From: Christoph Hellwig <hch@infradead.org>
To: nab@linux-iscsi.org
Cc: linux-scsi@vger.kernel.org
Subject: [PATCH 2/3] target: always assign t_task_lba in transport_generic_cmd_sequencer
Date: Mon, 29 Nov 2010 16:58:04 -0500 [thread overview]
Message-ID: <20101129215804.GA19726@infradead.org> (raw)
In-Reply-To: <20101129215734.GA19703@infradead.org>
We already sometimes directly calculate the lba in
transport_generic_cmd_sequencer, but sometimes use the transport_get_long_lba /
transport_get_lba callback in the se_cmd. Unify the code to always assign
it directly and remove the callbacks.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Index: lio-core/drivers/target/target_core_transport.c
===================================================================
--- lio-core.orig/drivers/target/target_core_transport.c 2010-11-29 19:42:05.046003633 +0100
+++ lio-core/drivers/target/target_core_transport.c 2010-11-29 19:47:31.670254576 +0100
@@ -1952,15 +1952,6 @@ int transport_generic_allocate_tasks(
DEBUG_CDB_H("Set cdb[0]: 0x%02x to "
"SCF_SCSI_DATA_SG_IO_CDB\n", cdb[0]);
cmd->se_cmd_flags |= SCF_SCSI_DATA_SG_IO_CDB;
-
- /*
- * Get the initial Logical Block Address from the Original
- * Command Descriptor Block that arrived on the iSCSI wire.
- */
- T_TASK(cmd)->t_task_lba = (cmd->transport_get_long_lba) ?
- cmd->transport_get_long_lba(cdb) :
- cmd->transport_get_lba(cdb);
-
break;
case TGCS_CONTROL_SG_IO_CDB:
DEBUG_CDB_H("Set cdb[0]: 0x%02x to"
@@ -3184,7 +3175,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_6;
- cmd->transport_get_lba = &transport_lba_21;
+ T_TASK(cmd)->t_task_lba = transport_lba_21(cdb);
ret = TGCS_DATA_SG_IO_CDB;
break;
case READ_10:
@@ -3193,7 +3184,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_10;
- cmd->transport_get_lba = &transport_lba_32;
+ T_TASK(cmd)->t_task_lba = transport_lba_32(cdb);
ret = TGCS_DATA_SG_IO_CDB;
break;
case READ_12:
@@ -3202,7 +3193,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_12;
- cmd->transport_get_lba = &transport_lba_32;
+ T_TASK(cmd)->t_task_lba = transport_lba_32(cdb);
ret = TGCS_DATA_SG_IO_CDB;
break;
case READ_16:
@@ -3211,7 +3202,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_16;
- cmd->transport_get_long_lba = &transport_lba_64;
+ T_TASK(cmd)->t_task_lba = transport_lba_64(cdb);
ret = TGCS_DATA_SG_IO_CDB;
break;
case WRITE_6:
@@ -3220,7 +3211,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_6;
- cmd->transport_get_lba = &transport_lba_21;
+ T_TASK(cmd)->t_task_lba = transport_lba_21(cdb);
ret = TGCS_DATA_SG_IO_CDB;
break;
case WRITE_10:
@@ -3229,7 +3220,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_10;
- cmd->transport_get_lba = &transport_lba_32;
+ T_TASK(cmd)->t_task_lba = transport_lba_32(cdb);
T_TASK(cmd)->t_tasks_fua = (cdb[1] & 0x8);
ret = TGCS_DATA_SG_IO_CDB;
break;
@@ -3239,7 +3230,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_12;
- cmd->transport_get_lba = &transport_lba_32;
+ T_TASK(cmd)->t_task_lba = transport_lba_32(cdb);
T_TASK(cmd)->t_tasks_fua = (cdb[1] & 0x8);
ret = TGCS_DATA_SG_IO_CDB;
break;
@@ -3249,7 +3240,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_16;
- cmd->transport_get_long_lba = &transport_lba_64;
+ T_TASK(cmd)->t_task_lba = transport_lba_64(cdb);
T_TASK(cmd)->t_tasks_fua = (cdb[1] & 0x8);
ret = TGCS_DATA_SG_IO_CDB;
break;
@@ -3262,7 +3253,7 @@ static int transport_generic_cmd_sequenc
return TGCS_UNSUPPORTED_CDB;
size = transport_get_size(sectors, cdb, cmd);
cmd->transport_split_cdb = &split_cdb_XX_10;
- cmd->transport_get_lba = &transport_lba_32;
+ T_TASK(cmd)->t_task_lba = transport_lba_32(cdb);
passthrough = (TRANSPORT(dev)->transport_type ==
TRANSPORT_PLUGIN_PHBA_PDEV);
/*
@@ -3297,7 +3288,7 @@ static int transport_generic_cmd_sequenc
* XDWRITE_READ_32 logic.
*/
cmd->transport_split_cdb = &split_cdb_XX_32;
- cmd->transport_get_long_lba = &transport_lba_64_ext;
+ T_TASK(cmd)->t_task_lba = transport_lba_64_ext(cdb);
/*
* Skip the remaining assignments for TCM/PSCSI passthrough
*/
Index: lio-core/include/target/target_core_base.h
===================================================================
--- lio-core.orig/include/target/target_core_base.h 2010-11-29 19:45:26.207004193 +0100
+++ lio-core/include/target/target_core_base.h 2010-11-29 19:45:30.786254087 +0100
@@ -531,8 +531,6 @@ struct se_cmd {
struct se_transport_task t_task_backstore;
struct target_core_fabric_ops *se_tfo;
int (*transport_emulate_cdb)(struct se_cmd *);
- u32 (*transport_get_lba)(unsigned char *);
- unsigned long long (*transport_get_long_lba)(unsigned char *);
void (*transport_split_cdb)(unsigned long long, u32 *, unsigned char *);
void (*transport_wait_for_tasks)(struct se_cmd *, int, int);
void (*transport_complete_callback)(struct se_cmd *);
next prev parent reply other threads:[~2010-11-29 21:58 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-11-29 21:57 [PATCH 1/3] target: remove unused get_dma_length subsystem method Christoph Hellwig
2010-11-29 21:58 ` Christoph Hellwig [this message]
2010-11-29 22:18 ` [PATCH 2/3] target: always assign t_task_lba in transport_generic_cmd_sequencer Nicholas A. Bellinger
2010-11-29 21:58 ` [PATCH 3/3] target: always assign se_cmd flags " Christoph Hellwig
2010-11-29 22:26 ` Nicholas A. Bellinger
2010-11-29 22:17 ` [PATCH 1/3] target: remove unused get_dma_length subsystem method Nicholas A. Bellinger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20101129215804.GA19726@infradead.org \
--to=hch@infradead.org \
--cc=linux-scsi@vger.kernel.org \
--cc=nab@linux-iscsi.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox