From: Ryan Harper <ryanh@us.ibm.com>
To: Ian Pratt <m+Ian.Pratt@cl.cam.ac.uk>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH] xend: update cpu config option
Date: Wed, 16 Nov 2005 16:30:36 -0600 [thread overview]
Message-ID: <20051116223036.GD15015@us.ibm.com> (raw)
In-Reply-To: <A95E2296287EAD4EB592B5DEEFCE0E9D32EA27@liverpoolst.ad.cl.cam.ac.uk>
* Ian Pratt <m+Ian.Pratt@cl.cam.ac.uk> [2005-11-16 16:07]:
> > The 'cpu' option in domain config files will pin VCPU0 of a domain.
> > This is not as useful now that domains can have more than 1
> > vcpu. This patch changes 'cpu' to 'cpus' and takes a list of
> > physical cpus the domains' vcpus can use and will pin the
> > vcpus upon domain creation.
> >
> > cpus = [1] # this starts all domain vcpus pinned to CPU1
>
> I think this patch is generally a good thing. Should we support cpu= as
> backward compatible legacy option?
I pondered that as well. No reason we can't. Some documentation should
avoid any confusion. I'll rework the patch to support cpu=X and
cpus=[].
>
> > The list is circular, so in a domain with the following config:
> >
> > vcpus = 4
> > cpus = [0,3,7] # Use any of 0, 3, 7 for this domain.
> >
> > would see vcpus 0-3 pinned to cpus 0,3,7,0 respectively.
>
> Actually, although this is a reasonable syntax, I think we'll probably
> interpret it differently in future when we have CPU load ballancing: I
> think we'll want to list the set of CPUs that a given _domain_ can use
> rather than pining individual CPUs.
I think the list is already representative of that idea: this is a list
of cpus that any of the vcpus in this domain can use. Currently without
a load balancer we only get one go at vcpu to cpu allocation.
Also, no reason we can't replace the current algorithm down the road
with a call out to the load balancer which would supply the mappings.
> However, I wander whether this should be a string so that we can list
> e.g. cpus='0-3,5,^1'
I like that notation better, but it opens up a few questions. Do you
mean the commas to indicate which cpus each vcpu is allowed to use, or
just a list of cpus the domain can use? Also, I take ^1 to mean any
cpu, yes?
I'll end up converting the string into a big list of possible cpus to
use and for each vcpu pick a cpu and pin it there.
--
Ryan Harper
Software Engineer; Linux Technology Center
IBM Corp., Austin, Tx
(512) 838-9253 T/L: 678-9253
ryanh@us.ibm.com
next prev parent reply other threads:[~2005-11-16 22:30 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-11-16 22:06 [PATCH] xend: update cpu config option Ian Pratt
2005-11-16 22:30 ` Ryan Harper [this message]
-- strict thread matches above, loose matches on Subject: below --
2005-11-16 22:59 Ian Pratt
2005-11-16 21:42 Ryan Harper
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20051116223036.GD15015@us.ibm.com \
--to=ryanh@us.ibm.com \
--cc=m+Ian.Pratt@cl.cam.ac.uk \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.