[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v4 01/31] libxl: fix libxl__build_hvm error handling

On Fri, Aug 07, 2015 at 12:55:21PM +0200, Roger Pau Monné wrote:
> El 07/08/15 a les 12.49, Wei Liu ha escrit:
> > On Fri, Aug 07, 2015 at 12:17:38PM +0200, Roger Pau Monne wrote:
> >> With the current code in libxl__build_hvm it is possible for the function 
> >> to
> >> fail and still return 0.

I care about this bit, which states clearly there is a bug that needs

> > 
> > It's hard to see where the bug is when this patch also does a bunch of
> > refactoring.
> It refactors the error paths only, mainly replacing:
> if (libxl_call_foo(bar))
>     <error>
> to
> rc = libxl_call_foo(bar)
> if (rc != 0)
>     <error>

But this suggests there is no bug? 

> So we can keep the error codes returned by auxiliary functions.
> > It would be good if you can separate the bug fix from other name
> > changing bits, so that we can apply that bug fix for 4.6 possibly queue
> > it up for backporting.
> There are no name changing bits AFAICT.

Changing ret for rc is naming changing to me. It's a good thing to do to
comply with coding style, but mixing this with bug fix makes it hard to
backport the fix itself.


> Roger.

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.