Hi Nelio, > -----Original Message----- > From: Nelio Laranjeiro [mailto:[email protected]] > Sent: Wednesday, November 22, 2017 8:10 AM > To: Akhil Goyal <[email protected]>; Doherty, Declan > <[email protected]> > Cc: [email protected]; De Lara Guarch, Pablo > <[email protected]>; [email protected] > Subject: [PATCH 2/3] crypto: fix pedentic compilation errors > > /root/dpdk/x86_64-native-linuxapp-gcc/include/rte_crypto.h:126:28: > error: ISO C forbids zero-size array ‘sym’ [-Werror=pedantic] > struct rte_crypto_sym_op sym[0]; > ^~~ > > Fixes: d2a4223c4c6d ("cryptodev: do not store pointer to op specific > params") > Cc: [email protected] > Cc: [email protected] > > Signed-off-by: Nelio Laranjeiro <[email protected]> > --- > lib/librte_cryptodev/rte_crypto.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/librte_cryptodev/rte_crypto.h > b/lib/librte_cryptodev/rte_crypto.h > index 3d672fe7d..dc6e91d1d 100644 > --- a/lib/librte_cryptodev/rte_crypto.h > +++ b/lib/librte_cryptodev/rte_crypto.h > @@ -123,7 +123,7 @@ struct rte_crypto_op { > > RTE_STD_C11 > union { > - struct rte_crypto_sym_op sym[0]; > + struct rte_crypto_sym_op *sym;
This would add extra 64 bits that are not necessary, as the rte_crypto_sym_op structure is always placed after rte_crypto_op (this is just a marker). Is there any other way to fix this error? Which compiler are you using? Thanks, Pablo
