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)
Sun, Feb 9, 10:00 PM
Unknown Object (File)
Mon, Jan 13, 7:44 AM
Unknown Object (File)
Jan 12 2025, 4:53 PM
Unknown Object (File)
Nov 13 2024, 10:27 AM
Unknown Object (File)
Nov 10 2024, 6:00 PM
Unknown Object (File)
Nov 2 2024, 10:56 AM
Unknown Object (File)
Oct 22 2024, 12:40 PM
Unknown Object (File)
Oct 13 2024, 1:42 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.