>>>>> "Ben" == Ben Collins <[email protected]> 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 [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to