OSDN Git Service

vhost: Fix worker hangs due to missed wake up calls
authorMike Christie <michael.christie@oracle.com>
Wed, 7 Jun 2023 19:23:38 +0000 (14:23 -0500)
committerMichael S. Tsirkin <mst@redhat.com>
Thu, 8 Jun 2023 19:43:09 +0000 (15:43 -0400)
commit4b13cbef797048fbb525f8c635a5279e9d209d93
tree59de4d4c9dc49411fe09d36c1390f58f08e7ad36
parenta284f09effea1908db7985dbfb5458c9100038d8
vhost: Fix worker hangs due to missed wake up calls

We can race where we have added work to the work_list, but
vhost_task_fn has passed that check but not yet set us into
TASK_INTERRUPTIBLE. wake_up_process will see us in TASK_RUNNING and
just return.

This bug was intoduced in commit f9010dbdce91 ("fork, vhost: Use
CLONE_THREAD to fix freezer/ps regression") when I moved the setting
of TASK_INTERRUPTIBLE to simplfy the code and avoid get_signal from
logging warnings about being in the wrong state. This moves the setting
of TASK_INTERRUPTIBLE back to before we test if we need to stop the
task to avoid a possible race there as well. We then have vhost_worker
set TASK_RUNNING if it finds work similar to before.

Fixes: f9010dbdce91 ("fork, vhost: Use CLONE_THREAD to fix freezer/ps regression")
Signed-off-by: Mike Christie <michael.christie@oracle.com>
Message-Id: <20230607192338.6041-3-michael.christie@oracle.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
drivers/vhost/vhost.c
kernel/vhost_task.c