Page MenuHomeFreeBSD
Differential D22186

Remove unnecessary object locking from the vnode pager.
ClosedPublic
Actions

Authored by jeff on Oct 30 2019, 2:39 AM.
Tags
None
Referenced Files
F154716963: D22186.id64595.diff
Wed, Apr 29, 7:23 AM
F154713357: D22186.id64595.diff
Wed, Apr 29, 7:14 AM
F154518223: D22186.id63781.diff
Tue, Apr 28, 7:43 PM
F154516004: D22186.id63781.diff
Tue, Apr 28, 7:32 PM
Unknown Object (File)
Mon, Apr 20, 3:04 PM
Unknown Object (File)
Sun, Apr 19, 4:45 PM
Unknown Object (File)
Sun, Apr 19, 10:03 AM
Unknown Object (File)
Sun, Apr 19, 6:49 AM
View All Files
Subscribers
bdragon
imp

Details

Reviewers
kib
alc
markj
dougm
Commits
rS354870: Remove unnecessary object locking from the vnode pager. Recent changes to
Summary

This is more lock pushdown as a result of the busy/valid/dirty changes.

Test Plan

pho

Diff Detail

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 27593
Build 25810: arc lint + arc unit

Event Timeline

jeff created this revision.Oct 30 2019, 2:39 AM
Harbormaster completed remote builds in B27269: Diff 63781.Oct 30 2019, 2:39 AM
jeff edited the summary of this revision. (Show Details)Oct 30 2019, 2:41 AM
jeff edited the test plan for this revision. (Show Details)
jeff added reviewers: kib, alc, markj, dougm.
kib added inline comments.Oct 30 2019, 4:46 PM
sys/vm/vnode_pager.c
1153

So the consistency of the operation against vnp_size is now depends on the vnode lock, right ? Note that ZFS still calls vnode_pager_setsize() with the vnode unlocked (I fixed this for NFS recently).

jeff added inline comments.Oct 30 2019, 9:25 PM
sys/vm/vnode_pager.c
1153

You are right. I left read locking around it elsewhere. I can restore that.

bdragon added a subscriber: bdragon.Oct 30 2019, 10:31 PM
jeff updated this revision to Diff 64474.Nov 17 2019, 1:21 AM

Readd a missing lock around the vnode size.

Harbormaster completed remote builds in B27593: Diff 64474.Nov 17 2019, 1:21 AM
markj accepted this revision.Nov 18 2019, 4:15 PM
This revision is now accepted and ready to land.Nov 18 2019, 4:15 PM
kib accepted this revision.Nov 19 2019, 3:15 PM
Closed by commit rS354870: Remove unnecessary object locking from the vnode pager. Recent changes to (authored by jeff). · Explain WhyNov 19 2019, 11:30 PM
This revision was automatically updated to reflect the committed changes.
jeff added a commit: rS354870: Remove unnecessary object locking from the vnode pager. Recent changes to.
Herald added a subscriber: imp. · View Herald TranscriptNov 19 2019, 11:30 PM

Revision Contents
Changeset List

PathSize
sys/
vm/
1 line
2 lines
31 lines

Diff 64474

View Options

sys/vm/vm_page.h

Loading...
View Options

sys/vm/vm_page.c

Loading...
View Options

sys/vm/vnode_pager.c

Loading...