Messages in this thread | | | Date | Fri, 28 May 2004 16:06:12 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH][RFC] 2.6.6 tty_io.c hangup locking |
| |
Paul Fulghum <paulkf@microgate.com> wrote: > > The following patch removes unnecessary disabling of > interrupts when processing hangup for tty devices.
Ho hum, this has been hanging around forever. Obviously the current locking is pointless and the only useful locking we have in there is lock_kernel().
The reason why a patch such as yours wasn't applied is that it was all kept as a reminder that we suck. Someone needs to get down and audit what's actually happening in there. It seems that you've now done that via comparison with other callers, but that is not necessarily a good approach when it comes to the tty layer ;)
We need to itemise all the affected memory storage in all impementations of ->flush_buffer() and ->write_wakeup() and then make sure that all _other_ users of those fields (whether or not they lie in the ->flush_buffer() and ->write_wakeup() codepaths) are using the same locking. Is that something you could do? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |