Page MenuHomeFreeBSD

kldload.2: Spell kld in lowercase
Needs RevisionPublic

Authored by tembun_bk.ru on Sat, Feb 7, 8:53 PM.

Details

Summary

for consistency with other kld*(2) manpages.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dab requested changes to this revision.Sun, Feb 8, 10:55 PM
dab added a subscriber: dab.
dab added inline comments.
lib/libsys/kldload.2
26

Need to update the revision date.

This revision now requires changes to proceed.Sun, Feb 8, 10:55 PM
lib/libsys/kldload.2
26

No. This is a typo. It would be especially bad to bump this date for a typo because the date reflects that the content is from 1999.

https://docs.freebsd.org/en/books/fdp-primer/manual-pages/#manual-pages-markup-guidelines

How about just load kernel module?

How about just load kernel module?

That's fine with me. However @kib already suggested to provide more information on what the 'kld file' is: https://reviews.freebsd.org/D55170#1261544. I thought maybe we can put this information in kld(4) for instance and use it as reference in kld*(2) man pages?

How about just load kernel module?

That's fine with me. However @kib already suggested to provide more information on what the 'kld file' is: https://reviews.freebsd.org/D55170#1261544. I thought maybe we can put this information in kld(4) for instance and use it as reference in kld*(2) man pages?

No, technically we load 'kld file', which is linked into the executing kernel. The file might contain some number of modules (from 0 to arbitrary), which are used as units of initialization.

In D55169#1262234, @kib wrote:

How about just load kernel module?

That's fine with me. However @kib already suggested to provide more information on what the 'kld file' is: https://reviews.freebsd.org/D55170#1261544. I thought maybe we can put this information in kld(4) for instance and use it as reference in kld*(2) man pages?

No, technically we load 'kld file', which is linked into the executing kernel. The file might contain some number of modules (from 0 to arbitrary), which are used as units of initialization.

Thank you! Do you think that kld(4) would be an appropriate place for this description? I guess I can incorporate it there.

Thank you! Do you think that kld(4) would be an appropriate place for this description? I guess I can incorporate it there.

Current kld(4) page needs a complete rewrite. Starting from the removal of LKM references, which were not relevant since 4.x.
The renovation for kld(4) is technically complicated work, requiring a lot of code reading to provide good manual.

In D55169#1262390, @kib wrote:

Thank you! Do you think that kld(4) would be an appropriate place for this description? I guess I can incorporate it there.

Current kld(4) page needs a complete rewrite. Starting from the removal of LKM references, which were not relevant since 4.x.
The renovation for kld(4) is technically complicated work, requiring a lot of code reading to provide good manual.

If kld(4) needs a review in the first place, it's better to do it first. I'm not experienced in kernel area, so it definitely should not be done by me :)