[RFC v2 07/12] present: Check the whole exec queue on event

Michel Dänzer michel at daenzer.net
Fri Sep 29 16:22:24 UTC 2017


On 27/09/17 07:19 AM, Louis-Francis Ratté-Boulianne wrote:
> Later events are sometimes added in front of the queue (e.g.
> if page flipping fails) so we need to check the whole queue
> on event.
> 
> Signed-off-by: Louis-Francis Ratté-Boulianne <lfrb at collabora.com>
> ---
>  present/present.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/present/present.c b/present/present.c
> index c47b28b20..262b6ce9c 100644
> --- a/present/present.c
> +++ b/present/present.c
> @@ -527,8 +527,6 @@ present_event_notify(uint64_t event_id, uint64_t ust, uint64_t msc)
>              present_execute(vblank, ust, msc);
>              return;
>          }
> -        if (match < 0)
> -            break;
>      }
>      xorg_list_for_each_entry(vblank, &present_flip_queue, event_queue) {
>          if (vblank->event_id == event_id) {
> @@ -996,8 +994,6 @@ present_abort_vblank(ScreenPtr screen, RRCrtcPtr crtc, uint64_t event_id, uint64
>              vblank->queued = FALSE;
>              return;
>          }
> -        if (match < 0)
> -            break;
>      }
>      xorg_list_for_each_entry(vblank, &present_flip_queue, event_queue) {
>          if (vblank->event_id == event_id) {
> 

This is a bug fix which is independent of the rest of the series, so it
should be submitted separately and applied for an upcoming 1.19.4 release.

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the xorg-devel mailing list