down_trylock() is the opposite of mutex_trylock(). It returns zero on success and one on failure. Fixes: f4ef95631ebf ('dm thin: do not run the worker thread if pool is suspended') Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/drivers/md/dm-thin.c b/drivers/md/dm-thin.c index 6cbe9d2e0385..7d8e03c21fa9 100644 --- a/drivers/md/dm-thin.c +++ b/drivers/md/dm-thin.c @@ -237,7 +237,7 @@ struct thin_c { */ static void wake_worker(struct pool *pool) { - if (down_trylock(&pool->queue_work_lock)) { + if (!down_trylock(&pool->queue_work_lock)) { queue_work(pool->wq, &pool->worker); up(&pool->queue_work_lock); } -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel