Messages in this thread | | | Date | Fri, 26 Jul 2013 19:25:13 +0200 | From | Oleg Nesterov <> | Subject | [PATCH v2 0/6] tracing: open/delete fixes |
| |
Hello.
Testing: seems to work, but assumes that debugs was fixed too. Hopefully "debugfs: debugfs_remove_recursive() must not rely on list_empty(d_subdirs)" is fine.
(We should probably make debugfs_remove_recursive() to return the error and trace_events.c should warn if it fails)
However: I am still testing this patches, so this is still mostly for review. I'll report if I find anything, but to remind we have other bugs which need more fixes. And in fact I need to re-read this series to convince myself it should work, I was distracted by the unexpected problem in debugfs.
Changes: a lot but all cosmetic, addresses the comments from Steven.
Steven, 1/6 was not changed, I assume you agreed with my reply.
Masami, I'll appreciate your review ;)
Oleg.
kernel/trace/trace_events.c | 155 +++++++++++++++++++----------------- kernel/trace/trace_events_filter.c | 17 ++--- 2 files changed, 87 insertions(+), 85 deletions(-)
| |