Commit bcbd379b authored by Alexey Marchuk's avatar Alexey Marchuk Committed by Tomasz Zawadzki
Browse files

accel: Reorder task structure 



Move fields used in completion path to the
beginning of the structure. In that way
we use only 1 cache line in cpl path

Signed-off-by: default avatarAlexey Marchuk <alexeymar@nvidia.com>
Change-Id: I0a3937ef5a07661aed9c5365e9832c2a972720dd
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21122


Reviewed-by: default avatarJacek Kalwas <jacek.kalwas@intel.com>
Reviewed-by: default avatarBen Walker <ben@nvidia.com>
Reviewed-by: default avatarKonrad Sztyber <konrad.sztyber@intel.com>
Community-CI: Mellanox Build Bot
Reviewed-by: default avatarJim Harris <jim.harris@samsung.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
parent f00c7cd7

include/spdk/accel_module.h

+7 −7
Original line number Diff line number Diff line
@@ -85,6 +85,13 @@ struct spdk_accel_task_aux_data { 
};



struct spdk_accel_task {

	TAILQ_ENTRY(spdk_accel_task)	seq_link;

	STAILQ_ENTRY(spdk_accel_task)	link;

	/* Uses enum spdk_accel_opcode */

	uint8_t				op_code;

	uint8_t				flags;

	bool				has_aux;

	int16_t				status;

	struct accel_io_channel		*accel_ch;

	struct spdk_accel_sequence	*seq;

	union {
@@ -134,13 +141,6 @@ struct spdk_accel_task { 
		uint32_t		block_size; /* for crypto op */

	};

	uint64_t			iv; /* Initialization vector (tweak) for crypto op */

	/* Uses enum spdk_accel_opcode */

	uint8_t				op_code;

	bool				has_aux;

	uint8_t				flags;

	int16_t				status;

	STAILQ_ENTRY(spdk_accel_task)	link;

	TAILQ_ENTRY(spdk_accel_task)	seq_link;

	struct spdk_accel_task_aux_data	*aux;

};