Re: [RESEND 1/3] Bluetooth: Use cancel_work instead of cancel_work_sync

Hi,

On Tue, Jan 31, 2012 at 1:11 PM, Marcel Holtmann [off-list ref] wrot=
e:

Hi Andrei,

quoted

quoted

quoted

- =A0 =A0 =A0 cancel_delayed_work_sync(&conn->disc_work);
+ =A0 =A0 =A0 cancel_delayed_work(&conn->disc_work);

I'm afraid we must use _sync variant here. disc_work is not supposed t=

o

quoted

quoted

be running after hci_conn is deleted.

BTW, I believe we already addressed this issue in patches [PATCH 1/2]
Bluetooth: Fix potential deadlock and [PATCH 2/2] Bluetooth: Remove
unneeded locking. These patches are now pushed upstream. Could you

I will check those patches from upstream and let you know.

crap, I just acked these.

Johan, forget about my acks and just ignore them. Lets wait until we get
a clean new series.

Regards

Marcel

This change is really wrong because we're on the delete path and Andre
sent other patches which I'm almost sure will address this problem.

Best regards,

--=20
Ulisses Furquim
ProFUSION embedded systems
http://profusion.mobi
Mobile: +55 19 9250 0942
Skype: ulissesffs