On Fri, Aug 29, 2014 at 03:09:49PM +0300, Ville Syrjälä wrote: > On Fri, Aug 29, 2014 at 12:21:27PM +0100, Chris Wilson wrote: > > On Fri, Aug 29, 2014 at 02:14:07PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > > > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > > > > > WaGsvRC0ResidenncyMethod is for vlv, it doesn't deal with chv > > > appropriately (eg. doesn't limit rps values to even numbers). > > > > > > Fix a typo in the w/a name while at it. > > > > You could review my changes for the vlv manual calculations that would > > fix it for chv as well... > > IIRC I glanced at them but there was too much noise in some of the > patches for my poor brain so I gave up. Could you clarify whether you mean WaGsvRC0ResidenncyMethod is only intended for vlv (fixed for chv), or whether just the implementation is currently only for chv. If the former, and the code is no longer required, Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
