Re: [PATCH] sg descriptor sense cleanup lk 2.6.11-rc1-bk1

[Douglas Gilbert <dougg@torque.net>]

Christoph Hellwig wrote:
> On Mon, Jan 17, 2005 at 09:39:37PM +1000, Douglas Gilbert wrote:
> 
>>Continuation of descriptor sense data expansion to
>>sg driver.
>>
>>Next will be the sd driver which awaits a
>>new utility function in constants.c to read the
>>sense data information field for medium/hardware
>>errors.
>>
>>Changelog:
>>  - expand sense data handling to descriptor format
>>  - make module parameters visible in
>>      /sys/modules/sg/parameters
>>  - add "readable" SCSI commands: READ(16), REPORT LUNS,
>>    SERVICE_ACTION_IN, RECEIVE_DIAGNOSTIC, READ_LONG
>>    and MAINTENANCE_IN_CMD to list
>>
>>Signed-off-by: Douglas Gilbert <dougg@torque.net>
> 
> 
> +#ifndef MAINTENANCE_IN_CMD
> +#define MAINTENANCE_IN_CMD 0xa3
> +#endif
> 
> Shouldn't we put it into <scsi/scsi.h> ?

Yes it should. For a larger list of missing "byte 0"
opcodes see the top of constants.c

The opcodes with a "in" suffix retrieve data from the
device so are generally safe to use when the device
is opened O_RDONLY.

So the various SCSI command sets are not limited
in number to 256, more use is being made of
service actions. Maintanance In contains these
commands (or at least they are documented separately
in SPC-3):
   Report device identifier [a3/5]
   Report target port groups [a3/a]
   Report aliases [a3/b]
   Report supported operation codes [a3/c]
   Report supported task management functions [a3/d]
   Report priority [a3/e]

And sooner or later we will need to support variable
length commands.

Doug Gilbert