Hi Ali, > -----Original Message----- > From: Ali Alnubani <[email protected]> > Sent: Tuesday, April 2, 2019 2:33 PM > To: Pavan Nikhilesh Bhagavatula <[email protected]> > Cc: [email protected]; Jerin Jacob Kollanukkaran <[email protected]>; Thomas > Monjalon <[email protected]>; [email protected]; > [email protected]; [email protected] > Subject: [EXT] RE: [dpdk-dev] [PATCH v5 1/2] app/testpmd: optimize testpmd > txonly mode > > External Email > > ---------------------------------------------------------------------- > Hi Pavan, > > > -----Original Message----- > > From: dev <[email protected]> On Behalf Of Pavan Nikhilesh > > Bhagavatula > > Sent: Sunday, March 31, 2019 4:14 PM > > To: Jerin Jacob Kollanukkaran <[email protected]>; Thomas Monjalon > > <[email protected]>; [email protected]; > > [email protected]; [email protected] > > Cc: [email protected]; Pavan Nikhilesh Bhagavatula > > <[email protected]> > > Subject: [dpdk-dev] [PATCH v5 1/2] app/testpmd: optimize testpmd > > txonly mode > > > > From: Pavan Nikhilesh <[email protected]> > > > > Optimize testpmd txonly mode by > > 1. Moving per packet ethernet header copy above the loop. > > 2. Use bulk ops for allocating segments instead of having a inner loop > > for every segment. > > > > Also, move the packet prepare logic into a separate function so that > > it can be reused later. > > > > Signed-off-by: Pavan Nikhilesh <[email protected]> > > --- > > v5 Changes > > - Remove unnecessary change to struct rte_port *txp (movement). > > (Bernard) > > > > v4 Changes: > > - Fix packet len calculation. > > > > v3 Changes: > > - Split the patches for easier review. (Thomas) > > - Remove unnecessary assignments to 0. (Bernard) > > > > v2 Changes: > > - Use bulk ops for fetching segments. (Andrew Rybchenko) > > - Fallback to rte_mbuf_raw_alloc if bulk get fails. (Andrew > > Rybchenko) > > - Fix mbufs not being freed when there is no more mbufs available for > > segments. (Andrew Rybchenko) > > > > app/test-pmd/txonly.c | 139 > > +++++++++++++++++++++++------------------- > > 1 file changed, 76 insertions(+), 63 deletions(-) > > > > The patch doesn't apply. It conflicts with 82010ef55 app/testpmd: make > txonly mode generate multiple flows > (http://patches.dpdk.org/patch/51869/). > Can you please rebase?
Yes, I'm in the process of rebasing/splitting the patch further. > > Thanks, > Ali Regards, Pavan.
