| Message ID | 1392417283-26185-1-git-send-email-chris@chris-wilson.co.uk (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Fri, Feb 14, 2014 at 10:34:43PM +0000, Chris Wilson wrote: > In commit e4e0c058a19c41150d12ad2d3023b3cf09c5de67 > Author: Eugeni Dodonov <eugeni.dodonov@intel.com> > Date: Wed Feb 8 12:53:50 2012 -0800 > > drm/i915: gen7: Implement an L3 caching workaround. > > the L3 cache aging was disabled. This was part of a shotgun response > to a number of GPU hang bugs, but there appears to be no documentation > to suggest that disabling the L3 cache age was ever required (to prevent > the GPU hangs). Bspec still lists the 0xF value in the w/a list, but the actual register description doesn't say that this bit needs to be set. All the other bits match either the default values, or specific notes about which bits need to be set. So I get the feeling this should be fine: Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> > > Restoring the L3 cache age is a minor performance win of around 2% > on IVB:GT2. (Note that this value seems to be consistent across a number > of tests and so appears to be above the usual noise.) > > Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> > Cc: Kenneth Graunke <kenneth@whitecape.org> > --- > drivers/gpu/drm/i915/i915_reg.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h > index a2c68c9492ba..d40d71bb66b6 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -4171,7 +4171,7 @@ > #define VLV_B0_WA_L3SQCREG1_VALUE 0x00D30000 > > #define GEN7_L3CNTLREG1 0xB01C > -#define GEN7_WA_FOR_GEN7_L3_CONTROL 0x3C4FFF8C > +#define GEN7_WA_FOR_GEN7_L3_CONTROL 0x3C47FF8C > #define GEN7_L3AGDIS (1<<19) > > #define GEN7_L3_CHICKEN_MODE_REGISTER 0xB030 > -- > 1.9.0.rc3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Mon, Feb 17, 2014 at 10:59:09AM +0200, Ville Syrjälä wrote: > On Fri, Feb 14, 2014 at 10:34:43PM +0000, Chris Wilson wrote: > > In commit e4e0c058a19c41150d12ad2d3023b3cf09c5de67 > > Author: Eugeni Dodonov <eugeni.dodonov@intel.com> > > Date: Wed Feb 8 12:53:50 2012 -0800 > > > > drm/i915: gen7: Implement an L3 caching workaround. > > > > the L3 cache aging was disabled. This was part of a shotgun response > > to a number of GPU hang bugs, but there appears to be no documentation > > to suggest that disabling the L3 cache age was ever required (to prevent > > the GPU hangs). > > Bspec still lists the 0xF value in the w/a list, but the actual register > description doesn't say that this bit needs to be set. All the other > bits match either the default values, or specific notes about which bits > need to be set. > > So I get the feeling this should be fine: > Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Queued for -next, thanks for the patch. -Daniel
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index a2c68c9492ba..d40d71bb66b6 100644 --- a/drivers/gpu/drm/i915/i915_reg.h +++ b/drivers/gpu/drm/i915/i915_reg.h @@ -4171,7 +4171,7 @@ #define VLV_B0_WA_L3SQCREG1_VALUE 0x00D30000 #define GEN7_L3CNTLREG1 0xB01C -#define GEN7_WA_FOR_GEN7_L3_CONTROL 0x3C4FFF8C +#define GEN7_WA_FOR_GEN7_L3_CONTROL 0x3C47FF8C #define GEN7_L3AGDIS (1<<19) #define GEN7_L3_CHICKEN_MODE_REGISTER 0xB030
In commit e4e0c058a19c41150d12ad2d3023b3cf09c5de67 Author: Eugeni Dodonov <eugeni.dodonov@intel.com> Date: Wed Feb 8 12:53:50 2012 -0800 drm/i915: gen7: Implement an L3 caching workaround. the L3 cache aging was disabled. This was part of a shotgun response to a number of GPU hang bugs, but there appears to be no documentation to suggest that disabling the L3 cache age was ever required (to prevent the GPU hangs). Restoring the L3 cache age is a minor performance win of around 2% on IVB:GT2. (Note that this value seems to be consistent across a number of tests and so appears to be above the usual noise.) Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Kenneth Graunke <kenneth@whitecape.org> --- drivers/gpu/drm/i915/i915_reg.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)