These changes are the raw update to linux-4.4.6-rt14. Kernel sources
[kvmfornfv.git] / kernel / drivers / target / target_core_fabric_configfs.c
index 1f7886b..f916d18 100644 (file)
@@ -35,9 +35,6 @@
 
 #include <target/target_core_base.h>
 #include <target/target_core_fabric.h>
-#include <target/target_core_fabric_configfs.h>
-#include <target/target_core_configfs.h>
-#include <target/configfs_macros.h>
 
 #include "target_core_internal.h"
 #include "target_core_alua.h"
 #define TF_CIT_SETUP(_name, _item_ops, _group_ops, _attrs)             \
 static void target_fabric_setup_##_name##_cit(struct target_fabric_configfs *tf) \
 {                                                                      \
-       struct target_fabric_configfs_template *tfc = &tf->tf_cit_tmpl; \
-       struct config_item_type *cit = &tfc->tfc_##_name##_cit;         \
+       struct config_item_type *cit = &tf->tf_##_name##_cit;           \
                                                                        \
        cit->ct_item_ops = _item_ops;                                   \
        cit->ct_group_ops = _group_ops;                                 \
        cit->ct_attrs = _attrs;                                         \
-       cit->ct_owner = tf->tf_module;                                  \
+       cit->ct_owner = tf->tf_ops->module;                             \
        pr_debug("Setup generic %s\n", __stringify(_name));             \
 }
 
 #define TF_CIT_SETUP_DRV(_name, _item_ops, _group_ops)         \
 static void target_fabric_setup_##_name##_cit(struct target_fabric_configfs *tf) \
 {                                                                      \
-       struct target_fabric_configfs_template *tfc = &tf->tf_cit_tmpl; \
-       struct config_item_type *cit = &tfc->tfc_##_name##_cit;         \
-       struct configfs_attribute **attrs = tf->tf_ops.tfc_##_name##_attrs; \
+       struct config_item_type *cit = &tf->tf_##_name##_cit;           \
+       struct configfs_attribute **attrs = tf->tf_ops->tfc_##_name##_attrs; \
                                                                        \
        cit->ct_item_ops = _item_ops;                                   \
        cit->ct_group_ops = _group_ops;                                 \
        cit->ct_attrs = attrs;                                          \
-       cit->ct_owner = tf->tf_module;                                  \
+       cit->ct_owner = tf->tf_ops->module;                             \
        pr_debug("Setup generic %s\n", __stringify(_name));             \
 }
 
@@ -83,7 +78,7 @@ static int target_fabric_mappedlun_link(
                        struct se_lun_acl, se_lun_group);
        struct se_portal_group *se_tpg;
        struct config_item *nacl_ci, *tpg_ci, *tpg_ci_s, *wwn_ci, *wwn_ci_s;
