mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@suse.cz>
To: Roman Zippel <zippel@linux-m68k.org>
Cc: Andries Brouwer <aebr@win.tue.nl>,
	linux-kernel@vger.kernel.org, vojtech@suse.cz
Subject: Re: Possible bug in keyboard.c (2.6.10)
Date: Mon, 31 Jan 2005 00:16:18 +0100	[thread overview]
Message-ID: <20050130231617.GA2781@elf.ucw.cz> (raw)
In-Reply-To: <Pine.LNX.4.61.0501272248380.6118@scrub.home>

Hi!

> > In short - raw mode in 2.6 is badly broken.
> 
> I think not just that. The whole keyboard input layer needs some serious 
> review. Just the complete lack of any locking is frightening, I'd really 
> like to know how the input layer could become the standard. I tried to 
> find a few times to find any discussion about the input layer design, but 
> I couldn't find anything.

Actually, at one point we went over inputs with vojtech and tried to
fix the locking. We tried adding spinlocks, but it got really ugly
really quickly. Then we figured out that set_bit() and friends should
be enough to do spinlock-equivalent without getting that ugly.

We did not have enough energy to solve plug/unplug races at that
point; but other races should have been fixed. It is possible that we
overlooked something and doing set_bit() instead of spinlocks is
broken. If it is so, please tell us. 

								Pavel
-- 
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

      parent reply	other threads:[~2005-01-30 23:16 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-01-27  3:16 Sasa Stevanovic
2005-01-27  4:50 ` Dmitry Torokhov
2005-01-27 10:16   ` Sasa Stevanovic
2005-01-27 12:56 ` Andries Brouwer
2005-01-28  0:39   ` Roman Zippel
2005-01-28 10:59     ` Vojtech Pavlik
2005-01-29  4:50       ` Al Viro
2005-01-29 11:25         ` Vojtech Pavlik
2005-01-29 23:35           ` Dmitry Torokhov
2005-01-31  9:01             ` Vojtech Pavlik
2005-01-30  8:41           ` Al Viro
2005-01-30 23:21             ` Dmitry Torokhov
2005-01-30 23:29               ` Dmitry Torokhov
2005-02-03  6:54               ` Dmitry Torokhov
2005-01-29 12:11       ` Roman Zippel
2005-01-29 14:04         ` Vojtech Pavlik
2005-01-30 20:13           ` Roman Zippel
2005-01-28 11:10     ` Vojtech Pavlik
2005-01-28 21:59       ` Andries Brouwer
2005-01-29 11:12         ` Vojtech Pavlik
2005-01-29 23:30           ` Dmitry Torokhov
2005-01-30 23:16     ` Pavel Machek [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20050130231617.GA2781@elf.ucw.cz \
    --to=pavel@suse.cz \
    --cc=aebr@win.tue.nl \
    --cc=linux-kernel@vger.kernel.org \
    --cc=vojtech@suse.cz \
    --cc=zippel@linux-m68k.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®