Page MenuHomeFreeBSD

dumpfs(8): add option to only print superblock information
ClosedPublic

Authored by rew on Jun 24 2021, 3:20 AM.
Tags
None
Referenced Files
F108481272: D30881.id91300.diff
Sat, Jan 25, 9:22 AM
F108441009: D30881.id91300.diff
Fri, Jan 24, 7:17 PM
Unknown Object (File)
Sat, Jan 18, 5:07 PM
Unknown Object (File)
Dec 4 2024, 11:22 PM
Unknown Object (File)
Nov 20 2024, 6:11 PM
Unknown Object (File)
Nov 11 2024, 11:16 PM
Unknown Object (File)
Oct 23 2024, 12:48 AM
Unknown Object (File)
Oct 22 2024, 1:36 AM
Subscribers

Details

Summary

Add an option to dumpfs, -s, that only prints the super block information.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 40071
Build 36960: arc lint + arc unit

Event Timeline

rew requested review of this revision.Jun 24 2021, 3:20 AM

This looks good. And I rather like it, since I've wanted something like this in the past.

Strictly speaking, dosb should be a bool, but that's a bigger change because dofreespace, domarshal, and dolabel are not yet bools and should before consistency. And that's well beyond the scope, unless you really wanted to follow up with that. :)

This revision is now accepted and ready to land.Jun 24 2021, 3:31 AM
bcr added a subscriber: bcr.

OK from manpages, too. Thanks for addition this feature.

In D30881#694691, @imp wrote:

Strictly speaking, dosb should be a bool, but that's a bigger change because dofreespace, domarshal, and dolabel are not yet bools and should before consistency. And that's well beyond the scope, unless you really wanted to follow up with that. :)

Would you rather see a follow-up done as part of this review or in a separate one?