Re: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Martin K. Petersen
> "Sumit" == Saxena, Sumit writes: Sumit> LSI has megaraid_sas driver with big endian support for driver Sumit> version "06.506.00.00". But to add big endian support for latest Sumit> upstream megaraid_sas driver (06.600.18.00), we need to make some Sumit> more changes. There are a few very

Re: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Martin K. Petersen
> "Sumit" == Saxena, Sumit writes: >> I realize that the big-endian patch from Martin isn't complete for >> fusion, but it is for other cards. Correct. I had no interest in Fury/Invader and no access to those boards. Sumit> Martin's patch does not have few changes needed in function Sumit

Re: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Ben Collins
> We have done a code review of what has been proposed and we believe it is > incomplete. We don't think that in its current state it will serve Linux as > well as we all want it to. I fully appreciate you're intent, but I can't imagine how you can think that it wont serve Linux in it's curre

RE: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Maloy, Joe
ay, August 26, 2013 2:42 PM To: Maloy, Joe Cc: Christoph Hellwig; Saxena, Sumit; Ben Collins; Martin K. Petersen; DL-MegaRAID Linux; linux-scsi; aradf...@gmail.com; Maly, JoLyn Subject: Re: State of big-endian for MegaRAID SAS driver On Mon, Aug 26, 2013 at 02:15:42PM -0600, Maloy, Joe wrote:

Re: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Christoph Hellwig
On Mon, Aug 26, 2013 at 04:26:30PM -0400, Ben Collins wrote: > Historically, waiting on commercial upstream to deliver what the community > has already done is wasted time. Likewise, historically, allowing patches > that partially implement things (in this respect, support one generation of > ca

Re: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Christoph Hellwig
On Mon, Aug 26, 2013 at 02:15:42PM -0600, Maloy, Joe wrote: > > Chris, > > I would appreciate you toning down your rhetoric. Please keep it > professional. If the b word is not in your business dictionary I have no idea how you get any work done, and I really don't care if you feel offended or

Re: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Ben Collins
On Aug 26, 2013, at 4:15 PM, "Maloy, Joe" wrote: > Chris, > > I would appreciate you toning down your rhetoric. Please keep it > professional. > > What Sumit reported is correct. We are polling our Marketing and Product > management organizations to see if we want to productize this type of

RE: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Maloy, Joe
, Sumit Cc: Ben Collins; Martin K. Petersen; DL-MegaRAID Linux; linux-scsi; aradf...@gmail.com; Maloy, Joe Subject: Re: State of big-endian for MegaRAID SAS driver On Mon, Aug 26, 2013 at 12:51:07PM +0530, Saxena, Sumit wrote: > We are conferring with Marketing and Product Management on direct

Re: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Christoph Hellwig
On Mon, Aug 26, 2013 at 12:51:07PM +0530, Saxena, Sumit wrote: > We are conferring with Marketing and Product Management on direction wrt > Big endian support for megaraid_sas driver. Our goal is to gain clear > direction within two weeks. Sorry guys, but this is just utter bullshit. There a

RE: State of big-endian for MegaRAID SAS driver

2013-08-26 Thread Saxena, Sumit
>-Original Message- >From: Ben Collins [mailto:be...@servergy.com] >Sent: Monday, August 19, 2013 10:55 PM >To: Saxena, Sumit >Cc: Martin K. Petersen; DL-MegaRAID Linux; linux-scsi; >aradf...@gmail.com >Subject: Re: State of big-endian for MegaRAID SAS driver > &

Re: State of big-endian for MegaRAID SAS driver

2013-08-19 Thread Ben Collins
through, since it actually does work, and more importantly, works right now. >> -Original Message- >> From: Martin K. Petersen [mailto:martin.peter...@oracle.com] >> Sent: Friday, August 16, 2013 6:42 AM >> To: Ben Collins >> Cc: Martin K. Petersen; DL-Meg

RE: State of big-endian for MegaRAID SAS driver

2013-08-19 Thread Saxena, Sumit
;aradf...@gmail.com >Subject: Re: State of big-endian for MegaRAID SAS driver > >>>>>> "Ben" == Ben Collins writes: > >Ben, > >Ben> Considering your patch has had two success stories (I suspect yours >Ben> was on SPARC, so that would even make

Re: State of big-endian for MegaRAID SAS driver

2013-08-15 Thread Martin K. Petersen
> "Ben" == Ben Collins 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 t

Re: State of big-endian for MegaRAID SAS driver

2013-08-14 Thread Ben Collins
On Aug 14, 2013, at 3:44 PM, "Martin K. Petersen" wrote: >> "Ben" == Ben Collins writes: > > Ben> You're the man. Forward ported this to 3.11-rc code base. Worked > Ben> like a charm. > > Heh, you just beat me to it :) > > Formatting differences aside there were a few glitches, however.

Re: State of big-endian for MegaRAID SAS driver

2013-08-14 Thread Martin K. Petersen
> "Ben" == Ben Collins writes: Ben> You're the man. Forward ported this to 3.11-rc code base. Worked Ben> like a charm. Heh, you just beat me to it :) Formatting differences aside there were a few glitches, however. I outlined them below... Btw. the reason I did cpu_to_le16(0) etc. was mos

Re: State of big-endian for MegaRAID SAS driver

2013-08-14 Thread Martin K. Petersen
> "Martin" == Martin K Petersen writes: And against upstream... -- Martin K. Petersen Oracle Linux Engineering commit 6d6db0df277b46ea538be5c70e89347436a8a114 Author: Martin K. Petersen Date: Sat Aug 10 17:55:58 2013 -0400 megaraid_sas: Big endian support Thi

Re: State of big-endian for MegaRAID SAS driver

2013-08-14 Thread Martin K. Petersen
> "Ben" == Ben Collins writes: Ben> Based on some searching, back when the megaraid_sas driver was Ben> first introduced in 2005, issues were raised with the lack of Ben> support for big-endian architectures[1]. In this conversation, it Ben> was said that LSI would revisit this later, and thu