Re: [PATCH weston] evdev: Handle SYN_DROPPED, atomically count buttons

2013-03-28 Thread Jonas Ã…dahl
On Wed, Mar 27, 2013 at 3:26 AM, Martin Minarik wrote: > When the kernel event queue overruns, the evdev.c will: > 1. Skip events until and up to the next SYN_REPORT > 2. Count depressed grab buttons on seat devices. > 3. If the depressed grab button count is different, update >it atomically.

Re: [PATCH weston] evdev: Handle SYN_DROPPED, atomically count buttons

2013-03-27 Thread Giulio Camuffo
Hi Martin, Apart some style issues this seems good to me, but i'ts the first time i look into evdev.c, so i might have missed something. I could not reproduce the two issues you describe, with 20 mpv instances running a full hd movie in the background. 2013/3/27 Martin Minarik : > When the kernel

[PATCH weston] evdev: Handle SYN_DROPPED, atomically count buttons

2013-03-26 Thread Martin Minarik
When the kernel event queue overruns, the evdev.c will: 1. Skip events until and up to the next SYN_REPORT 2. Count depressed grab buttons on seat devices. 3. If the depressed grab button count is different, update it atomically. If it changed to zero, call grab cancelation. Potential pitfalls: