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
F154168392: D27939.diff
Sun, Apr 26, 6:46 PM
F154163897: D27939.id.diff
Sun, Apr 26, 5:54 PM
F154130884: D27939.id81569.diff
Sun, Apr 26, 10:21 AM
Unknown Object (File)
Sat, Apr 25, 5:28 AM
Unknown Object (File)
Fri, Apr 24, 2:40 PM
Unknown Object (File)
Mon, Apr 20, 3:08 AM
Unknown Object (File)
Mon, Apr 6, 12:19 PM
Unknown Object (File)
Sun, Apr 5, 11:08 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
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...