ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

pc-dimm: pass PCDIMMDevice to pc_dimm_.*plug 

We're plugging/unplugging a PCDIMMDevice, so directly pass this type
instead of a more generic DeviceState.

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
Acked-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Igor Mammedov <imammedo@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: David Hildenbrand <david@redhat.com>
Message-Id: <20181005092024.14344-5-david@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
This commit is contained in:
David Hildenbrand 2018-10-05 11:20:12 +02:00 committed by Eduardo Habkost
parent f99d84b1fc
commit fd3416f5eb
4 changed files with 21 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
hw/i386/pc.c
View File

@ -1704,7 +1704,7 @@ static void pc_memory_pre_plug(HotplugHandler *hotplug_dev, DeviceState *dev,
return; return;
} }
pc_dimm_pre_plug(dev, MACHINE(hotplug_dev), pc_dimm_pre_plug(PC_DIMM(dev), MACHINE(hotplug_dev),
pcmc->enforce_aligned_dimm ? NULL : &legacy_align, errp); pcmc->enforce_aligned_dimm ? NULL : &legacy_align, errp);
} }
@ -1716,7 +1716,7 @@ static void pc_memory_plug(HotplugHandler *hotplug_dev,
PCMachineState *pcms = PC_MACHINE(hotplug_dev); PCMachineState *pcms = PC_MACHINE(hotplug_dev);
bool is_nvdimm = object_dynamic_cast(OBJECT(dev), TYPE_NVDIMM); bool is_nvdimm = object_dynamic_cast(OBJECT(dev), TYPE_NVDIMM);
pc_dimm_plug(dev, MACHINE(pcms), &local_err); pc_dimm_plug(PC_DIMM(dev), MACHINE(pcms), &local_err);
if (local_err) { if (local_err) {
goto out; goto out;
} }
@ -1776,7 +1776,7 @@ static void pc_memory_unplug(HotplugHandler *hotplug_dev,
goto out; goto out;
} }
pc_dimm_unplug(dev, MACHINE(pcms)); pc_dimm_unplug(PC_DIMM(dev), MACHINE(pcms));
object_unparent(OBJECT(dev)); object_unparent(OBJECT(dev));
out: out:

24
hw/mem/pc-dimm.c
View File

@ -29,24 +29,24 @@
static int pc_dimm_get_free_slot(const int *hint, int max_slots, Error **errp); static int pc_dimm_get_free_slot(const int *hint, int max_slots, Error **errp);
void pc_dimm_pre_plug(DeviceState *dev, MachineState *machine, void pc_dimm_pre_plug(PCDIMMDevice *dimm, MachineState *machine,
const uint64_t *legacy_align, Error **errp) const uint64_t *legacy_align, Error **errp)
{ {
PCDIMMDevice *dimm = PC_DIMM(dev);
PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm); PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm);
Error *local_err = NULL; Error *local_err = NULL;
MemoryRegion *mr; MemoryRegion *mr;
uint64_t addr, align; uint64_t addr, align;
int slot; int slot;
slot = object_property_get_int(OBJECT(dev), PC_DIMM_SLOT_PROP, slot = object_property_get_int(OBJECT(dimm), PC_DIMM_SLOT_PROP,
&error_abort); &error_abort);
slot = pc_dimm_get_free_slot(slot == PC_DIMM_UNASSIGNED_SLOT ? NULL : &slot, slot = pc_dimm_get_free_slot(slot == PC_DIMM_UNASSIGNED_SLOT ? NULL : &slot,
machine->ram_slots, &local_err); machine->ram_slots, &local_err);
if (local_err) { if (local_err) {
goto out; goto out;
} }
object_property_set_int(OBJECT(dev), slot, PC_DIMM_SLOT_PROP, &error_abort); object_property_set_int(OBJECT(dimm), slot, PC_DIMM_SLOT_PROP,
&error_abort);
trace_mhp_pc_dimm_assigned_slot(slot); trace_mhp_pc_dimm_assigned_slot(slot);
mr = ddc->get_memory_region(dimm, &local_err); mr = ddc->get_memory_region(dimm, &local_err);
@ -55,7 +55,7 @@ void pc_dimm_pre_plug(DeviceState *dev, MachineState *machine,
} }
align = legacy_align ? *legacy_align : memory_region_get_alignment(mr); align = legacy_align ? *legacy_align : memory_region_get_alignment(mr);
addr = object_property_get_uint(OBJECT(dev), PC_DIMM_ADDR_PROP, addr = object_property_get_uint(OBJECT(dimm), PC_DIMM_ADDR_PROP,
&error_abort); &error_abort);
addr = memory_device_get_free_addr(machine, !addr ? NULL : &addr, align, addr = memory_device_get_free_addr(machine, !addr ? NULL : &addr, align,
memory_region_size(mr), &local_err); memory_region_size(mr), &local_err);
@ -63,38 +63,36 @@ void pc_dimm_pre_plug(DeviceState *dev, MachineState *machine,
goto out; goto out;
} }
trace_mhp_pc_dimm_assigned_address(addr); trace_mhp_pc_dimm_assigned_address(addr);
object_property_set_uint(OBJECT(dev), addr, PC_DIMM_ADDR_PROP, object_property_set_uint(OBJECT(dimm), addr, PC_DIMM_ADDR_PROP,
&error_abort); &error_abort);
out: out:
error_propagate(errp, local_err); error_propagate(errp, local_err);
} }
void pc_dimm_plug(DeviceState *dev, MachineState *machine, Error **errp) void pc_dimm_plug(PCDIMMDevice *dimm, MachineState *machine, Error **errp)
{ {
PCDIMMDevice *dimm = PC_DIMM(dev);
PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm); PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm);
MemoryRegion *vmstate_mr = ddc->get_vmstate_memory_region(dimm, MemoryRegion *vmstate_mr = ddc->get_vmstate_memory_region(dimm,
&error_abort); &error_abort);
MemoryRegion *mr = ddc->get_memory_region(dimm, &error_abort); MemoryRegion *mr = ddc->get_memory_region(dimm, &error_abort);
uint64_t addr; uint64_t addr;
addr = object_property_get_uint(OBJECT(dev), PC_DIMM_ADDR_PROP, addr = object_property_get_uint(OBJECT(dimm), PC_DIMM_ADDR_PROP,
&error_abort); &error_abort);
memory_device_plug_region(machine, mr, addr); memory_device_plug_region(machine, mr, addr);
vmstate_register_ram(vmstate_mr, dev); vmstate_register_ram(vmstate_mr, DEVICE(dimm));
} }
void pc_dimm_unplug(DeviceState *dev, MachineState *machine) void pc_dimm_unplug(PCDIMMDevice *dimm, MachineState *machine)
{ {
PCDIMMDevice *dimm = PC_DIMM(dev);
PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm); PCDIMMDeviceClass *ddc = PC_DIMM_GET_CLASS(dimm);
MemoryRegion *vmstate_mr = ddc->get_vmstate_memory_region(dimm, MemoryRegion *vmstate_mr = ddc->get_vmstate_memory_region(dimm,
&error_abort); &error_abort);
MemoryRegion *mr = ddc->get_memory_region(dimm, &error_abort); MemoryRegion *mr = ddc->get_memory_region(dimm, &error_abort);
memory_device_unplug_region(machine, mr); memory_device_unplug_region(machine, mr);
vmstate_unregister_ram(vmstate_mr, dev); vmstate_unregister_ram(vmstate_mr, DEVICE(dimm));
} }
static int pc_dimm_slot2bitmap(Object *obj, void *opaque) static int pc_dimm_slot2bitmap(Object *obj, void *opaque)

