diff options
| author | Janne Peltonen <janne.peltonen@nokia.com> | 2021-02-19 16:05:47 +0200 |
|---|---|---|
| committer | Matias Elo <matias.elo@nokia.com> | 2021-02-23 10:48:31 +0200 |
| commit | 04e0316e828f118170a8075f5fc1c1d0dd60bddb (patch) | |
| tree | 25a0741ede464ea118a3c47c05c352885991e82d /platform/linux-dpdk/odp_crypto.c | |
| parent | 615d1afdaf0f03b8863c5f0ee9707c108837e924 (diff) | |
linux-dpdk: crypto: fix max_sessions capability
DPDK sets max_nb_sessions zero in rte_cryptodev_info_get() when the device
does not have any limit on the number of sessions. It has been so since
DPDK version 18.08 but ODP-DPDK still reports zero max_nb_sessions as
zero max sessions in ODP crypto capability. Fix the problem by taking
max_nb_sessions of a device into account only when it is non-zero.
Signed-off-by: Janne Peltonen <janne.peltonen@nokia.com>
Reviewed-by: Matias Elo <matias.elo@nokia.com>
Diffstat (limited to 'platform/linux-dpdk/odp_crypto.c')
| -rw-r--r-- | platform/linux-dpdk/odp_crypto.c | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/platform/linux-dpdk/odp_crypto.c b/platform/linux-dpdk/odp_crypto.c index 5ed52aaf2..d37dc40db 100644 --- a/platform/linux-dpdk/odp_crypto.c +++ b/platform/linux-dpdk/odp_crypto.c @@ -1,5 +1,5 @@ /* Copyright (c) 2017-2018, Linaro Limited - * Copyright (c) 2020, Nokia + * Copyright (c) 2018-2021, Nokia * All rights reserved. * * SPDX-License-Identifier: BSD-3-Clause @@ -622,6 +622,7 @@ int odp_crypto_capability(odp_crypto_capability_t *capability) capability->sync_mode = ODP_SUPPORT_YES; capability->async_mode = ODP_SUPPORT_PREFERRED; + capability->max_sessions = MAX_SESSIONS; for (cdev_id = 0; cdev_id < cdev_count; cdev_id++) { struct rte_cryptodev_info dev_info; @@ -635,18 +636,12 @@ int odp_crypto_capability(odp_crypto_capability_t *capability) capability->hw_auths = capability->auths; } - /* Read from the device with the lowest max_nb_sessions */ - if (capability->max_sessions > dev_info.sym.max_nb_sessions) - capability->max_sessions = dev_info.sym.max_nb_sessions; - - if (capability->max_sessions == 0) + /* Report the lowest max_nb_sessions of all devices */ + if (dev_info.sym.max_nb_sessions != 0 && + dev_info.sym.max_nb_sessions < capability->max_sessions) capability->max_sessions = dev_info.sym.max_nb_sessions; } - /* Make sure the session count doesn't exceed MAX_SESSIONS */ - if (capability->max_sessions > MAX_SESSIONS) - capability->max_sessions = MAX_SESSIONS; - return 0; } |