forked from luck/tmp_suning_uos_patched
2d93148ab6
This patch (as1229) fixes a few lifetime and locking problems in the usb-serial driver. The main symptom is that an invalid kevent is created when the serial device is unplugged while a connection is active. Ports should be unregistered when device is disconnected, not when the parent usb_serial structure is deallocated. Each open file should hold a reference to the corresponding port structure, and the reference should be released when the file is closed. serial->disc_mutex should be acquired in serial_open(), to resolve the classic race between open and disconnect. serial_close() doesn't need to hold both serial->disc_mutex and port->mutex at the same time. Release the subdriver's module reference only after releasing all the other references, in case one of the release routines needs to invoke some code in the subdriver module. Replace a call to flush_scheduled_work() (which is prone to deadlocks) with cancel_work_sync(). Also, add a call to cancel_work_sync() in the disconnect routine. Reduce the scope of serial->disc_mutex in serial_disconnect(). The only place it really needs to protect is where the "disconnected" flag is set. This fixes the bug reported in http://bugs.freedesktop.org/show_bug.cgi?id=20703 Signed-off-by: Alan Stern <stern@rowland.harvard.edu> Tested-by: Dan Williams <dcbw@redhat.com> Tested-by: Ming Lei <tom.leiming@gmail.com> Reviewed-by: Oliver Neukum <oliver@neukum.org> Acked-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> |
||
|---|---|---|
| .. | ||
| aircable.c | ||
| ark3116.c | ||
| belkin_sa.c | ||
| belkin_sa.h | ||
| bus.c | ||
| ch341.c | ||
| ChangeLog.history | ||
| console.c | ||
| cp210x.c | ||
| cyberjack.c | ||
| cypress_m8.c | ||
| cypress_m8.h | ||
| digi_acceleport.c | ||
| empeg.c | ||
| ezusb_convert.pl | ||
| ezusb.c | ||
| ftdi_sio.c | ||
| ftdi_sio.h | ||
| funsoft.c | ||
| garmin_gps.c | ||
| generic.c | ||
| hp4x.c | ||
| io_16654.h | ||
| io_edgeport.c | ||
| io_edgeport.h | ||
| io_ionsp.h | ||
| io_tables.h | ||
| io_ti.c | ||
| io_ti.h | ||
| io_usbvend.h | ||
| ipaq.c | ||
| ipaq.h | ||
| ipw.c | ||
| ir-usb.c | ||
| iuu_phoenix.c | ||
| iuu_phoenix.h | ||
| Kconfig | ||
| keyspan_pda.c | ||
| keyspan_usa26msg.h | ||
| keyspan_usa28msg.h | ||
| keyspan_usa49msg.h | ||
| keyspan_usa67msg.h | ||
| keyspan_usa90msg.h | ||
| keyspan.c | ||
| keyspan.h | ||
| kl5kusb105.c | ||
| kl5kusb105.h | ||
| kobil_sct.c | ||
| kobil_sct.h | ||
| Makefile | ||
| Makefile-keyspan_pda_fw | ||
| mct_u232.c | ||
| mct_u232.h | ||
| mos7720.c | ||
| mos7840.c | ||
| moto_modem.c | ||
| navman.c | ||
| omninet.c | ||
| opticon.c | ||
| option.c | ||
| oti6858.c | ||
| oti6858.h | ||
| pl2303.c | ||
| pl2303.h | ||
| qcserial.c | ||
| safe_serial.c | ||
| siemens_mpi.c | ||
| sierra.c | ||
| spcp8x5.c | ||
| symbolserial.c | ||
| ti_usb_3410_5052.c | ||
| ti_usb_3410_5052.h | ||
| usb_debug.c | ||
| usb-serial.c | ||
| visor.c | ||
| visor.h | ||
| whiteheat.c | ||
| whiteheat.h | ||