Page MenuHomeFreeBSD
Differential D3112

Describe how to load gmultipath at boot time.
ClosedPublic
Actions

Authored by araujo on Jul 17 2015, 1:44 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 16, 12:12 PM
Unknown Object (File)
Thu, Jan 15, 5:37 PM
Unknown Object (File)
Tue, Dec 23, 7:06 AM
Unknown Object (File)
Dec 20 2025, 3:05 PM
Unknown Object (File)
Dec 20 2025, 3:05 PM
Unknown Object (File)
Nov 30 2025, 12:52 PM
Unknown Object (File)
Nov 20 2025, 11:39 PM
Unknown Object (File)
Nov 20 2025, 11:38 PM
View All Files
Subscribers
imp
wblock

Details

Reviewers
allanjude
bapt
Group Reviewers
manpages
Commits
rS285666: Fix contraction spotted by igor(1) and remove 2 .Ed spotted by mandoc(1).
rS285647: Describe how to load gmultipath at boot time.
Summary

Add an entry just to explain how to load gmultipath at boot time.
I didn't find any documentation about it on FreeBSD.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

araujo updated this revision to Diff 7034.Jul 17 2015, 1:44 AM
araujo retitled this revision from to Describe how to load gmultipath at boot time..
araujo updated this object.
araujo edited the test plan for this revision. (Show Details)
araujo added reviewers: bapt, allanjude.
Herald added a reviewer: manpages. · View Herald TranscriptJul 17 2015, 1:44 AM
Herald added a subscriber: imp. · View Herald Transcript
allanjude added inline comments.Jul 17 2015, 1:48 AM
sbin/geom/class/multipath/gmultipath.8
355

to load the
.Nm
Module at boot time, add the following entry to
.Pa /boot/loader.conf :

358

missing closing quote

The quotes might need to be done as markup

360

this 2nd .ed is not capitalized correctly

allanjude edited edge metadata.Jul 17 2015, 1:48 AM

Sadly, I think the "official documentation" for this, was /boot/defaults/loader.conf

araujo updated this revision to Diff 7035.Jul 17 2015, 1:59 AM
araujo edited edge metadata.

Address the issues spotted by Allan, also add his suggestions :).
I saw in other man pages that double quote doesn't use any markup.

Thanks Allan for the very responsive and fast review,

allanjude added inline comments.Jul 17 2015, 3:22 AM
sbin/geom/class/multipath/gmultipath.8
355

Sentence should start with a capital

araujo updated this revision to Diff 7036.Jul 17 2015, 3:25 AM

Start sentence with a capital.

bapt accepted this revision.Jul 17 2015, 5:39 AM
bapt edited edge metadata.
This revision is now accepted and ready to land.Jul 17 2015, 5:39 AM
Closed by commit rS285647: Describe how to load gmultipath at boot time. (authored by araujo). · Explain WhyJul 17 2015, 6:35 AM
This revision was automatically updated to reflect the committed changes.
araujo mentioned this in rS285647: Describe how to load gmultipath at boot time..
wblock added a subscriber: wblock.Jul 17 2015, 10:58 PM

Remember to run igor -R and mandoc -Tlint on man page changes to check for problems. igor can also be used on source, text, man pages, and XML files (including vuxml).

head/sbin/geom/class/multipath/gmultipath.8
357 ↗(On Diff #7038)

Do not capitalize "module". "the following" is redundant here:

module at boot time, add this entry to

358 ↗(On Diff #7038)

The colon must be preceded by a space, or it will be included as part of the path:

.Pa /boot/loader.conf :

araujo added a comment.Jul 18 2015, 2:44 AM
In D3112#61957, @wblock wrote:

Remember to run igor -R and mandoc -Tlint on man page changes to check for problems. igor can also be used on source, text, man pages, and XML files (including vuxml).

Thanks @wblock, very appreciated your review, I'm fixing now.

Best,

araujo mentioned this in rS285666: Fix contraction spotted by igor(1) and remove 2 .Ed spotted by mandoc(1)..Jul 18 2015, 3:15 AM

Revision Contents
Changeset List

PathSize
sbin/
geom/
class/
multipath/
12 lines

Diff 7036

View Options

sbin/geom/class/multipath/gmultipath.8

Loading...