Re: [PATCH RFC] virtio_pci: fix virtio spec compliance on restore

From: Ben Hutchings
Date: Tue Sep 23 2014 - 21:27:59 EST


On Tue, 2014-09-23 at 13:32 +0300, Michael S. Tsirkin wrote:
> On restore, virtio pci does the following:
> + set features
> + init vqs etc - device can be used at this point!
> + set ACKNOWLEDGE,DRIVER and DRIVER_OK status bits
>
> This is in violation of the virtio spec, which
> requires the following order:
> - ACKNOWLEDGE
> - DRIVER
> - init vqs
> - DRIVER_OK
>
> Cc: stable@xxxxxxxxxxxxxxx
> Cc: Amit Shah <amit.shah@xxxxxxxxxx>
> Signed-off-by: Michael S. Tsirkin <mst@xxxxxxxxxx>
[...]

What concrete problem does this fix, such that it should be applied to
stable branches?

Ben.

--
Ben Hutchings
Everything should be made as simple as possible, but not simpler.
- Albert Einstein

Attachment: signature.asc
Description: This is a digitally signed message part