04/04/2019 17:13, Ferruh Yigit: > On 4/4/2019 3:14 PM, Eads, Gage wrote: > >> 04/04/2019 15:00, Thomas Monjalon: > >>> When enabling pedantic compilation with > >> CONFIG_RTE_LIBRTE_MLX5_DEBUG, > >>> the compiler complains about non standard 128-bit integer type: > >>> > >>> include/rte_atomic_64.h:223:3: error: > >>> ISO C does not support ‘__int128’ types [-Werror=pedantic] > >>> > >>> It must be marked as an extension of the standard C language to be > >>> accepted in pedantic compilation. > >>> > >>> Reported-by: Ferruh Yigit <ferruh.yi...@intel.com> > >>> Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > >> > >> Forgot one line: > >> > >> Fixes: 640c5f09ef2c ("eal/x86: add 128-bit atomic compare exchange") > >> > >> Note: It seems only Ferruh can reproduce it. > >> I wonder whether the RTE_STD_C11 of the union is hiding the issue. > > > > With that change: > > Acked-by: Gage Eads <gage.e...@intel.com> > > > > I confirm it fixes the build issue: > > Tested-by: Ferruh Yigit <ferruh.yi...@intel.com>
Applied, thanks