Hi Fan, Akhil, In that case, I don't think there is any other alternative than what Fan suggested (remove it now and add back along with capabilities). No objections to that from my end. Leaving that to Akhil's judgement.
I would still prefer adding a driver specific check, though. Thanks, Anoob > -----Original Message----- > From: Zhang, Roy Fan <roy.fan.zh...@intel.com> > Sent: Wednesday, February 23, 2022 8:22 PM > To: Akhil Goyal <gak...@marvell.com>; Anoob Joseph > <ano...@marvell.com>; Kusztal, ArkadiuszX <arkadiuszx.kusz...@intel.com> > Cc: Umesh Kartha <ukar...@marvell.com>; Ramkumar Balu > <rb...@marvell.com>; dev@dpdk.org > Subject: RE: [EXT] [PATCH] test/crypto: remove mod exp smaller modulus > cases > > Hi Akhil, > > Currently these tests will fail on QAT - is it possible to merge this patch > so that > QAT won't fail and we can add them back after the capability check is added > in the next release? > > Regards, > Fan > > > -----Original Message----- > > From: Akhil Goyal <gak...@marvell.com> > > Sent: Wednesday, February 23, 2022 2:38 PM > > To: Zhang, Roy Fan <roy.fan.zh...@intel.com>; Anoob Joseph > > <ano...@marvell.com>; Kusztal, ArkadiuszX > > <arkadiuszx.kusz...@intel.com> > > Cc: Umesh Kartha <ukar...@marvell.com>; Ramkumar Balu > > <rb...@marvell.com>; dev@dpdk.org > > Subject: RE: [EXT] [PATCH] test/crypto: remove mod exp smaller modulus > > cases > > > > Hi Anoob/Fan, > > We no more add driver specific checks in test app. Everything comes > > from capabilities. > > We may defer this patch to next release till we have something in > > capability or some other way. > > > > > Hi Anoob, > > > > > > Make sense. Will do ASAP. > > > > > > Regards, > > > Fan > > >
