Page MenuHomeFreeBSD

vm_page: Track number of allocated NOFREE pages
AbandonedPublic

Authored by bnovkov on Oct 7 2024, 3:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jun 23, 6:27 AM
Unknown Object (File)
Fri, Jun 5, 10:47 PM
Unknown Object (File)
Mon, Jun 1, 9:30 PM
Unknown Object (File)
May 6 2026, 10:40 PM
Unknown Object (File)
May 6 2026, 11:49 AM
Unknown Object (File)
May 3 2026, 3:51 AM
Unknown Object (File)
May 2 2026, 7:37 PM
Unknown Object (File)
Apr 27 2026, 9:48 PM
Subscribers

Details

Reviewers
markj
alc
kib

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 59810
Build 56695: arc lint + arc unit

Event Timeline

It's more a matter of taste than anything I guess, but I'd merge this with the previous patch. They are small enough that they can be reviewed as one patch, and it feels a bit silly to have a change which introduces a counter but doesn't use it. I almost pointed that out in the other revision before I saw this one, since it looked like an unintentional omission.

sys/vm/vm_page.c
2596

This can be done outside the lock; counter(9) is implemented using per-CPU counters that do not require explicit synchronization.

It's more a matter of taste than anything I guess, but I'd merge this with the previous patch. They are small enough that they can be reviewed as one patch, and it feels a bit silly to have a change which introduces a counter but doesn't use it. I almost pointed that out in the other revision before I saw this one, since it looked like an unintentional omission.

Sure, I was on the fence about that from the beginning, I'll go ahead and merge it into the parent review.