skd: Use __packed only when needed
authorBart Van Assche <bart.vanassche@wdc.com>
Thu, 17 Aug 2017 20:13:09 +0000 (13:13 -0700)
committerJens Axboe <axboe@kernel.dk>
Fri, 18 Aug 2017 14:45:29 +0000 (08:45 -0600)
Since needless use of __packed slows down access to data structures,
only use __packed when needed.

Signed-off-by: Bart Van Assche <bart.vanassche@wdc.com>
Cc: Christoph Hellwig <hch@lst.de>
Cc: Hannes Reinecke <hare@suse.de>
Cc: Johannes Thumshirn <jthumshirn@suse.de>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
drivers/block/skd_s1120.h

index 8044705cbbf94e7647e2447b298f65f16cc35517..de35f47e953c9e1b7a0e4773ce2d5f1a29880398 100644 (file)
@@ -10,8 +10,6 @@
 #ifndef SKD_S1120_H
 #define SKD_S1120_H
 
-#pragma pack(push, s1120_h, 1)
-
 /*
  * Q-channel, 64-bit r/w
  */
@@ -276,7 +274,7 @@ struct fit_comp_error_info {
        uint16_t        sks_low; /* 10: Sense Key Specific (LSW) */
        uint16_t        reserved3; /* 12: Part of additional sense bytes (unused) */
        uint16_t        uec; /* 14: Additional Sense Bytes */
-       uint64_t        per; /* 16: Additional Sense Bytes */
+       uint64_t        per __packed; /* 16: Additional Sense Bytes */
        uint8_t         reserved4[2]; /* 1E: Additional Sense Bytes (unused) */
 };
 
@@ -323,6 +321,4 @@ struct driver_inquiry_data {
        uint8_t         driver_version[0x14];
 };
 
-#pragma pack(pop, s1120_h)
-
 #endif /* SKD_S1120_H */