[PATCH 16/42] dix: if we run out of space for new touch events, resize the queue

Chase Douglas chase.douglas at canonical.com
Mon Dec 19 17:46:27 PST 2011


On Dec 14, 2011, at 9:01 PM, Peter Hutterer <peter.hutterer at who-t.net> wrote:
> @@ -133,7 +190,15 @@ TouchBeginDDXTouch(DeviceIntPtr dev, uint32_t ddx_id)
>         return ti;
>     }
> 
> -    /* If we get here, then we've run out of touches, drop the event */
> +    /* If we get here, then we've run out of touches and we need to drop the
> +     * event (we're inside the SIGIO handler here) schedule a WorkProc to
> +     * grow the queue for us for next time. */
> +    ErrorF("%s: not enough space for touch events. Dropping this event.\n", dev->name);

It would help diagnosing issues to know what the array size was when this occurred. Can you add a print out for this?

Otherwise,

Reviewed-by: Chase Douglas <chase.douglas at canonical.com>


More information about the xorg-devel mailing list