This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH] libxl: drop domid field from libxl_device_*

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libxl: drop domid field from libxl_device_*
From: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Date: Fri, 1 Apr 2011 11:29:59 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 01 Apr 2011 03:30:37 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19860.49831.975960.340177@xxxxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Organization: Citrix Systems, Inc.
References: <ca75b204dca4339dcdca.1300975847@xxxxxxxxxxxxxxxxxxxxxxxxx> <19860.49831.975960.340177@xxxxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Thu, 2011-03-31 at 19:06 +0100, Ian Jackson wrote:
> Ian Campbell writes ("[Xen-devel] [PATCH] libxl: drop domid field from 
> libxl_device_*"):
> > libxl: drop domid field from libxl_device_*
> I'd love to apply this but it gets lots of rejects from my current tip
> (23116:7f894a4cf702).

I'll repost once I've rebased.

> Also how backward-incompatible is it ?

It's API incompatible I guess but somewhat easy to fix since you can
just drop the initialiser in the caller since libxl always just
overwrote it anyway, IOW by dropping the initialiser you become
compatible with libxl both before and after this change.


Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>