dm: add cond_resched() to dm_wq_requeue_work()
authorMike Snitzer <snitzer@kernel.org>
Thu, 16 Feb 2023 17:10:05 +0000 (12:10 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 10 Mar 2023 08:34:21 +0000 (09:34 +0100)
commit f77692d65d54665d81815349cc727baa85e8b71d upstream.

Otherwise the while() loop in dm_wq_requeue_work() can result in a
"dead loop" on systems that have preemption disabled. This is
particularly problematic on single cpu systems.

Fixes: 8b211aaccb915 ("dm: add two stage requeue mechanism")
Cc: stable@vger.kernel.org
Signed-off-by: Mike Snitzer <snitzer@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/md/dm.c

index 2124416..d727ed9 100644 (file)
@@ -1020,6 +1020,7 @@ static void dm_wq_requeue_work(struct work_struct *work)
                io->next = NULL;
                __dm_io_complete(io, false);
                io = next;
+               cond_resched();
        }
 }