On 11/17/2010 11:38 PM, Christoph Hellwig wrote:
> The cmd to task mapping is almost the same for all control CDBs,
> except for calling different backend methods to do the backed-specific
> mapping. Merge the code performing the mapping into a single
> new transport_map_control_cmd_to_task helper, and kill the
> transport_map_task callback by simplify calling the backend methods
> directly for the single task for a control CDB, and hardcoding
> ->map_task_SG for data CDBs.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
>
<>
> +static int
> +transport_map_control_cmd_to_task(struct se_cmd *cmd,
> + struct se_transform_info *ti)
> +{
> + struct se_device *dev = SE_DEV(cmd);
> + unsigned char *cdb;
> + struct se_task *task;
> + int ret;
> +
> + task = transport_generic_get_task(ti, cmd, ti->se_obj_ptr,
> + cmd->data_direction);
> + if (!task)
> + return PYX_TRANSPORT_OUT_OF_MEMORY_RESOURCES;
> +
> + cdb = TRANSPORT(dev)->get_cdb(task);
> + if (cdb)
> + memcpy(cdb, cmd->t_task->t_task_cdb,
> + scsi_command_size(cmd->t_task->t_task_cdb));
> +
> + task->task_size = cmd->data_length;
> + task->task_sg_num =
> + (cmd->se_cmd_flags & SCF_SCSI_CONTROL_SG_IO_CDB) ? 1 : 0;
> +
> + atomic_inc(&cmd->t_task->t_fe_count);
> + atomic_inc(&cmd->t_task->t_se_count);
> +
> + if (cmd->se_cmd_flags & SCF_SCSI_CONTROL_SG_IO_CDB) {
> + struct se_mem *se_mem, *se_mem_lout = NULL;
> + u32 se_mem_cnt = 0, task_offset = 0;
> +
> + BUG_ON(list_empty(cmd->t_task->t_mem_list));
> +
> + ret = transport_do_se_mem_map(dev, task,
> + cmd->t_task->t_mem_list, NULL, se_mem,
> + &se_mem_lout, &se_mem_cnt, &task_offset);
> + if (ret < 0)
> + return PYX_TRANSPORT_OUT_OF_MEMORY_RESOURCES;
> +
> + if (dev->transport->map_task_SG)
> + return dev->transport->map_task_SG(task);
> + return 0;
> + } else if (cmd->se_cmd_flags & SCF_SCSI_CONTROL_NONSG_IO_CDB) {
> + if (dev->transport->map_task_non_SG(task))
> + return dev->transport->map_task_non_SG(task);
> + return 0;
> + } else if (cmd->se_cmd_flags & SCF_SCSI_NON_DATA_CDB) {
> + if (dev->transport->cdb_none(task))
> + return dev->transport->cdb_none(task);
> + return 0;
If all these are mutual exclusive (if..elseif..else) can we not make
them an enum and switch() for costless-ness and clarity?
Boaz
> + } else {
> + BUG();
> + return PYX_TRANSPORT_OUT_OF_MEMORY_RESOURCES;
> + }
> +}
> +
<>
--
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
