Re: [kvm-devel] Extboot Option ROM rewritten in C - v3

[H. P. A. <hp...@zy...>]

Nguyen Anh Quynh wrote:
> This patch replaces the current assembly code of Extboot option rom
> with new C code. Patch is against kvm-66.
> 
> This version returns an error code in case int 13 handler cannot
> handle a requested function.
> 
> Signed-off-by: Nguyen Anh Quynh <aq...@gm...>
> 

+	/* -fomit-frame-pointer might clobber %ebp */
+	pushl %ebp
+	call setup
+	popl  %ebp


No, it might not.  %ebx, %ebp, %esi, and %edi are guaranteed preserved; 
%eax, %ecx and %edx are clobbered.

It's also prudent to call cld before jumping to C code.

	-hpa