The definition of what's a data and what's a control CDB is not totally clear, but both of these set t_task_lba, so I'm inclined to call them data CDBs. Signed-off-by: Andy Grover <agrover@xxxxxxxxxx> --- drivers/target/target_core_transport.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/target/target_core_transport.c b/drivers/target/target_core_transport.c index 3dec696..fff099a 100644 --- a/drivers/target/target_core_transport.c +++ b/drivers/target/target_core_transport.c @@ -3078,7 +3078,7 @@ static int transport_generic_cmd_sequencer( size = dev->se_sub_dev->se_dev_attrib.block_size; cmd->t_task_lba = get_unaligned_be64(&cdb[12]); - cmd->se_cmd_flags |= SCF_SCSI_CONTROL_SG_IO_CDB; + cmd->se_cmd_flags |= SCF_SCSI_DATA_SG_IO_CDB; /* * Skip the remaining assignments for TCM/PSCSI passthrough @@ -3381,7 +3381,7 @@ static int transport_generic_cmd_sequencer( goto out_invalid_cdb_field; } } - cmd->se_cmd_flags |= SCF_SCSI_CONTROL_SG_IO_CDB; + cmd->se_cmd_flags |= SCF_SCSI_DATA_SG_IO_CDB; break; case ALLOW_MEDIUM_REMOVAL: case GPCMD_CLOSE_TRACK: -- 1.7.1 -- 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