ivampiresp/Spdk
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bdev/crypto: Fixed bdev_io double completion

Browse Source 
_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>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11629
(cherry picked from bfb676e93d)
Change-Id: I3d754d5aba7b222e99b7035cfe1d52461fefb9a2
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/11857
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Konrad Sztyber <konrad.sztyber@intel.com>
This commit is contained in:
Yuriy Umanets 2022-02-11 14:29:45 +02:00 committed by Keith Lucas
parent 703fceb4a9
commit bae4cb929a
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
module/bdev/crypto/vbdev_crypto.c
View File

@ -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 /* 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. * busy, fail it now as the poller won't know anything about it.
*/ */
_crypto_operation_complete(bdev_io);
rc = -EINVAL; rc = -EINVAL;
goto error_attach_session; goto error_attach_session;
} }