diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2015-09-16 06:22:51 (GMT) |
---|---|---|
committer | Doug Ledford <dledford@redhat.com> | 2015-09-18 15:28:47 (GMT) |
commit | 951842b0540d2ed49ae29ba968adc496baf46556 (patch) | |
tree | 62a4d17e2063cf9a221098072a489413aaf95050 /drivers/staging | |
parent | 50b19729ced72cfa8bb1c44fed9203f395f13991 (diff) | |
download | linux-951842b0540d2ed49ae29ba968adc496baf46556.tar.xz |
IB/hfi1: fix a locking bug
mutex_trylock() returns zero on failure, not EBUSY.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers/staging')
-rw-r--r-- | drivers/staging/rdma/hfi1/chip.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/staging/rdma/hfi1/chip.c b/drivers/staging/rdma/hfi1/chip.c index 654eafe..aa58e59 100644 --- a/drivers/staging/rdma/hfi1/chip.c +++ b/drivers/staging/rdma/hfi1/chip.c @@ -2710,7 +2710,7 @@ int acquire_lcb_access(struct hfi1_devdata *dd, int sleep_ok) if (sleep_ok) { mutex_lock(&ppd->hls_lock); } else { - while (mutex_trylock(&ppd->hls_lock) == EBUSY) + while (!mutex_trylock(&ppd->hls_lock)) udelay(1); } @@ -2758,7 +2758,7 @@ int release_lcb_access(struct hfi1_devdata *dd, int sleep_ok) if (sleep_ok) { mutex_lock(&dd->pport->hls_lock); } else { - while (mutex_trylock(&dd->pport->hls_lock) == EBUSY) + while (!mutex_trylock(&dd->pport->hls_lock)) udelay(1); } |