Update to FreeBSD handbook: virtualization chapter

[freebsd_ny-central.org]

Summary

Expanding documentation around bhyve

• snapshot feature (experimental)
• additional notice about persisting and changing network configuration and potential for locking oneself out
• jailing bhyve

I should probably add, that I'm more than willing to remove the part on snapshots, because it's certainly very experimental. On the other hand, this might invite more people to test and try things out.

Feedback - as usual - very welcome!

[emaste]

documentation/content/en/books/handbook/virtualization/_index.adoc

500–501

We mention Intel and AMD; should we reference Arm as well?

[freebsd_ny-central.org]

Valid point on ARM - I've added a suggestion on ARM though I ain't an expert on the topic.
I got those infos from here: https://cgit.freebsd.org/src/commit/?id=47e073941f4e

[freebsd_ny-central.org]

documentation/content/en/books/handbook/virtualization/_index.adoc

500–501

Actually, now rereading what I wrote... this makes no sense. I'll change this again.

[freebsd_ny-central.org]

Restructured into a bullet point list; certainly better to read - hope the content is correct though.

[pauamma_gundo.com]

Bunch of small things I spotted.

documentation/content/en/books/handbook/virtualization/_index.adoc

893

898

2nd-person pronoun use is discouraged.

937

944

988

995

1003

1011

1019

Shouldn't it be the same as the host's?

1024

Also, with 13.3 days from expected release, maybe wait a bit and use that as an example, for future-proofing?

1057

1062

1074

1135

1136

1181

1227

1232

1234

1247

For consistency with the actual examples

1267–1269

Why not pkg -j bhyve install bhyve-firmware?

[freebsd_ny-central.org]

Updated after feedback. Thanks for the great inputs!

[pauamma_gundo.com]

A few typos remain, but getting close, I think.

documentation/content/en/books/handbook/virtualization/_index.adoc

893

"additionally" and "reasonably" misspelled.

937

Spurious "if you"

[freebsd_ny-central.org]

Thanks for spotting those. I really need to run a spellcheck before I submit things. Apologies for the extra round.
Added additional typo and comma fixes and improvements for brevity.

[jrm]

documentation/content/en/books/handbook/virtualization/_index.adoc

562

775

Does this sound clearer?

800

801–802

I removed instances of the second person as Pau Amma requested in other reviews and suggested an alternative wording.

809

850

853

858

884

892

893

913

926–936

Would it be better to link to Quick Start section of the Updating and Upgrading FreeBSD chapter? The shell source at the top is nearly the same as here.

[freebsd_ny-central.org]

Thanks for the feedback. I've merged all suggested changes and also

• removed a few more "you" statements to a more neutral version
• added a note/recommendation on using UEFI vars file w/ Linux to prevent having to rename boot files

[jrm]

@freebsd_ny-central.org, this looks ready to me. Could you rebase your changes on top of the latest main after 3c1326a472ca67ff6159d644589ab5e5bfc1258c? Then we can commit.

Thanks,

Joe

[freebsd_ny-central.org]

• Rebased to 2698f302340dd1e06e8c683902f4a560ff114925.
• also edited the phrasing for firewire and USB drivers and removed "the" because in a previous statement it was excluded too.

[jrm]

What do you think about this commit log?

Handbook/Virtualization: Expand bhyve documentation

Notable changes:

- describe the experimental snapshot feature
- add notice about persisting and changing network configuration and the
  potential for locking oneself out
- describe jailing bhyve
- improve wording throughout the page

Reviewed by:    emaste, jrm, Pau Amma <[e-mail]>
Differential Revision:  https://reviews.freebsd.org/D44078

[freebsd_ny-central.org]

LGTM. Thanks for your help in getting this finalized!