Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 3 Jun 2020 20:58:53 +0100
From:   Russell King - ARM Linux admin <linux@armlinux.org.uk>
To:     Vincent Guittot <vincent.guittot@linaro.org>
Cc:     Valentin Schneider <valentin.schneider@arm.com>,
        Thara Gopinath <thara.gopinath@linaro.org>,
        linux-kernel <linux-kernel@vger.kernel.org>
Subject: Re: v5.7: new core kernel option missing help text
Message-ID: <20200603195853.GD1551@shell.armlinux.org.uk>
References: <20200603173150.GB1551@shell.armlinux.org.uk>
 <jhjh7vshvwl.mognet@arm.com>
 <20200603184500.GC1551@shell.armlinux.org.uk>
 <CAKfTPtBdN30ChMgFqqT1bzeU6HExXEQFrQjxbCK-hRT4HEiQkQ@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAKfTPtBdN30ChMgFqqT1bzeU6HExXEQFrQjxbCK-hRT4HEiQkQ@mail.gmail.com>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Wed, Jun 03, 2020 at 09:24:56PM +0200, Vincent Guittot wrote:
> On Wed, 3 Jun 2020 at 20:45, Russell King - ARM Linux admin
> <linux@armlinux.org.uk> wrote:
> > It's a start.  I'm still wondering whether I should answer yes or no
> > for the platforms I'm building for.
> >
> > So far, all I've found is:
> >
> > arch/arm/include/asm/topology.h:#define arch_scale_thermal_pressure topology_get_thermal_pressure
> >
> > which really doesn't tell me anything about this.  So I'm still in
> > the dark.
> >
> > I guess topology_get_thermal_pressure is provided by something in
> > drivers/ which will be conditional on some driver or something.
> 
> You need cpufreq_cooling device to make it useful and only for SMP
> I don't think that this should not be user configurable because even
> with the description above, it is not easy to choose.
> This should be set by the driver that implement the feature which is
> only cpufreq cooling device for now it

As I have CONFIG_CPU_FREQ_THERMAL=y in my config, I'm guessing (and it's
only a guess) that I should say y to SCHED_THERMAL_PRESSURE ?

> > > +     help
> > > +       This option allows the scheduler to be aware of CPU thermal throttling
> > > +       (i.e. thermal pressure), providing arch_scale_thermal_pressure() is
> > > +       implemented.

Is this feature documented in terms of what it does?  Do I assume that
as the thermal trip points start tripping, that has an influence on
the scheduler?  Or is it the case that the scheduler is wanting to
know when the cpu frequency changes?

Grepping for "thermal" in Documentation/scheduler brings up nothing.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTC for 0.8m (est. 1762m) line in suburbia: sync at 13.1Mbps down 424kbps up