8
hw/ppc/spapr.c
View File

@ -3135,7 +3135,7 @@ static void spapr_memory_plug(HotplugHandler *hotplug_dev, DeviceState *dev,
size = memory_region_size(mr); size = memory_region_size(mr);
pc_dimm_plug(dev, MACHINE(ms), &local_err); pc_dimm_plug(dimm, MACHINE(ms), &local_err);
if (local_err) { if (local_err) {
goto out; goto out;
} }
@ -3158,7 +3158,7 @@ static void spapr_memory_plug(HotplugHandler *hotplug_dev, DeviceState *dev,
return; return;
out_unplug: out_unplug:
pc_dimm_unplug(dev, MACHINE(ms)); pc_dimm_unplug(dimm, MACHINE(ms));
out: out:
error_propagate(errp, local_err); error_propagate(errp, local_err);
} }
@ -3202,7 +3202,7 @@ static void spapr_memory_pre_plug(HotplugHandler *hotplug_dev, DeviceState *dev,
return; return;
} }
pc_dimm_pre_plug(dev, MACHINE(hotplug_dev), NULL, errp); pc_dimm_pre_plug(dimm, MACHINE(hotplug_dev), NULL, errp);
} }
struct sPAPRDIMMState { struct sPAPRDIMMState {
@ -3314,7 +3314,7 @@ static void spapr_memory_unplug(HotplugHandler *hotplug_dev, DeviceState *dev)
sPAPRMachineState *spapr = SPAPR_MACHINE(hotplug_dev); sPAPRMachineState *spapr = SPAPR_MACHINE(hotplug_dev);
sPAPRDIMMState *ds = spapr_pending_dimm_unplugs_find(spapr, PC_DIMM(dev)); sPAPRDIMMState *ds = spapr_pending_dimm_unplugs_find(spapr, PC_DIMM(dev));
pc_dimm_unplug(dev, MACHINE(hotplug_dev)); pc_dimm_unplug(PC_DIMM(dev), MACHINE(hotplug_dev));
object_unparent(OBJECT(dev)); object_unparent(OBJECT(dev));
spapr_pending_dimm_unplugs_remove(spapr, ds); spapr_pending_dimm_unplugs_remove(spapr, ds);
} }

6
include/hw/mem/pc-dimm.h
View File

@ -79,8 +79,8 @@ typedef struct PCDIMMDeviceClass {
Error **errp); Error **errp);
} PCDIMMDeviceClass; } PCDIMMDeviceClass;
void pc_dimm_pre_plug(DeviceState *dev, MachineState *machine, void pc_dimm_pre_plug(PCDIMMDevice *dimm, MachineState *machine,
const uint64_t *legacy_align, Error **errp); const uint64_t *legacy_align, Error **errp);
void pc_dimm_plug(DeviceState *dev, MachineState *machine, Error **errp); void pc_dimm_plug(PCDIMMDevice *dimm, MachineState *machine, Error **errp);
void pc_dimm_unplug(DeviceState *dev, MachineState *machine); void pc_dimm_unplug(PCDIMMDevice *dimm, MachineState *machine);
#endif #endif