Commit db007fc5 authored by Martin K. Petersen's avatar Martin K. Petersen Committed by James Bottomley

[SCSI] Command protection operation

Controllers that support DMA of protection information must be told
explicitly how to handle the I/O.  The controller has no knowledge of
the protection capabilities of the target device so this information
must be passed in the scsi_cmnd.

 - The protection operation tells the HBA whether to generate, strip or
   verify protection information.

 - The protection type tells the HBA which layout the target is
   formatted with.  This is necessary because the controller must be
   able to correctly interpret the included protection information in
   order to verify it.

 - When a scsi_cmnd is reused for error handling the protection
   operation must be cleared and saved while error handling is in

 - prot_op and prot_type are placed in an existing hole in scsi_cmnd
   and don't cause the structure to grow.
Signed-off-by: default avatarMartin K. Petersen <>
Signed-off-by: default avatarJames Bottomley <>
parent 4469f987
......@@ -664,7 +664,9 @@ void scsi_eh_prep_cmnd(struct scsi_cmnd *scmd, struct scsi_eh_save *ses,
ses->sdb = scmd->sdb;
ses->next_rq = scmd->request->next_rq;
ses->result = scmd->result;
ses->prot_op = scmd->prot_op;
scmd->prot_op = SCSI_PROT_NORMAL;
scmd->cmnd = ses->eh_cmnd;
memset(scmd->cmnd, 0, BLK_MAX_CDB);
memset(&scmd->sdb, 0, sizeof(scmd->sdb));
......@@ -722,6 +724,7 @@ void scsi_eh_restore_cmnd(struct scsi_cmnd* scmd, struct scsi_eh_save *ses)
scmd->sdb = ses->sdb;
scmd->request->next_rq = ses->next_rq;
scmd->result = ses->result;
scmd->prot_op = ses->prot_op;
......@@ -77,6 +77,9 @@ struct scsi_cmnd {
int allowed;
int timeout_per_command;
unsigned char prot_op;
unsigned char prot_type;
unsigned short cmd_len;
enum dma_data_direction sc_data_direction;
......@@ -208,4 +211,67 @@ static inline int scsi_sg_copy_to_buffer(struct scsi_cmnd *cmd,
buf, buflen);
* The operations below are hints that tell the controller driver how
* to handle I/Os with DIF or similar types of protection information.
enum scsi_prot_operations {
/* Normal I/O */
/* OS-HBA: Protected, HBA-Target: Unprotected */
/* OS-HBA: Unprotected, HBA-Target: Protected */
/* OS-HBA: Protected, HBA-Target: Protected */
/* OS-HBA: Protected, HBA-Target: Protected, checksum conversion */
static inline void scsi_set_prot_op(struct scsi_cmnd *scmd, unsigned char op)
scmd->prot_op = op;
static inline unsigned char scsi_get_prot_op(struct scsi_cmnd *scmd)
return scmd->prot_op;
* The controller usually does not know anything about the target it
* is communicating with. However, when DIX is enabled the controller
* must be know target type so it can verify the protection
* information passed along with the I/O.
enum scsi_prot_target_type {
static inline void scsi_set_prot_type(struct scsi_cmnd *scmd, unsigned char type)
scmd->prot_type = type;
static inline unsigned char scsi_get_prot_type(struct scsi_cmnd *scmd)
return scmd->prot_type;
static inline sector_t scsi_get_lba(struct scsi_cmnd *scmd)
return scmd->request->sector;
#endif /* _SCSI_SCSI_CMND_H */
......@@ -75,6 +75,7 @@ struct scsi_eh_save {
int result;
enum dma_data_direction data_direction;
unsigned char cmd_len;
unsigned char prot_op;
unsigned char *cmnd;
struct scsi_data_buffer sdb;
struct request *next_rq;
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment