Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-18 Thread Akinobu Mita
> >[...] However, it might break slab. > >If I am not mistaken, slab code initializes multiple objects in > >CPU_UP_PREPARE and relies on the CPU_UP_CANCELLED to destroy the > >objects which successfully got initialized before the some object's > >initialization went bad. > > My testing machine

Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-18 Thread Gautham R Shenoy
Hi! On Wed, Jul 18, 2007 at 02:18:41AM +0900, Akinobu Mita wrote: > >> So it is natural to deliver CPU_UP_CANCELED event only to the functions > >> that have returned NOTIFY_OK with CPU_UP_PREPARE event and not to call > >> the function that have returned NOTIFY_BAD. This is what this patch is >

Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-18 Thread Gautham R Shenoy
Hi! On Wed, Jul 18, 2007 at 02:18:41AM +0900, Akinobu Mita wrote: So it is natural to deliver CPU_UP_CANCELED event only to the functions that have returned NOTIFY_OK with CPU_UP_PREPARE event and not to call the function that have returned NOTIFY_BAD. This is what this patch is doing.

Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-18 Thread Akinobu Mita
[...] However, it might break slab. If I am not mistaken, slab code initializes multiple objects in CPU_UP_PREPARE and relies on the CPU_UP_CANCELLED to destroy the objects which successfully got initialized before the some object's initialization went bad. My testing machine is ordinary

Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-17 Thread Akinobu Mita
> So it is natural to deliver CPU_UP_CANCELED event only to the functions > that have returned NOTIFY_OK with CPU_UP_PREPARE event and not to call > the function that have returned NOTIFY_BAD. This is what this patch is doing. Yes, this makes sense. Thank you for making sure of it. [...]

Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-17 Thread Gautham R Shenoy
Hi Akinobu, On Mon, Jul 16, 2007 at 10:53:47PM +0900, Akinobu Mita wrote: > The functions in a CPU notifier chain is called with CPU_UP_PREPARE event > before making the CPU online. If one of the callback returns NOTIFY_BAD, > it stops to deliver CPU_UP_PREPARE event, and CPU online operation is

Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-17 Thread Gautham R Shenoy
Hi Akinobu, On Mon, Jul 16, 2007 at 10:53:47PM +0900, Akinobu Mita wrote: The functions in a CPU notifier chain is called with CPU_UP_PREPARE event before making the CPU online. If one of the callback returns NOTIFY_BAD, it stops to deliver CPU_UP_PREPARE event, and CPU online operation is

Re: [PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-17 Thread Akinobu Mita
So it is natural to deliver CPU_UP_CANCELED event only to the functions that have returned NOTIFY_OK with CPU_UP_PREPARE event and not to call the function that have returned NOTIFY_BAD. This is what this patch is doing. Yes, this makes sense. Thank you for making sure of it. [...]

[PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-16 Thread Akinobu Mita
The functions in a CPU notifier chain is called with CPU_UP_PREPARE event before making the CPU online. If one of the callback returns NOTIFY_BAD, it stops to deliver CPU_UP_PREPARE event, and CPU online operation is canceled. Then CPU_UP_CANCELED event is delivered to the functions in a CPU

[PATCH 4/10] cpu: deliver CPU_UP_CANCELED only to NOTIFY_OKed callbacks with CPU_UP_PREPARE

2007-07-16 Thread Akinobu Mita
The functions in a CPU notifier chain is called with CPU_UP_PREPARE event before making the CPU online. If one of the callback returns NOTIFY_BAD, it stops to deliver CPU_UP_PREPARE event, and CPU online operation is canceled. Then CPU_UP_CANCELED event is delivered to the functions in a CPU