Page MenuHomeFreeBSD

tmpfs: Move partial page invalidation to a separate helper
ClosedPublic

Authored by khng on Aug 25 2021, 8:13 PM.
Tags
None
Referenced Files
F106113599: D31683.id94193.diff
Wed, Dec 25, 3:02 PM
F106113121: D31683.id94188.diff
Wed, Dec 25, 2:50 PM
F106085328: D31683.diff
Wed, Dec 25, 3:51 AM
Unknown Object (File)
Wed, Dec 11, 8:26 PM
Unknown Object (File)
Mon, Dec 9, 1:44 PM
Unknown Object (File)
Nov 18 2024, 5:42 PM
Unknown Object (File)
Nov 18 2024, 3:30 PM
Unknown Object (File)
Oct 27 2024, 7:28 PM
Subscribers

Details

Summary

The partial page invalidation code is factored out to be a separate
helper from tmpfs_reg_resize().

Sponsored by: The FreeBSD Foundation

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 41210
Build 38099: arc lint + arc unit

Event Timeline

khng requested review of this revision.Aug 25 2021, 8:13 PM
sys/fs/tmpfs/tmpfs_subr.c
404

Replace the whole 'else' branch with something along the lines:

vm_page_free(m);
m = NULL;
if (!ignerr)
    error = EIO;

and change return (0); at the end with return (error);

1755

By convention 'rv' var holds a Mach error code, not errno. Since tmpfs_partial_page_invalidate() returns errno, better rename rv there.

khng marked 2 inline comments as done.

Cleanups suggested by kib@

This revision is now accepted and ready to land.Aug 25 2021, 8:50 PM