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 <[email protected]> > >>> Signed-off-by: Thomas Monjalon <[email protected]> > >> > >> 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 <[email protected]> > > > > I confirm it fixes the build issue: > > Tested-by: Ferruh Yigit <[email protected]>
Applied, thanks
