Hi Pablo, Thanks for reviewing
> -----Original Message----- > From: De Lara Guarch, Pablo > Sent: Friday, December 21, 2018 4:52 AM > To: Trahe, Fiona <[email protected]>; [email protected] > Cc: Zhang, Roy Fan <[email protected]>; [email protected] > 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: [email protected] > > Cc: De Lara Guarch, Pablo <[email protected]>; Trahe, Fiona > > <[email protected]>; Zhang, Roy Fan <[email protected]>; > > [email protected] > > 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 <[email protected]> > > 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
