Received-SPF: pass (google.com: domain of linux-kernel+bounces-137117-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Precedence: bulk
MIME-Version: 1.0
References: <20240403-gpa-no-cooling-devs-v2-0-79bdd8439449@trvn.ru> <594fdc5a-3b09-40df-98a0-43671665e55f@lausen.nl>
In-Reply-To: <594fdc5a-3b09-40df-98a0-43671665e55f@lausen.nl>
From: Nikita Travkin <nikitos.tr@gmail.com>
Date: Tue, 9 Apr 2024 19:46:50 +0500
Message-ID: <CAN_S-bXxKBjB5knjcUXW=rVwhCx0f1-pDMjmf9V9+ZLYSGZxvg@mail.gmail.com>
Subject: Re: [PATCH v2 0/3] gov_power_allocator: Allow binding before cooling devices
To: Leonard Lausen <leonard@lausen.nl>
Cc: nikita@trvn.ru, Lukasz Luba <lukasz.luba@arm.com>, 
	"Rafael J. Wysocki" <rafael@kernel.org>, Daniel Lezcano <daniel.lezcano@linaro.org>, 
	Zhang Rui <rui.zhang@intel.com>, "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>, 
	linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, 
	regressions@lists.linux.dev
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

=D0=B2=D1=82, 9 =D0=B0=D0=BF=D1=80. 2024=E2=80=AF=D0=B3. =D0=B2 19:42, Leon=
ard Lausen <leonard@lausen.nl>:
>
> Hi Nikita, Hi =C5=81ukasz,
>
> thank you for fixing the e83747c2f8e3 ("thermal: gov_power_allocator: Set=
 up trip points earlier") and 912e97c67cc3 ("thermal: gov_power_allocator: =
Move memory allocation out of throttle()") regressions as part of v6.9-rc3.=
 As the regression was introduced in v6.8, would it be possible to include =
the fix in a v6.8 patch release?
>

Hi! I think these both have already been picked for stable:

https://lore.kernel.org/r/20240408125314.939341866@linuxfoundation.org
https://lore.kernel.org/r/20240408125314.969670696@linuxfoundation.org/

Nikita

> Thank you
> Leonard
>
> (Resending with regressions@lists.linux.dev in CC)
>
> #regzbot introduced: 912e97c67cc3f333c4c5df8f51498c651792e658
> #regzbot fixed-by: 1057c4c36ef8b236a2e28edef301da0801338c5f
>
>
> #regzbot introduced: e83747c2f8e3cc5e284e37a8921099f1901d79d8
> #regzbot fixed-by: da781936e7c301e6197eb6513775748e79fb2575
>
> On 4/3/24 07:31, Nikita Travkin via B4 Relay wrote:
> > Recent changes in IPA made it fail probing if the TZ has no cooling
> > devices attached on probe or no trip points defined.
> >
> > This series restores prior behavior to:
> >
> > - allow IPA to probe before cooling devices have attached;
> > - allow IPA to probe when the TZ has no passive/active trip points.
> >
> > I've noticed that all thermal zones fail probing with -EINVAL on my
> > sc7180 based Acer Aspire 1 since 6.8. This series allows me to bring
> > them back.
> >
> > Additionally there is a commit that supresses the "sustainable_power
> > will be estimated" warning on TZ that have no trip points (and thus IPA
> > will not be able to do anything for them anyway). This allowed me to
> > notice that some of the TZ with cooling_devices on my platform actually
> > lack the sustainable_power value.
> >
> > Signed-off-by: Nikita Travkin <nikita@trvn.ru>
> > ---
> > Changes in v2:
> > - Split to two changes (Lukasz)
> > - Return 0 in allocate_actors_buffer() instead of suppressing -EINVAL
> >   (Lukasz)
> > - Add a change to supress "sustainable_power will be estimated" warning
> >   on "empty" TZ
> > - Link to v1: https://lore.kernel.org/r/20240321-gpa-no-cooling-devs-v1=
-1-5c9e0ef2062e@trvn.ru
> >
> > ---
> > Nikita Travkin (3):
> >       thermal: gov_power_allocator: Allow binding without cooling devic=
es
> >       thermal: gov_power_allocator: Allow binding without trip points
> >       thermal: gov_power_allocator: Suppress sustainable_power warning =
without trip_points
> >
> >  drivers/thermal/gov_power_allocator.c | 16 ++++++----------
> >  1 file changed, 6 insertions(+), 10 deletions(-)
> > ---
> > base-commit: 727900b675b749c40ba1f6669c7ae5eb7eb8e837
> > change-id: 20240321-gpa-no-cooling-devs-c79ee3288325
> >
> > Best regards,