Commit 15538ba5 authored by Andrey Ryabinin's avatar Andrey Ryabinin Committed by Michael S. Tsirkin
Browse files

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: default avatarAndrey Ryabinin <arbn@yandex-team.com>
Reviewed-by: default avatarStefano Garzarella <sgarzare@redhat.com>
Signed-off-by: default avatarMike Christie <michael.christie@oracle.com>
Acked-by: default avatarJason Wang <jasowang@redhat.com>
Message-Id: <20220517180850.198915-6-michael.christie@oracle.com>
Signed-off-by: default avatarMichael S. Tsirkin <mst@redhat.com>
parent c5514758

drivers/vhost/vsock.c

+0 −5
Original line number Original line Diff line number Diff line
@@ -705,11 +705,6 @@ static int vhost_vsock_dev_open(struct inode *inode, struct file *file) 




static void vhost_vsock_flush(struct vhost_vsock *vsock)

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);

	vhost_work_dev_flush(&vsock->dev);

}

}