On Wed, 2022-01-12 at 12:24 +0100, Joachim Wiberg wrote: > From: Tobias Waldekranz <tob...@waldekranz.com> > > This means an idle guest won't needlessly consume an entire core on > the host, waiting for work to show up. > > Signed-off-by: Tobias Waldekranz <tob...@waldekranz.com> > Signed-off-by: Joachim Wiberg <troglo...@gmail.com> > --- > arch/powerpc/platforms/85xx/qemu_e500.c | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/arch/powerpc/platforms/85xx/qemu_e500.c > b/arch/powerpc/platforms/85xx/qemu_e500.c > index a4127b0b161f..4c4d577effd9 100644 > --- a/arch/powerpc/platforms/85xx/qemu_e500.c > +++ b/arch/powerpc/platforms/85xx/qemu_e500.c > @@ -67,4 +67,9 @@ define_machine(qemu_e500) { > .get_irq = mpic_get_coreint_irq, > .calibrate_decr = generic_calibrate_decr, > .progress = udbg_progress, > +#ifdef CONFIG_PPC64 > + .power_save = book3e_idle, > +#else > + .power_save = e500_idle, > +#endif > };
In the 32-bit case shouldn't this already be getting added by setup_power_save()? Though I see corenet_generic.c doing the same thing... -Scott