Re: [dpdk-dev] [PATCH] net/i40e: fix Tx fn selection when using new ethdev offloads

2018-05-01 Thread Ananyev, Konstantin
> -Original Message- > From: Richardson, Bruce > Sent: Tuesday, May 1, 2018 2:53 PM > To: Ananyev, Konstantin > Cc: Xing, Beilei ; Zhang, Qi Z ; > dev@dpdk.org; Yigit, Ferruh > Subject: Re: [dpdk-dev] [PATCH] net/i40e: fix Tx fn selection when using new > eth

Re: [dpdk-dev] [PATCH] net/i40e: fix Tx fn selection when using new ethdev offloads

2018-05-01 Thread Bruce Richardson
On Tue, May 01, 2018 at 02:24:39PM +0100, Ananyev, Konstantin wrote: > Hi Bruce, > > > > > The Tx function selection code in the driver only used the older txq > > flags values to check whether the scalar or vector functions should be > > used. This caused performance regressions with testpmd io-

Re: [dpdk-dev] [PATCH] net/i40e: fix Tx fn selection when using new ethdev offloads

2018-05-01 Thread Bruce Richardson
On Tue, May 01, 2018 at 02:24:39PM +0100, Ananyev, Konstantin wrote: > Hi Bruce, > > > > > The Tx function selection code in the driver only used the older txq > > flags values to check whether the scalar or vector functions should be > > used. This caused performance regressions with testpmd io-

Re: [dpdk-dev] [PATCH] net/i40e: fix Tx fn selection when using new ethdev offloads

2018-05-01 Thread Ananyev, Konstantin
Hi Bruce, > > The Tx function selection code in the driver only used the older txq > flags values to check whether the scalar or vector functions should be > used. This caused performance regressions with testpmd io-fwd as the > scalar path rather than the vector one was being used in the default

[dpdk-dev] [PATCH] net/i40e: fix Tx fn selection when using new ethdev offloads

2018-05-01 Thread Bruce Richardson
The Tx function selection code in the driver only used the older txq flags values to check whether the scalar or vector functions should be used. This caused performance regressions with testpmd io-fwd as the scalar path rather than the vector one was being used in the default case. Fix this by cha