public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Gunther.Mayer@t-online.de (Gunther Mayer)
To: linux-kernel@vger.kernel.org
Cc: Andries.Brouwer@cwi.nl
Subject: Re: [PATCH] make psaux reconnect adjustable
Date: Tue, 14 Aug 2001 19:10:26 +0200	[thread overview]
Message-ID: <3B795B82.195CA42B@t-online.de> (raw)
In-Reply-To: <200108141112.LAA99888@vlet.cwi.nl>

Andries.Brouwer@cwi.nl wrote:
> 
>     From garloff@garloff.de Tue Aug 14 11:57:23 2001
> 
>     I can confirm what you suggest:
>         My mouse (Logitech wheel USB/PS2) sends indeed AA 00.
>     So, I extended my patch:
>     psmouse_reconnect = 0: Do nothing (just pass all to userspace)
>     psmouse_reconnect = 1: Flush Q & ping mouse on AA 00 (default)
>     psmouse_reconnect = 2: Flush Q & ping mouse on AA (old behaviour)
> 
>     With reconnect 1 or 2: After reconnecting, mouse behaves strange
>         (jumping around the screen)

This is a serious bug in many user-space drivers. PS/2 mouse protocol
was designed to easily re-synchronize (think about transmission errors/
lost bytes).

>     With reconnect 0:      Mouse is dead

This is a bug in all user-space drivers (understandable as the
kernel tried to be too clever). They must send the proper ps2-enabling
sequence after they see "aa 00".

> 
>     In both cases restarting gpm gets the mouse back to work again.
>     It seems the imps2 driver does some initialization to the mouse.
> 
>     If I use the plain ps2 driver, then finally, I see the benefit of the
>     reconnect code in the kernel:
>     With reconnect = 1 or 2: It works after replugging
>     With reconnect = 0:      Mouse is dead after replugging
> 
>     In the latter case restarting gpm helps.
...

Add an "ioctl(PS2_TRANSPARENT)", to disable the current kernel policy.
So new drivers which understand about "aa 00" sequences can act properly.
Don't break existing apps.

  reply	other threads:[~2001-08-14 21:02 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-08-14 11:12 [PATCH] make psaux reconnect adjustable Andries.Brouwer
2001-08-14 17:10 ` Gunther Mayer [this message]
2001-08-14 21:56   ` Kurt Garloff
  -- strict thread matches above, loose matches on Subject: below --
2001-08-14 21:26 Andries.Brouwer
2001-08-14 18:29 Andries.Brouwer
2001-08-14 19:35 ` Gunther Mayer
     [not found] <no.id>
2001-08-02 15:03 ` Alan Cox
2001-08-02 17:34 ` Alan Cox
2001-08-02 17:27 Andries.Brouwer
2001-08-14  9:57 ` Kurt Garloff
2001-08-14 15:03   ` Kurt Garloff
2001-08-14 16:58     ` Linus Torvalds
2001-08-14 17:35       ` Kurt Garloff
2001-08-14 21:29       ` Kurt Garloff
2001-08-15 15:50         ` Gunther Mayer
2001-08-02 16:02 Andries.Brouwer
2001-08-02 16:18 ` Kurt Garloff
2001-08-02 11:55 Andries.Brouwer
2001-08-02  2:21 Kurt Garloff

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=3B795B82.195CA42B@t-online.de \
    --to=gunther.mayer@t-online.de \
    --cc=Andries.Brouwer@cwi.nl \
    --cc=linux-kernel@vger.kernel.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