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
Unknown Object (File)
Sat, Jan 18, 3:31 PM
Unknown Object (File)
Wed, Dec 25, 4:06 PM
Unknown Object (File)
Sun, Dec 22, 8:27 PM
Unknown Object (File)
Dec 4 2024, 5:50 AM
Unknown Object (File)
Oct 11 2024, 7:47 PM
Unknown Object (File)
Sep 30 2024, 4:13 PM
Unknown Object (File)
Sep 23 2024, 7:42 PM
Unknown Object (File)
Sep 22 2024, 5:52 PM
View All 63 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 27269
Build 25533: 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
1151

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
1151

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
28 lines

Diff 63781

View Options

sys/vm/vm_page.h

Loading...
View Options

sys/vm/vm_page.c

Loading...
View Options

sys/vm/vnode_pager.c

Loading...