Differential D53586

vmm: Move common accessors and vm_eventinfo into sys/dev/vmm
Needs ReviewPublic
Actions

Authored by markj on Nov 4 2025, 4:40 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Mon, Dec 1, 12:12 AM

	Unknown Object (File)
	Sun, Nov 30, 1:41 PM

	Unknown Object (File)
	Thu, Nov 27, 11:26 PM

	Unknown Object (File)
	Tue, Nov 18, 11:08 AM

	Unknown Object (File)
	Mon, Nov 17, 5:51 PM

	Unknown Object (File)
	Sun, Nov 16, 5:37 PM

	Unknown Object (File)
	Sat, Nov 15, 11:45 PM

	Unknown Object (File)
	Sat, Nov 15, 3:00 PM

View All 24 Files

Subscribers

Details

Reviewers

andrew
manu

Group Reviewers

Summary

Now that struct vm and struct vcpu are defined in headers, provide
inline accessors. We could just remove the accessors outright, but they
don't hurt and it would result in unneeded churn.

As a part of this, consolidate definitions related to struct
vm_eventinfo as well. I'm not sure if struct vm_eventinfo is really
needed now that vmmops_run implementations can directly access vm and
vcpu fields.

No functional change intended.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 68417
Build 65300: arc lint + arc unit

Event Timeline

markj created this revision.Nov 4 2025, 4:40 PM

Herald added a reviewer: andrew. · View Herald TranscriptNov 4 2025, 4:40 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.Nov 4 2025, 4:40 PM

markj added a parent revision: D53585: vmm: Deduplicate VM and vCPU state management code.

Harbormaster completed remote builds in B68417: Diff 165838.Nov 4 2025, 4:40 PM

Revision Contents
Changeset List

Path

Size

sys/

amd64/

include/

41 lines

vmm/

io/

1 line

37 lines

1 line

arm64/

include/

29 lines

vmm/

io/

2 lines

37 lines

2 lines

dev/

vmm/

2 lines

67 lines

riscv/

include/

30 lines

vmm/

40 lines

Diff 165838

sys/amd64/include/vmm.h

Loading...

sys/amd64/vmm/io/vhpet.c

Loading...

sys/amd64/vmm/vmm.c

Loading...

sys/amd64/vmm/vmm_ioport.c

Loading...

sys/arm64/include/vmm.h

Loading...

sys/arm64/vmm/io/vtimer.c

Loading...

sys/arm64/vmm/vmm.c

Loading...

sys/arm64/vmm/vmm_reset.c

Loading...

sys/dev/vmm/vmm_stat.h

Loading...

sys/dev/vmm/vmm_vm.h

Loading...

sys/riscv/include/vmm.h

Loading...

sys/riscv/vmm/vmm.c

Loading...