[dpdk-dev] [PATCH] cryptodev: add ff_disable field in cryptodev config

Akhil Goyal akhil.goyal at nxp.com
Fri Jun 21 15:10:20 CEST 2019


Hi Anoob,

> 
> Adding a new field, ff_disable, to allow applications to control the
> features enabled on the crypto device. This would allow for efficient
> usage of HW/SW offloads by disabling the features not required by the
> application.
> 
> Signed-off-by: Anoob Joseph <anoobj at marvell.com>

I see a compilation issue with this change. 
/home/akhil/up/dpdk-next-crypto/examples/fips_validation/fips_dev_self_test.c: In function 'fips_dev_auto_test_init':
/home/akhil/up/dpdk-next-crypto/examples/fips_validation/fips_dev_self_test.c:1560:9: error: missing initializer for field 'ff_disable' of 'struct rte_cryptodev_config' [-Werror=missing-field-initializers]
  struct rte_cryptodev_config conf = {rte_cryptodev_socket_id(dev_id), 1};
         ^~~~~~~~~~~~~~~~~~~~
In file included from /home/akhil/up/dpdk-next-crypto/examples/fips_validation/fips_dev_self_test.c:5:0:
/home/akhil/up/dpdk-next-crypto/arm64-dpaa-linuxapp-gcc/include/rte_cryptodev.h:615:11: note: 'ff_disable' declared here
  uint64_t ff_disable;
           ^~~~~~~~~~

Please add a patch for all applications which use struct rte_cryptodev_config and set the default values of its parameters.

> ---
>  doc/guides/rel_notes/deprecation.rst   | 14 --------------
>  doc/guides/rel_notes/release_19_08.rst | 11 ++++++++++-
>  lib/librte_cryptodev/Makefile          |  2 +-
>  lib/librte_cryptodev/meson.build       |  2 +-
>  lib/librte_cryptodev/rte_cryptodev.h   |  7 +++++++
>  5 files changed, 19 insertions(+), 17 deletions(-)
> 


More information about the dev mailing list