提交 c534d10f 编写于 作者: J John Ferlan

conf: Introduce virDomainObjListAddObjLocked

Create a common helper to add an object to the locked domain
objlist hash tables and use it.
Signed-off-by: NJohn Ferlan <jferlan@redhat.com>
上级 e0f7707c
...@@ -220,6 +220,42 @@ virDomainObjPtr virDomainObjListFindByName(virDomainObjListPtr doms, ...@@ -220,6 +220,42 @@ virDomainObjPtr virDomainObjListFindByName(virDomainObjListPtr doms,
} }
/**
* @doms: Domain object list pointer
* @vm: Domain object to be added
*
* Upon entry @vm should have at least 1 ref and be locked.
*
* Add the @vm into the @doms->objs and @doms->objsName hash
* tables.
*
* Returns 0 on success with 2 references and locked
* -1 on failure with 1 reference and locked
*/
static int
virDomainObjListAddObjLocked(virDomainObjListPtr doms,
virDomainObjPtr vm)
{
char uuidstr[VIR_UUID_STRING_BUFLEN];
virUUIDFormat(vm->def->uuid, uuidstr);
if (virHashAddEntry(doms->objs, uuidstr, vm) < 0)
return -1;
if (virHashAddEntry(doms->objsName, vm->def->name, vm) < 0) {
virObjectRef(vm);
virHashRemoveEntry(doms->objs, uuidstr);
return -1;
}
/* Since domain is in two hash tables, increment the
* reference counter */
virObjectRef(vm);
return 0;
}
/* /*
* virDomainObjListAddLocked: * virDomainObjListAddLocked:
* *
...@@ -294,22 +330,10 @@ virDomainObjListAddLocked(virDomainObjListPtr doms, ...@@ -294,22 +330,10 @@ virDomainObjListAddLocked(virDomainObjListPtr doms,
goto cleanup; goto cleanup;
vm->def = def; vm->def = def;
virUUIDFormat(def->uuid, uuidstr); if (virDomainObjListAddObjLocked(doms, vm) < 0) {
if (virHashAddEntry(doms->objs, uuidstr, vm) < 0) {
virDomainObjEndAPI(&vm);
return NULL;
}
if (virHashAddEntry(doms->objsName, def->name, vm) < 0) {
virObjectRef(vm);
virHashRemoveEntry(doms->objs, uuidstr);
virDomainObjEndAPI(&vm); virDomainObjEndAPI(&vm);
return NULL; return NULL;
} }
/* Since domain is in two hash tables, increment the
* reference counter */
virObjectRef(vm);
} }
cleanup: cleanup:
return vm; return vm;
...@@ -532,19 +556,9 @@ virDomainObjListLoadStatus(virDomainObjListPtr doms, ...@@ -532,19 +556,9 @@ virDomainObjListLoadStatus(virDomainObjListPtr doms,
goto error; goto error;
} }
if (virHashAddEntry(doms->objs, uuidstr, obj) < 0) if (virDomainObjListAddObjLocked(doms, obj) < 0)
goto error; goto error;
if (virHashAddEntry(doms->objsName, obj->def->name, obj) < 0) {
virObjectRef(obj);
virHashRemoveEntry(doms->objs, uuidstr);
goto error;
}
/* Since domain is in two hash tables, increment the
* reference counter */
virObjectRef(obj);
if (notify) if (notify)
(*notify)(obj, 1, opaque); (*notify)(obj, 1, opaque);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册