Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-04-09 Thread Preeti U Murthy
Hi Thomas, Any comments on this patch? Regards Preeti U Murthy On 04/01/2014 11:02 AM, Preeti U Murthy wrote: > On 03/28/2014 02:17 PM, Srivatsa S. Bhat wrote: >> On 03/27/2014 03:44 PM, Preeti U Murthy wrote: >>> On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: Actually, my suggestion

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-04-09 Thread Preeti U Murthy
Hi Thomas, Any comments on this patch? Regards Preeti U Murthy On 04/01/2014 11:02 AM, Preeti U Murthy wrote: On 03/28/2014 02:17 PM, Srivatsa S. Bhat wrote: On 03/27/2014 03:44 PM, Preeti U Murthy wrote: On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: Actually, my suggestion was to remove

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-31 Thread Preeti U Murthy
On 03/28/2014 02:17 PM, Srivatsa S. Bhat wrote: > On 03/27/2014 03:44 PM, Preeti U Murthy wrote: >> On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: >>> >>> Actually, my suggestion was to remove the dying CPU from the force_mask >>> alone, >>> in the CPU_DYING notifier. The rest of the cleanup

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-31 Thread Preeti U Murthy
On 03/28/2014 02:17 PM, Srivatsa S. Bhat wrote: On 03/27/2014 03:44 PM, Preeti U Murthy wrote: On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: Actually, my suggestion was to remove the dying CPU from the force_mask alone, in the CPU_DYING notifier. The rest of the cleanup (removing it from

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-28 Thread Srivatsa S. Bhat
On 03/27/2014 03:44 PM, Preeti U Murthy wrote: > On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: >> >> Actually, my suggestion was to remove the dying CPU from the force_mask >> alone, >> in the CPU_DYING notifier. The rest of the cleanup (removing it from the >> other >> masks, moving the

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-28 Thread Srivatsa S. Bhat
On 03/27/2014 03:44 PM, Preeti U Murthy wrote: On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: Actually, my suggestion was to remove the dying CPU from the force_mask alone, in the CPU_DYING notifier. The rest of the cleanup (removing it from the other masks, moving the broadcast duty to

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-27 Thread Preeti U Murthy
On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: > > Actually, my suggestion was to remove the dying CPU from the force_mask alone, > in the CPU_DYING notifier. The rest of the cleanup (removing it from the other > masks, moving the broadcast duty to someone else etc can still be done at > the

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-27 Thread Srivatsa S. Bhat
On 03/27/2014 08:32 AM, Preeti U Murthy wrote: > On 03/26/2014 04:51 PM, Srivatsa S. Bhat wrote: >> On 03/26/2014 09:26 AM, Preeti U Murthy wrote: >>> Its possible that the tick_broadcast_force_mask contains cpus which are not >>> in cpu_online_mask when a broadcast tick occurs. This could happen

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-27 Thread Srivatsa S. Bhat
On 03/27/2014 08:32 AM, Preeti U Murthy wrote: On 03/26/2014 04:51 PM, Srivatsa S. Bhat wrote: On 03/26/2014 09:26 AM, Preeti U Murthy wrote: Its possible that the tick_broadcast_force_mask contains cpus which are not in cpu_online_mask when a broadcast tick occurs. This could happen under the

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-27 Thread Preeti U Murthy
On 03/27/2014 11:58 AM, Srivatsa S. Bhat wrote: Actually, my suggestion was to remove the dying CPU from the force_mask alone, in the CPU_DYING notifier. The rest of the cleanup (removing it from the other masks, moving the broadcast duty to someone else etc can still be done at the CPU_DEAD

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-26 Thread Preeti U Murthy
On 03/26/2014 04:51 PM, Srivatsa S. Bhat wrote: > On 03/26/2014 09:26 AM, Preeti U Murthy wrote: >> Its possible that the tick_broadcast_force_mask contains cpus which are not >> in cpu_online_mask when a broadcast tick occurs. This could happen under the >> following circumstance assuming CPU1 is

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-26 Thread Preeti U Murthy
On 03/26/2014 04:51 PM, Srivatsa S. Bhat wrote: > On 03/26/2014 09:26 AM, Preeti U Murthy wrote: >> Its possible that the tick_broadcast_force_mask contains cpus which are not >> in cpu_online_mask when a broadcast tick occurs. This could happen under the >> following circumstance assuming CPU1 is

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-26 Thread Srivatsa S. Bhat
On 03/26/2014 09:26 AM, Preeti U Murthy wrote: > Its possible that the tick_broadcast_force_mask contains cpus which are not > in cpu_online_mask when a broadcast tick occurs. This could happen under the > following circumstance assuming CPU1 is among the CPUs waiting for broadcast. > > CPU0

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-26 Thread Srivatsa S. Bhat
On 03/26/2014 09:26 AM, Preeti U Murthy wrote: Its possible that the tick_broadcast_force_mask contains cpus which are not in cpu_online_mask when a broadcast tick occurs. This could happen under the following circumstance assuming CPU1 is among the CPUs waiting for broadcast. CPU0

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-26 Thread Preeti U Murthy
On 03/26/2014 04:51 PM, Srivatsa S. Bhat wrote: On 03/26/2014 09:26 AM, Preeti U Murthy wrote: Its possible that the tick_broadcast_force_mask contains cpus which are not in cpu_online_mask when a broadcast tick occurs. This could happen under the following circumstance assuming CPU1 is among

Re: [PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-26 Thread Preeti U Murthy
On 03/26/2014 04:51 PM, Srivatsa S. Bhat wrote: On 03/26/2014 09:26 AM, Preeti U Murthy wrote: Its possible that the tick_broadcast_force_mask contains cpus which are not in cpu_online_mask when a broadcast tick occurs. This could happen under the following circumstance assuming CPU1 is among

[PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-25 Thread Preeti U Murthy
Its possible that the tick_broadcast_force_mask contains cpus which are not in cpu_online_mask when a broadcast tick occurs. This could happen under the following circumstance assuming CPU1 is among the CPUs waiting for broadcast. CPU0CPU1 Run CPU_DOWN_PREPARE

[PATCH] tick, broadcast: Prevent false alarm when force mask contains offline cpus

2014-03-25 Thread Preeti U Murthy
Its possible that the tick_broadcast_force_mask contains cpus which are not in cpu_online_mask when a broadcast tick occurs. This could happen under the following circumstance assuming CPU1 is among the CPUs waiting for broadcast. CPU0CPU1 Run CPU_DOWN_PREPARE