Re: [PATCH 21.11] build: suppress rte_crypto_asym_op abi check

[Kevin Traynor <ktraynor@redhat.com>]

On 16/03/2022 16:14, David Marchand wrote:
> On Wed, Mar 16, 2022 at 12:45 PM Kevin Traynor <ktraynor@redhat.com> wrote:
>>
>> Suppress of rte_crypto_asym_op was added in dpdk main branch
>> as part of
>> a678c5d36589 ("cryptodev: add DSA random number k")
>>
>> That is not backported to 21.11 branch, but other commit
>> f092922c3674 ("cryptodev: fix RSA key type name")
>> also needs this suppression.
>>

This caused a failure in UNH CI
https://lab.dpdk.org/results/dashboard/tarballs/19132

>> Add the suppression here.
>>
>> Fixes: f092922c3674 ("cryptodev: fix RSA key type name")
>> Cc: arkadiuszx.kusztal@intel.com
>>
>> Reported-by: https://lab.dpdk.org/results/dashboard/tarballs/19132
> 
> In dpdk history, the Reported-by tag has always been used with people
> names, and not URI.
> It is probably not an issue (except maybe for people doing stats on
> those tags), but I would rather keep this URI as a comment in the
> commitlog.
> 

ok, sure. I changed it to free form in the commit msg as above instead.

>> Reported-by: David Marchand <david.marchand@redhat.com>
>> Signed-off-by: Kevin Traynor <ktraynor@redhat.com>
> 
> The change lgtm,
> 
> Reviewed-by: David Marchand <david.marchand@redhat.com>

Thanks. Applied with changes above.

> 
>