Page MenuHomeFreeBSD

swap_pager: small improvement to find_least
ClosedPublic

Authored by dougm on Jun 11 2024, 7:38 AM.
Tags
None
Referenced Files
F116101521: D45558.diff
Fri, May 2, 1:38 PM
Unknown Object (File)
Fri, Apr 25, 12:28 AM
Unknown Object (File)
Mar 11 2025, 9:43 AM
Unknown Object (File)
Mar 3 2025, 11:50 PM
Unknown Object (File)
Feb 26 2025, 7:52 PM
Unknown Object (File)
Feb 26 2025, 6:15 PM
Unknown Object (File)
Feb 26 2025, 3:57 PM
Unknown Object (File)
Feb 26 2025, 3:23 AM
Subscribers

Details

Summary

This change drops an unneeded test, a branch and avoids a needless computation. Altogether, it saves 10 bytes of instructions on amd64.

It will mean that, on the path to MPASS(0), the same lookup and iteration will happen twice.

Test Plan

The kernel still boots.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dougm requested review of this revision.Jun 11 2024, 7:38 AM
dougm created this revision.
kib added inline comments.
sys/vm/swap_pager.c
2279
2312

Perhaps it is the time to change these two lines into __unreachable().

This revision is now accepted and ready to land.Jun 11 2024, 2:39 PM
This revision was automatically updated to reflect the committed changes.