From mboxrd@z Thu Jan 1 00:00:00 1970 From: George Dunlap Subject: Re: [PATCH for xen-4.4] libxl: add option for discard support to xl disk configuration Date: Wed, 5 Feb 2014 14:28:01 +0000 Message-ID: <52F24A71.7080102@eu.citrix.com> References: <1391083364-29483-1-git-send-email-olaf@aepfle.de> <20140130162558.GA9033@aepfle.de> <1391099505.9495.23.camel@kazak.uk.xensource.com> <20140130173058.GA12133@aepfle.de> <1391530957.6497.56.camel@kazak.uk.xensource.com> <52F21C29.8090607@eu.citrix.com> <20140205113623.GA24025@aepfle.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20140205113623.GA24025@aepfle.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Olaf Hering Cc: anthony.perard@citrix.com, xen-devel@lists.xen.org, Ian.Jackson@eu.citrix.com, Ian Campbell , stefano.stabellini@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On 02/05/2014 11:36 AM, Olaf Hering wrote: > On Wed, Feb 05, George Dunlap wrote: > >> Well it looks like in order to keep ABI compatibility (which I don't think >> we ever promised), you're introducing this weird hack with overloading a >> putative boolean value with a magic number? > Yes, thats the point. If libxl_device_disk changes then IMO the SONAME > has to change as well. And is 4.4-rc4 is the right time to do that? > Likely not. I'm fine with carry the 4.4 patch to achieve the result, and > put the other version into 4.5. Right, and so this exposes another risk of accepting a patch so late: that of making poor interface decisions in a rush which either libxl or programs written against it have to deal with for a long time. (i.e., either we have to keep supporting the old interface, or the application developer has to special case the 4.4 interface if they want to be able to compile against it). -George