virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
  • [parent not found: <20220121202733.404989-2-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-3-eperezma@redhat.com>]
  • * Re: [PATCH 00/31] vDPA shadow virtqueue
           [not found] <20220121202733.404989-1-eperezma@redhat.com>
                       ` (2 preceding siblings ...)
           [not found] ` <20220121202733.404989-3-eperezma@redhat.com>
    @ 2022-01-28  6:02 ` Jason Wang
           [not found]   ` <CAJaqyWfWxQSJc3YMpF6g7VwZBN_ab0Z+1nXgWH1sg+uBaOYgBQ@mail.gmail.com>
           [not found] ` <20220121202733.404989-4-eperezma@redhat.com>
                       ` (15 subsequent siblings)
      19 siblings, 1 reply; 52+ messages in thread
    From: Jason Wang @ 2022-01-28  6:02 UTC (permalink / raw)
      To: Eugenio Pérez, qemu-devel
      Cc: Laurent Vivier, Parav Pandit, Cindy Lu, Michael S. Tsirkin,
    	Richard Henderson, Markus Armbruster, Gautam Dawar,
    	virtualization, Eduardo Habkost, Harpreet Singh Anand,
    	Xiao W Wang, Stefan Hajnoczi, Eli Cohen, Paolo Bonzini,
    	Zhu Lingshan, Eric Blake
    
    
    在 2022/1/22 上午4:27, Eugenio Pérez 写道:
    > This series enables shadow virtqueue (SVQ) for vhost-vdpa devices. This
    > is intended as a new method of tracking the memory the devices touch
    > during a migration process: Instead of relay on vhost device's dirty
    > logging capability, SVQ intercepts the VQ dataplane forwarding the
    > descriptors between VM and device. This way qemu is the effective
    > writer of guests memory, like in qemu's emulated virtio device
    > operation.
    >
    > When SVQ is enabled qemu offers a new virtual address space to the
    > device to read and write into, and it maps new vrings and the guest
    > memory in it. SVQ also intercepts kicks and calls between the device
    > and the guest. Used buffers relay would cause dirty memory being
    > tracked, but at this RFC SVQ is not enabled on migration automatically.
    >
    > Thanks of being a buffers relay system, SVQ can be used also to
    > communicate devices and drivers with different capabilities, like
    > devices that only support packed vring and not split and old guests with
    > no driver packed support.
    >
    > It is based on the ideas of DPDK SW assisted LM, in the series of
    > DPDK's https://patchwork.dpdk.org/cover/48370/ . However, these does
    > not map the shadow vq in guest's VA, but in qemu's.
    >
    > This version of SVQ is limited in the amount of features it can use with
    > guest and device, because this series is already very big otherwise.
    > Features like indirect or event_idx will be addressed in future series.
    >
    > SVQ needs to be enabled with cmdline parameter x-svq, like:
    >
    > -netdev type=vhost-vdpa,vhostdev=/dev/vhost-vdpa-0,id=vhost-vdpa0,x-svq=true
    >
    > In this version it cannot be enabled or disabled in runtime. Further
    > series will remove this limitation and will enable it only for migration
    > time.
    >
    > Some patches are intentionally very small to ease review, but they can
    > be squashed if preferred.
    >
    > Patches 1-10 prepares the SVQ and QEMU to support both guest to device
    > and device to guest notifications forwarding, with the extra qemu hop.
    > That part can be tested in isolation if cmdline change is reproduced.
    >
    > Patches from 11 to 18 implement the actual buffer forwarding, but with
    > no IOMMU support. It requires a vdpa device capable of addressing all
    > qemu vaddr.
    >
    > Patches 19 to 23 adds the iommu support, so the device with address
    > range limitations can access SVQ through this new virtual address space
    > created.
    >
    > The rest of the series add the last pieces needed for migration.
    >
    > Comments are welcome.
    
    
    I wonder the performance impact. So performance numbers are more than 
    welcomed.
    
    Thanks
    
    
    >
    > TODO:
    > * Event, indirect, packed, and other features of virtio.
    > * To separate buffers forwarding in its own AIO context, so we can
    >    throw more threads to that task and we don't need to stop the main
    >    event loop.
    > * Support virtio-net control vq.
    > * Proper documentation.
    >
    > Changes from v5 RFC:
    > * Remove dynamic enablement of SVQ, making less dependent of the device.
    > * Enable live migration if SVQ is enabled.
    > * Fix SVQ when driver reset.
    > * Comments addressed, specially in the iova area.
    > * Rebase on latest master, adding multiqueue support (but no networking
    >    control vq processing).
    > v5 link:
    > https://lists.gnu.org/archive/html/qemu-devel/2021-10/msg07250.html
    >
    > Changes from v4 RFC:
    > * Support of allocating / freeing iova ranges in IOVA tree. Extending
    >    already present iova-tree for that.
    > * Proper validation of guest features. Now SVQ can negotiate a
    >    different set of features with the device when enabled.
    > * Support of host notifiers memory regions
    > * Handling of SVQ full queue in case guest's descriptors span to
    >    different memory regions (qemu's VA chunks).
    > * Flush pending used buffers at end of SVQ operation.
    > * QMP command now looks by NetClientState name. Other devices will need
    >    to implement it's way to enable vdpa.
    > * Rename QMP command to set, so it looks more like a way of working
    > * Better use of qemu error system
    > * Make a few assertions proper error-handling paths.
    > * Add more documentation
    > * Less coupling of virtio / vhost, that could cause friction on changes
    > * Addressed many other small comments and small fixes.
    >
    > Changes from v3 RFC:
    >    * Move everything to vhost-vdpa backend. A big change, this allowed
    >      some cleanup but more code has been added in other places.
    >    * More use of glib utilities, especially to manage memory.
    > v3 link:
    > https://lists.nongnu.org/archive/html/qemu-devel/2021-05/msg06032.html
    >
    > Changes from v2 RFC:
    >    * Adding vhost-vdpa devices support
    >    * Fixed some memory leaks pointed by different comments
    > v2 link:
    > https://lists.nongnu.org/archive/html/qemu-devel/2021-03/msg05600.html
    >
    > Changes from v1 RFC:
    >    * Use QMP instead of migration to start SVQ mode.
    >    * Only accepting IOMMU devices, closer behavior with target devices
    >      (vDPA)
    >    * Fix invalid masking/unmasking of vhost call fd.
    >    * Use of proper methods for synchronization.
    >    * No need to modify VirtIO device code, all of the changes are
    >      contained in vhost code.
    >    * Delete superfluous code.
    >    * An intermediate RFC was sent with only the notifications forwarding
    >      changes. It can be seen in
    >      https://patchew.org/QEMU/20210129205415.876290-1-eperezma@redhat.com/
    > v1 link:
    > https://lists.gnu.org/archive/html/qemu-devel/2020-11/msg05372.html
    >
    > Eugenio Pérez (20):
    >        virtio: Add VIRTIO_F_QUEUE_STATE
    >        virtio-net: Honor VIRTIO_CONFIG_S_DEVICE_STOPPED
    >        virtio: Add virtio_queue_is_host_notifier_enabled
    >        vhost: Make vhost_virtqueue_{start,stop} public
    >        vhost: Add x-vhost-enable-shadow-vq qmp
    >        vhost: Add VhostShadowVirtqueue
    >        vdpa: Register vdpa devices in a list
    >        vhost: Route guest->host notification through shadow virtqueue
    >        Add vhost_svq_get_svq_call_notifier
    >        Add vhost_svq_set_guest_call_notifier
    >        vdpa: Save call_fd in vhost-vdpa
    >        vhost-vdpa: Take into account SVQ in vhost_vdpa_set_vring_call
    >        vhost: Route host->guest notification through shadow virtqueue
    >        virtio: Add vhost_shadow_vq_get_vring_addr
    >        vdpa: Save host and guest features
    >        vhost: Add vhost_svq_valid_device_features to shadow vq
    >        vhost: Shadow virtqueue buffers forwarding
    >        vhost: Add VhostIOVATree
    >        vhost: Use a tree to store memory mappings
    >        vdpa: Add custom IOTLB translations to SVQ
    >
    > Eugenio Pérez (31):
    >    vdpa: Reorder virtio/vhost-vdpa.c functions
    >    vhost: Add VhostShadowVirtqueue
    >    vdpa: Add vhost_svq_get_dev_kick_notifier
    >    vdpa: Add vhost_svq_set_svq_kick_fd
    >    vhost: Add Shadow VirtQueue kick forwarding capabilities
    >    vhost: Route guest->host notification through shadow virtqueue
    >    vhost: dd vhost_svq_get_svq_call_notifier
    >    vhost: Add vhost_svq_set_guest_call_notifier
    >    vhost-vdpa: Take into account SVQ in vhost_vdpa_set_vring_call
    >    vhost: Route host->guest notification through shadow virtqueue
    >    vhost: Add vhost_svq_valid_device_features to shadow vq
    >    vhost: Add vhost_svq_valid_guest_features to shadow vq
    >    vhost: Add vhost_svq_ack_guest_features to shadow vq
    >    virtio: Add vhost_shadow_vq_get_vring_addr
    >    vdpa: Add vhost_svq_get_num
    >    vhost: pass queue index to vhost_vq_get_addr
    >    vdpa: adapt vhost_ops callbacks to svq
    >    vhost: Shadow virtqueue buffers forwarding
    >    utils: Add internal DMAMap to iova-tree
    >    util: Store DMA entries in a list
    >    util: Add iova_tree_alloc
    >    vhost: Add VhostIOVATree
    >    vdpa: Add custom IOTLB translations to SVQ
    >    vhost: Add vhost_svq_get_last_used_idx
    >    vdpa: Adapt vhost_vdpa_get_vring_base to SVQ
    >    vdpa: Clear VHOST_VRING_F_LOG at vhost_vdpa_set_vring_addr in SVQ
    >    vdpa: Never set log_base addr if SVQ is enabled
    >    vdpa: Expose VHOST_F_LOG_ALL on SVQ
    >    vdpa: Make ncs autofree
    >    vdpa: Move vhost_vdpa_get_iova_range to net/vhost-vdpa.c
    >    vdpa: Add x-svq to NetdevVhostVDPAOptions
    >
    >   qapi/net.json                      |   5 +-
    >   hw/virtio/vhost-iova-tree.h        |  27 +
    >   hw/virtio/vhost-shadow-virtqueue.h |  46 ++
    >   include/hw/virtio/vhost-vdpa.h     |   7 +
    >   include/qemu/iova-tree.h           |  17 +
    >   hw/virtio/vhost-iova-tree.c        | 157 ++++++
    >   hw/virtio/vhost-shadow-virtqueue.c | 761 +++++++++++++++++++++++++++++
    >   hw/virtio/vhost-vdpa.c             | 740 ++++++++++++++++++++++++----
    >   hw/virtio/vhost.c                  |   6 +-
    >   net/vhost-vdpa.c                   |  58 ++-
    >   util/iova-tree.c                   | 161 +++++-
    >   hw/virtio/meson.build              |   2 +-
    >   12 files changed, 1852 insertions(+), 135 deletions(-)
    >   create mode 100644 hw/virtio/vhost-iova-tree.h
    >   create mode 100644 hw/virtio/vhost-shadow-virtqueue.h
    >   create mode 100644 hw/virtio/vhost-iova-tree.c
    >   create mode 100644 hw/virtio/vhost-shadow-virtqueue.c
    >
    
    _______________________________________________
    Virtualization mailing list
    Virtualization@lists.linux-foundation.org
    https://lists.linuxfoundation.org/mailman/listinfo/virtualization
    
    ^ permalink raw reply	[flat|nested] 52+ messages in thread
  • [parent not found: <20220121202733.404989-4-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-5-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-6-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-7-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-8-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-10-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-12-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-16-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-17-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-18-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-19-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-23-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-24-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-29-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-30-eperezma@redhat.com>]
  • [parent not found: <20220121202733.404989-31-eperezma@redhat.com>]

  • end of thread, other threads:[~2022-02-24  3:45 UTC | newest]
    
    Thread overview: 52+ messages (download: mbox.gz follow: Atom feed
    -- links below jump to the message on this page --
         [not found] <20220121202733.404989-1-eperezma@redhat.com>
         [not found] ` <20220121202733.404989-22-eperezma@redhat.com>
    2022-01-24  4:32   ` [PATCH 21/31] util: Add iova_tree_alloc Peter Xu
         [not found]     ` <CAJaqyWf--wbNZz5ZzbpixD9op_fO5fV01kbYXzG097c_NkqYrw@mail.gmail.com>
    2022-01-24 11:07       ` Peter Xu
         [not found]         ` <CAJaqyWcdpTr2X4VuAN2NLmpviCjDoAaY269+VQGZ7-F6myOhSw@mail.gmail.com>
    2022-01-27  8:06           ` Peter Xu
         [not found]             ` <CAJaqyWczZ7C_vbwugyN9bEgOVuRokGqVMb_g5UK_R4F8O+qKOA@mail.gmail.com>
    2022-01-28  3:57               ` Peter Xu
    2022-01-28  5:55                 ` Jason Wang
    2022-01-30  5:06       ` Jason Wang
         [not found] ` <20220121202733.404989-2-eperezma@redhat.com>
    2022-01-28  5:59   ` [PATCH 01/31] vdpa: Reorder virtio/vhost-vdpa.c functions Jason Wang
         [not found]     ` <CAJaqyWffGzYv2+HufFZzzBPtu5z3_vaKh4evGXqj7hqTB0WU3A@mail.gmail.com>
    2022-02-21  7:31       ` Jason Wang
         [not found] ` <20220121202733.404989-3-eperezma@redhat.com>
    2022-01-28  6:00   ` [PATCH 02/31] vhost: Add VhostShadowVirtqueue Jason Wang
    2022-01-28  6:02 ` [PATCH 00/31] vDPA shadow virtqueue Jason Wang
         [not found]   ` <CAJaqyWfWxQSJc3YMpF6g7VwZBN_ab0Z+1nXgWH1sg+uBaOYgBQ@mail.gmail.com>
    2022-02-08  8:27     ` Jason Wang
         [not found] ` <20220121202733.404989-4-eperezma@redhat.com>
    2022-01-28  6:03   ` [PATCH 03/31] vdpa: Add vhost_svq_get_dev_kick_notifier Jason Wang
         [not found] ` <20220121202733.404989-5-eperezma@redhat.com>
    2022-01-28  6:29   ` [PATCH 04/31] vdpa: Add vhost_svq_set_svq_kick_fd Jason Wang
         [not found]     ` <CAJaqyWc7fbgN-W7y3=iFqHsJzj+1Mg0cuwSu+my=62nu9vGOqA@mail.gmail.com>
    2022-02-08  8:47       ` Jason Wang
         [not found] ` <20220121202733.404989-6-eperezma@redhat.com>
    2022-01-28  6:32   ` [PATCH 05/31] vhost: Add Shadow VirtQueue kick forwarding capabilities Jason Wang
         [not found] ` <20220121202733.404989-7-eperezma@redhat.com>
    2022-01-28  6:56   ` [PATCH 06/31] vhost: Route guest->host notification through shadow virtqueue Jason Wang
         [not found]     ` <CAJaqyWeRbmwW80q3q52nFw=iz1xcPRFviFaRHo0nzXpEb+3m3A@mail.gmail.com>
    2022-02-08  9:02       ` Jason Wang
         [not found] ` <20220121202733.404989-8-eperezma@redhat.com>
    2022-01-29  7:57   ` [PATCH 07/31] vhost: dd vhost_svq_get_svq_call_notifier Jason Wang
         [not found] ` <20220121202733.404989-10-eperezma@redhat.com>
    2022-01-29  8:05   ` [PATCH 09/31] vhost-vdpa: Take into account SVQ in vhost_vdpa_set_vring_call Jason Wang
         [not found]     ` <CAJaqyWda5sBw9VGBrz8g60OJ07Eeq45RRYu9vwgOPZFwten9rw@mail.gmail.com>
    2022-02-08  3:23       ` Jason Wang
         [not found]         ` <CAJaqyWeisXmZ9+xw2Rj50K7aKx4khNZZjLZEz4MY97B9pQQm3w@mail.gmail.com>
    2022-02-21  7:39           ` Jason Wang
         [not found]             ` <CAJaqyWc5uR70a=hTpVpomuahF9iZouLmRpXPnWidga5CFxJOpA@mail.gmail.com>
    2022-02-22  7:18               ` Jason Wang
         [not found] ` <20220121202733.404989-12-eperezma@redhat.com>
    2022-01-29  8:11   ` [PATCH 11/31] vhost: Add vhost_svq_valid_device_features to shadow vq Jason Wang
         [not found]     ` <CAJaqyWfaf0RG9AzW4ktH2L3wyfOGuSk=rNm-j7xRkpdfVvkY-g@mail.gmail.com>
         [not found]       ` <CAJaqyWc6BqJBDcUE36AQ=bgWjJYkyMo1ZYxRwmc5ZgGj4T-pVg@mail.gmail.com>
    2022-02-08  3:37         ` Jason Wang
         [not found] ` <20220121202733.404989-16-eperezma@redhat.com>
    2022-01-29  8:14   ` [PATCH 15/31] vdpa: Add vhost_svq_get_num Jason Wang
         [not found] ` <20220121202733.404989-17-eperezma@redhat.com>
    2022-01-29  8:20   ` [PATCH 16/31] vhost: pass queue index to vhost_vq_get_addr Jason Wang
         [not found]     ` <CAJaqyWexu=VroHQxmtJDQm=iu1va-s1VGR8hqGOreG0SOisjYg@mail.gmail.com>
    2022-02-08  6:58       ` Jason Wang
         [not found] ` <20220121202733.404989-18-eperezma@redhat.com>
    2022-01-30  4:03   ` [PATCH 17/31] vdpa: adapt vhost_ops callbacks to svq Jason Wang
         [not found]     ` <CAJaqyWdRKZp6CwnE+HAr0JALhSRh-trJbZ01kddnLTuRX_tMKQ@mail.gmail.com>
    2022-02-08  3:57       ` Jason Wang
         [not found]         ` <CAJaqyWfEEg2PKgxBAFwYhF9LD1oDtwVYXSjHHnCbstT3dvL2GA@mail.gmail.com>
    2022-02-21  7:15           ` Jason Wang
         [not found]             ` <CAJaqyWcoHgToqsR-bVRctTnhgufmarR_2hh4O_VoCbCGp8WNhg@mail.gmail.com>
    2022-02-22  3:16               ` Jason Wang
         [not found]                 ` <CAJaqyWd2PQFedaEOV7YVZgp0m37snn-4LYYtNw7g4u+7hrtq=Q@mail.gmail.com>
    2022-02-22  7:59                   ` Jason Wang
         [not found] ` <20220121202733.404989-19-eperezma@redhat.com>
    2022-01-30  4:42   ` [PATCH 18/31] vhost: Shadow virtqueue buffers forwarding Jason Wang
         [not found]     ` <CAJaqyWdDax2+e3ZUEYyYNe5xAL=Oocu+72n89ygayrzYrQz2Yw@mail.gmail.com>
    2022-02-08  8:11       ` Jason Wang
         [not found]         ` <CAJaqyWfRWexq7jrCkJrPzLB4g_fK42pE8BarMhZwKNYtNXi7XA@mail.gmail.com>
    2022-02-23  2:03           ` Jason Wang
    2022-01-30  6:46   ` Jason Wang
         [not found]     ` <CAJaqyWfF01k3LntM7RLEmFcej=EY2d4+2MARKXPptQ2J7VnB9A@mail.gmail.com>
    2022-02-08  8:15       ` Jason Wang
         [not found]         ` <CAJaqyWedqtzRW=ur7upchneSc-oOkvkr3FUph_BfphV3zTmnkw@mail.gmail.com>
    2022-02-21  7:43           ` Jason Wang
         [not found]             ` <CAJaqyWcHhMpjJ4kde1ejV5c_vP7_8PvfXpi5u9rdWuaORFt_zg@mail.gmail.com>
    2022-02-22  7:26               ` Jason Wang
         [not found]                 ` <CAJaqyWePWg+eeQjjcMh24k0K+yUQUF2x0yXH32tPPWEw_wYP0Q@mail.gmail.com>
    2022-02-23  2:26                   ` Jason Wang
         [not found] ` <20220121202733.404989-23-eperezma@redhat.com>
    2022-01-30  5:21   ` [PATCH 22/31] vhost: Add VhostIOVATree Jason Wang
         [not found]     ` <CAJaqyWePW6hJKAm7nk+syqmXAgdTQSTtuv9jACu_+hgbg2bRHg@mail.gmail.com>
    2022-02-08  8:17       ` Jason Wang
         [not found] ` <20220121202733.404989-24-eperezma@redhat.com>
    2022-01-30  5:57   ` [PATCH 23/31] vdpa: Add custom IOTLB translations to SVQ Jason Wang
         [not found]     ` <CAJaqyWe1zH8bfaoxTyz_RXH=0q+Yk9H7QyUffaRB1fCV9oVLZQ@mail.gmail.com>
    2022-02-08  8:19       ` Jason Wang
         [not found] ` <20220121202733.404989-29-eperezma@redhat.com>
    2022-01-30  6:50   ` [PATCH 28/31] vdpa: Expose VHOST_F_LOG_ALL on SVQ Jason Wang
         [not found]     ` <CAJaqyWdBLU+maEhByepzeH7iwLmqUba0rRb8PM4VwBy2P8Vtow@mail.gmail.com>
    2022-02-08  8:25       ` Jason Wang
         [not found]         ` <CAJaqyWcvWjPas0=xp+U-c-kG+e6k73jg=C4phFD7S-tZY=niSQ@mail.gmail.com>
    2022-02-17  6:02           ` Jason Wang
         [not found]             ` <CAJaqyWdhHmD+tB_bY_YEMnBU1p7-LW=LP8f+3e_ZXDcOfSRiNA@mail.gmail.com>
    2022-02-22  7:41               ` Jason Wang
         [not found]                 ` <CAJaqyWfFC4SgxQ4zQeHgtDDJSd0tBa-W4HmtW0UASA2cVDWDUg@mail.gmail.com>
    2022-02-23  3:46                   ` Jason Wang
         [not found]                     ` <CAJaqyWds=97TjEpORiqhsj57KNxJ482jwcRS8TN59a4aank7-w@mail.gmail.com>
    2022-02-24  3:45                       ` Jason Wang
         [not found] ` <20220121202733.404989-30-eperezma@redhat.com>
    2022-01-30  6:51   ` [PATCH 29/31] vdpa: Make ncs autofree Jason Wang
         [not found] ` <20220121202733.404989-31-eperezma@redhat.com>
    2022-01-30  6:53   ` [PATCH 30/31] vdpa: Move vhost_vdpa_get_iova_range to net/vhost-vdpa.c Jason Wang
    

    This is a public inbox, see mirroring instructions
    for how to clone and mirror all data and code used for this inbox;
    as well as URLs for NNTP newsgroup(s).