Re: [PATCH 2.6] Mousedev - better button handling under load

From: Vojtech Pavlik
Date: Sun Jun 06 2004 - 11:04:17 EST


On Sun, Jun 06, 2004 at 09:40:20AM -0500, Dmitry Torokhov wrote:
> On Sunday 06 June 2004 04:58 am, Vojtech Pavlik wrote:
> > On Sat, Jun 05, 2004 at 02:49:00AM -0500, Dmitry Torokhov wrote:
> > > Hi,
> > >
> > > Currently mousedev combines all hardware motion data that arrivers since
> > > last time userspace read data into one cooked PS/2 packet. The problem is
> > > that under heavy or even moderate load, when userspace can't read data
> > > quickly enough, we start loosing valuable data which manifests in:
> > >
> > > - ignoring buton presses as by the time userspace gets to read the data
> > > button has already been released;
> > > - click starts in wrong place - by the time userspace got aroungd and read
> > > the packet mouse moved half way across the screen.
> > >
> > > The patch below corrects the issue - it will start accumulating new packet
> > > every time userspace is behind and button set changes. Size of the buffer
> > > is 16 packets, i.e. up to 8 pairs of press/release events which should be
> > > more than enough.
> > >
> > > The patch is against Vojtech's tree and shuld apply to -mm. I also have
> > > cumulative mousedev patch done against 2.6.7-pre2 at:
> > >
> > > http://www.geocities.com/dt_or/input/misc/mousedev-2.6.7-rc2-cumulative.patch.gz
> >
> > Thanks for this. Can I just pull from your tree, or is there more that I
> > shouldn't take?
> >
>
> I am exporting stuff to my bk tree on as-needed basis so there is nothing
> extra (and no mousedev changes yet). Do you want button handling changes
> only or you do also want tapping emulation exported?

Tapping is fine as well.

--
Vojtech Pavlik
SuSE Labs, SuSE CR
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/