2021-07-20 03:43:21

by Xianting Tian

Subject: [PATCH] vsock/virtio: set vsock frontend ready in virtio_vsock_probe()

Add the missed virtio_device_ready() to set vsock frontend ready.

Signed-off-by: Xianting Tian <[email protected]>
---
net/vmw_vsock/virtio_transport.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/net/vmw_vsock/virtio_transport.c b/net/vmw_vsock/virtio_transport.c
index e0c2c992a..eb4c607c4 100644
--- a/net/vmw_vsock/virtio_transport.c
+++ b/net/vmw_vsock/virtio_transport.c
@@ -637,6 +637,8 @@ static int virtio_vsock_probe(struct virtio_device *vdev)
vdev->priv = vsock;
rcu_assign_pointer(the_virtio_vsock, vsock);

+ virtio_device_ready(vdev);
+
mutex_unlock(&the_virtio_vsock_mutex);

return 0;
--
2.17.1

2021-07-20 06:28:07

by Jason Wang

Subject: Re: [PATCH] vsock/virtio: set vsock frontend ready in virtio_vsock_probe()

?? 2021/7/20 ????11:40, Xianting Tian д??:
> Add the missed virtio_device_ready() to set vsock frontend ready.
>
> Signed-off-by: Xianting Tian <[email protected]>
> ---
> net/vmw_vsock/virtio_transport.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/net/vmw_vsock/virtio_transport.c b/net/vmw_vsock/virtio_transport.c
> index e0c2c992a..eb4c607c4 100644
> --- a/net/vmw_vsock/virtio_transport.c
> +++ b/net/vmw_vsock/virtio_transport.c
> @@ -637,6 +637,8 @@ static int virtio_vsock_probe(struct virtio_device *vdev)
> vdev->priv = vsock;
> rcu_assign_pointer(the_virtio_vsock, vsock);
>
> + virtio_device_ready(vdev);
> +
> mutex_unlock(&the_virtio_vsock_mutex);


It's better to do this after the mutex.

Thanks


>
> return 0;