[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