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–506

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–506

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

898

903

2nd-person pronoun use is discouraged.

942

949

993

1000

1008

1016

1024

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

1029

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

1062

1067

1079

1140

1141

1186

1232

1237

1239

1252

For consistency with the actual examples

1272–1274

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

898

"additionally" and "reasonably" misspelled.

942

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

567

780

Does this sound clearer?

805

806–807

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

814

855

858

863

889

897

898

918

931–941

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!