cvs commit: src/sys/amd64/amd64 fpu.c

[David Schultz]

On Mon, Jun 07, 2004, David Malone wrote:
> On Sun, Jun 06, 2004 at 02:17:58AM -0700, David Schultz wrote:
> > It would be great if someone could pick this up and devise the
> > appropriate fix.  I want to see this bug fixed, but I don't have the
> > hardware to do amd64 kernel hacking.  I'm happy to help anyone who is
> > interested.
> 
> I think I've figured out the problem. On SMP systems, fpuinit() is
> called before enable_sse() for secondary processors. The ldmxcsr
> instruction counts as a sse instruction, so you get an illegal
> instruction fault. The patch below switches the order of fpuinit()
> and enable_sse() and fixes the problem on my system anyway.

Nice catch.  That would certainly explain the problem, and both of
the machines with reported problems (yours and sledge) are SMP.
Care to commit this?