mirror of
https://gitlab.com/qemu-project/qemu
synced 2024-11-05 20:35:44 +00:00
spapr: Introduce spapr_drc_reset_all()
No need to expose the way DRCs are traversed outside of spapr_drc.c. Signed-off-by: Greg Kurz <groug@kaod.org> Message-Id: <20201218103400.689660-4-groug@kaod.org> Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com> Tested-by: Daniel Henrique Barboza <danielhb413@gmail.com> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
parent
930ef3b5c2
commit
babb819f94
3 changed files with 43 additions and 34 deletions
|
@ -949,6 +949,37 @@ out:
|
|||
return ret;
|
||||
}
|
||||
|
||||
void spapr_drc_reset_all(SpaprMachineState *spapr)
|
||||
{
|
||||
Object *drc_container;
|
||||
ObjectProperty *prop;
|
||||
ObjectPropertyIterator iter;
|
||||
|
||||
drc_container = container_get(object_get_root(), DRC_CONTAINER_PATH);
|
||||
restart:
|
||||
object_property_iter_init(&iter, drc_container);
|
||||
while ((prop = object_property_iter_next(&iter))) {
|
||||
SpaprDrc *drc;
|
||||
|
||||
if (!strstart(prop->type, "link<", NULL)) {
|
||||
continue;
|
||||
}
|
||||
drc = SPAPR_DR_CONNECTOR(object_property_get_link(drc_container,
|
||||
prop->name,
|
||||
&error_abort));
|
||||
|
||||
/*
|
||||
* This will complete any pending plug/unplug requests.
|
||||
* In case of a unplugged PHB or PCI bridge, this will
|
||||
* cause some DRCs to be destroyed and thus potentially
|
||||
* invalidate the iterator.
|
||||
*/
|
||||
if (spapr_drc_reset(drc)) {
|
||||
goto restart;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/*
|
||||
* RTAS calls
|
||||
*/
|
||||
|
|
|
@ -1632,39 +1632,6 @@ static uint32_t cas_check_pvr(PowerPCCPU *cpu, uint32_t max_compat,
|
|||
return best_compat;
|
||||
}
|
||||
|
||||
static void spapr_handle_transient_dev_before_cas(SpaprMachineState *spapr)
|
||||
{
|
||||
Object *drc_container;
|
||||
ObjectProperty *prop;
|
||||
ObjectPropertyIterator iter;
|
||||
|
||||
drc_container = container_get(object_get_root(), "/dr-connector");
|
||||
restart:
|
||||
object_property_iter_init(&iter, drc_container);
|
||||
while ((prop = object_property_iter_next(&iter))) {
|
||||
SpaprDrc *drc;
|
||||
|
||||
if (!strstart(prop->type, "link<", NULL)) {
|
||||
continue;
|
||||
}
|
||||
drc = SPAPR_DR_CONNECTOR(object_property_get_link(drc_container,
|
||||
prop->name,
|
||||
&error_abort));
|
||||
|
||||
/*
|
||||
* This will complete any pending plug/unplug requests.
|
||||
* In case of a unplugged PHB or PCI bridge, this will
|
||||
* cause some DRCs to be destroyed and thus potentially
|
||||
* invalidate the iterator.
|
||||
*/
|
||||
if (spapr_drc_reset(drc)) {
|
||||
goto restart;
|
||||
}
|
||||
}
|
||||
|
||||
spapr_clear_pending_hotplug_events(spapr);
|
||||
}
|
||||
|
||||
target_ulong do_client_architecture_support(PowerPCCPU *cpu,
|
||||
SpaprMachineState *spapr,
|
||||
target_ulong vec,
|
||||
|
@ -1822,7 +1789,12 @@ target_ulong do_client_architecture_support(PowerPCCPU *cpu,
|
|||
|
||||
spapr_irq_update_active_intc(spapr);
|
||||
|
||||
spapr_handle_transient_dev_before_cas(spapr);
|
||||
/*
|
||||
* Process all pending hot-plug/unplug requests now. An updated full
|
||||
* rendered FDT will be returned to the guest.
|
||||
*/
|
||||
spapr_drc_reset_all(spapr);
|
||||
spapr_clear_pending_hotplug_events(spapr);
|
||||
|
||||
/*
|
||||
* If spapr_machine_reset() did not set up a HPT but one is necessary
|
||||
|
|
|
@ -245,6 +245,12 @@ int spapr_dt_drc(void *fdt, int offset, Object *owner, uint32_t drc_type_mask);
|
|||
void spapr_drc_attach(SpaprDrc *drc, DeviceState *d);
|
||||
void spapr_drc_detach(SpaprDrc *drc);
|
||||
|
||||
/*
|
||||
* Reset all DRCs, causing pending hot-plug/unplug requests to complete.
|
||||
* Safely handles potential DRC removal (eg. PHBs or PCI bridges).
|
||||
*/
|
||||
void spapr_drc_reset_all(struct SpaprMachineState *spapr);
|
||||
|
||||
static inline bool spapr_drc_unplug_requested(SpaprDrc *drc)
|
||||
{
|
||||
return drc->unplug_requested;
|
||||
|
|
Loading…
Reference in a new issue