Re: [kvm-devel] kvm errors when using greater than 3572m of memory

2007-12-07 Thread Brian Almeida
On Fri, Dec 07, 2007 at 09:28:42PM +0200, Izik Eidus wrote: > (note that kvm-54 should be enough, you just have to make sure that you use > the modules that come with it) Yes, this fixed the problem. Any idea why the ACPI opitmization patches (tpr-opt-1) are slated to make it in an upstream rele

Re: [kvm-devel] [PATCH] Use cmpxchg for pte updates on walk_addr()

2007-12-07 Thread Marcelo Tosatti
On Fri, Dec 07, 2007 at 07:06:26AM +0200, Avi Kivity wrote: > Marcelo Tosatti wrote: > >Right, patch at end of the message restarts the process if the pte > >changes under the walker. The goto is pretty ugly, but I fail to see any > >elegant way of doing that. Ideas? > > > > > > goto is fine for

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 1 of 3] export SMBIOS/DMI tables to PC machines

2007-12-07 Thread Anthony Liguori
Ryan Harper wrote: > * Anthony Liguori <[EMAIL PROTECTED]> [2007-12-07 15:07]: > >> Ryan Harper wrote: >> >>> * Anthony Liguori <[EMAIL PROTECTED]> [2007-12-07 14:49]: >>> >>> > + > +CPUState *first_cpu; > > > This should probably be static

Re: [kvm-devel] [Qemu-devel] Re: [PATCH 1 of 3] export SMBIOS/DMI tables to PC machines

2007-12-07 Thread Ryan Harper
* Anthony Liguori <[EMAIL PROTECTED]> [2007-12-07 15:07]: > Ryan Harper wrote: > >* Anthony Liguori <[EMAIL PROTECTED]> [2007-12-07 14:49]: > > > >>>+ > >>>+CPUState *first_cpu; > >>> > >>> > >>This should probably be static. > >> > > > >doh, should be extern actually since it's define

Re: [kvm-devel] [PATCH 1 of 3] export SMBIOS/DMI tables to PC machines

2007-12-07 Thread Anthony Liguori
Ryan Harper wrote: > * Anthony Liguori <[EMAIL PROTECTED]> [2007-12-07 14:49]: > >>> + >>> +CPUState *first_cpu; >>> >>> >> This should probably be static. >> > > doh, should be extern actually since it's defined elsewhere. > Perhaps you want cpu_single_env instead of first_cpu

Re: [kvm-devel] [PATCH 1 of 3] export SMBIOS/DMI tables to PC machines

2007-12-07 Thread Ryan Harper
* Anthony Liguori <[EMAIL PROTECTED]> [2007-12-07 14:49]: > Ryan Harper wrote: > >5 files changed, 754 insertions(+), 2 deletions(-) > >Makefile.target |4 > >hw/pc.c | 47 > >smbios.c| 519 > >+++ > >smbios_types.h |

Re: [kvm-devel] [PATCH 3 of 3] Add -uuid command line flag

2007-12-07 Thread Anthony Liguori
Ryan Harper wrote: > 2 files changed, 29 insertions(+), 1 deletion(-) > smbios.c | 12 +++- > vl.c | 18 ++ > > > # HG changeset patch > # User Ryan Harper <[EMAIL PROTECTED]> > # Date 1197058922 21600 > # Node ID 5fe703a5a7bde701686fc333c17836b308c17b4f > # Parent 34

Re: [kvm-devel] [PATCH 1 of 3] export SMBIOS/DMI tables to PC machines

2007-12-07 Thread Anthony Liguori
Ryan Harper wrote: > 5 files changed, 754 insertions(+), 2 deletions(-) > Makefile.target |4 > hw/pc.c | 47 > smbios.c| 519 +++ > smbios_types.h | 182 +++ > sysemu.h|4 > > > # HG changese

[kvm-devel] [PATCH 2 of 3] Optionally link against libuuid if present

2007-12-07 Thread Ryan Harper
3 files changed, 38 insertions(+), 2 deletions(-) Makefile.target |5 - configure | 26 ++ smbios.c|9 - # HG changeset patch # User Ryan Harper <[EMAIL PROTECTED]> # Date 1197058922 21600 # Node ID 3470dd05f46cc9f14c3fc9561d06031a4bc0ce7a

[kvm-devel] [PATCH 3 of 3] Add -uuid command line flag

