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)
Wed, May 6, 10:40 PM
Unknown Object (File)
Wed, May 6, 11:49 AM
Unknown Object (File)
Sun, May 3, 3:51 AM
Unknown Object (File)
Sat, May 2, 7:37 PM
Unknown Object (File)
Apr 27 2026, 9:48 PM
Unknown Object (File)
Apr 25 2026, 12:34 PM
Unknown Object (File)
Apr 22 2026, 11:38 AM
Unknown Object (File)
Apr 19 2026, 7:40 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.