Re: [PATCH V2] usbnet: fix race condition caused spinlock bad magic issue

2013-11-12 Thread Oliver Neukum
On Wed, 2013-11-13 at 09:11 +0800, wangbiao wrote: Hi, > 2, for the same reason,dev->wait should be judged again before use it, > as between the judge point(if(dev->wait)) and use point(wakeup(dev->wait)), > the dev->wait may be set NULL by another cpu. > > for issue 1, declare unlink_wakeup in

[PATCH V2] usbnet: fix race condition caused spinlock bad magic issue

2013-11-12 Thread wangbiao
From: wang, biao Date: Mon, 11 Nov 2013 10:23:40 +0800 Subject: [PATCH] usbnet: fix race condition caused spinlock bad magic issue 1, there is race between usbnet_terminate_urbs and usbnet_bh, when unlink_wakeup used in usbnet_bh, it may be already freed and used by other function as unlink_wakeu