vmm: Refactor VM reinitialization code a bit
Needs ReviewPublic
Actions

Authored by markj on Fri, Oct 30, 2:53 PM.

Project Tags

None

Referenced Files

	F177532961: D61043.1794648688.diff
	Fri, Nov 13, 9:31 AM

Subscribers

Details

Reviewers

jhb
andrew
manu

Group Reviewers

bhyve

Summary

We can avoid the bool parameter to vm_init() by moving a bit of code
around. This makes the VM (re)initialization code easier to read.

No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 78568
Build 75451: arc lint + arc unit

Event Timeline

markj created this revision.Fri, Oct 30, 2:53 PM

Herald added a reviewer: andrew. · View Herald TranscriptFri, Oct 30, 2:53 PM

Herald added a reviewer: andrew. · View Herald Transcript

Herald added a reviewer: manu. · View Herald Transcript

Herald added subscribers: riscv, bcran, rgrimes and 2 others. · View Herald Transcript

markj requested review of this revision.Fri, Oct 30, 2:53 PM

Harbormaster completed remote builds in B78568: Diff 190995.Fri, Oct 30, 2:53 PM

markj added a child revision: D61044: vmm: Use an rms lock to synchronize ioctls.Fri, Oct 30, 2:53 PM

Revision Contents
Changeset List

Path

Size

sys/

amd64/

vmm/

vmm.c

20 lines

arm64/

vmm/

vmm.c

19 lines

riscv/

vmm/

vmm.c

19 lines

Diff 190995

View Options

sys/amd64/vmm/vmm.c

View Options

sys/arm64/vmm/vmm.c

View Options

vmm: Refactor VM reinitialization code a bitNeeds ReviewPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 190995

sys/amd64/vmm/vmm.c

sys/arm64/vmm/vmm.c

sys/riscv/vmm/vmm.c

vmm: Refactor VM reinitialization code a bit
Needs ReviewPublic
Actions

Revision Contents
Changeset List