[patch 4/9] powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-15 Thread arnd
From: Christian Krafft <[EMAIL PROTECTED]> This patch adds a cpufreq governor that takes the number of running spus into account. It's very similar to the ondemand governor, but not as complex. Instead of hacking spu load into the ondemand governor it might be easier to have cpufreq accepting mult

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-15 Thread Arnd Bergmann
On Wednesday 09 July 2008, Dave Jones wrote: > > diff --git a/arch/powerpc/platforms/cell/Kconfig > b/arch/powerpc/platforms/cell/Kconfig > index 3959fcf..19f4b4d 100644 > --- a/arch/powerpc/platforms/cell/Kconfig > +++ b/arch/powerpc/platforms/cell/Kconfig > @@ -91,6 +91,7 @@ config CBE_THERM >

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-10 Thread Dominik Brodowski
On Thu, Jul 10, 2008 at 11:05:08PM +0200, Arnd Bergmann wrote: > On Thursday 10 July 2008, Dominik Brodowski wrote: > > Actually, "cpufreq-set -g spu_governor" should be enough. (BTW, do we really > > want this name?) > > Do you have a better suggestion for the name? I don't think there was a > pa

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-10 Thread Arnd Bergmann
On Thursday 10 July 2008, Dominik Brodowski wrote: > Actually, "cpufreq-set -g spu_governor" should be enough. (BTW, do we really > want this name?) Do you have a better suggestion for the name? I don't think there was a particularly strong reason to choose it over something else. Arnd <>

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-10 Thread Dominik Brodowski
On Tue, Jul 08, 2008 at 11:27:28AM -0400, Dave Jones wrote: > On Tue, Jul 08, 2008 at 08:43:43AM +0200, Arnd Bergmann wrote: > > On Monday 07 July 2008, Dave Jones wrote: > > > One question I do have though, is how userspace scripts are supposed > > > to know they're to echo cbe_spu_governor int

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-08 Thread Dave Jones
On Wed, Jul 09, 2008 at 03:18:59PM +1000, Ben Herrenschmidt wrote: > On Mon, 2008-07-07 at 17:02 +0200, Arnd Bergmann wrote: > > From: Christian Krafft <[EMAIL PROTECTED]> > > > > This patch adds a cpufreq governor that takes the number of running spus > > into account. It's very similar to t

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-08 Thread Benjamin Herrenschmidt
On Mon, 2008-07-07 at 17:02 +0200, Arnd Bergmann wrote: > From: Christian Krafft <[EMAIL PROTECTED]> > > This patch adds a cpufreq governor that takes the number of running spus > into account. It's very similar to the ondemand governor, but not as complex. > Instead of hacking spu load into the o

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-08 Thread Dave Jones
On Wed, Jul 09, 2008 at 01:41:38PM +1000, Ben Herrenschmidt wrote: > On Tue, 2008-07-08 at 11:27 -0400, Dave Jones wrote: > > On Tue, Jul 08, 2008 at 08:43:43AM +0200, Arnd Bergmann wrote: > > > On Monday 07 July 2008, Dave Jones wrote: > > > > One question I do have though, is how userspace

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-08 Thread Benjamin Herrenschmidt
On Tue, 2008-07-08 at 11:27 -0400, Dave Jones wrote: > On Tue, Jul 08, 2008 at 08:43:43AM +0200, Arnd Bergmann wrote: > > On Monday 07 July 2008, Dave Jones wrote: > > > One question I do have though, is how userspace scripts are supposed > > > to know they're to echo cbe_spu_governor into the r

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-08 Thread Dave Jones
On Tue, Jul 08, 2008 at 08:43:43AM +0200, Arnd Bergmann wrote: > On Monday 07 July 2008, Dave Jones wrote: > > One question I do have though, is how userspace scripts are supposed > > to know they're to echo cbe_spu_governor into the relevant parts of > > sysfs.  I've not used anything with a c

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-07 Thread Arnd Bergmann
On Monday 07 July 2008, Dave Jones wrote: > One question I do have though, is how userspace scripts are supposed > to know they're to echo cbe_spu_governor into the relevant parts of > sysfs.  I've not used anything with a cell. Do they expose the SPUs > as regular CPUs, or do they show up in a dif

Re: powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-07 Thread Dave Jones
On Mon, Jul 07, 2008 at 05:02:30PM +0200, Arnd Bergmann wrote: > From: Christian Krafft <[EMAIL PROTECTED]> > > This patch adds a cpufreq governor that takes the number of running spus > into account. It's very similar to the ondemand governor, but not as complex. > Instead of hacking spu loa

Re: [Cbe-oss-dev] powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-07 Thread Arnd Bergmann
On Monday 07 July 2008, Eric Blossom wrote: > I've got a basic question about this idea: > > Does it throttle only idle spus or is there a single control for > the entire PPE + SPE complex? There is one setting per socket, i.e. 8 SPUs and 1 PPU. In IBM speak, that is slow mode, which would be C s

Re: [Cbe-oss-dev] powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-07 Thread Eric Blossom
On Mon, Jul 07, 2008 at 05:02:30PM +0200, Arnd Bergmann wrote: > From: Christian Krafft <[EMAIL PROTECTED]> > > This patch adds a cpufreq governor that takes the number of running spus > into account. It's very similar to the ondemand governor, but not as complex. > Instead of hacking spu load int

powerpc/cell/cpufreq: add spu aware cpufreq governor

2008-07-07 Thread Arnd Bergmann
From: Christian Krafft <[EMAIL PROTECTED]> This patch adds a cpufreq governor that takes the number of running spus into account. It's very similar to the ondemand governor, but not as complex. Instead of hacking spu load into the ondemand governor it might be easier to have cpufreq accepting mult