>>>>> "Ben" == Ben Collins <ben.c@xxxxxxxxxxxx> writes: Ben, Ben> Considering your patch has had two success stories (I suspect yours Ben> was on SPARC, so that would even make two different platforms) and Ben> low likelihood of regression on little endian, I think it's pretty Ben> safe. I ordered a MegaRAID card today so I can test on x86 just to be sure. Ben> The only thing I think that could make it better is to annotate the Ben> structures in megaraid_sas.h as __le32 where applicable. I have an older patch that does just that. And another one that gets rid of the 32-bit DMA limitations on large systems. But I didn't want to clutter the initial submission. -- Martin K. Petersen Oracle Linux Engineering -- 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