-       int ret = 0, lun_access;
+       int lun_access;
 
        if (lun->lun_link_magic != SE_LUN_LINK_MAGIC) {
                pr_err("Bad lun->lun_link_magic, not a valid lun_ci pointer:"
@@ -93,12 +88,11 @@ static int target_fabric_mappedlun_link(
        /*
         * Ensure that the source port exists
         */
-       if (!lun->lun_sep || !lun->lun_sep->sep_tpg) {
-               pr_err("Source se_lun->lun_sep or lun->lun_sep->sep"
-                               "_tpg does not exist\n");
+       if (!lun->lun_se_dev) {
+               pr_err("Source se_lun->lun_se_dev does not exist\n");
                return -EINVAL;
        }
-       se_tpg = lun->lun_sep->sep_tpg;
+       se_tpg = lun->lun_tpg;
 
        nacl_ci = &lun_acl_ci->ci_parent->ci_group->cg_item;
        tpg_ci = &nacl_ci->ci_group->cg_item;
@@ -125,81 +119,66 @@ static int target_fabric_mappedlun_link(
         * which be will write protected (READ-ONLY) when
         * tpg_1/attrib/demo_mode_write_protect=1
         */
-       spin_lock_irq(&lacl->se_lun_nacl->device_list_lock);
-       deve = lacl->se_lun_nacl->device_list[lacl->mapped_lun];
-       if (deve->lun_flags & TRANSPORT_LUNFLAGS_INITIATOR_ACCESS)
+       rcu_read_lock();
+       deve = target_nacl_find_deve(lacl->se_lun_nacl, lacl->mapped_lun);
+       if (deve)
                lun_access = deve->lun_flags;
        else
                lun_access =
                        (se_tpg->se_tpg_tfo->tpg_check_prod_mode_write_protect(
                                se_tpg)) ? TRANSPORT_LUNFLAGS_READ_ONLY :
                                           TRANSPORT_LUNFLAGS_READ_WRITE;
-       spin_unlock_irq(&lacl->se_lun_nacl->device_list_lock);
+       rcu_read_unlock();
        /*
         * Determine the actual mapped LUN value user wants..
         *
         * This value is what the SCSI Initiator actually sees the
-        * iscsi/$IQN/$TPGT/lun/lun_* as on their SCSI Initiator Ports.
+        * $FABRIC/$WWPN/$TPGT/lun/lun_* as on their SCSI Initiator Ports.
         */
-       ret = core_dev_add_initiator_node_lun_acl(se_tpg, lacl,
-                       lun->unpacked_lun, lun_access);
-
-       return (ret < 0) ? -EINVAL : 0;
+       return core_dev_add_initiator_node_lun_acl(se_tpg, lacl, lun, lun_access);
 }
 
 static int target_fabric_mappedlun_unlink(
        struct config_item *lun_acl_ci,
        struct config_item *lun_ci)
 {
-       struct se_lun *lun;
        struct se_lun_acl *lacl = container_of(to_config_group(lun_acl_ci),
                        struct se_lun_acl, se_lun_group);
-       struct se_node_acl *nacl = lacl->se_lun_nacl;
-       struct se_dev_entry *deve = nacl->device_list[lacl->mapped_lun];
-       struct se_portal_group *se_tpg;
-       /*
-        * Determine if the underlying MappedLUN has already been released..
-        */
-       if (!deve->se_lun)
-               return 0;
-
-       lun = container_of(to_config_group(lun_ci), struct se_lun, lun_group);
-       se_tpg = lun->lun_sep->sep_tpg;
+       struct se_lun *lun = container_of(to_config_group(lun_ci),
+                       struct se_lun, lun_group);
 
-       core_dev_del_initiator_node_lun_acl(se_tpg, lun, lacl);
-       return 0;
+       return core_dev_del_initiator_node_lun_acl(lun, lacl);
 }
 
-CONFIGFS_EATTR_STRUCT(target_fabric_mappedlun, se_lun_acl);
-#define TCM_MAPPEDLUN_ATTR(_name, _mode)                               \
-static struct target_fabric_mappedlun_attribute target_fabric_mappedlun_##_name = \
-       __CONFIGFS_EATTR(_name, _mode,                                  \
-       target_fabric_mappedlun_show_##_name,                           \
-       target_fabric_mappedlun_store_##_name);
+static struct se_lun_acl *item_to_lun_acl(struct config_item *item)
+{
+       return container_of(to_config_group(item), struct se_lun_acl,
+                       se_lun_group);
+}
 
-static ssize_t target_fabric_mappedlun_show_write_protect(
-       struct se_lun_acl *lacl,
-       char *page)
+static ssize_t target_fabric_mappedlun_write_protect_show(
+               struct config_item *item, char *page)
 {
+       struct se_lun_acl *lacl = item_to_lun_acl(item);
        struct se_node_acl *se_nacl = lacl->se_lun_nacl;
        struct se_dev_entry *deve;
-       ssize_t len;
+       ssize_t len = 0;
 
-       spin_lock_irq(&se_nacl->device_list_lock);
-       deve = se_nacl->device_list[lacl->mapped_lun];
-       len = sprintf(page, "%d\n",
-                       (deve->lun_flags & TRANSPORT_LUNFLAGS_READ_ONLY) ?
-                       1 : 0);
-       spin_unlock_irq(&se_nacl->device_list_lock);
+       rcu_read_lock();
+       deve = target_nacl_find_deve(se_nacl, lacl->mapped_lun);
+       if (deve) {
+               len = sprintf(page, "%d\n",
+                       (deve->lun_flags & TRANSPORT_LUNFLAGS_READ_ONLY) ? 1 : 0);
+       }
+       rcu_read_unlock();
 
        return len;
 }
 
-static ssize_t target_fabric_mappedlun_store_write_protect(
-       struct se_lun_acl *lacl,
-       const char *page,
-       size_t count)
+static ssize_t target_fabric_mappedlun_write_protect_store(
+               struct config_item *item, const char *page, size_t count)
 {
+       struct se_lun_acl *lacl = item_to_lun_acl(item);
        struct se_node_acl *se_nacl = lacl->se_lun_nacl;
        struct se_portal_group *se_tpg = se_nacl->se_tpg;
        unsigned long op;
@@ -218,17 +197,20 @@ static ssize_t target_fabric_mappedlun_store_write_protect(
                        lacl->se_lun_nacl);
 
        pr_debug("%s_ConfigFS: Changed Initiator ACL: %s"
-               " Mapped LUN: %u Write Protect bit to %s\n",
+               " Mapped LUN: %llu Write Protect bit to %s\n",
                se_tpg->se_tpg_tfo->get_fabric_name(),
-               lacl->initiatorname, lacl->mapped_lun, (op) ? "ON" : "OFF");
+               se_nacl->initiatorname, lacl->mapped_lun, (op) ? "ON" : "OFF");
 
        return count;
 
 }
 
-TCM_MAPPEDLUN_ATTR(write_protect, S_IRUGO | S_IWUSR);
+CONFIGFS_ATTR(target_fabric_mappedlun_, write_protect);
 
-CONFIGFS_EATTR_OPS(target_fabric_mappedlun, se_lun_acl, se_lun_group);
+static struct configfs_attribute *target_fabric_mappedlun_attrs[] = {
+       &target_fabric_mappedlun_attr_write_protect,
+       NULL,
+};
 
 static void target_fabric_mappedlun_release(struct config_item *item)
 {
@@ -239,15 +221,8 @@ static void target_fabric_mappedlun_release(struct config_item *item)
        core_dev_free_initiator_node_lun_acl(se_tpg, lacl);
 }
 
-static struct configfs_attribute *target_fabric_mappedlun_attrs[] = {
-       &target_fabric_mappedlun_write_protect.attr,
-       NULL,
-};
-
 static struct configfs_item_operations target_fabric_mappedlun_item_ops = {
        .release                = target_fabric_mappedlun_release,
-       .show_attribute         = target_fabric_mappedlun_attr_show,
-       .store_attribute        = target_fabric_mappedlun_attr_store,
        .allow_link             = target_fabric_mappedlun_link,
        .drop_link              = target_fabric_mappedlun_unlink,
 };
@@ -283,49 +258,12 @@ TF_CIT_SETUP(tpg_mappedlun_stat, NULL, &target_fabric_mappedlun_stat_group_ops,
 
 /* End of tfc_tpg_mappedlun_port_cit */
 
-/* Start of tfc_tpg_nacl_attrib_cit */
-
-CONFIGFS_EATTR_OPS(target_fabric_nacl_attrib, se_node_acl, acl_attrib_group);
-
-static struct configfs_item_operations target_fabric_nacl_attrib_item_ops = {
-       .show_attribute         = target_fabric_nacl_attrib_attr_show,
-       .store_attribute        = target_fabric_nacl_attrib_attr_store,
-};
-
-TF_CIT_SETUP_DRV(tpg_nacl_attrib, &target_fabric_nacl_attrib_item_ops, NULL);
-
-/* End of tfc_tpg_nacl_attrib_cit */
-
-/* Start of tfc_tpg_nacl_auth_cit */
-
-CONFIGFS_EATTR_OPS(target_fabric_nacl_auth, se_node_acl, acl_auth_group);
-
-static struct configfs_item_operations target_fabric_nacl_auth_item_ops = {
-       .show_attribute         = target_fabric_nacl_auth_attr_show,
-       .store_attribute        = target_fabric_nacl_auth_attr_store,
-};
-
-TF_CIT_SETUP_DRV(tpg_nacl_auth, &target_fabric_nacl_auth_item_ops, NULL);
-
-/* End of tfc_tpg_nacl_auth_cit */
-
-/* Start of tfc_tpg_nacl_param_cit */
-
-CONFIGFS_EATTR_OPS(target_fabric_nacl_param, se_node_acl, acl_param_group);
-
-static struct configfs_item_operations target_fabric_nacl_param_item_ops = {
-       .show_attribute         = target_fabric_nacl_param_attr_show,
-       .store_attribute        = target_fabric_nacl_param_attr_store,
-};
-
-TF_CIT_SETUP_DRV(tpg_nacl_param, &target_fabric_nacl_param_item_ops, NULL);
-
-/* End of tfc_tpg_nacl_param_cit */
+TF_CIT_SETUP_DRV(tpg_nacl_attrib, NULL, NULL);
+TF_CIT_SETUP_DRV(tpg_nacl_auth, NULL, NULL);
+TF_CIT_SETUP_DRV(tpg_nacl_param, NULL, NULL);
 
 /* Start of tfc_tpg_nacl_base_cit */
 
-CONFIGFS_EATTR_OPS(target_fabric_nacl_base, se_node_acl, acl_group);
-
 static struct config_group *target_fabric_make_mappedlun(
        struct config_group *group,
        const char *name)
@@ -338,7 +276,7 @@ static struct config_group *target_fabric_make_mappedlun(
        struct config_item *acl_ci;
        struct config_group *lacl_cg = NULL, *ml_stat_grp = NULL;
        char *buf;
-       unsigned long mapped_lun;
+       unsigned long long mapped_lun;
        int ret = 0;
 
        acl_ci = &group->cg_item;
@@ -366,21 +304,9 @@ static struct config_group *target_fabric_make_mappedlun(
         * Determine the Mapped LUN value.  This is what the SCSI Initiator
         * Port will actually see.
         */
-       ret = kstrtoul(buf + 4, 0, &mapped_lun);
+       ret = kstrtoull(buf + 4, 0, &mapped_lun);
        if (ret)
                goto out;
-       if (mapped_lun > UINT_MAX) {
-               ret = -EINVAL;
-               goto out;
-       }
-       if (mapped_lun > (TRANSPORT_MAX_LUNS_PER_TPG-1)) {
-               pr_err("Mapped LUN: %lu exceeds TRANSPORT_MAX_LUNS_PER_TPG"
-                       "-1: %u for Target Portal Group: %u\n", mapped_lun,
-                       TRANSPORT_MAX_LUNS_PER_TPG-1,
-                       se_tpg->se_tpg_tfo->tpg_get_tag(se_tpg));
-               ret = -EINVAL;
-               goto out;
-       }
 
        lacl = core_dev_init_initiator_node_lun_acl(se_tpg, se_nacl,
                        mapped_lun, &ret);
@@ -399,9 +325,9 @@ static struct config_group *target_fabric_make_mappedlun(
        }
 
        config_group_init_type_name(&lacl->se_lun_group, name,
-                       &tf->tf_cit_tmpl.tfc_tpg_mappedlun_cit);
+                       &tf->tf_tpg_mappedlun_cit);
        config_group_init_type_name(&lacl->ml_stat_grps.stat_group,
-                       "statistics", &tf->tf_cit_tmpl.tfc_tpg_mappedlun_stat_cit);
+                       "statistics", &tf->tf_tpg_mappedlun_stat_cit);
        lacl_cg->default_groups[0] = &lacl->ml_stat_grps.stat_group;
        lacl_cg->default_groups[1] = NULL;
 
@@ -458,16 +384,15 @@ static void target_fabric_nacl_base_release(struct config_item *item)
 {
        struct se_node_acl *se_nacl = container_of(to_config_group(item),
                        struct se_node_acl, acl_group);
-       struct se_portal_group *se_tpg = se_nacl->se_tpg;
-       struct target_fabric_configfs *tf = se_tpg->se_tpg_wwn->wwn_tf;
+       struct target_fabric_configfs *tf = se_nacl->se_tpg->se_tpg_wwn->wwn_tf;
 
-       tf->tf_ops.fabric_drop_nodeacl(se_nacl);
+       if (tf->tf_ops->fabric_cleanup_nodeacl)
+               tf->tf_ops->fabric_cleanup_nodeacl(se_nacl);
+       core_tpg_del_initiator_node_acl(se_nacl);
 }
 
 static struct configfs_item_operations target_fabric_nacl_base_item_ops = {
        .release                = target_fabric_nacl_base_release,
-       .show_attribute         = target_fabric_nacl_base_attr_show,
-       .store_attribute        = target_fabric_nacl_base_attr_store,
 };
 
 static struct configfs_group_operations target_fabric_nacl_base_group_ops = {
@@ -501,15 +426,18 @@ static struct config_group *target_fabric_make_nodeacl(
        struct se_node_acl *se_nacl;
        struct config_group *nacl_cg;
 
-       if (!tf->tf_ops.fabric_make_nodeacl) {
-               pr_err("tf->tf_ops.fabric_make_nodeacl is NULL\n");
-               return ERR_PTR(-ENOSYS);
-       }
-
-       se_nacl = tf->tf_ops.fabric_make_nodeacl(se_tpg, group, name);
+       se_nacl = core_tpg_add_initiator_node_acl(se_tpg, name);
        if (IS_ERR(se_nacl))
                return ERR_CAST(se_nacl);
 
+       if (tf->tf_ops->fabric_init_nodeacl) {
+               int ret = tf->tf_ops->fabric_init_nodeacl(se_nacl, name);
+               if (ret) {
+                       core_tpg_del_initiator_node_acl(se_nacl);
+                       return ERR_PTR(ret);
+               }
+       }
+
        nacl_cg = &se_nacl->acl_group;
        nacl_cg->default_groups = se_nacl->acl_default_groups;
        nacl_cg->default_groups[0] = &se_nacl->acl_attrib_group;
@@ -519,16 +447,15 @@ static struct config_group *target_fabric_make_nodeacl(
        nacl_cg->default_groups[4] = NULL;
 
        config_group_init_type_name(&se_nacl->acl_group, name,
-                       &tf->tf_cit_tmpl.tfc_tpg_nacl_base_cit);
+                       &tf->tf_tpg_nacl_base_cit);
        config_group_init_type_name(&se_nacl->acl_attrib_group, "attrib",
-                       &tf->tf_cit_tmpl.tfc_tpg_nacl_attrib_cit);
+                       &tf->tf_tpg_nacl_attrib_cit);
        config_group_init_type_name(&se_nacl->acl_auth_group, "auth",
-                       &tf->tf_cit_tmpl.tfc_tpg_nacl_auth_cit);
+                       &tf->tf_tpg_nacl_auth_cit);
        config_group_init_type_name(&se_nacl->acl_param_group, "param",
-                       &tf->tf_cit_tmpl.tfc_tpg_nacl_param_cit);
+                       &tf->tf_tpg_nacl_param_cit);
        config_group_init_type_name(&se_nacl->acl_fabric_stat_group,
-                       "fabric_statistics",
-                       &tf->tf_cit_tmpl.tfc_tpg_nacl_stat_cit);
+                       "fabric_statistics", &tf->tf_tpg_nacl_stat_cit);
 
        return &se_nacl->acl_group;
 }
@@ -566,8 +493,6 @@ TF_CIT_SETUP(tpg_nacl, NULL, &target_fabric_nacl_group_ops, NULL);
 
 /* Start of tfc_tpg_np_base_cit */
 
-CONFIGFS_EATTR_OPS(target_fabric_np_base, se_tpg_np, tpg_np_group);
-
 static void target_fabric_np_base_release(struct config_item *item)
 {
        struct se_tpg_np *se_tpg_np = container_of(to_config_group(item),
@@ -575,13 +500,11 @@ static void target_fabric_np_base_release(struct config_item *item)
        struct se_portal_group *se_tpg = se_tpg_np->tpg_np_parent;
        struct target_fabric_configfs *tf = se_tpg->se_tpg_wwn->wwn_tf;
 
-       tf->tf_ops.fabric_drop_np(se_tpg_np);
+       tf->tf_ops->fabric_drop_np(se_tpg_np);
 }
 
 static struct configfs_item_operations target_fabric_np_base_item_ops = {
        .release                = target_fabric_np_base_release,
-       .show_attribute         = target_fabric_np_base_attr_show,
-       .store_attribute        = target_fabric_np_base_attr_store,
 };
 
 TF_CIT_SETUP_DRV(tpg_np_base, &target_fabric_np_base_item_ops, NULL);
@@ -599,18 +522,18 @@ static struct config_group *target_fabric_make_np(
        struct target_fabric_configfs *tf = se_tpg->se_tpg_wwn->wwn_tf;
        struct se_tpg_np *se_tpg_np;
 
-       if (!tf->tf_ops.fabric_make_np) {
+       if (!tf->tf_ops->fabric_make_np) {
                pr_err("tf->tf_ops.fabric_make_np is NULL\n");
                return ERR_PTR(-ENOSYS);
        }
 
-       se_tpg_np = tf->tf_ops.fabric_make_np(se_tpg, group, name);
+       se_tpg_np = tf->tf_ops->fabric_make_np(se_tpg, group, name);
        if (!se_tpg_np || IS_ERR(se_tpg_np))
                return ERR_PTR(-EINVAL);
 
        se_tpg_np->tpg_np_parent = se_tpg;
        config_group_init_type_name(&se_tpg_np->tpg_np_group, name,
-                       &tf->tf_cit_tmpl.tfc_tpg_np_base_cit);
+                       &tf->tf_tpg_np_base_cit);
 
        return &se_tpg_np->tpg_np_group;
 }
@@ -636,132 +559,113 @@ TF_CIT_SETUP(tpg_np, NULL, &target_fabric_np_group_ops, NULL);
 
 /* Start of tfc_tpg_port_cit */
 
-CONFIGFS_EATTR_STRUCT(target_fabric_port, se_lun);
-#define TCM_PORT_ATTR(_name, _mode)                                    \
-static struct target_fabric_port_attribute target_fabric_port_##_name =        \
-       __CONFIGFS_EATTR(_name, _mode,                                  \
-       target_fabric_port_show_attr_##_name,                           \
-       target_fabric_port_store_attr_##_name);
-
-#define TCM_PORT_ATTOR_RO(_name)                                       \
-       __CONFIGFS_EATTR_RO(_name,                                      \
-       target_fabric_port_show_attr_##_name);
+static struct se_lun *item_to_lun(struct config_item *item)
+{
+       return container_of(to_config_group(item), struct se_lun,
+                       lun_group);
+}
 
-/*
- * alua_tg_pt_gp
- */
-static ssize_t target_fabric_port_show_attr_alua_tg_pt_gp(
-       struct se_lun *lun,
-       char *page)
+static ssize_t target_fabric_port_alua_tg_pt_gp_show(struct config_item *item,
+               char *page)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
-       return core_alua_show_tg_pt_gp_info(lun->lun_sep, page);
+       return core_alua_show_tg_pt_gp_info(lun, page);
 }
 
-static ssize_t target_fabric_port_store_attr_alua_tg_pt_gp(
-       struct se_lun *lun,
-       const char *page,
-       size_t count)
+static ssize_t target_fabric_port_alua_tg_pt_gp_store(struct config_item *item,
+               const char *page, size_t count)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
-       return core_alua_store_tg_pt_gp_info(lun->lun_sep, page, count);
+       return core_alua_store_tg_pt_gp_info(lun, page, count);
 }
 
-TCM_PORT_ATTR(alua_tg_pt_gp, S_IRUGO | S_IWUSR);
-
-/*
- * alua_tg_pt_offline
- */
-static ssize_t target_fabric_port_show_attr_alua_tg_pt_offline(
-       struct se_lun *lun,
-       char *page)
+static ssize_t target_fabric_port_alua_tg_pt_offline_show(
+               struct config_item *item, char *page)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
        return core_alua_show_offline_bit(lun, page);
 }
 
-static ssize_t target_fabric_port_store_attr_alua_tg_pt_offline(
-       struct se_lun *lun,
-       const char *page,
-       size_t count)
+static ssize_t target_fabric_port_alua_tg_pt_offline_store(
+               struct config_item *item, const char *page, size_t count)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
        return core_alua_store_offline_bit(lun, page, count);
 }
 
-TCM_PORT_ATTR(alua_tg_pt_offline, S_IRUGO | S_IWUSR);
-
-/*
- * alua_tg_pt_status
- */
-static ssize_t target_fabric_port_show_attr_alua_tg_pt_status(
-       struct se_lun *lun,
-       char *page)
+static ssize_t target_fabric_port_alua_tg_pt_status_show(
+               struct config_item *item, char *page)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
        return core_alua_show_secondary_status(lun, page);
 }
 
