Re: [LART] pc_keyb.c changes

Alexander Viro
2001-11-30T09:04:34Z

On Fri, 30 Nov 2001, Alexander Viro wrote:

> Could the person who switched from BKL to spin_lock_irqsave() in
> pc_keyb.c please share whatever the hell he had been smoking? Free clue:
> disabling interrupts for long intervals to improve scalability is right up
> there with fighting for peace and fucking for virginity.
>
> Linus, could we please revert that crap and feed the authors to
> Larry? If they are religious about Scalability At Any Cost, Common Sense
> Be Damned(tm) - let's give them a chance to become martyrs...

BTW, while we are at it, one is _not_ supposed to do inclusion to/removal from
single-linked list without any exclusion whatsoever. Example:
drivers/input/evdev.c::evdev_{open,release}(). Or drivers/input/joydev.c
and drivers/input/mousedev.c, for that matter. Sigh...