Commit 8df37f7f authored by Ben Walker's avatar Ben Walker Committed by Jim Harris
Browse files

bdev: Pack some single bytes into the reserved region in 


spdk_bdev_io_internal_fields

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


Community-CI: Broadcom CI <spdk-ci.pdl@broadcom.com>
Reviewed-by: default avatarShuhei Matsumoto <smatsumoto@nvidia.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 14bad452

include/spdk/bdev_module.h

+20 −20
Original line number Diff line number Diff line
@@ -930,20 +930,6 @@ struct spdk_bdev_io_internal_fields { 
	/** The bdev I/O channel that this was handled on. */

	struct spdk_bdev_channel *ch;



	uint8_t	reserved[8];



	/** The bdev descriptor that was used when submitting this I/O. */

	struct spdk_bdev_desc *desc;



	/** User function that will be called when this completes */

	spdk_bdev_io_completion_cb cb;



	/** Context that will be passed to the completion callback */

	void *caller_ctx;



	/** Current tsc at submit time. Used to calculate latency at completion. */

	uint64_t submit_tsc;



	union {

		struct {
@@ -970,6 +956,26 @@ struct spdk_bdev_io_internal_fields { 
		uint8_t raw;

	} f;



	/** Status for the IO */

	int8_t status;



	/** Retry state (resubmit, re-pull, re-push, etc.) */

	uint8_t retry_state;



	uint8_t	reserved[5];



	/** The bdev descriptor that was used when submitting this I/O. */

	struct spdk_bdev_desc *desc;



	/** User function that will be called when this completes */

	spdk_bdev_io_completion_cb cb;



	/** Context that will be passed to the completion callback */

	void *caller_ctx;



	/** 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;
@@ -1001,12 +1007,6 @@ struct spdk_bdev_io_internal_fields { 
		int aio_result;

	} error;



	/** Status for the IO */

	int8_t status;



	/** Retry state (resubmit, re-pull, re-push, etc.) */

	uint8_t retry_state;



	struct {

		/** stored user callback in case we split the I/O and use a temporary callback */

		spdk_bdev_io_completion_cb stored_user_cb;