[Xen-changelog] [xen master] x86/mce: remove ASSERT's about mce_[u|d]handler_num in mce_action()

[patchbot@xxxxxxx]

commit 0489efef749e064efd0eba5fb45cf9f229fb22aa
Author:     Haozhong Zhang <haozhong.zhang@xxxxxxxxx>
AuthorDate: Wed Mar 8 15:11:06 2017 +0100
Commit:     Jan Beulich <jbeulich@xxxxxxxx>
CommitDate: Wed Mar 8 15:11:06 2017 +0100

    x86/mce: remove ASSERT's about mce_[u|d]handler_num in mce_action()
    
    Those assertions as well as mce_[u|d]handlers[], mce_[u|d]handler_num
    and mce_action() were intel only and lifted to the common code by c/s
    3a91769d6e1. However, MCE handling on AMD does not use mce_[u|d]handlers[]
    before and after that commit, so assertions in mce_action() about their
    size do not make sense for AMD. To be worse, they can crash the debug
    build on AMD. Remove them to make the debug build work on AMD.
    
    Signed-off-by: Haozhong Zhang <haozhong.zhang@xxxxxxxxx>
    Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>
---
 xen/arch/x86/cpu/mcheck/mce.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/xen/arch/x86/cpu/mcheck/mce.c b/xen/arch/x86/cpu/mcheck/mce.c
index 35117f8..cd4f0ee 100644
--- a/xen/arch/x86/cpu/mcheck/mce.c
+++ b/xen/arch/x86/cpu/mcheck/mce.c
@@ -1616,9 +1616,6 @@ static enum mce_result mce_action(const struct 
cpu_user_regs *regs,
         handlers = mce_uhandlers;
     }
 
-    /* At least a default handler should be registerd */
-    ASSERT(handler_num);
-
     local_mi = (struct mc_info*)mctelem_dataptr(mctc);
     x86_mcinfo_lookup(mic, local_mi, MC_TYPE_GLOBAL);
     if (mic == NULL) {
@@ -1651,7 +1648,6 @@ static enum mce_result mce_action(const struct 
cpu_user_regs *regs,
                 break;
             }
         }
-        ASSERT(i != handler_num);
     }
 
     return worst_result;
--
generated by git-patchbot for /home/xen/git/xen.git#master

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
https://lists.xenproject.org/xen-changelog