diff options
| author |   Michael S. Tsirkin <mst@redhat.com> | 2014-10-15 10:22:31 +1030 |
|---|---|---|
| committer |   Rusty Russell <rusty@rustcorp.com.au> | 2014-10-15 10:25:04 +1030 |
| commit | 64b4cc3911fe8284dfb3cfdb8065c100b818bab8 (patch) | |
| tree | 3c884644f7afc11d2c157bcc3b6518529adb2012 /net/9p | |
| parent | virtio_console: enable VQs early (diff) | |
| download | linux-dev-64b4cc3911fe8284dfb3cfdb8065c100b818bab8.tar.xz linux-dev-64b4cc3911fe8284dfb3cfdb8065c100b818bab8.zip | |
9p/trans_virtio: enable VQs early
virtio spec requires drivers to set DRIVER_OK before using VQs.
This is set automatically after probe returns, but virtio 9p device
adds self to channel list within probe, at which point VQ can be
used in violation of the spec.
To fix, call virtio_device_ready before using VQs.
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'net/9p')
| -rw-r--r-- | net/9p/trans_virtio.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/9p/trans_virtio.c b/net/9p/trans_virtio.c index 6940d8fe8971..daa749c8b3fb 100644 --- a/net/9p/trans_virtio.c +++ b/net/9p/trans_virtio.c @@ -575,6 +575,8 @@ static int p9_virtio_probe(struct virtio_device *vdev) /* Ceiling limit to avoid denial of service attacks */ chan->p9_max_pages = nr_free_buffer_pages()/4; + virtio_device_ready(vdev); + mutex_lock(&virtio_9p_lock); list_add_tail(&chan->chan_list, &virtio_chan_list); mutex_unlock(&virtio_9p_lock); |