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

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

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

892

897

2nd-person pronoun use is discouraged.

936

943

987

994

1002

1010

1018

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

1023

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

1056

1061

1073

1134

1135

1180

1226

1231

1233

1246

For consistency with the actual examples

1266–1268

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

892

"additionally" and "reasonably" misspelled.

936

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

561

774

Does this sound clearer?

799

800–801

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

808

849

852

857

883

891

892

912

925–935

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!