Re: [PATCH 00/11] Yet another FBC series, v3 part 2 v2

["Zanoni, Paulo R" <paulo.r.zanoni@intel.com>]

Em Qua, 2015-12-02 às 12:37 +0000, Chris Wilson escreveu:
> On Wed, Dec 02, 2015 at 10:15:16AM -0200, Paulo Zanoni wrote:
> > Hi
> > 
> > This series includes the implementation for the review feedback
> > given by Chris.
> > I also removed the patch that transformed our 50ms timeout into a
> > vblank-based
> > timeout due to performance concerns. The only patch that still
> > needs review is
> > patch 6.
> > 
> > Thanks,
> > Paulo
> > 
> > Paulo Zanoni (11):
> >   drm/i915: fix the CFB size check
> >   drm/i915: set dev_priv->fbc.crtc before scheduling the enable
> > work
> >   drm/i915: pass the crtc as an argument to intel_fbc_update()
> >   drm/i915: introduce is_active/activate/deactivate to the FBC
> >     terminology
> >   drm/i915: introduce intel_fbc_{enable,disable}
> >   drm/i915: alloc/free the FBC CFB during enable/disable
> >   drm/i915: check for FBC planes in the same place as the pipes
> >   drm/i915: use a single intel_fbc_work struct
> >   drm/i915: kill fbc.uncompressed_size
> >   drm/i915: get rid of FBC {,de}activation messages
> >   drm/i915: only recompress FBC after flushing a drawing operation
> 
> My r-b still stand.

What about patch 6? Any concerns or additional requests?

Thanks,
Paulo

> -Chris
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx