]> git.baikalelectronics.ru Git - kernel.git/commit
n_hdlc: fix read and write locking
authorPaul Fulghum <paulkf@microgate.com>
Tue, 18 Jan 2011 22:03:25 +0000 (14:03 -0800)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sun, 23 Jan 2011 03:15:01 +0000 (19:15 -0800)
commit5148662288e20c992f516f7e65cb548d42514045
treea1078793d172e79fdede20f30e371ba05d7c2d2b
parent46d1e1b70bed88085a975d7d670ac9f06945c6d8
n_hdlc: fix read and write locking

Fix locking in read and write code of n_hdlc line discipline.

2.6.36 replaced lock_kernel() with tty_lock().  The tty mutex is not
dropped automatically when the thread sleeps like the BKL.  This results
in a blocked read or write holding the tty mutex and stalling operations
by other devices that use the tty mutex.

A review of n_hdlc read and write code shows:
1. neither BKL or tty mutex are required for correct operation
2. read can block while read data is available if data is posted
   between availability check and call to interruptible_sleep_on()
3. write does not set process state to TASK_INTERRUPTIBLE
   on each pass through the processing loop which can cause
   unneeded scheduling of the thread

The unnecessary tty mutex references have been removed.

Read changed to use same code as n_tty read
for completing reads and blocking.

Write corrected to set process state to TASK_INTERRUPTIBLE on each pass
through processing loop.

Signed-off-by: Paul Fulghum <paulkf@microgate.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/tty/n_hdlc.c