diff options
author | Caleb Sander <csander@purestorage.com> | 2023-03-20 09:57:36 -0600 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2023-03-22 09:19:56 +0100 |
commit | aa01c67de5926fdb276793180564f172c55fb0d7 (patch) | |
tree | dd1ab337c4a58a228ff1aeb4d5d9258c27f1acb1 /include | |
parent | def84ab600b71ea3fcc422a876d5d0d0daa7d4f3 (diff) |
nvme-tcp: fix nvme_tcp_term_pdu to match spec
The FEI field of C2HTermReq/H2CTermReq is 4 bytes but not 4-byte-aligned
in the NVMe/TCP specification (it is located at offset 10 in the PDU).
Split it into two 16-bit integers in struct nvme_tcp_term_pdu
so no padding is inserted. There should also be 10 reserved bytes after.
There are currently no users of this type.
Fixes: fc221d05447aa6db ("nvme-tcp: Add protocol header")
Reported-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Caleb Sander <csander@purestorage.com>
Reviewed-by: Sagi Grimberg <sagi@grimberg.me>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'include')
-rw-r--r-- | include/linux/nvme-tcp.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/include/linux/nvme-tcp.h b/include/linux/nvme-tcp.h index 75470159a194..57ebe1267f7f 100644 --- a/include/linux/nvme-tcp.h +++ b/include/linux/nvme-tcp.h @@ -115,8 +115,9 @@ struct nvme_tcp_icresp_pdu { struct nvme_tcp_term_pdu { struct nvme_tcp_hdr hdr; __le16 fes; - __le32 fei; - __u8 rsvd[8]; + __le16 feil; + __le16 feiu; + __u8 rsvd[10]; }; /** |