[Libvirt-cim] [PATCH 04 of 10] DevicePool: reorganize get and enum functions

Thursday, 6 March 2008

# HG changeset patch
# User Heidi Eckhart <heidieck(a)linux.vnet.ibm.com&gt;
# Date 1204797486 -3600
# Node ID bcfdb432ac3a01af32dff70572a81e7c1823c038
# Parent  fe35567bbbb9af8ce77dac9dbd1148389aee9e06
DevicePool: reorganize get and enum functions

- the common look and feel of the get_..._by_name() and
  get_..._by_ref() is adopted
- get_all_pools() is renamed to enum_pools and rewritten
  to be configurable dynamically - return either a
  specific or all resource pools. This also replaces
  get_pool_by_type().
- the device_pool_names array is removed and types are
  identified by the CIM_RES_TYPE_..., as defined in
  svpc_types.h
- renamed device_type_by_poolid to res_type_by_pool_id

Signed-off-by: Heidi Eckhart <heidieck(a)linux.vnet.ibm.com&gt;

diff -r fe35567bbbb9 -r bcfdb432ac3a src/Virt_DevicePool.c

--- a/src/Virt_DevicePool.c	Thu Mar 06 10:58:04 2008 +0100
+++ b/src/Virt_DevicePool.c	Thu Mar 06 10:58:06 2008 +0100
@@ -45,12 +45,6 @@
 
 static const CMPIBroker *_BROKER;
 
