Re: [PATCH v4 1/6] x86,sched: Add support for frequency invariance

From: Peter Zijlstra
Date: Wed Dec 18 2019 - 14:34:45 EST


On Fri, Dec 06, 2019 at 12:57:46PM +0100, Giovanni Gherdovich wrote:
> > On Wednesday 13 Nov 2019 at 13:46:49 (+0100), Giovanni Gherdovich wrote:
> > I see above that you enable the static key (and therefore frequency
> > invariance before setting the max frequency ratio (if possible) and
> > before you initialise the counter references. Is there any reason for
> > doing this?

> This is a fair point; mine was a deliberate choice but you're the second
> person making this remark (Peter Zijlstra also suggested I find the max
> frequency before I set the static key), so it appears this design is
> unpopular
> enough to warrant a change in v5.

You actually 'fix' this in the next patch. I thought it was a patch
management 'fail' that it didn't end up in this patch.