[dpdk-dev] [PATCH 3/4] virtio: use indirect ring elements

2015-09-07 Thread Thomas Monjalon
General comment after reading this page: http://dpdk.org/dev/patchwork/patch/6905/ Please remove useless context when replying to make answers shorter and easier to read. Thanks

[dpdk-dev] [PATCH 3/4] virtio: use indirect ring elements

2015-09-06 Thread Stephen Hemminger
On Sun, 6 Sep 2015 08:40:44 + "Ouyang, Changchun" wrote: > > @@ -220,11 +221,26 @@ virtqueue_enqueue_xmit(struct virtqueue *txvq, > > struct rte_mbuf *cookie) > > dxp = &txvq->vq_descx[idx]; > > dxp->cookie = (void *)cookie; > > dxp->ndescs = needed; > > - > > start_dp = txvq-

[dpdk-dev] [PATCH 3/4] virtio: use indirect ring elements

2015-09-06 Thread Stephen Hemminger
On Sun, 6 Sep 2015 08:36:10 + "Ouyang, Changchun" wrote: > > -Original Message- > > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > > Sent: Saturday, September 5, 2015 4:58 AM > > To: Xie, Huawei; Ouyang, Changchun > > Cc: dev at dpdk.org; Stephen Hemminger > > Subjec

[dpdk-dev] [PATCH 3/4] virtio: use indirect ring elements

2015-09-06 Thread Ouyang, Changchun
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Saturday, September 5, 2015 4:58 AM > To: Xie, Huawei; Ouyang, Changchun > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH 3/4] virtio: use indirect ring elements > > The virtio ring in

[dpdk-dev] [PATCH 3/4] virtio: use indirect ring elements

2015-09-06 Thread Ouyang, Changchun
> -Original Message- > From: Stephen Hemminger [mailto:stephen at networkplumber.org] > Sent: Saturday, September 5, 2015 4:58 AM > To: Xie, Huawei; Ouyang, Changchun > Cc: dev at dpdk.org; Stephen Hemminger > Subject: [PATCH 3/4] virtio: use indirect ring elements > > The virtio ring in

[dpdk-dev] [PATCH 3/4] virtio: use indirect ring elements

2015-09-04 Thread Stephen Hemminger
The virtio ring in QEMU/KVM is usually limited to 256 entries and the normal way that virtio driver was queuing mbufs required nsegs + 1 ring elements. By using the indirect ring element feature if available, each packet will take only one ring slot even for multi-segment packets. Signed-off-by: S