All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] mptsas: Added code for checking the LU type is SSC for SILI bit in READ_6  and READ_16 CDB
@ 2012-03-02  9:56 nagalakshmi.nandigama
  2012-03-02 15:38 ` Greg KH
  2012-03-02 16:03 ` James Bottomley
  0 siblings, 2 replies; 3+ messages in thread
From: nagalakshmi.nandigama @ 2012-03-02  9:56 UTC (permalink / raw)
  To: stable, linux-scsi; +Cc: jejb

Added code for checking the LU type is SSC for SILI bit in READ_6  and READ_16 CDB 

Signed-off-by: Nagalakshmi Nandigama <nagalakshmi.nandigama@lsi.com>
---

diff --git a/drivers/message/fusion/mptscsih.c b/drivers/message/fusion/mptscsih.c
index 0c3ced7..266bb14 100644
--- a/drivers/message/fusion/mptscsih.c
+++ b/drivers/message/fusion/mptscsih.c
@@ -827,13 +827,16 @@ mptscsih_io_done(MPT_ADAPTER *ioc, MPT_FRAME_HDR *mf, MPT_FRAME_HDR *mr)
 				 * DID_SOFT_ERROR is set.
 				 */
 				if (ioc->bus_type == SPI) {
-					if ((pScsiReq->CDB[0] == READ_6  && ((pScsiReq->CDB[1] & 0x02) == 0)) ||
-					    pScsiReq->CDB[0] == READ_10 ||
-					    pScsiReq->CDB[0] == READ_12 ||
-						(pScsiReq->CDB[0] == READ_16 &&
-						((pScsiReq->CDB[1] & 0x02) == 0)) ||
-					    pScsiReq->CDB[0] == VERIFY  ||
-					    pScsiReq->CDB[0] == VERIFY_16) {
+					if ((pScsiReq->CDB[0] == READ_6  &&
+					((pScsiReq->CDB[1] & 0x02) == 0) &&
+					(sc->device->type == TYPE_TAPE)) ||
+					pScsiReq->CDB[0] == READ_10 ||
+					pScsiReq->CDB[0] == READ_12 ||
+					(pScsiReq->CDB[0] == READ_16 &&
+					((pScsiReq->CDB[1] & 0x02) == 0) &&
+					(sc->device->type == TYPE_TAPE)) ||
+					 pScsiReq->CDB[0] == VERIFY  ||
+					 pScsiReq->CDB[0] == VERIFY_16) {
 						if (scsi_bufflen(sc) !=
 							xfer_cnt) {
 							sc->result =

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] mptsas: Added code for checking the LU type is SSC for SILI bit in READ_6  and READ_16 CDB
  2012-03-02  9:56 [PATCH] mptsas: Added code for checking the LU type is SSC for SILI bit in READ_6 and READ_16 CDB nagalakshmi.nandigama
@ 2012-03-02 15:38 ` Greg KH
  2012-03-02 16:03 ` James Bottomley
  1 sibling, 0 replies; 3+ messages in thread
From: Greg KH @ 2012-03-02 15:38 UTC (permalink / raw)
  To: nagalakshmi.nandigama; +Cc: stable, linux-scsi, jejb

On Fri, Mar 02, 2012 at 03:26:55PM +0530, nagalakshmi.nandigama@lsi.com wrote:
> Added code for checking the LU type is SSC for SILI bit in READ_6  and READ_16 CDB 
> 
> Signed-off-by: Nagalakshmi Nandigama <nagalakshmi.nandigama@lsi.com>
> ---

<formletter>

This is not the correct way to submit patches for inclusion in the
stable kernel tree.  Please read Documentation/stable_kernel_rules.txt
for how to do this properly.

</formletter>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] mptsas: Added code for checking the LU type is SSC for SILI bit in READ_6  and READ_16 CDB
  2012-03-02  9:56 [PATCH] mptsas: Added code for checking the LU type is SSC for SILI bit in READ_6 and READ_16 CDB nagalakshmi.nandigama
  2012-03-02 15:38 ` Greg KH
@ 2012-03-02 16:03 ` James Bottomley
  1 sibling, 0 replies; 3+ messages in thread
From: James Bottomley @ 2012-03-02 16:03 UTC (permalink / raw)
  To: nagalakshmi.nandigama; +Cc: stable, linux-scsi, jejb

On Fri, 2012-03-02 at 15:26 +0530, nagalakshmi.nandigama@lsi.com wrote:
> Added code for checking the LU type is SSC for SILI bit in READ_6  and
> READ_16 CDB 
> 
> Signed-off-by: Nagalakshmi Nandigama <nagalakshmi.nandigama@lsi.com>

The description doesn't give me the impression that this is a bug
fix ... is this a bug fix and if so, what bug is being fixed (describe
the user visible issue)?

James



^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-03-02 16:03 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-03-02  9:56 [PATCH] mptsas: Added code for checking the LU type is SSC for SILI bit in READ_6 and READ_16 CDB nagalakshmi.nandigama
2012-03-02 15:38 ` Greg KH
2012-03-02 16:03 ` James Bottomley

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.