motorhead/qemu
1
0
Fork 0
mirror of https://github.com/Motorhead1991/qemu.git synced 2025-08-04 08:13:54 -06:00
Code Issues Projects Releases Packages Wiki Activity Actions

spapr: Call spapr_drc_reset() for all DRCs at CAS

Browse source 
Non-transient DRCs are either in the empty or the ready state,
which means spapr_drc_reset() doesn't change their state. It
is thus not needed to do any checking. Call spapr_drc_reset()
unconditionally and squash spapr_drc_transient() into its
only user, spapr_drc_needed().

Signed-off-by: Greg Kurz <groug@kaod.org>
Message-Id: <20201218103400.689660-2-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:
Greg Kurz 2020-12-18 11:33:55 +01:00 • committed by David Gibson
parent 30499fdd98
commit cd725bd748
3 changed files with 6 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

8
hw/ppc/spapr_drc.c
View file

@ -462,8 +462,9 @@ static const VMStateDescription vmstate_spapr_drc_unplug_requested = {
} }
}; };
bool spapr_drc_transient(SpaprDrc *drc) static bool spapr_drc_needed(void *opaque)
{ {
SpaprDrc *drc = opaque;
SpaprDrcClass *drck = SPAPR_DR_CONNECTOR_GET_CLASS(drc); SpaprDrcClass *drck = SPAPR_DR_CONNECTOR_GET_CLASS(drc);
/* /*
@ -483,11 +484,6 @@ bool spapr_drc_transient(SpaprDrc *drc)
spapr_drc_unplug_requested(drc); spapr_drc_unplug_requested(drc);
} }
static bool spapr_drc_needed(void *opaque)
{
return spapr_drc_transient(opaque);
}
static const VMStateDescription vmstate_spapr_drc = { static const VMStateDescription vmstate_spapr_drc = {
.name = "spapr_drc", .name = "spapr_drc",
.version_id = 1, .version_id = 1,

7
hw/ppc/spapr_hcall.c
View file

@ -1650,9 +1650,10 @@ static void spapr_handle_transient_dev_before_cas(SpaprMachineState *spapr)
prop->name, prop->name,
&error_abort)); &error_abort));
if (spapr_drc_transient(drc)) { /*
spapr_drc_reset(drc); * This will complete any pending plug/unplug requests.
} */
spapr_drc_reset(drc);
} }
spapr_clear_pending_hotplug_events(spapr); spapr_clear_pending_hotplug_events(spapr);

3
include/hw/ppc/spapr_drc.h
View file

@ -244,9 +244,6 @@ 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_attach(SpaprDrc *drc, DeviceState *d);
void spapr_drc_detach(SpaprDrc *drc); void spapr_drc_detach(SpaprDrc *drc);
/* Returns true if a hot plug/unplug request is pending */
bool spapr_drc_transient(SpaprDrc *drc);
static inline bool spapr_drc_unplug_requested(SpaprDrc *drc) static inline bool spapr_drc_unplug_requested(SpaprDrc *drc)
{ {
return drc->unplug_requested; return drc->unplug_requested;