RE: [EXTERNAL] [PATCH v5 1/4] crypto/ipsec_mb: bump minimum IPsec Multi-buffer version

["Power, Ciara" <ciara.power@intel.com>]

> -----Original Message-----
> From: Wathsala Wathawana Vithanage <wathsala.vithanage@arm.com>
> Sent: Thursday, March 7, 2024 4:21 PM
> To: Power, Ciara <ciara.power@intel.com>; Patrick Robb <probb@iol.unh.edu>
> Cc: Akhil Goyal <gakhil@marvell.com>; Dooley, Brian <brian.dooley@intel.com>;
> Ji, Kai <kai.ji@intel.com>; De Lara Guarch, Pablo
> <pablo.de.lara.guarch@intel.com>; Aaron Conole <aconole@redhat.com>;
> dev@dpdk.org; Sivaramakrishnan, VenkatX
> <venkatx.sivaramakrishnan@intel.com>; thomas@monjalon.net; Marchand,
> David <david.marchand@redhat.com>; nd <nd@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