[libvirt] [PATCH 05/12] libxl: libxl: Use per-domain ctx in libxlMakeDomCreateInfo
Michal Privoznik
mprivozn at redhat.com
Mon Sep 2 11:08:26 UTC 2013
On 30.08.2013 23:46, Jim Fehlig wrote:
> libxlMakeDomCreateInfo() uses the driver-wide libxl ctx when
> it would be more appropriate to use the per-domain ctx
> associated with the domain. Switch to using the per-domain
> libxl ctx.
>
> Signed-off-by: Jim Fehlig <jfehlig at suse.com>
> ---
> src/libxl/libxl_conf.c | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/src/libxl/libxl_conf.c b/src/libxl/libxl_conf.c
> index 81b4af4..231a53d 100644
> --- a/src/libxl/libxl_conf.c
> +++ b/src/libxl/libxl_conf.c
> @@ -395,7 +395,7 @@ libxlCapsInitGuests(libxl_ctx *ctx, virCapsPtr caps)
> }
>
> static int
> -libxlMakeDomCreateInfo(libxlDriverPrivatePtr driver,
> +libxlMakeDomCreateInfo(libxl_ctx *ctx,
> virDomainDefPtr def,
> libxl_domain_create_info *c_info)
> {
> @@ -413,7 +413,7 @@ libxlMakeDomCreateInfo(libxlDriverPrivatePtr driver,
>
> if (def->nseclabels &&
> def->seclabels[0]->type == VIR_DOMAIN_SECLABEL_STATIC) {
> - if (libxl_flask_context_to_sid(driver->ctx,
> + if (libxl_flask_context_to_sid(ctx,
> def->seclabels[0]->label,
> strlen(def->seclabels[0]->label),
> &c_info->ssidref)) {
> @@ -1024,10 +1024,11 @@ libxlBuildDomainConfig(libxlDriverPrivatePtr driver,
> virDomainObjPtr vm, libxl_domain_config *d_config)
> {
> virDomainDefPtr def = vm->def;
> + libxlDomainObjPrivatePtr priv = vm->privateData;
>
> libxl_domain_config_init(d_config);
>
> - if (libxlMakeDomCreateInfo(driver, def, &d_config->c_info) < 0)
> + if (libxlMakeDomCreateInfo(priv->ctx, def, &d_config->c_info) < 0)
> return -1;
>
> if (libxlMakeDomBuildInfo(vm, d_config) < 0)
>
ACK
Michal
More information about the libvir-list
mailing list