Re: [Xen-devel] [PATCH 4 of 4] Support new xl command cpupool-numa-split

[Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>]

On 12/08/10 14:38, Ian Campbell wrote:
> On Wed, 2010-12-08 at 13:17 +0000, Juergen Gross wrote:
> > On 12/08/10 14:12, Ian Campbell wrote:
> > > On Wed, 2010-12-08 at 12:20 +0000, Juergen Gross wrote:
> > > > On 12/08/10 12:16, Ian Campbell wrote:
> > > > > Can this loop be merged with the preceding loop, with the body being the
> > > > > else case of the if?
> > > > No. I have to add new cpus first to avoid a cpupool without cpus in between.
> > > ok.
> > >
> > > I was thinking that because this function only gets here if there is a
> > > single pool that all CPUs must be in that pool -- but that's not
> > > actually true is it? Even if that were the common case there's nothing
> > > to enforce that.
> > Perhaps I should add a comment to avoid a problem later...
> That would certainly help.
>
> The alternative would be to bail out if all cpus are not associated with
> Pool-0, not just when there are>  1 pools. That would be consistent with
> the function only acting on the default configuration.
I suspect NUMA systems are subject to cpu hot plug...

I'd prefer the way I've done it before. It isn't too complex and more
flexible.


Juergen

--
Juergen Gross                 Principal Developer Operating Systems
TSP ES&S SWE OS6                       Telephone: +49 (0) 89 3222 2967
Fujitsu Technology Solutions              e-mail: juergen.gross@xxxxxxxxxxxxxx
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel