> How about long instead of int. int leaves us with the possibility that > something else will expect 8 byte alignment. How about this? Align __cmd to ward off kernel unaligned access consoles messages on ia64 (and perhaps make an almost imperceptible performance improvement on other architectures that can handle unaligned access, but do so more slowly than aligned accesses). Signed-off-by: Tony Luck <tony.luck@xxxxxxxxx> --- diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h index e61f22b..a493a4b 100644 --- a/include/linux/blkdev.h +++ b/include/linux/blkdev.h @@ -211,7 +211,7 @@ struct request { * when request is used as a packet command carrier */ unsigned short cmd_len; - unsigned char __cmd[BLK_MAX_CDB]; + unsigned char __cmd[BLK_MAX_CDB] __attribute__((aligned(8))); unsigned char *cmd; unsigned int data_len; -- To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |