[Skiboot] [PATCH] phb4: Use the return value of phb4_fenced() in phb4_get_diag_data()

Vaibhav Jain vaibhav at linux.ibm.com
Tue Aug 14 21:14:43 AEST 2018


From: Cyril Bur <cyril.bur at au1.ibm.com>

phb4_get_diag_data() checks the flags for the PHB4_AIB_FENCED after
having called phb4_fenced(). This information is returned by
phb4_fenced().

This patch was prompted by an unused return value warning in Coverity.

Fixes: CID 163734
Signed-off-by: Cyril Bur <cyril.bur at au1.ibm.com>
Signed-off-by: Stewart Smith <stewart at linux.ibm.com>
---
 hw/phb4.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/hw/phb4.c b/hw/phb4.c
index 4b1acd9e..5e07e2bc 100644
--- a/hw/phb4.c
+++ b/hw/phb4.c
@@ -3755,6 +3755,7 @@ static int64_t phb4_get_diag_data(struct phb *phb,
 				  void *diag_buffer,
 				  uint64_t diag_buffer_len)
 {
+	bool fenced;
 	struct phb4 *p = phb_to_phb4(phb);
 	struct OpalIoPhb4ErrorData *data = diag_buffer;
 
@@ -3767,10 +3768,10 @@ static int64_t phb4_get_diag_data(struct phb *phb,
 	 * Dummy check for fence so that phb4_read_phb_status knows
 	 * whether to use ASB or AIB
 	 */
-	phb4_fenced(p);
+	fenced = phb4_fenced(p);
 	phb4_read_phb_status(p, data);
 
-	if (!(p->flags & PHB4_AIB_FENCED))
+	if (!fenced)
 		phb4_eeh_dump_regs(p);
 
 	/*
-- 
2.17.1



More information about the Skiboot mailing list