From: Xiubo Li <xiu...@redhat.com>
For some case we need some module wide configfs to contol some
attributes of the whole transport module.
Signed-off-by: Xiubo Li <xiu...@redhat.com>
---
drivers/target/target_core_configfs.c | 46 +++++++++++++++++++++++++++++++++++
drivers/target/target_core_hba.c | 11 +++++++++
drivers/target/target_core_internal.h | 5 ++++
include/target/target_core_backend.h | 1 +
4 files changed, 63 insertions(+)
diff --git a/drivers/target/target_core_configfs.c
b/drivers/target/target_core_configfs.c
index 3f4bf12..a1ee716 100644
--- a/drivers/target/target_core_configfs.c
+++ b/drivers/target/target_core_configfs.c
@@ -79,6 +79,7 @@
static struct config_group target_core_hbagroup;
static struct config_group alua_group;
static struct config_group alua_lu_gps_group;
+static struct config_group action_group;
static inline struct se_hba *
item_to_hba(struct config_item *item)
@@ -1198,6 +1199,7 @@ struct configfs_attribute *passthrough_attrib_attrs[] = {
TB_CIT_SETUP_DRV(dev_attrib, NULL, NULL);
TB_CIT_SETUP_DRV(dev_action, NULL, NULL);
+TB_CIT_SETUP_DRV(mod_action, NULL, NULL);
/* End functions for struct config_item_type tb_dev_attrib_cit */
@@ -2893,6 +2895,20 @@ static void target_core_alua_drop_tg_pt_gp(
/* End functions for struct config_item_type target_core_alua_cit */
+/* Start functions for struct config_item_type target_core_action_cit */
+
+/*
+ * target_core_action_cit is a ConfigFS group that lives under
+ * /sys/kernel/config/target/core/action.
+ */
+static const struct config_item_type target_core_action_cit = {
+ .ct_item_ops = NULL,
+ .ct_attrs = NULL,
+ .ct_owner = THIS_MODULE,
+};
+
+/* End functions for struct config_item_type target_core_action_cit */
+
/* Start functions for struct config_item_type tb_dev_stat_cit */
static struct config_group *target_core_stat_mkdir(
@@ -3211,6 +3227,30 @@ void target_setup_backend_cits(struct target_backend *tb)
target_core_setup_dev_wwn_cit(tb);
target_core_setup_dev_alua_tg_pt_gps_cit(tb);
target_core_setup_dev_stat_cit(tb);
+
+ target_core_setup_mod_action_cit(tb);
+}
+
+int target_setup_backend_action(struct target_backend *tb)
+{
+ if (!tb->ops->tb_mod_action_attrs)
+ return 0;
+
+ tb->action_group = configfs_register_default_group(&action_group,
+ tb->ops->name,
+ &tb->tb_mod_action_cit);
+ if (!tb->action_group)
+ return PTR_ERR(tb->action_group);
+
+ return 0;
+}
+
+void target_unset_backend_action(struct target_backend *tb)
+{
+ if (!tb->ops->tb_mod_action_attrs)
+ return;
+
+ configfs_unregister_default_group(tb->action_group);
}
static int __init target_core_init_configfs(void)
@@ -3267,6 +3307,12 @@ static int __init target_core_init_configfs(void)
default_lu_gp = lu_gp;
/*
+ * Create ALUA infrastructure under
/sys/kernel/config/target/core/action/
+ */
+ config_group_init_type_name(&action_group, "action",
&target_core_action_cit);
+ configfs_add_default_group(&action_group, &target_core_hbagroup);
+
+ /*
* Register the target_core_mod subsystem with configfs.
*/
ret = configfs_register_subsystem(subsys);
diff --git a/drivers/target/target_core_hba.c b/drivers/target/target_core_hba.c
index 22390e0..6903087 100644
--- a/drivers/target/target_core_hba.c
+++ b/drivers/target/target_core_hba.c
@@ -51,6 +51,7 @@
int transport_backend_register(const struct target_backend_ops *ops)
{
struct target_backend *tb, *old;
+ int ret;
tb = kzalloc(sizeof(*tb), GFP_KERNEL);
if (!tb)
@@ -67,11 +68,20 @@ int transport_backend_register(const struct
target_backend_ops *ops)
}
}
target_setup_backend_cits(tb);
+
+ ret = target_setup_backend_action(tb);
+ if (ret) {
+ mutex_unlock(&backend_mutex);
+ kfree(tb);
+ return ret;
+ }
+
list_add_tail(&tb->list, &backend_list);
mutex_unlock(&backend_mutex);
pr_debug("TCM: Registered subsystem plugin: %s struct module: %p\n",
ops->name, ops->owner);
+
return 0;
}
EXPORT_SYMBOL(transport_backend_register);
@@ -84,6 +94,7 @@ void target_backend_unregister(const struct
target_backend_ops *ops)
list_for_each_entry(tb, &backend_list, list) {
if (tb->ops == ops) {
list_del(&tb->list);
+ target_unset_backend_action(tb);
mutex_unlock(&backend_mutex);
/*
* Wait for any outstanding backend driver ->rcu_head
diff --git a/drivers/target/target_core_internal.h
b/drivers/target/target_core_internal.h
index 1d5afc3..73e8157 100644
--- a/drivers/target/target_core_internal.h
+++ b/drivers/target/target_core_internal.h
@@ -22,6 +22,9 @@ struct target_backend {
struct config_item_type tb_dev_wwn_cit;
struct config_item_type tb_dev_alua_tg_pt_gps_cit;
struct config_item_type tb_dev_stat_cit;
+
+ struct config_group *action_group;
+ struct config_item_type tb_mod_action_cit;
};
struct target_fabric_configfs {
@@ -92,6 +95,8 @@ int target_for_each_device(int (*fn)(struct se_device *dev,
void *data),
/* target_core_configfs.c */
extern struct configfs_item_operations target_core_dev_item_ops;
void target_setup_backend_cits(struct target_backend *);
+int target_setup_backend_action(struct target_backend *tb);
+void target_unset_backend_action(struct target_backend *tb);
/* target_core_fabric_configfs.c */
int target_fabric_setup_cits(struct target_fabric_configfs *);
diff --git a/include/target/target_core_backend.h
b/include/target/target_core_backend.h
index 34a15d5..5df7059 100644
--- a/include/target/target_core_backend.h
+++ b/include/target/target_core_backend.h
@@ -54,6 +54,7 @@ struct target_backend_ops {
struct configfs_attribute **tb_dev_attrib_attrs;
struct configfs_attribute **tb_dev_action_attrs;
+ struct configfs_attribute **tb_mod_action_attrs;
};
struct sbc_ops {
--
1.8.3.1
