* [PATCH] powercap/drivers/dtpm_cpu: Fix suspend failure and kernel warning [not found] <20211107095001.GC11442@xsang-OptiPlex-9020> @ 2021-11-08 6:23 ` Daniel Lezcano 2021-11-10 20:19 ` Rafael J. Wysocki 0 siblings, 1 reply; 2+ messages in thread From: Daniel Lezcano @ 2021-11-08 6:23 UTC (permalink / raw) To: rafael Cc: linux-pm, daniel.lezcano, Kenneth R . Crudup, kernel test robot, lkp, lkp, Daniel Lezcano, open list When the ENERGY_MODEL and DTPM_CPU are enabled but actually without any energy model, at cpu hotplug time, the dead cpuhp callback fails leading to the warning. Actually, the check could be simplified and we only do an action if the dtpm cpu is enabled, otherwise we bail out without error. Fixes: 7a89d7eacf8e ("powercap/drivers/dtpm: Simplify the dtpm table") Reported-by: Kenneth R. Crudup <kenny@panix.com> Tested-by: Kenneth R. Crudup <kenny@panix.com> Reported-by: kernel test robot <oliver.sang@intel.com> Cc: lkp@lists.01.org Cc: lkp@intel.com Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> --- drivers/powercap/dtpm_cpu.c | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/drivers/powercap/dtpm_cpu.c b/drivers/powercap/dtpm_cpu.c index 69eb5101b5c7..477042199d90 100644 --- a/drivers/powercap/dtpm_cpu.c +++ b/drivers/powercap/dtpm_cpu.c @@ -166,16 +166,13 @@ static struct dtpm_ops dtpm_ops = { static int cpuhp_dtpm_cpu_offline(unsigned int cpu) { - struct em_perf_domain *pd; struct dtpm_cpu *dtpm_cpu; - pd = em_cpu_get(cpu); - if (!pd) - return -EINVAL; - dtpm_cpu = per_cpu(dtpm_per_cpu, cpu); + if (dtpm_cpu) + dtpm_update_power(&dtpm_cpu->dtpm); - return dtpm_update_power(&dtpm_cpu->dtpm); + return 0; } static int cpuhp_dtpm_cpu_online(unsigned int cpu) -- 2.25.1 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] powercap/drivers/dtpm_cpu: Fix suspend failure and kernel warning 2021-11-08 6:23 ` [PATCH] powercap/drivers/dtpm_cpu: Fix suspend failure and kernel warning Daniel Lezcano @ 2021-11-10 20:19 ` Rafael J. Wysocki 0 siblings, 0 replies; 2+ messages in thread From: Rafael J. Wysocki @ 2021-11-10 20:19 UTC (permalink / raw) To: Daniel Lezcano Cc: Rafael J. Wysocki, Linux PM, Kenneth R . Crudup, kernel test robot, lkp, kbuild test robot, Daniel Lezcano, open list On Mon, Nov 8, 2021 at 7:24 AM Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > When the ENERGY_MODEL and DTPM_CPU are enabled but actually without > any energy model, at cpu hotplug time, the dead cpuhp callback fails > leading to the warning. > > Actually, the check could be simplified and we only do an action if > the dtpm cpu is enabled, otherwise we bail out without error. > > Fixes: 7a89d7eacf8e ("powercap/drivers/dtpm: Simplify the dtpm table") > Reported-by: Kenneth R. Crudup <kenny@panix.com> > Tested-by: Kenneth R. Crudup <kenny@panix.com> > Reported-by: kernel test robot <oliver.sang@intel.com> > Cc: lkp@lists.01.org > Cc: lkp@intel.com > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> > --- > drivers/powercap/dtpm_cpu.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-) > > diff --git a/drivers/powercap/dtpm_cpu.c b/drivers/powercap/dtpm_cpu.c > index 69eb5101b5c7..477042199d90 100644 > --- a/drivers/powercap/dtpm_cpu.c > +++ b/drivers/powercap/dtpm_cpu.c > @@ -166,16 +166,13 @@ static struct dtpm_ops dtpm_ops = { > > static int cpuhp_dtpm_cpu_offline(unsigned int cpu) > { > - struct em_perf_domain *pd; > struct dtpm_cpu *dtpm_cpu; > > - pd = em_cpu_get(cpu); > - if (!pd) > - return -EINVAL; > - > dtpm_cpu = per_cpu(dtpm_per_cpu, cpu); > + if (dtpm_cpu) > + dtpm_update_power(&dtpm_cpu->dtpm); > > - return dtpm_update_power(&dtpm_cpu->dtpm); > + return 0; > } > > static int cpuhp_dtpm_cpu_online(unsigned int cpu) > -- Applied as 5.16-rc material, thanks! ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-11-10 20:19 UTC | newest] Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <20211107095001.GC11442@xsang-OptiPlex-9020> 2021-11-08 6:23 ` [PATCH] powercap/drivers/dtpm_cpu: Fix suspend failure and kernel warning Daniel Lezcano 2021-11-10 20:19 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).