From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH 3/4] x86/domctl: Remove PV MSR parts of
	XEN_DOMCTL_[gs]et_ext_vcpucontext
Date: Thu, 5 Jun 2014 10:25:29 +0100
Message-ID: <53903789.5090300@citrix.com>
References: <1401902794-15542-1-git-send-email-andrew.cooper3@citrix.com>
	<1401902794-15542-4-git-send-email-andrew.cooper3@citrix.com>
	<1401954765.8669.1.camel@hamster.uk.xensource.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: <1401954765.8669.1.camel@hamster.uk.xensource.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: Frediano Ziglio <frediano.ziglio@citrix.com>
Cc: Keir Fraser <keir@xen.org>, Jan Beulich <JBeulich@suse.com>, Xen-devel <xen-devel@lists.xen.org>
List-Id: xen-devel@lists.xenproject.org

On 05/06/14 08:52, Frediano Ziglio wrote:
> On Wed, 2014-06-04 at 18:26 +0100, Andrew Cooper wrote:
>> diff --git a/xen/include/public/domctl.h b/xen/include/public/domctl.h
>> index 7a13e25..4321a64 100644
>> --- a/xen/include/public/domctl.h
>> +++ b/xen/include/public/domctl.h
>> @@ -543,16 +543,6 @@ typedef struct xen_domctl_pin_mem_cacheattr xen_domctl_pin_mem_cacheattr_t;
>>  DEFINE_XEN_GUEST_HANDLE(xen_domctl_pin_mem_cacheattr_t);
>>  
>>
>> -#if defined(__i386__) || defined(__x86_64__)
>> -struct xen_domctl_ext_vcpu_msr {
>> -    uint32_t         index;
>> -    uint32_t         reserved;
>> -    uint64_aligned_t value;
>> -};
>> -typedef struct xen_domctl_ext_vcpu_msr xen_domctl_ext_vcpu_msr_t;
>> -DEFINE_XEN_GUEST_HANDLE(xen_domctl_ext_vcpu_msr_t);
>> -#endif
>> -
>>  /* XEN_DOMCTL_set_ext_vcpucontext */
>>  /* XEN_DOMCTL_get_ext_vcpucontext */
>>  struct xen_domctl_ext_vcpucontext {
>> @@ -572,14 +562,6 @@ struct xen_domctl_ext_vcpucontext {
>>      uint16_t         sysenter_callback_cs;
>>      uint8_t          syscall32_disables_events;
>>      uint8_t          sysenter_disables_events;
>> -    /*
>> -     * When, for the "get" version, msr_count is too small to cover all MSRs
>> -     * the hypervisor needs to be saved, the call will return -ENOBUFS and
>> -     * set msr_count to the required (minimum) value. Furthermore, for both
>> -     * "get" and "set", that field as well as the msrs one only get looked at
>> -     * if the size field above covers the structure up to the entire msrs one.
>> -     */
>> -    uint16_t         msr_count;
>>  #if defined(__GNUC__)
>>      union {
>>          uint64_aligned_t mcg_cap;
>> @@ -588,7 +570,6 @@ struct xen_domctl_ext_vcpucontext {
>>  #else
>>      struct hvm_vmce_vcpu vmce;
>>  #endif
>> -    XEN_GUEST_HANDLE_64(xen_domctl_ext_vcpu_msr_t) msrs;
>>  #endif
>>  };
>>  typedef struct xen_domctl_ext_vcpucontext xen_domctl_ext_vcpucontext_t;
> This is an ABI change, you should bump XEN_DOMCTL_INTERFACE_VERSION too
> (if not already done, if done there should be a comment in the header
> stating when is bumped).
>
> Frediano
>
>

The ABI version only needs bumping once per Xen release.  The ABI bump
post-4.4 was in the patch which I am partially reverting.

~Andrew