linux-kernel - RE: [PATCH 1/5] scsi: megaraid_sas - Add Online Controller Reset to MegaRAID SAS driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4B6A08C587958942AA3002690DD4F8C3DC0D4B41@cosmail02.lsi.com>
Date:	Thu, 9 Sep 2010 11:28:45 -0600
From:	"Yang, Bo" <Bo.Yang@....com>
To:	"Yang, Bo" <Bo.Yang@....com>,
	James Bottomley <James.Bottomley@...e.de>,
	bo yang <boyang1288@...il.com>
CC:	"linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
	"akpm@...l.org" <akpm@...l.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Daftardar, Jayant" <Jayant.Daftardar@....com>
Subject: RE: [PATCH 1/5] scsi: megaraid_sas - Add Online Controller Reset to
 MegaRAID SAS driver

James,

Do you have any feedback for the patches?

Thanks,

Bo Yang

-----Original Message-----
From: Yang, Bo 
Sent: Tuesday, September 07, 2010 1:12 PM
To: Yang, Bo; 'James Bottomley'; 'bo yang'
Cc: 'linux-scsi@...r.kernel.org'; 'akpm@...l.org'; 'linux-kernel@...r.kernel.org'; Daftardar, Jayant
Subject: RE: [PATCH 1/5] scsi: megaraid_sas - Add Online Controller Reset to MegaRAID SAS driver

James,

Those were corrected and submitted.  

Thanks,

Bo Yang

-----Original Message-----
From: Yang, Bo 
Sent: Tuesday, September 07, 2010 11:54 AM
To: 'James Bottomley'; bo yang
Cc: linux-scsi@...r.kernel.org; akpm@...l.org; linux-kernel@...r.kernel.org; Daftardar, Jayant
Subject: RE: [PATCH 1/5] scsi: megaraid_sas - Add Online Controller Reset to MegaRAID SAS driver

Thanks James,

The e-mail broken lines of the description.  I am changing the description to avoid re-submit all the patches(5 of them).

Also 

>  megasas_complete_cmd(struct megasas_instance *instance, struct
> megasas_cmd *cmd,
>  		     u8 alt_status);

Not patch line(already in the src, but more than 80 chars).

Regards,

Bo Yang    

-----Original Message-----
From: James Bottomley [mailto:James.Bottomley@...e.de] 
Sent: Sunday, September 05, 2010 2:04 PM
To: bo yang
Cc: Yang, Bo; linux-scsi@...r.kernel.org; akpm@...l.org; linux-kernel@...r.kernel.org; Daftardar, Jayant
Subject: Re: [PATCH 1/5] scsi: megaraid_sas - Add Online Controller Reset to MegaRAID SAS driver

On Wed, 2010-08-11 at 16:45 -0400, bo yang wrote:
> To add the Online Controller Reset  (OCR) support, driver need to do:
> 		a). reset the controller chips -- Xscale and Gen2 which will change
> the function calls and add the reset function
> 		related to this two chips.
> 		
> 		b). during the reset, driver will store the pending cmds which not
> returned by FW to driver's pending queue.  Driver
> 		will re-issue those pending cmds again to FW after the OCR finished.
> 		
> 		c). In driver's timeout routine, driver will report to OS as reset.
> Also driver's queue routine will block the
> 		cmds until the OCR finished.
> 		
> 		d). in Driver's ISR routine, if driver get the FW state as state
> change, FW in Failure status and FW support online controller
> 		reset (OCR), driver will start to do the controller reset.
> 		
> 		e). In driver's IOCTL routine, the application cmds will wait for
> the OCR to finish, then issue the cmds to FW.
> 
> Signed-off-by Bo Yang<bo.yang@....com>

This patch is full of line breaks which prevent application, like this
one:

> diff -rupN old/Documentation/scsi/ChangeLog.megaraid_sas
> new/Documentation/scsi/ChangeLog.megaraid_sas
> --- old/Documentation/scsi/ChangeLog.megaraid_sas	2010-08-10
> 04:58:36.000000000 -0400
> +++ new/Documentation/scsi/ChangeLog.megaraid_sas	2010-08-10
> 04:56:30.000000000 -0400
> @@ -1,3 +1,36 @@
> +1 Release Date    : Thur.  May 03, 2010 09:12:45 PST 2009 -
> +			(emaild-id:megaraidlinux@....com)
> +			Bo Yang
[snip]

and this one
> diff -rupN old/drivers/scsi/megaraid/megaraid_sas.c
> new/drivers/scsi/megaraid/megaraid_sas.c
> --- old/drivers/scsi/megaraid/megaraid_sas.c	2010-08-10 04:55:41.000000000 -0400
> +++ new/drivers/scsi/megaraid/megaraid_sas.c	2010-08-10 04:56:30.000000000 -0400
> @@ -107,6 +107,12 @@ static void
>  megasas_complete_cmd(struct megasas_instance *instance, struct
> megasas_cmd *cmd,
>  		     u8 alt_status);

Since not every long line is broken, this looks like some type of cut
and paste error ... anyway, could you fix it up, please?

Thanks,

James