Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-07-01 Thread Thomas Monjalon
07/06/2017 19:20, Ferruh Yigit: > On 5/11/2017 12:51 PM, Gowrishankar wrote: > > From: Gowrishankar Muthukrishnan > > > > In kni_allocate_mbufs(), we attempt to add max_burst (32) count of mbuf > > always into alloc_q, which is excessively leading too many rte_pktmbuf_ > > free() when alloc_q is

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-06-07 Thread Ferruh Yigit
On 5/11/2017 12:51 PM, Gowrishankar wrote: > From: Gowrishankar Muthukrishnan > > In kni_allocate_mbufs(), we attempt to add max_burst (32) count of mbuf > always into alloc_q, which is excessively leading too many rte_pktmbuf_ > free() when alloc_q is contending at high packet rate (for eg 10Gig

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-06-07 Thread Ferruh Yigit
On 6/6/2017 3:43 PM, gowrishankar muthukrishnan wrote: > Hi Ferruh, > Just wanted to check with you on the verdict of this patch, whether we > are waiting for > any objection/ack ?. I was waiting for more comment, I will ack explicitly. > > Thanks, > Gowrishankar > > On Thursday 01 June 2017 0

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-06-06 Thread gowrishankar muthukrishnan
Hi Ferruh, Just wanted to check with you on the verdict of this patch, whether we are waiting for any objection/ack ?. Thanks, Gowrishankar On Thursday 01 June 2017 02:48 PM, Ferruh Yigit wrote: On 6/1/2017 6:56 AM, gowrishankar muthukrishnan wrote: Hi Ferruh, On Wednesday 31 May 2017 09:51

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-06-01 Thread Ferruh Yigit
On 6/1/2017 6:56 AM, gowrishankar muthukrishnan wrote: > Hi Ferruh, > > On Wednesday 31 May 2017 09:51 PM, Ferruh Yigit wrote: > >> I have sampled below data in x86_64 for KNI on ixgbe pmd. iperf server >>> runs on >>> remote interface connecting PMD and iperf client runs on KNI interface, >>> so

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-05-31 Thread gowrishankar muthukrishnan
Hi Ferruh, On Wednesday 31 May 2017 09:51 PM, Ferruh Yigit wrote: I have sampled below data in x86_64 for KNI on ixgbe pmd. iperf server runs on remote interface connecting PMD and iperf client runs on KNI interface, so as to create more egress from KNI into DPDK (w/o and with this patch) for

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-05-31 Thread Ferruh Yigit
Hi Gowrishankar, Sorry for late response. On 5/18/2017 6:45 PM, gowrishankar muthukrishnan wrote: > On Tuesday 16 May 2017 10:45 PM, Ferruh Yigit wrote: >> On 5/11/2017 12:51 PM, Gowrishankar wrote: >>> From: Gowrishankar Muthukrishnan >>> >>> In kni_allocate_mbufs(), we attempt to add max_burst

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-05-18 Thread gowrishankar muthukrishnan
On Tuesday 16 May 2017 10:45 PM, Ferruh Yigit wrote: On 5/11/2017 12:51 PM, Gowrishankar wrote: From: Gowrishankar Muthukrishnan In kni_allocate_mbufs(), we attempt to add max_burst (32) count of mbuf always into alloc_q, which is excessively leading too many rte_pktmbuf_ free() when alloc_q i

Re: [dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-05-16 Thread Ferruh Yigit
On 5/11/2017 12:51 PM, Gowrishankar wrote: > From: Gowrishankar Muthukrishnan > > In kni_allocate_mbufs(), we attempt to add max_burst (32) count of mbuf > always into alloc_q, which is excessively leading too many rte_pktmbuf_ > free() when alloc_q is contending at high packet rate (for eg 10Gig

[dpdk-dev] [PATCH v2] kni: add new mbuf in alloc_q only based on its empty slots

2017-05-11 Thread Gowrishankar
From: Gowrishankar Muthukrishnan In kni_allocate_mbufs(), we attempt to add max_burst (32) count of mbuf always into alloc_q, which is excessively leading too many rte_pktmbuf_ free() when alloc_q is contending at high packet rate (for eg 10Gig data). In a situation when alloc_q fifo can only acc