[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20241031040626.631-1-zhengshaobo1@xiaomi.com>
Date: Thu, 31 Oct 2024 12:06:23 +0800
From: ZhengShaobo <zhengshaobo1@...omi.com>
To: <rafael@...nel.org>
CC: <chendejun@...omi.com>, <chuci@...omi.com>, <daniel.lezcano@...aro.org>,
<dingchongchong@...omi.com>, <linux-kernel@...r.kernel.org>,
<linux-pm@...r.kernel.org>, <lukasz.luba@....com>, <rui.zhang@...el.com>,
<zhengshaobo1@...omi.com>
Subject: Re: [PATCH] thermal: gov_power_allocator: Granted power set to max when nobody request power
On Thu, Oct 24, 2024 at 05:26:49PM +0200, Rafael J. Wysocki wrote:
> On Thu, Oct 24, 2024 at 10:01 AM ZhengShaobo <zhengshaobo1@...omi.com> wrote:
> >
> > On Wed, Oct 23, 2024 at 12:09:44PM +0200, Rafael J. Wysocki wrote:
> > > On Mon, Oct 21, 2024 at 2:12 PM ZhengShaobo <zhengshaobo1@...omi.com> wrote:
> > > >
> > > > From: zhengshaobo1 <zhengshaobo1@...omi.com>
> > > >
> > > > When total_req_power is 0, divvy_up_power() will set granted_power to 0,
> > > > and cdev will be limited to the lowest performance. If our polling delay
> > > > is set to 200ms, it means that cdev cannot perform better within 200ms
> > > > even if cdev has a sudden load. This will affect the performance of cdev
> > > > and is not as expected.
> > > >
> > > > For this reason, if nobody requests power, then set the granted power to
> > > > the max_power.
> > > >
> > > > Signed-off-by: zhengshaobo1 <zhengshaobo1@...omi.com>
> > >
> > > I would have applied this, but your S-o-b above needs to be fixed.
> > > Why don't you use your real name there?
> > >
> > > If it can be changed to "ZhengShaobo <zhengshaobo1@...omi.com>",
> > > please let me know, and I will fix it for you when applying the patch.
> > >
> > Yes, it should be "ZhengShaobo <zhengshaobo1@...omi.com>".
> > I would really appreciate your help in solving this problem.
>
> OK, applied as 6.13 material with the S-o-b tag as per the above, thanks!
I would like to backport the patch to android15-6.6, but this needs to be merged
into the maintainer tree. I am wondering when this patch will be merged into your
maintainer tree, and I hope you can let me know.
Thanks,
ZhengShaobo
Powered by blists - more mailing lists