Re: [PATCH vhost v7 4/6] virtio: vring_create_virtqueue: pass struct instead of multi parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2024-03-28 at 16:03 +0800, Xuan Zhuo wrote:
> Now, we pass multi parameters to vring_create_virtqueue. These
> parameters
> may from transport or from driver.
> 
> vring_create_virtqueue is called by many places.
> Every time, we try to add a new parameter, that is difficult.
> 
> If parameters from the driver, that should directly be passed to
> vring.
> Then the vring can access the config from driver directly.
> 
> If parameters from the transport, we squish the parameters to a
> structure. That will be helpful to add new parameter.
> 
> Signed-off-by: Xuan Zhuo <xuanzhuo@xxxxxxxxxxxxxxxxx>
> Acked-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx>
> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@xxxxxxxxxxxxxxx>
> Acked-by: Jason Wang <jasowang@xxxxxxxxxx>
> ---
>  arch/um/drivers/virtio_uml.c       | 14 +++++---
>  drivers/s390/virtio/virtio_ccw.c   | 14 ++++----
>  drivers/virtio/virtio_mmio.c       | 14 ++++----
>  drivers/virtio/virtio_pci_legacy.c | 15 ++++----
>  drivers/virtio/virtio_pci_modern.c | 15 ++++----
>  drivers/virtio/virtio_ring.c       | 57 ++++++++++++----------------
> --
>  drivers/virtio/virtio_vdpa.c       | 21 +++++------
>  include/linux/virtio_ring.h        | 51 +++++++++++++-------------
>  8 files changed, 101 insertions(+), 100 deletions(-)

Acked-by: Eric Farman <farman@xxxxxxxxxxxxx> # s390





[Index of Archives]     [Linux Kernel Development]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux