Thread (34 messages) 34 messages, 3 authors, 2023-07-21

Re: [PATCH v4 00/10] tracing: introducing eventfs

From: Steven Rostedt <rostedt@goodmis.org>
Date: 2023-07-21 19:14:21
Also in: linux-kselftest, lkml, oe-lkp

On Fri, 21 Jul 2023 09:18:43 -0400
Steven Rostedt [off-list ref] wrote:
OK, I got this working and it appears to pass all my tests. I actually got
it working Wednesday night, but I tried a different approach on Thursday
that got rid of the evenfs_file and only used eventfs_inodes and makes the
files more dynamic. There's still a couple of corner cases that are not
working with this approach (the dentry counters are getting out of sync).
This should not stop this from going in. I'll continue working on that
approach for the next merge cycle. But for now, here's the patch to this
series that works.
Just figured out my bug with my new design. It was in the eventfs_release()
code, where I have a loop that does the dput on the children, but I wasn't
dput(child), I was dput(parent) in that loop!!

Anyway, for this merge window I much rather have this code in, and for the
next merge window I'll add this update.

I can post the new design too, but first let's focus on this.

-- Steve
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help