Re: [PATCH]: Fix MSR_K7_EVNTSEL{0,3} for SVM

Chris Lalancette
2008-04-24T15:44:06Z

Avi Kivity wrote:
>
>> 1811: 8d 86 00 00 ff 3f lea 0x3fff0000(%rsi),%eax
>> 1817: 83 f8 03 cmp $0x3,%eax
>> 181a: 0f 87 e2 01 00 00 ja 1a02 <svm_set_msr+0x27f>
> Now it uses %rsi instead of %esi, and any junk in the upper bits will
> cause the ja to be taken.
>
> We need to get a reduced testcase to the gcc folks, this is a serious
> bug. Any changes in the code to work around this would be fragile.
>

Ouch, I missed this on my reading of it. I'll try to come up with a standalone
program that shows this.

Thanks, Avi.

Chris Lalancette