Oh, I misuse () when casting. Now it works.
-----Original Message----- From: Ye, Xiaolong Sent: Tuesday, September 3, 2019 4:35 PM To: Pei, Andy <andy....@intel.com> Cc: dev@dpdk.org; Zhang, Qi Z <qi.z.zh...@intel.com>; Yigit, Ferruh <ferruh.yi...@intel.com>; Xu, Rosen <rosen...@intel.com> Subject: Re: [PATCH v3] net/ipn3ke: setup MTU when HW init On 09/03, Pei, Andy wrote: >Hi Xiaolong, > >I tried, but compile issue exist. Er, what's the exact compiling issue after the type cast? We can see the similar usage in function like replay_num_bucket @lib/librte_ipsec/sa.c. Thanks, Xiaolong > >-----Original Message----- >From: Ye, Xiaolong >Sent: Tuesday, September 3, 2019 2:23 PM >To: Pei, Andy <andy....@intel.com> >Cc: dev@dpdk.org; Zhang, Qi Z <qi.z.zh...@intel.com>; Yigit, Ferruh ><ferruh.yi...@intel.com>; Xu, Rosen <rosen...@intel.com> >Subject: Re: [PATCH v3] net/ipn3ke: setup MTU when HW init > >On 09/03, Pei, Andy wrote: >[snip] >> tmp = RTE_MAX(tmp, RTE_ETHER_MIN_MTU); >> >>>+ if (tmp > IPN3KE_MAC_FRAME_SIZE_MAX - IPN3KE_ETH_OVERHEAD) >>>+ tmp = IPN3KE_MAC_FRAME_SIZE_MAX - IPN3KE_ETH_OVERHEAD; >> >> tmp = RTE_MIN(tmp, IPN3KE_MAC_FRAME_SIZE_MAX - IPN3KE_ETH_OVERHEAD); >> >>in the definition of TRE_MIN and TRE_MAX, typeof() is used. >>RTE_ETHER_MIN_MTU is MACRO, your expression cause complie issue. >> > >How about RTE_MAX(tmp, (uint32_t)RTE_ETHER_MIN_MTU) > >Thanks, >Xiaolong
