On Wed, May 06, 2015 at 07:32:08PM +0300, Imre Deak wrote: > On Wed, 2015-05-06 at 17:01 +0100, Nick Hoath wrote: > > On 29/04/2015 14:02, Deak, Imre wrote: > > > On pe, 2015-04-10 at 13:12 +0100, Nick Hoath wrote: > > >> Signed-off-by: Nick Hoath <nicholas.hoath@xxxxxxxxx> > > >> --- > > >> drivers/gpu/drm/i915/intel_ringbuffer.c | 5 +++++ > > >> 1 file changed, 5 insertions(+) > > >> > > >> diff --git a/drivers/gpu/drm/i915/intel_ringbuffer.c b/drivers/gpu/drm/i915/intel_ringbuffer.c > > >> index eebee73..cc62e5c 100644 > > >> --- a/drivers/gpu/drm/i915/intel_ringbuffer.c > > >> +++ b/drivers/gpu/drm/i915/intel_ringbuffer.c > > >> @@ -1054,6 +1054,10 @@ static int bxt_init_workarounds(struct intel_engine_cs *ring) > > >> GEN7_SBE_SS_CACHE_DISPATCH_PORT_SHARING_DISABLE); > > >> } > > >> > > >> + /* WaForceContextSaveRestoreNonCoherent:bxt */ > > >> + WA_SET_BIT_MASKED(HDC_CHICKEN0, > > >> + HDC_FORCE_CONTEXT_SAVE_RESTORE_NON_COHERENT); > > >> + > > > > > > Looks ok, but again it needs to be added for SKL (all steppings) in > > > gen9_init_workarounds. > > > > The same argument as for patch 2/3 applies? > > Yes, applying this on SKL as a follow up is ok: > > Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx> Queued for -next, thanks for the patch. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
