[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 14/33] pc-dimm: drop the prefix of pc-dimm
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v6 14/33] pc-dimm: drop the prefix of pc-dimm |
Date: |
Mon, 9 Nov 2015 22:05:05 +0200 |
On Fri, Oct 30, 2015 at 01:56:08PM +0800, Xiao Guangrong wrote:
> This patch is generated by this script:
>
> find ./ -name "*.[ch]" -o -name "*.json" -o -name "trace-events" -type f \
> | xargs sed -i "s/PC_DIMM/DIMM/g"
>
> find ./ -name "*.[ch]" -o -name "*.json" -o -name "trace-events" -type f \
> | xargs sed -i "s/PCDIMM/DIMM/g"
>
> find ./ -name "*.[ch]" -o -name "*.json" -o -name "trace-events" -type f \
> | xargs sed -i "s/pc_dimm/dimm/g"
>
> find ./ -name "trace-events" -type f | xargs sed -i "s/pc-dimm/dimm/g"
>
> It prepares the work which abstracts dimm device type for both pc-dimm and
> nvdimm
>
> Signed-off-by: Xiao Guangrong <address@hidden>
I can see two ways this patchset can get merged
- merge refactorings first, nvdimm support on top
- merge nvdimm support first, refactor code on top
The way you put it in the middle of the series
allows neither. And I definitely favor option 2:
it's easier to reason about the best way to refactor code
when you have multiple users before you.
> ---
> hmp.c | 2 +-
> hw/acpi/ich9.c | 6 +-
> hw/acpi/memory_hotplug.c | 16 ++---
> hw/acpi/piix4.c | 6 +-
> hw/i386/pc.c | 32 ++++-----
> hw/mem/pc-dimm.c | 148
> ++++++++++++++++++++--------------------
> hw/ppc/spapr.c | 18 ++---
> include/hw/mem/pc-dimm.h | 62 ++++++++---------
> numa.c | 2 +-
> qapi-schema.json | 8 +--
> qmp.c | 2 +-
> stubs/qmp_pc_dimm_device_list.c | 2 +-
> trace-events | 8 +--
> 13 files changed, 156 insertions(+), 156 deletions(-)
>
> diff --git a/hmp.c b/hmp.c
> index 5048eee..5c617d2 100644
> --- a/hmp.c
> +++ b/hmp.c
> @@ -1952,7 +1952,7 @@ void hmp_info_memory_devices(Monitor *mon, const QDict
> *qdict)
> MemoryDeviceInfoList *info_list = qmp_query_memory_devices(&err);
> MemoryDeviceInfoList *info;
> MemoryDeviceInfo *value;
> - PCDIMMDeviceInfo *di;
> + DIMMDeviceInfo *di;
>
> for (info = info_list; info; info = info->next) {
> value = info->value;
> diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
> index 1c7fcfa..b0d6a67 100644
> --- a/hw/acpi/ich9.c
> +++ b/hw/acpi/ich9.c
> @@ -440,7 +440,7 @@ void ich9_pm_add_properties(Object *obj, ICH9LPCPMRegs
> *pm, Error **errp)
> void ich9_pm_device_plug_cb(ICH9LPCPMRegs *pm, DeviceState *dev, Error
> **errp)
> {
> if (pm->acpi_memory_hotplug.is_enabled &&
> - object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> acpi_memory_plug_cb(&pm->acpi_regs, pm->irq,
> &pm->acpi_memory_hotplug,
> dev, errp);
> } else if (object_dynamic_cast(OBJECT(dev), TYPE_CPU)) {
> @@ -455,7 +455,7 @@ void ich9_pm_device_unplug_request_cb(ICH9LPCPMRegs *pm,
> DeviceState *dev,
> Error **errp)
> {
> if (pm->acpi_memory_hotplug.is_enabled &&
> - object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> acpi_memory_unplug_request_cb(&pm->acpi_regs, pm->irq,
> &pm->acpi_memory_hotplug, dev, errp);
> } else {
> @@ -468,7 +468,7 @@ void ich9_pm_device_unplug_cb(ICH9LPCPMRegs *pm,
> DeviceState *dev,
> Error **errp)
> {
> if (pm->acpi_memory_hotplug.is_enabled &&
> - object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> acpi_memory_unplug_cb(&pm->acpi_memory_hotplug, dev, errp);
> } else {
> error_setg(errp, "acpi: device unplug for not supported device"
> diff --git a/hw/acpi/memory_hotplug.c b/hw/acpi/memory_hotplug.c
> index ce428df..e687852 100644
> --- a/hw/acpi/memory_hotplug.c
> +++ b/hw/acpi/memory_hotplug.c
> @@ -54,23 +54,23 @@ static uint64_t acpi_memory_hotplug_read(void *opaque,
> hwaddr addr,
> o = OBJECT(mdev->dimm);
> switch (addr) {
> case 0x0: /* Lo part of phys address where DIMM is mapped */
> - val = o ? object_property_get_int(o, PC_DIMM_ADDR_PROP, NULL) : 0;
> + val = o ? object_property_get_int(o, DIMM_ADDR_PROP, NULL) : 0;
> trace_mhp_acpi_read_addr_lo(mem_st->selector, val);
> break;
> case 0x4: /* Hi part of phys address where DIMM is mapped */
> - val = o ? object_property_get_int(o, PC_DIMM_ADDR_PROP, NULL) >> 32
> : 0;
> + val = o ? object_property_get_int(o, DIMM_ADDR_PROP, NULL) >> 32 : 0;
> trace_mhp_acpi_read_addr_hi(mem_st->selector, val);
> break;
> case 0x8: /* Lo part of DIMM size */
> - val = o ? object_property_get_int(o, PC_DIMM_SIZE_PROP, NULL) : 0;
> + val = o ? object_property_get_int(o, DIMM_SIZE_PROP, NULL) : 0;
> trace_mhp_acpi_read_size_lo(mem_st->selector, val);
> break;
> case 0xc: /* Hi part of DIMM size */
> - val = o ? object_property_get_int(o, PC_DIMM_SIZE_PROP, NULL) >> 32
> : 0;
> + val = o ? object_property_get_int(o, DIMM_SIZE_PROP, NULL) >> 32 : 0;
> trace_mhp_acpi_read_size_hi(mem_st->selector, val);
> break;
> case 0x10: /* node proximity for _PXM method */
> - val = o ? object_property_get_int(o, PC_DIMM_NODE_PROP, NULL) : 0;
> + val = o ? object_property_get_int(o, DIMM_NODE_PROP, NULL) : 0;
> trace_mhp_acpi_read_pxm(mem_st->selector, val);
> break;
> case 0x14: /* pack and return is_* fields */
> @@ -151,13 +151,13 @@ static void acpi_memory_hotplug_write(void *opaque,
> hwaddr addr, uint64_t data,
> /* call pc-dimm unplug cb */
> hotplug_handler_unplug(hotplug_ctrl, dev, &local_err);
> if (local_err) {
> - trace_mhp_acpi_pc_dimm_delete_failed(mem_st->selector);
> + trace_mhp_acpi_dimm_delete_failed(mem_st->selector);
> qapi_event_send_mem_unplug_error(dev->id,
> error_get_pretty(local_err),
> &error_abort);
> break;
> }
> - trace_mhp_acpi_pc_dimm_deleted(mem_st->selector);
> + trace_mhp_acpi_dimm_deleted(mem_st->selector);
> }
> break;
> default:
> @@ -206,7 +206,7 @@ acpi_memory_slot_status(MemHotplugState *mem_st,
> DeviceState *dev, Error **errp)
> {
> Error *local_err = NULL;
> - int slot = object_property_get_int(OBJECT(dev), PC_DIMM_SLOT_PROP,
> + int slot = object_property_get_int(OBJECT(dev), DIMM_SLOT_PROP,
> &local_err);
>
> if (local_err) {
> diff --git a/hw/acpi/piix4.c b/hw/acpi/piix4.c
> index 2cd2fee..0b2cb6e 100644
> --- a/hw/acpi/piix4.c
> +++ b/hw/acpi/piix4.c
> @@ -344,7 +344,7 @@ static void piix4_device_plug_cb(HotplugHandler
> *hotplug_dev,
> PIIX4PMState *s = PIIX4_PM(hotplug_dev);
>
> if (s->acpi_memory_hotplug.is_enabled &&
> - object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> acpi_memory_plug_cb(&s->ar, s->irq, &s->acpi_memory_hotplug, dev,
> errp);
> } else if (object_dynamic_cast(OBJECT(dev), TYPE_PCI_DEVICE)) {
> acpi_pcihp_device_plug_cb(&s->ar, s->irq, &s->acpi_pci_hotplug, dev,
> @@ -363,7 +363,7 @@ static void piix4_device_unplug_request_cb(HotplugHandler
> *hotplug_dev,
> PIIX4PMState *s = PIIX4_PM(hotplug_dev);
>
> if (s->acpi_memory_hotplug.is_enabled &&
> - object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> acpi_memory_unplug_request_cb(&s->ar, s->irq,
> &s->acpi_memory_hotplug,
> dev, errp);
> } else if (object_dynamic_cast(OBJECT(dev), TYPE_PCI_DEVICE)) {
> @@ -381,7 +381,7 @@ static void piix4_device_unplug_cb(HotplugHandler
> *hotplug_dev,
> PIIX4PMState *s = PIIX4_PM(hotplug_dev);
>
> if (s->acpi_memory_hotplug.is_enabled &&
> - object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> acpi_memory_unplug_cb(&s->acpi_memory_hotplug, dev, errp);
> } else {
> error_setg(errp, "acpi: device unplug for not supported device"
> diff --git a/hw/i386/pc.c b/hw/i386/pc.c
> index 0cb8afd..67ecc4f 100644
> --- a/hw/i386/pc.c
> +++ b/hw/i386/pc.c
> @@ -1610,14 +1610,14 @@ void ioapic_init_gsi(GSIState *gsi_state, const char
> *parent_name)
> }
> }
>
> -static void pc_dimm_plug(HotplugHandler *hotplug_dev,
> +static void dimm_plug(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp)
> {
> HotplugHandlerClass *hhc;
> Error *local_err = NULL;
> PCMachineState *pcms = PC_MACHINE(hotplug_dev);
> - PCDIMMDevice *dimm = PC_DIMM(dev);
> - PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm);
> + DIMMDevice *dimm = DIMM(dev);
> + DIMMDeviceClass *ddc = DIMM_GET_CLASS(dimm);
> MemoryRegion *mr = ddc->get_memory_region(dimm);
> uint64_t align = TARGET_PAGE_SIZE;
>
> @@ -1631,7 +1631,7 @@ static void pc_dimm_plug(HotplugHandler *hotplug_dev,
> goto out;
> }
>
> - pc_dimm_memory_plug(dev, &pcms->hotplug_memory, mr, align, &local_err);
> + dimm_memory_plug(dev, &pcms->hotplug_memory, mr, align, &local_err);
> if (local_err) {
> goto out;
> }
> @@ -1642,7 +1642,7 @@ out:
> error_propagate(errp, local_err);
> }
>
> -static void pc_dimm_unplug_request(HotplugHandler *hotplug_dev,
> +static void dimm_unplug_request(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp)
> {
> HotplugHandlerClass *hhc;
> @@ -1662,12 +1662,12 @@ out:
> error_propagate(errp, local_err);
> }
>
> -static void pc_dimm_unplug(HotplugHandler *hotplug_dev,
> +static void dimm_unplug(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp)
> {
> PCMachineState *pcms = PC_MACHINE(hotplug_dev);
> - PCDIMMDevice *dimm = PC_DIMM(dev);
> - PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm);
> + DIMMDevice *dimm = DIMM(dev);
> + DIMMDeviceClass *ddc = DIMM_GET_CLASS(dimm);
> MemoryRegion *mr = ddc->get_memory_region(dimm);
> HotplugHandlerClass *hhc;
> Error *local_err = NULL;
> @@ -1679,7 +1679,7 @@ static void pc_dimm_unplug(HotplugHandler *hotplug_dev,
> goto out;
> }
>
> - pc_dimm_memory_unplug(dev, &pcms->hotplug_memory, mr);
> + dimm_memory_unplug(dev, &pcms->hotplug_memory, mr);
> object_unparent(OBJECT(dev));
>
> out:
> @@ -1718,8 +1718,8 @@ out:
> static void pc_machine_device_plug_cb(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp)
> {
> - if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> - pc_dimm_plug(hotplug_dev, dev, errp);
> + if (object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> + dimm_plug(hotplug_dev, dev, errp);
> } else if (object_dynamic_cast(OBJECT(dev), TYPE_CPU)) {
> pc_cpu_plug(hotplug_dev, dev, errp);
> }
> @@ -1728,8 +1728,8 @@ static void pc_machine_device_plug_cb(HotplugHandler
> *hotplug_dev,
> static void pc_machine_device_unplug_request_cb(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error
> **errp)
> {
> - if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> - pc_dimm_unplug_request(hotplug_dev, dev, errp);
> + if (object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> + dimm_unplug_request(hotplug_dev, dev, errp);
> } else {
> error_setg(errp, "acpi: device unplug request for not supported
> device"
> " type: %s", object_get_typename(OBJECT(dev)));
> @@ -1739,8 +1739,8 @@ static void
> pc_machine_device_unplug_request_cb(HotplugHandler *hotplug_dev,
> static void pc_machine_device_unplug_cb(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp)
> {
> - if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> - pc_dimm_unplug(hotplug_dev, dev, errp);
> + if (object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> + dimm_unplug(hotplug_dev, dev, errp);
> } else {
> error_setg(errp, "acpi: device unplug for not supported device"
> " type: %s", object_get_typename(OBJECT(dev)));
> @@ -1752,7 +1752,7 @@ static HotplugHandler
> *pc_get_hotpug_handler(MachineState *machine,
> {
> PCMachineClass *pcmc = PC_MACHINE_GET_CLASS(machine);
>
> - if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM) ||
> + if (object_dynamic_cast(OBJECT(dev), TYPE_DIMM) ||
> object_dynamic_cast(OBJECT(dev), TYPE_CPU)) {
> return HOTPLUG_HANDLER(machine);
> }
> diff --git a/hw/mem/pc-dimm.c b/hw/mem/pc-dimm.c
> index 2dcbbcd..67afc53 100644
> --- a/hw/mem/pc-dimm.c
> +++ b/hw/mem/pc-dimm.c
> @@ -27,21 +27,21 @@
> #include "trace.h"
> #include "hw/virtio/vhost.h"
>
> -typedef struct pc_dimms_capacity {
> +typedef struct dimms_capacity {
> uint64_t size;
> Error **errp;
> -} pc_dimms_capacity;
> +} dimms_capacity;
>
> static int existing_dimms_capacity_internal(Object *obj, void *opaque)
> {
> - pc_dimms_capacity *cap = opaque;
> + dimms_capacity *cap = opaque;
> uint64_t *size = &cap->size;
>
> - if (object_dynamic_cast(obj, TYPE_PC_DIMM)) {
> + if (object_dynamic_cast(obj, TYPE_DIMM)) {
> DeviceState *dev = DEVICE(obj);
>
> if (dev->realized) {
> - (*size) += object_property_get_int(obj, PC_DIMM_SIZE_PROP,
> + (*size) += object_property_get_int(obj, DIMM_SIZE_PROP,
> cap->errp);
> }
>
> @@ -55,7 +55,7 @@ static int existing_dimms_capacity_internal(Object *obj,
> void *opaque)
>
> static uint64_t existing_dimms_capacity(Error **errp)
> {
> - pc_dimms_capacity cap;
> + dimms_capacity cap;
>
> cap.size = 0;
> cap.errp = errp;
> @@ -64,22 +64,22 @@ static uint64_t existing_dimms_capacity(Error **errp)
> return cap.size;
> }
>
> -void pc_dimm_memory_plug(DeviceState *dev, MemoryHotplugState *hpms,
> +void dimm_memory_plug(DeviceState *dev, MemoryHotplugState *hpms,
> MemoryRegion *mr, uint64_t align, Error **errp)
> {
> int slot;
> MachineState *machine = MACHINE(qdev_get_machine());
> - PCDIMMDevice *dimm = PC_DIMM(dev);
> + DIMMDevice *dimm = DIMM(dev);
> Error *local_err = NULL;
> uint64_t dimms_capacity = 0;
> uint64_t addr;
>
> - addr = object_property_get_int(OBJECT(dimm), PC_DIMM_ADDR_PROP,
> &local_err);
> + addr = object_property_get_int(OBJECT(dimm), DIMM_ADDR_PROP, &local_err);
> if (local_err) {
> goto out;
> }
>
> - addr = pc_dimm_get_free_addr(hpms->base,
> + addr = dimm_get_free_addr(hpms->base,
> memory_region_size(&hpms->mr),
> !addr ? NULL : &addr, align,
> memory_region_size(mr), &local_err);
> @@ -100,27 +100,27 @@ void pc_dimm_memory_plug(DeviceState *dev,
> MemoryHotplugState *hpms,
> goto out;
> }
>
> - object_property_set_int(OBJECT(dev), addr, PC_DIMM_ADDR_PROP,
> &local_err);
> + object_property_set_int(OBJECT(dev), addr, DIMM_ADDR_PROP, &local_err);
> if (local_err) {
> goto out;
> }
> - trace_mhp_pc_dimm_assigned_address(addr);
> + trace_mhp_dimm_assigned_address(addr);
>
> - slot = object_property_get_int(OBJECT(dev), PC_DIMM_SLOT_PROP,
> &local_err);
> + slot = object_property_get_int(OBJECT(dev), DIMM_SLOT_PROP, &local_err);
> if (local_err) {
> goto out;
> }
>
> - slot = pc_dimm_get_free_slot(slot == PC_DIMM_UNASSIGNED_SLOT ? NULL :
> &slot,
> + slot = dimm_get_free_slot(slot == DIMM_UNASSIGNED_SLOT ? NULL : &slot,
> machine->ram_slots, &local_err);
> if (local_err) {
> goto out;
> }
> - object_property_set_int(OBJECT(dev), slot, PC_DIMM_SLOT_PROP,
> &local_err);
> + object_property_set_int(OBJECT(dev), slot, DIMM_SLOT_PROP, &local_err);
> if (local_err) {
> goto out;
> }
> - trace_mhp_pc_dimm_assigned_slot(slot);
> + trace_mhp_dimm_assigned_slot(slot);
>
> if (kvm_enabled() && !kvm_has_free_slot(machine)) {
> error_setg(&local_err, "hypervisor has no free memory slots left");
> @@ -141,29 +141,29 @@ out:
> error_propagate(errp, local_err);
> }
>
> -void pc_dimm_memory_unplug(DeviceState *dev, MemoryHotplugState *hpms,
> +void dimm_memory_unplug(DeviceState *dev, MemoryHotplugState *hpms,
> MemoryRegion *mr)
> {
> - PCDIMMDevice *dimm = PC_DIMM(dev);
> + DIMMDevice *dimm = DIMM(dev);
>
> numa_unset_mem_node_id(dimm->addr, memory_region_size(mr), dimm->node);
> memory_region_del_subregion(&hpms->mr, mr);
> vmstate_unregister_ram(mr, dev);
> }
>
> -int qmp_pc_dimm_device_list(Object *obj, void *opaque)
> +int qmp_dimm_device_list(Object *obj, void *opaque)
> {
> MemoryDeviceInfoList ***prev = opaque;
>
> - if (object_dynamic_cast(obj, TYPE_PC_DIMM)) {
> + if (object_dynamic_cast(obj, TYPE_DIMM)) {
> DeviceState *dev = DEVICE(obj);
>
> if (dev->realized) {
> MemoryDeviceInfoList *elem = g_new0(MemoryDeviceInfoList, 1);
> MemoryDeviceInfo *info = g_new0(MemoryDeviceInfo, 1);
> - PCDIMMDeviceInfo *di = g_new0(PCDIMMDeviceInfo, 1);
> + DIMMDeviceInfo *di = g_new0(DIMMDeviceInfo, 1);
> DeviceClass *dc = DEVICE_GET_CLASS(obj);
> - PCDIMMDevice *dimm = PC_DIMM(obj);
> + DIMMDevice *dimm = DIMM(obj);
>
> if (dev->id) {
> di->has_id = true;
> @@ -174,7 +174,7 @@ int qmp_pc_dimm_device_list(Object *obj, void *opaque)
> di->addr = dimm->addr;
> di->slot = dimm->slot;
> di->node = dimm->node;
> - di->size = object_property_get_int(OBJECT(dimm),
> PC_DIMM_SIZE_PROP,
> + di->size = object_property_get_int(OBJECT(dimm), DIMM_SIZE_PROP,
> NULL);
> di->memdev = object_get_canonical_path(OBJECT(dimm->hostmem));
>
> @@ -186,7 +186,7 @@ int qmp_pc_dimm_device_list(Object *obj, void *opaque)
> }
> }
>
> - object_child_foreach(obj, qmp_pc_dimm_device_list, opaque);
> + object_child_foreach(obj, qmp_dimm_device_list, opaque);
> return 0;
> }
>
> @@ -197,7 +197,7 @@ ram_addr_t get_current_ram_size(void)
> MemoryDeviceInfoList *info;
> ram_addr_t size = ram_size;
>
> - qmp_pc_dimm_device_list(qdev_get_machine(), &prev);
> + qmp_dimm_device_list(qdev_get_machine(), &prev);
> for (info = info_list; info; info = info->next) {
> MemoryDeviceInfo *value = info->value;
>
> @@ -216,28 +216,28 @@ ram_addr_t get_current_ram_size(void)
> return size;
> }
>
> -static int pc_dimm_slot2bitmap(Object *obj, void *opaque)
> +static int dimm_slot2bitmap(Object *obj, void *opaque)
> {
> unsigned long *bitmap = opaque;
>
> - if (object_dynamic_cast(obj, TYPE_PC_DIMM)) {
> + if (object_dynamic_cast(obj, TYPE_DIMM)) {
> DeviceState *dev = DEVICE(obj);
> if (dev->realized) { /* count only realized DIMMs */
> - PCDIMMDevice *d = PC_DIMM(obj);
> + DIMMDevice *d = DIMM(obj);
> set_bit(d->slot, bitmap);
> }
> }
>
> - object_child_foreach(obj, pc_dimm_slot2bitmap, opaque);
> + object_child_foreach(obj, dimm_slot2bitmap, opaque);
> return 0;
> }
>
> -int pc_dimm_get_free_slot(const int *hint, int max_slots, Error **errp)
> +int dimm_get_free_slot(const int *hint, int max_slots, Error **errp)
> {
> unsigned long *bitmap = bitmap_new(max_slots);
> int slot = 0;
>
> - object_child_foreach(qdev_get_machine(), pc_dimm_slot2bitmap, bitmap);
> + object_child_foreach(qdev_get_machine(), dimm_slot2bitmap, bitmap);
>
> /* check if requested slot is not occupied */
> if (hint) {
> @@ -262,10 +262,10 @@ out:
> return slot;
> }
>
> -static gint pc_dimm_addr_sort(gconstpointer a, gconstpointer b)
> +static gint dimm_addr_sort(gconstpointer a, gconstpointer b)
> {
> - PCDIMMDevice *x = PC_DIMM(a);
> - PCDIMMDevice *y = PC_DIMM(b);
> + DIMMDevice *x = DIMM(a);
> + DIMMDevice *y = DIMM(b);
> Int128 diff = int128_sub(int128_make64(x->addr), int128_make64(y->addr));
>
> if (int128_lt(diff, int128_zero())) {
> @@ -276,22 +276,22 @@ static gint pc_dimm_addr_sort(gconstpointer a,
> gconstpointer b)
> return 0;
> }
>
> -static int pc_dimm_built_list(Object *obj, void *opaque)
> +static int dimm_built_list(Object *obj, void *opaque)
> {
> GSList **list = opaque;
>
> - if (object_dynamic_cast(obj, TYPE_PC_DIMM)) {
> + if (object_dynamic_cast(obj, TYPE_DIMM)) {
> DeviceState *dev = DEVICE(obj);
> if (dev->realized) { /* only realized DIMMs matter */
> - *list = g_slist_insert_sorted(*list, dev, pc_dimm_addr_sort);
> + *list = g_slist_insert_sorted(*list, dev, dimm_addr_sort);
> }
> }
>
> - object_child_foreach(obj, pc_dimm_built_list, opaque);
> + object_child_foreach(obj, dimm_built_list, opaque);
> return 0;
> }
>
> -uint64_t pc_dimm_get_free_addr(uint64_t address_space_start,
> +uint64_t dimm_get_free_addr(uint64_t address_space_start,
> uint64_t address_space_size,
> uint64_t *hint, uint64_t align, uint64_t size,
> Error **errp)
> @@ -321,7 +321,7 @@ uint64_t pc_dimm_get_free_addr(uint64_t
> address_space_start,
> }
>
> assert(address_space_end > address_space_start);
> - object_child_foreach(qdev_get_machine(), pc_dimm_built_list, &list);
> + object_child_foreach(qdev_get_machine(), dimm_built_list, &list);
>
> if (hint) {
> new_addr = *hint;
> @@ -331,9 +331,9 @@ uint64_t pc_dimm_get_free_addr(uint64_t
> address_space_start,
>
> /* find address range that will fit new DIMM */
> for (item = list; item; item = g_slist_next(item)) {
> - PCDIMMDevice *dimm = item->data;
> + DIMMDevice *dimm = item->data;
> uint64_t dimm_size = object_property_get_int(OBJECT(dimm),
> - PC_DIMM_SIZE_PROP,
> + DIMM_SIZE_PROP,
> errp);
> if (errp && *errp) {
> goto out;
> @@ -363,20 +363,20 @@ out:
> return ret;
> }
>
> -static Property pc_dimm_properties[] = {
> - DEFINE_PROP_UINT64(PC_DIMM_ADDR_PROP, PCDIMMDevice, addr, 0),
> - DEFINE_PROP_UINT32(PC_DIMM_NODE_PROP, PCDIMMDevice, node, 0),
> - DEFINE_PROP_INT32(PC_DIMM_SLOT_PROP, PCDIMMDevice, slot,
> - PC_DIMM_UNASSIGNED_SLOT),
> +static Property dimm_properties[] = {
> + DEFINE_PROP_UINT64(DIMM_ADDR_PROP, DIMMDevice, addr, 0),
> + DEFINE_PROP_UINT32(DIMM_NODE_PROP, DIMMDevice, node, 0),
> + DEFINE_PROP_INT32(DIMM_SLOT_PROP, DIMMDevice, slot,
> + DIMM_UNASSIGNED_SLOT),
> DEFINE_PROP_END_OF_LIST(),
> };
>
> -static void pc_dimm_get_size(Object *obj, Visitor *v, void *opaque,
> +static void dimm_get_size(Object *obj, Visitor *v, void *opaque,
> const char *name, Error **errp)
> {
> int64_t value;
> MemoryRegion *mr;
> - PCDIMMDevice *dimm = PC_DIMM(obj);
> + DIMMDevice *dimm = DIMM(obj);
>
> mr = host_memory_backend_get_memory(dimm->hostmem, errp);
> value = memory_region_size(mr);
> @@ -384,7 +384,7 @@ static void pc_dimm_get_size(Object *obj, Visitor *v,
> void *opaque,
> visit_type_int(v, &value, name, errp);
> }
>
> -static void pc_dimm_check_memdev_is_busy(Object *obj, const char *name,
> +static void dimm_check_memdev_is_busy(Object *obj, const char *name,
> Object *val, Error **errp)
> {
> MemoryRegion *mr;
> @@ -399,65 +399,65 @@ static void pc_dimm_check_memdev_is_busy(Object *obj,
> const char *name,
> }
> }
>
> -static void pc_dimm_init(Object *obj)
> +static void dimm_init(Object *obj)
> {
> - PCDIMMDevice *dimm = PC_DIMM(obj);
> + DIMMDevice *dimm = DIMM(obj);
>
> - object_property_add(obj, PC_DIMM_SIZE_PROP, "int", pc_dimm_get_size,
> + object_property_add(obj, DIMM_SIZE_PROP, "int", dimm_get_size,
> NULL, NULL, NULL, &error_abort);
> - object_property_add_link(obj, PC_DIMM_MEMDEV_PROP, TYPE_MEMORY_BACKEND,
> + object_property_add_link(obj, DIMM_MEMDEV_PROP, TYPE_MEMORY_BACKEND,
> (Object **)&dimm->hostmem,
> - pc_dimm_check_memdev_is_busy,
> + dimm_check_memdev_is_busy,
> OBJ_PROP_LINK_UNREF_ON_RELEASE,
> &error_abort);
> }
>
> -static void pc_dimm_realize(DeviceState *dev, Error **errp)
> +static void dimm_realize(DeviceState *dev, Error **errp)
> {
> - PCDIMMDevice *dimm = PC_DIMM(dev);
> + DIMMDevice *dimm = DIMM(dev);
>
> if (!dimm->hostmem) {
> - error_setg(errp, "'" PC_DIMM_MEMDEV_PROP "' property is not set");
> + error_setg(errp, "'" DIMM_MEMDEV_PROP "' property is not set");
> return;
> }
> if (((nb_numa_nodes > 0) && (dimm->node >= nb_numa_nodes)) ||
> (!nb_numa_nodes && dimm->node)) {
> - error_setg(errp, "'DIMM property " PC_DIMM_NODE_PROP " has value %"
> + error_setg(errp, "'DIMM property " DIMM_NODE_PROP " has value %"
> PRIu32 "' which exceeds the number of numa nodes: %d",
> dimm->node, nb_numa_nodes ? nb_numa_nodes : 1);
> return;
> }
> }
>
> -static MemoryRegion *pc_dimm_get_memory_region(PCDIMMDevice *dimm)
> +static MemoryRegion *dimm_get_memory_region(DIMMDevice *dimm)
> {
> return host_memory_backend_get_memory(dimm->hostmem, &error_abort);
> }
>
> -static void pc_dimm_class_init(ObjectClass *oc, void *data)
> +static void dimm_class_init(ObjectClass *oc, void *data)
> {
> DeviceClass *dc = DEVICE_CLASS(oc);
> - PCDIMMDeviceClass *ddc = PC_DIMM_CLASS(oc);
> + DIMMDeviceClass *ddc = DIMM_CLASS(oc);
>
> - dc->realize = pc_dimm_realize;
> - dc->props = pc_dimm_properties;
> + dc->realize = dimm_realize;
> + dc->props = dimm_properties;
> dc->desc = "DIMM memory module";
>
> - ddc->get_memory_region = pc_dimm_get_memory_region;
> + ddc->get_memory_region = dimm_get_memory_region;
> }
>
> -static TypeInfo pc_dimm_info = {
> - .name = TYPE_PC_DIMM,
> +static TypeInfo dimm_info = {
> + .name = TYPE_DIMM,
> .parent = TYPE_DEVICE,
> - .instance_size = sizeof(PCDIMMDevice),
> - .instance_init = pc_dimm_init,
> - .class_init = pc_dimm_class_init,
> - .class_size = sizeof(PCDIMMDeviceClass),
> + .instance_size = sizeof(DIMMDevice),
> + .instance_init = dimm_init,
> + .class_init = dimm_class_init,
> + .class_size = sizeof(DIMMDeviceClass),
> };
>
> -static void pc_dimm_register_types(void)
> +static void dimm_register_types(void)
> {
> - type_register_static(&pc_dimm_info);
> + type_register_static(&dimm_info);
> }
>
> -type_init(pc_dimm_register_types)
> +type_init(dimm_register_types)
> diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c
> index 288b57e..ab6eb83 100644
> --- a/hw/ppc/spapr.c
> +++ b/hw/ppc/spapr.c
> @@ -2144,8 +2144,8 @@ static void spapr_memory_plug(HotplugHandler
> *hotplug_dev, DeviceState *dev,
> {
> Error *local_err = NULL;
> sPAPRMachineState *ms = SPAPR_MACHINE(hotplug_dev);
> - PCDIMMDevice *dimm = PC_DIMM(dev);
> - PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm);
> + DIMMDevice *dimm = DIMM(dev);
> + DIMMDeviceClass *ddc = DIMM_GET_CLASS(dimm);
> MemoryRegion *mr = ddc->get_memory_region(dimm);
> uint64_t align = memory_region_get_alignment(mr);
> uint64_t size = memory_region_size(mr);
> @@ -2157,14 +2157,14 @@ static void spapr_memory_plug(HotplugHandler
> *hotplug_dev, DeviceState *dev,
> goto out;
> }
>
> - pc_dimm_memory_plug(dev, &ms->hotplug_memory, mr, align, &local_err);
> + dimm_memory_plug(dev, &ms->hotplug_memory, mr, align, &local_err);
> if (local_err) {
> goto out;
> }
>
> - addr = object_property_get_int(OBJECT(dimm), PC_DIMM_ADDR_PROP,
> &local_err);
> + addr = object_property_get_int(OBJECT(dimm), DIMM_ADDR_PROP, &local_err);
> if (local_err) {
> - pc_dimm_memory_unplug(dev, &ms->hotplug_memory, mr);
> + dimm_memory_unplug(dev, &ms->hotplug_memory, mr);
> goto out;
> }
>
> @@ -2179,14 +2179,14 @@ static void spapr_machine_device_plug(HotplugHandler
> *hotplug_dev,
> {
> sPAPRMachineClass *smc = SPAPR_MACHINE_GET_CLASS(qdev_get_machine());
>
> - if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + if (object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> int node;
>
> if (!smc->dr_lmb_enabled) {
> error_setg(errp, "Memory hotplug not supported for this
> machine");
> return;
> }
> - node = object_property_get_int(OBJECT(dev), PC_DIMM_NODE_PROP, errp);
> + node = object_property_get_int(OBJECT(dev), DIMM_NODE_PROP, errp);
> if (*errp) {
> return;
> }
> @@ -2220,7 +2220,7 @@ static void spapr_machine_device_plug(HotplugHandler
> *hotplug_dev,
> static void spapr_machine_device_unplug(HotplugHandler *hotplug_dev,
> DeviceState *dev, Error **errp)
> {
> - if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + if (object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> error_setg(errp, "Memory hot unplug not supported by sPAPR");
> }
> }
> @@ -2228,7 +2228,7 @@ static void spapr_machine_device_unplug(HotplugHandler
> *hotplug_dev,
> static HotplugHandler *spapr_get_hotpug_handler(MachineState *machine,
> DeviceState *dev)
> {
> - if (object_dynamic_cast(OBJECT(dev), TYPE_PC_DIMM)) {
> + if (object_dynamic_cast(OBJECT(dev), TYPE_DIMM)) {
> return HOTPLUG_HANDLER(machine);
> }
> return NULL;
> diff --git a/include/hw/mem/pc-dimm.h b/include/hw/mem/pc-dimm.h
> index 8a43548..ece8786 100644
> --- a/include/hw/mem/pc-dimm.h
> +++ b/include/hw/mem/pc-dimm.h
> @@ -13,39 +13,39 @@
> *
> */
>
> -#ifndef QEMU_PC_DIMM_H
> -#define QEMU_PC_DIMM_H
> +#ifndef QEMU_DIMM_H
> +#define QEMU_DIMM_H
>
> #include "exec/memory.h"
> #include "sysemu/hostmem.h"
> #include "hw/qdev.h"
>
> -#define TYPE_PC_DIMM "pc-dimm"
> -#define PC_DIMM(obj) \
> - OBJECT_CHECK(PCDIMMDevice, (obj), TYPE_PC_DIMM)
> -#define PC_DIMM_CLASS(oc) \
> - OBJECT_CLASS_CHECK(PCDIMMDeviceClass, (oc), TYPE_PC_DIMM)
> -#define PC_DIMM_GET_CLASS(obj) \
> - OBJECT_GET_CLASS(PCDIMMDeviceClass, (obj), TYPE_PC_DIMM)
> +#define TYPE_DIMM "pc-dimm"
> +#define DIMM(obj) \
> + OBJECT_CHECK(DIMMDevice, (obj), TYPE_DIMM)
> +#define DIMM_CLASS(oc) \
> + OBJECT_CLASS_CHECK(DIMMDeviceClass, (oc), TYPE_DIMM)
> +#define DIMM_GET_CLASS(obj) \
> + OBJECT_GET_CLASS(DIMMDeviceClass, (obj), TYPE_DIMM)
>
> -#define PC_DIMM_ADDR_PROP "addr"
> -#define PC_DIMM_SLOT_PROP "slot"
> -#define PC_DIMM_NODE_PROP "node"
> -#define PC_DIMM_SIZE_PROP "size"
> -#define PC_DIMM_MEMDEV_PROP "memdev"
> +#define DIMM_ADDR_PROP "addr"
> +#define DIMM_SLOT_PROP "slot"
> +#define DIMM_NODE_PROP "node"
> +#define DIMM_SIZE_PROP "size"
> +#define DIMM_MEMDEV_PROP "memdev"
>
> -#define PC_DIMM_UNASSIGNED_SLOT -1
> +#define DIMM_UNASSIGNED_SLOT -1
>
> /**
> - * PCDIMMDevice:
> - * @addr: starting guest physical address, where @PCDIMMDevice is mapped.
> + * DIMMDevice:
> + * @addr: starting guest physical address, where @DIMMDevice is mapped.
> * Default value: 0, means that address is auto-allocated.
> - * @node: numa node to which @PCDIMMDevice is attached.
> - * @slot: slot number into which @PCDIMMDevice is plugged in.
> + * @node: numa node to which @DIMMDevice is attached.
> + * @slot: slot number into which @DIMMDevice is plugged in.
> * Default value: -1, means that slot is auto-allocated.
> - * @hostmem: host memory backend providing memory for @PCDIMMDevice
> + * @hostmem: host memory backend providing memory for @DIMMDevice
> */
> -typedef struct PCDIMMDevice {
> +typedef struct DIMMDevice {
> /* private */
> DeviceState parent_obj;
>
> @@ -54,19 +54,19 @@ typedef struct PCDIMMDevice {
> uint32_t node;
> int32_t slot;
> HostMemoryBackend *hostmem;
> -} PCDIMMDevice;
> +} DIMMDevice;
>
> /**
> - * PCDIMMDeviceClass:
> + * DIMMDeviceClass:
> * @get_memory_region: returns #MemoryRegion associated with @dimm
> */
> -typedef struct PCDIMMDeviceClass {
> +typedef struct DIMMDeviceClass {
> /* private */
> DeviceClass parent_class;
>
> /* public */
> - MemoryRegion *(*get_memory_region)(PCDIMMDevice *dimm);
> -} PCDIMMDeviceClass;
> + MemoryRegion *(*get_memory_region)(DIMMDevice *dimm);
> +} DIMMDeviceClass;
>
> /**
> * MemoryHotplugState:
> @@ -79,16 +79,16 @@ typedef struct MemoryHotplugState {
> MemoryRegion mr;
> } MemoryHotplugState;
>
> -uint64_t pc_dimm_get_free_addr(uint64_t address_space_start,
> +uint64_t dimm_get_free_addr(uint64_t address_space_start,
> uint64_t address_space_size,
> uint64_t *hint, uint64_t align, uint64_t size,
> Error **errp);
>
> -int pc_dimm_get_free_slot(const int *hint, int max_slots, Error **errp);
> +int dimm_get_free_slot(const int *hint, int max_slots, Error **errp);
>
> -int qmp_pc_dimm_device_list(Object *obj, void *opaque);
> -void pc_dimm_memory_plug(DeviceState *dev, MemoryHotplugState *hpms,
> +int qmp_dimm_device_list(Object *obj, void *opaque);
> +void dimm_memory_plug(DeviceState *dev, MemoryHotplugState *hpms,
> MemoryRegion *mr, uint64_t align, Error **errp);
> -void pc_dimm_memory_unplug(DeviceState *dev, MemoryHotplugState *hpms,
> +void dimm_memory_unplug(DeviceState *dev, MemoryHotplugState *hpms,
> MemoryRegion *mr);
> #endif
> diff --git a/numa.c b/numa.c
> index e9b18f5..cb69965 100644
> --- a/numa.c
> +++ b/numa.c
> @@ -482,7 +482,7 @@ static void numa_stat_memory_devices(uint64_t node_mem[])
> MemoryDeviceInfoList **prev = &info_list;
> MemoryDeviceInfoList *info;
>
> - qmp_pc_dimm_device_list(qdev_get_machine(), &prev);
> + qmp_dimm_device_list(qdev_get_machine(), &prev);
> for (info = info_list; info; info = info->next) {
> MemoryDeviceInfo *value = info->value;
>
> diff --git a/qapi-schema.json b/qapi-schema.json
> index f60be29..aeae833 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -3717,9 +3717,9 @@
> { 'command': 'query-memdev', 'returns': ['Memdev'] }
>
> ##
> -# @PCDIMMDeviceInfo:
> +# @DIMMDeviceInfo:
> #
> -# PCDIMMDevice state information
> +# DIMMDevice state information
> #
> # @id: #optional device's ID
> #
> @@ -3739,7 +3739,7 @@
> #
> # Since: 2.1
> ##
> -{ 'struct': 'PCDIMMDeviceInfo',
> +{ 'struct': 'DIMMDeviceInfo',
> 'data': { '*id': 'str',
> 'addr': 'int',
> 'size': 'int',
> @@ -3758,7 +3758,7 @@
> #
> # Since: 2.1
> ##
> -{ 'union': 'MemoryDeviceInfo', 'data': {'dimm': 'PCDIMMDeviceInfo'} }
> +{ 'union': 'MemoryDeviceInfo', 'data': {'dimm': 'DIMMDeviceInfo'} }
>
> ##
> # @query-memory-devices
> diff --git a/qmp.c b/qmp.c
> index ff54e5a..169b981 100644
> --- a/qmp.c
> +++ b/qmp.c
> @@ -712,7 +712,7 @@ MemoryDeviceInfoList *qmp_query_memory_devices(Error
> **errp)
> MemoryDeviceInfoList *head = NULL;
> MemoryDeviceInfoList **prev = &head;
>
> - qmp_pc_dimm_device_list(qdev_get_machine(), &prev);
> + qmp_dimm_device_list(qdev_get_machine(), &prev);
>
> return head;
> }
> diff --git a/stubs/qmp_pc_dimm_device_list.c b/stubs/qmp_pc_dimm_device_list.c
> index b584bd8..b2704c6 100644
> --- a/stubs/qmp_pc_dimm_device_list.c
> +++ b/stubs/qmp_pc_dimm_device_list.c
> @@ -1,7 +1,7 @@
> #include "qom/object.h"
> #include "hw/mem/pc-dimm.h"
>
> -int qmp_pc_dimm_device_list(Object *obj, void *opaque)
> +int qmp_dimm_device_list(Object *obj, void *opaque)
> {
> return 0;
> }
> diff --git a/trace-events b/trace-events
> index bdfe79f..d1f258d 100644
> --- a/trace-events
> +++ b/trace-events
> @@ -1652,12 +1652,12 @@ mhp_acpi_write_ost_ev(uint32_t slot, uint32_t ev)
> "slot[0x%"PRIx32"] OST EVENT:
> mhp_acpi_write_ost_status(uint32_t slot, uint32_t st) "slot[0x%"PRIx32"] OST
> STATUS: 0x%"PRIx32
> mhp_acpi_clear_insert_evt(uint32_t slot) "slot[0x%"PRIx32"] clear insert
> event"
> mhp_acpi_clear_remove_evt(uint32_t slot) "slot[0x%"PRIx32"] clear remove
> event"
> -mhp_acpi_pc_dimm_deleted(uint32_t slot) "slot[0x%"PRIx32"] pc-dimm deleted"
> -mhp_acpi_pc_dimm_delete_failed(uint32_t slot) "slot[0x%"PRIx32"] pc-dimm
> delete failed"
> +mhp_acpi_dimm_deleted(uint32_t slot) "slot[0x%"PRIx32"] dimm deleted"
> +mhp_acpi_dimm_delete_failed(uint32_t slot) "slot[0x%"PRIx32"] dimm delete
> failed"
>
> # hw/i386/pc.c
> -mhp_pc_dimm_assigned_slot(int slot) "0x%d"
> -mhp_pc_dimm_assigned_address(uint64_t addr) "0x%"PRIx64
> +mhp_dimm_assigned_slot(int slot) "0x%d"
> +mhp_dimm_assigned_address(uint64_t addr) "0x%"PRIx64
>
> # target-s390x/kvm.c
> kvm_enable_cmma(int rc) "CMMA: enabling with result code %d"
> --
> 1.8.3.1
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v6 14/33] pc-dimm: drop the prefix of pc-dimm,
Michael S. Tsirkin <=