[kvm-devel] [PATCH] lapic3: various cleanups on user/kernel irqchip

2007-08-10 Thread He, Qing
KVM: cleanups on user/kernel irqchip checking The patch removes the union of userspace irq_pending and kernel apic in struct vcpu, which minimizes the impact of careless overwriting and avoids accidentally dereferencing invalid apic pointer. It also adds a protective checking on KVM_INTERRUPT

Re: [kvm-devel] kvm very slow

2007-08-10 Thread Ulrich Schreiner
the system is now MUCH faster! it boots really fast, i think, in the init-scripts there are much sleep x, and every x were 2*x in reality. other tasks are also much faster i'm happy now :-) although i cannot reboot ... (it hangs after halted), but that is another thread. Am Donnerstag, den

[kvm-devel] KVM version reporting?

2007-08-10 Thread Jim Paris
Hi, Could qemu/VERSION get updated with each KVM release? Currently it seems there is no way to determine the KVM version from the binaries. Putting it in MODULE_VERSION for the kernel modules would also be useful. -jim -

Re: [kvm-devel] clean APIC_TMCCT setting in APIC timer

2007-08-10 Thread Dong, Eddie
Forgot the attachment :-( [EMAIL PROTECTED] wrote: APIC_TMCCT is a dynamically running count which always need to be recalculated at the time it is read. Setting intermediate value is meaningless. Signed-off-by: Yaozu (Eddie) Dong [EMAIL PROTECTED] diff --git

Re: [kvm-devel] Storing command line options in images

2007-08-10 Thread Anthony Liguori
Jorge Lucángeli Obes wrote: Hi all, From what I've gathered, it seems that we have basically four options at hand. I think it's important to notice, however, that whatever comes out of this will probably be, as Avi said, a low-end solution. IMHO there's room to have both the high-end

[kvm-devel] [PATCH 1/2][KVM] introduce a new field in cpustat

2007-08-10 Thread Laurent Vivier
introduce a new field, guest, in cpustat to store the time used by the CPU to run virtual CPU. Modify /proc/stat to display this new field. Signed-Off-by: Laurent Vivier [EMAIL PROTECTED] -- - [EMAIL PROTECTED] -- Software is hard - Donald Knuth Index:

[kvm-devel] [PATCH 2/2][KVM] modify account_system_time() to add cputime to cpustat-guest

2007-08-10 Thread Laurent Vivier
modify account_system_time() to add cputime to cpustat-guest if we are running a VCPU. We add this cputime to cpustat-user too instead of cpustat-system because this part of KVM code is in fact user code although it is executed in the kernel. We duplicate VCPU time between guest and user to allow

Re: [kvm-devel] add back pending timer irqs for kernel APIC timer

2007-08-10 Thread Dong, Eddie
[EMAIL PROTECTED] wrote: Add back pending irqs for apic timer to get precise guestAPIC timer interrupt. Signed-off-by: Yaozu (Eddie) Dong [EMAIL PROTECTED] a typo, please use this one. With above patches, now guest timer is much accurate! sleep 60 get exactly 60 seconds in

Re: [kvm-devel] help with understanding GPA/GVA issues

2007-08-10 Thread Avi Kivity
Gregory Haskins wrote: Yes, you can pass through any gva. There are couple of things to be aware of though. When passing a gva, you have to be sure that the gva is actually mapped in memory as KVM cannot cause Linux to fault in a page. Cool! Ya, I totally understand

Re: [kvm-devel] KVM version reporting?

2007-08-10 Thread Avi Kivity
Jim Paris wrote: Hi, Could qemu/VERSION get updated with each KVM release? Currently it seems there is no way to determine the KVM version from the binaries. Putting it in MODULE_VERSION for the kernel modules would also be useful. There will be a version string for the external

Re: [kvm-devel] [PATCH 2/2][KVM] modify account_system_time() to add cputime to cpustat-guest

2007-08-10 Thread Laurent Vivier
You can find attached to this email the patch to apply to procps-3.2.7[1] to allow top(1) to display the cpu guest time. Laurent [1] http://procps.sf.net/procps-3.2.7.tar.gz -- - [EMAIL PROTECTED] -- Software is hard - Donald Knuth Index: procps-3.2.7/top.c

[kvm-devel] add back pending timer irqs for kernel APIC timer

2007-08-10 Thread Dong, Eddie
Add back pending irqs for apic timer to get precise guest APIC timer interrupt. Signed-off-by: Yaozu (Eddie) Dong [EMAIL PROTECTED] diff --git a/drivers/kvm/irq.h b/drivers/kvm/irq.h index ed6d20a..8867c82 100644 --- a/drivers/kvm/irq.h +++ b/drivers/kvm/irq.h @@ -110,7 +110,7

[kvm-devel] modify apic timer last_update to precisely reflect hrtimer fire point.

2007-08-10 Thread Dong, Eddie
cleanup: last_update should precisely reflect the time when an hrtimer is fired, and store the period of apic timer to avoid calculation each time. Signed-off-by: Yaozu (Eddie) Dong [EMAIL PROTECTED] diff --git a/drivers/kvm/irq.h b/drivers/kvm/irq.h index

[kvm-devel] clean APIC_TMCCT setting in APIC timer

2007-08-10 Thread Dong, Eddie
APIC_TMCCT is a dynamically running count which always need to be recalculated at the time it is read. Setting intermediate value is meaningless. Signed-off-by: Yaozu (Eddie) Dong [EMAIL PROTECTED] diff --git a/drivers/kvm/lapic.c b/drivers/kvm/lapic.c index d59b69f..fc53e88

Re: [kvm-devel] [Qemu-devel] Re: Storing command line options in images

2007-08-10 Thread Anthony Liguori
Avi Kivity wrote: This is a big effort but a config file is the right long term solution. For which use case? management-full or management-less? Both. A config file will be useful not just for expressing the functionality we have today, but also for describing the guest's

Re: [kvm-devel] [Qemu-devel] Re: Storing command line options in images

2007-08-10 Thread Avi Kivity
Anthony Liguori wrote: Jorge Lucángeli Obes wrote: Hi all, From what I've gathered, it seems that we have basically four options at hand. I think it's important to notice, however, that whatever comes out of this will probably be, as Avi said, a low-end solution. IMHO there's room to have

Re: [kvm-devel] add back pending timer irqs for kernel APIC timer

2007-08-10 Thread Avi Kivity
Dong, Eddie wrote: Add back pending irqs for apic timer to get precise guest APIC timer interrupt. Can you explain the problem and the solution in more detail? - This SF.net email is sponsored by: Splunk Inc.

[kvm-devel] [PATCH 0/2][KVM] guest time accounting

2007-08-10 Thread Laurent Vivier
The aim of these two patches is to measure the CPU time used by a virtual machine. All comments are welcome... I'm not sure it's the good way to do that. [PATCH 1/2] introduce a new field, guest, in cpustat to store the time used by the CPU to run virtual CPU. Modify /proc/stat to display this

[kvm-devel] [PATCH] qemu: free migration structure if migration fails

2007-08-10 Thread Jim Paris
Without this, memory is leaked and later attempts fail with Migration already active. Signed-off-by: Jim Paris [EMAIL PROTECTED] --- qemu/migration.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/qemu/migration.c b/qemu/migration.c index bbeed86..ed1fd29 100644 ---

Re: [kvm-devel] [Qemu-devel] Re: Storing command line options in images

2007-08-10 Thread Anthony Liguori
Avi Kivity wrote: Anthony Liguori wrote: Avi Kivity wrote: This is a big effort but a config file is the right long term solution. For which use case? management-full or management-less? Both. A config file will be useful not just for expressing the functionality we have

Re: [kvm-devel] help with understanding GPA/GVA issues

2007-08-10 Thread Anthony Liguori
Gregory Haskins wrote: Hi All, I am working on some PV stuff and had some questions about the ability to share memory across the Guest/Host boundary. It seems that most examples of how to do this always involve starting with a *page, converting it to a gfn via page_to_gfn(), and using that

Re: [kvm-devel] help with understanding GPA/GVA issues

2007-08-10 Thread Gregory Haskins
On Fri, 2007-08-10 at 19:59 +0300, Avi Kivity wrote: Note that passing a virtual address is highly discouraged as its meaning can change from vcpu to vcpu, it might not be mapped, translation is slow, etc. Just let the guest do the translation. Yeah, Hollis and Anthony straighted me out

Re: [kvm-devel] WARNING: at arch/x86_64/kernel/smp.c:379 smp_call_function_single()

2007-08-10 Thread Michal Piotrowski
Hi, On 10/08/07, Paolo Ornati [EMAIL PROTECTED] wrote: Just got this warning during suspend2ram (2.6.23-rc2-gac078602). Config and full dmesg attached. [ 756.707601] Disabling non-boot CPUs ... [ 756.712034] kvm: disabling virtualization on CPU1 [ 756.712037] WARNING: at

Re: [kvm-devel] help with understanding GPA/GVA issues

2007-08-10 Thread Gregory Haskins
On Fri, 2007-08-10 at 09:32 -0500, Anthony Liguori wrote: Gregory Haskins wrote: Hi All, I am working on some PV stuff and had some questions about the ability to share memory across the Guest/Host boundary. It seems that most examples of how to do this always involve starting with

Re: [kvm-devel] help with understanding GPA/GVA issues

2007-08-10 Thread Hollis Blanchard
On Fri, 10 Aug 2007 00:09:18 -0400, Gregory Haskins wrote: If so, how would I do this: E.g. can I just pass the pointer, and then do gva_to_hpa() on the host? Or do I need to prep the pointer before sending it? We've talked about this a little in the past: see the discussion around

[kvm-devel] [Patch] (big) real mode emulation - jmp rel

2007-08-10 Thread Nitin A Kamble
Hi Avi, The patch to implement jmp rel emulation is attached. Thanks Regards, Nitin Open Source Technology Center, Intel Corporation. - The mind is like a parachute; it works much better when it's open. commit

[kvm-devel] [Patch] (big) real mode emulation - jmp rel short

2007-08-10 Thread Nitin A Kamble
Hi Avi, Attached is the patch for the instruction jmp rel short, opcode 0xeb. This patch goes on top of last patch (jmp rel). Thanks Regards, Nitin Open Source Technology Center, Intel Corporation. - The mind is

[kvm-devel] [PATCH] Fix network boot whether KVM is enabled or disabled

2007-08-10 Thread Chuan-kai Lin
From: Chuan-kai Lin [EMAIL PROTECTED] This patch is based on the one Anthony Liguori submitted to kvm-devel on July 2nd, which fixes PXE booting with KVM enabled but breaks PXE booting when not using KVM. I simplified Anthony's patch and duplicated the cpu_register_physical_memory call to ensure

Re: [kvm-devel] add back pending timer irqs for kernel APIC timer

2007-08-10 Thread Dong, Eddie
Avi Kivity wrote: Dong, Eddie wrote: Add back pending irqs for apic timer to get precise guest APIC timer interrupt. Can you explain the problem and the solution in more detail? Today guest sleep 10 seconds only get about 5-6 seconds in host time. A design philosiphy in various

Re: [kvm-devel] [Qemu-devel] Re: Storing command line options in images

2007-08-10 Thread Jorge Lucángeli Obes
My feeling is that config files are outdated. When used with a gui, you end up writing silly parsers and stuff and still wrecking things horribly when the the gui writer's expectations don't match reality. When used without a gui, they increase the amount of details one has to remember

Re: [kvm-devel] [PATCH] Fix network boot whether KVM is enabled or disabled

2007-08-10 Thread Anthony Liguori
Chuan-kai Lin wrote: From: Chuan-kai Lin [EMAIL PROTECTED] This patch is based on the one Anthony Liguori submitted to kvm-devel on July 2nd, which fixes PXE booting with KVM enabled but breaks PXE booting when not using KVM. I simplified Anthony's patch and duplicated the

[kvm-devel] problem upgrading to kvm-33

2007-08-10 Thread Tony Smolar
Hi, I was currently using kvm-24, and I decided to try to upgrade to the latest. My guest is XP-SP2 Home full edition.When I tried to boot it under kvm-33, I eventually got a blue screen and a IRQL_NOT_LESS_OR_EQUAL error. I removed 33, and installed kvm-26.I was able to boot the VM