> It should arrive soon, it awaits moderation approval. Sounds good. So could you help get my patch pushed together with your upcoming patch? I don't think I have permission for the push now. Thanks!
Regards, Jammy -----Original Message----- From: Axel Davy [mailto:[email protected]] Sent: Tuesday, November 03, 2015 4:19 PM To: Zhou, Jammy; [email protected] Subject: Re: [PATCH] present: Execute right away if target_msc equals current_msc Reviewed-by: Axel Davy <[email protected]> BTW, I've just sent a patch to fix the PresentOptionAsync behaviour (we want to use sync flips when target_msc is above crtc_msc). It should arrive soon, it awaits moderation approval. On 28/10/2015 11:39, Jammy Zhou wrote: > It is according to the protocol: > > "If 'options' contains PresentOptionAsync, and the 'target-msc' > is less than or equal to the current msc for 'window', then the > operation will be performed as soon as possible, not necessarily > waiting for the next vertical blank interval." > > Signed-off-by: Jammy Zhou <[email protected]> > --- > present/present.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/present/present.c b/present/present.c index > beb4ff0..5900c22 100644 > --- a/present/present.c > +++ b/present/present.c > @@ -871,7 +871,7 @@ present_pixmap(WindowPtr window, > > xorg_list_add(&vblank->event_queue, &present_exec_queue); > vblank->queued = TRUE; > - if ((pixmap && target_msc >= crtc_msc) || (!pixmap && target_msc > > crtc_msc)) { > + if (target_msc > crtc_msc) { > ret = present_queue_vblank(screen, target_crtc, vblank->event_id, > target_msc); > if (ret == Success) > return Success; _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