-static ssize_t target_fabric_port_store_attr_alua_tg_pt_status(
-       struct se_lun *lun,
-       const char *page,
-       size_t count)
+static ssize_t target_fabric_port_alua_tg_pt_status_store(
+               struct config_item *item, const char *page, size_t count)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
        return core_alua_store_secondary_status(lun, page, count);
 }
 
-TCM_PORT_ATTR(alua_tg_pt_status, S_IRUGO | S_IWUSR);
-
-/*
- * alua_tg_pt_write_md
- */
-static ssize_t target_fabric_port_show_attr_alua_tg_pt_write_md(
-       struct se_lun *lun,
-       char *page)
+static ssize_t target_fabric_port_alua_tg_pt_write_md_show(
+               struct config_item *item, char *page)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
        return core_alua_show_secondary_write_metadata(lun, page);
 }
 
-static ssize_t target_fabric_port_store_attr_alua_tg_pt_write_md(
-       struct se_lun *lun,
-       const char *page,
-       size_t count)
+static ssize_t target_fabric_port_alua_tg_pt_write_md_store(
+               struct config_item *item, const char *page, size_t count)
 {
-       if (!lun || !lun->lun_sep)
+       struct se_lun *lun = item_to_lun(item);
+
+       if (!lun || !lun->lun_se_dev)
                return -ENODEV;
 
        return core_alua_store_secondary_write_metadata(lun, page, count);
 }
 
