WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [Patch] use full-size cpumask for vcpu-pin

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [Patch] use full-size cpumask for vcpu-pin
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Fri, 13 Aug 2010 14:25:30 +0100
Cc: Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 13 Aug 2010 06:27:46 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <C88B0585.1DB52%keir.fraser@xxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: Acs66P6ihsUYSx3hTHqLrb2bGAI6HQAAKSSBAAA3bqQAAB++rg==
Thread-topic: [Xen-devel] [Patch] use full-size cpumask for vcpu-pin
User-agent: Microsoft-Entourage/12.24.0.100205
On 13/08/2010 14:21, "Keir Fraser" <keir.fraser@xxxxxxxxxxxxx> wrote:

>> Since Xen discards any CPUs which are not online at the time of the
>> setvcpuaffinity hypercall, expressing "all CPUs present and future" is a bit
>> pointless.
> 
> Hm, no, I'm talking rubbish. Argh. Okay, his patch is fine then. :-D
> 
> You can check it in to xen-unstable-tools.hg.

One suggestion: that we rename the syctl.physinfo field 'max_phys_cpus' to
'max_possible_cpus'. The 'phys' is kind of redundant since this is the
physinfo sysctl, and 'possible' provides a better hint that this fields
indicates maximum possible supported CPUs now and forever on this boot of
the system.

 -- Keir



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