On Mon, 25 Oct 2010 17:13:57 +0300 Pauli Nieminen <ext-pauli.niemi...@nokia.com> wrote:
> DDX driver may implement schedule swap without GetMSC. In that case we > can't call GetMSC in DRI2SwapBuffers. > > Signed-off-by: Pauli Nieminen <ext-pauli.niemi...@nokia.com> > Reviewed-by: Mario Kleiner <mario.klei...@tuebingen.mpg.de> > CC: Keith Packard <kei...@keithp.com> > CC: Kristian Høgsberg <k...@bitplanet.net> > --- > hw/xfree86/dri2/dri2.c | 11 +++++++---- > 1 files changed, 7 insertions(+), 4 deletions(-) > > diff --git a/hw/xfree86/dri2/dri2.c b/hw/xfree86/dri2/dri2.c > index 228c0e0..d9b9d57 100644 > --- a/hw/xfree86/dri2/dri2.c > +++ b/hw/xfree86/dri2/dri2.c > @@ -856,11 +856,14 @@ DRI2SwapBuffers(ClientPtr client, DrawablePtr pDraw, > CARD64 target_msc, > * is moved to a crtc with a lower refresh rate, or a crtc that just > * got enabled. > */ > - if (!(*ds->GetMSC)(pDraw, &ust, ¤t_msc)) > - pPriv->last_swap_target = 0; > + if (ds->GetMSC) { > + if (!(*ds->GetMSC)(pDraw, &ust, ¤t_msc)) > + pPriv->last_swap_target = 0; > > - if (current_msc < pPriv->last_swap_target) > - pPriv->last_swap_target = current_msc; > + if (current_msc < pPriv->last_swap_target) > + pPriv->last_swap_target = current_msc; > + > + } > > /* > * Swap target for this swap is last swap target + swap interval since Yeah, that's fine. Reviewed-by: Jesse Barnes <jbar...@virtuousgeek.org> -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel