From: Markus Armbruster <armbru@redhat.com>
To: xen-devel@lists.xensource.com
Subject: RFC: drop frontend support for relative pointer
Date: Wed, 07 Oct 2009 19:16:51 +0200 [thread overview]
Message-ID: <87vdirw2d8.fsf@pike.pond.sub.org> (raw)
The protocol between PVFB frontend and backend supports relative and
absolute pointer.
In theory, support for absolute pointer in the backend is optional. In
practice, our backend has always supported it.
Absolute pointers provide a superior user experience, and our frontend
always enabled them.
However, because the backend could theoretically not offer the absolute
pointer option, the frontend still supports both. This has worked fine
so far, but it's starting to cause trouble now.
We support both relative and absolute pointer by setting both EV_REL and
EV_ABS in input device, then use whatever the backend sends us. The
backend either sends only relative or only absolute events.
Nothing in the kernel suggests setting both EV_REL and EV_ABS is bad.
In fact, drivers/hid/hid-wacom.c and drivers/input/tablet/aiptek.c seem
to do the same.
Unfortunately, X is having difficulties with it. It worked only because
of a bug in evdev. This bug was fixed recently, and the fix broke the
Xen PV pointer. Impact: pointer doesn't work at all with Fedora Rawhide
guests. See [*] for the gory details.
X will eventually be fixed, but waiting for that isn't practical. We
need to work around the problem in xen-kbdfront, or possibly evdev.
My preferred solution is dropping support for relative pointer in the
frontend, then set only EV_ABS. It's easy, safe, and sends user space
down well-trodden paths. Requires a backend supporting absolute
pointers, but as mentioned above, they all do.
Opinions?
[*] https://bugzilla.redhat.com/show_bug.cgi?id=523914
next reply other threads:[~2009-10-07 17:16 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-07 17:16 Markus Armbruster [this message]
2009-10-07 17:39 ` RFC: drop frontend support for relative pointer Stefano Stabellini
2009-10-07 18:16 ` Markus Armbruster
2009-10-07 20:01 ` Daniel P. Berrange
2009-10-08 11:25 ` Stefano Stabellini
2009-10-08 11:32 ` Daniel P. Berrange
2009-10-08 12:59 ` Markus Armbruster
2009-10-12 18:02 ` Jean Guyader
2009-10-12 18:31 ` Jean Guyader
2009-10-12 20:42 ` Markus Armbruster
2009-10-12 21:18 ` Jean Guyader
2009-10-13 16:05 ` Markus Armbruster
2009-10-13 16:21 ` Stefano Stabellini
2009-10-13 18:13 ` Markus Armbruster
2009-10-13 20:58 ` Jean Guyader
2009-10-14 11:56 ` Stefano Stabellini
2009-10-14 12:14 ` Markus Armbruster
2009-10-14 12:42 ` Stefano Stabellini
2009-10-09 17:33 ` Markus Armbruster
2009-10-12 17:27 ` Stefano Stabellini
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=87vdirw2d8.fsf@pike.pond.sub.org \
--to=armbru@redhat.com \
--cc=xen-devel@lists.xensource.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.