[PATCH vhost v9 0/6] refactor the params of find_vqs()

Xuan Zhuo xuanzhuo at linux.alibaba.com
Thu May 16 18:25:58 PDT 2024


Hi, Michael

I hope this in your for_linus branch to merge to Linux 6.9.

	https://git.kernel.org/pub/scm/linux/kernel/git/mst/vhost.git/log/?h=linux-next

And some commits from me in your branch are changed after you picked them.
And there are merged by net-next.

virtio_net: remove the misleading comment
virtio_net: rx remove premapped failover code
virtio_net: enable premapped by default
virtio_net: big mode support premapped
virtio_net: replace private by pp struct inside page
virtio_ring: enable premapped mode whatever use_dma_api
virtio_ring: introduce dma map api for page

Thanks.



On Wed, 24 Apr 2024 17:15:27 +0800, Xuan Zhuo <xuanzhuo at linux.alibaba.com> wrote:
> This pathset is splited from the
>
>      http://lore.kernel.org/all/20240229072044.77388-1-xuanzhuo@linux.alibaba.com
>
> That may needs some cycles to discuss. But that notifies too many people.
>
> But just the four commits need to notify so many people.
> And four commits are independent. So I split that patch set,
> let us review these first.
>
> The patch set try to  refactor the params of find_vqs().
> Then we can just change the structure, when introducing new
> features.
>
> Thanks.
>
> v8:
>   1. rebase the vhost branch
>
> v7:
>   1. fix two bugs. @Jason
>
> v6:
>   1. virtio_balloon: a single variable for both purposes.
>   2. if names[i] is null, return error
>
> v5:
>   1. virtio_balloon: follow David Hildenbrand's suggest
>     http://lore.kernel.org/all/3620be9c-e288-4ff2-a7be-1fcf806e6e6e@redhat.com
>   2. fix bug of the reference of "cfg_idx"
>     http://lore.kernel.org/all/202403222227.Sdp23Lcb-lkp@intel.com
>
> v4:
>   1. remove support for names array entries being null
>   2. remove cfg_idx from virtio_vq_config
>
> v3:
>   1. fix the bug: "assignment of read-only location '*cfg.names'"
>
> v2:
>   1. add kerneldoc for "struct vq_transport_config" @ilpo.jarvinen
>
> v1:
>   1. fix some comments from ilpo.jarvinen at linux.intel.com
>
>
>
>
>
>
>
>
>
> Xuan Zhuo (6):
>   virtio_balloon: remove the dependence where names[] is null
>   virtio: remove support for names array entries being null.
>   virtio: find_vqs: pass struct instead of multi parameters
>   virtio: vring_create_virtqueue: pass struct instead of multi
>     parameters
>   virtio: vring_new_virtqueue(): pass struct instead of multi parameters
>   virtio_ring: simplify the parameters of the funcs related to
>     vring_create/new_virtqueue()
>
>  arch/um/drivers/virtio_uml.c             |  36 +++--
>  drivers/platform/mellanox/mlxbf-tmfifo.c |  23 +--
>  drivers/remoteproc/remoteproc_virtio.c   |  37 +++--
>  drivers/s390/virtio/virtio_ccw.c         |  38 ++---
>  drivers/virtio/virtio_balloon.c          |  48 +++---
>  drivers/virtio/virtio_mmio.c             |  36 +++--
>  drivers/virtio/virtio_pci_common.c       |  69 ++++-----
>  drivers/virtio/virtio_pci_common.h       |   9 +-
>  drivers/virtio/virtio_pci_legacy.c       |  16 +-
>  drivers/virtio/virtio_pci_modern.c       |  37 +++--
>  drivers/virtio/virtio_ring.c             | 177 ++++++++---------------
>  drivers/virtio/virtio_vdpa.c             |  51 +++----
>  include/linux/virtio_config.h            |  76 +++++++---
>  include/linux/virtio_ring.h              |  93 +++++++-----
>  tools/virtio/virtio_test.c               |   4 +-
>  tools/virtio/vringh_test.c               |  28 ++--
>  16 files changed, 384 insertions(+), 394 deletions(-)
>
> --
> 2.32.0.3.g01195cf9f
>



More information about the linux-um mailing list