-TCM_PORT_ATTR(alua_tg_pt_write_md, S_IRUGO | S_IWUSR);
-
+CONFIGFS_ATTR(target_fabric_port_, alua_tg_pt_gp);
+CONFIGFS_ATTR(target_fabric_port_, alua_tg_pt_offline);
+CONFIGFS_ATTR(target_fabric_port_, alua_tg_pt_status);
+CONFIGFS_ATTR(target_fabric_port_, alua_tg_pt_write_md);
 
 static struct configfs_attribute *target_fabric_port_attrs[] = {
-       &target_fabric_port_alua_tg_pt_gp.attr,
-       &target_fabric_port_alua_tg_pt_offline.attr,
-       &target_fabric_port_alua_tg_pt_status.attr,
-       &target_fabric_port_alua_tg_pt_write_md.attr,
+       &target_fabric_port_attr_alua_tg_pt_gp,
+       &target_fabric_port_attr_alua_tg_pt_offline,
+       &target_fabric_port_attr_alua_tg_pt_status,
+       &target_fabric_port_attr_alua_tg_pt_write_md,
        NULL,
 };
 
-CONFIGFS_EATTR_OPS(target_fabric_port, se_lun, lun_group);
-
 static int target_fabric_port_link(
        struct config_item *lun_ci,
        struct config_item *se_dev_ci)
