Page MenuHomeFreeBSD
Differential D54051

sys: Remove/update references to the swapper process in various comments
ClosedPublic
Actions

Authored by jhb on Dec 3 2025, 4:33 PM.
Tags
None
Referenced Files
F163284588: D54051.id167553.diff
Tue, Jul 21, 5:51 PM
Unknown Object (File)
Sun, Jul 19, 1:53 PM
Unknown Object (File)
Wed, Jul 15, 6:57 AM
Unknown Object (File)
Wed, Jul 15, 6:57 AM
Unknown Object (File)
Sat, Jul 11, 6:10 PM
Unknown Object (File)
Thu, Jul 9, 11:57 PM
Unknown Object (File)
Thu, Jun 25, 9:04 PM
Unknown Object (File)
Mon, Jun 22, 11:20 PM
View All 46 Files
Subscribers
imp
olce

Details

Reviewers
markj
andrew
olce
Commits
rG6cafc75b3930: sys: Remove/update references to the swapper process in various comments
rGf6910b09a230: sys: Remove/update references to the swapper process in various comments

Diff Detail

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

Event Timeline

jhb created this revision.Dec 3 2025, 4:33 PM
Herald added a reviewer: andrew. · View Herald TranscriptDec 3 2025, 4:33 PM
Herald added a subscriber: imp. · View Herald Transcript
jhb requested review of this revision.Dec 3 2025, 4:33 PM
Harbormaster completed remote builds in B69015: Diff 167479.Dec 3 2025, 4:33 PM
jhb added a child revision: D54052: MAC: Rename mac_cred_create_swapper to mac_cred_create_kproc0.Dec 3 2025, 4:33 PM
jhb added a parent revision: D54039: thread0: Stop calling thread0 "swapper".Dec 3 2025, 4:33 PM
olce accepted this revision.Dec 3 2025, 4:58 PM
olce added a subscriber: olce.

There is an occurrence of "swapper" in sys/vm/vnode_pager.c that seems to also point to the swapper process.

This revision is now accepted and ready to land.Dec 3 2025, 4:58 PM
markj accepted this revision.Dec 3 2025, 5:10 PM
jhb added a comment.Dec 4 2025, 3:46 PM
In D54051#1234938, @olce wrote:

There is an occurrence of "swapper" in sys/vm/vnode_pager.c that seems to also point to the swapper process.

I think that might be a reference to the swap pager as swap_pager_putpages() can do asynchronous I/O. I probably should fix that comment though to reference the swap pager now that I've looked at it in more detail though. There are a couple of "swapper" mentions in swap_pager.c that are clearly about the swap pager, as well, but they are not as trivial to rewrite.

Closed by commit rGf6910b09a230: sys: Remove/update references to the swapper process in various comments (authored by jhb). · Explain WhyDec 4 2025, 7:53 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGf6910b09a230: sys: Remove/update references to the swapper process in various comments.
jhb added a commit: rG6cafc75b3930: sys: Remove/update references to the swapper process in various comments.Dec 19 2025, 3:32 PM

Revision Contents
Changeset List

PathSize
sys/
arm/
arm/
2 lines
i386/
i386/
2 lines
sys/
2 lines
5 lines
vm/
2 lines

Diff 167553

View Options

sys/arm/arm/pmap-v6.c

Loading...
View Options

sys/i386/i386/pmap.c

Loading...
View Options

sys/sys/proc.h

Loading...
View Options

sys/sys/sleepqueue.h

Loading...
View Options

sys/vm/vnode_pager.c

Loading...