[libvirt] [PATCH 1/2] parallels: move up updating parameter in prlsdkLoadDomain
Dmitry Guryanov
dguryanov at virtuozzo.com
Mon May 25 21:39:10 UTC 2015
On 05/21/2015 04:49 PM, Maxim Nestratov wrote:
> It is better to get all necessary parameters and check them on newly
> created configuration before actually creating a domain with them or
> applying them to an existing domain.
>
> Signed-off-by: Maxim Nestratov <mnestratov at parallels.com>
Thanks, ACKED and pushed.
> ---
> src/parallels/parallels_sdk.c | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> diff --git a/src/parallels/parallels_sdk.c b/src/parallels/parallels_sdk.c
> index 786e0ad..4d4582f 100644
> --- a/src/parallels/parallels_sdk.c
> +++ b/src/parallels/parallels_sdk.c
> @@ -1312,6 +1312,12 @@ prlsdkLoadDomain(parallelsConnPtr privconn,
> *s = '\0';
> }
>
> + pret = PrlVmCfg_GetAutoStart(sdkdom, &autostart);
> + prlsdkCheckRetGoto(pret, error);
> +
> + if (prlsdkGetDomainState(sdkdom, &domainState) < 0)
> + goto error;
> +
> if (virDomainDefAddImplicitControllers(def) < 0)
> goto error;
>
> @@ -1349,15 +1355,6 @@ prlsdkLoadDomain(parallelsConnPtr privconn,
> dom->privateDataFreeFunc = prlsdkDomObjFreePrivate;
> dom->persistent = 1;
>
> - if (prlsdkGetDomainState(sdkdom, &domainState) < 0)
> - goto error;
> -
> - if (prlsdkConvertDomainState(domainState, envId, dom) < 0)
> - goto error;
> -
> - pret = PrlVmCfg_GetAutoStart(sdkdom, &autostart);
> - prlsdkCheckRetGoto(pret, error);
> -
> switch (autostart) {
> case PAO_VM_START_ON_LOAD:
> dom->autostart = 1;
> @@ -1371,6 +1368,9 @@ prlsdkLoadDomain(parallelsConnPtr privconn,
> goto error;
> }
>
> + if (prlsdkConvertDomainState(domainState, envId, dom) < 0)
> + goto error;
> +
> if (!pdom->sdkdom) {
> pret = PrlHandle_AddRef(sdkdom);
> prlsdkCheckRetGoto(pret, error);
More information about the libvir-list
mailing list