On Tue, 3 Oct 2006, Andi Kleen wrote:
>
> > void mxcsr_feature_mask_init(void)
> > {
> > unsigned long mask = 0;
> > clts();
> > if (cpu_has_fxsr) {
> > memset(¤t->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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
