| Message ID | 1460574097-14073-1-git-send-email-paulo.r.zanoni@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Apr 13, 2016 at 04:01:37PM -0300, Paulo Zanoni wrote: > Now that we're more protected against user space doing frontbuffer > mmap rendering, the last - how many times did I say this before? - > SKL problem seems to be solved. So let's give it a try. > > And since the other newer platforms are still under preliminary > support, let's enable FBC on them too to make sure we fix FBC on them. > Also, it seems KBL is passing the tests. > > If you reached this commit through git bisect or if you just want more > information about FBC, please see: > commit a98ee79317b4091cafb502b4ffdbbbe1335e298c > Author: Paulo Zanoni <paulo.r.zanoni@intel.com> > Date: Tue Feb 16 18:47:21 2016 -0200 > drm/i915/fbc: enable FBC by default on HSW and BDW > > v2: Enable not only on SKL, but for everything new (Daniel). > v3: Rebase after the intel_sanitize_fbc_option() change. > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> \o/ let's pop the champange! Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> > --- > drivers/gpu/drm/i915/intel_fbc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c > index c3dffba..1043482 100644 > --- a/drivers/gpu/drm/i915/intel_fbc.c > +++ b/drivers/gpu/drm/i915/intel_fbc.c > @@ -1230,7 +1230,8 @@ static int intel_sanitize_fbc_option(struct drm_i915_private *dev_priv) > if (i915.enable_fbc >= 0) > return !!i915.enable_fbc; > > - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) > + if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv) || > + INTEL_GEN(dev_priv) >= 9) > return 1; > > return 0; > -- > 2.8.0.rc3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/intel-gfx
diff --git a/drivers/gpu/drm/i915/intel_fbc.c b/drivers/gpu/drm/i915/intel_fbc.c index c3dffba..1043482 100644 --- a/drivers/gpu/drm/i915/intel_fbc.c +++ b/drivers/gpu/drm/i915/intel_fbc.c @@ -1230,7 +1230,8 @@ static int intel_sanitize_fbc_option(struct drm_i915_private *dev_priv) if (i915.enable_fbc >= 0) return !!i915.enable_fbc; - if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv)) + if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv) || + INTEL_GEN(dev_priv) >= 9) return 1; return 0;
Now that we're more protected against user space doing frontbuffer mmap rendering, the last - how many times did I say this before? - SKL problem seems to be solved. So let's give it a try. And since the other newer platforms are still under preliminary support, let's enable FBC on them too to make sure we fix FBC on them. Also, it seems KBL is passing the tests. If you reached this commit through git bisect or if you just want more information about FBC, please see: commit a98ee79317b4091cafb502b4ffdbbbe1335e298c Author: Paulo Zanoni <paulo.r.zanoni@intel.com> Date: Tue Feb 16 18:47:21 2016 -0200 drm/i915/fbc: enable FBC by default on HSW and BDW v2: Enable not only on SKL, but for everything new (Daniel). v3: Rebase after the intel_sanitize_fbc_option() change. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> --- drivers/gpu/drm/i915/intel_fbc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)