On Thu, Mar 14, 2024 at 11:24:33AM +0800, Jason Wang wrote:
> On Thu, Mar 14, 2024 at 3:52 AM Michael S. Tsirkin <m...@redhat.com> wrote:
> >
> > 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 <ch...@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 +++++++++++++++++++-
>
> I think those modifications should belong to a separate file as it
> might conflict with virito features in the future.
>
> > > > 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.
>
> Right and this part, it's not a part of standard virtio.
>
> Thanks
I'm thinking of reverting this patch unless there's a resolution
soon, and reapplying later after the release.
> >
> > > > +
> > > > #endif /* VIRTIO_PCI_NO_MODERN */
> > > >
> > > > #endif
> >
