virtio-net: fix race between ndo_open() and virtio_device_ready()
authorJason Wang <jasowang@redhat.com>
Fri, 17 Jun 2022 07:29:49 +0000 (15:29 +0800)
committerMichael S. Tsirkin <mst@redhat.com>
Mon, 27 Jun 2022 12:02:59 +0000 (08:02 -0400)
commit50c0ada627f56c92f5953a8bf9158b045ad026a1
treed31a4d46395675b1e007389063beb794bf910d2d
parentc346dae4f3fbce51bbd4f2ec5e8c6f9b91e93163
virtio-net: fix race between ndo_open() and virtio_device_ready()

We currently call virtio_device_ready() after netdev
registration. Since ndo_open() can be called immediately
after register_netdev, this means there exists a race between
ndo_open() and virtio_device_ready(): the driver may start to use the
device before DRIVER_OK which violates the spec.

Fix this by switching to use register_netdevice() and protect the
virtio_device_ready() with rtnl_lock() to make sure ndo_open() can
only be called after virtio_device_ready().

Fixes: 4baf1e33d0842 ("virtio_net: enable VQs early")
Signed-off-by: Jason Wang <jasowang@redhat.com>
Message-Id: <20220617072949.30734-1-jasowang@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
drivers/net/virtio_net.c