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

[Daniel Vetter <daniel@ffwll.ch>]

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.
-Daniel

> 
> > -----Original Message-----
> > From: Daniel Vetter [mailto:daniel.vetter@ffwll.ch] On Behalf Of Daniel
> > Vetter
> > Sent: Friday, December 06, 2013 12:07 PM
> > To: Bloomfield, Jon
> > Cc: Chris Wilson; intel-gfx@lists.freedesktop.org
> > Subject: Re: [Intel-gfx] [PATCH] drm/i915: Introduce vblank work function
> > 
> > On Fri, Dec 06, 2013 at 10:44:15AM +0000, Bloomfield, Jon wrote:
> > > What's the status of this patch ? I can't find any subsequent mention
> > > of it, but we currently use it in one of our Android development
> > > trees. I'm trying to work out whether to retain it or replace it.
> > >
> > > Was it killed off, or is it still in the pipeline ?
> > 
> > Stalled atm I think. The overall concept of a vblank worker/timer support
> > code is still useful imo. I think I've written up all the bikesheds Chris&I
> > discussed on irc in my other reply.
> > -Daniel
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > +41 (0) 79 365 57 48 - http://blog.ffwll.ch

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch