The iu struct definitions are usb packet definitions, so no alignment should
happen. Notice that assuming 32 bit alignment this does not make any
difference at all.

Signed-off-by: Hans de Goede <hdego...@redhat.com>
---
 include/linux/usb/uas.h | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/include/linux/usb/uas.h b/include/linux/usb/uas.h
index 772b66b..3fc8e8b 100644
--- a/include/linux/usb/uas.h
+++ b/include/linux/usb/uas.h
@@ -9,7 +9,7 @@ struct iu {
        __u8 iu_id;
        __u8 rsvd1;
        __be16 tag;
-};
+} __attribute__((__packed__));
 
 enum {
        IU_ID_COMMAND           = 0x01,
@@ -52,7 +52,7 @@ struct command_iu {
        __u8 rsvd7;
        struct scsi_lun lun;
        __u8 cdb[16];   /* XXX: Overflow-checking tools may misunderstand */
-};
+} __attribute__((__packed__));
 
 struct task_mgmt_iu {
        __u8 iu_id;
@@ -62,7 +62,7 @@ struct task_mgmt_iu {
        __u8 rsvd2;
        __be16 task_tag;
        struct scsi_lun lun;
-};
+} __attribute__((__packed__));
 
 /*
  * Also used for the Read Ready and Write Ready IUs since they have the
@@ -77,7 +77,7 @@ struct sense_iu {
        __u8 rsvd7[7];
        __be16 len;
        __u8 sense[SCSI_SENSE_BUFFERSIZE];
-};
+} __attribute__((__packed__));
 
 struct response_iu {
        __u8 iu_id;
@@ -85,7 +85,7 @@ struct response_iu {
        __be16 tag;
        __u8 add_response_info[3];
        __u8 response_code;
-};
+} __attribute__((__packed__));
 
 struct usb_pipe_usage_descriptor {
        __u8  bLength;
-- 
1.8.4.2

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to