| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20091215114922.GA23415@linux.vnet.ibm.com>
Date: Tue, 15 Dec 2009 17:19:22 +0530
From: Arun R Bharadwaj <arun@...ux.vnet.ibm.com>
To: Benjamin Herrenschmidt <benh@...nel.crashing.org>
Cc: Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Ingo Molnar <mingo@...e.hu>,
Vaidyanathan Srinivasan <svaidy@...ux.vnet.ibm.com>,
Dipankar Sarma <dipankar@...ibm.com>,
Balbir Singh <balbir@...ibm.com>,
Venkatesh Pallipadi <venkatesh.pallipadi@...el.com>,
linux-kernel@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org,
linux-arch@...r.kernel.org, linux-acpi@...r.kernel.org,
Arun Bharadwaj <arun@...ux.vnet.ibm.com>
Subject: Re: [v10 PATCH 8/9]: pSeries: implement pSeries processor idle
module
* Benjamin Herrenschmidt <benh@...nel.crashing.org> [2009-12-04 21:00:52]:
> On Fri, 2009-12-04 at 13:45 +0530, Arun R Bharadwaj wrote:
>
> >
> > Hi Ben,
> >
> > I forgot to attach the patch which enables cpuidle for the rest of the
> > POWER platforms. Attaching it below.
> >
> > So for these platforms, ppc_md.power_save will be called from from the
> > cpuidle_idle_call idle loop itself. Also, this cpuidle_idle_call is
> > not a pseries specific idle loop. It is a common loop for Intel and
> > PPC which use cpuidle infrastructure.
>
> Ok, so there was a missing piece in the puzzle ;-)
>
> I'll review asap.
>
Hi Ben,
Did you get time to review this?
thanks
arun
> Cheers,
> Ben.
>
> > arun
> >
> >
> >
> > This patch enables cpuidle for the rest of the POWER platforms like
> > 44x, Cell, Pasemi etc.
> >
> > Signed-off-by: Arun R Bharadwaj <arun@...ux.vnet.ibm.com>
> > ---
> > arch/powerpc/include/asm/system.h | 2 ++
> > arch/powerpc/kernel/idle.c | 28 ++++++++++++++++++++++++++++
> > arch/powerpc/kernel/setup_32.c | 8 ++++++--
> > arch/powerpc/platforms/44x/idle.c | 2 ++
> > arch/powerpc/platforms/cell/pervasive.c | 2 ++
> > arch/powerpc/platforms/pasemi/idle.c | 2 ++
> > arch/powerpc/platforms/ps3/setup.c | 2 ++
> > 7 files changed, 44 insertions(+), 2 deletions(-)
> >
> > Index: linux.trees.git/arch/powerpc/include/asm/system.h
> > ===================================================================
> > --- linux.trees.git.orig/arch/powerpc/include/asm/system.h
> > +++ linux.trees.git/arch/powerpc/include/asm/system.h
> > @@ -551,8 +551,10 @@ void cpu_idle_wait(void);
> >
> > #ifdef CONFIG_CPU_IDLE
> > extern void update_smt_snooze_delay(int snooze);
> > +extern void setup_cpuidle_ppc(void);
> > #else
> > static inline void update_smt_snooze_delay(int snooze) {}
> > +static inline void setup_cpuidle_ppc(void) {}
> > #endif
> >
> > #endif /* __KERNEL__ */
> > Index: linux.trees.git/arch/powerpc/kernel/idle.c
> > ===================================================================
> > --- linux.trees.git.orig/arch/powerpc/kernel/idle.c
> > +++ linux.trees.git/arch/powerpc/kernel/idle.c
> > @@ -129,6 +129,34 @@ void default_idle(void)
> > HMT_very_low();
> > }
> >
> > +#ifdef CONFIG_CPU_IDLE
> > +DEFINE_PER_CPU(struct cpuidle_device, ppc_idle_devices);
> > +struct cpuidle_driver cpuidle_ppc_driver = {
> > + .name = "cpuidle_ppc",
> > +};
> > +
> > +static void ppc_idle_loop(struct cpuidle_device *dev, struct cpuidle_state *st)
> > +{
> > + ppc_md.power_save();
> > +}
> > +
> > +void setup_cpuidle_ppc(void)
> > +{
> > + struct cpuidle_device *dev;
> > + int cpu;
> > +
> > + cpuidle_register_driver(&cpuidle_ppc_driver);
> > +
> > + for_each_online_cpu(cpu) {
> > + dev = &per_cpu(ppc_idle_devices, cpu);
> > + dev->cpu = cpu;
> > + dev->states[0].enter = ppc_idle_loop;
> > + dev->state_count = 1;
> > + cpuidle_register_device(dev);
> > + }
> > +}
> > +#endif
> > +
> > int powersave_nap;
> >
> > #ifdef CONFIG_SYSCTL
> > Index: linux.trees.git/arch/powerpc/kernel/setup_32.c
> > ===================================================================
> > --- linux.trees.git.orig/arch/powerpc/kernel/setup_32.c
> > +++ linux.trees.git/arch/powerpc/kernel/setup_32.c
> > @@ -133,14 +133,18 @@ notrace void __init machine_init(unsigne
> >
> > #ifdef CONFIG_6xx
> > if (cpu_has_feature(CPU_FTR_CAN_DOZE) ||
> > - cpu_has_feature(CPU_FTR_CAN_NAP))
> > + cpu_has_feature(CPU_FTR_CAN_NAP)) {
> > ppc_md.power_save = ppc6xx_idle;
> > + setup_cpuidle_ppc();
> > + }
> > #endif
> >
> > #ifdef CONFIG_E500
> > if (cpu_has_feature(CPU_FTR_CAN_DOZE) ||
> > - cpu_has_feature(CPU_FTR_CAN_NAP))
> > + cpu_has_feature(CPU_FTR_CAN_NAP)) {
> > ppc_md.power_save = e500_idle;
> > + setup_cpuidle_ppc();
> > + }
> > #endif
> > if (ppc_md.progress)
> > ppc_md.progress("id mach(): done", 0x200);
> > Index: linux.trees.git/arch/powerpc/platforms/44x/idle.c
> > ===================================================================
> > --- linux.trees.git.orig/arch/powerpc/platforms/44x/idle.c
> > +++ linux.trees.git/arch/powerpc/platforms/44x/idle.c
> > @@ -24,6 +24,7 @@
> > #include <linux/of.h>
> > #include <linux/kernel.h>
> > #include <asm/machdep.h>
> > +#include <asm/system.h>
> >
> > static int mode_spin;
> >
> > @@ -46,6 +47,7 @@ int __init ppc44x_idle_init(void)
> > /* If we are not setting spin mode
> > then we set to wait mode */
> > ppc_md.power_save = &ppc44x_idle;
> > + setup_cpuidle_ppc();
> > }
> >
> > return 0;
> > Index: linux.trees.git/arch/powerpc/platforms/cell/pervasive.c
> > ===================================================================
> > --- linux.trees.git.orig/arch/powerpc/platforms/cell/pervasive.c
> > +++ linux.trees.git/arch/powerpc/platforms/cell/pervasive.c
> > @@ -35,6 +35,7 @@
> > #include <asm/pgtable.h>
> > #include <asm/reg.h>
> > #include <asm/cell-regs.h>
> > +#include <asm/system.h>
> >
> > #include "pervasive.h"
> >
> > @@ -128,5 +129,6 @@ void __init cbe_pervasive_init(void)
> > }
> >
> > ppc_md.power_save = cbe_power_save;
> > + setup_cpuidle_ppc();
> > ppc_md.system_reset_exception = cbe_system_reset_exception;
> > }
> > Index: linux.trees.git/arch/powerpc/platforms/pasemi/idle.c
> > ===================================================================
> > --- linux.trees.git.orig/arch/powerpc/platforms/pasemi/idle.c
> > +++ linux.trees.git/arch/powerpc/platforms/pasemi/idle.c
> > @@ -27,6 +27,7 @@
> > #include <asm/machdep.h>
> > #include <asm/reg.h>
> > #include <asm/smp.h>
> > +#include <asm/system.h>
> >
> > #include "pasemi.h"
> >
> > @@ -81,6 +82,7 @@ static int __init pasemi_idle_init(void)
> >
> > ppc_md.system_reset_exception = pasemi_system_reset_exception;
> > ppc_md.power_save = modes[current_mode].entry;
> > + setup_cpuidle_ppc();
> > printk(KERN_INFO "Using PA6T idle loop (%s)\n", modes[current_mode].name);
> >
> > return 0;
> > Index: linux.trees.git/arch/powerpc/platforms/ps3/setup.c
> > ===================================================================
> > --- linux.trees.git.orig/arch/powerpc/platforms/ps3/setup.c
> > +++ linux.trees.git/arch/powerpc/platforms/ps3/setup.c
> > @@ -33,6 +33,7 @@
> > #include <asm/prom.h>
> > #include <asm/lv1call.h>
> > #include <asm/ps3gpu.h>
> > +#include <asm/system.h>
> >
> > #include "platform.h"
> >
> > @@ -214,6 +215,7 @@ static void __init ps3_setup_arch(void)
> > prealloc_ps3flash_bounce_buffer();
> >
> > ppc_md.power_save = ps3_power_save;
> > + setup_cpuidle_ppc();
> > ps3_os_area_init();
> >
> > DBG(" <- %s:%d\n", __func__, __LINE__);
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-arch" in
> > the body of a message to majordomo@...r.kernel.org
> > More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists