Page MenuHomeFreeBSD

Use unlocked page lookup for inmem() to avoid object lock contention
ClosedPublic

Authored by mlaier on Oct 3 2020, 6:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 31, 3:50 AM
Unknown Object (File)
Nov 3 2024, 4:04 PM
Unknown Object (File)
Oct 30 2024, 10:50 AM
Unknown Object (File)
Oct 2 2024, 12:28 PM
Unknown Object (File)
Sep 30 2024, 4:30 PM
Unknown Object (File)
Sep 30 2024, 10:34 AM
Unknown Object (File)
Sep 28 2024, 11:39 AM
Unknown Object (File)
Sep 23 2024, 8:09 PM
Subscribers

Details

Summary

Let's try this again:

  • make a vm_page_lookup_unlocked
  • really bool-ify
  • while here, export inmem alongside incore (we have a local diff for this)

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mlaier requested review of this revision.Oct 3 2020, 6:57 PM
mlaier created this revision.

Moving the tinc calculation back up - needed for valid range

markj added inline comments.
sys/vm/vm_page.c
1701 ↗(On Diff #77910)

I think this deserves a comment to match the others. e.g., the description for vm_page_lookup(), plus, "The page may be no longer be present in the object at the time that this function returns."

This revision is now accepted and ready to land.Oct 6 2020, 12:47 PM

Add comment per markj's request.

This revision now requires review to proceed.Oct 8 2020, 10:14 PM
This revision is now accepted and ready to land.Oct 8 2020, 10:48 PM