Thanks for the reminder. Will fix it. BRs, Xiao
> -----Original Message----- > From: Thomas Monjalon [mailto:tho...@monjalon.net] > Sent: Tuesday, April 17, 2018 2:07 AM > To: Wang, Xiao W <xiao.w.w...@intel.com> > Cc: Yigit, Ferruh <ferruh.yi...@intel.com>; Burakov, Anatoly > <anatoly.bura...@intel.com>; dev@dpdk.org; maxime.coque...@redhat.com; > Wang, Zhihong <zhihong.w...@intel.com>; Bie, Tiwei <tiwei....@intel.com>; > Tan, Jianfeng <jianfeng....@intel.com>; Liang, Cunming > <cunming.li...@intel.com>; Daly, Dan <dan.d...@intel.com> > Subject: Re: [PATCH v8 0/5] add ifcvf vdpa driver > > 16/04/2018 18:36, Ferruh Yigit: > > Hi Xiao, > > > > Getting following build error for 32bit [1], can you please check them? > > > > [1] > > .../dpdk/drivers/net/ifc/ifcvf_vdpa.c: In function ‘ifcvf_dma_map’: > > .../dpdk/drivers/net/ifc/ifcvf_vdpa.c:24:3: error: format ‘%lx’ expects > argument > > of type ‘long unsigned int’, but argument 6 has type ‘uint64_t {aka long > > long > > unsigned int}’ [-Werror=format=] > > Reminder from this recent post: > http://dpdk.org/ml/archives/dev/2018-February/090882.html > " > Most of the times, using %l is wrong (except when printing a long). > So next time you write %l, please think "I am probably wrong". > " > >
