[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