From: "Dmitry Torokhov" <dmitry.torokhov@gmail.com>
To: "Vojtech Pavlik" <vojtech@suse.cz>
Cc: "Pete Zaitcev" <zaitcev@redhat.com>,
linux-kernel@vger.kernel.org, stuart_hayes@dell.com
Subject: Re: Suppressing softrepeat
Date: Wed, 22 Feb 2006 16:09:33 -0500 [thread overview]
Message-ID: <d120d5000602221309n58cad283q41a79e6fe013042d@mail.gmail.com> (raw)
In-Reply-To: <20060222204024.GA7477@suse.cz>
On 2/22/06, Vojtech Pavlik <vojtech@suse.cz> wrote:
> On Wed, Feb 22, 2006 at 12:00:47PM -0800, Pete Zaitcev wrote:
> > On Tue, 21 Feb 2006 22:08:00 +0100, Vojtech Pavlik <vojtech@suse.cz> wrote:
> >
> > > A much simpler workaround for the DRAC3 is to set the softrepeat delay
> > > to at least 750ms, using kbdrate(8), which will call the proper console
> > > ioctl, resulting in updating the softrepeat parameters.
> > >
> > > I prefer workarounds for problematic hardware done outside the kernel,
> > > if possible.
> >
> > I agree with the sentiment when posed in the abstract way, but let me
> > tell you why this case is different.
> >
> > Firstly, there's nothing "problematic" about this. It's just how it is.
> > The only problematic thing here is our code. Currently, the situation is
> > assymetric. It is possible to force softrepeat on, but not possible to
> > force softrepeat off. Isn't it broken?
> >
> > Secondly, 750ms may be not enough. Stuart is being shy here and posting
> > explanations to Bugzilla for some reason.
> >
> > Lastly, it's such a PITA to add these things into the userland, that
> > it's completely impractical. Console is needed the most when things go
> > wrong. In such case, that echo(1) may not be reached before the single
> > user shell. And stuffing it into the initrd is for Linux weenies only,
> > unless automated by mkinitrd.
> >
> > I think you're being unreasonable here. I am not asking for NFS root
> > or IP autoconfiguration and sort of complicated process which ought to
> > be done in userland indeed.
>
> I'm definitely not intending to be unreasonable, and I understand your
> need to have the keyboard working all the way from the grub/lilo prompt.
>
> I just don't like adding more module options to one that already has so
> many it's hard to understand what they're used for.
>
> How about simply this patch instead?
>
> Setting autorepeat will not be possible on 'dumb' keyboards anymore by
> default, but since these usually are special forms of hardware anyway,
> like the DRAC3, this shouldn't be an issue for most users. Using
> 'softrepeat' on these keyboards will restore the behavior for users that
> need it.
>
I am not keen on changing the default behaviour... How many dumb
keybaords are out there? I'd rather turn atkbd.softrepeat into a
3-state switch...
--
Dmitry
next prev parent reply other threads:[~2006-02-22 21:09 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-21 20:43 Pete Zaitcev
2006-02-21 21:08 ` Vojtech Pavlik
2006-02-21 21:32 ` Dmitry Torokhov
2006-02-21 21:40 ` Vojtech Pavlik
2006-02-21 21:57 ` Dmitry Torokhov
2006-02-22 20:00 ` Pete Zaitcev
2006-02-22 20:40 ` Vojtech Pavlik
2006-02-22 21:09 ` Dmitry Torokhov [this message]
2006-02-22 22:09 ` Vojtech Pavlik
2006-02-22 22:13 ` Dmitry Torokhov
2006-02-22 21:16 ` Pete Zaitcev
2006-02-21 21:15 ` Dmitry Torokhov
2006-02-21 21:35 ` Vojtech Pavlik
2006-02-22 20:46 Stuart_Hayes
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=d120d5000602221309n58cad283q41a79e6fe013042d@mail.gmail.com \
--to=dmitry.torokhov@gmail.com \
--cc=dtor_core@ameritech.net \
--cc=linux-kernel@vger.kernel.org \
--cc=stuart_hayes@dell.com \
--cc=vojtech@suse.cz \
--cc=zaitcev@redhat.com \
/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®