Page MenuHomeFreeBSD
Differential D22998

7/7 vfs: per-cpu batched requeuing of free vnodes
ClosedPublic
Actions

Authored by mjg on Jan 1 2020, 11:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jan 2, 9:24 PM
Unknown Object (File)
Thu, Jan 2, 12:07 PM
Unknown Object (File)
Dec 24 2024, 12:35 AM
Unknown Object (File)
Nov 22 2024, 9:16 PM
Unknown Object (File)
Nov 22 2024, 10:15 AM
Unknown Object (File)
Nov 5 2024, 4:11 AM
Unknown Object (File)
Oct 23 2024, 2:29 PM
Unknown Object (File)
Oct 23 2024, 2:29 PM
View All 51 Files
Subscribers
imp

Details

Reviewers
kib
jeff
Commits
rS356673: vfs: per-cpu batched requeuing of free vnodes
Summary

Constant requeuing adds significant lock contention in certain
workloads. Lessen the problem by batching this.

Per-cpu areas are locked in order to synchronize against UMA freeing
memory.

vnode's v_mflag is converted to short to prevent the struct from
growing.

Sample result from an incremental make -s -j 104 bzImage on tmpfs:
stock: 122.38s user 1780.45s system 6242% cpu 30.480 total
patched: 144.84s user 985.90s system 4856% cpu 23.282 total

Diff Detail

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

Event Timeline

mjg created this revision.Jan 1 2020, 11:00 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 1 2020, 11:00 PM
Harbormaster completed remote builds in B28426: Diff 66227.Jan 1 2020, 11:00 PM
mjg mentioned this in D22694: [wip] vfs: defer vdrop if mnt_listmtx is contended.Jan 1 2020, 11:05 PM
mjg updated this revision to Diff 66297.Jan 3 2020, 12:36 PM
  • rename vnode_defer to vdbatch
mjg updated this revision to Diff 66299.Jan 3 2020, 2:52 PM
  • consistnely use vdbatch* names everywhere
  • add more asserts
  • fix a counting bug reported by pho
mjg retitled this revision from 5/5 vfs: per-cpu batched requeuing of free vnodes to 7/7 vfs: per-cpu batched requeuing of free vnodes.Jan 5 2020, 4:07 AM
mjg updated this revision to Diff 66450.Jan 7 2020, 5:14 PM
  • rebase
jeff accepted this revision.Jan 12 2020, 7:59 PM
jeff added inline comments.
sys/kern/vfs_subr.c
299 ↗(On Diff #66450)

We are starting to repeat this pattern in multiple places. Might be worth making a new api eventually.

3261 ↗(On Diff #66450)

I would leave this pinned until vd is unlocked. It probably won't matter but it will potentially reduce contention.

This revision is now accepted and ready to land.Jan 12 2020, 7:59 PM
Closed by commit rS356673: vfs: per-cpu batched requeuing of free vnodes (authored by mjg). · Explain WhyJan 13 2020, 2:39 AM
This revision was automatically updated to reflect the committed changes.
mjg added a commit: rS356673: vfs: per-cpu batched requeuing of free vnodes.

Revision Contents
Changeset List

PathSize
head/
sys/
kern/
120 lines
sys/
3 lines

Diff 66670

View Options

head/sys/kern/vfs_subr.c

Loading...
View Options

head/sys/sys/vnode.h

Loading...