Re: [PATCH -v3] KVM: MCE: Add MCE support to KVM

Avi Kivity
2009-05-26T08:12:05Z

Jan Kiszka wrote:
>> Well, if I look at the definition of MCI_STATUS_VAL as (1UL<<63),
>> something tells me: "Hey, only use me on 64-bit hosts!" But I have no
>> clue about details of this stuff, and from a second glance at it is
>> seems to include at least some parts that are valid on 32-bit as well.
>>
>
> Ah, interesting: /someone/ removed the #ifdef __x86_64__ from
> arch/x86/include/asm/mce.h, but that's not mainline yet...
>

Well that someone missed the MCI_STATUS_VAL bit. Do you see anything
else fishy in there?

--
error compiling committee.c: too many arguments to function