[Libvirt-cim] [PATCH] Add PoolID into DevicePool instances

# HG changeset patch # User Dan Smith <danms(a)us.ibm.com&gt; # Date 1210263836 25200 # Node ID 1eef845c77d096d65b256384423b93cda19fb8db # Parent a46dd9e2e4efbea37e7e5d2cf8c285e1a56bd726 Add PoolID into DevicePool instances Changes: - Consolidate the setting of common ResourcePool attributes Signed-off-by: Dan Smith <danms(a)us.ibm.com&gt; diff -r a46dd9e2e4ef -r 1eef845c77d0 src/Virt_DevicePool.c --- a/src/Virt_DevicePool.c Thu May 08 08:25:23 2008 -0700 +++ b/src/Virt_DevicePool.c Thu May 08 09:23:56 2008 -0700 @@ -561,13 +561,28 @@ static bool procpool_set_total(CMPIInsta return procs != 0; } -static bool set_units(CMPIInstance *inst, - const char *units) -{ - CMSetProperty(inst, "AllocationUnits", - (CMPIValue *)units, CMPI_chars); - - return true; +static void set_params(CMPIInstance *inst, + uint16_t type, + const char *id, + const char *units, + const char *caption) +{ + CMSetProperty(inst, "InstanceID", + (CMPIValue *)id, CMPI_chars); + + CMSetProperty(inst, "PoolID", + (CMPIValue *)id, CMPI_chars); + + CMSetProperty(inst, "ResourceType", + (CMPIValue *)&type, CMPI_uint16); + + if (units != NULL) + CMSetProperty(inst, "AllocationUnits", + (CMPIValue *)units, CMPI_chars); + + if (caption != NULL) + CMSetProperty(inst, "Caption", + (CMPIValue *)caption, CMPI_chars); } static CMPIStatus mempool_instance(virConnectPtr conn, @@ -577,7 +592,6 @@ static CMPIStatus mempool_instance(virCo const CMPIBroker *broker) { const char *id = "MemoryPool/0"; - uint16_t type = CIM_RES_TYPE_MEM; CMPIInstance *inst; CMPIStatus s = {CMPI_RC_OK, NULL}; @@ -595,13 +609,8 @@ static CMPIStatus mempool_instance(virCo mempool_set_total(inst, conn); mempool_set_reserved(inst, conn); - set_units(inst, "KiloBytes"); - - CMSetProperty(inst, "InstanceID", - (CMPIValue *)id, CMPI_chars); - - CMSetProperty(inst, "ResourceType", - (CMPIValue *)&type, CMPI_uint16); + + set_params(inst, CIM_RES_TYPE_MEM, id, "KiloBytes", NULL); inst_list_add(list, inst); @@ -615,7 +624,6 @@ static CMPIStatus procpool_instance(virC const CMPIBroker *broker) { const char *id = "ProcessorPool/0"; - uint16_t type = CIM_RES_TYPE_PROC; CMPIInstance *inst; CMPIStatus s = {CMPI_RC_OK, NULL}; @@ -632,13 +640,8 @@ static CMPIStatus procpool_instance(virC ns); procpool_set_total(inst, conn); - set_units(inst, "Processors"); - - CMSetProperty(inst, "InstanceID", - (CMPIValue *)id, CMPI_chars); - - CMSetProperty(inst, "ResourceType", - (CMPIValue *)&type, CMPI_uint16); + + set_params(inst, CIM_RES_TYPE_PROC, id, "Processors", NULL); inst_list_add(list, inst); @@ -653,9 +656,9 @@ static CMPIStatus _netpool_for_network(s const CMPIBroker *broker) { CMPIStatus s = {CMPI_RC_OK, NULL}; - char *str = NULL; + char *id = NULL; + char *cap = NULL; char *bridge = NULL; - uint16_t type = CIM_RES_TYPE_NET; CMPIInstance *inst; virNetworkPtr network = NULL; @@ -681,36 +684,27 @@ static CMPIStatus _netpool_for_network(s goto out; } - if (asprintf(&str, "NetworkPool/%s", netname) == -1) { + if (asprintf(&id, "NetworkPool/%s", netname) == -1) { cu_statusf(broker, &s, CMPI_RC_ERR_FAILED, ""); goto out; } - CMSetProperty(inst, "InstanceID", - (CMPIValue *)str, CMPI_chars); - free(str); - bridge = virNetworkGetBridgeName(network); - if (asprintf(&str, "Bridge: %s", bridge) == -1) { + if (asprintf(&cap, "Bridge: %s", bridge) == -1) { cu_statusf(broker, &s, CMPI_RC_ERR_FAILED, ""); goto out; } - CMSetProperty(inst, "Caption", - (CMPIValue *)str, CMPI_chars); - free(str); + set_params(inst, CIM_RES_TYPE_NET, id, NULL, cap); + free(id); + free(cap); free(bridge); - CMSetProperty(inst, "ResourceType", - (CMPIValue *)&type, CMPI_uint16); - - inst_list_add(list, inst); - out: virNetworkFree(network); @@ -778,24 +772,13 @@ static CMPIInstance *diskpool_from_path( { CMPIInstance *inst; char *poolid = NULL; - const uint16_t type = CIM_RES_TYPE_DISK; inst = get_typed_instance(broker, refcn, "DiskPool", ns); if (asprintf(&poolid, "DiskPool/%s", pool->tag) == -1) return NULL; - CMSetProperty(inst, "InstanceID", - (CMPIValue *)poolid, CMPI_chars); - - CMSetProperty(inst, "ResourceType", - (CMPIValue *)&type, CMPI_uint16); - - CMSetProperty(inst, "AllocationUnits", - (CMPIValue *)"Megabytes", CMPI_chars); - - CMSetProperty(inst, "Caption", - (CMPIValue *)pool->tag, CMPI_chars); + set_params(inst, CIM_RES_TYPE_DISK, poolid, "Megabytes", pool->tag); if (!diskpool_set_capacity(conn, inst, pool)) CU_DEBUG("Failed to set capacity for disk pool: %s",