Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-29 Thread David Marchand
On Tue, Oct 29, 2019 at 1:21 PM Flavio Leitner wrote: > On Tue, 29 Oct 2019 10:02:57 +0100 > David Marchand wrote: > > Using rte_malloc() means that the allocation can end up on any numa > > node. This external buffer might end up on a different node than the > > mbuf (which resides on mp->socket

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-29 Thread Flavio Leitner
On Tue, 29 Oct 2019 10:02:57 +0100 David Marchand wrote: > On Tue, Oct 15, 2019 at 9:00 PM Flavio Leitner > wrote: > > diff --git a/lib/librte_vhost/virtio_net.c > > b/lib/librte_vhost/virtio_net.c index 5b85b832d..da69ab1db 100644 > > --- a/lib/librte_vhost/virtio_net.c > > +++ b/lib/librte_vho

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-29 Thread David Marchand
On Tue, Oct 15, 2019 at 9:00 PM Flavio Leitner wrote: > diff --git a/lib/librte_vhost/virtio_net.c b/lib/librte_vhost/virtio_net.c > index 5b85b832d..da69ab1db 100644 > --- a/lib/librte_vhost/virtio_net.c > +++ b/lib/librte_vhost/virtio_net.c > @@ -1289,6 +1289,93 @@ get_zmbuf(struct vhost_virtque

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Flavio Leitner
On Wed, 16 Oct 2019 16:08:54 +0200 Ilya Maximets wrote: > On 16.10.2019 16:02, Flavio Leitner wrote: > > On Wed, 16 Oct 2019 15:46:15 +0200 > > Maxime Coquelin wrote: > > > >> On 10/16/19 3:32 PM, Ilya Maximets wrote: > >>> On 16.10.2019 13:13, Maxime Coquelin wrote: > > >

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Ilya Maximets
On 16.10.2019 16:02, Flavio Leitner wrote: On Wed, 16 Oct 2019 15:46:15 +0200 Maxime Coquelin wrote: On 10/16/19 3:32 PM, Ilya Maximets wrote: On 16.10.2019 13:13, Maxime Coquelin wrote: On 10/15/19 8:59 PM, Flavio Leitner wrote: The rte_vhost_dequeue_burst supports two ways of dequeuing

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Ilya Maximets
On 16.10.2019 15:46, Maxime Coquelin wrote: On 10/16/19 3:32 PM, Ilya Maximets wrote: On 16.10.2019 13:13, Maxime Coquelin wrote: On 10/15/19 8:59 PM, Flavio Leitner wrote: The rte_vhost_dequeue_burst supports two ways of dequeuing data. If the data fits into a buffer, then all data is cop

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Flavio Leitner
On Wed, 16 Oct 2019 15:46:15 +0200 Maxime Coquelin wrote: > On 10/16/19 3:32 PM, Ilya Maximets wrote: > > On 16.10.2019 13:13, Maxime Coquelin wrote: > >> > >> > >> On 10/15/19 8:59 PM, Flavio Leitner wrote: > >>> The rte_vhost_dequeue_burst supports two ways of dequeuing data. > >>> If the d

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Maxime Coquelin
On 10/16/19 3:32 PM, Ilya Maximets wrote: > On 16.10.2019 13:13, Maxime Coquelin wrote: >> >> >> On 10/15/19 8:59 PM, Flavio Leitner wrote: >>> The rte_vhost_dequeue_burst supports two ways of dequeuing data. >>> If the data fits into a buffer, then all data is copied and a >>> single linear buf

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Ilya Maximets
On 16.10.2019 13:13, Maxime Coquelin wrote: On 10/15/19 8:59 PM, Flavio Leitner wrote: The rte_vhost_dequeue_burst supports two ways of dequeuing data. If the data fits into a buffer, then all data is copied and a single linear buffer is returned. Otherwise it allocates additional mbufs and ch

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Maxime Coquelin
On 10/15/19 8:59 PM, Flavio Leitner wrote: > The rte_vhost_dequeue_burst supports two ways of dequeuing data. > If the data fits into a buffer, then all data is copied and a > single linear buffer is returned. Otherwise it allocates > additional mbufs and chains them together to return a multipl

Re: [dpdk-dev] [PATCH v5] vhost: add support for large buffers

2019-10-16 Thread Maxime Coquelin
Hi Flavio, On 10/15/19 8:59 PM, Flavio Leitner wrote: > The rte_vhost_dequeue_burst supports two ways of dequeuing data. > If the data fits into a buffer, then all data is copied and a > single linear buffer is returned. Otherwise it allocates > additional mbufs and chains them together to return