On 14/11/2018 18:17, Wei Liu wrote: > If that's set, the stub is going to contain that UUID. > > No functional change. > > Signed-off-by: Wei Liu <wei.l...@citrix.com> > --- > tools/helpers/init-dom-json.c | 5 ++++- > tools/helpers/init-dom-json.h | 3 ++- > tools/helpers/init-xenstore-domain.c | 2 +- > tools/helpers/xen-init-dom0.c | 2 +- > 4 files changed, 8 insertions(+), 4 deletions(-) > > diff --git a/tools/helpers/init-dom-json.c b/tools/helpers/init-dom-json.c > index 704e7cb4f0..9514b3ceb6 100644 > --- a/tools/helpers/init-dom-json.c > +++ b/tools/helpers/init-dom-json.c > @@ -7,7 +7,7 @@ > #include <xenctrl.h> > #include <libxl.h> > > -int gen_stub_json_config(uint32_t domid) > +int gen_stub_json_config(uint32_t domid, libxl_uuid *uuid) > { > int rc = 1; > xentoollog_logger_stdiostream *logger; > @@ -40,6 +40,9 @@ int gen_stub_json_config(uint32_t domid) > libxl_domain_build_info_init_type(&dom_config.b_info, > dom_config.c_info.type); > > + if (uuid && !libxl_uuid_is_nil(uuid)) > + libxl_uuid_copy(ctx, &dom_config.c_info.uuid, uuid);
Is the nil check necessary, given that nil is the default value for dom_config.c_info.uuid ? Either way, Acked-by: Andrew Cooper <andrew.coop...@citrix.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel