Commit 5d25e3fc authored by Konrad Sztyber's avatar Konrad Sztyber
Browse files

bdev/malloc: don't retry failed requests 



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: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Change-Id: I85a522a7934d2d6f415620b9a323effefb91f299
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/17196


Reviewed-by: default avatarAleksey Marchuk <alexeymar@nvidia.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent d8f63f39

module/bdev/malloc/bdev_malloc.c

+3 −1
Original line number Diff line number Diff line
@@ -98,7 +98,9 @@ malloc_done(void *ref, int status) 


	if (status != 0) {

		if (status == -ENOMEM) {

			if (task->status == SPDK_BDEV_IO_STATUS_SUCCESS) {

				task->status = SPDK_BDEV_IO_STATUS_NOMEM;

			}

		} else {

			task->status = SPDK_BDEV_IO_STATUS_FAILED;

		}