Hi Nelio, > -----Original Message----- > From: Nelio Laranjeiro [mailto:nelio.laranje...@6wind.com] > Sent: Wednesday, November 22, 2017 8:10 AM > To: Akhil Goyal <akhil.go...@nxp.com>; Doherty, Declan > <declan.dohe...@intel.com> > Cc: dev@dpdk.org; De Lara Guarch, Pablo > <pablo.de.lara.gua...@intel.com>; sta...@dpdk.org > 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: pablo.de.lara.gua...@intel.com > Cc: sta...@dpdk.org > > Signed-off-by: Nelio Laranjeiro <nelio.laranje...@6wind.com> > --- > 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
