Re: [Xen-devel] [PATCH v2 4/4] nestedhvm: replace VMCX_EADDR by INVALID_PADDR

2016-12-14 Thread Boris Ostrovsky
On 12/14/2016 05:18 AM, Haozhong Zhang wrote: > On 12/14/16 18:11 +0800, Haozhong Zhang wrote: >> ... because INVALID_PADDR is a more general one. >> >> Suggested-by: Jan Beulich >> Signed-off-by: Haozhong Zhang >> --- >> xen/arch/x86/hvm/nestedhvm.c | 2 +- >> xen/arch/x86/hvm/svm/nestedsvm.

Re: [Xen-devel] [PATCH v2 4/4] nestedhvm: replace VMCX_EADDR by INVALID_PADDR

2016-12-14 Thread Konrad Rzeszutek Wilk
On Wed, Dec 14, 2016 at 03:16:33AM -0700, Jan Beulich wrote: > >>> On 14.12.16 at 11:11, wrote: > > ... because INVALID_PADDR is a more general one. > > > > Suggested-by: Jan Beulich > > Signed-off-by: Haozhong Zhang > > Reviewed-by: Jan Beulich > > Thanks for doing this! Indeed! Thank you.

Re: [Xen-devel] [PATCH v2 4/4] nestedhvm: replace VMCX_EADDR by INVALID_PADDR

2016-12-14 Thread Tian, Kevin
> From: Zhang, Haozhong > Sent: Wednesday, December 14, 2016 6:12 PM > > ... because INVALID_PADDR is a more general one. > > Suggested-by: Jan Beulich > Signed-off-by: Haozhong Zhang Reviewed-by: Kevin Tian ___ Xen-devel mailing list Xen-devel@lis

Re: [Xen-devel] [PATCH v2 4/4] nestedhvm: replace VMCX_EADDR by INVALID_PADDR

2016-12-14 Thread Haozhong Zhang
On 12/14/16 18:11 +0800, Haozhong Zhang wrote: ... because INVALID_PADDR is a more general one. Suggested-by: Jan Beulich Signed-off-by: Haozhong Zhang --- xen/arch/x86/hvm/nestedhvm.c | 2 +- xen/arch/x86/hvm/svm/nestedsvm.c | 18 +- xen/arch/x86/hvm/svm/vmcb.c | 2 +-

Re: [Xen-devel] [PATCH v2 4/4] nestedhvm: replace VMCX_EADDR by INVALID_PADDR

2016-12-14 Thread Jan Beulich
>>> On 14.12.16 at 11:11, wrote: > ... because INVALID_PADDR is a more general one. > > Suggested-by: Jan Beulich > Signed-off-by: Haozhong Zhang Reviewed-by: Jan Beulich Thanks for doing this! Jan ___ Xen-devel mailing list Xen-devel@lists.xen.

[Xen-devel] [PATCH v2 4/4] nestedhvm: replace VMCX_EADDR by INVALID_PADDR

2016-12-14 Thread Haozhong Zhang
... because INVALID_PADDR is a more general one. Suggested-by: Jan Beulich Signed-off-by: Haozhong Zhang --- xen/arch/x86/hvm/nestedhvm.c | 2 +- xen/arch/x86/hvm/svm/nestedsvm.c | 18 +- xen/arch/x86/hvm/svm/vmcb.c | 2 +- xen/arch/x86/hvm/vmx/vvmx.c | 16 ++