All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <Ian.Campbell@citrix.com>
To: Jan Beulich <JBeulich@suse.com>
Cc: Keir Fraser <keir@xen.org>,
	Andrew Cooper <andrew.cooper3@citrix.com>,
	Ian Jackson <Ian.Jackson@eu.citrix.com>, Tim Deegan <tim@xen.org>,
	Xen-devel <xen-devel@lists.xen.org>
Subject: Re: [PATCH v2 for-4.5] xen: Bump Xen interface for Xen-4.5
Date: Tue, 4 Nov 2014 12:08:50 +0000	[thread overview]
Message-ID: <1415102930.11486.40.camel@citrix.com> (raw)
In-Reply-To: <5458CDF70200007800044C30@mail.emea.novell.com>

On Tue, 2014-11-04 at 12:00 +0000, Jan Beulich wrote:
> >>> On 04.11.14 at 12:50, <andrew.cooper3@citrix.com> wrote:
> > c/s fce5281c "x86/mem_access: Deprecate the HVM mem_access ops" removes the
> > structures associated with xen_hvm_{get,set}_mem_access from the Xen public
> > API.
> > 
> > While these were toolstack hypercalls and documented as liable to change in
> > the future, it causes build issues for certain tools (valgrind, strace).
> > 
> > As HVM ops have no specific interface version, the main Xen interface 
> > version
> > needs to be bumped to compensate.
> 
> Content-wise I don't really object to this patch, but I view it as
> merely cosmetic rather than fixing anything: Tool stack interfaces
> are declared to be volatile just because we want to avoid exactly
> this need for bumping versions or anything when altering or
> dropping them. If there are out of tree consumers of them, it is
> their responsibility to keep up with our changes (or have their
> own clones of the canonical headers).
> 
> Also we didn't bother incrementing the version just because of a
> release on earlier occasions: 3.3 and 3.4 as well as 4.0 and 4.1
> shared interface versions, yet especially in the case of 4.1 I'm
> pretty certain even without explicitly checking that there were
> tool stack interface changes.

I always thought __XEN_(LATEST_)INTERFACE_VERSION__ were more to do with
API rather than ABI, i.e. it gets used to revert
__HYPERVISOR_sched_op_compat into providing __HYPERVISOR_sched_op for
older consumers and things like that.

Ian.

  reply	other threads:[~2014-11-04 12:08 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-04 11:44 [PATCH for-4.5] xen: Bump several interface versions in preparation for Xen-4.5 Andrew Cooper
2014-11-04 11:46 ` Andrew Cooper
2014-11-04 11:50   ` [PATCH v2 for-4.5] xen: Bump Xen interface " Andrew Cooper
2014-11-04 12:00     ` Jan Beulich
2014-11-04 12:08       ` Ian Campbell [this message]
2014-11-04 12:24         ` Jan Beulich
2014-11-04 12:07     ` Ian Campbell

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1415102930.11486.40.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=JBeulich@suse.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=keir@xen.org \
    --cc=tim@xen.org \
    --cc=xen-devel@lists.xen.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.