[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-ppc] [PATCH v5 09/12] nvdimm: convert "unarmed" into a static prop
From: |
David Hildenbrand |
Subject: |
[Qemu-ppc] [PATCH v5 09/12] nvdimm: convert "unarmed" into a static property |
Date: |
Tue, 19 Jun 2018 15:41:38 +0200 |
We don't allow to modify it after realization. So we can simply turn
it into a static property.
Reviewed-by: David Gibson <address@hidden>
Signed-off-by: David Hildenbrand <address@hidden>
---
hw/mem/nvdimm.c | 32 +++++++-------------------------
1 file changed, 7 insertions(+), 25 deletions(-)
diff --git a/hw/mem/nvdimm.c b/hw/mem/nvdimm.c
index df9716231f..7260c9c6b1 100644
--- a/hw/mem/nvdimm.c
+++ b/hw/mem/nvdimm.c
@@ -64,36 +64,11 @@ out:
error_propagate(errp, local_err);
}
-static bool nvdimm_get_unarmed(Object *obj, Error **errp)
-{
- NVDIMMDevice *nvdimm = NVDIMM(obj);
-
- return nvdimm->unarmed;
-}
-
-static void nvdimm_set_unarmed(Object *obj, bool value, Error **errp)
-{
- NVDIMMDevice *nvdimm = NVDIMM(obj);
- Error *local_err = NULL;
-
- if (memory_region_size(&nvdimm->nvdimm_mr)) {
- error_setg(&local_err, "cannot change property value");
- goto out;
- }
-
- nvdimm->unarmed = value;
-
- out:
- error_propagate(errp, local_err);
-}
-
static void nvdimm_init(Object *obj)
{
object_property_add(obj, NVDIMM_LABEL_SIZE_PROP, "int",
nvdimm_get_label_size, nvdimm_set_label_size, NULL,
NULL, NULL);
- object_property_add_bool(obj, NVDIMM_UNARMED_PROP,
- nvdimm_get_unarmed, nvdimm_set_unarmed, NULL);
}
static MemoryRegion *nvdimm_get_memory_region(PCDIMMDevice *dimm, Error **errp)
@@ -166,13 +141,20 @@ static void nvdimm_write_label_data(NVDIMMDevice *nvdimm,
const void *buf,
memory_region_set_dirty(mr, backend_offset, size);
}
+static Property nvdimm_properties[] = {
+ DEFINE_PROP_BOOL(NVDIMM_UNARMED_PROP, NVDIMMDevice, unarmed, false),
+ DEFINE_PROP_END_OF_LIST(),
+};
+
static void nvdimm_class_init(ObjectClass *oc, void *data)
{
PCDIMMDeviceClass *ddc = PC_DIMM_CLASS(oc);
NVDIMMClass *nvc = NVDIMM_CLASS(oc);
+ DeviceClass *dc = DEVICE_CLASS(oc);
ddc->realize = nvdimm_realize;
ddc->get_memory_region = nvdimm_get_memory_region;
+ dc->props = nvdimm_properties;
nvc->read_label_data = nvdimm_read_label_data;
nvc->write_label_data = nvdimm_write_label_data;
--
2.17.1
- [Qemu-ppc] [PATCH v5 00/12] pc-dimm: next bunch of cleanups, David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 01/12] pc-dimm: remove leftover "struct pc_dimms_capacity", David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 03/12] pc-dimm: rename pc_dimm_memory_* to pc_dimm_*, David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 02/12] pc: rename pc_dimm_(plug|unplug|...)* into pc_memory_(plug|unplug|...)*, David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 04/12] pc-dimm: remove pc_dimm_get_free_slot() from header, David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 05/12] pc: factor out pc specific dimm checks into pc_memory_pre_plug(), David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 06/12] nvdimm: no need to overwrite get_vmstate_memory_region(), David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 09/12] nvdimm: convert "unarmed" into a static property,
David Hildenbrand <=
- [Qemu-ppc] [PATCH v5 08/12] pc-dimm: merge get_(vmstate_)memory_region(), David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 10/12] nvdimm: convert nvdimm_mr into a pointer, David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 07/12] hostmem: drop error variable from host_memory_backend_get_memory(), David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 12/12] pc-dimm: get_memory_region() will not fail after realize, David Hildenbrand, 2018/06/19
- [Qemu-ppc] [PATCH v5 11/12] nvdimm: make get_memory_region() perform checks and initialization, David Hildenbrand, 2018/06/19
- Re: [Qemu-ppc] [PATCH v5 00/12] pc-dimm: next bunch of cleanups, Paolo Bonzini, 2018/06/20