kldload(8): Improve phrasing
ClosedPublic
Actions

Authored by debdrup on Feb 8 2021, 7:12 PM.

Details

Reviewers

eugen_grosbein.net
imp
kevans

Group Reviewers

manpages

Commits

rG2a7d4d954504: kldload(8): Improve phrasing

Summary

Mountroot isn't documented in the extant manual pages - so this
phrasing, while less absolute and concise, still conveys which
modules are recommended to be handled via loader.conf(5), and it also
does a better job of elucidating that the modules can include filesystem
drivers.

Submitted by: kevans@ (earlier version)
Reported by: imp@, kevans@, Eugene Grosbein <eugen at grosbein.net>

Test Plan

mandoc and igor

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

debdrup created this revision.Feb 8 2021, 7:12 PM

Herald added a reviewer: manpages. · View Herald TranscriptFeb 8 2021, 7:12 PM

Herald added subscribers: Contributor Reviews (src), imp. · View Herald Transcript

debdrup requested review of this revision.Feb 8 2021, 7:12 PM

Harbormaster completed remote builds in B36811: Diff 83559.Feb 8 2021, 7:12 PM

debdrup edited the test plan for this revision. (Show Details)Feb 8 2021, 7:13 PM

debdrup added reviewers: eugen_grosbein.net, imp, kevans.

I could probably bikeshed this all day just by myself, but this is a substantial improvement -- it's still somewhat strong wording ("Only those ... should be handled by loader.conf(5)"), but IMO it includes a good enough example of the set of modules it's talking about for one to naturally exercise caution when moving stuff from loader.conf(5) to kld_list and those of us that know better will ignore the strength of the message anyways.

This revision is now accepted and ready to land.Feb 11 2021, 4:54 AM

Closed by commit rG2a7d4d954504: kldload(8): Improve phrasing (authored by debdrup). · Explain WhyFeb 11 2021, 5:05 AM

This revision was automatically updated to reflect the committed changes.

debdrup added a commit: rG2a7d4d954504: kldload(8): Improve phrasing.