Just one comment so far now that I'm using this code.

On Fri, 11 Sep 2009 14:33:34 -0400
"Kristian Høgsberg" <k...@bitplanet.net> wrote:
> +     file_priv->event_space -= sizeof e->event;
> +     seq = drm_vblank_count(dev, pipe);
> +     if ((vblwait->request.type & _DRM_VBLANK_NEXTONMISS) &&
> +         (seq - vblwait->request.sequence) <= (1 << 23)) {
> +             vblwait->request.sequence = seq + 1;
> +     }
> +
> +     DRM_DEBUG("event on vblank count %d, current %d, crtc %d\n",
> +               vblwait->request.sequence, seq, pipe);

When we queue the event we should probably return the queued sequence
number to the client since in some cases they may be doing a relative
wait and will need to know the ultimate sequence number to look for in
their wake handler.

-- 
Jesse Barnes, Intel Open Source Technology Center

------------------------------------------------------------------------------
Come build with us! The BlackBerry&reg; Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9&#45;12, 2009. Register now&#33;
http://p.sf.net/sfu/devconf
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to