[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH V4.1] mcheck, vmce: Allow vmce_amd_* functions to handle AMD thresolding MSRs

On 2/13/2014 2:38 AM, Jan Beulich wrote:
    *val = 0;
- switch ( msr & (MSR_IA32_MC0_CTL | 3) )
+    /* Allow only first 3 MC banks into switch() */
+    switch ( msr & (-MSR_IA32_MC0_CTL | 3) )
      case MSR_IA32_MC0_CTL:
          /* stick all 1's to MCi_CTL */
I'm confused: You now add a comment as if the mask was including
bit 4, which it doesn't. What am I missing?

Darn. Sorry about that. Will fix..
Also, please get used to mention (commonly at the bottom of the
commit message, after a --- separator) what changed compared to
the previous iteration.

Okay, will do.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.