Messages in this thread | | | Date | Tue, 4 Dec 2012 07:17:34 -0800 | From | Tejun Heo <> | Subject | Re: [RFC PATCH 01/10] CPU hotplug: Introduce "stable" cpu online mask, for atomic hotplug readers |
| |
Hello, Srivatsa.
On Tue, Dec 04, 2012 at 02:23:41PM +0530, Srivatsa S. Bhat wrote: > extern const struct cpumask *const cpu_possible_mask; > extern const struct cpumask *const cpu_online_mask; > +extern const struct cpumask *const cpu_online_stable_mask; > extern const struct cpumask *const cpu_present_mask; > extern const struct cpumask *const cpu_active_mask;
This is a bit nasty. The distinction between cpu_online_mask and the stable one is quite subtle and there's no mechanism to verify the right one is in use. IIUC, the only time cpu_online_mask and cpu_online_stable_mask can deviate is during the final stage CPU take down, right? If so, why not just make cpu_online_mask the stable one and the few cases where the actual online state matters deal with the internal version? Anyone outside cpu hotplug proper should be happy with the stable version anyway, no?
Thanks.
-- tejun
| |