ron minnich wrote:
On Dec 29, 2007 9:00 PM, joe@smittys.pointclark.net wrote:
Quoting Corey Osgood corey.osgood@gmail.com:
Acked-by: Corey Osgood corey.osgood@gmail.com
how long has it been since we've updated the microcode updates?
No idea, but remember the email below from 12/12/07? Looks like an update is in order, but I wouldn't know where to start.
Thanks - Joe
Quoting Martin.Karlsson@emerson.com:
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.
Just wanted to let you know.
Best regards, Martin Karlsson
-- linuxbios mailing list linuxbios@linuxbios.org http://www.linuxbios.org/mailman/listinfo/linuxbios
I submitted the patch to remove this extraneous wrmsr a while back. I can't recall if it was acked and committed, but I thought it was.
No, this doesn't seem to have been taken care of yet. Is it read_microcode_rev() or intel_update_microcode() that needs the wrmsr removed?
Microcode updates coming in the morning, if the abuild passes.
-Corey