lkml.org 
[lkml]   [2006]   [Oct]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH/RFC] Math-emu kills the kernel on Athlon64 X2


On Tue, 3 Oct 2006, Andi Kleen wrote:

>
> > void mxcsr_feature_mask_init(void)
> > {
> > unsigned long mask = 0;
> > clts();
> > if (cpu_has_fxsr) {
> > memset(&current->thread.i387.fxsave, 0, sizeof(struct i387_fxsave_struct));
> > asm volatile("fxsave %0" : : "m" (current->thread.i387.fxsave));
> > mask = current->thread.i387.fxsave.mxcsr_mask;
> > if (mask == 0) mask = 0x0000ffbf;
> > }
>
> This just needs an ifdef. I'll fix that thanks.

No, it doesn't need "an ifdef" at all.

There is no #define to even _test_. The fact that FPU math emulation is
compiled in in _no_ way means that it should statically be used. It just
means that if the user asks the math coprocessor to not be used, or if no
such coprocessor exists, we shouldn't do an "fxsave".

The real fix is to clear "fxsr" when the user said "no387". If you don't
have a math coprocessor, you sure as heck don't have fxsr either.

However, that's apparently not enough, since "nofxsr" was reported to not
fix it entirely. However, that might well be true due to just handling
that flag too late, or something.

And THAT is the real bug. Don't try to make it anything else. The bug is
simply that we use the math coprocessor when we've been told it doesn't
exist.

Linus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2009-11-18 23:46    [W:0.074 / U:0.036 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site