2007-12-07 Thread Ryan Harper
2 files changed, 29 insertions(+), 1 deletion(-) smbios.c | 12 +++- vl.c | 18 ++ # HG changeset patch # User Ryan Harper <[EMAIL PROTECTED]> # Date 1197058922 21600 # Node ID 5fe703a5a7bde701686fc333c17836b308c17b4f # Parent 3470dd05f46cc9f14c3fc9561d06031a4bc0ce7

[kvm-devel] [PATCH 1 of 3] export SMBIOS/DMI tables to PC machines

2007-12-07 Thread Ryan Harper
5 files changed, 754 insertions(+), 2 deletions(-) Makefile.target |4 hw/pc.c | 47 smbios.c| 519 +++ smbios_types.h | 182 +++ sysemu.h|4 # HG changeset patch # User Ryan Harper <[EMAIL

[kvm-devel] [PATCH 0 of 3] Add SMBIOS/DMI table generation to PC machine

2007-12-07 Thread Ryan Harper
This patchset introduces SMBIOS/DMI table generation to qemu for PC machines. The intial patch includes all changes needed to create the tables and load them into memory. This patch depends on libuuid. The subsequent patches detect for libuuid and optionally link to the library if present. The

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Luca Tettamanti
On Dec 7, 2007 7:38 PM, Anthony Liguori <[EMAIL PROTECTED]> wrote: > Izik Eidus wrote: > > Anthony Liguori wrote: > >> Glauber de Oliveira Costa wrote: > >> > >>> Currently, the -kernel option is not working. > >>> > >>> Reason is, because we're registering chunks for regions 0-0xa and > >>> 0x

[kvm-devel] kvm errors when using greater than 3572m of memory

2007-12-07 Thread Brian Almeida
Hi, I get the following error when I give -m 3752 or higher: stormcrow:/data/kvm# kvm -no-acpi -m 3572 -cdrom /data/iso/w2k3_server_ent_r2_disc1.iso -boot d -usbdevice tablet tharkun.img create_kernel_phys_mem: Invalid argumentset_vram_mapping: cannot allocate memory: Invalid argument NULL vram_

Re: [kvm-devel] kvm errors when using greater than 3572m of memory

