2005-01-19 16:05:36

by Kirill Korotaev

[permalink] [raw]
Subject: possible CPU bug and request for Intel contacts

Hello Linus,

Linus, Ingo, I've got one strange CPU bug leading to oopses, reboots and
so on. This bug can be reproduced with a little bit modified 4gb split
and is probably related to CPU speculative execution. I'll post more
information about this bug later, but I would like to ask you for Intel
guys contacts who maybe interested in this information, so I could CC
them as well.

Thank you,
Kirill


2005-01-21 12:43:52

by Kirill Korotaev

[permalink] [raw]
Subject: Re: possible CPU bug and request for Intel contacts

--- ./arch/i386/kernel/entry_trampoline.c.4gbglb 2005-01-19 11:01:17.000000000 +0300
+++ ./arch/i386/kernel/entry_trampoline.c 2005-01-19 11:01:28.275121416 +0300
@@ -24,14 +24,16 @@ void __init init_entry_mappings(void)

void *tramp;
int p;
+ pgprot_t prot;

/*
* We need a high IDT and GDT for the 4G/4G split:
*/
trap_init_virtual_IDT();

- __set_fixmap(FIX_ENTRY_TRAMPOLINE_0, __pa((unsigned long)&__entry_tramp_start), PAGE_KERNEL_EXEC);
- __set_fixmap(FIX_ENTRY_TRAMPOLINE_1, __pa((unsigned long)&__entry_tramp_start) + PAGE_SIZE, PAGE_KERNEL_EXEC);
+ prot = __pgprot(pgprot_val(PAGE_KERNEL_EXEC) | _PAGE_GLOBAL);
+ __set_fixmap(FIX_ENTRY_TRAMPOLINE_0, __pa((unsigned long)&__entry_tramp_start), prot);
+ __set_fixmap(FIX_ENTRY_TRAMPOLINE_1, __pa((unsigned long)&__entry_tramp_start) + PAGE_SIZE, prot);
tramp = (void *)fix_to_virt(FIX_ENTRY_TRAMPOLINE_0);

printk("mapped 4G/4G trampoline to %p.\n", tramp);


Attachments:
4gbtest.c (2.24 kB)
diff-arch-4gb-global (937.00 B)
Download all attachments

2005-01-22 03:03:41

by Rohit Seth

[permalink] [raw]
Subject: RE: possible CPU bug and request for Intel contacts

Hello Kirill,

