[PATCH v2] test/crypto: skip plain text compare for null cipher

[De Lara Guarch, Pablo]

Hi Anoob,

> -----Original Message-----
> From: Power, Ciara <ciara.power at intel.com>
> Sent: Wednesday, November 10, 2021 5:01 PM
> To: Anoob Joseph <anoobj at marvell.com>; Akhil Goyal <gakhil at marvell.com>;
> Doherty, Declan <declan.doherty at intel.com>; Zhang, Roy Fan
> <roy.fan.zhang at intel.com>; De Lara Guarch, Pablo
> <pablo.de.lara.guarch at intel.com>
> Cc: Jerin Jacob <jerinj at marvell.com>; Archana Muniganti
> <marchana at marvell.com>; Tejasree Kondoj <ktejasree at marvell.com>; Hemant
> Agrawal <hemant.agrawal at nxp.com>; Nicolau, Radu
> <radu.nicolau at intel.com>; Gagandeep Singh <g.singh at nxp.com>; Ji, Kai
> <kai.ji at intel.com>; dev at dpdk.org; adamx.dybkowski at intel.com; De Lara
> Guarch, Pablo <pablo.de.lara.guarch at intel.com>
> Subject: RE: [PATCH v2] test/crypto: skip plain text compare for null cipher
> 
> Hi Anoob,
> 
> >-----Original Message-----
> >From: Anoob Joseph <anoobj at marvell.com>
> >Sent: Wednesday 10 November 2021 13:04
> >To: Akhil Goyal <gakhil at marvell.com>; Doherty, Declan
> ><declan.doherty at intel.com>; Zhang, Roy Fan <roy.fan.zhang at intel.com>;
> >De Lara Guarch, Pablo <pablo.de.lara.guarch at intel.com>
> >Cc: Anoob Joseph <anoobj at marvell.com>; Jerin Jacob
> ><jerinj at marvell.com>; Archana Muniganti <marchana at marvell.com>;
> >Tejasree Kondoj <ktejasree at marvell.com>; Hemant Agrawal
> ><hemant.agrawal at nxp.com>; Nicolau, Radu <radu.nicolau at intel.com>;
> >Power, Ciara <ciara.power at intel.com>; Gagandeep Singh
> ><g.singh at nxp.com>; Ji, Kai <kai.ji at intel.com>; dev at dpdk.org;
> >adamx.dybkowski at intel.com
> >Subject: [PATCH v2] test/crypto: skip plain text compare for null
> >cipher
> >
> >NULL cipher is used for validating auth only cases. With NULL cipher,
> >validating plain text should not be done as the PMD is only expected to
> >update auth data.
> >
> >Fixes: e847fc512817 ("test/crypto: add encrypted digest case for
> >AES-CTR-
> >CMAC")
> >Cc: adamx.dybkowski at intel.com
> >
> >Signed-off-by: Anoob Joseph <anoobj at marvell.com>
> >---
> >Changes in v2:
> >- Made the skip applicable for in place also (comment from Pablo)
> >- Re-organized code as suggested by Pablo
> >- Made same change in test_mixed_auth_cipher_sgl (comment from Pablo)
> >
> > app/test/test_cryptodev.c | 63
> > ++++++++++++++++++++++++++--------------
> 
> Acked-by: Ciara Power <ciara.power at intel.com>

Wonder if this patch needs to CC stable, as it's fixing a commit from 2019?

Apart from this:
Acked-by: Pablo de Lara <pablo.de.lara.guarch at intel.com>