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

Roff requires one sentence per line, so period always starts a new line. The linter will catch these basic issues, you can use it with mandoc -Tlint here/is/the/manual.

I agree with this from a doc principle but I don't know enough from a C standpoint, I'd like @emaste or @imp to check it.

remove set eficom

Hey Rick! Thanks for the ping. Tip: use the linter,

mandoc -Tlint here/is/the/manual

this uses saner code variant for change just done recently

I think this looks great! Let's wait for a second opinion on the content and then I'd be happy to take this.

Unfortunately parsing diffs from git format-patch from phabricator side seems to be broken. It only shows the first (and the most trivial) commit. I'm using basic git diff here.

Personally, I prefer a manual to stick strictly to its own contents and avoid referencing other libraries in the main text which keeps it concise and easier to read

This is such a beautifully structured manual. People often abuse .Sh when really they should be subsections of DESCRIPTION. Thanks for the patch Artem!

Thanks for doing this!

I still do (nonblocking, if anyone else is free) but I'm swamped at $dayjob and idk when I'll be able to get to it.

Thanks for the ping! I get busy and forget about stuff so I really appreciate pings!

Sunpoet committed this from bugzilla

I tried to test this, but the patch failed to apply

Ah, I misunderstood then, sorry. Over lunch I'll test this.

I don't think this is the diff you intended to upload based on the commit message title. Also, please format patches with -U9999 so we can see the context

Phabricator is disconnected somehow, but I updated this and just commit it

corrected info, tagged bug. thanks @adrian!

I think we have made a lot of progress on this, and the greatest points of controversy are resolved. To me, if this is acceptable to the major stake holders, I can merge this and we can continue iterating in the tree.

If the point is to align the two driver strings, I think that's one logical change and could/should be one commit.

I think if you want to make them the same you should remove FreeBSD from both. If people search logs for FreeBSD they shouldn't get these drivers, I think.

correctly capitalizing MegaRAID, and expanding version all sound good to me.

In D54433#1244080, @pi wrote:

Will it also explain pkgbasify ?

I think my text still could use improvement, but to me almost all of these suggestions is going the wrong way towards "explain them to death".

Committed without review

Don't forget to mfc

Please fix the commit message though, this is not a commit to bhyve.8 and it is not just adding things to the manual

remove Intel CPUs

added PORTREVISION, fixed OPTIONS_DEFAULT; thanks @jrm!

make mdoc(7) a bit clearer

Also, in your last commit to stable/13, you violated two rules:

I finished MFCing my end. Now, you should wait two days minimum, and then if nobody complains, you switch to the stable branches and git cherry-pick -x ccec94bf63d. This is written in the comitters guide.

This is mostly my commit, I will MFC it, then you should be able to MFC yours without conflict

We don't really do it this way. If you want to sync this manual with current, you MFC the changes. It's a lot easier then to keep track of everything, and not have merge conflicts.

Manual LGTM, idk about the code.

Okay, I'm fine now. Thanks!

This list is by controller, and the 2208 controller is already listed, so this would make the table inconsistent. I think we should not add it to this table.

Thanks for doing this!

Note that deferred mbuf freeing is not enabled by default, and can be

enabled using the dev.$DEV.$UNIT.iflib.tx_defer_mfree sysctl.

In D54520#1246460, @michaelo wrote:

In fact, mrsas(4) mentions:

HARDWARE
     The mrsas driver supports the following hardware:

     [ Thunderbolt 6Gb/s MR controller ]
     •   LSI MegaRAID SAS 9265
     •   LSI MegaRAID SAS 9266
     •   LSI MegaRAID SAS 9267
     •   LSI MegaRAID SAS 9270
     •   LSI MegaRAID SAS 9271
     •   LSI MegaRAID SAS 9272
     •   LSI MegaRAID SAS 9285
     •   LSI MegaRAID SAS 9286
     •   DELL PERC H810
     •   DELL PERC H710/P

     [ Invader/Fury 12Gb/s MR controller ]
     •   LSI MegaRAID SAS 9380
     •   LSI MegaRAID SAS 9361
     •   LSI MegaRAID SAS 9341
     •   DELL PERC H830
     •   DELL PERC H730/P
     •   DELL PERC H330

Let me evaluate this in a separate review.

I see, thanks michaelo!

If it works for both, should we list it for both?

Thanks for tagging me!

don't forget to fix the commit message typo!

Woah, sorry I hit the wrong button, which you can't deselect on phabricator mobile. Did not intend to commander revision!