Re: Cpupools and pdata_alloc

[Keir Fraser <keir.fraser@eu.citrix.com>]

On 11/05/2010 19:52, "George Dunlap" <George.Dunlap@eu.citrix.com> wrote:

> On Tue, May 11, 2010 at 1:41 PM, Keir Fraser <keir.fraser@eu.citrix.com>
> wrote:
>> Well, once secondary CPUs are fully up they are of course running their
>> respective idle VCPUs, so we can't call schedule_init() very late. The right
>> thing to do is dynamically allocate state as CPUs are brought online -- and
>> potentially free that state when CPUs are taken offline. Which is pretty
>> much the direction the code points in already.
> 
> Except that it's done a little indirectly, via initializing the idle
> vcpu, rather than calling something like sched_init_pcpu() when the
> cpu is brought up before creating an idle vcpu for it. :-)  Anyway,
> maybe once I get the credit2 stuff working again, I may think about
> writing a patch to clean it up a bit... mostly so that it's easier to
> follow what's going on.

Splitting it into an explicit separate call into the scheduling subsystem
would be acceptable, I think.

 -- Keir