All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ryan Harper <ryanh@us.ibm.com>
To: Keir Fraser <Keir.Fraser@cl.cam.ac.uk>
Cc: xen-devel@lists.xensource.com
Subject: Re: [PATCH] tools: convert cpumap bitmap to list
Date: Thu, 20 Oct 2005 11:23:42 -0500	[thread overview]
Message-ID: <20051020162342.GE2004@us.ibm.com> (raw)
In-Reply-To: <d3d3e7baeb58004b381f58c0c87e9e02@cl.cam.ac.uk>

* Keir Fraser <Keir.Fraser@cl.cam.ac.uk> [2005-10-20 11:00]:
> 
> On 20 Oct 2005, at 16:24, Ryan Harper wrote:
> 
> >Are you interested in any modification of the cpumap to list 
> >conversion?
> >Or is it ok if vcpu-list shows a long list of integers?
> 
> I think an ordered list of cpus is most user-friendly, perhaps using 
> range notation to keep it short (so, for example, mask 0x77 would print 
> as '0-2, 4-6' rather than '0, 1, 2, 4, 5, 6'). A mask is a more 
> succinct representation but maybe not so good for users. :-)

Agreed.  I'll work up something that tries to shrink the list into
something smaller but still readable.

> 
> >I would think mapping cpu % nr_cpus, throwing away duplicates would be
> >best.  With [0,1,2,3] and nr_cpus=2, cpumap would be [0,1].
> 
> I think Xen should ignore non-existent cpus in a cpumap. I'm not really 
> fussed what you do with them in xend -- ignoring them there too would 
> probably make most sense.

I am a little concerned that if a user pins a vcpu to cpus 0-7 , on a
two-way, and the output of vcpu-list shows him [0,1], that it might look
like the pin operation didn't work.  If this isn't a big deal, then I'll
also look at throwing out invalid cpus from the cpumap representation.

-- 
Ryan Harper
Software Engineer; Linux Technology Center
IBM Corp., Austin, Tx
(512) 838-9253   T/L: 678-9253
ryanh@us.ibm.com

  reply	other threads:[~2005-10-20 16:23 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-19 22:45 [PATCH] tools: convert cpumap bitmap to list Ryan Harper
2005-10-20 10:44 ` Keir Fraser
2005-10-20 15:24   ` Ryan Harper
2005-10-20 16:03     ` Keir Fraser
2005-10-20 16:23       ` Ryan Harper [this message]
2005-10-20 20:10         ` Keir Fraser
2005-10-21 14:03           ` Ryan Harper
2005-10-21 15:04             ` Keir Fraser
2005-10-21 15:02               ` Ryan Harper
2005-10-21 16:59               ` [PATCH] build cpumap from list of ints Ryan Harper
2005-10-21 17:52               ` 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=20051020162342.GE2004@us.ibm.com \
    --to=ryanh@us.ibm.com \
    --cc=Keir.Fraser@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.