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

module/accel/idxd: don't increment flow control on submission error

Browse Source 
Shouldn't really ever happen but is somehow things get out of whack
and we get a busy back from the low level lib we don't want to
increment our flow control counter.

Signed-off-by: paul luse <paul.e.luse@intel.com>
Change-Id: I49176e8ad2bf6f658ac970efea5db89eebc747f1
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/9232
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Ben Walker <benjamin.walker@intel.com>
This commit is contained in:
paul luse 2021-08-19 16:00:53 -04:00 committed by Tomasz Zawadzki
parent ab48766fd1
commit 95e4b24d5e
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
module/accel/idxd/accel_engine_idxd.c
View File

@ -144,7 +144,6 @@ _process_single_task(struct spdk_io_channel *ch, struct spdk_accel_task *task)
chan->state = IDXD_CHANNEL_PAUSED; chan->state = IDXD_CHANNEL_PAUSED;
return -EBUSY; return -EBUSY;
} }
chan->num_outstanding++;
switch (task->op_code) { switch (task->op_code) {
case ACCEL_OPCODE_MEMMOVE: case ACCEL_OPCODE_MEMMOVE:
@ -178,6 +177,10 @@ _process_single_task(struct spdk_io_channel *ch, struct spdk_accel_task *task)
break; break;
} }
if (rc == 0) {
chan->num_outstanding++;
}
return rc; return rc;
} }