All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v9 00/32] virtio pci support VIRTIO_F_RING_RESET (refactor vring)
@ 2022-04-06  3:43 ` Xuan Zhuo
  0 siblings, 0 replies; 255+ messages in thread
From: Xuan Zhuo @ 2022-04-06  3:43 UTC (permalink / raw)
  To: virtualization
  Cc: Vadim Pasternak, Michael S. Tsirkin, linux-remoteproc,
	Alexei Starovoitov, Bjorn Andersson, Alexander Gordeev,
	Anton Ivanov, Johannes Berg, Daniel Borkmann, Richard Weinberger,
	Vincent Whitchurch, John Fastabend, Halil Pasic, Jakub Kicinski,
	Heiko Carstens, Jesper Dangaard Brouer, Vasily Gorbik,
	linux-s390, Jeff Dike, linux-um, Mark Gross, Hans de Goede, kvm,
	platform-driver-x86, Mathieu Poirier, netdev, Cornelia Huck,
	Sven Schnelle, bpf, David S. Miller

The virtio spec already supports the virtio queue reset function. This patch set
is to add this function to the kernel. The relevant virtio spec information is
here:

    https://github.com/oasis-tcs/virtio-spec/issues/124

Also regarding MMIO support for queue reset, I plan to support it after this
patch is passed.

This patch set implements the refactoring of vring. Finally, the
virtuque_resize() interface is provided based on the reset function of the
transport layer.

Test environment:
    Host: 4.19.91
    Qemu: QEMU emulator version 6.2.50 (with vq reset support)
    Test Cmd:  ethtool -G eth1 rx $1 tx $2; ethtool -g eth1

    The default is split mode, modify Qemu virtio-net to add PACKED feature to test
    packed mode.

Qemu code:
    https://github.com/fengidri/qemu/compare/89f3bfa3265554d1d591ee4d7f1197b6e3397e84...master

In order to simplify the review of this patch set, the function of reusing
the old buffers after resize will be introduced in subsequent patch sets.

Please review. Thanks.

v9:
  1. Provide a virtqueue_resize() interface directly
  2. A patch set including vring resize, virtio pci reset, virtio-net resize
  3. No more separate structs

v8:
  1. Provide a virtqueue_reset() interface directly
  2. Split the two patch sets, this is the first part
  3. Add independent allocation helper for allocating state, extra

v7:
  1. fix #6 subject typo
  2. fix #6 ring_size_in_bytes is uninitialized
  3. check by: make W=12

v6:
  1. virtio_pci: use synchronize_irq(irq) to sync the irq callbacks
  2. Introduce virtqueue_reset_vring() to implement the reset of vring during
     the reset process. May use the old vring if num of the vq not change.
  3. find_vqs() support sizes to special the max size of each vq

v5:
  1. add virtio-net support set_ringparam

v4:
  1. just the code of virtio, without virtio-net
  2. Performing reset on a queue is divided into these steps:
    1. reset_vq: reset one vq
    2. recycle the buffer from vq by virtqueue_detach_unused_buf()
    3. release the ring of the vq by vring_release_virtqueue()
    4. enable_reset_vq: re-enable the reset queue
  3. Simplify the parameters of enable_reset_vq()
  4. add container structures for virtio_pci_common_cfg

v3:
  1. keep vq, irq unreleased

Xuan Zhuo (32):
  virtio: add helper virtqueue_get_vring_max_size()
  virtio: struct virtio_config_ops add callbacks for queue_reset
  virtio_ring: update the document of the virtqueue_detach_unused_buf
    for queue reset
  virtio_ring: remove the arg vq of vring_alloc_desc_extra()
  virtio_ring: extract the logic of freeing vring
  virtio_ring: split: extract the logic of alloc queue
  virtio_ring: split: extract the logic of alloc state and extra
  virtio_ring: split: extract the logic of attach vring
  virtio_ring: split: extract the logic of vq init
  virtio_ring: split: introduce virtqueue_reinit_split()
  virtio_ring: split: introduce virtqueue_resize_split()
  virtio_ring: packed: extract the logic of alloc queue
  virtio_ring: packed: extract the logic of alloc state and extra
  virtio_ring: packed: extract the logic of attach vring
  virtio_ring: packed: extract the logic of vq init
  virtio_ring: packed: introduce virtqueue_reinit_packed()
  virtio_ring: packed: introduce virtqueue_resize_packed()
  virtio_ring: introduce virtqueue_resize()
  virtio_pci: struct virtio_pci_common_cfg add queue_notify_data
  virtio: queue_reset: add VIRTIO_F_RING_RESET
  virtio_pci: queue_reset: update struct virtio_pci_common_cfg and
    option functions
  virtio_pci: queue_reset: extract the logic of active vq for modern pci
  virtio_pci: queue_reset: support VIRTIO_F_RING_RESET
  virtio: find_vqs() add arg sizes
  virtio_pci: support the arg sizes of find_vqs()
  virtio_mmio: support the arg sizes of find_vqs()
  virtio: add helper virtio_find_vqs_ctx_size()
  virtio_net: set the default max ring size by find_vqs()
  virtio_net: get ringparam by virtqueue_get_vring_max_size()
  virtio_net: split free_unused_bufs()
  virtio_net: support rx/tx queue resize
  virtio_net: support set_ringparam

 arch/um/drivers/virtio_uml.c             |   3 +-
 drivers/net/virtio_net.c                 | 219 +++++++-
 drivers/platform/mellanox/mlxbf-tmfifo.c |   3 +
 drivers/remoteproc/remoteproc_virtio.c   |   3 +
 drivers/s390/virtio/virtio_ccw.c         |   4 +
 drivers/virtio/virtio_mmio.c             |  11 +-
 drivers/virtio/virtio_pci_common.c       |  28 +-
 drivers/virtio/virtio_pci_common.h       |   3 +-
 drivers/virtio/virtio_pci_legacy.c       |   8 +-
 drivers/virtio/virtio_pci_modern.c       | 149 +++++-
 drivers/virtio/virtio_pci_modern_dev.c   |  36 ++
 drivers/virtio/virtio_ring.c             | 626 ++++++++++++++++++-----
 drivers/virtio/virtio_vdpa.c             |   3 +
 include/linux/virtio.h                   |   6 +
 include/linux/virtio_config.h            |  38 +-
 include/linux/virtio_pci_modern.h        |   2 +
 include/uapi/linux/virtio_config.h       |   7 +-
 include/uapi/linux/virtio_pci.h          |  14 +
 18 files changed, 964 insertions(+), 199 deletions(-)

--
2.31.0

_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

^ permalink raw reply	[flat|nested] 255+ messages in thread

end of thread, other threads:[~2022-04-26 10:25 UTC | newest]

Thread overview: 255+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-06  3:43 [PATCH v9 00/32] virtio pci support VIRTIO_F_RING_RESET (refactor vring) Xuan Zhuo
2022-04-06  3:43 ` Xuan Zhuo
2022-04-06  3:43 ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 01/32] virtio: add helper virtqueue_get_vring_max_size() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  2:41   ` Jason Wang
2022-04-12  2:41     ` Jason Wang
2022-04-12  2:41     ` Jason Wang
2022-04-13  2:24     ` Xuan Zhuo
2022-04-13  2:24       ` Xuan Zhuo
2022-04-13  2:24       ` Xuan Zhuo
2022-04-14  9:16       ` Jason Wang
2022-04-14  9:16         ` Jason Wang
2022-04-14  9:16         ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 02/32] virtio: struct virtio_config_ops add callbacks for queue_reset Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  2:47   ` Jason Wang
2022-04-12  2:47     ` Jason Wang
2022-04-12  2:47     ` Jason Wang
2022-04-12  2:49     ` Jason Wang
2022-04-12  2:49       ` Jason Wang
2022-04-12  2:49       ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 03/32] virtio_ring: update the document of the virtqueue_detach_unused_buf for queue reset Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  2:49   ` Jason Wang
2022-04-12  2:49     ` Jason Wang
2022-04-12  2:49     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 04/32] virtio_ring: remove the arg vq of vring_alloc_desc_extra() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  2:53   ` Jason Wang
2022-04-12  2:53     ` Jason Wang
2022-04-12  2:53     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 05/32] virtio_ring: extract the logic of freeing vring Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  2:58   ` Jason Wang
2022-04-12  2:58     ` Jason Wang
2022-04-12  2:58     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 06/32] virtio_ring: split: extract the logic of alloc queue Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  3:22   ` Jason Wang
2022-04-12  3:22     ` Jason Wang
2022-04-12  3:22     ` Jason Wang
2022-04-13  6:52     ` Xuan Zhuo
2022-04-13  6:52       ` Xuan Zhuo
2022-04-13  6:52       ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 07/32] virtio_ring: split: extract the logic of alloc state and extra Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  3:26   ` Jason Wang
2022-04-12  3:26     ` Jason Wang
2022-04-12  3:26     ` Jason Wang
2022-04-13  6:48     ` Xuan Zhuo
2022-04-13  6:48       ` Xuan Zhuo
2022-04-13  6:48       ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 08/32] virtio_ring: split: extract the logic of attach vring Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  3:31   ` Jason Wang
2022-04-12  3:31     ` Jason Wang
2022-04-12  3:31     ` Jason Wang
2022-04-13  6:44     ` Xuan Zhuo
2022-04-13  6:44       ` Xuan Zhuo
2022-04-13  6:44       ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 09/32] virtio_ring: split: extract the logic of vq init Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  3:42   ` Jason Wang
2022-04-12  3:42     ` Jason Wang
2022-04-12  3:42     ` Jason Wang
2022-04-13  7:04     ` Xuan Zhuo
2022-04-13  7:04       ` Xuan Zhuo
2022-04-13  7:04       ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 10/32] virtio_ring: split: introduce virtqueue_reinit_split() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  5:49   ` Jason Wang
2022-04-12  5:49     ` Jason Wang
2022-04-12  5:49     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 11/32] virtio_ring: split: introduce virtqueue_resize_split() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  5:53   ` Jason Wang
2022-04-12  5:53     ` Jason Wang
2022-04-12  5:53     ` Jason Wang
2022-04-13  6:32     ` Xuan Zhuo
2022-04-13  6:32       ` Xuan Zhuo
2022-04-13  6:32       ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 12/32] virtio_ring: packed: extract the logic of alloc queue Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  6:28   ` Jason Wang
2022-04-12  6:28     ` Jason Wang
2022-04-12  6:28     ` Jason Wang
2022-04-13  3:23     ` Xuan Zhuo
2022-04-13  3:23       ` Xuan Zhuo
2022-04-13  3:23       ` Xuan Zhuo
2022-04-14  9:18       ` Jason Wang
2022-04-14  9:18         ` Jason Wang
2022-04-14  9:18         ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 13/32] virtio_ring: packed: extract the logic of alloc state and extra Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 14/32] virtio_ring: packed: extract the logic of attach vring Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 15/32] virtio_ring: packed: extract the logic of vq init Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 16/32] virtio_ring: packed: introduce virtqueue_reinit_packed() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 17/32] virtio_ring: packed: introduce virtqueue_resize_packed() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 18/32] virtio_ring: introduce virtqueue_resize() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  6:41   ` Jason Wang
2022-04-12  6:41     ` Jason Wang
2022-04-12  6:41     ` Jason Wang
2022-04-13 12:14     ` Xuan Zhuo
2022-04-13 12:14       ` Xuan Zhuo
2022-04-13 12:14       ` Xuan Zhuo
2022-04-13 12:21     ` Xuan Zhuo
2022-04-13 12:21       ` Xuan Zhuo
2022-04-13 12:21       ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 19/32] virtio_pci: struct virtio_pci_common_cfg add queue_notify_data Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 20/32] virtio: queue_reset: add VIRTIO_F_RING_RESET Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 21/32] virtio_pci: queue_reset: update struct virtio_pci_common_cfg and option functions Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  6:43   ` Jason Wang
2022-04-12  6:43     ` Jason Wang
2022-04-12  6:43     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 22/32] virtio_pci: queue_reset: extract the logic of active vq for modern pci Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  6:58   ` Jason Wang
2022-04-12  6:58     ` Jason Wang
2022-04-12  6:58     ` Jason Wang
2022-04-14  6:22     ` Xuan Zhuo
2022-04-14  6:22       ` Xuan Zhuo
2022-04-14  6:22       ` Xuan Zhuo
2022-04-14  9:37       ` Jason Wang
2022-04-14  9:37         ` Jason Wang
2022-04-14  9:37         ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 23/32] virtio_pci: queue_reset: support VIRTIO_F_RING_RESET Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  7:07   ` Jason Wang
2022-04-12  7:07     ` Jason Wang
2022-04-12  7:07     ` Jason Wang
2022-04-13  3:21     ` Xuan Zhuo
2022-04-13  3:21       ` Xuan Zhuo
2022-04-13  3:21       ` Xuan Zhuo
2022-04-14  9:17       ` Jason Wang
2022-04-14  9:17         ` Jason Wang
2022-04-14  9:17         ` Jason Wang
2022-04-13  8:48     ` Xuan Zhuo
2022-04-13  8:48       ` Xuan Zhuo
2022-04-13  8:48       ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 24/32] virtio: find_vqs() add arg sizes Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  7:10   ` Jason Wang
2022-04-12  7:10     ` Jason Wang
2022-04-12  7:10     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 25/32] virtio_pci: support the arg sizes of find_vqs() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  7:12   ` Jason Wang
2022-04-12  7:12     ` Jason Wang
2022-04-12  7:12     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 26/32] virtio_mmio: " Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  7:13   ` Jason Wang
2022-04-12  7:13     ` Jason Wang
2022-04-12  7:13     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 27/32] virtio: add helper virtio_find_vqs_ctx_size() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  7:15   ` Jason Wang
2022-04-12  7:15     ` Jason Wang
2022-04-12  7:15     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 28/32] virtio_net: set the default max ring size by find_vqs() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  7:18   ` Jason Wang
2022-04-12  7:18     ` Jason Wang
2022-04-12  7:18     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 29/32] virtio_net: get ringparam by virtqueue_get_vring_max_size() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-12  7:19   ` Jason Wang
2022-04-12  7:19     ` Jason Wang
2022-04-12  7:19     ` Jason Wang
2022-04-06  3:43 ` [PATCH v9 30/32] virtio_net: split free_unused_bufs() Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 31/32] virtio_net: support rx/tx queue resize Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-13  8:00   ` Jason Wang
2022-04-13  8:00     ` Jason Wang
2022-04-13  8:00     ` Jason Wang
2022-04-13  8:35     ` Xuan Zhuo
2022-04-13  8:35       ` Xuan Zhuo
2022-04-13  8:35       ` Xuan Zhuo
2022-04-14  9:30       ` Jason Wang
2022-04-14  9:30         ` Jason Wang
2022-04-14  9:30         ` Jason Wang
2022-04-15  2:18         ` Xuan Zhuo
2022-04-15  2:18           ` Xuan Zhuo
2022-04-15  2:18           ` Xuan Zhuo
2022-04-15  5:53           ` Jason Wang
2022-04-15  5:53             ` Jason Wang
2022-04-15  5:53             ` Jason Wang
2022-04-15  9:17             ` Xuan Zhuo
2022-04-15  9:17               ` Xuan Zhuo
2022-04-15  9:17               ` Xuan Zhuo
2022-04-18  7:57               ` Jason Wang
2022-04-18  7:57                 ` Jason Wang
2022-04-18  7:57                 ` Jason Wang
2022-04-18  3:21     ` Xuan Zhuo
2022-04-18  3:21       ` Xuan Zhuo
2022-04-18  3:21       ` Xuan Zhuo
2022-04-18  7:49       ` Jason Wang
2022-04-18  7:49         ` Jason Wang
2022-04-18  7:49         ` Jason Wang
2022-04-18  8:48         ` Xuan Zhuo
2022-04-18  8:48           ` Xuan Zhuo
2022-04-18  8:48           ` Xuan Zhuo
2022-04-06  3:43 ` [PATCH v9 32/32] virtio_net: support set_ringparam Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-06  3:43   ` Xuan Zhuo
2022-04-13  8:06   ` Jason Wang
2022-04-13  8:06     ` Jason Wang
2022-04-13  8:06     ` Jason Wang
2022-04-26  9:55 ` [PATCH v9 00/32] virtio pci support VIRTIO_F_RING_RESET (refactor vring) Michael S. Tsirkin
2022-04-26  9:55   ` Michael S. Tsirkin
2022-04-26  9:55   ` Michael S. Tsirkin
2022-04-26  9:59   ` Xuan Zhuo
2022-04-26  9:59     ` Xuan Zhuo
2022-04-26  9:59     ` Xuan Zhuo

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.