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)
Sat, Nov 29, 3:06 AM
Unknown Object (File)
Thu, Nov 27, 3:08 PM
Unknown Object (File)
Wed, Nov 26, 3:53 AM
Unknown Object (File)
Tue, Nov 25, 5:37 AM
Unknown Object (File)
Mon, Nov 24, 8:12 PM
Unknown Object (File)
Fri, Nov 21, 3:36 AM
Unknown Object (File)
Thu, Nov 20, 5:22 PM
Unknown Object (File)
Wed, Nov 19, 12:24 PM
View All 96 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...