Re: [coreboot] [LinuxBIOS] Intel microcode revision code

[Stefan Reinauer]

* ron minnich  [071212 17:19]:
> > 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 :-)

Here's a patch that resolves the issue.