usb-gadget-ether: prevent oops caused by error interrupt race
authorBenedikt Spranger <bene@linutronix.de>
Tue, 2 Oct 2007 21:40:48 +0000 (14:40 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 12 Oct 2007 21:55:30 +0000 (14:55 -0700)
commit5395353e0c8272fe73ac914acd7e4add0da2bef0
tree258ce18442ed45116f842cd13d0969cea7e5d353
parent9dcfbd97a695a3c28a867501127fa35ac49bc805
usb-gadget-ether: prevent oops caused by error interrupt race

Fix a longstanding race in the Ethernet gadget driver, which can cause an
oops on device disconnect.  The fix is just to make the TX path check
whether its freelist is empty.  That check is otherwise not necessary,
since the queue is always stopped when that list empties (and restarted
when request completion puts an entry back on that freelist).

The race window starts when the network code decides to transmit a packet,
and ends when hard_start_xmit() grabs the freelist lock.  When disconnect()
is called inside that window, it shuts down the TX queue and breaks the
otherwise-solid assumption that packets are never sent through a TX queue
that's stopped.

Signed-off-by: Benedikt Spranger <bene@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Cc: stable <stable@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/gadget/ether.c