Re: [Intel-gfx] [PATCH 0/4] Enable FBC on SKL, v3

2016-04-06 Thread Thulasimani, Sivakumar
On 4/6/2016 7:24 PM, Zanoni, Paulo R wrote: Em Qua, 2016-04-06 às 10:36 +0530, Thulasimani, Sivakumar escreveu: dont want to hijack thread but wanted to point out a possible regression in the previous patches of this series. intel_fbc_can_choose: returns true for gen 4/5/6/7. (possible bug)

Re: [Intel-gfx] [PATCH 0/4] Enable FBC on SKL, v3

2016-04-06 Thread Zanoni, Paulo R
Em Qua, 2016-04-06 às 10:36 +0530, Thulasimani, Sivakumar escreveu: > dont want to hijack thread but wanted to point out a possible > regression  > in the > previous patches of this series. > > intel_fbc_can_choose: returns true for gen 4/5/6/7. (possible bug) How? It will check for i915.enable_f

Re: [Intel-gfx] [PATCH 0/4] Enable FBC on SKL, v3

2016-04-05 Thread Thulasimani, Sivakumar
dont want to hijack thread but wanted to point out a possible regression in the previous patches of this series. intel_fbc_can_choose: returns true for gen 4/5/6/7. (possible bug) so intel_crtc_state->enable_fbc = true; will be executed for first crtc everytime intel_fbc_choose_crtc is called.

[Intel-gfx] [PATCH 0/4] Enable FBC on SKL, v3

2016-04-04 Thread Paulo Zanoni
Now with the suggestion from Chris instead of the old workaround. We don't need new DDX patches anymore, but now we need new IGT patches. Chris Wilson (1): drm/i915: use ORIGIN_CPU for frontbuffer invalidation on WC mmaps Paulo Zanoni (3): drm/i915/fbc: update busy_bits even for GTT and flip

[Intel-gfx] [PATCH 0/4] Enable FBC on SKL, v2

2016-03-24 Thread Paulo Zanoni
Hi The Kernel patches are mostly the same, but with the review suggestions implemented. The DDX patch is new and should really help things now. Thanks, Paulo Paulo Zanoni (4): drm/i915/fbc: update busy_bits even for GTT and flip flushes drm/i915/fbc: sanitize i915.enable_fbc during FBC init

[Intel-gfx] [PATCH 0/4] Enable FBC on SKL

2016-03-21 Thread Paulo Zanoni
Hi Here's a patch series to workaround the current SKL FBC problems, then try to enable it by default. Between patch 2/4 and the DDX patch, we only need one of them for things to work. We'll also be fine if we merge both, but we need at least one. No IGT changes are needed due to the automatic w