回复: [PATCH v1] net/i40e: remove redundant number of packets check

[Feifei Wang]

> -----邮件原件-----
> 发件人: Zhang, Qi Z <qi.z.zhang at intel.com>
> 发送时间: Friday, March 4, 2022 9:24 AM
> 收件人: Yigit, Ferruh <ferruh.yigit at intel.com>; Feifei Wang
> <Feifei.Wang2 at arm.com>; Xing, Beilei <beilei.xing at intel.com>; David
> Christensen <drc at linux.vnet.ibm.com>; Richardson, Bruce
> <bruce.richardson at intel.com>; Ananyev, Konstantin
> <konstantin.ananyev at intel.com>; Ruifeng Wang <Ruifeng.Wang at arm.com>
> 抄送: dev at dpdk.org; nd <nd at arm.com>; Honnappa Nagarahalli
> <Honnappa.Nagarahalli at arm.com>; thomas at monjalon.net; David Marchand
> <david.marchand at redhat.com>
> 主题: RE: [PATCH v1] net/i40e: remove redundant number of packets check
> 
> 
> 
> > -----Original Message-----
> > From: Yigit, Ferruh <ferruh.yigit at intel.com>
> > Sent: Friday, March 4, 2022 12:44 AM
> > To: Zhang, Qi Z <qi.z.zhang at intel.com>; Feifei Wang
> > <feifei.wang2 at arm.com>; Xing, Beilei <beilei.xing at intel.com>; David
> > Christensen <drc at linux.vnet.ibm.com>; Richardson, Bruce
> > <bruce.richardson at intel.com>; Ananyev, Konstantin
> > <konstantin.ananyev at intel.com>; Ruifeng Wang <ruifeng.wang at arm.com>
> > Cc: dev at dpdk.org; nd at arm.com; Honnappa Nagarahalli
> > <honnappa.nagarahalli at arm.com>; Thomas Monjalon
> <thomas at monjalon.net>;
> > David Marchand <david.marchand at redhat.com>
> > Subject: Re: [PATCH v1] net/i40e: remove redundant number of packets
> > check
> >
> > On 3/3/2022 2:28 AM, Zhang, Qi Z wrote:
> > >
> > >
> > >> -----Original Message-----
> > >> From: Feifei Wang <feifei.wang2 at arm.com>
> > >> Sent: Thursday, March 3, 2022 9:54 AM
> > >> To: Xing, Beilei <beilei.xing at intel.com>; David Christensen
> > >> <drc at linux.vnet.ibm.com>; Richardson, Bruce
> > >> <bruce.richardson at intel.com>; Ananyev, Konstantin
> > >> <konstantin.ananyev at intel.com>; Ruifeng Wang
> <ruifeng.wang at arm.com>
> > >> Cc: dev at dpdk.org; nd at arm.com; Feifei Wang <feifei.wang2 at arm.com>;
> > >> Honnappa Nagarahalli <honnappa.nagarahalli at arm.com>
> > >> Subject: [PATCH v1] net/i40e: remove redundant number of packets
> > >> check
> > >>
> > >> For i40e_xmit_pkts_vec_xx function, it checks nb_pkts to ensure
> > >> nb_pkts does not cross rs_thresh.
> > >>
> > >> However, in i40e_xmit_fixed_burst_vec_xx function, this check will
> > >> be performed again. To improve code, delete this redundant check.
> > >>
> > >> Suggested-by: Honnappa Nagarahalli <honnappa.nagarahalli at arm.com>
> > >> Signed-off-by: Feifei Wang <feifei.wang2 at arm.com>
> > >> Reviewed-by: Ruifeng Wang <ruifeng.wang at arm.com>
> > >
> > > Applied to dpdk-next-net-intel.
> > >
> >
> > Hi Qi,
> >
> > This patch is not acked by the i40e maintainers.
> >
> > And this is changing the datapath for the -rc3, two weeks before the
> > release. Is it tested enough?

Hi, Ferruh

Thanks for your remind. Test the performance in l3fwd with this patch:
In arm server, there is no performance change.
In x86 server, there is also no obvious performance change
according to performance test in public CI.

Best Regards
Feifei

> >
> > What is the gain with this patch, I don't see any numbers in the commit log.
> > If the gain is small, can we postpone this patch to next release
> > instead of getting it for -rc3?
> 
> The patch applied the same thing as below which I have reviewed.
> https://patchwork.dpdk.org/project/dpdk/patch/20220222224818.8612-1-
> kathleen.capella at arm.com/
> 
> I didn't see the risk of having it, and I will add a "reviewed-by" to avoid
> confusion, but if you think it's risky, we can still defer it to next-net.

Hi, Qi

Thanks for your review.

Best Regards
Feifei

> 
> Thanks
> Qi