On Fri, Oct 23, 2015 at 5:40 PM, Dario Faggioli <dario.faggi...@citrix.com>
wrote:
> > int main_cpupooldestroy(int argc, char **argv)
> > @@ -7580,13 +7580,13 @@ int main_cpupooldestroy(int argc, char
> > **argv)
> > if (libxl_cpupool_qualifier_to_cpupoolid(ctx, pool, &poolid,
> > NULL) ||
> > !libxl_cpupoolid_is_valid(ctx, poolid)) {
> > fprintf(stderr, "unknown cpupool '%s'\n", pool);
> > - return 1;
> > + return EXIT_FAILURE;
> > }
> >
> > if (libxl_cpupool_destroy(ctx, poolid))
> > - return 1;
> > + return EXIT_FAILURE;
> >
> > - return 0;
> > + return EXIT_SUCCESS;
> > }
> >
> For this one: I've sent a patch for another reason yesterday, and while
> there I did the exit code adjustment myself. So, update your tree and,
> if my patch has been committed already, just skip this function.
>
> https://www.mail-archive.com/xen-devel@lists.xen.org/msg42850.html
>
> Which brings up a question: what git tree are you using for
> development? You should stay either on master or staging branches (and
> I recommend staging) of the official repository:
>
> http://wiki.xenproject.org/wiki/Xen_Project_Repositories
I was on master branch, now switching to staging.
Thank you so much for reviewing my patch and helping on this tight
timeline. And regarding your other questions (patch 1 and 2) I will be
answering as I digest all the information you just passed :)
Again, thank you Dario. I really appreciate your help.
Warmest Regards :)
Harman
> > @@ -7653,7 +7653,7 @@ int main_cpupoolcpuadd(int argc, char **argv)
> >
> > out:
> > libxl_bitmap_dispose(&cpumap);
> > - return rc;
> > + return rc ? EXIT_FAILURE : EXIT_SUCCESS;
> >
> Same as already said for main_cpupoolcreate, just us rc.
>
> > @@ -7691,7 +7691,7 @@ int main_cpupoolcpuremove(int argc, char
> > **argv)
> >
> > out:
> > libxl_bitmap_dispose(&cpumap);
> > - return rc;
> > + return rc ? EXIT_FAILURE : EXIT_SUCCESS;
> >
> And here.
>
> > int main_cpupoolnumasplit(int argc, char **argv)
> > @@ -7758,7 +7758,7 @@ int main_cpupoolnumasplit(int argc, char
> > **argv)
> > poolinfo = libxl_list_cpupool(ctx, &n_pools);
> > if (!poolinfo) {
> > fprintf(stderr, "error getting cpupool info\n");
> > - return 1;
> > + return EXIT_FAILURE;
> > }
> > poolid = poolinfo[0].poolid;
> > sched = poolinfo[0].sched;
> > @@ -7766,13 +7766,13 @@ int main_cpupoolnumasplit(int argc, char
> > **argv)
> >
> > if (n_pools > 1) {
> > fprintf(stderr, "splitting not possible, already cpupools in
> > use\n");
> > - return 1;
> > + return EXIT_FAILURE;
> > }
> >
> > topology = libxl_get_cpu_topology(ctx, &n_cpus);
> > if (topology == NULL) {
> > fprintf(stderr, "libxl_get_topologyinfo failed\n");
> > - return 1;
> > + return EXIT_FAILURE;
> > }
> >
> > if (libxl_cpu_bitmap_alloc(ctx, &cpumap, 0)) {
> > @@ -7869,7 +7869,7 @@ out:
> > libxl_dominfo_dispose(&info);
> > free(name);
> >
> > - return rc;
> > + return rc ? EXIT_FAILURE : EXIT_SUCCESS;
> > }
> >
> And here too.
>
> Thanks and regards,
> Dario
> --
> <<This happens because I choose it to happen!>> (Raistlin Majere)
> -----------------------------------------------------------------
> Dario Faggioli, Ph.D, http://about.me/dario.faggioli
> Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
>
>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
