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

2015-12-02 Thread ch...@chris-wilson.co.uk
On Wed, Dec 02, 2015 at 05:19:33PM +, Zanoni, Paulo R wrote: > Em Qua, 2015-12-02 às 12:37 +, Chris Wilson escreveu: > > My r-b still stand. > > What about patch 6? Any concerns or additional requests? >   drm/i915: alloc/free the FBC CFB during enable/disable I thought I had spotted one

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

2015-12-02 Thread Zanoni, Paulo R
Em Qua, 2015-12-02 às 12:37 +, 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 > > vbl

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

2015-12-02 Thread Chris Wilson
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 stil

[Intel-gfx] [PATCH 00/11] Yet another FBC series, v3 part 2 v2

2015-12-02 Thread Paulo Zanoni
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: f