Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 31 Jul 2021 00:21:38 GMT
From:      Warner Losh <imp@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: d85d7244d8bb - stable/12 - nvme: Fix alignment on nvme structures
Message-ID:  <202107310021.16V0LctC052240@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/12 has been updated by imp:

URL: https://cgit.FreeBSD.org/src/commit/?id=d85d7244d8bb20d5d5040045856c8038333736ce

commit d85d7244d8bb20d5d5040045856c8038333736ce
Author:     Warner Losh <imp@FreeBSD.org>
AuthorDate: 2021-07-02 21:58:19 +0000
Commit:     Warner Losh <imp@FreeBSD.org>
CommitDate: 2021-07-31 00:02:53 +0000

    nvme: Fix alignment on nvme structures
    
    Remove __packed from nvme_command, nvme_completion and
    nvme_dsm_trim. Add super-alignment to nvme_completion since it's always
    at least that aligned in hardware (and in our existing uses of it
    embedded in structures). It generates better code in
    nvme_qpair_process_completions on riscv64 because otherwise the ABI
    assumes a 4-byte alignment, and the same on all other platforms.
    
    Reviewed by:            jrtc27@, mav@, chuck@
    Sponsored by:           Netflix
    Differential Revision:  https://reviews.freebsd.org/D31001
    
    (cherry picked from commit fea3cf1d6da0acf40bc1d3dadeeea7eeccbc10dd)
---
 sys/dev/nvme/nvme.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/sys/dev/nvme/nvme.h b/sys/dev/nvme/nvme.h
index d53bf3dd9357..fa0ac546e9c8 100644
--- a/sys/dev/nvme/nvme.h
+++ b/sys/dev/nvme/nvme.h
@@ -578,8 +578,8 @@ struct nvme_registers {
 	struct {
 	    uint32_t	sq_tdbl; /* submission queue tail doorbell */
 	    uint32_t	cq_hdbl; /* completion queue head doorbell */
-	} doorbell[1] __packed;
-} __packed;
+	} doorbell[1];
+};
 
 _Static_assert(sizeof(struct nvme_registers) == 0x1008, "bad size for nvme_registers");
 
@@ -612,7 +612,7 @@ struct nvme_command {
 	uint32_t cdw13;		/* command-specific */
 	uint32_t cdw14;		/* command-specific */
 	uint32_t cdw15;		/* command-specific */
-} __packed;
+};
 
 _Static_assert(sizeof(struct nvme_command) == 16 * 4, "bad size for nvme_command");
 
@@ -631,7 +631,7 @@ struct nvme_completion {
 	/* dword 3 */
 	uint16_t		cid;	/* command identifier */
 	uint16_t		status;
-} __packed;
+} __aligned(8);	/* riscv: nvme_qpair_process_completions has better code gen */
 
 _Static_assert(sizeof(struct nvme_completion) == 4 * 4, "bad size for nvme_completion");
 
@@ -639,7 +639,7 @@ struct nvme_dsm_range {
 	uint32_t attributes;
 	uint32_t length;
 	uint64_t starting_lba;
-} __packed;
+};
 
 /* Largest DSM Trim that can be done */
 #define NVME_MAX_DSM_TRIM		4096



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202107310021.16V0LctC052240>