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

accel: insert freed tasks at the head of the tailq

Browse Source 
We allocate tasks from the head, so it's better to
free them to the head too for better cache utilization.

Signed-off-by: Jim Harris <james.r.harris@intel.com>
Change-Id: I67c23e3d89cda16f94b1770eada5465015ddb6ff
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10192
Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Community-CI: Mellanox Build Bot
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: John Kariuki <John.K.Kariuki@intel.com>
Reviewed-by: Changpeng Liu <changpeng.liu@intel.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
This commit is contained in:
Jim Harris 2021-11-11 13:17:01 -07:00 committed by Keith Lucas
parent 847f30d986
commit 09f887bdc4
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/accel/accel_engine.c
View File

@ -122,7 +122,7 @@ spdk_accel_task_complete(struct spdk_accel_task *accel_task, int status)
* the accel task list is exhausted when there is recursive call to * the accel task list is exhausted when there is recursive call to
* allocate accel_task in user's call back function (cb_fn) * allocate accel_task in user's call back function (cb_fn)
*/ */
TAILQ_INSERT_TAIL(&accel_ch->task_pool, accel_task, link); TAILQ_INSERT_HEAD(&accel_ch->task_pool, accel_task, link);
cb_fn(cb_arg, status); cb_fn(cb_arg, status);
/* If this task is part of a batch, check for completion of the batch. */ /* If this task is part of a batch, check for completion of the batch. */