scsi: hpsa: Add an assert to prevent __packed reintroduction

Link: https://lore.kernel.org/r/[e-mail]
Fixes: f749d8b7a989 ("scsi: hpsa: Correct dev cmds outstanding for retried cmds")
CC: [e-mail]
CC: [e-mail]
CC: [e-mail]
CC: Joe Szczypek <[e-mail]>
CC: Scott Benesh <[e-mail]>
CC: Scott Teel <[e-mail]>
CC: Tomas Henzl <[e-mail]>
CC: "Martin K. Petersen" <[e-mail]>
CC: Don Brace <[e-mail]>
Reported-by: John Paul Adrian Glaubitz <[e-mail]>
Suggested-by: Don Brace <[e-mail]>
Reviewed-by: Arnd Bergmann <[e-mail]>
Signed-off-by: Sergei Trofimovich <[e-mail]>
Signed-off-by: Martin K. Petersen <[e-mail]>

drivers/scsi/hpsa_cmd.h

···
 
 #include <linux/compiler.h>
 
+#include <linux/build_bug.h> /* static_assert */
+#include <linux/stddef.h> /* offsetof */
+
 /* general boundary defintions */
 #define SENSEINFOBYTES          32 /* may vary between hbas */
 #define SG_ENTRIES_IN_CMD	32 /* Max SG entries excluding chain blocks */
···
 	struct hpsa_scsi_dev_t *device;
 	atomic_t refcount; /* Must be last to avoid memset in hpsa_cmd_init() */
 } __aligned(COMMANDLIST_ALIGNMENT);
+
+/*
+ * Make sure our embedded atomic variable is aligned. Otherwise we break atomic
+ * operations on architectures that don't support unaligned atomics like IA64.
+ *
+ * The assert guards against reintroductin against unwanted __packed to
+ * the struct CommandList.
+ */
+static_assert(offsetof(struct CommandList, refcount) % __alignof__(atomic_t) == 0);
 
 /* Max S/G elements in I/O accelerator command */
 #define IOACCEL1_MAXSGENTRIES           24