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

2008-04-18 Thread Nguyen Anh Quynh
On Thu, Apr 17, 2008 at 2:58 PM, H. Peter Anvin [EMAIL PROTECTED] wrote: 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

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

2008-04-18 Thread Nguyen Anh Quynh
On Thu, Apr 17, 2008 at 3:00 PM, H. Peter Anvin [EMAIL PROTECTED] wrote: + .globl linux_boot +linux_boot: + cli + cld + mov $0x9000, %ax + mov %ax, %ds + mov %ax, %es + mov %ax, %fs + mov %ax, %gs + mov %ax, %ss +

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

2008-04-18 Thread Nguyen Anh Quynh
On Thu, Apr 17, 2008 at 4:36 PM, Carlo Marcelo Arenas Belon [EMAIL PROTECTED] wrote: On Thu, Apr 17, 2008 at 10:30:27AM +0900, Nguyen Anh Quynh wrote: +++ b/extboot/farvar.h @@ -0,0 +1,113 @@ +// Code to access multiple segments within gcc. +// +// Copyright (C) 2008 Kevin O'Connor

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

2008-04-18 Thread H. Peter Anvin
Nguyen Anh Quynh wrote: Actually, this code is left from the original code of Anthony, and it seems he took it from qemu 0.8 version. Anthony, may you explain why you want to hijact the linux boot process here? If I understand correctly, we can just let the original int19 execute, and if

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

2008-04-18 Thread H. Peter Anvin
Nguyen Anh Quynh wrote: Actually, this code is left from the original code of Anthony, and it seems he took it from qemu 0.8 version. Anthony, may you explain why you want to hijact the linux boot process here? If I understand correctly, we can just let the original int19 execute, and if

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

2008-04-18 Thread Anthony Liguori
Nguyen Anh Quynh wrote: On Thu, Apr 17, 2008 at 3:00 PM, H. Peter Anvin [EMAIL PROTECTED] wrote: + .globl linux_boot +linux_boot: + cli + cld + mov $0x9000, %ax + mov %ax, %ds + mov %ax, %es + mov %ax, %fs + mov %ax, %gs + mov

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

2008-04-18 Thread Anthony Liguori
H. Peter Anvin wrote: Nguyen Anh Quynh wrote: Actually, this code is left from the original code of Anthony, and it seems he took it from qemu 0.8 version. Anthony, may you explain why you want to hijact the linux boot process here? If I understand correctly, we can just let the original

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

2008-04-18 Thread Nguyen Anh Quynh
On 4/18/08, Anthony Liguori [EMAIL PROTECTED] wrote: Nguyen Anh Quynh wrote: On Thu, Apr 17, 2008 at 3:00 PM, H. Peter Anvin [EMAIL PROTECTED] wrote: + .globl linux_boot +linux_boot: + cli + cld + mov $0x9000, %ax + mov %ax, %ds

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

2008-04-18 Thread H. Peter Anvin
Anthony Liguori wrote: The thinking is to eliminate the need to hijack the boot sector when using the -kernel option. However, the linux boot stuff in extboot has been broken since hpa rewrote the boot code. It can be removed for now and I'll eventually revisit it. It probably makes

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

2008-04-18 Thread H. Peter Anvin
Anthony Liguori wrote: Nguyen Anh Quynh wrote: The thinking is to eliminate the need to hijack the boot sector when using the -kernel option. I see, but does that offer any advantage over the current approach? You no longer have to specify a -hda option when using -kernel.

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

2008-04-18 Thread Nguyen Anh Quynh
On 4/18/08, Anthony Liguori [EMAIL PROTECTED] wrote: Nguyen Anh Quynh wrote: The thinking is to eliminate the need to hijack the boot sector when using the -kernel option. I see, but does that offer any advantage over the current approach? You no longer have to

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

2008-04-18 Thread Anthony Liguori
Nguyen Anh Quynh wrote: You no longer have to specify a -hda option when using -kernel. Without -hda, how can we load disk image? Or you mean you only want to test the kernel? Right. You may be booting from NFS, iSCSI, or something like that. Regards, Anthony Liguori Thanks,

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

2008-04-17 Thread H. Peter Anvin
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 [EMAIL PROTECTED] +

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

2008-04-17 Thread H. Peter Anvin
+ .globl linux_boot +linux_boot: + cli + cld + mov $0x9000, %ax + mov %ax, %ds + mov %ax, %es + mov %ax, %fs + mov %ax, %gs + mov %ax, %ss + mov $0x8ffe, %sp + ljmp $0x9000 + 0x20, $0 The hard use of segment 9000 is really highly unfortunate

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

2008-04-17 Thread Carlo Marcelo Arenas Belon
On Thu, Apr 17, 2008 at 10:30:27AM +0900, Nguyen Anh Quynh wrote: +++ b/extboot/farvar.h @@ -0,0 +1,113 @@ +// Code to access multiple segments within gcc. +// +// Copyright (C) 2008 Kevin O'Connor [EMAIL PROTECTED] +// +// This file may be distributed under the terms of the GNU GPLv3 license.

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

2008-04-16 Thread Nguyen Anh Quynh
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 [EMAIL PROTECTED] # diffstat extboot3.diff