On Mon, Apr 08, 2019 at 04:12:11PM +0800, Zhao Yakui wrote:
When acrn_hypervisor is detected, the hypercall is needed so that the
acrn guest can query/config some settings. For example: it can be used
to query the resources in hypervisor and manage the CPU/memory/device/
interrupt for Guest system.
Good example.
What is "Guest system" and why is capitalized? Do you mean "the guest
operating system" or simply "the guest"?
So the hypercall is added so that the kernel can communicate with the
"So add the hypercall so that..."
low-level acrn-hypervisor.
Is it acrn_hypervisor or acrn-hypervisor or the ACRN hypervisor or ...?
Unify the naming pls.
On x86 it is implemented by using vmacll when
During last review Thomas said:
"is implemented with the VMCALL instruction"
You still have it wrong. Which makes me think you haven't even gone over
*all* review comments as this is the second missed review comment in a
4-patches set.
So I'm going to stop reviewing here and won't look at your patches until
you incorporate *all* review comments from all people.