Re: [PATCH] cpu hotplug, sched: Introduce cpu_active_map and redoscheddomainmanagment (take 2)

[Peter Zijlstra <a.p.zijlstra@chello.nl>]

On Thu, 2008-07-17 at 13:46 -0600, Gregory Haskins wrote:
> >>> On Thu, Jul 17, 2008 at  2:52 PM, in message <487F9509.9050802@qualcomm..com>,
> Max Krasnyansky <maxk@qualcomm.com> wrote: 
> > Gregory Haskins wrote:
> >>
> >> Hi Max,
> >>   Thanks for the pointers.  I see that I did indeed misunderstand the intent 
> >> of the patch.  It seems you already solved the rebuild problem, and were
> >> just trying to solve the "migrate to a dead cpu" problem that Linus mentions
> >> as a solution with cpu_active_map.
> >
> > Yes. btw they are definitely related, because the reason we were blowing 
> > away the domains is to avoid "migration to a dead cpu". ie We were relying
> > on the fact that domain masks never contain cpus that are either dying or
> > already dead.
> 
> Agreed.
> 
> >> 
> >> Thoughts?
> >
> > None at this point :). I need to run right now and will try to look at this
> > later today. My knowledge of the internal sched structs is definitely 
> > lacking so I need to look at the rq->rd thing to have and opinion.
> 
> Sounds good, Max.  Thanks!

I'm thinking doing it explicitly with the new cpu mask is clearer and
easier to understand than 'hiding' the variable in the root domain and
having to understand all the domain/root-domain stuff.

I think this was Linus' main point. It should be easier to understand
this code.


So, if there is functional overlap with the root domain stuff, it might
be good to remove that bit and use the cpu_active_map stuff for that
instead.