Re: [PATCH 2/3] drm/i915/guc/slpc: Enable GuC SLPC default strategies

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Wed, Jan 08, 2025 at 09:18:41PM -0800, Dixit, Ashutosh wrote:
> On Wed, 08 Jan 2025 13:19:16 -0800, Rodrigo Vivi wrote:
> >
> 
> Hi Rodrigo,
> 
> > On Wed, Jan 08, 2025 at 11:30:59AM -0800, Dixit, Ashutosh wrote:
> > > On Wed, 08 Jan 2025 11:11:53 -0800, Belgaumkar, Vinay wrote:
> > > >
> > > >
> > > > On 1/8/2025 6:13 AM, Rodrigo Vivi wrote:
> > > > > The Balancer and DCC strategies were left off on a fear that
> > > > > these strategies would conflict with the i915's waitboost.
> > > > > However, these strategies are only active in certain conditions where
> > > > > the system is TDP limited. So, they don't conflict, but help the
> > > > > waitboost by guaranteeing a bit more of GT frequency.
> > > > >
> > > > > Without these strategies we were likely leaving some performance
> > > > > behind on some scenarious.
> > > >
> > > > s/scenarious/scenarios
> >
> > I do need to re-enable my spell checker during git commit :)
> >
> > > >
> > > > We should also mention that the platform defaults for enabling/disabling
> > > > DCC/Balancer will now be chosen by GuC.
> > > >
> > > > Reviewed-by: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> > > >
> > > > >
> > > > > Cc: Vinay Belgaumkar <vinay.belgaumkar@intel.com>
> > > > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > >
> > > Does this need a Fixes: and Cc:stable?
> >
> > nope... this is more like enabling a new feature...
> 
> Hmm, if this really improves performance, as the commit message seems to
> suggest, why wouldn't we want this in stable, or at least LTS, kernels? So
> why isn't this a performance bug fix?

hmm... performance is not listed in the stable-rules as a obvious case, and
it is not a regression, is really a new feature, that has the potential to
find some later corner cases. So, safer to just enable as a new feature...

Also a feature that is really a corner case for the end user... needs to
be using both cpu and gpu with high utilization and a limited power budget.
General benchmarks and usages don't hit this scenario very easily.

> 
> Ashutosh