Page MenuHomeFreeBSD
Differential D20485

Use the vm_page_wired() predicate where appropriate.
ClosedPublic
Actions

Authored by markj on May 31 2019, 8:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 4:00 AM
Unknown Object (File)
Nov 23 2024, 10:30 AM
Unknown Object (File)
Nov 23 2024, 7:21 AM
Unknown Object (File)
Nov 13 2024, 3:25 PM
Unknown Object (File)
Nov 8 2024, 3:12 AM
Unknown Object (File)
Oct 28 2024, 8:01 AM
Unknown Object (File)
Oct 24 2024, 5:03 AM
Unknown Object (File)
Sep 29 2024, 11:32 PM
View All 84 Files
Subscribers
imp

Details

Reviewers
alc
kib
jeff
Commits
rS348502: Add a vm_page_wired() predicate.
Summary

This is a follow-up for D19247. I would like to minimize naked accesses to wire_count
as much as possible, since D20486 makes use of several bits in that field to
store special references.

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 24650
Build 23427: arc lint + arc unit

Event Timeline

markj created this revision.May 31 2019, 8:45 PM
Harbormaster completed remote builds in B24634: Diff 58127.May 31 2019, 8:45 PM
markj added a parent revision: D19247: Merge hold_count into wire_count..May 31 2019, 9:20 PM
markj added a child revision: D20486: Change synchonization rules for page reference counting..
markj mentioned this in D19247: Merge hold_count into wire_count..May 31 2019, 9:25 PM
markj edited the summary of this revision. (Show Details)
markj added reviewers: alc, kib, jeff.
alc added a comment.Jun 1 2019, 4:21 AM

Where is vm_page_wired() defined?

markj added a comment.Jun 1 2019, 2:54 PM
In D20485#442354, @alc wrote:

Where is vm_page_wired() defined?

In D19247, I renamed it from vm_page_held().

alc accepted this revision.Jun 1 2019, 4:19 PM
In D20485#442381, @markj wrote:
In D20485#442354, @alc wrote:

Where is vm_page_wired() defined?

In D19247, I renamed it from vm_page_held().

Okay.

This revision is now accepted and ready to land.Jun 1 2019, 4:19 PM
kib added a comment.Jun 1 2019, 5:47 PM

Could you please update the diff to include the definition if vm_page_wired() ? I expect it to be trivial.

markj updated this revision to Diff 58162.Jun 1 2019, 9:43 PM

Add a vm_page_wired() definition and rebase on FreeBSD head
instead of D19247.

This revision now requires review to proceed.Jun 1 2019, 9:43 PM
Harbormaster completed remote builds in B24650: Diff 58162.Jun 1 2019, 9:43 PM
markj added a comment.Jun 1 2019, 9:43 PM
In D20485#442437, @kib wrote:

Could you please update the diff to include the definition if vm_page_wired() ? I expect it to be trivial.

I rebased the change so that it does not depend on the hold/wire count merge diff. Once this diff is committed, I will rebase that diff.

alc accepted this revision.Jun 1 2019, 9:59 PM
This revision is now accepted and ready to land.Jun 1 2019, 9:59 PM
kib accepted this revision.Jun 1 2019, 10:18 PM
Closed by commit rS348502: Add a vm_page_wired() predicate. (authored by markj). · Explain WhyJun 2 2019, 1:00 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: imp. · View Herald TranscriptJun 2 2019, 1:00 AM

Revision Contents
Changeset List

PathSize
sys/
vm/
4 lines
2 lines
6 lines
8 lines
7 lines
18 lines
6 lines

Diff 58162

View Options

sys/vm/memguard.c

Loading...
View Options

sys/vm/swap_pager.c

Loading...
View Options

sys/vm/vm_fault.c

Loading...
View Options

sys/vm/vm_object.c

Loading...
View Options

sys/vm/vm_page.h

Loading...
View Options

sys/vm/vm_page.c

Loading...
View Options

sys/vm/vm_pageout.c

Loading...