[Libvirt-cim] [PATCH 1 of 3] Device: implement get_dev_by_ref and get_dev_by_name for usage in assocs
Heidi Eckhart
heidieck at linux.vnet.ibm.com
Tue Feb 26 12:51:48 UTC 2008
# HG changeset patch
# User Heidi Eckhart <heidieck at linux.vnet.ibm.com>
# Date 1204029552 -3600
# Node ID c9ac31ec5a89cd5b2d165c34d218412cbbb6c47d
# Parent 8ce1fe84f5831b7f3e32832901845a749a429c78
Device: implement get_dev_by_ref and get_dev_by_name for usage in assocs
Signed-off-by: Heidi Eckhart <heidieck at linux.vnet.ibm.com>
diff -r 8ce1fe84f583 -r c9ac31ec5a89 src/Virt_Device.c
--- a/src/Virt_Device.c Tue Feb 26 10:08:24 2008 +0100
+++ b/src/Virt_Device.c Tue Feb 26 13:39:12 2008 +0100
@@ -268,7 +268,7 @@ static CMPIInstance *device_instance(con
return instance;
}
-int device_type_from_classname(const char *classname)
+uint16_t device_type_from_classname(const char *classname)
{
if (strstr(classname, "NetworkPort"))
return VIRT_DEV_NET;
@@ -419,79 +419,96 @@ static struct virt_device *find_dom_dev(
return dev;
}
-CMPIInstance *instance_from_devid(const CMPIBroker *broker,
- virConnectPtr conn,
- const char *devid,
- const char *ns,
- int type)
-{
+CMPIStatus get_device_by_name(const CMPIBroker *broker,
+ const CMPIObjectPath *reference,
+ const char *name,
+ const uint16_t type,
+ CMPIInstance **_inst)
+{
+ CMPIStatus s = {CMPI_RC_OK, NULL};
char *domain = NULL;
char *device = NULL;
CMPIInstance *instance = NULL;
+ virConnectPtr conn = NULL;
virDomainPtr dom = NULL;
struct virt_device *dev = NULL;
- if (!parse_devid(devid, &domain, &device))
- return NULL;
-
- dom = virDomainLookupByName(conn, domain);
- if (!dom)
- goto out;
-
- dev = find_dom_dev(dom, device, type);
- if (!dev)
- goto out;
-
- instance = device_instance(broker, dev, dom, ns);
- cleanup_virt_device(dev);
-
- out:
- virDomainFree(dom);
- free(domain);
- free(device);
-
- return instance;
-}
-
-CMPIStatus get_device(const CMPIBroker *broker,
- const CMPIObjectPath *reference,
- CMPIInstance **_inst)
-{
- CMPIStatus s = {CMPI_RC_OK, NULL};
- virConnectPtr conn;
- CMPIInstance *inst;
- const char *cn;
- const char *devid;
-
- cn = CLASSNAME(reference);
-
- conn = connect_by_classname(broker, cn, &s);
+ conn = connect_by_classname(broker, CLASSNAME(reference), &s);
if (conn == NULL) {
cu_statusf(broker, &s,
CMPI_RC_ERR_NOT_FOUND,
"No such instance");
goto out;
- }
-
- if (cu_get_str_path(reference, "DeviceID", &devid) != CMPI_RC_OK) {
+ }
+
+ if (parse_devid(name, &domain, &device) != 1) {
+ cu_statusf(broker, &s,
+ CMPI_RC_ERR_NOT_FOUND,
+ "No such instance (%s)",
+ name);
+ goto out;
+ }
+
+ dom = virDomainLookupByName(conn, domain);
+ if (dom == NULL) {
+ cu_statusf(broker, &s,
+ CMPI_RC_ERR_NOT_FOUND,
+ "No such instance (%s)",
+ name);
+ goto err;
+ }
+
+ dev = find_dom_dev(dom, device, type);
+ if (!dev) {
+ cu_statusf(broker, &s,
+ CMPI_RC_ERR_NOT_FOUND,
+ "No such instance (%s)",
+ name);
+ goto err;
+ }
+
+ instance = device_instance(broker,
+ dev,
+ dom,
+ NAMESPACE(reference));
+ cleanup_virt_device(dev);
+
+ *_inst = instance;
+
+ err:
+ virDomainFree(dom);
+ free(domain);
+ free(device);
+
+ out:
+ virConnectClose(conn);
+
+ return s;
+}
+
+CMPIStatus get_device_by_ref(const CMPIBroker *broker,
+ const CMPIObjectPath *reference,
+ CMPIInstance **_inst)
+{
+ CMPIStatus s = {CMPI_RC_OK, NULL};
+ CMPIInstance *inst = NULL;
+ const char *name = NULL;
+
+ if (cu_get_str_path(reference, "DeviceID", &name) != CMPI_RC_OK) {
cu_statusf(broker, &s,
CMPI_RC_ERR_FAILED,
"No DeviceID specified");
goto out;
}
-
- inst = instance_from_devid(broker,
- conn,
- devid,
- NAMESPACE(reference),
- device_type_from_classname(cn));
- if (inst == NULL) {
- cu_statusf(broker, &s,
- CMPI_RC_ERR_NOT_FOUND,
- "No such instance (%s)", devid);
- goto out;
- }
-
+
+ s = get_device_by_name(broker,
+ reference,
+ name,
+ device_type_from_classname(CLASSNAME(reference)),
+ &inst);
+ if (s.rc != CMPI_RC_OK)
+ goto out;
+
s = cu_validate_ref(broker, reference, inst);
if (s.rc != CMPI_RC_OK)
goto out;
@@ -499,24 +516,7 @@ CMPIStatus get_device(const CMPIBroker *
*_inst = inst;
out:
- virConnectClose(conn);
-
return s;
-}
-
-static CMPIStatus return_device(const CMPIResult *results,
- const CMPIObjectPath *reference)
-{
- CMPIStatus s = {CMPI_RC_OK, NULL};
- CMPIInstance *inst = NULL;
-
- s = get_device(_BROKER, reference, &inst);
- if (s.rc != CMPI_RC_OK || inst == NULL)
- return s;
-
- CMReturnInstance(results, inst);
-
- return s;
}
static CMPIStatus EnumInstanceNames(CMPIInstanceMI *self,
@@ -542,7 +542,17 @@ static CMPIStatus GetInstance(CMPIInstan
const CMPIObjectPath *reference,
const char **properties)
{
- return return_device(results, reference);
+ CMPIStatus s = {CMPI_RC_OK, NULL};
+ CMPIInstance *inst = NULL;
+
+ s = get_device_by_ref(_BROKER, reference, &inst);
+ if (s.rc != CMPI_RC_OK)
+ goto out;
+
+ CMReturnInstance(results, inst);
+
+ out:
+ return s;
}
DEFAULT_CI();
diff -r 8ce1fe84f583 -r c9ac31ec5a89 src/Virt_Device.h
--- a/src/Virt_Device.h Tue Feb 26 10:08:24 2008 +0100
+++ b/src/Virt_Device.h Tue Feb 26 13:39:12 2008 +0100
@@ -46,30 +46,31 @@ int dom_devices(const CMPIBroker *broker
*
* @param broker A pointer to the CIM broker
* @param reference The object path identifying the instance
- * @param inst Contains the pointer to the instance in case
+ * @param _inst Contains the pointer to the instance in case
* of success
* @returns CMPIStatus of the operation
*/
-CMPIStatus get_device(const CMPIBroker *broker,
- const CMPIObjectPath *reference,
- CMPIInstance **_inst);
+CMPIStatus get_device_by_ref(const CMPIBroker *broker,
+ const CMPIObjectPath *reference,
+ CMPIInstance **_inst);
/**
- * Return a device instance for a given devid
+ * Returns the device instance for a given name and type
*
* @param broker A pointer to the CIM broker
- * @param conn The libvirt connection to use
- * @param devid The device id
- * @param reference the namespace
- * @returns The instance, or NULL if not found
+ * @param reference The object path containing namespace info
+ * @param name The name "<vm>/<resource>"
+ * @param type The resource type
+ * @param _inst The instance pointer in case of success
+ * @returns The result as CMPIStatus
*/
-CMPIInstance *instance_from_devid(const CMPIBroker *broker,
- virConnectPtr conn,
- const char *devid,
- const char *ns,
- int type);
+CMPIStatus get_device_by_name(const CMPIBroker *broker,
+ const CMPIObjectPath *reference,
+ const char *name,
+ const uint16_t type,
+ CMPIInstance **_inst);
-int device_type_from_classname(const char *classname);
+uint16_t device_type_from_classname(const char *classname);
#endif
More information about the Libvirt-cim
mailing list