On 4/23/08, H. Peter Anvin <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
>
> > Hi,
> >
> > I am thinking about comibing this ROM with the extboot. Both two ROM
> > are about "booting", so I think that is reasonable. So we will have
> > o
On 4/22/08, Anthony Liguori <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
>
> > Hi,
> >
> > This should be submitted to upstream (but not to kvm-devel list), but
> > this is only the test code that I want to quickly send out for
> > comments. In ca
On 4/22/08, Alexander Graf <[EMAIL PROTECTED]> wrote:
> I believe that's the way to go. If you have spare time on your hands, feel
> free to integrate my multiboot patches as well.
>
OK, that looks straightforward enough.
Thanks,
Q
>
> On Apr 22, 2008, at 11:07
Hi,
I am thinking about comibing this ROM with the extboot. Both two ROM
are about "booting", so I think that is reasonable. So we will have
only 1 ROM that supports both external boot and Linux boot.
Is that desirable or not?
Thanks,
Quynh
On 4/21/08, Nguyen Anh Quynh <[EM
On Tue, Apr 22, 2008 at 12:59 AM, Damjan <[EMAIL PROTECTED]> wrote:
> > > If I try
> > > $ qemu -kernel minimal-kernel -drive file=jeos-devel.img,if=virtio
> > > I get the following error:
> > > A disk image must be given for 'hda' when booting a Linux kernel
> > > is this neccesseary?
>
>
/hw/linuxboot.c | 39 +++
qemu/hw/pc.c | 22 +++-
qemu/hw/pc.h |5 +
12 files changed, 600 insertions(+), 9 deletions(-)
On Mon, Apr 21, 2008 at 12:33 PM, Nguyen Anh Quynh <[EMAIL PROTECTED]> wrote:
> Forget to say that this patch is again
Forget to say that this patch is against kvm-66.
Thanks,
Q
On Mon, Apr 21, 2008 at 12:32 PM, Nguyen Anh Quynh <[EMAIL PROTECTED]> wrote:
> Hi,
>
> This should be submitted to upstream (but not to kvm-devel list), but
> this is only the test code that I want to quickly send o
/linuxboot.c | 39 +++
qemu/hw/pc.c | 22 +++-
qemu/hw/pc.h |5 +
13 files changed, 600 insertions(+), 9 deletions(-)
commit f4f1178898c8a4bbbc0a432354dbcc56353099c3
Author: Nguyen Anh Quynh <[EMAIL PROTECTED]>
Date: Mon Apr 21 12:27:47 2008
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.
> > >
> > >
> >
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
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.
> +//
> +/
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
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 erro
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
b/e
On Thu, Apr 17, 2008 at 12:02 AM, Anthony Liguori <[EMAIL PROTECTED]> wrote:
> A couple general comments.
>
> I'd feel a lot more comfortable with the int13 handler returning an int and
> the asm stub code uses that result to determine how to set CF. You set CF
> deep within the function stack an
Hi Anthony,
I found a bug in the last code: send_command() failed to copy back the
result into extboot_cmd structure. This patch fixes it.
I succesfully tested this version with guest Win2K (fully updated,
scsi boot) and Linux 2.6.25-rc8 (virtio).
Let me know if you can boot Windows with this ve
On 4/15/08, Nguyen Anh Quynh <[EMAIL PROTECTED]> wrote:
> > You should be able to get very far along the Windows boot process with
> > extboot. If you just install the LSI driver in Win2k, you can boot up
> > completely. Someone just posted instructions for doing scs
On 4/15/08, Anthony Liguori <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
>
> > On 4/15/08, Anthony Liguori <[EMAIL PROTECTED]> wrote:
> >
> >
> > > Hi Nguyen,
> > >
> > > Nguyen Anh Quynh wrote:
> > >
> > >
On 4/15/08, Anthony Liguori <[EMAIL PROTECTED]> wrote:
> Hi Nguyen,
>
> Nguyen Anh Quynh wrote:
>
> > Hi Anthony,
> >
> > I spot a bug and few dead code in the extboot option rom. Perhaps the
> > reason they are there is because less people want to look
Hi Anthony,
I spot a bug and few dead code in the extboot option rom. Perhaps the
reason they are there is because less people want to look at assembly
code, and it looks indeed scary.
So I decided to rewrite it in C, using .code16gcc directive. Some advantages:
- C code is easier to understand,
On 4/14/08, Anthony Liguori <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
>
> >
> > This patch fixes a wrong calculation in the physical address of
> > extboot (qemu/hw/extboot.c). Looks like a typo.
> >
> > Signed-off-by: Nguyen Anh Quynh <
This patch fixes a wrong calculation in the physical address of
extboot (qemu/hw/extboot.c). Looks like a typo.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
diff --git a/qemu/hw/extboot.c b/qemu/hw/extboot.c
index 056fb59..0dac34c 100644
--- a/qemu/hw/extboot.c
+++ b/qemu/hw/ext
On 7/25/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
> > On 7/24/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> >> Nguyen Anh Quynh wrote:
> >> > This patch removes some unused functions in mmu.c
> >>
> >> > -static
On 7/24/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
> > This patch removes some unused functions in mmu.c
>
> > -static int is_nx(struct kvm_vcpu *vcpu)
> > -{
> > - return vcpu->shadow_efer & EFER_NX;
> > -}
> > -
This patch removes some unused functions in mmu.c
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
diff --git a/drivers/kvm/mmu.c b/drivers/kvm/mmu.c
index a7985b1..13fb6bb 100644
--- a/drivers/kvm/mmu.c
+++ b/drivers/kvm/mmu.c
@@ -156,21 +156,6 @@ static struct kmem_cache *pte_chain
On 7/23/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
> > This patch makes "install" compile code, and "install-rpm" package rpms.
>
> 'install' and 'install-rpm' are typically run as root, and we don'
This patch makes "install" compile code, and "install-rpm" package rpms.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
diff --git a/Makefile b/Makefile
index 37b01d2..dd3f51c 100644
--- a/Makefile
+++ b/Makefile
@@ -22,7 +22,7 @@ initdir = /etc/init.d
confdir =
kvmctl.c: The first param of mmap() should be NULL rather than 0.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
diff --git a/user/kvmctl.c b/user/kvmctl.c
index 3b0d8bc..43b374d 100644
--- a/user/kvmctl.c
+++ b/user/kvmctl.c
@@ -222,7 +222,7 @@ int kvm_create_vcpu(kvm_context_t kv
On 7/19/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
> > Currently, when handling hardware breakpoints, we always set values
> > for all the 4 hardware debug registers, regardless it is needed or
> > not. This patch fixes the bug.
>
> Why
Currently, when handling hardware breakpoints, we always set values
for all the 4 hardware debug registers, regardless it is needed or
not. This patch fixes the bug.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
diff --git a/drivers/kvm/kvm.h b/drivers/kvm/kvm.h
index 7bdffe6..c
s when installing
this avoids an ambiguity when using the kvm external module in conjuction
with 2.6.20 or later, which may also have kvm built.
This patch raises a problem if
/lib/modules//kernel/drivers/kvm/ does not exist.
The enclosed patch fixes the bug.
Signed-off-by: Nguyen
On 7/11/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
> > *nopage() in kvm_main.c should only store the type of mmap() fault if
> > the pointers are not NULL. This patch fixes the problem.
> >
>
> What caller of ->nopage() passes a NULL
*nopage() in kvm_main.c should only store the type of mmap() fault if
the pointers are not NULL. This patch fixes the problem.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
diff --git a/drivers/kvm/kvm_main.c b/drivers/kvm/kvm_main.c
index 83bb284..0520d15 100644
--- a/drive
patch is fine for you, or how do you want to take
patch from gmail.
On 6/4/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
Nguyen Anh Quynh wrote:
> "memslot" in makr_page_dirty() should be verified before dereferencing
> it (kvm_main.c)
The patches still don't show up inline
Hi Avi,
Perhaps I miss something, but I dont see you initialize ncpus anywhere
in the patch?
Cheers,
Quynh
On 6/5/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> repository: /home/avi/kvm
> branch: master
> commit 9d02d1c4345c77ffbc95136c12011a112a8d1c7e
> Author: Avi Kivity <[EMAIL PROTECTED]>
> Da
"memslot" in makr_page_dirty() should be verified before dereferencing
it (kvm_main.c)
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch9.patch
Description: Binary data
-
This SF.net email is
This patch removes a dead struct in kvmctl.c
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch8.patch
Description: Binary data
-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - th
This patch uses symbol rather than a constant in kvm_show_code(), so
more readable :-)
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch7.patch
Description: Binary data
-
This SF.net email is sponsored
On 6/3/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
> > On 6/3/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> >> Nguyen Anh Quynh wrote:
> >> > This patch replaces 2 constant variables "dosmem" and "exmem" in
>
This patch makes kvm_get_msr_list() returns NULL rather than plain 0
in error cases.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch6.patch
Description: Binary data
-
This SF.net email is sponsored by DB2 E
This patch turnes a local function in kvmctl.c static.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch5.patch
Description: Binary data
-
This SF.net email is sponsored by DB2 Express
Download DB2 Express C
On 6/3/07, Avi Kivity <[EMAIL PROTECTED]> wrote:
> Nguyen Anh Quynh wrote:
> > This patch replaces 2 constant variables "dosmem" and "exmem" in
> > kvm_create() with symbols.
>
> What is the benefit of this? Readability is not improved.
>
>
This patch moves a comment in kvmctl.h to the right place, and fixes
some stuffs for doxygen.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch4.patch
Description: Binary data
-
This SF.net email is sponso
This patch replaces 2 constant variables "dosmem" and "exmem" in
kvm_create() with symbols.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch3.patch
Description: Binary data
-
This SF.net
This patch makes hypercall.c use symbol instead of number with
hypercall-ioport-size.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch2
Description: Binary data
-
This SF.net email is sponsored by DB2 E
This patch turns some local functions and variables in hypercall.c static.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
patch1
Description: Binary data
-
This SF.net email is sponsored by DB2 Express
Downlo
sometime because the old modules (which is available in 2.6.20) is
not supported by the current libkvm.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
# diffstat patch2.patch
Makefile |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/Makefile b/kernel/Makefile
This patch clarifies some "old version" error message, and deletes
some unused functions in userspace code.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
diffstat patch1.patch
kvmctl.c | 36 +++-
1 file changed, 3 insertions(+), 33 deletio
Hi,
This patch renames few MSR registers in vmx.h and did few trivial cleanups.
Signed-off-by: Nguyen Anh Quynh <[EMAIL PROTECTED]>
$ diffstat msr.patch
vmx.c | 13 ++---
vmx.h | 10 +-
2 files changed, 11 insertions(+), 12 deletions(-)
Index: kernel
49 matches
Mail list logo