[PATCH] evdev: add phys property (EVIOCGPHYS) as stable identifier

Peter Korsgaard jacmet at sunsite.dk
Sat May 8 00:42:26 PDT 2010


>>>>> "Peter" == Peter Korsgaard <jacmet at sunsite.dk> writes:

Hi,

 >>> Really? That sounds like a bug in valgrind then. EVIOCGPHYS is correctly
 >>> marked as an input ioctl (_IOC(_IOC_READ, 'E', 0x07, len)), so it should
 >>> know that phys contains valid data after the ioctl.
 >>> 
 >>> But I can certainly add the initializer, even though it's unneeded
 >>> bloat.

 Peter> I'll double-check again, but I saw those warnings this
 Peter> morning. Don't know which distro and what version of valgrind
 Peter> though...

 Peter> I had a look, and it was a bug in valgrind. I sent them a patch:

 Peter> http://thread.gmane.org/gmane.comp.debugging.valgrind.devel/11049

FYI, this is now in valgrind svn (r11124).

-- 
Bye, Peter Korsgaard


More information about the xorg-devel mailing list