Stable apply request [was: Bluetooth: bnep: fix possible might sleep error in bnep_session]

4 messages, 4 authors, 2017-08-27 · open the first message on its own page

Stable apply request [was: Bluetooth: bnep: fix possible might sleep error in bnep_session]

From: Jiri Slaby <hidden>
Date: 2017-08-23 07:29:50

On 06/27/2017, 07:32 PM, Marcel Holtmann wrote:
quoted
It looks like bnep_session has same pattern as the issue reported in
old rfcomm:

	while (1) {
		set_current_state(TASK_INTERRUPTIBLE);
		if (condition)
			break;
		// may call might_sleep here
		schedule();
	}
	__set_current_state(TASK_RUNNING);

Which fixed at:
	dfb2fae Bluetooth: Fix nested sleeps

So let's fix it at the same way, also follow the suggestion of:
https://lwn.net/Articles/628628/
...
all 3 patches have been applied to bluetooth-next tree.
Hi,

given users are hitting it in at least 4.4 and 4.12, can we have all
three in all stables where this applies?

5da8e47d849d Bluetooth: hidp: fix possible might sleep error in
hidp_session_thread
f06d977309d0 Bluetooth: cmtp: fix possible might sleep error in cmtp_session
25717382c1dd Bluetooth: bnep: fix possible might sleep error in bnep_session

I am not sure: to stable directly or via net stable?

thanks,
-- 
js
suse labs

Re: Stable apply request

From: David Miller <davem@davemloft.net>
Date: 2017-08-23 18:07:29

From: Jiri Slaby <redacted>
Date: Wed, 23 Aug 2017 09:29:44 +0200
On 06/27/2017, 07:32 PM, Marcel Holtmann wrote:
quoted
quoted
It looks like bnep_session has same pattern as the issue reported in
old rfcomm:

	while (1) {
		set_current_state(TASK_INTERRUPTIBLE);
		if (condition)
			break;
		// may call might_sleep here
		schedule();
	}
	__set_current_state(TASK_RUNNING);

Which fixed at:
	dfb2fae Bluetooth: Fix nested sleeps

So let's fix it at the same way, also follow the suggestion of:
https://lwn.net/Articles/628628/
...
quoted
all 3 patches have been applied to bluetooth-next tree.
Hi,

given users are hitting it in at least 4.4 and 4.12, can we have all
three in all stables where this applies?

5da8e47d849d Bluetooth: hidp: fix possible might sleep error in
hidp_session_thread
f06d977309d0 Bluetooth: cmtp: fix possible might sleep error in cmtp_session
25717382c1dd Bluetooth: bnep: fix possible might sleep error in bnep_session

I am not sure: to stable directly or via net stable?
I generally let the wireless family handle their own -stable submissions
and that includes bluetooth.

Re: Stable apply request [was: Bluetooth: bnep: fix possible might sleep error in bnep_session]

From: Marcel Holtmann <marcel@holtmann.org>
Date: 2017-08-23 18:14:20

Hi Jiri,
quoted
quoted
It looks like bnep_session has same pattern as the issue reported in
old rfcomm:

	while (1) {
		set_current_state(TASK_INTERRUPTIBLE);
		if (condition)
			break;
		// may call might_sleep here
		schedule();
	}
	__set_current_state(TASK_RUNNING);

Which fixed at:
	dfb2fae Bluetooth: Fix nested sleeps

So let's fix it at the same way, also follow the suggestion of:
https://lwn.net/Articles/628628/
...
quoted
all 3 patches have been applied to bluetooth-next tree.
Hi,

given users are hitting it in at least 4.4 and 4.12, can we have all
three in all stables where this applies?

5da8e47d849d Bluetooth: hidp: fix possible might sleep error in
hidp_session_thread
f06d977309d0 Bluetooth: cmtp: fix possible might sleep error in cmtp_session
25717382c1dd Bluetooth: bnep: fix possible might sleep error in bnep_session

I am not sure: to stable directly or via net stable?
as Dave said, just email -stable directly and have Greg pick them up.

Regards

Marcel

Re: Stable apply request [was: Bluetooth: bnep: fix possible might sleep error in bnep_session]

From: Greg KH <hidden>
Date: 2017-08-27 13:10:10

On Wed, Aug 23, 2017 at 08:14:15PM +0200, Marcel Holtmann wrote:
Hi Jiri,
quoted
quoted
quoted
It looks like bnep_session has same pattern as the issue reported in
old rfcomm:

	while (1) {
		set_current_state(TASK_INTERRUPTIBLE);
		if (condition)
			break;
		// may call might_sleep here
		schedule();
	}
	__set_current_state(TASK_RUNNING);

Which fixed at:
	dfb2fae Bluetooth: Fix nested sleeps

So let's fix it at the same way, also follow the suggestion of:
https://lwn.net/Articles/628628/
...
quoted
all 3 patches have been applied to bluetooth-next tree.
Hi,

given users are hitting it in at least 4.4 and 4.12, can we have all
three in all stables where this applies?

5da8e47d849d Bluetooth: hidp: fix possible might sleep error in
hidp_session_thread
f06d977309d0 Bluetooth: cmtp: fix possible might sleep error in cmtp_session
25717382c1dd Bluetooth: bnep: fix possible might sleep error in bnep_session

I am not sure: to stable directly or via net stable?
as Dave said, just email -stable directly and have Greg pick them up.
All now picked up :)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help