Page MenuHomeFreeBSD
Differential D51696

ufs: Remove uses of DEBUG_VFS_LOCKS
ClosedPublic
Actions

Authored by markj on Aug 2 2025, 1:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 11, 3:23 AM
Unknown Object (File)
Tue, Sep 8, 1:39 PM
Unknown Object (File)
Sun, Sep 6, 5:04 AM
Unknown Object (File)
Sun, Sep 6, 3:25 AM
Unknown Object (File)
Sun, Sep 6, 1:19 AM
Unknown Object (File)
Sun, Sep 6, 12:56 AM
Unknown Object (File)
Sat, Sep 5, 6:25 PM
Unknown Object (File)
Sat, Sep 5, 1:29 PM
View All Files
Subscribers
imp

Details

Reviewers
kib
imp
olce
Commits
rGd033848a5e4c: ufs: Remove uses of DEBUG_VFS_LOCKS
rG4798f1e8f28d: ufs: Remove uses of DEBUG_VFS_LOCKS
Summary

All of these assertions can reasonably be checked when plain INVARIANTS
is configured, there's no need to configure a separate option.

Diff Detail

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

Event Timeline

markj created this revision.Aug 2 2025, 1:14 PM
Herald added a subscriber: imp. · View Herald TranscriptAug 2 2025, 1:14 PM
markj requested review of this revision.Aug 2 2025, 1:14 PM
Harbormaster completed remote builds in B65917: Diff 159624.Aug 2 2025, 1:14 PM
markj added a parent revision: D51402: vfs: Move DEBUG_VFS_LOCKS checks to INVARIANTS.Aug 2 2025, 1:14 PM
markj added a child revision: D51697: tmpfs: Remove uses of DEBUG_VFS_LOCKS.
kib added a comment.Aug 2 2025, 2:07 PM

O think #ifdef's lines could be removed for the whole patch.

sys/ufs/ffs/ffs_softdep.c
14524–14525

I think the braces are not needed.

sys/ufs/ffs/ffs_vnops.c
510–511

Same there.

imp accepted this revision.Aug 2 2025, 6:02 PM
This revision is now accepted and ready to land.Aug 2 2025, 6:02 PM
markj updated this revision to Diff 159685.Aug 3 2025, 3:58 PM
markj marked 2 inline comments as done.

Simply remove the ifdefs.

This revision now requires review to proceed.Aug 3 2025, 3:58 PM
Harbormaster completed remote builds in B65943: Diff 159685.Aug 3 2025, 3:58 PM
olce accepted this revision.Aug 3 2025, 4:47 PM
This revision is now accepted and ready to land.Aug 3 2025, 4:47 PM
kib accepted this revision.Aug 3 2025, 6:06 PM
Closed by commit rG4798f1e8f28d: ufs: Remove uses of DEBUG_VFS_LOCKS (authored by markj). · Explain WhyAug 3 2025, 10:16 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG4798f1e8f28d: ufs: Remove uses of DEBUG_VFS_LOCKS.
markj added a commit: rGd033848a5e4c: ufs: Remove uses of DEBUG_VFS_LOCKS.Aug 25 2025, 3:23 PM

Revision Contents
Changeset List

PathSize
sys/
ufs/
ffs/
2 lines
2 lines
ufs/
2 lines
4 lines

Diff 159694

View Options

sys/ufs/ffs/ffs_softdep.c

Loading...
View Options

sys/ufs/ffs/ffs_vnops.c

Loading...
View Options

sys/ufs/ufs/ufs_lookup.c

Loading...
View Options

sys/ufs/ufs/ufs_quota.c

Loading...