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

bdev/malloc: don't retry failed requests

Browse Source 
If a request was marked as failed, we don't want to retry it, so we
shouldn't override its status with NOMEM.

Signed-off-by: Konrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I85a522a7934d2d6f415620b9a323effefb91f299
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17196
Reviewed-by: Aleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
This commit is contained in:
Konrad Sztyber 2023-03-14 11:01:17 +01:00 committed by David Ko
parent 177a74ee0c
commit a2a8f19ce2
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
module/bdev/malloc/bdev_malloc.c
View File

@ -98,7 +98,9 @@ malloc_done(void *ref, int status)
if (status != 0) { if (status != 0) {
if (status == -ENOMEM) { if (status == -ENOMEM) {
task->status = SPDK_BDEV_IO_STATUS_NOMEM; if (task->status == SPDK_BDEV_IO_STATUS_SUCCESS) {
task->status = SPDK_BDEV_IO_STATUS_NOMEM;
}
} else { } else {
task->status = SPDK_BDEV_IO_STATUS_FAILED; task->status = SPDK_BDEV_IO_STATUS_FAILED;
} }