Skip to content

virtualbox: VBoxVMI batched physical writes - #307

Merged
Wenzel merged 2 commits into
devfrom
vbox/vboxvmi-write
Oct 12, 2026
Merged

Wenzel merged 2 commits into
devfrom
vbox/vboxvmi-write

Conversation

@Wenzel

@Wenzel Wenzel commented Oct 12, 2026

Copy link
Copy Markdown
Owner

Closes #285.

  • VBoxVMI driver implements write_physical through batched PHYS_WRITE requests (multiple ranges, per-page status) via rs-vboxvmi 9a349ca. Entire range set is prevalidated; completed-batch statuses survive a later malformed reply. Writes are non-atomic across pages; bytes_read/errors identify the readable/writable prefix consistently.
  • Existing core-ground-truth write test now runs under both methods. A VM-state guard is armed before pause and all fallible page-selection/core operations; the byte-restoration guard drops before resume.
  • Driver/test docs: both methods now support physical writes.

Server: Wenzel/virtualbox#8 (merged into vmi/vboxvmi).

Verified live: VBoxVMI 512 MiB (seed 1791797247855151235; pages 0xf000, 0xffff000, 0x1ffff000) and 5120 MiB (seed 1791798462874390042; pages 0xf000, 0xaffff000, 0xdffff000, 0x15fff9000 >4 GiB), each checked in fresh cores before/after resume and after restore. Full VBoxVMI regression and MachineDebugger write/core pass. fmt/clippy/tests green locally.

@Wenzel
Wenzel merged commit b1307c5 into dev Oct 12, 2026
22 checks passed
@Wenzel Wenzel mentioned this pull request Oct 12, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant