Page MenuHomeFreeBSD
Differential D27939

fusefs: only check vnode locks with DEBUG_VFS_LOCKS
ClosedPublic
Actions

Authored by asomers on Jan 3 2021, 3:34 AM.
Tags
None
Referenced Files
F167214142: D27939.diff
Thu, Aug 20, 12:09 AM
Unknown Object (File)
Sun, Aug 9, 6:40 PM
Unknown Object (File)
Sun, Aug 9, 6:39 PM
Unknown Object (File)
Wed, Aug 5, 2:20 AM
Unknown Object (File)
Tue, Aug 4, 8:28 AM
Unknown Object (File)
Tue, Jul 28, 10:34 PM
Unknown Object (File)
Tue, Jul 21, 10:10 AM
Unknown Object (File)
Tue, Jul 21, 7:19 AM
View All Files
Subscribers
imp

Details

Reviewers
cem
Commits
rG34477e25c1d4: fusefs: only check vnode locks with DEBUG_VFS_LOCKS
Summary

MFC-With: 37df9d3bba8577fcdd63382ff5a4a5cbb4aa55b4

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 35903
Build 32792: arc lint + arc unit

Event Timeline

asomers created this revision.Jan 3 2021, 3:34 AM
Herald added a subscriber: imp. · View Herald TranscriptJan 3 2021, 3:34 AM
Harbormaster completed remote builds in B35903: Diff 81569.Jan 3 2021, 3:34 AM
asomers requested review of this revision.Jan 3 2021, 3:34 AM
cem accepted this revision.Jan 3 2021, 12:48 PM

The other nice thing about this routine (ok, conditional on debug.vfs_badlock_vnode=1, but it is the default) — or VNASSERT(), VPASS() — is that they'll print out details of the vp. MPASS can't do that.

This revision is now accepted and ready to land.Jan 3 2021, 12:48 PM
Closed by commit rG34477e25c1d4: fusefs: only check vnode locks with DEBUG_VFS_LOCKS (authored by asomers). · Explain WhyJan 3 2021, 4:20 PM
This revision was automatically updated to reflect the committed changes.
asomers added a commit: rG34477e25c1d4: fusefs: only check vnode locks with DEBUG_VFS_LOCKS.

Revision Contents
Changeset List

PathSize
sys/
fs/
fuse/
2 lines

Diff 81569

View Options

sys/fs/fuse/fuse_vnops.c

Loading...