target: Rename core_tpg_{pre,post}_addlun for clarity
authorAndy Grover <agrover@redhat.com>
Tue, 26 Nov 2013 20:07:54 +0000 (12:07 -0800)
committerNicholas Bellinger <nab@linux-iscsi.org>
Mon, 16 Dec 2013 21:28:19 +0000 (13:28 -0800)
"pre" is really an allocation function. The only time it isn't called is
for virtual_lun0, which is statically allocated. Renaming that to "alloc"
lets the other function not need to be "post", and just be called
core_tpg_add_lun.

(nab: fix minor applying fuzz in core_tpg_setup_virtual_lun0)

Signed-off-by: Andy Grover <agrover@redhat.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
drivers/target/target_core_device.c
drivers/target/target_core_internal.h
drivers/target/target_core_tpg.c

index 4b6b787d2c62debd4f7987be8c795e9b4ffbb011..dbd78a176ddb173a361aabe22a49dd04e7a97de1 100644 (file)
@@ -1117,11 +1117,11 @@ struct se_lun *core_dev_add_lun(
        struct se_lun *lun;
        int rc;
 
-       lun = core_tpg_pre_addlun(tpg, unpacked_lun);
+       lun = core_tpg_alloc_lun(tpg, unpacked_lun);
        if (IS_ERR(lun))
                return lun;
 
-       rc = core_tpg_post_addlun(tpg, lun,
+       rc = core_tpg_add_lun(tpg, lun,
                                TRANSPORT_LUNFLAGS_READ_WRITE, dev);
        if (rc < 0)
                return ERR_PTR(rc);
index 47b63b094cdcacd0ace2e6fe2dee7c18d4091b49..f67e764a42bd3c9d6a3bdf95e6cbd99d99eb98e9 100644 (file)
@@ -77,8 +77,8 @@ struct se_node_acl *__core_tpg_get_initiator_node_acl(struct se_portal_group *tp
                const char *);
 void   core_tpg_add_node_to_devs(struct se_node_acl *, struct se_portal_group *);
 void   core_tpg_wait_for_nacl_pr_ref(struct se_node_acl *);
-struct se_lun *core_tpg_pre_addlun(struct se_portal_group *, u32);
-int    core_tpg_post_addlun(struct se_portal_group *, struct se_lun *,
+struct se_lun *core_tpg_alloc_lun(struct se_portal_group *, u32);
+int    core_tpg_add_lun(struct se_portal_group *, struct se_lun *,
                u32, void *);
 struct se_lun *core_tpg_pre_dellun(struct se_portal_group *, u32 unpacked_lun);
 int    core_tpg_post_dellun(struct se_portal_group *, struct se_lun *);
index f697f8baec5418d13484904bf3730880f90639fd..a1dbc94881836570679f4fc86636cef7fef17fac 100644 (file)
@@ -662,7 +662,7 @@ static int core_tpg_setup_virtual_lun0(struct se_portal_group *se_tpg)
        if (ret < 0)
                return ret;
 
-       ret = core_tpg_post_addlun(se_tpg, lun, lun_access, dev);
+       ret = core_tpg_add_lun(se_tpg, lun, lun_access, dev);
        if (ret < 0) {
                percpu_ref_cancel_init(&lun->lun_ref);
                return ret;
@@ -789,7 +789,7 @@ int core_tpg_deregister(struct se_portal_group *se_tpg)
 }
 EXPORT_SYMBOL(core_tpg_deregister);
 
-struct se_lun *core_tpg_pre_addlun(
+struct se_lun *core_tpg_alloc_lun(
        struct se_portal_group *tpg,
        u32 unpacked_lun)
 {
@@ -819,7 +819,7 @@ struct se_lun *core_tpg_pre_addlun(
        return lun;
 }
 
-int core_tpg_post_addlun(
+int core_tpg_add_lun(
        struct se_portal_group *tpg,
        struct se_lun *lun,
        u32 lun_access,