On Thu, Nov 17, 2022 at 01:08:03AM +0000, Andrew Cooper wrote:
> diff --git a/tools/libs/light/libxl_dom.c b/tools/libs/light/libxl_dom.c
> index 2abaab439c4f..f8f7b7e81837 100644
> --- a/tools/libs/light/libxl_dom.c
> +++ b/tools/libs/light/libxl_dom.c
> @@ -1448,6 +1448,25 @@ int libxl_userdata_unlink(libxl_ctx *ctx, uint32_t
> domid,
> return rc;
> }
>
> +int libxl__domain_set_p2m_pool_size(
> + libxl__gc *gc, libxl_domain_config *d_config, uint32_t domid)
> +{
> + libxl_ctx *ctx = libxl__gc_owner(gc);
> + uint64_t shadow_mem;
> +
> + shadow_mem = d_config->b_info.shadow_memkb;
> + shadow_mem <<= 10;
> +
> + int r = xc_set_paging_mempool_size(ctx->xch, domid, shadow_mem);
Nit: 'ctx' could be replace by using the macro 'CTX', without having to
declare a local ctx variable.
With or without this changed, patch looks fine:
Reviewed-by: Anthony PERARD <[email protected]>
Thanks,
--
Anthony PERARD
