aboutsummaryrefslogtreecommitdiff
path: root/hw/mem
diff options
context:
space:
mode:
authorBharata B Rao <bharata@linux.vnet.ibm.com>2015-01-27 09:35:02 +0530
committerMichael S. Tsirkin <mst@redhat.com>2015-01-27 14:46:18 +0200
commit37153450436f58449ce7e41d13a23821611e889e (patch)
tree8113dc857b997ee97320d5d0926cc3098f6dc819 /hw/mem
parent9967c94957f60c2d1cadbe0bfd4373a13fb1a226 (diff)
downloadqemu-37153450436f58449ce7e41d13a23821611e889e.zip
qemu-37153450436f58449ce7e41d13a23821611e889e.tar.gz
qemu-37153450436f58449ce7e41d13a23821611e889e.tar.bz2
pc-dimm: Add Error argument to pc_existing_dimms_capacity
Now that pc_existing_dimms_capacity() is an API, include Error pointer as an argument and modify the caller appropriately. Suggested-by: Igor Mammedov <imammedo@redhat.com> Signed-off-by: Bharata B Rao <bharata@linux.vnet.ibm.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Diffstat (limited to 'hw/mem')
-rw-r--r--hw/mem/pc-dimm.c32
1 files changed, 22 insertions, 10 deletions
diff --git a/hw/mem/pc-dimm.c b/hw/mem/pc-dimm.c
index f02ce6e..18cdc54 100644
--- a/hw/mem/pc-dimm.c
+++ b/hw/mem/pc-dimm.c
@@ -22,32 +22,44 @@
#include "qemu/config-file.h"
#include "qapi/visitor.h"
#include "qemu/range.h"
-#include "qapi/qmp/qerror.h"
-int pc_existing_dimms_capacity(Object *obj, void *opaque)
+typedef struct pc_dimms_capacity {
+ uint64_t size;
+ Error **errp;
+} pc_dimms_capacity;
+
+static int pc_existing_dimms_capacity_internal(Object *obj, void *opaque)
{
- Error *local_err = NULL;
- uint64_t *size = opaque;
+ pc_dimms_capacity *cap = opaque;
+ uint64_t *size = &cap->size;
if (object_dynamic_cast(obj, TYPE_PC_DIMM)) {
DeviceState *dev = DEVICE(obj);
if (dev->realized) {
(*size) += object_property_get_int(obj, PC_DIMM_SIZE_PROP,
- &local_err);
+ cap->errp);
}
- if (local_err) {
- qerror_report_err(local_err);
- error_free(local_err);
+ if (cap->errp && *cap->errp) {
return 1;
}
}
-
- object_child_foreach(obj, pc_existing_dimms_capacity, opaque);
+ object_child_foreach(obj, pc_existing_dimms_capacity_internal, opaque);
return 0;
}
+uint64_t pc_existing_dimms_capacity(Error **errp)
+{
+ pc_dimms_capacity cap;
+
+ cap.size = 0;
+ cap.errp = errp;
+
+ pc_existing_dimms_capacity_internal(qdev_get_machine(), &cap);
+ return cap.size;
+}
+
int qmp_pc_dimm_device_list(Object *obj, void *opaque)
{
MemoryDeviceInfoList ***prev = opaque;