[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v3 08/10] hw/i386/pc: Remove PC_MACHINE_DEVMEM_REGION_SIZE
From: |
David Hildenbrand |
Subject: |
[PATCH v3 08/10] hw/i386/pc: Remove PC_MACHINE_DEVMEM_REGION_SIZE |
Date: |
Thu, 1 Jun 2023 14:14:45 +0200 |
There are no remaining users in the tree. Libvirt never used that
property and a quick internet search revealed no other users.
Further, we renamed that property already in commit f2ffbe2b7dd0
("pc: rename "hotplug memory" terminology to "device memory"") without
anybody complaining.
So let's just get rid of it.
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: Richard Henderson <richard.henderson@linaro.org>
Cc: Eduardo Habkost <eduardo@habkost.net>
Cc: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: David Hildenbrand <david@redhat.com>
---
hw/i386/pc.c | 19 -------------------
include/hw/i386/pc.h | 1 -
2 files changed, 20 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 3d7f991368..96334cf60d 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -1646,21 +1646,6 @@ static HotplugHandler
*pc_get_hotplug_handler(MachineState *machine,
return NULL;
}
-static void
-pc_machine_get_device_memory_region_size(Object *obj, Visitor *v,
- const char *name, void *opaque,
- Error **errp)
-{
- MachineState *ms = MACHINE(obj);
- int64_t value = 0;
-
- if (ms->device_memory) {
- value = memory_region_size(&ms->device_memory->mr);
- }
-
- visit_type_int(v, name, &value, errp);
-}
-
static void pc_machine_get_vmport(Object *obj, Visitor *v, const char *name,
void *opaque, Error **errp)
{
@@ -1980,10 +1965,6 @@ static void pc_machine_class_init(ObjectClass *oc, void
*data)
object_class_property_set_description(oc, PC_MACHINE_MAX_RAM_BELOW_4G,
"Maximum ram below the 4G boundary (32bit boundary)");
- object_class_property_add(oc, PC_MACHINE_DEVMEM_REGION_SIZE, "int",
- pc_machine_get_device_memory_region_size, NULL,
- NULL, NULL);
-
object_class_property_add(oc, PC_MACHINE_VMPORT, "OnOffAuto",
pc_machine_get_vmport, pc_machine_set_vmport,
NULL, NULL);
diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
index c661e9cc80..6c9ad2d132 100644
--- a/include/hw/i386/pc.h
+++ b/include/hw/i386/pc.h
@@ -60,7 +60,6 @@ typedef struct PCMachineState {
#define PC_MACHINE_ACPI_DEVICE_PROP "acpi-device"
#define PC_MACHINE_MAX_RAM_BELOW_4G "max-ram-below-4g"
-#define PC_MACHINE_DEVMEM_REGION_SIZE "device-memory-region-size"
#define PC_MACHINE_VMPORT "vmport"
#define PC_MACHINE_SMBUS "smbus"
#define PC_MACHINE_SATA "sata"
--
2.40.1
- Re: [PATCH v3 00/10] memory-device: Some cleanups, (continued)
- [PATCH v3 02/10] memory-device: Introduce machine_memory_devices_init(), David Hildenbrand, 2023/06/01
- [PATCH v3 03/10] hw/arm/virt: Use machine_memory_devices_init(), David Hildenbrand, 2023/06/01
- [PATCH v3 04/10] hw/ppc/spapr: Use machine_memory_devices_init(), David Hildenbrand, 2023/06/01
- [PATCH v3 01/10] memory-device: Unify enabled vs. supported error messages, David Hildenbrand, 2023/06/01
- [PATCH v3 05/10] hw/loongarch/virt: Use machine_memory_devices_init(), David Hildenbrand, 2023/06/01
- [PATCH v3 06/10] hw/i386/pc: Use machine_memory_devices_init(), David Hildenbrand, 2023/06/01
- [PATCH v3 07/10] hw/i386/acpi-build: Rely on machine->device_memory when building SRAT, David Hildenbrand, 2023/06/01
- [PATCH v3 10/10] memory-device: Track used region size in DeviceMemoryState, David Hildenbrand, 2023/06/01
- [PATCH v3 08/10] hw/i386/pc: Remove PC_MACHINE_DEVMEM_REGION_SIZE,
David Hildenbrand <=
- [PATCH v3 09/10] memory-device: Refactor memory_device_pre_plug(), David Hildenbrand, 2023/06/01
- Re: [PATCH] memory-device: Track used region size in DeviceMemoryState, David Hildenbrand, 2023/06/01