[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