> -----Original Message----- > From: Wathsala Wathawana Vithanage <wathsala.vithan...@arm.com> > Sent: Thursday, March 7, 2024 4:21 PM > To: Power, Ciara <ciara.po...@intel.com>; Patrick Robb <pr...@iol.unh.edu> > Cc: Akhil Goyal <gak...@marvell.com>; Dooley, Brian <brian.doo...@intel.com>; > Ji, Kai <kai...@intel.com>; De Lara Guarch, Pablo > <pablo.de.lara.gua...@intel.com>; Aaron Conole <acon...@redhat.com>; > dev@dpdk.org; Sivaramakrishnan, VenkatX > <venkatx.sivaramakrish...@intel.com>; tho...@monjalon.net; Marchand, > David <david.march...@redhat.com>; nd <n...@arm.com> > Subject: RE: [EXTERNAL] [PATCH v5 1/4] crypto/ipsec_mb: bump minimum IPsec > Multi-buffer version > > > > > If the correct version is being installed and picked up, maybe there > > is something missing for that function definition in arm-ipsec-mb repo. > > Wathsala, can you check that please? > > We are working on reproducing this issue. Will update asap.
Thanks Wathsala. I think, for basic compile to work for both repos, any functions in the ipsec_mb.h file need to be defined in a C file to avoid issues. But, I understand only SNOW3G + ZUC are the focus for the Arm ipsec-mb repo, so a HMAC related function like the one throwing the error isn't used. Perhaps having empty stubs for all other functions such as the imb_hmac_ipad_opad would be sufficient to allow compiling, because PMD expects it to exist somewhere in the library. Thanks, Ciara
