[PATCH (v5) 0/15] Multitouch take five

Cyril Brulebois kibi at debian.org
Sun Jan 23 17:34:26 PST 2011


Daniel Stone <daniel at fooishbar.org> (19/01/2011):
> Another round of multitouch patches ... this lot addresses all the
> review comments from the previous patchset, and seems to work quite
> reliably and consistently for me.  It's a fairly large change, but the
> code is now a fair bit easier to understand and verify.

So I played a bit with those patches and here's the extra comments I've
got:
 * server side: make && make check are OK for */8 except for 7/8 (then
   8/8), due to the wrong parameter count I mentioned earlier. I “fixed”
   it locally by only passing the first two, which seemed to do the job.
   With that change, the only failing “make check” was with 7/8:
   | /xi2/protocol/XISelectEvents: **
   | ERROR:protocol-xiselectevents.c:110:request_XISelectEvent: assertion failed: (rc == error)
   | /bin/sh: line 5: 21379 Aborted                 ${dir}$tst
   | FAIL: protocol-xiselectevents
 * driver side: your patches led to a segfaulting X, after a few events
   from my non-multitouch-enabled touchpad, in valuator_mask_copy().
   I mentioned it on IRC, and Chase pointed me to his v2 of the evdev
   patch, which gave me an apparently functional X again.

KiBi.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.x.org/archives/xorg-devel/attachments/20110124/998415e5/attachment.pgp>


More information about the xorg-devel mailing list