RE: [PATCH][RESUBMIT] scsi_dh_rdac: Fix for returning correct mode select cmd return info

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



James,
   Can you please pick this up for next scsi update.. I have made sure that checkpatch.pl runs fine on this . ):
Thanks
Babu Moger 

> -----Original Message-----
> From: Chandra Seetharaman [mailto:sekharan@xxxxxxxxxx]
> Sent: Friday, August 28, 2009 12:51 PM
> To: Moger, Babu
> Cc: 'linux-scsi@xxxxxxxxxxxxxxx'; 'dm-devel@xxxxxxxxxx'; Chauhan,
> Vijay; Stankey, Robert; Dachepalli, Sudhir
> Subject: Re: [PATCH][RESUBMIT] scsi_dh_rdac: Fix for returning correct
> mode select cmd return info
> 
> 
> On Fri, 2009-08-28 at 08:18 -0600, Moger, Babu wrote:
> > The function mode_select_handle_sense returns SCSI_DH_OK even when
> there is a sense code which is incorrect. Removing it so that it
> returns SCSI_DH_IO when there is sense that is not handled by this
> function.
> >
> > Signed-off-by: Babu Moger <babu.moger@xxxxxxx>
> > Reviewed-by: Vijay Chauhan <vijay.chauhan@xxxxxxx>
> > Reviewed-by: Bob Stankey <Robert.stankey@xxxxxxx>
> 
> Reviewed-by: Chandra Seetharaman <sekharan@xxxxxxxxxx>
> 
> >
> > ---
> > --- linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c.orig
> 	2009-08-22 06:20:45.000000000 -0500
> > +++ linux-2.6.31-rc5/drivers/scsi/device_handler/scsi_dh_rdac.c	2009-
> 08-22 06:18:36.000000000 -0500
> > @@ -455,8 +455,6 @@ static int mode_select_handle_sense(stru
> >  	if (!ret)
> >  		goto done;
> >
> > -	err = SCSI_DH_OK;
> > -
> >  	switch (sense_hdr.sense_key) {
> >  	case NO_SENSE:
> >  	case ABORTED_COMMAND:
> >
> >
> > --
> > 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

--
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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux