[Patch] Small fix for telem info dump in Intel CMCI code

[Patch] Small fix for telem info dump in Intel CMCI code

[Ke, Liping]

[-- Attachment #1.1: Type: text/plain, Size: 123 bytes --]

Hi, Keir& Frank.

This is a small fix for telem info dump in Intel CMCI code.

Thanks a lot for your help!

Criping

[-- Attachment #1.2: Type: text/html, Size: 1028 bytes --]

[-- Attachment #2: intel_small_fix.patch --]
[-- Type: application/octet-stream, Size: 1154 bytes --]

diff -r 9fc957e63f8d xen/arch/x86/cpu/mcheck/mce_intel.c
--- a/xen/arch/x86/cpu/mcheck/mce_intel.c	Tue Mar 17 15:40:25 2009 +0000
+++ b/xen/arch/x86/cpu/mcheck/mce_intel.c	Fri Mar 20 10:14:01 2009 +0800
@@ -198,7 +198,6 @@
 {
     unsigned long flags;
     int i;
-    struct mc_info *mi = NULL;
     mctelem_cookie_t mctc;
     struct mca_summary bs;
 
@@ -226,7 +225,7 @@
             mctelem_commit(mctc);
             send_guest_global_virq(dom0, VIRQ_MCA);
         } else {
-            x86_mcinfo_dump(mi);
+            x86_mcinfo_dump(mctelem_dataptr(mctc));
             mctelem_dismiss(mctc);
        }
     } else if (mctc != NULL)
@@ -326,7 +325,6 @@
 
 fastcall void smp_cmci_interrupt(struct cpu_user_regs *regs)
 {
-    struct mc_info *mi = NULL;
     mctelem_cookie_t mctc;
     struct mca_summary bs;
 
@@ -341,9 +339,9 @@
             mctelem_commit(mctc);
             send_guest_global_virq(dom0, VIRQ_MCA);
         } else {
-            x86_mcinfo_dump(mi);
+            x86_mcinfo_dump(mctelem_dataptr(mctc));
             mctelem_dismiss(mctc);
-       }
+        }
     } else if (mctc != NULL)
         mctelem_dismiss(mctc);
 

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Re: [Patch] Small fix for telem info dump in Intel CMCI code

[Frank van der Linden]

Ke, Liping wrote:
> Hi, Keir& Frank.
>  
> This is a small fix for telem info dump in Intel CMCI code.
>  
> Thanks a lot for your help!
>  
> Criping

Thanks for catching that one! Can you tell that our testsuite has no 
CMCI injection support right now..

- Frank