> -----Original Message----- > From: dev <[email protected]> On Behalf Of Vikas Gupta > Sent: Wednesday, 12 August, 2020 08:31 > To: [email protected]; [email protected] > Cc: [email protected]; [email protected]; Vikas > Gupta <[email protected]>; Raveendra Padasalagi > <[email protected]> > Subject: [dpdk-dev] [PATCH v1 8/8] crypto/bcmfs: add crypto pmd into > cryptodev test > > Add test suites for supported algorithms by bcmfs crypto pmd > > Signed-off-by: Vikas Gupta <[email protected]> > Signed-off-by: Raveendra Padasalagi > <[email protected]> > Reviewed-by: Ajit Khaparde <[email protected]> > --- > app/test/test_cryptodev.c | 261
[Adam] The test suites set was refactored in recent months to use one big list of tests and run/skip them depending on the capability checks. I strongly propose to do this also for your new BCMFS PMD. Have a look at e.g. QAT PMD test suite in the same file. It has one function to properly initialize the run (check the device id) and then it uses big common area of individual tests (some of them are skipped because of requiring specific capabilities). Adam Dybkowski
