Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sat, 17 Dec 2022 22:01:33 +0000
From:   Qais Yousef <qyousef@layalina.io>
To:     Bagas Sanjaya <bagasdotme@gmail.com>
Cc:     Ingo Molnar <mingo@kernel.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Vincent Guittot <vincent.guittot@linaro.org>,
        Dietmar Eggemann <dietmar.eggemann@arm.com>,
        Jonathan Corbet <corbet@lwn.net>, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        Lukasz Luba <lukasz.luba@arm.com>,
        Xuewen Yan <xuewen.yan94@gmail.com>, Wei Wang <wvw@google.com>,
        Jonathan JMChen <Jonathan.JMChen@mediatek.com>,
        Hank <han.lin@mediatek.com>, Paul Bone <pbone@mozilla.com>,
        Russell King <linux@armlinux.org.uk>,
        Linus Walleij <linus.walleij@linaro.org>,
        Catalin Marinas <catalin.marinas@arm.com>
Subject: Re: [PATCH v3] Documentation: sched: Document util clamp feature
Message-ID: <20221217220133.p5wh7drpvkmkups2@airbuntu>
References: <20221216235716.201923-1-qyousef@layalina.io>
 <Y52Ri28ThsM4iU8X@debian.me>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <Y52Ri28ThsM4iU8X@debian.me>
Precedence: bulk

On 12/17/22 16:53, Bagas Sanjaya wrote:
> On Fri, Dec 16, 2022 at 11:57:16PM +0000, Qais Yousef wrote:
> > +Another example is in Android where tasks are classified as background,
> > +foreground, top-app, etc. Util clamp can be used to constrain how much
> > +resources background tasks are consuming by capping the performance point they
> > +can run at. This constraint helps reserve resources for important tasks, like
> > +the ones belonging to the currently active app (top-app group). Beside this
> > +helps in limiting how much power they consume. This can be more obvious in
> > +heterogeneous systems (e.g. Arm big.LITTLE); the constraint will help bias the
> > +background tasks to stay on the little cores which will ensure that:
> > +
> > +        1. The big cores are free to run top-app tasks immediately. top-app
> > +           tasks are the tasks the user is currently interacting with, hence
> > +           the most important tasks in the system.
> > +        2. They don't run on a power hungry core and drain battery even if they
> > +           are CPU intensive tasks.
> > +
> > +.. note::
> > +  **little cores**:
> > +    CPUs with capacity < 1024
> > +
> > +  **big cores**:
> > +    CPUs with capacity = 1024
> 
> Processing capacity (CPU frequency) in MHz? This is the first time I

This is a scheduler abstraction value from 0-1024. Biggest CPU always has
a capacity of 1024. On SMP all cpus appear as 1024 CPUs.

> hear Arm big.LITTLE architecture. CC'ing several Arm folks and
> linux-arm-kernel list for I'm unsure on this.

This has my Arm SoB :-)

This was added based on Lukasz comments. Both Lukasz and Dietmar were my
colleagues at Arm and are on CC.

I don't think it was necessary to expand the recipients list. But probably more
is better though all the scheduler people are covered. This has nothing to do
with Arm but it's the prime example of heterogeneous systems.

> 
> > +
> > +By making these uclamp performance requests, or rather hints, user space can
> > +ensure system resources are used optimally to deliver the best possible user
> > +experience.
> > +
> > +Another use case is to help with **overcoming the ramp up latency inherit in
> > +how scheduler utilization signal is calculated**.
> 
> IMO the bold text isn't needed (why did you highlight the phrase above)?
> 
> > +When task @p is running, **the scheduler should try its best to ensure it
> > +starts at 40% performance level**. If the task runs for a long enough time so
> > +that its actual utilization goes above 80%, the utilization, or performance
> > +level, will be capped.
> 
> Same here.
> 
> > +**Generally it is advised to perceive the input as performance level or point
> > +which will imply both task placement and frequency selection**.
> 
> Same here too.

All of these were added based on Lukasz review comments. He thought they're
important pieces and should be highlighted, I opted to do that by making them
bold text.


Thanks!

--
Qais Yousef