[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH RFC] tools: Drop xc_cpuid_check() and bindings



On Mon, Jul 17, 2017 at 01:38:03PM +0100, Andrew Cooper wrote:
> There are no current users which I can locate.  One piece of xend which didn't
> move forwards into xl/libxl is this:
> 
>   #   Configure host CPUID consistency checks, which must be satisfied for 
> this
>   #   VM to be allowed to run on this host's processor type:
>   #cpuid_check=[ '1:ecx=xxxxxxxxxxxxxxxxxxxxxxxxxx1xxxxx' ]
>   # - Host must have VMX feature flag set
> 
> The implementation of xc_cpuid_check() is conceptually broken.  Dom0's view of
> CPUID is not the approprite view to check, and will be wrong in the presence
> of CPUID masking/faulting, and for HVM-based toolstack domains.
> 
> If it turns out that the functionality is required, it should be implemented
> in terms of XEN_SYSCTL_get_cpuid_policy to use the proper CPUID view.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> ---
> CC: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
> CC: Wei Liu <wei.liu2@xxxxxxxxxx>
> CC: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>
> CC: David Scott <dave@xxxxxxxxxx>
> CC: Christian Lindig <christian.lindig@xxxxxxxxxx>
> CC: Juergen Gross <jgross@xxxxxxxx>
> CC: Jim Fehlig <jfehlig@xxxxxxxx>
> CC: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
> CC: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> 
> RFC initially for feedback, and to see if anyone does expect to be using this
> call.  It turns out that Xapi has a library function using it, but that
> function is dead so can be removed.

FAOD I am still waiting for Oracle and Suse folks to express their
opinions.

I will wait until August 4th. If I don't hear objection by then I will
just apply this patch.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.