aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCédric Le Goater <clg@kaod.org>2018-03-21 18:12:37 +0100
committerStewart Smith <stewart@linux.vnet.ibm.com>2018-03-22 22:13:32 -0700
commitdcbf18c1f0e16111d5c86e845f37e8c2cad4f9ea (patch)
tree5e1e246bb955b9647cd0e89ada7997e7fb73c540
parent87145c6bad5bc6bb712ff1ae7ae3464037153424 (diff)
downloadskiboot-dcbf18c1f0e16111d5c86e845f37e8c2cad4f9ea.zip
skiboot-dcbf18c1f0e16111d5c86e845f37e8c2cad4f9ea.tar.gz
skiboot-dcbf18c1f0e16111d5c86e845f37e8c2cad4f9ea.tar.bz2
xive: fix opal_xive_set_vp_info() error path
In case of error, opal_xive_set_vp_info() will return without unlocking the xive object. This is most certainly a typo. Signed-off-by: Cédric Le Goater <clg@kaod.org> Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
-rw-r--r--hw/xive.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/xive.c b/hw/xive.c
index 998705e..196c070 100644
--- a/hw/xive.c
+++ b/hw/xive.c
@@ -4421,8 +4421,8 @@ static int64_t opal_xive_set_vp_info(uint64_t vp_id,
if (!(flags & OPAL_XIVE_VP_ENABLED))
xive_vpc_scrub_clean(x, blk, idx);
- unlock(&x->lock);
bail:
+ unlock(&x->lock);
return rc;
}