Re: Inquiry data and emulated SG devices

From: Mathieu Fluhr
Date: Thu Oct 18 2007 - 06:00:19 EST


On Wed, 2007-10-17 at 21:31 -0400, Jeff Garzik wrote:
> Robert Hancock wrote:
> > This doesn't seem a very reliable way to identify an IDE device, as all
> > that 0 means is that the device does not claim conformance to any
> > standard. I would think it would be legitimate for an IDE device to put
> > a value like 5 in there as well, if it complies with SPC-4..
>
> Via the this-doesnt-really-matter-but-it-should-be-noted department:
>
> According to the latest on t10.org, MMC retroactively permitted SCSI
> version to be zero, for MMC-compliant USB and ATAPI devices.
>

Quoting to the latest MtFuji draft (Section 17.7.1):
"The ANSI Version field shall contain a non-zero value to comply with
this version of the Specification for a SCSI logical unit or zero for
an ATAPI logical unit."


> > In the case of libata though, that appears to be due to this code in
> > drivers/ata/libata-scsi.c:
> >
> > /* ATAPI devices typically report zero for their SCSI version,
> > * and sometimes deviate from the spec WRT response data
> > * format. If SCSI version is reported as zero like normal,
> > * then we make the following fixups: 1) Fake MMC-5 version,
> > * to indicate to the Linux scsi midlayer this is a modern
> > * device. 2) Ensure response data format / ATAPI information
> > * are always correct.
> > */
> > if (buf[2] == 0) {
> > buf[2] = 0x5;
> > buf[3] = 0x32;
> > }
> >

This explain a lot... But (Sorry I am not a scsi mid-layer expert) why
faking what the device outputs?

>From the application side, every device is then saw as real SCSI
devices. So from the developer point of view, this is really bad as
afterwards you have no way to optimize your code regarding the original
hardware architecture.


Mathieu

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/