[Skiboot] [PATCH 1/3] opal/hmi: Fix double unlock of hmi lock in failure path.

Mahesh J Salgaonkar mahesh at linux.vnet.ibm.com
Wed Feb 13 04:47:37 AEDT 2019


From: Mahesh Salgaonkar <mahesh at linux.vnet.ibm.com>

unlock once and goto error_out.

Signed-off-by: Mahesh Salgaonkar <mahesh at linux.vnet.ibm.com>
---
 core/hmi.c |    6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/core/hmi.c b/core/hmi.c
index c01a2c3bf..7094b4ca5 100644
--- a/core/hmi.c
+++ b/core/hmi.c
@@ -1042,13 +1042,9 @@ static int handle_all_core_tfac_error(uint64_t tfmr, uint64_t *out_flags)
 			if (!recover_corrupt_tfmr()) {
 				unlock(&hmi_lock);
 				recover = 0;
+				goto error_out;
 			}
 
-		if (!recover) {
-			unlock(&hmi_lock);
-			goto error_out;
-		}
-
 		tfmr = mfspr(SPR_TFMR);
 
 		/* We could have got new thread errors in the meantime */



More information about the Skiboot mailing list