Re: [Piglit] [PATCH] glx: Rerun failing tests in auto mode if there is a pending expose event

2017-09-26 Thread Thomas Hellstrom
On 09/26/2017 08:47 AM, Brian Paul wrote: On 09/26/2017 10:32 AM, Thomas Hellstrom wrote: Tests that check frontbuffer contents after drawing may fail because the X server touch the front contents between drawing and checking. In those cases, there's typically a pending expose event. So check

Re: [Piglit] [PATCH] glx: Rerun failing tests in auto mode if there is a pending expose event

2017-09-26 Thread Brian Paul
On 09/26/2017 10:32 AM, Thomas Hellstrom wrote: Tests that check frontbuffer contents after drawing may fail because the X server touch the front contents between drawing and checking. In those cases, there's typically a pending expose event. So check for that situation and optionally rerun the

Re: [Piglit] [PATCH] glx: Rerun failing tests in auto mode if there is a pending expose event

2017-09-26 Thread Brian Paul
On 09/26/2017 10:55 AM, Thomas Hellstrom wrote: On 09/26/2017 08:47 AM, Brian Paul wrote: On 09/26/2017 10:32 AM, Thomas Hellstrom wrote: Tests that check frontbuffer contents after drawing may fail because the X server touch the front contents between drawing and checking. In those cases,

Re: [Piglit] [PATCH] glx: Rerun failing tests in auto mode if there is a pending expose event

2017-09-26 Thread Eric Anholt
Thomas Hellstrom writes: > Tests that check frontbuffer contents after drawing may fail because > the X server touch the front contents between drawing and checking. In those > cases, there's typically a pending expose event. So check for that > situation and optionally

[Piglit] [PATCH] glx: Rerun failing tests in auto mode if there is a pending expose event

2017-09-26 Thread Thomas Hellstrom
Tests that check frontbuffer contents after drawing may fail because the X server touch the front contents between drawing and checking. In those cases, there's typically a pending expose event. So check for that situation and optionally rerun the test. Signed-off-by: Thomas Hellstrom