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
Unknown Object (File)
Sun, Sep 22, 6:43 PM
Unknown Object (File)
Thu, Sep 19, 3:06 PM
Unknown Object (File)
Wed, Sep 18, 4:58 PM
Unknown Object (File)
Sun, Sep 15, 10:40 PM
Unknown Object (File)
Sun, Sep 8, 6:53 PM
Unknown Object (File)
Thu, Sep 5, 6:47 AM
Unknown Object (File)
Aug 16 2024, 5:55 AM
Unknown Object (File)
Jun 30 2024, 1: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