Commit bfb676e9 authored Feb 11, 2022 by Yuriy Umanets Committed by Jim Harris Mar 08, 2022

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 <yumanets@nvidia.com>
Change-Id: Ie03412c72f41abf661b069d4b00eaf74f40261d6
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11629


Community-CI: Mellanox Build Bot
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: Paul Luse <paul.e.luse@intel.com>
Reviewed-by: Shuhei Matsumoto <smatsumoto@nvidia.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>

parent 3d0bae35

module/bdev/crypto/vbdev_crypto.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -1082,7 +1082,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;
		}