Re: [Xen-devel] [PATCH v3 2/5] libxl: make GC_FREE reachable in libxl_get_scheduler()

[czylin@xxxxxxxxxxxx]

Quoting Ian Campbell <ian.campbell@xxxxxxxxxx>:

> On Mon, 2016-01-25 at 11:59 +0000, Ian Campbell wrote:
> > On Sun, 2016-01-24 at 19:45 -0500, Chester Lin wrote:
> > > Coverity CID 1343309
> > >
> > > Make GC_FREE reachable in all cases in libxl_get_scheduler() by
> > > eliminating the error-path return and instead storing the error code in
> > > the returned variable.
> > >
> > > To make this semantically consistent, change the return type of
> > > libxl_get_scheduler() from libxl_scheduler to int, and make a note of
> > > the interpretation of the return value in libxl.h.  N.B. This change
> > > does not change the API in a way that affects functionality.
> > >
> > > The libxl_scheduler enum is consistent with the sched_id return value
> > > of xc_sched_id and this must continue to be true.
> > >
> > > Suggested-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > > Signed-off-by: Chester Lin <czylin@xxxxxxxxxxxx>
> > > Reviewed-by: Dario Faggioli <dario.faggioli@xxxxxxxxxx>
> >
> > Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> Applied.
>
> > I can't see any sign of v3 versions of #3, #4 or #5/5 in my INBOX, have
> > they gone astray?
> >
> > Also, both #1 and #2 seem to have been sent independently using separate
> > git send-email invocations. Please send them from one single invocation, as
> > reply to a cover letter (which may be referenced with --in-reply-to, or
> > written using --compose) such that they are all properly threaded together
> > into a message thread.
> I've retired the previous #3..#5 out of my queue (without applying
> anything) in anticipation of a resend iff they are still needed.
>
> Ian.
I'm sorry for any confusion. I didn't send patch #3 because I thought  
there was agreement that it would be better to disable the check in  
the tool than make any code changes.
We also do not intend to follow up with the IDL refactoring as per  
your earlier comments. If you would like us to resend any of the  
patches for further debate or archiving, I will.
On behalf of Josh, Harley, and myself, I'd like to thank the  
maintainers for reviewing our patches.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel