From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v2 for 4.5] xl: Return proper error codes for block-attach and block-detach Date: Thu, 20 Nov 2014 15:56:18 +0000 Message-ID: <1416498978.14429.32.camel@citrix.com> References: <1415905446-32168-1-git-send-email-george.dunlap@eu.citrix.com> <1415963574.7113.7.camel@citrix.com> <5469EBE5.4070209@eu.citrix.com> <1416498424.14429.30.camel@citrix.com> <546E0E0E.4050102@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <546E0E0E.4050102@eu.citrix.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: George Dunlap Cc: Ian Jackson , Wei Liu , xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Thu, 2014-11-20 at 15:51 +0000, George Dunlap wrote: > On 11/20/2014 03:47 PM, Ian Campbell wrote: > > On Mon, 2014-11-17 at 12:36 +0000, George Dunlap wrote: > >> On 11/14/2014 11:12 AM, Ian Campbell wrote: > >>> On Thu, 2014-11-13 at 19:04 +0000, George Dunlap wrote: > >>>> Return proper error codes on failure so that scripts can tell > >>>> whether the command completed properly or not. > >>>> > >>>> Also while we're at it, normalize init and dispose of > >>>> libxl_device_disk structures. This means calling init and > >>>> dispose in the actual functions where they are declared. > >>>> > >>>> This in turn means changing parse_disk_config_multistring() > >>>> to not call libxl_device_disk_init. And that in turn means > >>>> changes to callers of parse_disk_config(). > >>>> > >>>> It also means removing libxl_device_disk_init() from > >>>> libxl.c:libxl_vdev_to_device_disk(). This is only called > >>>> from xl_cmdimpl.c. > >>>> > >>>> Signed-off-by: George Dunlap > >>>> --- CC: Ian Campbell CC: Ian > >>>> Jackson CC: Wei Liu > >>>> CC: Konrad Wilk > >>>> > >>>> > >>>> Release justification: This is a bug fix. It's a fairly > >>>> minor one, but it's also a very simple one. > >>>> > >>>> v2: - Restructure functions to make sure init and dispose are > >>>> properly called. > >>> Sadly this bit has somewhat reduced the truth of the second > >>> half of your release justification, since the patch is a fair > >>> bit more subtle though. Although IMHO it hasn't invalidated the > >>> case for taking the patch for 4.5 (modulo comments below). > >> > >> Well, I think we can take the hacky short-term fix I posted > >> before, which is simple, and do a proper fix after the 4.6 dev > >> window opens up; or we can do a more complete fix now. > > > > Specifically the "hacky short-term fix" is > > <1415813493-25330-1-git-send-email-george.dunlap@eu.citrix.com> ? > > Yes -- the one you found somewhat wanting. :-) > > > I could live with that, perhaps with the commit log explaining that > > a little. > > Do you want to add that, or shall I add it and re-submit? If you provide the text I'll just fold it in, unless having written it you find invoking send-email to be so trivial it's actually easier. Ian.