Add "kldstat -h"; showing module sizes in hex is rather weird.

[emaste]

kldstat.8 update?

[trasz]

Update manual page, and drop the "-h" from usage() in one case it doesn't apply.

[emaste]

Looks good other than two minor comments

sbin/kldstat/kldstat.8

53 ↗

(On Diff #12419)

I believe for standard manpage style this should something like "Display the size field in a human-readable form..."

sbin/kldstat/kldstat.c

38–39 ↗

(On Diff #12419)

any reason this shouldn't sort in with the other headers?

69–71 ↗

(On Diff #12419)

fwiw I think it's better in these cases to always use the temporary buf, so that there aren't two strings that could get out of sync

[trasz]

More input from emaste@.

[emaste]

LGTM with two nits

sbin/kldstat/kldstat.c

59 ↗

(On Diff #12467)

buf[6] now too small for %-8zx format string

69 ↗

(On Diff #12467)

note that there's a %-8s in the printf below so this shouldn't need any width/alignment set explicitly

[trasz]

Improve formatting - the "-h" output looks much better now - and go
back to two separate printfs.