Re: [PATCH] hw/virtio: Add support for VDPA network simulation devices

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

On Wed, Mar 13, 2024 at 07:51:08PM +0100, Thomas Weißschuh wrote:
> On 2024-02-21 15:38:02+0800, Hao Chen wrote:
> > This patch adds support for VDPA network simulation devices.
> > The device is developed based on virtio-net and tap backend,
> > and supports hardware live migration function.
> > 
> > For more details, please refer to "docs/system/devices/vdpa-net.rst"
> > 
> > Signed-off-by: Hao Chen <chenh@yusur.tech>
> > ---
> >  MAINTAINERS                                 |   5 +
> >  docs/system/device-emulation.rst            |   1 +
> >  docs/system/devices/vdpa-net.rst            | 121 +++++++++++++
> >  hw/net/virtio-net.c                         |  16 ++
> >  hw/virtio/virtio-pci.c                      | 189 +++++++++++++++++++-
> >  hw/virtio/virtio.c                          |  39 ++++
> >  include/hw/virtio/virtio-pci.h              |   5 +
> >  include/hw/virtio/virtio.h                  |  19 ++
> >  include/standard-headers/linux/virtio_pci.h |   7 +
> >  9 files changed, 399 insertions(+), 3 deletions(-)
> >  create mode 100644 docs/system/devices/vdpa-net.rst
> 
> [..]
> 
> > diff --git a/include/standard-headers/linux/virtio_pci.h b/include/standard-headers/linux/virtio_pci.h
> > index b7fdfd0668..fb5391cef6 100644
> > --- a/include/standard-headers/linux/virtio_pci.h
> > +++ b/include/standard-headers/linux/virtio_pci.h
> > @@ -216,6 +216,13 @@ struct virtio_pci_cfg_cap {
> >  #define VIRTIO_PCI_COMMON_Q_NDATA	56
> >  #define VIRTIO_PCI_COMMON_Q_RESET	58
> >  
> > +#define LM_LOGGING_CTRL                 0
> > +#define LM_BASE_ADDR_LOW                4
> > +#define LM_BASE_ADDR_HIGH               8
> > +#define LM_END_ADDR_LOW                 12
> > +#define LM_END_ADDR_HIGH                16
> > +#define LM_VRING_STATE_OFFSET           0x20
> 
> These changes are not in upstream Linux and will be undone by
> ./scripts/update-linux-headers.sh.
> 
> Are they intentionally in this header?


Good point. Pls move.

> > +
> >  #endif /* VIRTIO_PCI_NO_MODERN */
> >  
> >  #endif