2007-12-07 Thread Izik Eidus
Brian Almeida wrote: > On Fri, Dec 07, 2007 at 09:14:37PM +0200, Izik Eidus wrote: > >> yea it does look like the reason is that you use old kvm module version, >> i ran it on my box with 3572 mb of ram and it work >> > > Ok - I will try with the latest version. Thanks! > > Brian > (not

Re: [kvm-devel] kvm errors when using greater than 3572m of memory

2007-12-07 Thread Brian Almeida
On Fri, Dec 07, 2007 at 09:14:37PM +0200, Izik Eidus wrote: > yea it does look like the reason is that you use old kvm module version, > i ran it on my box with 3572 mb of ram and it work Ok - I will try with the latest version. Thanks! Brian

Re: [kvm-devel] kvm errors when using greater than 3572m of memory

2007-12-07 Thread Izik Eidus
Izik Eidus wrote: > Brian Almeida wrote: >> Hi, >> >> I get the following error when I give -m 3752 or higher: >> >> stormcrow:/data/kvm# kvm -no-acpi -m 3572 -cdrom >> /data/iso/w2k3_server_ent_r2_disc1.iso -boot d -usbdevice tablet >> tharkun.img create_kernel_phys_mem: Invalid argumentset_vram_m

Re: [kvm-devel] kvm errors when using greater than 3572m of memory

2007-12-07 Thread Izik Eidus
Brian Almeida wrote: > Hi, > > I get the following error when I give -m 3752 or higher: > > stormcrow:/data/kvm# kvm -no-acpi -m 3572 -cdrom > /data/iso/w2k3_server_ent_r2_disc1.iso -boot d -usbdevice tablet > tharkun.img > create_kernel_phys_mem: Invalid argumentset_vram_mapping: cannot > allocat

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Izik Eidus
Anthony Liguori wrote: > Izik Eidus wrote: >> Anthony Liguori wrote: >>> Glauber de Oliveira Costa wrote: >>> Currently, the -kernel option is not working. Reason is, because we're registering chunks for regions 0-0xa and 0x10-ram_size, the phys_ram_addr + PA is broken

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Anthony Liguori
Izik Eidus wrote: > Anthony Liguori wrote: >> Glauber de Oliveira Costa wrote: >> >>> Currently, the -kernel option is not working. >>> >>> Reason is, because we're registering chunks for regions 0-0xa and >>> 0x10-ram_size, the phys_ram_addr + PA is broken. >>> >> >> I think this

Re: [kvm-devel] [PATCH] Use cmpxchg for pte updates on walk_addr()

2007-12-07 Thread Andrea Arcangeli
On Fri, Dec 07, 2007 at 07:56:58AM -0500, Marcelo Tosatti wrote: > I see that possibility, but why on earth would the guest be continuously > updating a pagetable entry ? If I understood correctly this would be just to be more robust against malicious guests that could try to create unkillable tig

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Izik Eidus
Glauber de Oliveira Costa wrote: > Currently, the -kernel option is not working. > > Reason is, because we're registering chunks for regions 0-0xa and > 0x10-ram_size, the phys_ram_addr + PA is broken. > The real fix should be to rewrite all the load_linux() code to not rely > on this, but

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Izik Eidus
Anthony Liguori wrote: > Glauber de Oliveira Costa wrote: > >> Currently, the -kernel option is not working. >> >> Reason is, because we're registering chunks for regions 0-0xa and >> 0x10-ram_size, the phys_ram_addr + PA is broken. >> >> > > I think this patch is a reasonable wo

Re: [kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Anthony Liguori
Glauber de Oliveira Costa wrote: > Currently, the -kernel option is not working. > > Reason is, because we're registering chunks for regions 0-0xa and > 0x10-ram_size, the phys_ram_addr + PA is broken. > I think this patch is a reasonable work-around but the long term solution is proba

[kvm-devel] [PATCH] fix -kernel option

2007-12-07 Thread Glauber de Oliveira Costa
Currently, the -kernel option is not working. Reason is, because we're registering chunks for regions 0-0xa and 0x10-ram_size, the phys_ram_addr + PA is broken. The real fix should be to rewrite all the load_linux() code to not rely on this, but meanwhile, filling in the gap up to 0xc

[kvm-devel] kvm-55 test

2007-12-07 Thread Farkas Levente
hi, after we test kvm-55 no regression found to kvm-54, but 32bit guest on a 64bit host still not working (but now the boot process hang earlier the starting udev). another strange thing that win xp with 4vcpu guest still see onlu one cpu inside the guest. setup: - host: - Intel(R) Core(TM)2 Qua

[kvm-devel] [PATCH] align gdbstub with QEMU

2007-12-07 Thread Jan Kiszka
We've noticed some problems with current gdbstub in kvm's qemu: # qemu-system-x86_64 -hda myimage -S -s & # gdb (gdb) tar re :1234 Remote debugging using :1234 Remote 'g' packet reply is too long: [...] This issue did not occur with QEMU from CVS. As I was awar

Re: [kvm-devel] [PATCH] Use cmpxchg for pte updates on walk_addr()

2007-12-07 Thread Marcelo Tosatti
On Fri, Dec 07, 2007 at 07:06:26AM +0200, Avi Kivity wrote: > Marcelo Tosatti wrote: > >Right, patch at end of the message restarts the process if the pte > >changes under the walker. The goto is pretty ugly, but I fail to see any > >elegant way of doing that. Ideas? > > > > > > goto is fine for

Re: [kvm-devel] [PATCH] [0/1] simplify kvm-userspace to qemu-kvm callback structure

2007-12-07 Thread Avi Kivity
Christian Ehrhardt wrote: > Well no response might mean that there is nothing to be said against this > simplification ;-) > I did respond, see http://article.gmane.org/gmane.comp.emulators.kvm.devel/10467. -- Any sufficiently difficult bug is indistinguishable from a feature.

[kvm-devel] [PATCH] [1/1] simplify kvm-userspace to qemu-kvm callback structure

2007-12-07 Thread Christian Ehrhardt
Subject: [PATCH] simplify read[bwlq]/write[bwlq] callback structure From: Christian Ehrhardt <[EMAIL PROTECTED]> Merging the read[bwlq]/write[bwlq] callback callback pointers to one mmio_rw function simplifies the callback interface (a lot of lines in libkvm.c and main.c) and eventually uses t

[kvm-devel] [PATCH] [0/1] simplify kvm-userspace to qemu-kvm callback structure

2007-12-07 Thread Christian Ehrhardt
Well no response might mean that there is nothing to be said against this simplification ;-) Because of that I completed the patch (main.c as well) and corrected some whitespace issues I had before. Christian Ehrhardt wrote: > Background: > In our ppc code for the demo we only needed a call to >