diff options
author | Josh Poimboeuf <jpoimboe@kernel.org> | 2023-02-24 17:50:01 +0100 |
---|---|---|
committer | Peter Zijlstra <peterz@infradead.org> | 2023-03-22 17:09:29 +0100 |
commit | 05bfb338fa8dd40b008ce443e397fc374f6bd107 (patch) | |
tree | d8808615de78a14bc5489b574be930de9d20c045 /drivers/vhost | |
parent | livepatch,sched: Add livepatch task switching to cond_resched() (diff) | |
download | linux-05bfb338fa8dd40b008ce443e397fc374f6bd107.tar.xz linux-05bfb338fa8dd40b008ce443e397fc374f6bd107.zip |
vhost: Fix livepatch timeouts in vhost_worker()
Livepatch timeouts were reported due to busy vhost_worker() kthreads.
Now that cond_resched() can do livepatch task switching, use
cond_resched() in vhost_worker(). That's the better way to
conditionally call schedule() anyway.
Reported-by: Seth Forshee (DigitalOcean) <sforshee@kernel.org>
Signed-off-by: Josh Poimboeuf <jpoimboe@kernel.org>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Reviewed-by: Petr Mladek <pmladek@suse.com>
Tested-by: Seth Forshee (DigitalOcean) <sforshee@kernel.org>
Link: https://lore.kernel.org/r/509f6ea6fe6505f0a75a66026ba531c765ef922f.1677257135.git.jpoimboe@kernel.org
Diffstat (limited to 'drivers/vhost')
-rw-r--r-- | drivers/vhost/vhost.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c index f11bdbe4c2c5..822fbdb54c4f 100644 --- a/drivers/vhost/vhost.c +++ b/drivers/vhost/vhost.c @@ -363,8 +363,7 @@ static int vhost_worker(void *data) kcov_remote_start_common(dev->kcov_handle); work->fn(work); kcov_remote_stop(); - if (need_resched()) - schedule(); + cond_resched(); } } kthread_unuse_mm(dev->mm); |