Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-20 Thread Gautham R Shenoy
On Thu, Nov 15, 2007 at 05:05:20PM +0100, Ingo Molnar wrote: > > * Ingo Molnar <[EMAIL PROTECTED]> wrote: > > > FYI, i've put these 3 patches into the scheduler git tree and it's > > looking very good so far. So unless Andrew or Linus objects to put > > this into v2.6.24, or there's serious

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-20 Thread Gautham R Shenoy
On Thu, Nov 15, 2007 at 05:05:20PM +0100, Ingo Molnar wrote: * Ingo Molnar [EMAIL PROTECTED] wrote: FYI, i've put these 3 patches into the scheduler git tree and it's looking very good so far. So unless Andrew or Linus objects to put this into v2.6.24, or there's serious questions

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Oleg Nesterov
On 11/15, Gautham R Shenoy wrote: > > +static struct { > + struct task_struct *active_writer; > + struct mutex lock; /* Synchronizes accesses to refcount, */ > + /* > + * Also blocks the new readers during > + * an ongoing cpu hotplug operation. > + */ > + int

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Ingo Molnar
* Ingo Molnar <[EMAIL PROTECTED]> wrote: > FYI, i've put these 3 patches into the scheduler git tree and it's > looking very good so far. So unless Andrew or Linus objects to put > this into v2.6.24, or there's serious questions during review, could > we merge it this way? i've got this

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Ingo Molnar
* Ingo Molnar <[EMAIL PROTECTED]> wrote: > * Gautham R Shenoy <[EMAIL PROTECTED]> wrote: > > > void lock_cpu_hotplug(void) > > { > > - struct task_struct *tsk = current; > > - > > - if (tsk == recursive) { > > - static int warnings = 10; > > - if (warnings) { > > -

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Ingo Molnar
* Gautham R Shenoy <[EMAIL PROTECTED]> wrote: > void lock_cpu_hotplug(void) > { > - struct task_struct *tsk = current; > - > - if (tsk == recursive) { > - static int warnings = 10; > - if (warnings) { > - printk(KERN_ERR "Lukewarm IQ detected

[RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Gautham R Shenoy
From: Gautham R Shenoy <[EMAIL PROTECTED]> Date: Thu, 15 Nov 2007 18:14:20 +0530 Subject: [PATCH 1/3] cpu-hotplug: Refcount based Cpu Hotplug implementation This patch implements a Refcount + Waitqueue based model for cpu-hotplug. Now, a thread which wants to prevent cpu-hotplug, will bump up a

[RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Gautham R Shenoy
From: Gautham R Shenoy [EMAIL PROTECTED] Date: Thu, 15 Nov 2007 18:14:20 +0530 Subject: [PATCH 1/3] cpu-hotplug: Refcount based Cpu Hotplug implementation This patch implements a Refcount + Waitqueue based model for cpu-hotplug. Now, a thread which wants to prevent cpu-hotplug, will bump up a

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Ingo Molnar
* Gautham R Shenoy [EMAIL PROTECTED] wrote: void lock_cpu_hotplug(void) { - struct task_struct *tsk = current; - - if (tsk == recursive) { - static int warnings = 10; - if (warnings) { - printk(KERN_ERR Lukewarm IQ detected in hotplug

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Oleg Nesterov
On 11/15, Gautham R Shenoy wrote: +static struct { + struct task_struct *active_writer; + struct mutex lock; /* Synchronizes accesses to refcount, */ + /* + * Also blocks the new readers during + * an ongoing cpu hotplug operation. + */ + int refcount; +

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Ingo Molnar
* Ingo Molnar [EMAIL PROTECTED] wrote: FYI, i've put these 3 patches into the scheduler git tree and it's looking very good so far. So unless Andrew or Linus objects to put this into v2.6.24, or there's serious questions during review, could we merge it this way? i've got this trivial

Re: [RFC PATCH 1/3] cpu-hotplug: Refcount Based Cpu Hotplug implementation

2007-11-15 Thread Ingo Molnar
* Ingo Molnar [EMAIL PROTECTED] wrote: * Gautham R Shenoy [EMAIL PROTECTED] wrote: void lock_cpu_hotplug(void) { - struct task_struct *tsk = current; - - if (tsk == recursive) { - static int warnings = 10; - if (warnings) { -