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

Reply via email to