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, Aug 25, 9:09 AM
Unknown Object (File)
Sun, Aug 23, 10:20 PM
Unknown Object (File)
Mon, Aug 17, 4:13 PM
Unknown Object (File)
Mon, Aug 17, 3:52 PM
Unknown Object (File)
Thu, Aug 13, 12:22 PM
Unknown Object (File)
Aug 5 2026, 8:37 AM
Unknown Object (File)
Jun 23 2026, 6:27 AM
Unknown Object (File)
Jun 5 2026, 10:47 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.