[prev in list] [next in list] [prev in thread] [next in thread] 

List:       xen-cvs
Subject:    [Xen-changelog] [xen master] x86/nested HAP: don't BUG() on legitimate error
From:       patchbot () xen ! org
Date:       2014-04-16 11:03:56
Message-ID: E1WaNdE-0000kO-6H () xenbits ! xen ! org
[Download RAW message or body]

commit 1ca73aaf51eba14256794bf045c2eb01e88e1324
Author:     Jan Beulich <jbeulich@suse.com>
AuthorDate: Mon Apr 14 12:50:56 2014 +0200
Commit:     Jan Beulich <jbeulich@suse.com>
CommitDate: Mon Apr 14 12:50:56 2014 +0200

    x86/nested HAP: don't BUG() on legitimate error
    
    p2m_set_entry() can fail without there being a bug in the code - crash
    the domain rather than the host in that case.
    
    Signed-off-by: Jan Beulich <jbeulich@suse.com>
    Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
    Acked-by: Tim Deegan <tim@xen.org>
---
 xen/arch/x86/mm/hap/nested_hap.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/xen/arch/x86/mm/hap/nested_hap.c b/xen/arch/x86/mm/hap/nested_hap.c
index 5c41725..9d8bfc8 100644
--- a/xen/arch/x86/mm/hap/nested_hap.c
+++ b/xen/arch/x86/mm/hap/nested_hap.c
@@ -134,7 +134,7 @@ nestedhap_fix_p2m(struct vcpu *v, struct p2m_domain *p2m,
         gdprintk(XENLOG_ERR,
                  "failed to set entry for %#"PRIx64" -> %#"PRIx64" rc:%d\n",
                  L2_gpa, L0_gpa, rc);
-        BUG();
+        domain_crash(p2m->domain);
     }
 }
 
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@lists.xen.org
http://lists.xensource.com/xen-changelog
[prev in list] [next in list] [prev in thread] [next in thread]