Page MenuHomeFreeBSD
Differential D38091

Improve debuggability of VOP_* locking assertions
ClosedPublic
Actions

Authored by jah on Jan 16 2023, 9:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, May 10, 7:20 AM
Unknown Object (File)
Tue, Apr 28, 11:24 PM
Unknown Object (File)
Tue, Apr 28, 3:51 PM
Unknown Object (File)
Tue, Apr 28, 11:44 AM
Unknown Object (File)
Mon, Apr 27, 11:06 AM
Unknown Object (File)
Sun, Apr 26, 12:53 AM
Unknown Object (File)
Fri, Apr 24, 12:36 PM
Unknown Object (File)
Thu, Apr 23, 2:12 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
markj
mjg
Commits
rG5a4a83fd0e67: Improve debuggability of VOP_* locking assertions
Summary

Include the phase and argument field to make it easier to determine
at a glance where the failure originated.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jah created this revision.Jan 16 2023, 9:41 PM
Herald added a subscriber: imp. · View Herald TranscriptJan 16 2023, 9:41 PM
jah requested review of this revision.Jan 16 2023, 9:41 PM
Harbormaster completed remote builds in B49136: Diff 115215.Jan 16 2023, 9:41 PM
kib added a comment.Jan 16 2023, 9:56 PM

Could you please show an example of the new output?

jah added a comment.Jan 16 2023, 10:03 PM
In D38091#865308, @kib wrote:

Could you please show an example of the new output?

This is what I saw a few hours ago when I used it to help me debug a unionfs issue:

VOP_MKDIR Error (dvp): vnode <...> is not exclusive locked but should be

kib accepted this revision.Jan 17 2023, 12:27 AM
This revision is now accepted and ready to land.Jan 17 2023, 12:27 AM
markj accepted this revision.Jan 18 2023, 3:52 PM
Closed by commit rG5a4a83fd0e67: Improve debuggability of VOP_* locking assertions (authored by jah). · Explain WhyJan 19 2023, 3:59 AM
This revision was automatically updated to reflect the committed changes.
jah added a commit: rG5a4a83fd0e67: Improve debuggability of VOP_* locking assertions.

Revision Contents
Changeset List

PathSize
sys/
tools/
8 lines

Diff 115298

View Options

sys/tools/vnode_if.awk

Loading...