In D55169#1262390, @kib wrote:

In D55169#1262376, @tembun_bk.ru 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.

In D55169#1262376, @tembun_bk.ru wrote:

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

In D55169#1262234, @kib wrote:

In D55169#1262072, @tembun_bk.ru wrote:

In D55169#1262058, @emaste 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.

In D55169#1262072, @tembun_bk.ru wrote:

In D55169#1262058, @emaste 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?

In D55169#1262058, @emaste wrote:

How about just load kernel module?

How about just load kernel module?

Correct wording.

• Remove redundant 'of the function' part.
• Change "it's" -> "it is".

looks great.

Don't touch kldfind(2), leave changes only for kldload(2).

This patch does not make sense (to me at least). kldfind(2) looks up kld from the loaded files in kernel, it does not search anything in the file system.
What the syscall does it strips the passed filename from the non-base path components, and then compares the base name with names of all loaded klds, returning the finding, if any.

The system call is a better place for this info. But it's ok to have it botg places in this case

The information is basically taken from kldload(8).

Fixed based on the remarks of @marck.
Added clarification of boot and fsck interaction with journals (check by source code).
Added Q&A from forums.freebsd.org.

Add MLINKS: wheel.7 and operator.7.

Rebase the patch.

• Document that games group is used by hgame(4) (since b6cad334e64).
• Correct several mandoc -T lint warnings.
• Document that bind(8) was removed from the base in FreeBSD 10.0.
• Replace reference to unbound(8) with local-unbound(8).
• Remove references to setuid(7), since NetBSD has this page, but FreeBSD does not.

I suppose, review from native English speaker is needed, otherwise it looks almost perfect!

I got comment from @marck to change 'filesystem' to 'file system', I will change it in the next diff for this review. In the documentation 'file system' won 'filesystem' with score 614:320 times in the texts.

In D54904#1255287, @ziaee wrote:

Thank you so much for doing this. Ufs is amazing and it doesn't get enough love

Thank you so much for doing this. Ufs is amazing and it doesn't get enough love

In D54801#1252698, @ziaee wrote:

Why? When you call invoke less as "more" it works exactly the same but gives a classic BSD status line at the bottom.

Why? When you call invoke less as "more" it works exactly the same but gives a classic BSD status line at the bottom.

Approved, thanks!