Re: [Xen-devel] [V10 PATCH 00/23]PVH xen: Phase I, Version 10 patches...

[Mukesh Rathor <mukesh.rathor@xxxxxxxxxx>]

On Thu, 25 Jul 2013 17:39:28 +0100
Tim Deegan <tim@xxxxxxx> wrote:

> Hi,
...

> #19 is probably OK for correctness, though I haven't reviewed the VMCS
> settings in enough detail to be sure they're complete.  My main
> reservation is that it seems to duplicate a bunch of code from the
> HVM VMCS setup.

Forgot to respond to this. I factored out some common code into
vmx_set_common_host_vmcs_fields. Other lets just leave it, otherwise it
clutters the existing function way too much with if (PVH) statements. 

thanks
Mukesh


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel