Page MenuHomeFreeBSD

linuxkpi: Use pageproc instead of vmproc
ClosedPublic

Authored by bwidawsk on Nov 21 2018, 3:52 AM.
Tags
None
Referenced Files
F112009394: D18061.diff
Tue, Mar 11, 12:47 PM
Unknown Object (File)
Feb 8 2025, 1:51 AM
Unknown Object (File)
Jan 24 2025, 3:23 PM
Unknown Object (File)
Jan 15 2025, 2:41 AM
Unknown Object (File)
Dec 12 2024, 5:27 PM
Unknown Object (File)
Oct 24 2024, 9:52 AM
Unknown Object (File)
Oct 5 2024, 2:20 PM
Unknown Object (File)
Oct 5 2024, 2:20 PM
Subscribers

Details

Summary

According to markj@:
pageproc contains the page daemon and laundry threads, which are
responsible for managing the LRU page queues and writing back dirty
pages. vmproc's main task is to swap out kernel stacks when the system
is under memory pressure, and swap them back in when necessary. It's a
somewhat legacy component of the system and isn't required. You can
build a kernel without it by specifying "options NO_SWAPPING" (which is
a somewhat misleading name), in which vm_swapout_dummy.c is compiled
instead of vm_swapout.c.

Based on this, we want pageproc to emulate kswapd, not vmproc.

Test Plan

N/A

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 20915
Build 20280: arc lint + arc unit

Event Timeline

This looks ok to me. In general I'd suggest considering having the definition in a .c file so that we're not forced to pollute the namespace with FreeBSD definitions. It's probably not going to be an issue here, but it's something that does come up from time to time.

sys/compat/linuxkpi/common/include/linux/swap.h
49

style(9) puts parens around return values.

This revision is now accepted and ready to land.Nov 21 2018, 4:06 AM
This revision was automatically updated to reflect the committed changes.