-char *device_pool_names[] = {"ProcessorPool",
-                             "MemoryPool",
-                             "DiskPool",
-                             "NetworkPool",
-                             NULL};
-
 struct disk_pool {
         char *tag;
         char *path;
@@ -174,7 +168,7 @@ static char *diskpool_member_of(const CM
         if (dom == NULL)
                 goto out;
 
-        count = get_devices(dom, &devs, VIRT_DEV_DISK);
+        count = get_devices(dom, &devs, CIM_RES_TYPE_DISK);
 
         for (i = 0; i < count; i++) {
                 if (STREQ((devs[i].dev.disk.virtual_dev), dev)) {
@@ -290,7 +284,7 @@ static char *netpool_member_of(const CMP
         if (dom == NULL)
                 goto out;
 
-        count = get_devices(dom, &devs, VIRT_DEV_NET);
+        count = get_devices(dom, &devs, CIM_RES_TYPE_NET);
 
         for (i = 0; i < count; i++) {
                 if (STREQ((devs[i].id), dev)) {
@@ -318,13 +312,13 @@ char *pool_member_of(const CMPIBroker *b
 {
         char *poolid = NULL;
 
-        if (type == CIM_RASD_TYPE_PROC)
+        if (type == CIM_RES_TYPE_PROC)
                 poolid = strdup("ProcessorPool/0");
-        else if (type == CIM_RASD_TYPE_MEM)
+        else if (type == CIM_RES_TYPE_MEM)
                 poolid = strdup("MemoryPool/0");
-        else if (type == CIM_RASD_TYPE_NET)
+        else if (type == CIM_RES_TYPE_NET)
                 poolid = netpool_member_of(broker, id, refcn);
-        else if (type == CIM_RASD_TYPE_DISK)
+        else if (type == CIM_RES_TYPE_DISK)
                 poolid = diskpool_member_of(broker, id, refcn);
         else
                 return NULL;
@@ -332,18 +326,32 @@ char *pool_member_of(const CMPIBroker *b
         return poolid;
 }
 
-uint16_t device_type_from_poolid(const char *id)
+uint16_t res_type_from_pool_classname(const char *classname)
+{
+        if (strstr(classname, "NetworkPool"))
+                return CIM_RES_TYPE_NET;
+        else if (strstr(classname, "DiskPool"))
+                return CIM_RES_TYPE_DISK;
+        else if (strstr(classname, "MemoryPool"))
+                return CIM_RES_TYPE_MEM;
+        else if (strstr(classname, "ProcessorPool"))
+                return CIM_RES_TYPE_PROC;
+        else
+                return CIM_RES_TYPE_UNKNOWN;
+}
+
+uint16_t res_type_from_pool_id(const char *id)
 {
         if (STARTS_WITH(id, "NetworkPool"))
-                return VIRT_DEV_NET;
+                return CIM_RES_TYPE_NET;
         else if (STARTS_WITH(id, "DiskPool"))
-                return VIRT_DEV_DISK;
+                return CIM_RES_TYPE_DISK;
         else if (STARTS_WITH(id, "MemoryPool"))
-                return VIRT_DEV_MEM;
+                return CIM_RES_TYPE_MEM;
         else if (STARTS_WITH(id, "ProcessorPool"))
-                return VIRT_DEV_VCPU;
+                return CIM_RES_TYPE_PROC;
         else
-                return VIRT_DEV_UNKNOWN;
+                return CIM_RES_TYPE_UNKNOWN;
 }
 
 static bool mempool_set_total(CMPIInstance *inst, virConnectPtr conn)
@@ -412,7 +420,7 @@ static CMPIStatus mempool_instance(virCo
                                    const CMPIBroker *broker)
 {
         const char *id = "MemoryPool/0";
-        uint16_t type = CIM_RASD_TYPE_MEM;
+        uint16_t type = CIM_RES_TYPE_MEM;
         CMPIInstance *inst;
         CMPIStatus s = {CMPI_RC_OK, NULL};
 
@@ -450,7 +458,7 @@ static CMPIStatus procpool_instance(virC
                                     const CMPIBroker *broker)
 {
         const char *id = "ProcessorPool/0";
-        uint16_t type = CIM_RASD_TYPE_PROC;
+        uint16_t type = CIM_RES_TYPE_PROC;
         CMPIInstance *inst;
         CMPIStatus s = {CMPI_RC_OK, NULL};
 
@@ -489,7 +497,7 @@ static CMPIStatus _netpool_for_network(s
 {
         char *str = NULL;
         char *bridge = NULL;
-        uint16_t type = CIM_RASD_TYPE_NET;
+        uint16_t type = CIM_RES_TYPE_NET;
         CMPIInstance *inst;
         virNetworkPtr network = NULL;
 
@@ -595,7 +603,7 @@ static CMPIInstance *diskpool_from_path(
 {
         CMPIInstance *inst;
         char *poolid = NULL;
-        const uint16_t type = CIM_RASD_TYPE_DISK;
+        const uint16_t type = CIM_RES_TYPE_DISK;
         struct statvfs vfs;
         uint64_t cap;
         uint64_t res;
@@ -673,177 +681,190 @@ static CMPIStatus diskpool_instance(virC
         return s;
 }
 
-static CMPIStatus _get_pool(const CMPIBroker *broker,
-                            virConnectPtr conn,
-                            const char *type,
+static CMPIStatus _get_pools(const CMPIBroker *broker,
+                             const CMPIObjectPath *reference,
+                             const uint16_t type,
+                             const char *id,
+                             struct inst_list *list)
+{
+        CMPIStatus s = {CMPI_RC_OK, NULL};
+        virConnectPtr conn;
+
+        conn = connect_by_classname(broker, CLASSNAME(reference), &s);
+        if (conn == NULL)
+                goto out;
+
+        if ((type == CIM_RES_TYPE_PROC) || 
+            (type == CIM_RES_TYPE_ALL))
+                s = procpool_instance(conn, list,
+                                      NAMESPACE(reference),
+                                      id, broker);
+
+        if ((type == CIM_RES_TYPE_MEM) || 
+            (type == CIM_RES_TYPE_ALL))
+                s = mempool_instance(conn, list,
+                                     NAMESPACE(reference),
+                                     id, broker);
+
+        if ((type == CIM_RES_TYPE_NET) || 
+            (type == CIM_RES_TYPE_ALL))
+                s = netpool_instance(conn, list,
+                                     NAMESPACE(reference),
+                                     id, broker);
+
+        if ((type == CIM_RES_TYPE_DISK) || 
+            (type == CIM_RES_TYPE_ALL))
+                s = diskpool_instance(conn, list,
+                                      NAMESPACE(reference),
+                                      id, broker);
+
+        if (type == CIM_RES_TYPE_UNKNOWN)
+                cu_statusf(broker, &s,
+                           CMPI_RC_ERR_NOT_FOUND,
+                           "No such instance - resource pool type unknown");
+
+        if (id && list->cur == 0)
+                cu_statusf(broker, &s,
+                           CMPI_RC_ERR_NOT_FOUND,
+                           "No such instance (%s)", id);
+
+ out:
+        virConnectClose(conn);
+        return s;
+}
+
+CMPIStatus get_pool_by_name(const CMPIBroker *broker,
+                            const CMPIObjectPath *reference,
                             const char *id,
-                            const char *ns,
-                            struct inst_list *list)
-{
-        if (STARTS_WITH(type, "MemoryPool"))
-                return mempool_instance(conn, list, ns, id, broker);
-        else if (STARTS_WITH(type, "ProcessorPool"))
-                return procpool_instance(conn, list, ns, id, broker);
-        else if (STARTS_WITH(type, "NetworkPool"))
-                return netpool_instance(conn, list, ns, id, broker);
-        else if (STARTS_WITH(type, "DiskPool"))
-                return diskpool_instance(conn, list, ns, id, broker);
-
-        return (CMPIStatus){CMPI_RC_ERR_NOT_FOUND, NULL};
-}
-
-CMPIStatus get_pool_by_type(const CMPIBroker *broker,
-                            virConnectPtr conn,
-                            const char *type,
-                            const char *ns,
-                            struct inst_list *list)
-{
-        return _get_pool(broker, conn, type, NULL, ns, list);
-}
-
-CMPIInstance *get_pool_by_id(const CMPIBroker *broker,
-                             virConnectPtr conn,
-                             const char *id,
-                             const char *ns)
-{
-        CMPIInstance *inst = NULL;
-        CMPIStatus s;
-        char *type = NULL;
+                            CMPIInstance **_inst)
+{
+        CMPIStatus s = {CMPI_RC_OK, NULL};
+        virConnectPtr conn = NULL;
+        struct inst_list list;
         char *poolid = NULL;
         int ret;
-        struct inst_list list;
+        uint16_t type;
 
         inst_list_init(&list);
 
-        ret = sscanf(id, "%a[^/]/%as", &type, &poolid);
-        if (ret != 2)
-                goto out;
-
-        s = _get_pool(broker, conn, type, poolid, ns, &list);
-        if ((s.rc == CMPI_RC_OK) && (list.cur > 0))
-                inst = list.list[0];
-
- out:
+        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;
+        }
+
+        type = res_type_from_pool_id(id);
+
+        if (type == CIM_RES_TYPE_UNKNOWN) {
+                cu_statusf(broker, &s,
+                           CMPI_RC_ERR_NOT_FOUND,
+                           "No such instance (%s) - resource pool type
mismatch",
+                           id);
+                goto out;
+        }
+
+        ret = sscanf(id, "%*[^/]/%as", &poolid);
+        if (ret != 1) {
+                cu_statusf(broker, &s,
+                           CMPI_RC_ERR_NOT_FOUND,
+                           "No such instance (%s)",
+                           id);
+                goto out;
+        }
+
+        s = _get_pools(broker, reference, type, poolid, &list);
+        if (s.rc != CMPI_RC_OK)
+                goto out;
+
+        *_inst = list.list[0];
+
+ out:
+        free(poolid);
+        virConnectClose(conn);
         inst_list_free(&list);
 
-        return inst;
-}
-
-CMPIStatus get_all_pools(const CMPIBroker *broker,
-                         virConnectPtr conn,
-                         const char *ns,
-                         struct inst_list *list)
-{
-        int i;
+        return s;
+}
+
+CMPIStatus get_pool_by_ref(const CMPIBroker *broker,
+                           const CMPIObjectPath *reference,
+                           CMPIInstance **instance)
+{
         CMPIStatus s = {CMPI_RC_OK};
-
-        for (i = 0; device_pool_names[i]; i++) {
-                s = get_pool_by_type(broker,
-                                     conn,
-                                     device_pool_names[i],
-                                     ns,
-                                     list);
-                if (s.rc != CMPI_RC_OK)
-                        goto out;
-        }
-
+        CMPIInstance *inst = NULL;
+        const char *id = NULL;
+        uint16_t type_cls;
+        uint16_t type_id;
+
+        if (cu_get_str_path(reference, "InstanceID", &id) != CMPI_RC_OK) {
+                cu_statusf(broker, &s,
+                           CMPI_RC_ERR_FAILED,
+                           "Missing InstanceID");
+                goto out;
+        }
+        
+        type_cls = res_type_from_pool_classname(CLASSNAME(reference));
+        type_id = res_type_from_pool_id(id);
+
+        if ((type_cls != type_id) || 
+            (type_cls == CIM_RES_TYPE_UNKNOWN)) {
+                cu_statusf(broker, &s,
+                           CMPI_RC_ERR_NOT_FOUND,
+                           "No such instance (%s) - resource pool type
mismatch",
+                           id);
+                goto out;
+        }
+
+        s = get_pool_by_name(broker, reference, id, &inst);
+        if (s.rc != CMPI_RC_OK)
+                goto out;
+
+        s = cu_validate_ref(broker, reference, inst);
+        if (s.rc != CMPI_RC_OK)
+                goto out;
+
+        *instance = inst;
+        
  out:
         return s;
 }
 
-static void __return_pool(const CMPIResult *results,
-                          struct inst_list *list,
-                          bool name_only)
-{
-        if (name_only)
-                cu_return_instance_names(results, list);
-        else
-                cu_return_instances(results, list);
+CMPIStatus enum_pools(const CMPIBroker *broker,
+                      const CMPIObjectPath *reference,
+                      const uint16_t type,
+                      struct inst_list *list)
+{
+        return _get_pools(broker, reference, type, NULL, list);
 }
 
 static CMPIStatus return_pool(const CMPIObjectPath *ref,
                               const CMPIResult *results,
-                              bool name_only,
-                              bool single_only)
-{
-        CMPIStatus s;
-        char *type;
-        virConnectPtr conn;
+                              bool names_only)
+{
+        CMPIStatus s = {CMPI_RC_OK, NULL};
         struct inst_list list;
 
         if (!provider_is_responsible(_BROKER, ref, &s))
-                return s;
-
-        type = class_base_name(CLASSNAME(ref));
-        if (type == NULL) {
-                cu_statusf(_BROKER, &s,
-                           CMPI_RC_ERR_FAILED,
-                           "Invalid classname `%s'", CLASSNAME(ref));
-                return s;
-        }
+                goto out;
 
         inst_list_init(&list);
 
-        conn = connect_by_classname(_BROKER, CLASSNAME(ref), &s);
-        if (conn == NULL)
-                goto out;
-
-        s = get_pool_by_type(_BROKER,
-                             conn,
-                             type,
-                             NAMESPACE(ref),
-                             &list);
-        if (s.rc == CMPI_RC_OK) {
-                __return_pool(results, &list, name_only);
-                cu_statusf(_BROKER, &s,
-                           CMPI_RC_OK,
-                           "");
-        } else {
-                cu_statusf(_BROKER, &s,
-                           CMPI_RC_ERR_FAILED,
-                           "Pool type %s not found", type);
-        }
-
- out:
-        free(type);
+        s = enum_pools(_BROKER,
+                       ref,
+                       res_type_from_pool_classname(CLASSNAME(ref)),
+                       &list);
+        if (s.rc != CMPI_RC_OK)
+                goto out;
+        
+        if (names_only)
+                cu_return_instance_names(results, &list);
+        else
+                cu_return_instances(results, &list);
+
+ out:
         inst_list_free(&list);
-
-        return s;
-}
-
-CMPIStatus get_pool_inst(const CMPIBroker *broker,
-                         const CMPIObjectPath *reference,
-                         CMPIInstance **instance)
-{
-        CMPIStatus s;
-        CMPIInstance *inst = NULL;
-        virConnectPtr conn = NULL;
-        const char *id = NULL;
-
-        if (cu_get_str_path(reference, "InstanceID", &id) != CMPI_RC_OK) {
-                cu_statusf(broker, &s,
-                           CMPI_RC_ERR_FAILED,
-                           "Missing InstanceID");
-                goto out;
-        }
-
-        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;
-        }
-
-        inst = get_pool_by_id(broker, conn, id, NAMESPACE(reference));
-        if (inst == NULL)
-                cu_statusf(broker, &s,
-                           CMPI_RC_ERR_NOT_FOUND,
-                           "No such instance (%s)", id);
-        
- out:
-        virConnectClose(conn);
-        *instance = inst;
 
         return s;
 }
@@ -853,7 +874,7 @@ static CMPIStatus EnumInstanceNames(CMPI
                                     const CMPIResult *results,
                                     const CMPIObjectPath *reference)
 {
-        return return_pool(reference, results, true, false);
+        return return_pool(reference, results, true);
 }
 
 static CMPIStatus EnumInstances(CMPIInstanceMI *self,
@@ -862,7 +883,7 @@ static CMPIStatus EnumInstances(CMPIInst
                                 const CMPIObjectPath *reference,
                                 const char **properties)
 {
-        return return_pool(reference, results, false, false);
+        return return_pool(reference, results, false);
 }
 
 static CMPIStatus GetInstance(CMPIInstanceMI *self,
@@ -871,13 +892,16 @@ static CMPIStatus GetInstance(CMPIInstan
                               const CMPIObjectPath *reference,
                               const char **properties)
 {
-        CMPIStatus s;
+        CMPIStatus s = {CMPI_RC_OK};
         CMPIInstance *inst = NULL;
 
-        s = get_pool_inst(_BROKER, reference, &inst);
-        if ((s.rc == CMPI_RC_OK) && (inst != NULL))
-                CMReturnInstance(results, inst);
-
+        s = get_pool_by_ref(_BROKER, reference, &inst);
+        if (s.rc != CMPI_RC_OK)
+                goto out;
+
+        CMReturnInstance(results, inst);
+
+ out:
         return s;
 }
 
diff -r fe35567bbbb9 -r bcfdb432ac3a src/Virt_DevicePool.h
--- a/src/Virt_DevicePool.h	Thu Mar 06 10:58:04 2008 +0100
+++ b/src/Virt_DevicePool.h	Thu Mar 06 10:58:06 2008 +0100
@@ -26,20 +26,6 @@
 #include <libcmpiutil/libcmpiutil.h>
 #include <stdint.h>
 
-extern char *device_pool_names[];
-
-CMPIStatus get_pool_by_type(const CMPIBroker *broker,
-                            virConnectPtr conn,
-                            const char *type,
-                            const char *ns,
-                            struct inst_list *list);
-
-CMPIInstance *get_pool_by_id(const CMPIBroker *broker,
-                             virConnectPtr conn,
-                             const char *id,
-                             const char *ns);
-
-
 /**
  * Get the InstanceID of a pool that a given RASD id (for type) is in
  *
@@ -56,24 +42,35 @@ char *pool_member_of(const CMPIBroker *b
                      const char *id);
 
 /**
- * Get the device type of a given pool from the pool's InstanceID
+ * Get the resource type of a given pool from the pool's classname
+ *
+ * @param classname The classname of the pool
+ * Returns the resource type
+ */
+uint16_t res_type_from_pool_classname(const char *classname);
+
+/**
+ * Get the resource type of a given pool from the pool's InstanceID
  *
  * @param id The InstanceID of the pool
+ * Returns the resource type
  */
-uint16_t device_type_from_poolid(const char *id);
+uint16_t res_type_from_pool_id(const char *id);
 
 /**
- * Get all device pools on the system for the given connection
+ * Get all device pools on the system for the given type
+ * 
  *
  * @param broker The current Broker
- * @param conn The libvirt connection to use
- * @param ns Namespace for the pools
- * @param list Return instances in this struct
+ * @param reference Defines the libvirt connection to use
+ * @param type The device pool type or CIM_RES_TYPE_ALL
+ *             to get all resource pools
+ * @param list The list of returned instances
  */
-CMPIStatus get_all_pools(const CMPIBroker *broker,
-                         virConnectPtr conn,
-                         const char *ns,
-                         struct inst_list *list);
+CMPIStatus enum_pools(const CMPIBroker *broker,
+                      const CMPIObjectPath *reference,
+                      const uint16_t type,
+                      struct inst_list *list);
 
 /**
  * Get a device pools instance for the given reference 
@@ -82,9 +79,23 @@ CMPIStatus get_all_pools(const CMPIBroke
  * @param reference The reference passed to the CIMOM 
  * @param instance Return corresponding instance 
  */
-CMPIStatus get_pool_inst(const CMPIBroker *broker,
-                         const CMPIObjectPath *reference,
-                         CMPIInstance **instance);
+CMPIStatus get_pool_by_ref(const CMPIBroker *broker,
+                           const CMPIObjectPath *reference,
+                           CMPIInstance **instance);
+
+/**
+ * Get device pool instance specified by the id
+ *
+ * @param broker A pointer to the current broker
+ * @param ref The object path containing namespace and prefix info
+ * @param name The device pool id
+ * @param _inst In case of success the pointer to the instance
+ * @returns CMPIStatus
+ */
+CMPIStatus get_pool_by_name(const CMPIBroker *broker,
+                            const CMPIObjectPath *reference,
+                            const char *id,
+                            CMPIInstance **_inst);
 
 #endif
 


    

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

[Libvirt-cim] [PATCH 04 of 10] DevicePool: reorganize get and enum functions