| Message ID | 1396336408-20954-3-git-send-email-ego@linux.vnet.ibm.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
On Tue, 2014-04-01 at 12:43 +0530, Gautham R. Shenoy wrote: > From: "Gautham R. Shenoy" <ego@linux.vnet.ibm.com> > > Enable CPUFreq for PowerNV. Select "performance", "powersave", > "userspace" and "ondemand" governors. Choose "ondemand" to be the > default governor. Rafael, are you going to take these or should I send them to Linus ? (I'd rather you take them :-) Cheers, Ben. > > Signed-off-by: Gautham R. Shenoy <ego@linux.vnet.ibm.com> > Signed-off-by: Srivatsa S. Bhat <srivatsa.bhat@linux.vnet.ibm.com> > --- > arch/powerpc/configs/pseries_defconfig | 1 + > arch/powerpc/configs/pseries_le_defconfig | 1 + > arch/powerpc/platforms/powernv/Kconfig | 6 ++++++ > 3 files changed, 8 insertions(+) > > diff --git a/arch/powerpc/configs/pseries_defconfig b/arch/powerpc/configs/pseries_defconfig > index 9ea8342b..a905063 100644 > --- a/arch/powerpc/configs/pseries_defconfig > +++ b/arch/powerpc/configs/pseries_defconfig > @@ -306,3 +306,4 @@ CONFIG_KVM_BOOK3S_64=m > CONFIG_KVM_BOOK3S_64_HV=y > CONFIG_TRANSPARENT_HUGEPAGE=y > CONFIG_TRANSPARENT_HUGEPAGE_ALWAYS=y > +CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y > diff --git a/arch/powerpc/configs/pseries_le_defconfig b/arch/powerpc/configs/pseries_le_defconfig > index 3c84f9d..58e3dbf 100644 > --- a/arch/powerpc/configs/pseries_le_defconfig > +++ b/arch/powerpc/configs/pseries_le_defconfig > @@ -301,3 +301,4 @@ CONFIG_CRYPTO_LZO=m > # CONFIG_CRYPTO_ANSI_CPRNG is not set > CONFIG_CRYPTO_DEV_NX=y > CONFIG_CRYPTO_DEV_NX_ENCRYPT=m > +CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y > diff --git a/arch/powerpc/platforms/powernv/Kconfig b/arch/powerpc/platforms/powernv/Kconfig > index 895e8a2..c252ee9 100644 > --- a/arch/powerpc/platforms/powernv/Kconfig > +++ b/arch/powerpc/platforms/powernv/Kconfig > @@ -11,6 +11,12 @@ config PPC_POWERNV > select PPC_UDBG_16550 > select PPC_SCOM > select ARCH_RANDOM > + select CPU_FREQ > + select CPU_FREQ_GOV_PERFORMANCE > + select CPU_FREQ_GOV_POWERSAVE > + select CPU_FREQ_GOV_USERSPACE > + select CPU_FREQ_GOV_ONDEMAND > + select CPU_FREQ_GOV_CONSERVATIVE > default y > > config PPC_POWERNV_RTAS
On Tuesday, April 01, 2014 08:46:49 PM Benjamin Herrenschmidt wrote: > On Tue, 2014-04-01 at 12:43 +0530, Gautham R. Shenoy wrote: > > From: "Gautham R. Shenoy" <ego@linux.vnet.ibm.com> > > > > Enable CPUFreq for PowerNV. Select "performance", "powersave", > > "userspace" and "ondemand" governors. Choose "ondemand" to be the > > default governor. > > Rafael, are you going to take these or should I send them to Linus ? > > (I'd rather you take them :-) I can do that, but the timing is pretty bad. How urgent are they? Rafael
On Wed, 2014-04-02 at 00:03 +0200, Rafael J. Wysocki wrote: > > Rafael, are you going to take these or should I send them to Linus ? > > > > (I'd rather you take them :-) > > I can do that, but the timing is pretty bad. How urgent are they? To be honest pretty urgent. It's a new drop-in driver so it shouldn't be a huge deal. We have distros really waiting for this to hit upstream... But I'll leave that decision to you :-) Cheers, Ben.
On Wednesday, April 02, 2014 03:23:28 PM Benjamin Herrenschmidt wrote: > On Wed, 2014-04-02 at 00:03 +0200, Rafael J. Wysocki wrote: > > > Rafael, are you going to take these or should I send them to Linus ? > > > > > > (I'd rather you take them :-) > > > > I can do that, but the timing is pretty bad. How urgent are they? > > To be honest pretty urgent. It's a new drop-in driver so it shouldn't > be a huge deal. > > We have distros really waiting for this to hit upstream... But I'll > leave that decision to you :-) Well, I've already sent a second pull request for this merge window and I'm not going to send the next one before it is merged. So, I can push them next week realistically. Rafael
On Wed, 2014-04-02 at 13:03 +0200, Rafael J. Wysocki wrote: > > To be honest pretty urgent. It's a new drop-in driver so it > shouldn't > > be a huge deal. > > > > We have distros really waiting for this to hit upstream... But I'll > > leave that decision to you :-) > > Well, I've already sent a second pull request for this merge window > and I'm not going to send the next one before it is merged. > > So, I can push them next week realistically. Thanks. You can blame me for the lateness if you want (and the debate as to which tree should carry them :-) Cheers, Ben.
diff --git a/arch/powerpc/configs/pseries_defconfig b/arch/powerpc/configs/pseries_defconfig index 9ea8342b..a905063 100644 --- a/arch/powerpc/configs/pseries_defconfig +++ b/arch/powerpc/configs/pseries_defconfig @@ -306,3 +306,4 @@ CONFIG_KVM_BOOK3S_64=m CONFIG_KVM_BOOK3S_64_HV=y CONFIG_TRANSPARENT_HUGEPAGE=y CONFIG_TRANSPARENT_HUGEPAGE_ALWAYS=y +CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y diff --git a/arch/powerpc/configs/pseries_le_defconfig b/arch/powerpc/configs/pseries_le_defconfig index 3c84f9d..58e3dbf 100644 --- a/arch/powerpc/configs/pseries_le_defconfig +++ b/arch/powerpc/configs/pseries_le_defconfig @@ -301,3 +301,4 @@ CONFIG_CRYPTO_LZO=m # CONFIG_CRYPTO_ANSI_CPRNG is not set CONFIG_CRYPTO_DEV_NX=y CONFIG_CRYPTO_DEV_NX_ENCRYPT=m +CONFIG_CPU_FREQ_DEFAULT_GOV_ONDEMAND=y diff --git a/arch/powerpc/platforms/powernv/Kconfig b/arch/powerpc/platforms/powernv/Kconfig index 895e8a2..c252ee9 100644 --- a/arch/powerpc/platforms/powernv/Kconfig +++ b/arch/powerpc/platforms/powernv/Kconfig @@ -11,6 +11,12 @@ config PPC_POWERNV select PPC_UDBG_16550 select PPC_SCOM select ARCH_RANDOM + select CPU_FREQ + select CPU_FREQ_GOV_PERFORMANCE + select CPU_FREQ_GOV_POWERSAVE + select CPU_FREQ_GOV_USERSPACE + select CPU_FREQ_GOV_ONDEMAND + select CPU_FREQ_GOV_CONSERVATIVE default y config PPC_POWERNV_RTAS