tracing/events: Only define remove_subsystem_dir() if CONFIG_MODULES
If we disable modules, we get the following warning in ftrace events file: kernel/trace/trace_events.c:912: attention : ‘remove_subsystem_dir’ defined but not used remove_subystem_dir() is useless if !CONFIG_MODULES, then move it to the appropriate #ifdef section of trace_events.c Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com> Cc: Steven Rostedt <rostedt@goodmis.org>
This commit is contained in:
parent
1a0799a8fe
commit
a2ca5e03b6
|
@ -908,32 +908,6 @@ event_subsystem_dir(const char *name, struct dentry *d_events)
|
||||||
return system->entry;
|
return system->entry;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void remove_subsystem_dir(const char *name)
|
|
||||||
{
|
|
||||||
struct event_subsystem *system;
|
|
||||||
|
|
||||||
if (strcmp(name, TRACE_SYSTEM) == 0)
|
|
||||||
return;
|
|
||||||
|
|
||||||
list_for_each_entry(system, &event_subsystems, list) {
|
|
||||||
if (strcmp(system->name, name) == 0) {
|
|
||||||
if (!--system->nr_events) {
|
|
||||||
struct event_filter *filter = system->filter;
|
|
||||||
|
|
||||||
debugfs_remove_recursive(system->entry);
|
|
||||||
list_del(&system->list);
|
|
||||||
if (filter) {
|
|
||||||
kfree(filter->filter_string);
|
|
||||||
kfree(filter);
|
|
||||||
}
|
|
||||||
kfree(system->name);
|
|
||||||
kfree(system);
|
|
||||||
}
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
static int
|
static int
|
||||||
event_create_dir(struct ftrace_event_call *call, struct dentry *d_events,
|
event_create_dir(struct ftrace_event_call *call, struct dentry *d_events,
|
||||||
const struct file_operations *id,
|
const struct file_operations *id,
|
||||||
|
@ -1018,6 +992,32 @@ struct ftrace_module_file_ops {
|
||||||
struct file_operations filter;
|
struct file_operations filter;
|
||||||
};
|
};
|
||||||
|
|
||||||
|
static void remove_subsystem_dir(const char *name)
|
||||||
|
{
|
||||||
|
struct event_subsystem *system;
|
||||||
|
|
||||||
|
if (strcmp(name, TRACE_SYSTEM) == 0)
|
||||||
|
return;
|
||||||
|
|
||||||
|
list_for_each_entry(system, &event_subsystems, list) {
|
||||||
|
if (strcmp(system->name, name) == 0) {
|
||||||
|
if (!--system->nr_events) {
|
||||||
|
struct event_filter *filter = system->filter;
|
||||||
|
|
||||||
|
debugfs_remove_recursive(system->entry);
|
||||||
|
list_del(&system->list);
|
||||||
|
if (filter) {
|
||||||
|
kfree(filter->filter_string);
|
||||||
|
kfree(filter);
|
||||||
|
}
|
||||||
|
kfree(system->name);
|
||||||
|
kfree(system);
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
static struct ftrace_module_file_ops *
|
static struct ftrace_module_file_ops *
|
||||||
trace_create_file_ops(struct module *mod)
|
trace_create_file_ops(struct module *mod)
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in New Issue