From 09f887bdc4c727626c38a0c8be54a8faa4ab5f8b Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Thu, 11 Nov 2021 13:17:01 -0700 Subject: [PATCH] accel: insert freed tasks at the head of the tailq 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 Change-Id: I67c23e3d89cda16f94b1770eada5465015ddb6ff Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/10192 Community-CI: Broadcom CI Community-CI: Mellanox Build Bot Tested-by: SPDK CI Jenkins Reviewed-by: John Kariuki Reviewed-by: Changpeng Liu Reviewed-by: Tomasz Zawadzki --- lib/accel/accel_engine.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/accel/accel_engine.c b/lib/accel/accel_engine.c index 949208aab..8da19cf85 100644 --- a/lib/accel/accel_engine.c +++ b/lib/accel/accel_engine.c @@ -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 * 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); /* If this task is part of a batch, check for completion of the batch. */