[dpdk-dev] [PATCH v5 1/3] app/testpmd: move dumping packets to a separate function

Raslan Darawsheh rasland at mellanox.com
Wed Oct 17 17:22:26 CEST 2018


Sending a newer version with the fix. 

Kindest regards,
Raslan Darawsheh

> -----Original Message-----
> From: Iremonger, Bernard <bernard.iremonger at intel.com>
> Sent: Wednesday, October 17, 2018 3:59 PM
> To: Yigit, Ferruh <ferruh.yigit at intel.com>; Raslan Darawsheh
> <rasland at mellanox.com>; Wu, Jingjing <jingjing.wu at intel.com>
> Cc: Thomas Monjalon <thomas at monjalon.net>; dev at dpdk.org; Shahaf
> Shuler <shahafs at mellanox.com>; Xueming(Steven) Li
> <xuemingl at mellanox.com>; Ori Kam <orika at mellanox.com>;
> jerin.jacob at caviumnetworks.com; david.marchand at 6wind.com
> Subject: RE: [PATCH v5 1/3] app/testpmd: move dumping packets to a
> separate function
> 
> Hi Raslan,
> 
> > -----Original Message-----
> > From: Yigit, Ferruh
> > Sent: Wednesday, October 17, 2018 1:28 PM
> > To: Raslan Darawsheh <rasland at mellanox.com>; Wu, Jingjing
> > <jingjing.wu at intel.com>
> > Cc: thomas at monjalon.net; dev at dpdk.org; shahafs at mellanox.com;
> > xuemingl at mellanox.com; orika at mellanox.com;
> > jerin.jacob at caviumnetworks.com; david.marchand at 6wind.com;
> Iremonger,
> > Bernard <bernard.iremonger at intel.com>
> > Subject: Re: [PATCH v5 1/3] app/testpmd: move dumping packets to a
> > separate function
> >
> > On 10/17/2018 8:27 AM, Raslan Darawsheh wrote:
> > > verbosity for the received/sent packets is needed in all of the
> > > forwarding engines so moving it to be in a separate function
> > >
> > > Acked-by: Bernard Iremonger <bernard.iremonger at intel.com>
> > > Signed-off-by: Raslan Darawsheh <rasland at mellanox.com>
> >
> > <...>
> >
> > > +static  inline void
> > > +dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf
> *pkts[],
> > > +	      uint16_t nb_pkts, int is_rx) {
> >
> > This is causing build error when only this patch applied:
> >
> > .../app/test-pmd/util.c:25:1: error: unused function 'dump_pkt_burst'
> > [-Werror,-Wunused-function]
> > dump_pkt_burst(uint16_t port_id, uint16_t queue, struct rte_mbuf
> > *pkts[], ^
> 
> Builds ok with gcc but fails with clang
> 
> Regards,
> 
> Bernard.



More information about the dev mailing list