On Dec 12, 2007 12:28 AM, <[EMAIL PROTECTED]> wrote: > > > > Hello, > > Yesterday I wrote some code for the Syslinux boot loader that is meant to > boot different images based on which revision of the CPU microcode that is > currently loaded in the CPU (yes, this is a bit strange but we have a very > good reason for it). I then compared my implementation with a few others an > ran across yours, in > http://www.openbios.org/viewvc/trunk/LinuxBIOSv2/src/cpu/intel/microcode/microcode.c?view=markup > > Question to you guys: why is the first wrmsr instruction there? From my > understanding, by not properly initialising ECX, EAX and EDX this will > overwrite whatever is in the MSR pointed to by ECX?! > > BTW I tried out your code on our target hardware (Intel Celeron M, 600 MHz) > and with that first wrmsr line in place it hangs and without it, it runs > just fine. >
Thanks Martin. That looks like quite a nice bug catch you've done :-) We'll have to figure out if this code was ever (a) used and (b) worked :-) Thanks again, this is great! ron -- linuxbios mailing list [email protected] http://www.linuxbios.org/mailman/listinfo/linuxbios
