Add API trace_usb_configfs_make_group() and trace_usb_configfs_drop_group()
to trace user create gadget/function/configuration... groups,
it also trace group create in a specific function.
Signed-off-by: Linyu Yuan <quic_linyyuan@xxxxxxxxxxx>
---
drivers/usb/gadget/configfs.c | 88 ++++++++++++++++++++++++++++
drivers/usb/gadget/function/f_mass_storage.c | 4 ++
drivers/usb/gadget/function/uvc_configfs.c | 12 ++++
include/linux/usb/composite.h | 9 +++
include/linux/usb/gadget_configfs.h | 2 +
5 files changed, 115 insertions(+)
diff --git a/drivers/usb/gadget/configfs.c b/drivers/usb/gadget/configfs.c
index f35a226..be9e70a 100644
--- a/drivers/usb/gadget/configfs.c
+++ b/drivers/usb/gadget/configfs.c
@@ -624,6 +624,8 @@ static struct config_group *function_make(
gi = container_of(group, struct gadget_info, functions_group);
+ trace_usb_configfs_make_group(&group->cg_item, &fi->group.cg_item);
+
mutex_lock(&gi->lock);
list_add_tail(&fi->cfs_list, &gi->available_func);
mutex_unlock(&gi->lock);
@@ -637,6 +639,8 @@ static void function_drop(
struct usb_function_instance *fi = to_usb_function_instance(item);
struct gadget_info *gi;
+ trace_usb_configfs_drop_group(&group->cg_item, item);
+
gi = container_of(group, struct gadget_info, functions_group);
mutex_lock(&gi->lock);
@@ -732,6 +736,7 @@ static struct config_group *config_desc_make(
if (ret)
goto err;
+ trace_usb_configfs_make_group(&group->cg_item, &cfg->group.cg_item);
return &cfg->group;
err:
kfree(cfg->c.label);
@@ -743,6 +748,7 @@ static void config_desc_drop(
struct config_group *group,
struct config_item *item)
{
+ trace_usb_configfs_drop_group(&group->cg_item, item);
config_item_put(item);
}
@@ -1086,6 +1092,7 @@ static struct config_item *ext_prop_make(
ext_prop_type->ct_owner = desc->owner;
config_item_init_type_name(&ext_prop->item, name, ext_prop_type);
+ trace_usb_configfs_make_group(&group->cg_item, &ext_prop->item);
ext_prop->name = kstrdup(name, GFP_KERNEL);
if (!ext_prop->name) {
@@ -1110,6 +1117,8 @@ static void ext_prop_drop(struct config_group *group, struct config_item *item)
struct usb_os_desc_ext_prop *ext_prop = to_usb_os_desc_ext_prop(item);
struct usb_os_desc *desc = to_usb_os_desc(&group->cg_item);
+ trace_usb_configfs_drop_group(&group->cg_item, item);
+
if (desc->opts_mutex)
mutex_lock(desc->opts_mutex);
list_del(&ext_prop->entry);
@@ -1629,6 +1638,7 @@ static struct config_group *gadgets_make(
if (!gi->composite.gadget_driver.function)
goto err;
+ trace_usb_configfs_make_group(&group->cg_item, &gi->group.cg_item);
return &gi->group;
err:
kfree(gi);
@@ -1637,6 +1647,7 @@ static struct config_group *gadgets_make(
static void gadgets_drop(struct config_group *group, struct config_item *item)
{
+ trace_usb_configfs_drop_group(&group->cg_item, item);
config_item_put(item);
}
@@ -1670,6 +1681,83 @@ void unregister_gadget_item(struct config_item *item)
}
EXPORT_SYMBOL_GPL(unregister_gadget_item);
+#ifdef CONFIG_TRACEPOINTS
+#define GROUP_LEN 128
+static int gadget_configfs_group(char *group, struct config_item *item)
+{
+ char *tmpgroup = kzalloc(GROUP_LEN, GFP_KERNEL);
+
+ if (!tmpgroup)
+ return -1;
+
+ for (;;) {
+ if (item->ci_type == &gadgets_type) {
+ kfree(tmpgroup);
+ return 0;
+ }
+
+ if (tmpgroup[0] == '\0')
+ snprintf(group, GROUP_LEN, "%s",
+ config_item_name(item));
+ else
+ snprintf(group, GROUP_LEN, "%s/%s",
+ config_item_name(item), tmpgroup);
+
+ item = item->ci_parent;
+ if (item->ci_type == &gadgets_type) {
+ kfree(tmpgroup);
+ return 0;
+ }
+
+ strcpy(tmpgroup, group);
+ }
+
+ return 0;
+}
+
+static void trace_usb_configfs_make_drop_group(struct config_item *pitem,
+ struct config_item *item, char *make_drop)
+{
+ char *group = kzalloc(2 * GROUP_LEN, GFP_KERNEL);
+ char *tmpgroup;
+ int ret;
+
+ if (!group)
+ goto out;
+
+ tmpgroup = group + GROUP_LEN;
+ ret = gadget_configfs_group(tmpgroup, pitem);
+ if (ret)
+ goto out;
+
+ if (tmpgroup[0] == '\0')
+ snprintf(group, GROUP_LEN, "%s %s", make_drop,
+ config_item_name(item));
+ else
+ snprintf(group, GROUP_LEN, "%s %s/%s", make_drop, tmpgroup,
+ config_item_name(item));
+
+ trace_gadget_configfs(group);
+
+out:
+ kfree(group);
+}
+
+void trace_usb_configfs_make_group(struct config_item *pitem,
+ struct config_item *item)
+{
+ trace_usb_configfs_make_drop_group(pitem, item, "mkdir");
+}
+EXPORT_SYMBOL(trace_usb_configfs_make_group);
+
+void trace_usb_configfs_drop_group(struct config_item *pitem,
+ struct config_item *item)
+{
+ trace_usb_configfs_make_drop_group(pitem, item, "rmdir");
+}
+EXPORT_SYMBOL(trace_usb_configfs_drop_group);
+#endif
+
static int __init gadget_cfs_init(void)
{
int ret;
diff --git a/drivers/usb/gadget/function/f_mass_storage.c b/drivers/usb/gadget/function/f_mass_storage.c
index ba899ca..8bf1f37 100644
--- a/drivers/usb/gadget/function/f_mass_storage.c
+++ b/drivers/usb/gadget/function/f_mass_storage.c
@@ -3288,6 +3288,8 @@ static struct config_group *fsg_lun_make(struct config_group *group,
config_group_init_type_name(&opts->group, name, &fsg_lun_type);
+ trace_usb_configfs_make_group(&group->cg_item, &opts->group.cg_item);
+
return &opts->group;
out:
mutex_unlock(&fsg_opts->lock);
@@ -3299,6 +3301,8 @@ static void fsg_lun_drop(struct config_group *group, struct config_item *item)
struct fsg_lun_opts *lun_opts;
struct fsg_opts *fsg_opts;
+ trace_usb_configfs_drop_group(&group->cg_item, item);
+
lun_opts = to_fsg_lun_opts(item);
fsg_opts = to_fsg_opts(&group->cg_item);
diff --git a/drivers/usb/gadget/function/uvc_configfs.c b/drivers/usb/gadget/function/uvc_configfs.c
index 77d6403..cc0f2eb 100644
--- a/drivers/usb/gadget/function/uvc_configfs.c
+++ b/drivers/usb/gadget/function/uvc_configfs.c
@@ -236,6 +236,8 @@ static struct config_item *uvcg_control_header_make(struct config_group *group,
config_item_init_type_name(&h->item, name, &uvcg_control_header_type);
+ trace_usb_configfs_make_group(&group->cg_item, &h->item);
+
return &h->item;
}
@@ -1039,6 +1041,8 @@ static struct config_item
config_item_init_type_name(&h->item, name, &uvcg_streaming_header_type);
+ trace_usb_configfs_make_group(&group->cg_item, &h->item);
+
return &h->item;
}
@@ -1380,6 +1384,8 @@ static struct config_item *uvcg_frame_make(struct config_group *group,
config_item_init_type_name(&h->item, name, &uvcg_frame_type);
+ trace_usb_configfs_make_group(&group->cg_item, &h->item);
+
return &h->item;
}
@@ -1389,6 +1395,8 @@ static void uvcg_frame_drop(struct config_group *group, struct config_item *item
struct f_uvc_opts *opts;
struct config_item *opts_item;
+ trace_usb_configfs_drop_group(&group->cg_item, item);
+
opts_item = group->cg_item.ci_parent->ci_parent->ci_parent;
opts = to_f_uvc_opts(opts_item);
@@ -1649,6 +1657,8 @@ static struct config_group *uvcg_uncompressed_make(struct config_group *group,
config_group_init_type_name(&h->fmt.group, name,
&uvcg_uncompressed_type);
+ trace_usb_configfs_make_group(&group->cg_item, &h->fmt.group.cg_item);
+
return &h->fmt.group;
}
@@ -1835,6 +1845,8 @@ static struct config_group *uvcg_mjpeg_make(struct config_group *group,
config_group_init_type_name(&h->fmt.group, name,
&uvcg_mjpeg_type);
+ trace_usb_configfs_make_group(&group->cg_item, &h->fmt.group.cg_item);
+
return &h->fmt.group;
}
diff --git a/include/linux/usb/composite.h b/include/linux/usb/composite.h
index 9d27622..611029d 100644
--- a/include/linux/usb/composite.h
+++ b/include/linux/usb/composite.h
@@ -603,6 +603,15 @@ void usb_put_function_instance(struct usb_function_instance *fi);
void usb_put_function(struct usb_function *f);
struct usb_function_instance *usb_get_function_instance(const char *name);
struct usb_function *usb_get_function(struct usb_function_instance *fi);
+#ifdef CONFIG_TRACEPOINTS
+void trace_usb_configfs_make_group(struct config_item *pitem,
+ struct config_item *item);
+void trace_usb_configfs_drop_group(struct config_item *pitem,
+ struct config_item *item);
+#else
+#define trace_usb_configfs_make_group(pitem, item) do {} while(0)
+#define trace_usb_configfs_drop_group(pitem, item) do {} while(0)
+#endif
struct usb_configuration *usb_get_config(struct usb_composite_dev *cdev,
int val);
diff --git a/include/linux/usb/gadget_configfs.h b/include/linux/usb/gadget_configfs.h
index d61aebd..a89f177 100644
--- a/include/linux/usb/gadget_configfs.h
+++ b/include/linux/usb/gadget_configfs.h
@@ -63,6 +63,7 @@ static struct config_item_type struct_in##_langid_type = { \
goto err; \
config_group_init_type_name(&new->group, name, \
&struct_in##_langid_type); \
+ trace_usb_configfs_make_group(&group->cg_item, &new->group.cg_item); \
\
gi = container_of(group, struct struct_member, strings_group); \
ret = -EEXIST; \
@@ -86,6 +87,7 @@ static void struct_in##_strings_drop( \
struct config_group *group, \
struct config_item *item) \
{ \
+ trace_usb_configfs_drop_group(&group->cg_item, item); \
config_item_put(item); \
} \
\
--
2.7.4
