On 7/18/2025 1:25 PM, Easwar Hariharan wrote:
On 7/18/2025 10:13 AM, Michael Kelley wrote:
From: Easwar Hariharan <eahar...@linux.microsoft.com> Sent: Friday, July 18,
2025 9:33 AM
On 7/17/2025 9:55 PM, mhkelle...@gmail.com wrote:
From: Michael Kelley <mhkli...@outlook.com>
[snip]
The new code compiles and runs successfully on x86 and arm64. However,
basic smoke tests cover only a limited number of hypercall call sites
that have been modified. I don't have the hardware or Hyper-V
configurations needed to test running in the Hyper-V root partition
or running in a VTL other than VTL 0. The related hypercall call sites
still need to be tested to make sure I didn't break anything. Hopefully
someone with the necessary configurations and Hyper-V versions can
help with that testing.
Easwar --
Thanks for reviewing.
Any chance you (or someone else) could do a quick smoke test of this
patch set when running in the Hyper-V root partition, and separately,
when running in VTL2? Some hypercall call sites are modified that
don't get called in normal VTL0 execution. It just needs a quick
verification that nothing is obviously broken for the root partition and
VTL2 cases.
Michael
I'm working almost entirely in VTL0, so I'd call on Nuno, Naman, and Roman
(cc'ed) to help.
Michael,
I'll try to squeeze that in during the next week. Folks should feel free
to beat me to that :) The caveat would be that there are scenarios that
are beyond the capabilities of the hardware that I have readily
available, and would need to run in test clusters in Azure, and these
are pretty busy.
VTL2 currently uses a limited number hypercalls that are set as enabled
in the OpenVMM code (`set_allowed_hypercalls`). You could take a look
and conclude if these hypercalls require any adjustments in the patches.
My opinion has been to have two pages (input and output ones). As the
new code introduces just one page I do feel a bit apprehensive, got no
hard evidence that this is a bad approach though. If we tweak the code
to have 2 pages, perhaps there would be no need to run a full-blown
validation, and even smoke tests will suffice?
- Easwar
--
Thank you,
Roman