@@ -769,7 +673,6 @@ static int target_fabric_port_link(
        struct config_item *tpg_ci;
        struct se_lun *lun = container_of(to_config_group(lun_ci),
                                struct se_lun, lun_group);
-       struct se_lun *lun_p;
        struct se_portal_group *se_tpg;
        struct se_device *dev =
                container_of(to_config_group(se_dev_ci), struct se_device, dev_group);
@@ -797,20 +700,19 @@ static int target_fabric_port_link(
                return -EEXIST;
        }
 
-       lun_p = core_dev_add_lun(se_tpg, dev, lun->unpacked_lun);
-       if (IS_ERR(lun_p)) {
-               pr_err("core_dev_add_lun() failed\n");
-               ret = PTR_ERR(lun_p);
+       ret = core_dev_add_lun(se_tpg, dev, lun);
+       if (ret) {
+               pr_err("core_dev_add_lun() failed: %d\n", ret);
                goto out;
        }
 
-       if (tf->tf_ops.fabric_post_link) {
+       if (tf->tf_ops->fabric_post_link) {
                /*
                 * Call the optional fabric_post_link() to allow a
                 * fabric module to setup any additional state once
                 * core_dev_add_lun() has been called..
                 */
-               tf->tf_ops.fabric_post_link(se_tpg, lun);
+               tf->tf_ops->fabric_post_link(se_tpg, lun);
        }
 
        return 0;
@@ -824,25 +726,32 @@ static int target_fabric_port_unlink(
 {
        struct se_lun *lun = container_of(to_config_group(lun_ci),
                                struct se_lun, lun_group);
-       struct se_portal_group *se_tpg = lun->lun_sep->sep_tpg;
+       struct se_portal_group *se_tpg = lun->lun_tpg;
        struct target_fabric_configfs *tf = se_tpg->se_tpg_wwn->wwn_tf;
 
-       if (tf->tf_ops.fabric_pre_unlink) {
+       if (tf->tf_ops->fabric_pre_unlink) {
                /*
                 * Call the optional fabric_pre_unlink() to allow a
                 * fabric module to release any additional stat before
                 * core_dev_del_lun() is called.
                */
-               tf->tf_ops.fabric_pre_unlink(se_tpg, lun);
+               tf->tf_ops->fabric_pre_unlink(se_tpg, lun);
        }
 
        core_dev_del_lun(se_tpg, lun);
        return 0;
 }
 
+static void target_fabric_port_release(struct config_item *item)
+{
+       struct se_lun *lun = container_of(to_config_group(item),
+                                         struct se_lun, lun_group);
+
+       kfree_rcu(lun, rcu_head);
+}
+
 static struct configfs_item_operations target_fabric_port_item_ops = {
-       .show_attribute         = target_fabric_port_attr_show,
-       .store_attribute        = target_fabric_port_attr_store,
+       .release                = target_fabric_port_release,
        .allow_link             = target_fabric_port_link,
        .drop_link              = target_fabric_port_unlink,
 };
@@ -887,7 +796,7 @@ static struct config_group *target_fabric_make_lun(
                        struct se_portal_group, tpg_lun_group);
        struct target_fabric_configfs *tf = se_tpg->se_tpg_wwn->wwn_tf;
        struct config_group *lun_cg = NULL, *port_stat_grp = NULL;
-       unsigned long unpacked_lun;
+       unsigned long long unpacked_lun;
        int errno;
 
        if (strstr(name, "lun_") != name) {
@@ -895,28 +804,27 @@ static struct config_group *target_fabric_make_lun(
                                " \"lun_$LUN_NUMBER\"\n");
                return ERR_PTR(-EINVAL);
        }
-       errno = kstrtoul(name + 4, 0, &unpacked_lun);
+       errno = kstrtoull(name + 4, 0, &unpacked_lun);
        if (errno)
                return ERR_PTR(errno);
-       if (unpacked_lun > UINT_MAX)
-               return ERR_PTR(-EINVAL);
 
-       lun = core_get_lun_from_tpg(se_tpg, unpacked_lun);
-       if (!lun)
-               return ERR_PTR(-EINVAL);
+       lun = core_tpg_alloc_lun(se_tpg, unpacked_lun);
+       if (IS_ERR(lun))
+               return ERR_CAST(lun);
 
        lun_cg = &lun->lun_group;
        lun_cg->default_groups = kmalloc(sizeof(struct config_group *) * 2,
                                GFP_KERNEL);
        if (!lun_cg->default_groups) {
                pr_err("Unable to allocate lun_cg->default_groups\n");
+               kfree(lun);
                return ERR_PTR(-ENOMEM);
        }
 
        config_group_init_type_name(&lun->lun_group, name,
-                       &tf->tf_cit_tmpl.tfc_tpg_port_cit);
+                       &tf->tf_tpg_port_cit);
        config_group_init_type_name(&lun->port_stat_grps.stat_group,
-                       "statistics", &tf->tf_cit_tmpl.tfc_tpg_port_stat_cit);
+                       "statistics", &tf->tf_tpg_port_stat_cit);
        lun_cg->default_groups[0] = &lun->port_stat_grps.stat_group;
        lun_cg->default_groups[1] = NULL;
 
@@ -926,6 +834,7 @@ static struct config_group *target_fabric_make_lun(
        if (!port_stat_grp->default_groups) {
                pr_err("Unable to allocate port_stat_grp->default_groups\n");
                kfree(lun_cg->default_groups);
+               kfree(lun);
                return ERR_PTR(-ENOMEM);
        }
        target_stat_setup_port_default_groups(lun);
@@ -971,50 +880,11 @@ TF_CIT_SETUP(tpg_lun, NULL, &target_fabric_lun_group_ops, NULL);
 
 /* End of tfc_tpg_lun_cit */
 
-/* Start of tfc_tpg_attrib_cit */
-
-CONFIGFS_EATTR_OPS(target_fabric_tpg_attrib, se_portal_group, tpg_attrib_group);
-
-static struct configfs_item_operations target_fabric_tpg_attrib_item_ops = {
-       .show_attribute         = target_fabric_tpg_attrib_attr_show,
-       .store_attribute        = target_fabric_tpg_attrib_attr_store,
-};
-
-TF_CIT_SETUP_DRV(tpg_attrib, &target_fabric_tpg_attrib_item_ops, NULL);
-
-/* End of tfc_tpg_attrib_cit */
-
-/* Start of tfc_tpg_auth_cit */
-
-CONFIGFS_EATTR_OPS(target_fabric_tpg_auth, se_portal_group, tpg_auth_group);
-
-static struct configfs_item_operations target_fabric_tpg_auth_item_ops = {
-       .show_attribute         = target_fabric_tpg_auth_attr_show,
-       .store_attribute        = target_fabric_tpg_auth_attr_store,
-};
-
-TF_CIT_SETUP_DRV(tpg_auth, &target_fabric_tpg_auth_item_ops, NULL);
-
-/* End of tfc_tpg_attrib_cit */
-
-/* Start of tfc_tpg_param_cit */
-
-CONFIGFS_EATTR_OPS(target_fabric_tpg_param, se_portal_group, tpg_param_group);
-
-static struct configfs_item_operations target_fabric_tpg_param_item_ops = {
-       .show_attribute         = target_fabric_tpg_param_attr_show,
-       .store_attribute        = target_fabric_tpg_param_attr_store,
-};
-
-TF_CIT_SETUP_DRV(tpg_param, &target_fabric_tpg_param_item_ops, NULL);
-
-/* End of tfc_tpg_param_cit */
+TF_CIT_SETUP_DRV(tpg_attrib, NULL, NULL);
+TF_CIT_SETUP_DRV(tpg_auth, NULL, NULL);
+TF_CIT_SETUP_DRV(tpg_param, NULL, NULL);
 
 /* Start of tfc_tpg_base_cit */
-/*
- * For use with TF_TPG_ATTR() and TF_TPG_ATTR_RO()
- */
-CONFIGFS_EATTR_OPS(target_fabric_tpg, se_portal_group, tpg_group);
 
 static void target_fabric_tpg_release(struct config_item *item)
 {
@@ -1023,13 +893,11 @@ static void target_fabric_tpg_release(struct config_item *item)
        struct se_wwn *wwn = se_tpg->se_tpg_wwn;
        struct target_fabric_configfs *tf = wwn->wwn_tf;
 
-       tf->tf_ops.fabric_drop_tpg(se_tpg);
+       tf->tf_ops->fabric_drop_tpg(se_tpg);
 }
 
 static struct configfs_item_operations target_fabric_tpg_base_item_ops = {
        .release                = target_fabric_tpg_release,
-       .show_attribute         = target_fabric_tpg_attr_show,
-       .store_attribute        = target_fabric_tpg_attr_store,
 };
 
 TF_CIT_SETUP_DRV(tpg_base, &target_fabric_tpg_base_item_ops, NULL);
@@ -1046,12 +914,12 @@ static struct config_group *target_fabric_make_tpg(
        struct target_fabric_configfs *tf = wwn->wwn_tf;
        struct se_portal_group *se_tpg;
 
-       if (!tf->tf_ops.fabric_make_tpg) {
-               pr_err("tf->tf_ops.fabric_make_tpg is NULL\n");
+       if (!tf->tf_ops->fabric_make_tpg) {
+               pr_err("tf->tf_ops->fabric_make_tpg is NULL\n");
                return ERR_PTR(-ENOSYS);
        }
 
-       se_tpg = tf->tf_ops.fabric_make_tpg(wwn, group, name);
+       se_tpg = tf->tf_ops->fabric_make_tpg(wwn, group, name);
        if (!se_tpg || IS_ERR(se_tpg))
                return ERR_PTR(-EINVAL);
        /*
@@ -1067,19 +935,19 @@ static struct config_group *target_fabric_make_tpg(
        se_tpg->tpg_group.default_groups[6] = NULL;
 
        config_group_init_type_name(&se_tpg->tpg_group, name,
-                       &tf->tf_cit_tmpl.tfc_tpg_base_cit);
+                       &tf->tf_tpg_base_cit);
        config_group_init_type_name(&se_tpg->tpg_lun_group, "lun",
-                       &tf->tf_cit_tmpl.tfc_tpg_lun_cit);
+                       &tf->tf_tpg_lun_cit);
        config_group_init_type_name(&se_tpg->tpg_np_group, "np",
-                       &tf->tf_cit_tmpl.tfc_tpg_np_cit);
+                       &tf->tf_tpg_np_cit);
        config_group_init_type_name(&se_tpg->tpg_acl_group, "acls",
-                       &tf->tf_cit_tmpl.tfc_tpg_nacl_cit);
+                       &tf->tf_tpg_nacl_cit);
        config_group_init_type_name(&se_tpg->tpg_attrib_group, "attrib",
-                       &tf->tf_cit_tmpl.tfc_tpg_attrib_cit);
+                       &tf->tf_tpg_attrib_cit);
        config_group_init_type_name(&se_tpg->tpg_auth_group, "auth",
-                       &tf->tf_cit_tmpl.tfc_tpg_auth_cit);
+                       &tf->tf_tpg_auth_cit);
        config_group_init_type_name(&se_tpg->tpg_param_group, "param",
-                       &tf->tf_cit_tmpl.tfc_tpg_param_cit);
+                       &tf->tf_tpg_param_cit);
 
        return &se_tpg->tpg_group;
 }
@@ -1112,7 +980,7 @@ static void target_fabric_release_wwn(struct config_item *item)
                                struct se_wwn, wwn_group);
        struct target_fabric_configfs *tf = wwn->wwn_tf;
 
-       tf->tf_ops.fabric_drop_wwn(wwn);
+       tf->tf_ops->fabric_drop_wwn(wwn);
 }
 
 static struct configfs_item_operations target_fabric_tpg_item_ops = {
@@ -1148,12 +1016,12 @@ static struct config_group *target_fabric_make_wwn(
                                struct target_fabric_configfs, tf_group);
        struct se_wwn *wwn;
 
-       if (!tf->tf_ops.fabric_make_wwn) {
+       if (!tf->tf_ops->fabric_make_wwn) {
                pr_err("tf->tf_ops.fabric_make_wwn is NULL\n");
                return ERR_PTR(-ENOSYS);
        }
 
-       wwn = tf->tf_ops.fabric_make_wwn(tf, group, name);
+       wwn = tf->tf_ops->fabric_make_wwn(tf, group, name);
        if (!wwn || IS_ERR(wwn))
                return ERR_PTR(-EINVAL);
 
@@ -1165,10 +1033,9 @@ static struct config_group *target_fabric_make_wwn(
        wwn->wwn_group.default_groups[0] = &wwn->fabric_stat_group;
        wwn->wwn_group.default_groups[1] = NULL;
 
-       config_group_init_type_name(&wwn->wwn_group, name,
-                       &tf->tf_cit_tmpl.tfc_tpg_cit);
+       config_group_init_type_name(&wwn->wwn_group, name, &tf->tf_tpg_cit);
        config_group_init_type_name(&wwn->fabric_stat_group, "fabric_statistics",
-                       &tf->tf_cit_tmpl.tfc_wwn_fabric_stats_cit);
+                       &tf->tf_wwn_fabric_stats_cit);
 
        return &wwn->wwn_group;
 }
@@ -1196,33 +1063,9 @@ static struct configfs_group_operations target_fabric_wwn_group_ops = {
        .make_group     = target_fabric_make_wwn,
        .drop_item      = target_fabric_drop_wwn,
 };
-/*
- * For use with TF_WWN_ATTR() and TF_WWN_ATTR_RO()
- */
-CONFIGFS_EATTR_OPS(target_fabric_wwn, target_fabric_configfs, tf_group);
-
-static struct configfs_item_operations target_fabric_wwn_item_ops = {
-       .show_attribute         = target_fabric_wwn_attr_show,
-       .store_attribute        = target_fabric_wwn_attr_store,
-};
-
-TF_CIT_SETUP_DRV(wwn, &target_fabric_wwn_item_ops, &target_fabric_wwn_group_ops);
-
-/* End of tfc_wwn_cit */
-
-/* Start of tfc_discovery_cit */
-
-CONFIGFS_EATTR_OPS(target_fabric_discovery, target_fabric_configfs,
-               tf_disc_group);
-
-static struct configfs_item_operations target_fabric_discovery_item_ops = {
-       .show_attribute         = target_fabric_discovery_attr_show,
-       .store_attribute        = target_fabric_discovery_attr_store,
-};
-
-TF_CIT_SETUP_DRV(discovery, &target_fabric_discovery_item_ops, NULL);
 
-/* End of tfc_discovery_cit */
+TF_CIT_SETUP_DRV(wwn, NULL, &target_fabric_wwn_group_ops);
+TF_CIT_SETUP_DRV(discovery, NULL, NULL);
 
 int target_fabric_setup_cits(struct target_fabric_configfs *tf)
 {