11:44 a.m.
Add some external interfaces to DevicePool
Unchanged from previous versions in the NetRASD set
11:44 a.m.
New subject: [PATCH 1 of 2] Add function to parse out pool name from InstanceID
# HG changeset patch
# User Dan Smith <danms(a)us.ibm.com>
# Date 1208191443 25200
# Node ID 2afa2a7d1d5a4e7d026f2f6cf8b5a1a8a266b5a0
# Parent 857848be56f0bdc3480bb9f5a7ae6d0598b4d9a8
Add function to parse out pool name from InstanceID
Signed-off-by: Dan Smith <danms(a)us.ibm.com>
diff -r 857848be56f0 -r 2afa2a7d1d5a src/Virt_DevicePool.c
--- a/src/Virt_DevicePool.c Mon Apr 14 09:42:32 2008 -0700
+++ b/src/Virt_DevicePool.c Mon Apr 14 09:44:03 2008 -0700
@@ -500,6 +500,17 @@ uint16_t res_type_from_pool_id(const cha
return CIM_RES_TYPE_UNKNOWN;
}
+char *name_from_pool_id(const char *id)
+{
+ char *s;
+
+ s = strchr(id, '/');
+ if (s == NULL)
+ return NULL;
+
+ return strdup((char *)s+1);
+}
+
static bool mempool_set_total(CMPIInstance *inst, virConnectPtr conn)
{
virNodeInfo info;
diff -r 857848be56f0 -r 2afa2a7d1d5a src/Virt_DevicePool.h
--- a/src/Virt_DevicePool.h Mon Apr 14 09:42:32 2008 -0700
+++ b/src/Virt_DevicePool.h Mon Apr 14 09:44:03 2008 -0700
@@ -58,6 +58,14 @@ uint16_t res_type_from_pool_id(const cha
uint16_t res_type_from_pool_id(const char *id);
/**
+ * Get the pool name from a given pool's InstanceID
+ *
+ * @param id The InstanceID of the pool
+ * @returns the name (must be free'd by the caller)
+ */
+char *name_from_pool_id(const char *id);
+
+/**
* Get all device pools on the system for the given type
*
*
11:44 a.m.
New subject: [PATCH 2 of 2] Add ability to get a default pool of a given type to the pool provider
# HG changeset patch
# User Dan Smith <danms(a)us.ibm.com>
# Date 1208191444 25200
# Node ID 327cfc9f69191f5a2f0454290c8ecf02e223d4dd
# Parent 2afa2a7d1d5a4e7d026f2f6cf8b5a1a8a266b5a0
Add ability to get a default pool of a given type to the pool provider
Signed-off-by: Dan Smith <danms(a)us.ibm.com>
diff -r 2afa2a7d1d5a -r 327cfc9f6919 src/Virt_DevicePool.c
--- a/src/Virt_DevicePool.c Mon Apr 14 09:44:03 2008 -0700
+++ b/src/Virt_DevicePool.c Mon Apr 14 09:44:04 2008 -0700
@@ -991,6 +991,39 @@ CMPIStatus enum_pools(const CMPIBroker *
return _get_pools(broker, reference, type, NULL, list);
}
+CMPIInstance *default_device_pool(const CMPIBroker *broker,
+ const CMPIObjectPath *reference,
+ uint16_t type,
+ CMPIStatus *s)
+{
+ CMPIInstance *inst = NULL;
+ struct inst_list list;
+
+ inst_list_init(&list);
+
+ if (type == CIM_RES_TYPE_MEM) {
+ *s = get_pool_by_name(broker, reference, "MemoryPool/0",
&inst);
+ } else if (type == CIM_RES_TYPE_PROC) {
+ *s = get_pool_by_name(broker, reference, "ProcPool/0",
&inst);
+ } else if (type == CIM_RES_TYPE_DISK) {
+ *s = enum_pools(broker, reference, type, &list);
+ if (s->rc == CMPI_RC_OK)
+ inst = list.list[0];
+ } else if (type == CIM_RES_TYPE_NET) {
+ *s = enum_pools(broker, reference, type, &list);
+ if (s->rc == CMPI_RC_OK)
+ inst = list.list[0];
+ } else {
+ cu_statusf(broker, s,
+ CMPI_RC_ERR_INVALID_PARAMETER,
+ "No such device type `%s'", type);
+ }
+
+ inst_list_free(&list);
+
+ return inst;
+}
+
static CMPIStatus return_pool(const CMPIObjectPath *ref,
const CMPIResult *results,
bool names_only)
diff -r 2afa2a7d1d5a -r 327cfc9f6919 src/Virt_DevicePool.h
--- a/src/Virt_DevicePool.h Mon Apr 14 09:44:03 2008 -0700
+++ b/src/Virt_DevicePool.h Mon Apr 14 09:44:04 2008 -0700
@@ -105,6 +105,20 @@ CMPIStatus get_pool_by_name(const CMPIBr
const char *id,
CMPIInstance **_inst);
+/**
+ * Get the default pool for a given device type
+ *
+ * @param broker A pointer to the current broker
+ * @param ref The object path containing namespace and prefix info
+ * @param type The device type in question
+ * @param status The returned status
+ * @returns Default instance of a pool
+ */
+CMPIInstance *default_device_pool(const CMPIBroker *broker,
+ const CMPIObjectPath *reference,
+ uint16_t type,
+ CMPIStatus *status);
+
#endif
/*
12:53 p.m.
New subject: [PATCH 2 of 2] Add ability to get a default pool of a given type to the pool provider
Dan Smith wrote:
# HG changeset patch
# User Dan Smith <danms(a)us.ibm.com>
# Date 1208191444 25200
# Node ID 327cfc9f69191f5a2f0454290c8ecf02e223d4dd
# Parent 2afa2a7d1d5a4e7d026f2f6cf8b5a1a8a266b5a0
Add ability to get a default pool of a given type to the pool provider
Signed-off-by: Dan Smith <danms(a)us.ibm.com>
diff -r 2afa2a7d1d5a -r 327cfc9f6919 src/Virt_DevicePool.c
--- a/src/Virt_DevicePool.c Mon Apr 14 09:44:03 2008 -0700
+++ b/src/Virt_DevicePool.c Mon Apr 14 09:44:04 2008 -0700
@@ -991,6 +991,39 @@ CMPIStatus enum_pools(const CMPIBroker *
return _get_pools(broker, reference, type, NULL, list);
}
+CMPIInstance *default_device_pool(const CMPIBroker *broker,
+ const CMPIObjectPath *reference,
+ uint16_t type,
+ CMPIStatus *s)
+{
+ CMPIInstance *inst = NULL;
+ struct inst_list list;
+
+ inst_list_init(&list);
+
+ if (type == CIM_RES_TYPE_MEM) {
+ *s = get_pool_by_name(broker, reference, "MemoryPool/0",
&inst);
+ } else if (type == CIM_RES_TYPE_PROC) {
+ *s = get_pool_by_name(broker, reference, "ProcPool/0",
&inst);
This should be ProcessorPool, which is how the InstanceID is set
elsewhere.
--
Kaitlin Rupert
IBM Linux Technology Center
kaitlin(a)linux.vnet.ibm.com
1:08 p.m.
New subject: [PATCH 2 of 2] Add ability to get a default pool of a given type to the pool provider
KR> This should be ProcessorPool, which is how the InstanceID is set
KR> elsewhere.
Well, that was stupid of me :)
Replacement on the way soon.
--
Dan Smith
IBM Linux Technology Center
Open Hypervisor Team
email: danms(a)us.ibm.com
6058
days inactive
6058
days old
4 comments
2 participants
participants (2)
-
Dan Smith -
Kaitlin Rupert