On Thu, Jul 23, 2020 at 02:21:10PM +0530, Srikar Dronamraju wrote: > Move topology fixup based on the platform attributes into its own > function which is called just before set_sched_topology. > > Cc: linuxppc-dev <[email protected]> > Cc: LKML <[email protected]> > Cc: Michael Ellerman <[email protected]> > Cc: Nicholas Piggin <[email protected]> > Cc: Anton Blanchard <[email protected]> > Cc: Oliver O'Halloran <[email protected]> > Cc: Nathan Lynch <[email protected]> > Cc: Michael Neuling <[email protected]> > Cc: Gautham R Shenoy <[email protected]> > Cc: Ingo Molnar <[email protected]> > Cc: Peter Zijlstra <[email protected]> > Cc: Valentin Schneider <[email protected]> > Cc: Jordan Niethe <[email protected]> > Signed-off-by: Srikar Dronamraju <[email protected]>
Reviewed-by: Gautham R. Shenoy <[email protected]> > --- > Changelog v2 -> v3: > Rewrote changelog (Gautham) > Renamed to powerpc/smp: Move topology fixups into a new function > > arch/powerpc/kernel/smp.c | 17 +++++++++++------ > 1 file changed, 11 insertions(+), 6 deletions(-) > > diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c > index a685915e5941..da27f6909be1 100644 > --- a/arch/powerpc/kernel/smp.c > +++ b/arch/powerpc/kernel/smp.c > @@ -1368,6 +1368,16 @@ int setup_profiling_timer(unsigned int multiplier) > return 0; > } > > +static void fixup_topology(void) > +{ > +#ifdef CONFIG_SCHED_SMT > + if (has_big_cores) { > + pr_info("Big cores detected but using small core scheduling\n"); > + powerpc_topology[0].mask = smallcore_smt_mask; > + } > +#endif > +} > + > void __init smp_cpus_done(unsigned int max_cpus) > { > /* > @@ -1381,12 +1391,7 @@ void __init smp_cpus_done(unsigned int max_cpus) > > dump_numa_cpu_topology(); > > -#ifdef CONFIG_SCHED_SMT > - if (has_big_cores) { > - pr_info("Big cores detected but using small core scheduling\n"); > - powerpc_topology[0].mask = smallcore_smt_mask; > - } > -#endif > + fixup_topology(); > set_sched_topology(powerpc_topology); > } > > -- > 2.18.2 >
