Skip to content

fuse: invalidate page cache after sync and async direct writes (for redfs-ubuntu-noble-6.8.0-58.60)#127

Merged
hbirth merged 1 commit intoDDNStorage:redfs-ubuntu-noble-6.8.0-58.60from
cding-ddn:fix-xfstest-451-redfs-ubuntu-noble-6.8.0-58.60
Apr 15, 2026
Merged

fuse: invalidate page cache after sync and async direct writes (for redfs-ubuntu-noble-6.8.0-58.60)#127
hbirth merged 1 commit intoDDNStorage:redfs-ubuntu-noble-6.8.0-58.60from
cding-ddn:fix-xfstest-451-redfs-ubuntu-noble-6.8.0-58.60

Conversation

@cding-ddn
Copy link
Copy Markdown
Collaborator

Fixes xfstests generic/451, similar to how commit b359af8 ("fuse: Invalidate the page cache after FOPEN_DIRECT_IO write") fixes xfstests generic/209.

Fixes xfstests generic/451, similar to how commit b359af8 ("fuse:
Invalidate the page cache after FOPEN_DIRECT_IO write") fixes xfstests
generic/209.

Signed-off-by: Cheng Ding <cding@ddn.com>
@cding-ddn cding-ddn requested review from bsbernd and hbirth March 16, 2026 09:14
@hbirth
Copy link
Copy Markdown
Collaborator

hbirth commented Mar 20, 2026

LGTM, but have we checked how much this affects performance?

@hbirth
Copy link
Copy Markdown
Collaborator

hbirth commented Apr 14, 2026

@cding-ddn do we still need this?

@cding-ddn
Copy link
Copy Markdown
Collaborator Author

@cding-ddn do we still need this?

Sorry for my slow response. We still need this. About the performance, this code works only when aio and buffered io mixed workload, the upstream kernel always has Bernd's patch which does similar thing for dio and buffered io mixed case.
See
https://lore.kernel.org/linux-fsdevel/20260303-async-dio-aio-cache-invalidation-v1-1-fba0fd0426c3@ddn.com/
https://lore.kernel.org/linux-fsdevel/20260309-xfstests-generic-451-v3-1-bb6ad2f59512@ddn.com/

@hbirth hbirth merged commit 51e0799 into DDNStorage:redfs-ubuntu-noble-6.8.0-58.60 Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants