Page MenuHomeFreeBSD
Differential D1672

per-cpu page cache
AbandonedPublic
Actions

Authored by cem on Jan 25 2015, 10:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Mar 19, 1:12 AM
Unknown Object (File)
Wed, Mar 18, 5:23 AM
Unknown Object (File)
Wed, Mar 18, 5:22 AM
Unknown Object (File)
Wed, Mar 18, 12:08 AM
Unknown Object (File)
Tue, Mar 17, 10:10 AM
Unknown Object (File)
Tue, Mar 17, 8:09 AM
Unknown Object (File)
Sun, Mar 8, 3:31 AM
Unknown Object (File)
Fri, Feb 27, 7:05 PM
View All Files
Subscribers
cem
emaste
imp

Details

Reviewers
kmacy
alc
adrian
Summary

Integrate from OneFS on behalf of Netflix:

  • Add per-cpu page caches to avoid contention on the page queue free mutex or large objects
  • Add the option of passing PQ_NONE to vm_page_unwire for callers that are going to subsequently free the page (making the add to the inactive queue simply unneeded overhead)
    • bump WITNESS_PENDLIST to avoid panic on boot
Test Plan

stress2, buildworld, netflix deployment

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

kmacy updated this revision to Diff 3433.Jan 25 2015, 10:42 PM
kmacy retitled this revision from to per-cpu page cache.
kmacy updated this object.
kmacy edited the test plan for this revision. (Show Details)
kmacy added a reviewer: alc.
emaste added a subscriber: emaste.Jan 26 2015, 1:13 AM
cse_cem_gmail_com added a reviewer: adrian.Jul 17 2015, 4:12 PM
cse_cem_gmail_com added a subscriber: cse_cem_gmail_com.

What can I do to help this along?

cse_cem_gmail_com removed a subscriber: cse_cem_gmail_com.Jul 17 2015, 4:13 PM
cem added a subscriber: cem.Jul 17 2015, 4:13 PM

Oops, wrong account.

adrian edited edge metadata.Jul 17 2015, 4:38 PM

Update it to the latest -HEAD? I have a whole bunch of 2 and 4 socket NUMA boxes I can test this out on.

Thanks!

-a

cem commandeered this revision.Jul 17 2015, 6:29 PM
cem added a reviewer: kmacy.

Sure. It applied cleanly against HEAD for me; took me a while to get a build (10.1 can't build CURRENT today). It boots, seems to run ok in a VM.

cem updated this revision to Diff 7055.Jul 17 2015, 6:29 PM
cem edited edge metadata.

Update to latest HEAD.

Herald added a subscriber: imp. · View Herald TranscriptJul 17 2015, 6:29 PM
adrian added a comment.Jul 19 2015, 1:31 AM

ok, so this adds a per-cpu free list /and/ disables the page cache piece. What's the effect going to be, besides reducing lock contention?

cem updated this object.Jul 19 2015, 1:43 AM
cem abandoned this revision.Dec 22 2016, 11:59 PM

Revision Contents
Changeset List

PathSize
sys/
kern/
2 lines
vm/
187 lines

Diff 7055

View Options

sys/kern/subr_witness.c

Loading...
View Options

sys/vm/vm_page.c

Loading...