Skip to content

eventfs: Linus's updates for 6.7

Steven Rostedt lkml oe-kbuild-all stable 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest23 patches 1 replyIn review
  2. v123 patches 2 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/23eventfs: Remove "lookup" parameter from create_dir/file_dentry()fs/tracefs/event_inode.c+20 -350 replies
2/23eventfs: Stop using dcache_readdir() for getdents()fs/tracefs/event_inode.c+64 -1300 replies
3/23tracefs/eventfs: Use root and instance inodes as default ownershipfs/tracefs/event_inode.c fs/tracefs/inode.c fs/tracefs/internal.h+190 -900 replies
4/23eventfs: Have eventfs_iterate() stop immediately if ei->is_freed is setfs/tracefs/event_inode.c+6 -50 replies
5/23eventfs: Do ctx->pos update for all iterations in eventfs_iterate()fs/tracefs/event_inode.c+14 -70 replies
6/23eventfs: Read ei->entries before ei->children in eventfs_iterate()fs/tracefs/event_inode.c+23 -230 replies
7/23eventfs: Shortcut eventfs_iterate() by skipping entries already readfs/tracefs/event_inode.c+10 -130 replies
8/23eventfs: Have the inodes all for files and directories all be the samefs/tracefs/event_inode.c+10 -00 replies
9/23eventfs: Do not create dentries nor inodes in iterate_sharedfs/tracefs/event_inode.c+5 -150 replies
10/23eventfs: Use kcalloc() instead of kzalloc()fs/tracefs/event_inode.c+3 -30 replies
11/23eventfs: Save directory inodes in the eventfs_inode structurefs/tracefs/event_inode.c fs/tracefs/internal.h+15 -60 replies
12/23tracefs: remove stale update_gid codefs/tracefs/event_inode.c fs/tracefs/internal.h+0 -390 replies
13/23tracefs: Zero out the tracefs_inode when allocating itfs/tracefs/inode.c fs/tracefs/internal.h+6 -30 replies
14/23eventfs: Initialize the tracefs inode properlyfs/tracefs/event_inode.c+2 -40 replies
15/23tracefs: Avoid using the ei->dentry pointer unnecessarilyfs/tracefs/event_inode.c+12 -140 replies
16/23tracefs: dentry lookup crapectomyfs/tracefs/event_inode.c fs/tracefs/inode.c fs/tracefs/internal.h+50 -2970 replies
17/23eventfs: Remove unused d_parent pointer fieldfs/tracefs/event_inode.c fs/tracefs/internal.h+1 -50 replies
18/23eventfs: Clean up dentry ops and add revalidate functionfs/tracefs/event_inode.c fs/tracefs/inode.c fs/tracefs/internal.h+22 -130 replies
19/23eventfs: Get rid of dentry pointers without refcountsfs/tracefs/event_inode.c fs/tracefs/internal.h+78 -1770 replies
20/23eventfs: Warn if an eventfs_inode is freed without is_freed being setfs/tracefs/event_inode.c+14 -40 replies
21/23eventfs: Restructure eventfs_inode structure to be more condensedfs/tracefs/internal.h+12 -150 replies
22/23eventfs: Remove fsnotify*() functions from lookup()fs/tracefs/event_inode.c+0 -20 replies
23/23eventfs: Keep all directory links at 1fs/tracefs/event_inode.c+10 -40 replies

Reviews trailers per patch

2 of 23 patches have a Reviewed-by or Acked-by.

PatchGustavo A. R. SilvaGeert UytterhoevenKees Cook
1/23
2/23
3/23
4/23
5/23
6/23
7/23
8/23
9/23
10/23R-b
11/23T-b R-b
12/23
13/23
14/23
15/23
16/23
17/23
18/23
19/23
20/23
21/23
22/23
23/23

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v2 thread

Cover letter [v6.7][PATCH v2 00/23] eventfs: Linus's updates for 6.7

This is a backport of all the work that lead up to the work that Linus made on eventfs. I trust Linus's version more so than the versions in 6.6 and 6.7. There may be plenty of hidden issues due to the design.

This is the update for 6.7. It includes Linus's updates as well as all the patches leading up to them.

I ran these through my full test suite that I use before sending anyting to Linus, althouh I did not run my "bisect" test that walks through the patches. The tests were just run on the end result. I'm currently running my 6.6 version through my tests.

This was created with the following command against v6.7.3:

git log --reverse --no-merges --pretty=oneline v6.7..origin/master fs/tracefs/ | cut -d' ' -f1 |
while read a; do if ! git cherry-pick -x $a; then break; fi ; done

Which adds -x to the cherry pick to add the upstream commit SHAs.

Erick Archer (1):
eventfs: Use kcalloc() instead of kzalloc()

Linus Torvalds (7):
tracefs: remove stale 'update_gid' code
eventfs: Initialize the tracefs inode properly
tracefs: Avoid using the ei->dentry pointer unnecessarily
tracefs: dentry lookup crapectomy
eventfs: Remove unused d_parent pointer field
eventfs: Clean up dentry ops and add revalidate function
eventfs: Get rid of dentry pointers without refcounts

Steven Rostedt (Google) (15):
eventfs: Remove "lookup" parameter from create_dir/file_dentry()
eventfs: Stop using dcache_readdir() for getdents() tracefs/eventfs: Use root and instance inodes as default ownership
eventfs: Have eventfs_iterate() stop immediately if ei->is_freed is set
eventfs: Do ctx->pos update for all iterations in eventfs_iterate()
eventfs: Read ei->entries before ei->children in eventfs_iterate()
eventfs: Shortcut eventfs_iterate() by skipping entries already read
eventfs: Have the inodes all for files and directories all be the same
eventfs: Do not create dentries nor inodes in iterate_shared
eventfs: Save directory inodes in the eventfs_inode structure
tracefs: Zero out the tracefs_inode when allocating it
eventfs: Warn if an eventfs_inode is freed without is_freed being set
eventfs: Restructure eventfs_inode structure to be more condensed
eventfs: Remove fsnotify*() functions from lookup()
eventfs: Keep all directory links at 1

----

 fs/tracefs/event_inode.c | 905 ++++++++++++++++-------------------------------
 fs/tracefs/inode.c       | 286 +++++++--------
 fs/tracefs/internal.h    |  48 ++-
 3 files changed, 451 insertions(+), 788 deletions(-)