2008/7/16, Kevin O'Connor kevin@koconnor.net:
and added this at the end of post_coreboot() function: asm( "movl %esi, %esp\n" "retl" );
That's dangerous because the compiler might use %esi in the post_coreboot() function. If you really want to do this, you should move %esp to %eax and declare the stack pointer as an argument to post_coreboot() and then pass the argument to the inline asm.
I see. I will change this.
Zhang Rui