similar to: [RFC PATCH 0/2] virtio: allow per vq DMA domain

Hi, This patch set implements packed ring support in virtio driver. A performance test between pktgen (pktgen_sample03_burst_single_flow.sh) and DPDK vhost (testpmd/rxonly/vhost-PMD) has been done, I saw ~30% performance gain in packed ring in this case. To make this patch set work with below patch set for vhost, some hacks are needed to set the _F_NEXT flag in indirect descriptors (this should

Hi, This patch set implements packed ring support in virtio driver. A performance test between pktgen (pktgen_sample03_burst_single_flow.sh) and DPDK vhost (testpmd/rxonly/vhost-PMD) has been done, I saw ~30% performance gain in packed ring in this case. To make this patch set work with below patch set for vhost, some hacks are needed to set the _F_NEXT flag in indirect descriptors (this should

These patches are not ready to be merged because I was unable to measure a performance improvement. I'm publishing them so they are archived in case someone picks up this work again in the future. The goal of these patches is to allocate virtqueues and driver state from the device's NUMA node for optimal memory access latency. Only guests with a vNUMA topology and virtio devices spread

These patches are not ready to be merged because I was unable to measure a performance improvement. I'm publishing them so they are archived in case someone picks up this work again in the future. The goal of these patches is to allocate virtqueues and driver state from the device's NUMA node for optimal memory access latency. Only guests with a vNUMA topology and virtio devices spread

This patch validate the used buffer length provided by the device before trying to use it. This is done by remembering the in buffer length in a dedicated array during virtqueue_add(), then we can fail the virtqueue_get_buf() when we find the device is trying to give us a used buffer length which is greater than we stored before. This validation is disable by default via module parameter to

On Wed, May 31, 2023 at 03:36:51PM +0800, Jason Wang wrote: > On Wed, May 31, 2023 at 1:50?PM Michael S. Tsirkin <mst at redhat.com> wrote: > > > > On Wed, May 31, 2023 at 09:05:00AM +0800, Jason Wang wrote: > > > On Mon, May 29, 2023 at 6:03?PM Michael S. Tsirkin <mst at redhat.com> wrote: > > > > > > > > On Mon, May 29, 2023 at

On Wed, May 31, 2023 at 04:26:38PM +0800, Jason Wang wrote: > On Wed, May 31, 2023 at 3:36?PM Jason Wang <jasowang at redhat.com> wrote: > > > > On Wed, May 31, 2023 at 1:50?PM Michael S. Tsirkin <mst at redhat.com> wrote: > > > > > > On Wed, May 31, 2023 at 09:05:00AM +0800, Jason Wang wrote: > > > > On Mon, May 29, 2023 at 6:03?PM Michael

On Wed, May 31, 2023 at 5:55?PM Michael S. Tsirkin <mst at redhat.com> wrote: > > On Wed, May 31, 2023 at 03:36:51PM +0800, Jason Wang wrote: > > On Wed, May 31, 2023 at 1:50?PM Michael S. Tsirkin <mst at redhat.com> wrote: > > > > > > On Wed, May 31, 2023 at 09:05:00AM +0800, Jason Wang wrote: > > > > On Mon, May 29, 2023 at 6:03?PM Michael S.

On Wed, May 31, 2023 at 6:25?PM Michael S. Tsirkin <mst at redhat.com> wrote: > > On Wed, May 31, 2023 at 04:26:38PM +0800, Jason Wang wrote: > > On Wed, May 31, 2023 at 3:36?PM Jason Wang <jasowang at redhat.com> wrote: > > > > > > On Wed, May 31, 2023 at 1:50?PM Michael S. Tsirkin <mst at redhat.com> wrote: > > > > > > > >

On 2018?03?16? 18:04, Tiwei Bie wrote: > On Fri, Mar 16, 2018 at 04:34:28PM +0800, Jason Wang wrote: >> On 2018?03?16? 15:40, Tiwei Bie wrote: >>> On Fri, Mar 16, 2018 at 02:44:12PM +0800, Jason Wang wrote: >>>> On 2018?03?16? 14:10, Tiwei Bie wrote: >>>>> On Fri, Mar 16, 2018 at 12:03:25PM +0800, Jason Wang wrote: >>>>>> On

On 2018?03?16? 18:04, Tiwei Bie wrote: > On Fri, Mar 16, 2018 at 04:34:28PM +0800, Jason Wang wrote: >> On 2018?03?16? 15:40, Tiwei Bie wrote: >>> On Fri, Mar 16, 2018 at 02:44:12PM +0800, Jason Wang wrote: >>>> On 2018?03?16? 14:10, Tiwei Bie wrote: >>>>> On Fri, Mar 16, 2018 at 12:03:25PM +0800, Jason Wang wrote: >>>>>> On

virtio_disable_cb is currently a nop for split ring with event index. This is because it used to be always called from a callback when we know device won't trigger more events until we update the index. However, now that we run with interrupts enabled a lot we also poll without a callback so that is different: disabling callbacks will help reduce the number of spurious interrupts. Further, if

On 2018?03?16? 15:40, Tiwei Bie wrote: > On Fri, Mar 16, 2018 at 02:44:12PM +0800, Jason Wang wrote: >> On 2018?03?16? 14:10, Tiwei Bie wrote: >>> On Fri, Mar 16, 2018 at 12:03:25PM +0800, Jason Wang wrote: >>>> On 2018?02?23? 19:18, Tiwei Bie wrote: >>>>> Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> >>>>> ---

On 2018?03?16? 15:40, Tiwei Bie wrote: > On Fri, Mar 16, 2018 at 02:44:12PM +0800, Jason Wang wrote: >> On 2018?03?16? 14:10, Tiwei Bie wrote: >>> On Fri, Mar 16, 2018 at 12:03:25PM +0800, Jason Wang wrote: >>>> On 2018?02?23? 19:18, Tiwei Bie wrote: >>>>> Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> >>>>> ---

On 2018?03?16? 14:10, Tiwei Bie wrote: > On Fri, Mar 16, 2018 at 12:03:25PM +0800, Jason Wang wrote: >> On 2018?02?23? 19:18, Tiwei Bie wrote: >>> Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> >>> --- >>> drivers/virtio/virtio_ring.c | 699 +++++++++++++++++++++++++++++++++++++------ >>> include/linux/virtio_ring.h | 8 +- >>>

On 2018?03?16? 14:10, Tiwei Bie wrote: > On Fri, Mar 16, 2018 at 12:03:25PM +0800, Jason Wang wrote: >> On 2018?02?23? 19:18, Tiwei Bie wrote: >>> Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> >>> --- >>> drivers/virtio/virtio_ring.c | 699 +++++++++++++++++++++++++++++++++++++------ >>> include/linux/virtio_ring.h | 8 +- >>>

On 2018?02?23? 19:18, Tiwei Bie wrote: > Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> > --- > drivers/virtio/virtio_ring.c | 699 +++++++++++++++++++++++++++++++++++++------ > include/linux/virtio_ring.h | 8 +- > 2 files changed, 618 insertions(+), 89 deletions(-) > > diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c > index

On 2018?02?23? 19:18, Tiwei Bie wrote: > Signed-off-by: Tiwei Bie <tiwei.bie at intel.com> > --- > drivers/virtio/virtio_ring.c | 699 +++++++++++++++++++++++++++++++++++++------ > include/linux/virtio_ring.h | 8 +- > 2 files changed, 618 insertions(+), 89 deletions(-) > > diff --git a/drivers/virtio/virtio_ring.c b/drivers/virtio/virtio_ring.c > index

Hello everyone, This RFC implements a subset of packed ring which is described at https://github.com/oasis-tcs/virtio-docs/blob/master/virtio-v1.1-packed-wd08.pdf The code was tested with DPDK vhost (testpmd/vhost-PMD) implemented by Jens at http://dpdk.org/ml/archives/dev/2018-January/089417.html Minor changes are needed for the vhost code, e.g. to kick the guest. It's not a complete

Hello everyone, This RFC implements packed ring support in virtio driver. Some simple functional tests have been done with Jason's packed ring implementation in vhost (RFC v4): https://lkml.org/lkml/2018/5/16/501 Both of ping and netperf worked as expected w/ EVENT_IDX disabled. Ping worked as expected w/ EVENT_IDX enabled, but netperf didn't (A hack has been added in the driver to