I've applied this too as part of my testing, I don't see a dri3 file - does
that mean the dri2 one covers both dri2&3?

If it doesn't please ignore

Tested-by: Mike Lothian <[email protected]>

On Mon, 16 Apr 2018 at 21:17 Frank Binns <[email protected]> wrote:

> Adam Jackson <[email protected]> writes:
>
> > ms_queue_vblank() returns false on failure.
> >
> > Reported-by: Chris Wilson <[email protected]>
> > Signed-off-by: Adam Jackson <[email protected]>
>
> Reviewed-by: Frank Binns <[email protected]>
>
> > ---
> >  hw/xfree86/drivers/modesetting/dri2.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/hw/xfree86/drivers/modesetting/dri2.c
> b/hw/xfree86/drivers/modesetting/dri2.c
> > index fd36aa118..96eaaaaf7 100644
> > --- a/hw/xfree86/drivers/modesetting/dri2.c
> > +++ b/hw/xfree86/drivers/modesetting/dri2.c
> > @@ -749,7 +749,7 @@ ms_dri2_schedule_wait_msc(ClientPtr client,
> DrawablePtr draw, CARD64 target_msc,
> >              target_msc = current_msc;
> >
> >          ret = ms_queue_vblank(crtc, MS_QUEUE_ABSOLUTE, target_msc,
> &queued_msc, seq);
> > -        if (ret) {
> > +        if (!ret) {
> >              static int limit = 5;
> >              if (limit) {
> >                  xf86DrvMsg(scrn->scrnIndex, X_WARNING,
>
> _______________________________________________
> [email protected]: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: https://lists.x.org/mailman/listinfo/xorg-devel
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: https://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to