Thanks Pablo for the fix. I am ok with this change Acked-by: Vipin Varghese <[email protected]>
> -----Original Message----- > From: dev [mailto:[email protected]] On Behalf Of Pablo de Lara > Sent: Friday, May 18, 2018 6:54 PM > To: Doherty, Declan <[email protected]> > Cc: [email protected]; De Lara Guarch, Pablo <[email protected]>; > [email protected] > Subject: [dpdk-dev] [PATCH] cryptodev: add missing feature string > > Security protocol flag string was not added when the actual flag was added. > > Fixes: eadb4fa1e1fe ("cryptodev: support security APIs") > Cc: [email protected] > > Signed-off-by: Pablo de Lara <[email protected]> > --- > lib/librte_cryptodev/rte_cryptodev.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/lib/librte_cryptodev/rte_cryptodev.c > b/lib/librte_cryptodev/rte_cryptodev.c > index 2a95a351f..3712caf73 100644 > --- a/lib/librte_cryptodev/rte_cryptodev.c > +++ b/lib/librte_cryptodev/rte_cryptodev.c > @@ -346,6 +346,8 @@ rte_cryptodev_get_feature_name(uint64_t flag) > return "CPU_NEON"; > case RTE_CRYPTODEV_FF_CPU_ARM_CE: > return "CPU_ARM_CE"; > + case RTE_CRYPTODEV_FF_SECURITY: > + return "SECURITY_PROTOCOL"; > default: > return NULL; > } > -- > 2.17.0
