On Thu, 26 Feb 2015, Wei Liu wrote: > On Thu, Feb 26, 2015 at 06:53:29PM +0000, Stefano Stabellini wrote: > [...] > > > > } > > > > > > > > - libxl_dominfo_init(&ptr); > > > > - xcinfo2xlinfo(ctx, &info, &ptr); > > > > > > If I'm not mistaken, &info is only used here. I think you can delete > > > info and relevant code all together. > > > > info is used later as an argument to xc_domain_getinfolist > > > > > > What I meant was, the sole purpose of info and two function calls > xc_domain_getinfolist + xcinfo2xlinfo is to fill in ptr, which is done > by a single call to libxl_domain_info at the beginning of your patch, > so it's possible to remove info and those two function calls all > together.
You are right! I'll resend. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
