Page MenuHomeFreeBSD

fusefs: fix an interaction between copy_file_range and mmap
ClosedPublic

Authored by asomers on Jan 13 2024, 10:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Feb 4, 11:13 PM
Unknown Object (File)
Thu, Jan 30, 2:47 AM
Unknown Object (File)
Thu, Jan 30, 12:46 AM
Unknown Object (File)
Wed, Jan 29, 5:19 AM
Unknown Object (File)
Sun, Jan 26, 7:55 PM
Unknown Object (File)
Sun, Jan 26, 6:07 PM
Unknown Object (File)
Sat, Jan 25, 3:04 PM
Unknown Object (File)
Fri, Jan 24, 11:29 PM
Subscribers

Details

Summary

If a copy_file_range operation tries to read from a page that was
previously written via mmap, that page must be flushed first.

MFC after: 2 weeks

Test Plan

two test cases added to the fusefs test suite

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sys/fs/fuse/fuse_vnops.c
912

Indent is strange.

I suggest to use (newly added) vnode_pager_clear_sync() instead, I do not think that possible optimization of flushing a range is significant there.

  • Use vnode_pager_clean_sync
This revision is now accepted and ready to land.Jan 15 2024, 9:53 PM