Re: [PATCH] add i386 idle notifier (take 3)

2007-01-09 Thread Stephane Eranian
Hello, On Fri, Jan 05, 2007 at 02:36:53PM +0100, Adrian Bunk wrote: > > > > > Where does the perfmon code use the EXPORT_SYMBOL's? > > > > > > > > The perfmon patch includes several kernel modules which make use of > > > > the exported entry points. The following symbols are exported: > > > > >

Re: [PATCH] add i386 idle notifier (take 3)

2007-01-05 Thread Adrian Bunk
On Fri, Jan 05, 2007 at 02:55:14AM -0800, Stephane Eranian wrote: > Adrian, > > On Thu, Jan 04, 2007 at 12:07:08AM +0100, Adrian Bunk wrote: > > > I am hearing conflicting opinions on this one. > > > > > > Perfmon is a fairly big patch. It is hard to take it as one. I have tried > > > to > > > s

Re: [PATCH] add i386 idle notifier (take 3)

2007-01-05 Thread Stephane Eranian
Adrian, On Thu, Jan 04, 2007 at 12:07:08AM +0100, Adrian Bunk wrote: > > I am hearing conflicting opinions on this one. > > > > Perfmon is a fairly big patch. It is hard to take it as one. I have tried to > > split it up in smaller, more manageable pieces as requested by top-level > > maintainers

Re: [PATCH] add i386 idle notifier (take 3)

2007-01-03 Thread Adrian Bunk
On Wed, Jan 03, 2007 at 05:20:15AM -0800, Stephane Eranian wrote: > Adrian, > > On Sat, Dec 23, 2006 at 12:40:15PM +0100, Adrian Bunk wrote: > > > > > > If you look at the perfmon-new-base patch, you'll see a base.diff patch > > > which > > > includes this one. I am slowly getting rid of this re

Re: [PATCH] add i386 idle notifier (take 3)

2007-01-03 Thread Stephane Eranian
Adrian, On Sat, Dec 23, 2006 at 12:40:15PM +0100, Adrian Bunk wrote: > > > > If you look at the perfmon-new-base patch, you'll see a base.diff patch > > which > > includes this one. I am slowly getting rid of this requirement by pushing > > those "infrastructure patches" to mainline so that the

Re: [PATCH] add i386 idle notifier (take 3)

2006-12-23 Thread Adrian Bunk
On Fri, Dec 22, 2006 at 02:07:00AM -0800, Stephane Eranian wrote: > Andrian, > > On Fri, Dec 22, 2006 at 02:06:41AM +0100, Adrian Bunk wrote: > > > changelog: > > > - add a notifier mechanism to the low level idle loop. You can > > > register a callback function which gets invoked on entry a

Re: [PATCH] add i386 idle notifier (take 3)

2006-12-22 Thread Stephane Eranian
Andrian, On Fri, Dec 22, 2006 at 02:06:41AM +0100, Adrian Bunk wrote: > > changelog: > > - add a notifier mechanism to the low level idle loop. You can > > register a callback function which gets invoked on entry and exit > > from the low level idle loop. The low level idle loop is

Re: [PATCH] add i386 idle notifier (take 3)

2006-12-21 Thread Adrian Bunk
On Thu, Dec 21, 2006 at 01:12:42AM -0800, Stephane Eranian wrote: > Andrew, > > On Wed, Dec 20, 2006 at 09:05:14PM -0800, Andrew Morton wrote: > > On Wed, 20 Dec 2006 06:05:00 -0800 > > Stephane Eranian <[EMAIL PROTECTED]> wrote: > > > > > Hello, > > > > > > Here is the latest version of the idl

Re: [PATCH] add i386 idle notifier (take 3)

2006-12-21 Thread Stephane Eranian
Andrew, On Wed, Dec 20, 2006 at 09:05:14PM -0800, Andrew Morton wrote: > On Wed, 20 Dec 2006 06:05:00 -0800 > Stephane Eranian <[EMAIL PROTECTED]> wrote: > > > Hello, > > > > Here is the latest version of the idle notifier for i386. > > This patch is against 2.6.20-rc1 (GIT). In this kernel, the

Re: [PATCH] add i386 idle notifier (take 3)

2006-12-20 Thread Andrew Morton
On Wed, 20 Dec 2006 06:05:00 -0800 Stephane Eranian <[EMAIL PROTECTED]> wrote: > Hello, > > Here is the latest version of the idle notifier for i386. > This patch is against 2.6.20-rc1 (GIT). In this kernel, the idle > loop code was modified such that the lowest level idle > routines do not have

[PATCH] add i386 idle notifier (take 3)

2006-12-20 Thread Stephane Eranian
Hello, Here is the latest version of the idle notifier for i386. This patch is against 2.6.20-rc1 (GIT). In this kernel, the idle loop code was modified such that the lowest level idle routines do not have loops anymore (e.g., poll_idle). As such, we do not need to call enter_idle() in all the int