app/test: handle -ENOTSUP from session init
Checks
Commit Message
The session init routine rte_cryptodev_sym_session_init(),
could return -ENOTSUP when the requested algo combination
is not supported by the PMD. This should be treated as
unsupported features. For other return values like -EINVAL
or -ENOMEM the test can be treated as failure.
Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
Acked-by: Anoob Joseph <anoobj@marvell.com>
---
v2:
* Updated test_msg to print UNSUPPORTED for the test case.
app/test/test_cryptodev_blockcipher.c | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
Comments
> The session init routine rte_cryptodev_sym_session_init(),
> could return -ENOTSUP when the requested algo combination
> is not supported by the PMD. This should be treated as
> unsupported features. For other return values like -EINVAL
> or -ENOMEM the test can be treated as failure.
>
> Signed-off-by: Ankur Dwivedi <adwivedi@marvell.com>
>
> Acked-by: Anoob Joseph <anoobj@marvell.com>
Applied to dpdk-next-crypto
Thanks.
@@ -427,9 +427,13 @@
nb_iterates == 0) {
sess = rte_cryptodev_sym_session_create(sess_mpool);
- rte_cryptodev_sym_session_init(dev_id, sess, init_xform,
- sess_priv_mpool);
- if (!sess) {
+ status = rte_cryptodev_sym_session_init(dev_id, sess,
+ init_xform, sess_priv_mpool);
+ if (status == -ENOTSUP) {
+ snprintf(test_msg, BLOCKCIPHER_TEST_MSG_LEN, "UNSUPPORTED");
+ goto error_exit;
+ }
+ if (!sess || status < 0) {
snprintf(test_msg, BLOCKCIPHER_TEST_MSG_LEN, "line %u "
"FAILED: %s", __LINE__,
"Session creation failed");