From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH RFC 2/2] xen/x86: Introduce XEN_SYSCTL_cpuid hypercall Date: Thu, 27 Feb 2014 15:57:22 +0000 Message-ID: <530F6062.2040302@citrix.com> References: <1393499497-9162-1-git-send-email-andrew.cooper3@citrix.com> <1393499497-9162-4-git-send-email-andrew.cooper3@citrix.com> <530F365B020000780011FD03@nat28.tlf.novell.com> <530F2B85.6060403@citrix.com> <530F3CF8020000780011FD40@nat28.tlf.novell.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <530F3CF8020000780011FD40@nat28.tlf.novell.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Jan Beulich Cc: Keir Fraser , Tim Deegan , Ian Jackson , Ian Campbell , Xen-devel List-Id: xen-devel@lists.xenproject.org On 27/02/14 12:26, Jan Beulich wrote: >>>> On 27.02.14 at 13:11, Andrew Cooper wrote: >> Apart from forcibly messing with a balanced numa setup, what about cpu >> pools, or toolstack disaggregation where pinning is restricted? > There no messing with a balanced setup here - everything should > of course be transient, i.e. get restored to previous values right > after. But anything else happening on that vcpu at the same time is transiently out of balance. > And I can't see a reason not to permit the hardware domain > to temporarily escape its enclave, as it can do worse to the overall > system anyway. True > > The new hypercall is simple enough (yet very x86-centric) that I'm > not really against it; what I'm against is adding functionality to the > hypervisor that is already available by other means. > > Jan > As I said before, the cache information is faked up by the cpuid policy, so might be subject to policy depending on faulting or non-dom0 hardware domian, or PVH dom0 in the near future. I think there is a legitimate case for "I really truely need the real hardware values, without anything in the policy getting in the way". 'cpuid' is indeed very x86 specific, but it is not information readily available by other means. ~Andrew