Hi Akhil, > -----Original Message----- > From: dev <dev-boun...@dpdk.org> On Behalf Of Akhil Goyal > Sent: Friday, October 9, 2020 3:10 PM > To: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Doherty, Declan > <declan.dohe...@intel.com> > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH v4 2/2] test/crypto: add GMAC SGL tests > > Hi Pablo, > > > diff --git a/app/test/test_cryptodev.h b/app/test/test_cryptodev.h > > index 41542e055..57003733b 100644 > > --- a/app/test/test_cryptodev.h > > +++ b/app/test/test_cryptodev.h > > @@ -17,9 +17,9 @@ > > #define DEFAULT_NUM_QPS_PER_QAT_DEVICE (2) > > #define DEFAULT_BURST_SIZE (64) > > #define DEFAULT_NUM_XFORMS (2) > > -#define NUM_MBUFS (8191) > > +#define NUM_MBUFS (4095) > > #define MBUF_CACHE_SIZE (256) > > -#define MBUF_DATAPAYLOAD_SIZE (2048 + > > DIGEST_BYTE_LENGTH_SHA512) > > +#define MBUF_DATAPAYLOAD_SIZE (4096 + > > DIGEST_BYTE_LENGTH_SHA512) > > #define MBUF_SIZE (sizeof(struct rte_mbuf) + \ > > RTE_PKTMBUF_HEADROOM + MBUF_DATAPAYLOAD_SIZE) > > > Why are these changes done? Any specific requirement for GMAC? I believe > 2048 is also good enough.
I needed to reduce the maximum number of segments for the SGL tests, since Intel QAT PMD does not support more than 16, and this way both PMDs can be tested with the new tests. Thanks, Pablo > > Regards, > Akhil
