>
> Hi Anoob.
>
> Thanks for this longer explanation.
> I think can remove the test as intended.
>
> Acked-by: Adam Dybkowski
>
Acked-by: Akhil Goyal
Applied to dpdk-next-crypto
Thanks.
rayana Prasad Raju Athreya ; Ankur Dwivedi
> ; De Lara Guarch, Pablo
> ; dev@dpdk.org
> Subject: RE: [dpdk-dev] [PATCH] test/crypto: remove tests for unsupported
> descriptors
>
> Hi Adam,
>
> The crypto library doesn't specify an upper limit for the nb_descriptors to b
; To: Akhil Goyal ; Doherty, Declan
> >
> > Cc: Anoob Joseph ; Narayana Prasad
> > ; Ankur Dwivedi ; De Lara
> > Guarch, Pablo ; dev@dpdk.org
> > Subject: [dpdk-dev] [PATCH] test/crypto: remove tests for unsupported
> > descriptors
> &g
Hi Adam,
>
> Hi Anoob.
>
> What's wrong with this test? Is this unit test failing on any PMD now?
> I've checked on several (QAT, OpenSSL, SW ZUC, SW KASUMI, SW SNOW3G,
> SCHEDULER) and it passes everywhere. Then why should we remove it
> completely?
>
The problem is not with the valid cases. I
Behalf Of Anoob Joseph
> Sent: Tuesday, 12 May, 2020 09:12
> To: Akhil Goyal ; Doherty, Declan
>
> Cc: Anoob Joseph ; Narayana Prasad
> ; Ankur Dwivedi ; De Lara
> Guarch, Pablo ; dev@dpdk.org
> Subject: [dpdk-dev] [PATCH] test/crypto: remove tests for unsupported
> desc
Cryptodev doesn't limit the number of descriptors that can be supported
by the PMD.
Signed-off-by: Anoob Joseph
---
app/test/test_cryptodev.c | 45 -
1 file changed, 45 deletions(-)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptodev.c
in
6 matches
Mail list logo
