On Wed, Sep 19, 2012 at 12:59 AM, James Bottomley <James.Bottomley@xxxxxxxxxxxxxxxxxxxxx> wrote: > Is this also true on Big Endian Hardware? Because the fix you have > assumes that the TIO IOCB with SCSI status mode 1 should be CPU > endian ... that doesn't look right since this is passed directly over > the PCI bus (and the PCI bus is little endian), so shouldn't the correct > fix be to replace cpu_to_be32 with cpu_to_le32? After my patch the assignment is to a u8, and that byte is in the right place in memory. So I don't think there's any endianness bug. - R. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html