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)
Fri, Mar 27, 8:27 AM
Unknown Object (File)
Wed, Mar 25, 6:43 AM
Unknown Object (File)
Wed, Mar 25, 3:42 AM
Unknown Object (File)
Sat, Mar 21, 12:37 AM
Unknown Object (File)
Sat, Mar 21, 12:37 AM
Unknown Object (File)
Tue, Mar 17, 4:35 PM
Unknown Object (File)
Mon, Mar 16, 3:30 PM
Unknown Object (File)
Fri, Mar 13, 12:10 AM
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...