From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH 8/9] xen/arch/x86: use is_hardware_domain
 instead of domid == 0
Date: Fri, 12 Apr 2013 17:50:24 +0100
Message-ID: <51683B50.1070408@citrix.com>
References: <CD8DE9A2.20C73%keir.xen@gmail.com>
	<1365781784.15783.106.camel@zakaz.uk.xensource.com>
	<51683AE1.4090503@citrix.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <51683AE1.4090503@citrix.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: Daniel De Graaf <dgdegra@tycho.nsa.gov>, Keir Fraser <keir.xen@gmail.com>, Jan Beulich <JBeulich@suse.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On 12/04/13 17:48, Andrew Cooper wrote:
> On 12/04/13 16:49, Ian Campbell wrote:
>> On Fri, 2013-04-12 at 16:41 +0100, Keir Fraser wrote:
>>> We could get rid of this not-always-faulting-cpuid hack in the
>>> hypervisor altogether, if we could provide a paravirtualised interface
>>> that guarantees access to the real CPUID space, for the toolstack to
>>> use. 
>> Doesn't XCP have a patch which allows access to the non-levelled cpuid
>> values for the toolstacks benefit? Andy C CCd...
>>
>>
> XCP/XenServer has a specific patch which records the real CPUID values
> before applying the MSRs for feature masking, which can then subsequently
queried by the toolstack.

(Apologies - I sent mid-sentence)

~Andrew

>
> This allows the toolstack to recalculate the host level parity bits
> correctly whenever a host is added or removed from the pool, without
> suffering the problems of getting into a diminishing subset of features
> as slightly heterogeneous hardware is moved in and out of the pool.
>
> The patch is currently in the state of "Needing to be cleaned up for
> upstream" and is in my todo list (albeit quite far down given some
> recent Xen crashes found during stress testing)
>
> I can up its priority if there is a need for it.
>
> ~Andrew
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel