Re: [Xen-devel] [PATCH v3 50/52] libxl: add libxl_set_parameters() function

[Wei Liu <wei.liu2@xxxxxxxxxx>]

On Wed, Aug 16, 2017 at 02:52:17PM +0200, Juergen Gross wrote:
> Add a new libxl function to set hypervisor parameters at runtime
> similar to boot time parameters via command line.
> 
> Cc: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> Cc: Wei Liu <wei.liu2@xxxxxxxxxx>
> Signed-off-by: Juergen Gross <jgross@xxxxxxxx>

Acked-by: Wei Liu <wei.liu2@xxxxxxxxxx>


> ---
> V2:
> - corrected coding style (Wei Liu)
> - removed superfluous #ifdef (Wei Liu)
> 
> V3:
> - use LOGEV() for error message
> ---
>  tools/libxl/libxl.c | 15 +++++++++++++++
>  tools/libxl/libxl.h |  8 ++++++++
>  2 files changed, 23 insertions(+)
> 
> diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c
> index 0ef874406f..247c56cf83 100644
> --- a/tools/libxl/libxl.c
> +++ b/tools/libxl/libxl.c
> @@ -652,6 +652,21 @@ int libxl_send_debug_keys(libxl_ctx *ctx, char *keys)
>      return 0;
>  }
>  
> +int libxl_set_parameters(libxl_ctx *ctx, char *params)
> +{
> +    int ret;
> +    GC_INIT(ctx);
> +
> +    ret = xc_set_parameters(ctx->xch, params);
> +    if (ret < 0) {
> +        LOGEV(ERROR, ret, "setting parameters");
> +        GC_FREE;
> +        return ERROR_FAIL;
> +    }

In case you repost, can you add a blank line here? Thanks.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel