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>

diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c
index 7f5ee7fc02a66a152fb4be14d26e6f021482acf8..62d2a18e1b419225b312326690ec624563f5aa7b 100644 (file)
--- 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;
        }