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
Unknown Object (File)
Thu, May 14, 4:52 AM
Unknown Object (File)
Thu, May 14, 4:07 AM
Unknown Object (File)
Sun, May 3, 5:38 AM
Unknown Object (File)
Thu, Apr 30, 11:39 PM
Unknown Object (File)
Thu, Apr 30, 10:01 AM
Unknown Object (File)
Mon, Apr 27, 6:32 AM
Unknown Object (File)
Sun, Apr 26, 6:46 PM
Unknown Object (File)
Sun, Apr 26, 5:54 PM
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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 81586

View Options

sys/fs/fuse/fuse_vnops.c

Loading...