Page MenuHomeFreeBSD

vmm: Rework snapshotting of CPU-specific per-vCPU data.
ClosedPublic

Authored by jhb on Oct 27 2022, 3:03 PM.
Tags
None
Referenced Files
F82011937: D37150.id113290.diff
Wed, Apr 24, 12:45 PM
F82011504: D37150.id112577.diff
Wed, Apr 24, 12:38 PM
F82011500: D37150.id.diff
Wed, Apr 24, 12:38 PM
F82011497: D37150.id112259.diff
Wed, Apr 24, 12:38 PM
Unknown Object (File)
Sun, Apr 21, 8:16 PM
Unknown Object (File)
Feb 15 2024, 5:37 AM
Unknown Object (File)
Feb 15 2024, 5:37 AM
Unknown Object (File)
Feb 15 2024, 5:37 AM
Subscribers

Details

Summary

Previously some per-vCPU state was saved in vmmops_snapshot and other
state was saved in vmmops_vcmx_snapshot. Consolidate all per-vCPU
state into the latter routine and rename the hook to the more generic
'vcpu_snapshot'. Note that the CPU-independent per-vCPU data is still
stored in a separate blob as well as the per-vCPU local APIC data.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

markj added inline comments.
sys/amd64/vmm/amd/svm.c
2599

"i" is gone.

2644

Elsewhere this is done with svm_set_dirty(svm_sc, vcpu, 0xffffffff);. This code is missing VMCB_CACHE_LBR.

This revision is now accepted and ready to land.Nov 2 2022, 2:43 PM
sys/amd64/vmm/amd/svm.c
2644

I might fix that in a separate commit I'll add to the series. This is just a copy/paste of the old code.

This revision now requires review to proceed.Nov 3 2022, 11:48 PM
corvink added a reviewer: manu.
This revision is now accepted and ready to land.Nov 4 2022, 7:20 AM