Re: [Qemu-devel] [PATCH 5/5] qdev/compat: virtio-net-pci 0.10 compatibility.

["Michael S. Tsirkin" <mst@redhat.com>]

On Thu, Jul 09, 2009 at 03:02:33PM +0200, Gerd Hoffmann wrote:
> 
> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
> ---
>  hw/pc.c         |    4 ++++
>  hw/virtio-pci.c |   13 ++++++++++++-
>  2 files changed, 16 insertions(+), 1 deletions(-)
> 
> diff --git a/hw/pc.c b/hw/pc.c
> index 0b7c24b..ecb618d 100644
> --- a/hw/pc.c
> +++ b/hw/pc.c
> @@ -1471,6 +1471,10 @@ static QEMUMachine pc_machine_v0_10 = {
>              .driver   = "virtio-console-pci",
>              .property = "class",
>              .value    = "0x0380", /* PCI_CLASS_DISPLAY_OTHER */
> +        },{
> +            .driver   = "virtio-net-pci",
> +            .property = "msi",
> +            .value    = "0",
>          },
>          { /* end of list */ }
>      },

The number of vectors is part of hardware config so it's not a good idea
to use a binary property here. Since <= vectors is not a legal msi
configuration, it's enough to implement a "vectors" property instead:
virtio already interprets zero vectors as no msi.  This also matches
command line/monitor syntax, which is a good thing.

Most virtio-net changes become then unnecessary.

-- 
MST