> > > + if (temp & VSYNC_PIPEA_FLAG)
> > > +         atomic_add(i915_get_vblank_counter(dev, 0),
> > > +                    &dev->vblank_count[0]);
> > > + if (temp & VSYNC_PIPEB_FLAG)
> > > +         atomic_add(i915_get_vblank_counter(dev, 1),
> > > +                    &dev->vblank_count[1]);
> >
> > I think atomic_add is wrong here.
>
> Hm yeah it should be just atomic_set(), duh.

On second thought, maybe I'll just go back to atomic_inc, otherwise I'll 
have to figure out the difference between the last count and the new 
count, then atomically add that to the current count, to deal with 
missed interrupts.

Then again, maybe I could unify it with the counter update code that I 
pull out of drm_vblank_get.  I'll see about that.

Jesse

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to