Ping.

> -----Original Message-----
> From: Jerin Jacob <jerinjac...@gmail.com>
> Sent: Thursday, June 2, 2022 1:19 PM
> To: Rahul Bhansali <rbhans...@marvell.com>
> Cc: dpdk-dev <dev@dpdk.org>; Bruce Richardson
> <bruce.richard...@intel.com>; Thomas Monjalon <tho...@monjalon.net>;
> Ferruh Yigit <ferruh.yi...@xilinx.com>; Ajit Khaparde
> <ajit.khapa...@broadcom.com>; Andrew Boyer <abo...@pensando.io>;
> Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>; Beilei Xing
> <beilei.x...@intel.com>; Chas Williams <ch...@att.com>; Xia, Chenbo
> <chenbo....@intel.com>; Ciara Loftus <ciara.lof...@intel.com>; Devendra
> Singh Rawat <dsinghra...@marvell.com>; Ed Czeck
> <ed.cz...@atomicrules.com>; Evgeny Schemeilin <evge...@amazon.com>;
> Gaetan Rivet <gr...@u256.net>; Gagandeep Singh <g.si...@nxp.com>;
> Guoyang Zhou <zhouguoy...@huawei.com>; Haiyue Wang
> <haiyue.w...@intel.com>; Harman Kalra <hka...@marvell.com>;
> heinrich.k...@corigine.com; Hemant Agrawal <hemant.agra...@nxp.com>;
> Hyong Youb Kim <hyon...@cisco.com>; Igor Chauskin <igo...@amazon.com>;
> Igor Russkikh <irussk...@marvell.com>; Jakub Grajciar <jgraj...@cisco.com>;
> Jasvinder Singh <jasvinder.si...@intel.com>; Jian Wang
> <jianw...@trustnetic.com>; Jiawen Wu <jiawe...@trustnetic.com>; Jingjing
> Wu <jingjing...@intel.com>; John Daley <johnd...@cisco.com>; John Miller
> <john.mil...@atomicrules.com>; John W. Linville <linvi...@tuxdriver.com>;
> Wiles, Keith <keith.wi...@intel.com>; Kiran Kumar Kokkilagadda
> <kirankum...@marvell.com>; Lijun Ou <ouli...@huawei.com>; Liron Himi
> <lir...@marvell.com>; Long Li <lon...@microsoft.com>; Marcin Wojtas
> <m...@semihalf.com>; Martin Spinler <spin...@cesnet.cz>; Matan Azrad
> <ma...@nvidia.com>; Matt Peters <matt.pet...@windriver.com>; Maxime
> Coquelin <maxime.coque...@redhat.com>; Michal Krawczyk
> <m...@semihalf.com>; Min Hu (Connor <humi...@huawei.com>; Pradeep
> Kumar Nalla <pna...@marvell.com>; Nithin Kumar Dabilpuram
> <ndabilpu...@marvell.com>; Qiming Yang <qiming.y...@intel.com>; Qi Zhang
> <qi.z.zh...@intel.com>; Radha Chintakuntla <rad...@marvell.com>; Rahul
> Lakkireddy <rahul.lakkire...@chelsio.com>; Rasesh Mody
> <rm...@marvell.com>; Rosen Xu <rosen...@intel.com>; Sachin Saxena
> <sachin.sax...@oss.nxp.com>; Satha Koteswara Rao Kottidi
> <skotesh...@marvell.com>; Shahed Shaikh <shsha...@marvell.com>; Shai
> Brandes <shaib...@amazon.com>; Shepard Siegel
> <shepard.sie...@atomicrules.com>; Somalapuram Amaranath
> <asoma...@amd.com>; Somnath Kotur <somnath.ko...@broadcom.com>;
> Stephen Hemminger <sthem...@microsoft.com>; Steven Webster
> <steven.webs...@windriver.com>; Sunil Kumar Kori <sk...@marvell.com>;
> Tetsuya Mukawa <mtetsu...@gmail.com>; Veerasenareddy Burru
> <vbu...@marvell.com>; Viacheslav Ovsiienko <viachesl...@nvidia.com>; Xiao
> Wang <xiao.w.w...@intel.com>; Xiaoyun Wang
> <cloud.wangxiao...@huawei.com>; Yisen Zhuang
> <yisen.zhu...@huawei.com>; Yong Wang <yongw...@vmware.com>; Ziyang
> Xuan <xuanziya...@huawei.com>
> Subject: [EXT] Re: [PATCH] examples/l2fwd: add check of Rx packets count
> 
> External Email
> 
> ----------------------------------------------------------------------
> On Wed, May 25, 2022 at 2:44 PM Rahul Bhansali <rbhans...@marvell.com>
> wrote:
> >
> > An additional check is added to avoid extra processing if receive
> > packets are 0.
> >
> > Performance impact: with Marvell OCTEON TX2 platform, observed an
> > improvement by ~14%.
> >
> > Signed-off-by: Rahul Bhansali <rbhans...@marvell.com>
> 
> Acked-by: Jerin Jacob <jer...@marvell.com>
> 
> 
> > ---
> >  examples/l2fwd/main.c | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/examples/l2fwd/main.c b/examples/l2fwd/main.c index
> > 281c6b7a3f..28c498712e 100644
> > --- a/examples/l2fwd/main.c
> > +++ b/examples/l2fwd/main.c
> > @@ -286,6 +286,9 @@ l2fwd_main_loop(void)
> >                         nb_rx = rte_eth_rx_burst(portid, 0,
> >                                                  pkts_burst,
> > MAX_PKT_BURST);
> >
> > +                       if (unlikely(nb_rx == 0))
> > +                               continue;
> > +
> >                         port_statistics[portid].rx += nb_rx;
> >
> >                         for (j = 0; j < nb_rx; j++) {
> > --
> > 2.25.1
> >

Reply via email to