Re: [PATCH] drm/i915: Introduce vblank work function

[Jesse Barnes <jbarnes@virtuousgeek.org>]

On Fri, 6 Dec 2013 14:42:29 +0100
Daniel Vetter <daniel@ffwll.ch> wrote:

> On Fri, Dec 06, 2013 at 12:12:21PM +0000, Bloomfield, Jon wrote:
> > Ok thanks. 
> > 
> > To add weight to it becoming official in some form, we're using it for various deferred operations:
> > 	drm/i915: Make plane switching asynchronous
> > 	drm/i915: Asynchronously unpin the old framebuffer after the next vblank	
> > 
> > They aren't my patches but I believe they should be upstreamed in the near future. The claim is that these give a noticeable performance boost.
> > 
> > I'll leave it in and hope it becomes official.
> 
> For this stuff the upstream plane is to merge Ville's nuclear pageflip
> code, which is the full deal solution for all these issues. I haven't read
> his latest wip code to see what exactly he's using for all the vblank
> work.

I don't think that should block getting the vblank worker code in along
with the trivial change to the sprite code to avoid a wait_vblank in
the buffer switching case.  We've needed that since this code went in,
but for some reason Chris's code has been stalled the whole time (and
didn't we have a wq patch for the sprite unpin even before that?).

-- 
Jesse Barnes, Intel Open Source Technology Center