diff options
| author | Nicholas Bellinger | 2015-05-21 06:48:03 +0200 |
|---|---|---|
| committer | Nicholas Bellinger | 2015-06-16 08:23:22 +0200 |
| commit | bc0c94b1404b225b19b6b53a0e508f43e269ed1e (patch) | |
| tree | 8bd5e91d21589d1779d8d9fba62a71d609d93e31 /drivers/target/iscsi/iscsi_target_tpg.c | |
| parent | target: Drop se_lun->lun_active for existing percpu lun_ref (diff) | |
| download | kernel-qcow2-linux-bc0c94b1404b225b19b6b53a0e508f43e269ed1e.tar.gz kernel-qcow2-linux-bc0c94b1404b225b19b6b53a0e508f43e269ed1e.tar.xz kernel-qcow2-linux-bc0c94b1404b225b19b6b53a0e508f43e269ed1e.zip | |
target: Drop unnecessary core_tpg_register TFO parameter
This patch drops unnecessary target_core_fabric_ops parameter usage
for core_tpg_register() during fabric driver TFO->fabric_make_tpg()
se_portal_group creation callback execution.
Instead, use the existing se_wwn->wwn_tf->tf_ops pointer to ensure
fabric driver is really using the same TFO provided at module_init
time.
Also go ahead and drop the forward TFO declarations tree-wide, and
handling the special case for iscsi-target discovery TPG.
Cc: Christoph Hellwig <hch@lst.de>
Reviewed-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target/iscsi/iscsi_target_tpg.c')
| -rw-r--r-- | drivers/target/iscsi/iscsi_target_tpg.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/drivers/target/iscsi/iscsi_target_tpg.c b/drivers/target/iscsi/iscsi_target_tpg.c index de8829102ab4..968068ffcb1c 100644 --- a/drivers/target/iscsi/iscsi_target_tpg.c +++ b/drivers/target/iscsi/iscsi_target_tpg.c @@ -66,8 +66,12 @@ int iscsit_load_discovery_tpg(void) pr_err("Unable to allocate struct iscsi_portal_group\n"); return -1; } - - ret = core_tpg_register(&iscsi_ops, NULL, &tpg->tpg_se_tpg, -1); + /* + * Save iscsi_ops pointer for special case discovery TPG that + * doesn't exist as se_wwn->wwn_group within configfs. + */ + tpg->tpg_se_tpg.se_tpg_tfo = &iscsi_ops; + ret = core_tpg_register(NULL, &tpg->tpg_se_tpg, -1); if (ret < 0) { kfree(tpg); return -1; |