|
 |
9bac43 |
From 6984d8d007ed94067b37af90b1e20a0aad76a205 Mon Sep 17 00:00:00 2001
|
|
|
9bac43 |
From: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Date: Wed, 4 Oct 2017 05:40:11 +0200
|
|
|
9bac43 |
Subject: [PATCH 04/34] hw/ppc/spapr_drc.c: change spapr_drc_needed to use
|
|
|
9bac43 |
drc->dev
|
|
|
9bac43 |
|
|
|
9bac43 |
RH-Author: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Message-id: <20171004054014.14159-2-dgibson@redhat.com>
|
|
|
9bac43 |
Patchwork-id: 76799
|
|
|
9bac43 |
O-Subject: [RHEL-7.5 qemu-kvm-rhev PATCH 1/4] hw/ppc/spapr_drc.c: change spapr_drc_needed to use drc->dev
|
|
|
9bac43 |
Bugzilla: 1448344
|
|
|
9bac43 |
RH-Acked-by: Laurent Vivier <lvivier@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Thomas Huth <thuth@redhat.com>
|
|
|
9bac43 |
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
From: Daniel Henrique Barboza <danielhb@linux.vnet.ibm.com>
|
|
|
9bac43 |
|
|
|
9bac43 |
This patch makes a small fix in 'spapr_drc_needed' to change how we detect
|
|
|
9bac43 |
if a DRC has a device attached. Previously it used dr_entity_sense for this,
|
|
|
9bac43 |
which works for physical DRCs.
|
|
|
9bac43 |
|
|
|
9bac43 |
However, for logical DRCs, it didn't cover the case where a logical DRC has
|
|
|
9bac43 |
a drc->dev but the state is LOGICAL_UNUSABLE (e.g. a hotplugged CPU before
|
|
|
9bac43 |
CAS). In this case, the dr_entity_sense of this DRC returns UNUSABLE and the
|
|
|
9bac43 |
code was considering that there were no dev attached, making spapr_drc_needed
|
|
|
9bac43 |
return 'false' when in fact we would like to migrate the DRC.
|
|
|
9bac43 |
|
|
|
9bac43 |
Changing it to check for drc->dev instead works for all DRC types.
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: Daniel Henrique Barboza <danielhb@linux.vnet.ibm.com>
|
|
|
9bac43 |
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
|
|
|
9bac43 |
(cherry picked from commit c618e300eb2276996e7004100686768cf1445128)
|
|
|
9bac43 |
|
|
|
9bac43 |
Signed-off-by: David Gibson <dgibson@redhat.com>
|
|
|
9bac43 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
9bac43 |
---
|
|
|
9bac43 |
hw/ppc/spapr_drc.c | 3 +--
|
|
|
9bac43 |
1 file changed, 1 insertion(+), 2 deletions(-)
|
|
|
9bac43 |
|
|
|
9bac43 |
diff --git a/hw/ppc/spapr_drc.c b/hw/ppc/spapr_drc.c
|
|
|
9bac43 |
index 605697d..031ba7c 100644
|
|
|
9bac43 |
--- a/hw/ppc/spapr_drc.c
|
|
|
9bac43 |
+++ b/hw/ppc/spapr_drc.c
|
|
|
9bac43 |
@@ -464,10 +464,9 @@ static bool spapr_drc_needed(void *opaque)
|
|
|
9bac43 |
{
|
|
|
9bac43 |
sPAPRDRConnector *drc = (sPAPRDRConnector *)opaque;
|
|
|
9bac43 |
sPAPRDRConnectorClass *drck = SPAPR_DR_CONNECTOR_GET_CLASS(drc);
|
|
|
9bac43 |
- sPAPRDREntitySense value = drck->dr_entity_sense(drc);
|
|
|
9bac43 |
|
|
|
9bac43 |
/* If no dev is plugged in there is no need to migrate the DRC state */
|
|
|
9bac43 |
- if (value != SPAPR_DR_ENTITY_SENSE_PRESENT) {
|
|
|
9bac43 |
+ if (!drc->dev) {
|
|
|
9bac43 |
return false;
|
|
|
9bac43 |
}
|
|
|
9bac43 |
|
|
|
9bac43 |
--
|
|
|
9bac43 |
1.8.3.1
|
|
|
9bac43 |
|