Hi Pablo, Thanks for reviewing
> -----Original Message----- > From: De Lara Guarch, Pablo > Sent: Friday, December 21, 2018 4:52 AM > To: Trahe, Fiona <fiona.tr...@intel.com>; dev@dpdk.org > Cc: Zhang, Roy Fan <roy.fan.zh...@intel.com>; akhil.go...@nxp.com > Subject: RE: [PATCH] test/crypto: don't attempt unsupported SGL tests on > aesni mb PMD > > > > > -----Original Message----- > > From: Trahe, Fiona > > Sent: Friday, December 21, 2018 12:02 AM > > To: dev@dpdk.org > > Cc: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; Trahe, Fiona > > <fiona.tr...@intel.com>; Zhang, Roy Fan <roy.fan.zh...@intel.com>; > > akhil.go...@nxp.com > > Subject: [PATCH] test/crypto: don't attempt unsupported SGL tests on aesni > > mb PMD > > > > Remove AESNI_MB flag from SGL test cases which it doesn't support. > > > > Signed-off-by: Fiona Trahe <fiona.tr...@intel.com> > > Patch looks good, but I wonder if this is a fix and therefore, if it needs to > be backported. [Fiona] It did previously print a message saying the device doesn't support the test case, so I think more of a style change than a bug. I think this way is clearer but not necessary to backport cause only really of interest to developers and dev of new test cases are usually done on latest. > > Thanks, > Pablo
