On Aug 19, 2013, at 7:16 AM, "Saxena, Sumit" <Sumit.Saxena@xxxxxxx> wrote: > This patch: Big endian support for megaraid_sas driver is not complete. In few areas, we need to do changes like this patch does not have changes for Thunderbolt/invader/Fury controllers(file: megaraid_sas_fusion.c), and needs few changes in file: megaraid_sas_fp.c(while host reads some member of raidmap, it needs endianness change there also).. > > LSI has megaraid_sas driver with big endian support for driver version "06.506.00.00". But to add big endian support for latest upstream megaraid_sas driver (06.600.18.00), we need to make some more changes. > We will get back with big endian support patch on top of latest upstream megaraid_sas driver. > Is this work-in-progress available somewhere? What is the time frame you expect to have this supported by LSI directly? Based on the fact that big-endian support in megaraid_sas is 8 years in the making, forgive me if I'm a little skeptical of progress in the near term. I realize that the big-endian patch from Martin isn't complete for fusion, but it is for other cards. Given that our company has a POWER based product using MegaRAID LSI SAS due on the market in a matter of months, I'd like to have stock kernels supporting it sooner rather than later. Since this patch has zero effect on currently supported platforms (little-endian), I don't see the harm in letting it through, since it actually does work, and more importantly, works right now. >> -----Original Message----- >> From: Martin K. Petersen [mailto:martin.petersen@xxxxxxxxxx] >> Sent: Friday, August 16, 2013 6:42 AM >> To: Ben Collins >> Cc: Martin K. Petersen; DL-MegaRAID Linux; linux-scsi; >> aradford@xxxxxxxxx >> Subject: Re: State of big-endian for MegaRAID SAS driver >> >>>>>>> "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 > > -- Benjamin M. Collins | Principal Architect Servergy, Inc. | Save Energy. Work Smart. ® m (757) 243-7557 | e ben.c@xxxxxxxxxxxx L LinkedIn | T twitter.com/Servergy www.servergy.com CONFIDENTIALITY NOTICE: This communication contains privileged and/or confidential information to be maintained with the strictest confidence. It is intended solely for the use of the person or entity in which it is addressed. If you are not the intended recipient, you are STRICTLY PROHIBITED from disclosing, copying, distributing or using any of this information. If you received this communication in error, please contact the sender immediately and destroy the material in its entirety, whether electronic or hard copy. Email may contain forward looking comments that are based on managements best-faith estimates which are subject to change. For information only. Not an offer, or solicitation of any offer to buy or sell any security, investment or other product. -- 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
