From bae4cb929a9120fc67eb9deb0e03c0f69dbbf123 Mon Sep 17 00:00:00 2001 From: Yuriy Umanets Date: Fri, 11 Feb 2022 14:29:45 +0200 Subject: [PATCH] bdev/crypto: Fixed bdev_io double completion _crypto_operation_complete(bdev_io) should not be called in _crypto_operation() because it is done by caller function on read or write. Signed-off-by: Yuriy Umanets Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11629 (cherry picked from bfb676e93df207bb60decd82ac083ec23d44bb5e) Change-Id: I3d754d5aba7b222e99b7035cfe1d52461fefb9a2 Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11857 Tested-by: SPDK CI Jenkins Community-CI: Broadcom CI Reviewed-by: Tomasz Zawadzki Reviewed-by: Konrad Sztyber --- module/bdev/crypto/vbdev_crypto.c | 1 - 1 file changed, 1 deletion(-) diff --git a/module/bdev/crypto/vbdev_crypto.c b/module/bdev/crypto/vbdev_crypto.c index 3f6ba39c9..a8cc8e155 100644 --- a/module/bdev/crypto/vbdev_crypto.c +++ b/module/bdev/crypto/vbdev_crypto.c @@ -899,7 +899,6 @@ _crypto_operation(struct spdk_bdev_io *bdev_io, enum rte_crypto_cipher_operation /* If nothing was enqueued, but the last one wasn't because of * busy, fail it now as the poller won't know anything about it. */ - _crypto_operation_complete(bdev_io); rc = -EINVAL; goto error_attach_session; }