Thanks for sending the detailed information. Based on our experiments
and analysis, we believe at this point that this is a known E80 issue
mentioned in the PIII spec update at this location
(http://www.intel.com/design/pentiumiii/specupdt/24445351.pdf)

Could you please try one of the suggested work arounds for this issue.

Thanks, rohit

Kirill Korotaev <mailto:[email protected]> wrote on Friday, January 21, 2005
4:47 AM:

> Hello,
>
> Here are the details about CPU bug I mentioned in my previous post.
> Though it turned out later that it happens on P-III systems only I
> still
> hope it can be of interest.
>
> Brief description
> ~~~~~~~~~~~~~~~~~
>
> This issue was found by Vasily Averin ([email protected]) when playing
> with uselib security exploit on kernels with my 4gb split patch.
>
> This bug results in strange effects such as calltraces below,
> reboots, impossible call traces and so on.
>
> I started to resolve the bug, narrowed down uselib exploit and
> got a simple testcase for the bug, which can be found in attach.
> This testcase does a simple thing - it maps pages at low addresses
> from 0x04000000 downto 0x00000000, page by page and touches them
> for write. Sometimes when running this exploit I got oopses,
> sometimes reboots and I found that this is sensitive to the page
> addresses which exploit maps.
>
> Why it crashes? I think this is due to virtual addresses of
> kernel code and mapped user space pages overlap. I was able even to
> reboot machine if mapped user space pages were filled with some
> appropriate asm code.
>
> I found that Ingo Molnar 4gb split is not vulnerable, and after
> investigations I found that Ingo patch doesn't map kernel entry code
> (trampline) as _PAGE_GLOBAL. This was the answer.
>
> I tested it on 4 different P-III machines - all of them were
> vulnerable.
> But lately I tested it on Celeron 2.4Ghz and P4 systems - it doesn't
> happen, so this bug can be of low interest to Intel people :(
>
> Below you can find the way how to reproduce the bug, call traces
> and why I think it's a hardware bug.
>
> How to reproduce a bug
> ~~~~~~~~~~~~~~~~~~~~~~
>
> - take any FedoraCore kernel with Ingo Molnar 4gb split patch
> or mainstream kernel and apply 4GB split patch
> - apply attached diff-arch-4gb-global patch to make
> trampline code to be GLOBAL
> - compile kernel with turned on 4gb split, i.e. CONFIG_X86_4GB=y
> - boot the kernel and run the attached testcase:
>
> # while true; do ./4gbtest; done;
>
> or
>
> # ./elflbl -l ./lib -a 0x4000000 (where elflbl is uselib exploit)
>
> During each 4-5 test runs I get the following oops:
>
> Jan 21 12:15:17 ts Unable to handle kernel NULL pointer dereference at
> virtual address 000000c0
> Jan 21 12:15:17 ts printing eip:
> Jan 21 12:15:17 ts 02114450
> Jan 21 12:15:17 ts *pde = 00000000
> Jan 21 12:15:17 ts Oops: 0002
> Jan 21 12:15:17 ts SMP
> Jan 21 12:15:17 ts Modules linked in:
> Jan 21 12:15:17 ts CPU: 0
> Jan 21 12:15:17 ts EIP: 0060:[<02114450>] Not tainted
> Jan 21 12:15:17 ts EFLAGS: 00010246 (2.6.8-dev)
> Jan 21 12:15:17 ts EIP is at sys_mmap2+0x0/0xb0
> Jan 21 12:15:17 ts eax: 000000c0 ebx: 31524fc4 ecx: 00001000
> edx: 004ec000
> Jan 21 12:15:17 ts esi: 00000032 edi: 00000000 ebp: 31524000
> esp: 31524fc0
> Jan 21 12:15:17 ts ds: 007b es: 007b ss: 0068
> Jan 21 12:15:17 ts Process test (pid: 25, threadinfo=31524000
> task=31f680c0) Jan 21 12:15:17 ts Stack: fffec200 01a2a000 00001000
> 00000003 00000032 00000000 00000000 000000c0
> Jan 21 12:15:17 ts 0000007b 0000007b 000000c0 08048541 00000073
> 00000282 bffffdcc 0000007b
> Jan 21 12:15:17 ts Call Trace:
> Jan 21 12:15:17 ts Code: 55 bd f7 ff ff ff 57 31 ff 56 53 83 ec 18 8b
> 44 24 38 89 c6
>
> Unable to handle kernel NULL pointer dereference at virtual address
> 000000c0
> 02114450
> *pde = 00000000
> Oops: 0002
> CPU: 0
> EIP: 0060:[<02114450>] Not tainted
> EFLAGS: 00010246 (2.6.8-dev)
> eax: 000000c0 ebx: 31524fc4 ecx: 00001000 edx: 004ec000
> esi: 00000032 edi: 00000000 ebp: 31524000 esp: 31524fc0
> ds: 007b es: 007b ss: 0068
> Stack: fffec200 01a2a000 00001000 00000003 00000032 00000000
> 00000000 000000c0
> 0000007b 0000007b 000000c0 08048541 00000073 00000282
> bffffdcc 0000007b
> Call Trace:
> Code: 55 bd f7 ff ff ff 57 31 ff 56 53 83 ec 18 8b 44 24 38 89 c6
>
>
> >>EIP; 02114450 <sys_mmap2+0/b0> <=====
>
> >>ebx; 31524fc4 <pg0+2eff8fc4/fdac0000>
> >>ebp; 31524000 <pg0+2eff8000/fdac0000>
> >>esp; 31524fc0 <pg0+2eff8fc0/fdac0000>
>
> Code; 02114450 <sys_mmap2+0/b0>
> 00000000 <_EIP>:
> Code; 02114450 <sys_mmap2+0/b0> <=====
> 0: 55 push %ebp <=====
> Code; 02114451 <sys_mmap2+1/b0>
> 1: bd f7 ff ff ff mov $0xfffffff7,%ebp
> Code; 02114456 <sys_mmap2+6/b0>
> 6: 57 push %edi
> Code; 02114457 <sys_mmap2+7/b0>
> 7: 31 ff xor %edi,%edi
> Code; 02114459 <sys_mmap2+9/b0>
> 9: 56 push %esi
> Code; 0211445a <sys_mmap2+a/b0>
> a: 53 push %ebx
> Code; 0211445b <sys_mmap2+b/b0>
> b: 83 ec 18 sub $0x18,%esp
> Code; 0211445e <sys_mmap2+e/b0>
> e: 8b 44 24 38 mov 0x38(%esp,1),%eax
> Code; 02114462 <sys_mmap2+12/b0>
> 12: 89 c6 mov %eax,%esi
>
> Why CPU is unable to handle paging request at 0x000000c0? There is no
> access to
> this addr in executing code! What has "push %ebp" to do with 0xc0?
> The answer is that %eax contains 0xc0 and the touched in user space
> pages contain 4092 zero bytes. And 0x0000 is an opcode for "addl %al,
> (%eax)".
> So we see the situation when CPU is executing code from user space
> pages though we are in kernel space already and data peeks from these
> addresses
> shows us the correct code (code in call trace is correct!).
> I checked it and if these pages are filled with some other values,
> not zeroes, than it's possible to make CPU execute this code.
>
> And why this happens on sys_mmap2+0? Because entry code (system_call)
> is mapped at high addresses (> 0xffc00000) and is the same both in
> kernel
> and user spaces, so entry.S code works ok.
>
> So we found 2 ways of curing this bug:
> - make trampline code to be non-GLOBAL
> - another observation was that PAE turned ON helps as well.
>
> Hypothesis
> ~~~~~~~~~~
> I think that the problem is in code prefetch queue or somewhere in
> CPU.
> It looks like CPU doesn't flush code prefetch queue after %cr3 reload
> (to kernel space) in entry.S and continues to execute prefetched code
> from user space pages.
>
> Why making entry code non-global helps the problem?
> I think that if the code at %eip is flushed on %cr3 reload than the
> _whole_ prefetch queue is flushed and when entry code is global than
> it is
> not flushed on %cr3 reload and prefetch queue (including call to
> flushed sys_mmap2 code) is not flushed.
>
> Kirill
>
>
>> Hi Kirill,
>>
>> I appreciate you bringing this issue up. Could you please send us
>> the information on how you are able to reproduce this issue (System
>> config, Linux kernel version and any test case). We would like to
>> root cause the failure here at Intel.
>>
>> Appreciate your help,
>> Thanks,
>> -rohit
>>
>> Kirill Korotaev <> wrote on Wednesday, January 19, 2005 8:08 AM:
>>
>>
>>> Hello Linus,
>>>
>>> Linus, Ingo, I've got one strange CPU bug leading to oopses, reboots
>>> and so on. This bug can be reproduced with a little bit modified 4gb
>>> split and is probably related to CPU speculative execution. I'll
>>> post more information about this bug later, but I would like to ask
>>> you for Intel guys contacts who maybe interested in this
>>> information, so I could CC them as well.
>>>
>>> Thank you,
>>> Kirill
>>>
>>> -
>>> To unsubscribe from this list: send the line "unsubscribe
>>> linux-kernel" in the body of a message to [email protected]
>>> More majordomo info at http://vger.kernel.org/majordomo-info.html
>>> Please read the FAQ at http://www.tux.org/lkml/

2005-01-22 10:20:47

by Pavel Machek

[permalink] [raw]
Subject: Re: possible CPU bug and request for Intel contacts

Hi!

> Here are the details about CPU bug I mentioned in my previous post.
> Though it turned out later that it happens on P-III systems only I still
> hope it can be of interest.

What about Pentium-M? They are based on P-III and are certainly *very*
interesting.
Pavel

--
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

2005-01-24 09:47:50

by Kirill Korotaev

[permalink] [raw]
Subject: Re: possible CPU bug and request for Intel contacts

Hello Rohit,

> Thanks for sending the detailed information. Based on our experiments
> and analysis, we believe at this point that this is a known E80 issue
> mentioned in the PIII spec update at this location
> (http://www.intel.com/design/pentiumiii/specupdt/24445351.pdf)

> Could you please try one of the suggested work arounds for this issue.
Yes, double cr3 reload and cpuid helps us. But rdtsc doesn't.

BTW, can you explain why making pages non-global is the cure? Is it safe
workaround for this bug?
Double cr3 reload looks a bit unsafe to me, since interrupts or NMI can
occur between the reloads and probably reuse stale iTLB mappings... But
I'm not sure about this since it is much harder to catch and I have no
access to CPU internals. What is your opinion about this?

Sorry for taking your time, I should have checked ERRATAs more
attentively myself.

Thank you,
Kirill

>>Hello,
>>
>>Here are the details about CPU bug I mentioned in my previous post.
>>Though it turned out later that it happens on P-III systems only I
>>still
>>hope it can be of interest.
>>
>>Brief description
>>~~~~~~~~~~~~~~~~~
>>
>>This issue was found by Vasily Averin ([email protected]) when playing
>>with uselib security exploit on kernels with my 4gb split patch.
>>
>>This bug results in strange effects such as calltraces below,
>>reboots, impossible call traces and so on.
>>
>>I started to resolve the bug, narrowed down uselib exploit and
>>got a simple testcase for the bug, which can be found in attach.
>>This testcase does a simple thing - it maps pages at low addresses
>>from 0x04000000 downto 0x00000000, page by page and touches them
>>for write. Sometimes when running this exploit I got oopses,
>>sometimes reboots and I found that this is sensitive to the page
>>addresses which exploit maps.
>>
>>Why it crashes? I think this is due to virtual addresses of
>>kernel code and mapped user space pages overlap. I was able even to
>>reboot machine if mapped user space pages were filled with some
>>appropriate asm code.
>>
>>I found that Ingo Molnar 4gb split is not vulnerable, and after
>>investigations I found that Ingo patch doesn't map kernel entry code
>>(trampline) as _PAGE_GLOBAL. This was the answer.
>>
>>I tested it on 4 different P-III machines - all of them were
>>vulnerable.
>>But lately I tested it on Celeron 2.4Ghz and P4 systems - it doesn't
>>happen, so this bug can be of low interest to Intel people :(
>>
>>Below you can find the way how to reproduce the bug, call traces
>>and why I think it's a hardware bug.
>>
>>How to reproduce a bug
>>~~~~~~~~~~~~~~~~~~~~~~
>>
>>- take any FedoraCore kernel with Ingo Molnar 4gb split patch
>> or mainstream kernel and apply 4GB split patch
>>- apply attached diff-arch-4gb-global patch to make
>> trampline code to be GLOBAL
>>- compile kernel with turned on 4gb split, i.e. CONFIG_X86_4GB=y
>>- boot the kernel and run the attached testcase:
>>
>># while true; do ./4gbtest; done;
>>
>>or
>>
>># ./elflbl -l ./lib -a 0x4000000 (where elflbl is uselib exploit)
>>
>>During each 4-5 test runs I get the following oops:
>>
>>Jan 21 12:15:17 ts Unable to handle kernel NULL pointer dereference at
>>virtual address 000000c0
>>Jan 21 12:15:17 ts printing eip:
>>Jan 21 12:15:17 ts 02114450
>>Jan 21 12:15:17 ts *pde = 00000000
>>Jan 21 12:15:17 ts Oops: 0002
>>Jan 21 12:15:17 ts SMP
>>Jan 21 12:15:17 ts Modules linked in:
>>Jan 21 12:15:17 ts CPU: 0
>>Jan 21 12:15:17 ts EIP: 0060:[<02114450>] Not tainted
>>Jan 21 12:15:17 ts EFLAGS: 00010246 (2.6.8-dev)
>>Jan 21 12:15:17 ts EIP is at sys_mmap2+0x0/0xb0
>>Jan 21 12:15:17 ts eax: 000000c0 ebx: 31524fc4 ecx: 00001000
>>edx: 004ec000
>>Jan 21 12:15:17 ts esi: 00000032 edi: 00000000 ebp: 31524000
>>esp: 31524fc0
>>Jan 21 12:15:17 ts ds: 007b es: 007b ss: 0068
>>Jan 21 12:15:17 ts Process test (pid: 25, threadinfo=31524000
>>task=31f680c0) Jan 21 12:15:17 ts Stack: fffec200 01a2a000 00001000
>>00000003 00000032 00000000 00000000 000000c0
>>Jan 21 12:15:17 ts 0000007b 0000007b 000000c0 08048541 00000073
>>00000282 bffffdcc 0000007b
>>Jan 21 12:15:17 ts Call Trace:
>>Jan 21 12:15:17 ts Code: 55 bd f7 ff ff ff 57 31 ff 56 53 83 ec 18 8b
>>44 24 38 89 c6
>>
>> Unable to handle kernel NULL pointer dereference at virtual address
>>000000c0
>> 02114450
>> *pde = 00000000
>> Oops: 0002
>> CPU: 0
>> EIP: 0060:[<02114450>] Not tainted
>> EFLAGS: 00010246 (2.6.8-dev)
>> eax: 000000c0 ebx: 31524fc4 ecx: 00001000 edx: 004ec000
>> esi: 00000032 edi: 00000000 ebp: 31524000 esp: 31524fc0
>> ds: 007b es: 007b ss: 0068
>> Stack: fffec200 01a2a000 00001000 00000003 00000032 00000000
>>00000000 000000c0
>> 0000007b 0000007b 000000c0 08048541 00000073 00000282
>>bffffdcc 0000007b
>> Call Trace:
>> Code: 55 bd f7 ff ff ff 57 31 ff 56 53 83 ec 18 8b 44 24 38 89 c6
>>
>>
>> >>EIP; 02114450 <sys_mmap2+0/b0> <=====
>>
>> >>ebx; 31524fc4 <pg0+2eff8fc4/fdac0000>
>> >>ebp; 31524000 <pg0+2eff8000/fdac0000>
>> >>esp; 31524fc0 <pg0+2eff8fc0/fdac0000>
>>
>>Code; 02114450 <sys_mmap2+0/b0>
>>00000000 <_EIP>:
>>Code; 02114450 <sys_mmap2+0/b0> <=====
>> 0: 55 push %ebp <=====
>>Code; 02114451 <sys_mmap2+1/b0>
>> 1: bd f7 ff ff ff mov $0xfffffff7,%ebp
>>Code; 02114456 <sys_mmap2+6/b0>
>> 6: 57 push %edi
>>Code; 02114457 <sys_mmap2+7/b0>
>> 7: 31 ff xor %edi,%edi
>>Code; 02114459 <sys_mmap2+9/b0>
>> 9: 56 push %esi
>>Code; 0211445a <sys_mmap2+a/b0>
>> a: 53 push %ebx
>>Code; 0211445b <sys_mmap2+b/b0>
>> b: 83 ec 18 sub $0x18,%esp
>>Code; 0211445e <sys_mmap2+e/b0>
>> e: 8b 44 24 38 mov 0x38(%esp,1),%eax
>>Code; 02114462 <sys_mmap2+12/b0>
>> 12: 89 c6 mov %eax,%esi
>>
>>Why CPU is unable to handle paging request at 0x000000c0? There is no
>>access to
>>this addr in executing code! What has "push %ebp" to do with 0xc0?
>>The answer is that %eax contains 0xc0 and the touched in user space
>>pages contain 4092 zero bytes. And 0x0000 is an opcode for "addl %al,
>>(%eax)".
>>So we see the situation when CPU is executing code from user space
>>pages though we are in kernel space already and data peeks from these
>>addresses
>>shows us the correct code (code in call trace is correct!).
>>I checked it and if these pages are filled with some other values,
>>not zeroes, than it's possible to make CPU execute this code.
>>
>>And why this happens on sys_mmap2+0? Because entry code (system_call)
>>is mapped at high addresses (> 0xffc00000) and is the same both in
>>kernel
>>and user spaces, so entry.S code works ok.
>>
>>So we found 2 ways of curing this bug:
>>- make trampline code to be non-GLOBAL
>>- another observation was that PAE turned ON helps as well.
>>
>>Hypothesis
>>~~~~~~~~~~
>>I think that the problem is in code prefetch queue or somewhere in
>>CPU.
>>It looks like CPU doesn't flush code prefetch queue after %cr3 reload
>>(to kernel space) in entry.S and continues to execute prefetched code
>>from user space pages.
>>
>>Why making entry code non-global helps the problem?
>>I think that if the code at %eip is flushed on %cr3 reload than the
>>_whole_ prefetch queue is flushed and when entry code is global than
>>it is
>>not flushed on %cr3 reload and prefetch queue (including call to
>>flushed sys_mmap2 code) is not flushed.
>>
>>Kirill
>>
>>
>>
>>>Hi Kirill,
>>>
>>>I appreciate you bringing this issue up. Could you please send us
>>>the information on how you are able to reproduce this issue (System
>>>config, Linux kernel version and any test case). We would like to
>>>root cause the failure here at Intel.
>>>
>>>Appreciate your help,
>>>Thanks,
>>>-rohit
>>>
>>>Kirill Korotaev <> wrote on Wednesday, January 19, 2005 8:08 AM:
>>>
>>>
>>>
>>>>Hello Linus,
>>>>
>>>>Linus, Ingo, I've got one strange CPU bug leading to oopses, reboots
>>>>and so on. This bug can be reproduced with a little bit modified 4gb
>>>>split and is probably related to CPU speculative execution. I'll
>>>>post more information about this bug later, but I would like to ask
>>>>you for Intel guys contacts who maybe interested in this
>>>>information, so I could CC them as well.
>>>>
>>>>Thank you,
>>>>Kirill
>>>>
>>>>-
>>>>To unsubscribe from this list: send the line "unsubscribe
>>>>linux-kernel" in the body of a message to [email protected]
>>>>More majordomo info at http://vger.kernel.org/majordomo-info.html
>>>>Please read the FAQ at http://www.tux.org/lkml/
>
>


2005-01-25 07:16:56

by Rohit Seth

[permalink] [raw]
Subject: RE: possible CPU bug and request for Intel contacts

Pavel Machek <mailto:[email protected]> wrote on Saturday, January 22, 2005
2:03 AM:

> Hi!
>
>> Here are the details about CPU bug I mentioned in my previous post.
>> Though it turned out later that it happens on P-III systems only I
>> still hope it can be of interest.
>
> What about Pentium-M? They are based on P-III and are certainly *very*
> interesting.
> Pavel

This issue does not happen on Pentium-M. This issue happens only on
some PIII steppings. Information about the affected stepping is
provided in the spec update link.

http://www.intel.com/design/pentiumiii/specupdt/24445351.pdf

2005-01-25 07:23:57

by Rohit Seth

[permalink] [raw]
Subject: RE: possible CPU bug and request for Intel contacts

Kirill Korotaev <mailto:[email protected]> wrote on Monday, January 24, 2005
1:51 AM:

> Hello Rohit,
>
>> Thanks for sending the detailed information. Based on our experiments
>> and analysis, we believe at this point that this is a known E80 issue
>> mentioned in the PIII spec update at this location
>> (http://www.intel.com/design/pentiumiii/specupdt/24445351.pdf)
>
>> Could you please try one of the suggested work arounds for this
>> issue.
> Yes, double cr3 reload and cpuid helps us. But rdtsc doesn't.
>

I will have to get back to you about rdtsc.


> BTW, can you explain why making pages non-global is the cure? Is it
> safe workaround for this bug?

There is a boundary condition that can have non-global pages containing
the CR3 load to also hit this issue on affected PIII. Though for this
to happen, mov to cr3 has to be the very last instruction on a page.
And the page following that page (containing CR3 load) has to have
different mapping between user and kernel spaces.

> Double cr3 reload looks a bit unsafe to me, since interrupts or NMI
> can
> occur between the reloads and probably reuse stale iTLB mappings...

Interruptions will ensure that stale mapping don't exist in ITLB fill
buffer. So, you are okay with double CR3 laods.

Thanks, rohit

2005-01-25 14:09:28

by Kirill Korotaev

[permalink] [raw]
Subject: Re: possible CPU bug and request for Intel contacts

Hello Rohit,

>>BTW, can you explain why making pages non-global is the cure? Is it
>> safe workaround for this bug?
> There is a boundary condition that can have non-global pages containing
> the CR3 load to also hit this issue on affected PIII. Though for this
> to happen, mov to cr3 has to be the very last instruction on a page.
> And the page following that page (containing CR3 load) has to have
> different mapping between user and kernel spaces.
but in our case "mov %edx, %cr3" is not the last instruction on a page.
It is in the middle of it.
Well, another remark is that after cr3 load there are only few
instructions before the "call system_call_table(%edx)" which references
the page with different user and kernel mappings.

also, this bug can be cured via inserting about 20 simple operations
between cr3 load and call to the page with overlapping mappings.

I'm just trying to understand is it the bug referenced in E80 or not and
is it safe to use non-global mappings as a cure.

Kirill

2005-01-26 01:39:58

by Rohit Seth

[permalink] [raw]
Subject: RE: possible CPU bug and request for Intel contacts

Kirill Korotaev <mailto:[email protected]> wrote on Tuesday, January 25, 2005
6:12 AM:

> Hello Rohit,
>
>>> BTW, can you explain why making pages non-global is the cure? Is it
>>> safe workaround for this bug?

>> There is a boundary condition that can have non-global pages
>> containing the CR3 load to also hit this issue on affected PIII.
>> Though for this to happen, mov to cr3 has to be the very last
>> instruction on a page. And the page following that page (containing
>> CR3 load) has to have different mapping between user and kernel
>> spaces.

> but in our case "mov %edx, %cr3" is not the last instruction on a
> page.
> It is in the middle of it.

So, in this scenario (where trampoline code is mapped by non global
page), we will not hit this issue.

> Well, another remark is that after cr3 load there are only few
> instructions before the "call system_call_table(%edx)" which
> references
> the page with different user and kernel mappings.
>
> also, this bug can be cured via inserting about 20 simple operations
> between cr3 load and call to the page with overlapping mappings.
>

This is not a recommended solution.

> I'm just trying to understand is it the bug referenced in E80 or not
> and
> is it safe to use non-global mappings as a cure.

Our analysis has shown that this is E80 issue. In this 4G-4G kernel
context, we are safe to use non-global mapping as a workaround for this
issue. (Or we can use any of the other recommendations given in the spec
update except rdtsc with global pages).

We have also seen that inserting rdtsc instruction is not a workaround.
We will update the spec update with this information.

On a little different note, while running the 4G-4G kernel on our
machine, we saw occasional hangs. Those are root caused to the fact
that this kernel was first chaging the stack pointer from virtual stack
to kernel and then changing the CR3 to that of kernel. Any interrupt
between these two instructions will result in those hangs as the
interruption handler will execute with user's CR3(as the kernel thinks
that it is already in kernel because of the value of esp). Swapping the
order, first loading the CR3 with kernel and then switching the stack to
kernel fixes this issue. Venki will generate that patch and send to
lkml.

Thanks, rohit

2005-02-13 20:10:26

by Ingo Molnar

[permalink] [raw]
Subject: Re: possible CPU bug and request for Intel contacts


* Seth, Rohit <[email protected]> wrote:

> On a little different note, while running the 4G-4G kernel on our
> machine, we saw occasional hangs. Those are root caused to the fact
> that this kernel was first chaging the stack pointer from virtual
> stack to kernel and then changing the CR3 to that of kernel. Any
> interrupt between these two instructions will result in those hangs as
> the interruption handler will execute with user's CR3(as the kernel
> thinks that it is already in kernel because of the value of esp).
> Swapping the order, first loading the CR3 with kernel and then
> switching the stack to kernel fixes this issue. Venki will generate
> that patch and send to lkml.

i'm not sure what you mean. Here's the relevant 4:4 code from Fedora:

#define __SWITCH_KERNELSPACE \
...
movl %edx, %cr3; \
movl %ebx, %esp; \

i.e. we _first_ load cr3 with the kernel pagetable value, then do we
switch esp to the real kernel stack.

Ingo

2005-02-14 16:56:38

by Pallipadi, Venkatesh

[permalink] [raw]
Subject: RE: possible CPU bug and request for Intel contacts


>-----Original Message-----
>From: Ingo Molnar [mailto:[email protected]]
>Sent: Sunday, February 13, 2005 12:10 PM
>To: Seth, Rohit
>Cc: Kirill Korotaev; Linus Torvalds; Saxena, Sunil; Pallipadi,
>Venkatesh; Andrey Savochkin; [email protected]
>Subject: Re: possible CPU bug and request for Intel contacts
>
>
>* Seth, Rohit <[email protected]> wrote:
>
>> On a little different note, while running the 4G-4G kernel on our
>> machine, we saw occasional hangs. Those are root caused to the fact
>> that this kernel was first chaging the stack pointer from virtual
>> stack to kernel and then changing the CR3 to that of kernel. Any
>> interrupt between these two instructions will result in
>those hangs as
>> the interruption handler will execute with user's CR3(as the kernel
>> thinks that it is already in kernel because of the value of esp).
>> Swapping the order, first loading the CR3 with kernel and then
>> switching the stack to kernel fixes this issue. Venki will generate
>> that patch and send to lkml.
>
>i'm not sure what you mean. Here's the relevant 4:4 code from Fedora:
>
>#define __SWITCH_KERNELSPACE \
>...
> movl %edx, %cr3; \
> movl %ebx, %esp; \
>
>i.e. we _first_ load cr3 with the kernel pagetable value, then do we
>switch esp to the real kernel stack.
>

Yes. I verified that and that's the reason I didn't send any patch. But,

the kernel we were using in our testing of this bug, came from some
earlier version of 4:4 code and had this cr3 switch and esp switch in
reverse order.

With the latest kernels there is no issue related to this.

Thanks,
Venki