On 3/21/2017 9:18 AM, Yu Zhang wrote:
On 3/21/2017 1:03 AM, Jan Beulich wrote:
On 18.03.17 at 11:53, <yu.c.zh...@linux.intel.com> wrote:
--- a/xen/arch/x86/hvm/hvm.c
+++ b/xen/arch/x86/hvm/hvm.c
@@ -1870,18 +1870,14 @@ int hvm_hap_nested_page_fault(paddr_t gpa,
unsigned long gla,
(npfec.write_access &&
(p2m_is_discard_write(p2mt) || (p2mt ==
p2m_ioreq_server))) )
{
- __put_gfn(p2m, gfn);
- if ( ap2m_active )
- __put_gfn(hostp2m, gfn);
-
rc = 0;
if ( unlikely(is_pvh_domain(currd)) )
- goto out;
+ goto out_put_gfn;
Did you forget to re-base onto staging before you've sent this?
is_pvh_domain() was gone before your submission already. When
re-basing, feel free to also drop the dead "rc = 0;" line.
Oops...
I did a rebase days ago, but did not send the patch directly. There
were some XenGT test performed based on these patch.
Sorry, I should have a check. Will resend the patch. :)
BTW, since there's another change - "drop the rc = 0", I'd like to send
the new patchset with a new version nubmer. :-)
Yu
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
