Re: [PATCH 0/3] SCSI & usb-storage: new flags for VPD pages and REPORT LUNS

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

 



On Tue, Jan 10, 2012 at 01:42:40PM -0500, Alan Stern wrote:
> James and Greg:
> 
> Perry Wagle reports that his new USB-ESATA drives don't work properly 
> because they don't like getting the LUN bits in the second byte of the 
> command string.  We set those bits only because usb-storage adjusts the 
> SCSI revision level for all devices to SCSI-2; the drives themselves 
> report a value of 0x06 (which would be SCSI-7).
> 
> The reason for mangling the scsi_level value was that some USB devices 
> reported SCSI-3 but couldn't handle some of the mandatory features: VPD 
> pages and the REPORT LUNS command.  It's now clear that instead of 
> abusing scsi_level, we should have separate flags for the individual 
> features we need to control.
> 
> That's what this patch series does.  The first merely corrects an 
> existing typo in the definition of struct scsi_target.  The second adds 
> the new flags and makes usb-storage use them.  The third takes this 
> opportunity to do a little code clean-up in usb-storage.
> 
> The changes are more or less equally divided between SCSI and
> usb-storage.  James, if they look okay to you and you don't mind taking
> them through your tree, that would be fine.  Or if you prefer, I'm sure
> Greg won't mind taking them.

I'll be glad to take these, James, any objection?

thanks,

greg k-h
--
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