Skip to content

eventfs: More updates to eventfs_iterate()

Steven Rostedt linux-fsdevel linux-trace-kernel lkml 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4eventfs: Have eventfs_iterate() stop immediately if ei->is_freed is setfs/tracefs/event_inode.c+6 -50 replies
2/4eventfs: Do ctx->pos update for all iterations in eventfs_iterate()fs/tracefs/event_inode.c+14 -70 replies
3/4eventfs: Read ei->entries before ei->children in eventfs_iterate()fs/tracefs/event_inode.c+23 -230 replies
4/4eventfs: Shortcut eventfs_iterate() by skipping entries already readfs/tracefs/event_inode.c+10 -130 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/4] eventfs: More updates to eventfs_iterate()

With the ongoing descussion of eventfs iterator, a few more changes are required and some changes are just enhancements.

- Stop immediately in the loop if the ei is found to be in the process of being freed.

- Make the ctx->pos update consistent with the skipped previous read index. This fixes a bug with duplicate files being showned by 'ls'.

- Swap reading ei->entries with ei->children to make the next change easier to read

- Add a "shortcut" in the ei->entries array to skip over already read entries.

Steven Rostedt (Google) (4):

      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

----

 fs/tracefs/event_inode.c | 67 ++++++++++++++++++++++++++----------------------
 1 file changed, 36 insertions(+), 31 deletions(-)