> + if (target_init_cmd(&cmd->se_cmd, cmd->sess->se_sess, > + &cmd->ft_sense_buffer[0], scsilun_to_int(&fcp->fc_lun), > + ntohl(fcp->fc_dl), task_attr, data_dir, It would be nice to avoid the overly long line here. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>