mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2025-01-22 12:35:17 +00:00
vz: use state variable sdkdom in prlsdkApplyConfig
sdk domain handle is unique per connection so there is no sense to query it again if we have it in vzDomObjPtr. Side effect of prlsdkSdkDomainLookupByUUID is refreshing domain config is of no use too as PrlVm_BeginEdit do it too.
This commit is contained in:
parent
30d27f24d8
commit
2abef7fb55
@ -3906,28 +3906,22 @@ prlsdkApplyConfig(vzDriverPtr driver,
|
||||
virDomainObjPtr dom,
|
||||
virDomainDefPtr new)
|
||||
{
|
||||
PRL_HANDLE sdkdom = PRL_INVALID_HANDLE;
|
||||
vzDomObjPtr privdom = dom->privateData;
|
||||
PRL_HANDLE job = PRL_INVALID_HANDLE;
|
||||
int ret;
|
||||
|
||||
sdkdom = prlsdkSdkDomainLookupByUUID(driver, dom->def->uuid);
|
||||
if (sdkdom == PRL_INVALID_HANDLE)
|
||||
return -1;
|
||||
|
||||
job = PrlVm_BeginEdit(sdkdom);
|
||||
job = PrlVm_BeginEdit(privdom->sdkdom);
|
||||
if (PRL_FAILED(waitJob(job)))
|
||||
return -1;
|
||||
|
||||
ret = prlsdkDoApplyConfig(driver, sdkdom, new, dom->def);
|
||||
ret = prlsdkDoApplyConfig(driver, privdom->sdkdom, new, dom->def);
|
||||
|
||||
if (ret == 0) {
|
||||
job = PrlVm_CommitEx(sdkdom, PVCF_DETACH_HDD_BUNDLE);
|
||||
job = PrlVm_CommitEx(privdom->sdkdom, PVCF_DETACH_HDD_BUNDLE);
|
||||
if (PRL_FAILED(waitJob(job)))
|
||||
ret = -1;
|
||||
}
|
||||
|
||||
PrlHandle_Free(sdkdom);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
|
Loading…
x
Reference in New Issue
Block a user