Messages in this thread | | | From | Michael Ellerman <> | Subject | Re: [PATCH] cpuidle: pseries: Mark pseries_idle_proble() as __init | Date | Wed, 04 Aug 2021 10:54:04 +1000 |
| |
Nathan Chancellor <nathan@kernel.org> writes: > After commit 7cbd631d4dec ("cpuidle: pseries: Fixup CEDE0 latency only > for POWER10 onwards"), pseries_idle_probe() is no longer inlined when > compiling with clang, which causes a modpost warning: > > WARNING: modpost: vmlinux.o(.text+0xc86a54): Section mismatch in > reference from the function pseries_idle_probe() to the function > .init.text:fixup_cede0_latency() > The function pseries_idle_probe() references > the function __init fixup_cede0_latency(). > This is often because pseries_idle_probe lacks a __init > annotation or the annotation of fixup_cede0_latency is wrong. > > pseries_idle_probe() is a non-init function, which calls > fixup_cede0_latency(), which is an init function, explaining the > mismatch. pseries_idle_probe() is only called from > pseries_processor_idle_init(), which is an init function, so mark > pseries_idle_probe() as __init so there is no more warning. > > Fixes: 054e44ba99ae ("cpuidle: pseries: Add function to parse extended CEDE records") > Signed-off-by: Nathan Chancellor <nathan@kernel.org> > --- > drivers/cpuidle/cpuidle-pseries.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
I don't see this in my builds for some reason, but I guess toolchain or config differences probably explain it.
Regardless, the patch is correct so I'll pick it up, thanks.
cheers
> diff --git a/drivers/cpuidle/cpuidle-pseries.c b/drivers/cpuidle/cpuidle-pseries.c > index bba449b77641..7e7ab5597d7a 100644 > --- a/drivers/cpuidle/cpuidle-pseries.c > +++ b/drivers/cpuidle/cpuidle-pseries.c > @@ -403,7 +403,7 @@ static void __init fixup_cede0_latency(void) > * pseries_idle_probe() > * Choose state table for shared versus dedicated partition > */ > -static int pseries_idle_probe(void) > +static int __init pseries_idle_probe(void) > { > > if (cpuidle_disable != IDLE_NO_OVERRIDE) > > base-commit: a6cae77f1bc89368a4e2822afcddc45c3062d499 > -- > 2.33.0.rc0
| |