summaryrefslogtreecommitdiff
path: root/drivers/infiniband/ulp
diff options
context:
space:
mode:
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>2014-08-14 08:56:22 +0800
committerRoland Dreier <roland@purestorage.com>2014-08-14 08:57:54 -0700
commitda05be290f5040e488ea4d13edc4da889f6f6929 (patch)
tree3dd1517388178e0c1660560f448bc58780f5d819 /drivers/infiniband/ulp
parent2aa1cf64aa576671b3593d73a6cb0a28b90ee444 (diff)
IB/srp: Fix return value check in srp_init_module()
In case of error, the function create_workqueue() returns NULL pointer not ERR_PTR(). The IS_ERR() test in the return value check should be replaced with NULL test. Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn> Acked-by: Bart Van Assche <bvanassche@acm.org> Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband/ulp')
-rw-r--r--drivers/infiniband/ulp/srp/ib_srp.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c
index 7f5ee7fc02a6..62d2a18e1b41 100644
--- a/drivers/infiniband/ulp/srp/ib_srp.c
+++ b/drivers/infiniband/ulp/srp/ib_srp.c
@@ -3320,8 +3320,8 @@ static int __init srp_init_module(void)
}
srp_remove_wq = create_workqueue("srp_remove");
- if (IS_ERR(srp_remove_wq)) {
- ret = PTR_ERR(srp_remove_wq);
+ if (!srp_remove_wq) {
+ ret = -ENOMEM;
goto out;
}