diff options
author | Andrey Ryabinin <[email protected]> | 2022-05-17 13:08:47 -0500 |
---|---|---|
committer | Michael S. Tsirkin <[email protected]> | 2022-05-31 12:45:10 -0400 |
commit | 15538ba5ffaa5cbdfd11161ded503b27f5c1f0af (patch) | |
tree | 2b0a5efd9fc85a50ccb869d978201a7897c99383 | |
parent | c5514758ddd9a774d879b098bf7364c45a76bed5 (diff) |
vhost_vsock: simplify vhost_vsock_flush()
vhost_vsock_flush() calls vhost_work_dev_flush(vsock->vqs[i].poll.dev)
before vhost_work_dev_flush(&vsock->dev). This seems pointless
as vsock->vqs[i].poll.dev is the same as &vsock->dev and several flushes
in a row doesn't do anything useful, one is just enough.
Signed-off-by: Andrey Ryabinin <[email protected]>
Reviewed-by: Stefano Garzarella <[email protected]>
Signed-off-by: Mike Christie <[email protected]>
Acked-by: Jason Wang <[email protected]>
Message-Id: <[email protected]>
Signed-off-by: Michael S. Tsirkin <[email protected]>
-rw-r--r-- | drivers/vhost/vsock.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/drivers/vhost/vsock.c b/drivers/vhost/vsock.c index a4c8ae92a0fb..96be63697117 100644 --- a/drivers/vhost/vsock.c +++ b/drivers/vhost/vsock.c @@ -705,11 +705,6 @@ out: static void vhost_vsock_flush(struct vhost_vsock *vsock) { - int i; - - for (i = 0; i < ARRAY_SIZE(vsock->vqs); i++) - if (vsock->vqs[i].handle_kick) - vhost_work_dev_flush(vsock->vqs[i].poll.dev); vhost_work_dev_flush(&vsock->dev); } |