Re: [PATCH 1/3] Input: sysrq - drop tty argument from sysrq ops handlers

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Benjamin Herrenschmidt
Date: Wednesday, August 4, 2010 - 1:33 am

On Wed, 2010-08-04 at 00:59 -0700, Dmitry Torokhov wrote:

While talking about sysrq...

I noticed that serial drivers call sysrq with the uart lock held.

I've seen deadlocks caused by that, for example, when entering the
debugger, it waits forever for a CPU which is itself waiting on the UART
lock.

I have a patch to drop the lock in serial_core.h, I'll post that
tomorrow hopefully, just checking if there's any objection there ? The
serial drivers might need to be audited a bit to make sure they cope
with the lock being dropped and re-acquired around the sysrq call.

Cheers,
Ben.
 

--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/3] SysRq: do not pass tty argument around, Dmitry Torokhov, (Wed Aug 4, 12:58 am)
Re: [PATCH 1/3] Input: sysrq - drop tty argument from sysr ..., Benjamin Herrenschmidt, (Wed Aug 4, 1:33 am)
Re: [PATCH 1/3] Input: sysrq - drop tty argument from sysr ..., Benjamin Herrenschmidt, (Wed Aug 4, 6:59 pm)
Re: [PATCH 1/3] Input: sysrq - drop tty argument from sysr ..., Benjamin Herrenschmidt, (Wed Aug 4, 7:03 pm)