diff options
author | Andy Grover <agrover@redhat.com> | 2013-11-26 14:55:22 -0500 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2013-12-16 16:24:38 -0500 |
commit | 2af7973a37059c92330014ee5f39adc94900e7e1 (patch) | |
tree | 549985d826744db03f797a05456afef685514bdb | |
parent | ab6dae8236767f9815bb00c29a56d045e33cd470 (diff) |
target: Refer to u32 luns as unpacked_lun
It's clearer to refer to pointers to the struct se_lun as "lun" and the
actual number itself as "unpacked_lun".
Signed-off-by: Andy Grover <agrover@redhat.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
-rw-r--r-- | drivers/target/target_core_device.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/target/target_core_device.c b/drivers/target/target_core_device.c index 207b340498a3..4b6b787d2c62 100644 --- a/drivers/target/target_core_device.c +++ b/drivers/target/target_core_device.c | |||
@@ -1112,23 +1112,23 @@ int se_dev_set_block_size(struct se_device *dev, u32 block_size) | |||
1112 | struct se_lun *core_dev_add_lun( | 1112 | struct se_lun *core_dev_add_lun( |
1113 | struct se_portal_group *tpg, | 1113 | struct se_portal_group *tpg, |
1114 | struct se_device *dev, | 1114 | struct se_device *dev, |
1115 | u32 lun) | 1115 | u32 unpacked_lun) |
1116 | { | 1116 | { |
1117 | struct se_lun *lun_p; | 1117 | struct se_lun *lun; |
1118 | int rc; | 1118 | int rc; |
1119 | 1119 | ||
1120 | lun_p = core_tpg_pre_addlun(tpg, lun); | 1120 | lun = core_tpg_pre_addlun(tpg, unpacked_lun); |
1121 | if (IS_ERR(lun_p)) | 1121 | if (IS_ERR(lun)) |
1122 | return lun_p; | 1122 | return lun; |
1123 | 1123 | ||
1124 | rc = core_tpg_post_addlun(tpg, lun_p, | 1124 | rc = core_tpg_post_addlun(tpg, lun, |
1125 | TRANSPORT_LUNFLAGS_READ_WRITE, dev); | 1125 | TRANSPORT_LUNFLAGS_READ_WRITE, dev); |
1126 | if (rc < 0) | 1126 | if (rc < 0) |
1127 | return ERR_PTR(rc); | 1127 | return ERR_PTR(rc); |
1128 | 1128 | ||
1129 | pr_debug("%s_TPG[%u]_LUN[%u] - Activated %s Logical Unit from" | 1129 | pr_debug("%s_TPG[%u]_LUN[%u] - Activated %s Logical Unit from" |
1130 | " CORE HBA: %u\n", tpg->se_tpg_tfo->get_fabric_name(), | 1130 | " CORE HBA: %u\n", tpg->se_tpg_tfo->get_fabric_name(), |
1131 | tpg->se_tpg_tfo->tpg_get_tag(tpg), lun_p->unpacked_lun, | 1131 | tpg->se_tpg_tfo->tpg_get_tag(tpg), lun->unpacked_lun, |
1132 | tpg->se_tpg_tfo->get_fabric_name(), dev->se_hba->hba_id); | 1132 | tpg->se_tpg_tfo->get_fabric_name(), dev->se_hba->hba_id); |
1133 | /* | 1133 | /* |
1134 | * Update LUN maps for dynamically added initiators when | 1134 | * Update LUN maps for dynamically added initiators when |
@@ -1149,7 +1149,7 @@ struct se_lun *core_dev_add_lun( | |||
1149 | spin_unlock_irq(&tpg->acl_node_lock); | 1149 | spin_unlock_irq(&tpg->acl_node_lock); |
1150 | } | 1150 | } |
1151 | 1151 | ||
1152 | return lun_p; | 1152 | return lun; |
1153 | } | 1153 | } |
1154 | 1154 | ||
1155 | /* core_dev_del_lun(): | 1155 | /* core_dev_del_lun(): |