Date: Sun, 6 Jun 2004 18:02:45 +0200
From: Vojtech Pavlik <vojtech@suse.cz>
To: Dmitry Torokhov <dtor_core@ameritech.net>
Cc: linux-kernel@vger.kernel.org, Andrew Morton <akpm@osdl.org>
Subject: Re: [PATCH 2.6] Mousedev - better button handling under load
Message-ID: <20040606160245.GA1350@ucw.cz>
References: <200406050249.02523.dtor_core@ameritech.net> <20040606095843.GC1646@ucw.cz> <200406060940.21209.dtor_core@ameritech.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <200406060940.21209.dtor_core@ameritech.net>
User-Agent: Mutt/1.4.1i
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1920
Lines: 43

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@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/