When kthread_run fails, it returns ERR, not NULL.
Therefore, NULL checking is redundant. 
(https://www.kernel.org/doc/htmldocs/device-drivers/API-kthread-run.html)

Signed-off-by: Insu Yun <[email protected]>
---
 drivers/block/aoe/aoecmd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/block/aoe/aoecmd.c b/drivers/block/aoe/aoecmd.c
index d048d20..4213114 100644
--- a/drivers/block/aoe/aoecmd.c
+++ b/drivers/block/aoe/aoecmd.c
@@ -1327,7 +1327,7 @@ aoe_ktstart(struct ktstate *k)
 
        init_completion(&k->rendez);
        task = kthread_run(kthread, k, "%s", k->name);
-       if (task == NULL || IS_ERR(task))
+       if (IS_ERR(task))
                return -ENOMEM;
        k->task = task;
        wait_for_completion(&k->rendez); /* allow kthread to start */
-- 
1.9.1

Reply via email to