Commit 51e155e4 authored by Ben Walker's avatar Ben Walker Committed by Konrad Sztyber
Browse files

bdev: Rearrange spdk_bdev_io_internal_fields to touch fewer cache lines 



This is just a simple swap for now. The following patches will do much
better packing to get the common case down to a single cache line.

Change-Id: Ia9c44043df699cf80002e344082bfbf8db1925cd
Signed-off-by: default avatarBen Walker <ben@nvidia.com>
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/21944


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

include/spdk/bdev_module.h

+6 −6
Original line number Diff line number Diff line
@@ -933,6 +933,12 @@ struct spdk_bdev_io_internal_fields { 
	/** Current tsc at submit time. Used to calculate latency at completion. */

	uint64_t submit_tsc;



	/** Entry to the list io_submitted of struct spdk_bdev_channel */

	TAILQ_ENTRY(spdk_bdev_io) ch_link;



	/** bdev_io pool entry */

	STAILQ_ENTRY(spdk_bdev_io) buf_link;



	/** Error information from a device */

	union {

		struct {
@@ -1019,12 +1025,6 @@ struct spdk_bdev_io_internal_fields { 
	 */

	TAILQ_ENTRY(spdk_bdev_io) link;



	/** Entry to the list need_buf of struct spdk_bdev. */

	STAILQ_ENTRY(spdk_bdev_io) buf_link;



	/** Entry to the list io_submitted of struct spdk_bdev_channel */

	TAILQ_ENTRY(spdk_bdev_io) ch_link;



	/** iobuf queue entry */

	struct spdk_iobuf_entry iobuf;