mirror of
https://github.com/qemu/qemu.git
synced 2025-08-14 11:43:46 +00:00
pc-dimm: pass in the machine and to the MemoryHotplugState
We use the machine internally either way, so let's just pass it in then. Signed-off-by: David Hildenbrand <david@redhat.com> Message-Id: <20180423165126.15441-5-david@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
This commit is contained in:
parent
acc7fa17e6
commit
bd6c3e4a49
@ -1715,7 +1715,7 @@ static void pc_dimm_plug(HotplugHandler *hotplug_dev,
|
|||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
pc_dimm_memory_plug(dev, MACHINE(pcms)->device_memory, align, &local_err);
|
pc_dimm_memory_plug(dev, MACHINE(pcms), align, &local_err);
|
||||||
if (local_err) {
|
if (local_err) {
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
@ -1775,7 +1775,7 @@ static void pc_dimm_unplug(HotplugHandler *hotplug_dev,
|
|||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
pc_dimm_memory_unplug(dev, MACHINE(pcms)->device_memory);
|
pc_dimm_memory_unplug(dev, MACHINE(pcms));
|
||||||
object_unparent(OBJECT(dev));
|
object_unparent(OBJECT(dev));
|
||||||
|
|
||||||
out:
|
out:
|
||||||
|
@ -36,11 +36,11 @@ typedef struct pc_dimms_capacity {
|
|||||||
Error **errp;
|
Error **errp;
|
||||||
} pc_dimms_capacity;
|
} pc_dimms_capacity;
|
||||||
|
|
||||||
void pc_dimm_memory_plug(DeviceState *dev, MemoryHotplugState *hpms,
|
void pc_dimm_memory_plug(DeviceState *dev, MachineState *machine,
|
||||||
uint64_t align, Error **errp)
|
uint64_t align, Error **errp)
|
||||||
{
|
{
|
||||||
int slot;
|
int slot;
|
||||||
MachineState *machine = MACHINE(qdev_get_machine());
|
MemoryHotplugState *hpms = machine->device_memory;
|
||||||
PCDIMMDevice *dimm = PC_DIMM(dev);
|
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);
|
||||||
@ -122,14 +122,14 @@ out:
|
|||||||
error_propagate(errp, local_err);
|
error_propagate(errp, local_err);
|
||||||
}
|
}
|
||||||
|
|
||||||
void pc_dimm_memory_unplug(DeviceState *dev, MemoryHotplugState *hpms)
|
void pc_dimm_memory_unplug(DeviceState *dev, MachineState *machine)
|
||||||
{
|
{
|
||||||
PCDIMMDevice *dimm = PC_DIMM(dev);
|
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);
|
||||||
MemoryRegion *mr = ddc->get_memory_region(dimm, &error_abort);
|
MemoryRegion *mr = ddc->get_memory_region(dimm, &error_abort);
|
||||||
|
|
||||||
memory_region_del_subregion(&hpms->mr, mr);
|
memory_region_del_subregion(&machine->device_memory->mr, mr);
|
||||||
vmstate_unregister_ram(vmstate_mr, dev);
|
vmstate_unregister_ram(vmstate_mr, dev);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -3153,7 +3153,7 @@ static void spapr_memory_plug(HotplugHandler *hotplug_dev, DeviceState *dev,
|
|||||||
align = memory_region_get_alignment(mr);
|
align = memory_region_get_alignment(mr);
|
||||||
size = memory_region_size(mr);
|
size = memory_region_size(mr);
|
||||||
|
|
||||||
pc_dimm_memory_plug(dev, MACHINE(ms)->device_memory, align, &local_err);
|
pc_dimm_memory_plug(dev, MACHINE(ms), align, &local_err);
|
||||||
if (local_err) {
|
if (local_err) {
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
@ -3174,7 +3174,7 @@ static void spapr_memory_plug(HotplugHandler *hotplug_dev, DeviceState *dev,
|
|||||||
return;
|
return;
|
||||||
|
|
||||||
out_unplug:
|
out_unplug:
|
||||||
pc_dimm_memory_unplug(dev, MACHINE(ms)->device_memory);
|
pc_dimm_memory_unplug(dev, MACHINE(ms));
|
||||||
out:
|
out:
|
||||||
error_propagate(errp, local_err);
|
error_propagate(errp, local_err);
|
||||||
}
|
}
|
||||||
@ -3311,7 +3311,7 @@ void spapr_lmb_release(DeviceState *dev)
|
|||||||
* Now that all the LMBs have been removed by the guest, call the
|
* Now that all the LMBs have been removed by the guest, call the
|
||||||
* pc-dimm unplug handler to cleanup up the pc-dimm device.
|
* pc-dimm unplug handler to cleanup up the pc-dimm device.
|
||||||
*/
|
*/
|
||||||
pc_dimm_memory_unplug(dev, MACHINE(spapr)->device_memory);
|
pc_dimm_memory_unplug(dev, MACHINE(spapr));
|
||||||
object_unparent(OBJECT(dev));
|
object_unparent(OBJECT(dev));
|
||||||
spapr_pending_dimm_unplugs_remove(spapr, ds);
|
spapr_pending_dimm_unplugs_remove(spapr, ds);
|
||||||
}
|
}
|
||||||
|
@ -84,7 +84,7 @@ uint64_t pc_dimm_get_free_addr(uint64_t address_space_start,
|
|||||||
int pc_dimm_get_free_slot(const int *hint, int max_slots, Error **errp);
|
int pc_dimm_get_free_slot(const int *hint, int max_slots, Error **errp);
|
||||||
|
|
||||||
uint64_t pc_existing_dimms_capacity(Error **errp);
|
uint64_t pc_existing_dimms_capacity(Error **errp);
|
||||||
void pc_dimm_memory_plug(DeviceState *dev, MemoryHotplugState *hpms,
|
void pc_dimm_memory_plug(DeviceState *dev, MachineState *machine,
|
||||||
uint64_t align, Error **errp);
|
uint64_t align, Error **errp);
|
||||||
void pc_dimm_memory_unplug(DeviceState *dev, MemoryHotplugState *hpms);
|
void pc_dimm_memory_unplug(DeviceState *dev, MachineState *machine);
|
||||||
#endif
|
#endif
|
||||||
|
Loading…
Reference in New Issue
Block a user