Re: [PATCH 1/3] kvm: fix irq 0 assignment

2009-03-04 Thread Chris Wedgwood
On Wed, Mar 04, 2009 at 03:54:27PM +0800, Sheng Yang wrote: Shouldn't update assigned irq if host irq is 0, which means uninitialized or don't support INTx. Is that generally true? -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to

kvm-84 kernel panic

2009-03-04 Thread Johannes Baumann
Hi, i had serveral kernel panics in the last days on a DELL R300. First i had ubuntu 8.10 running with latest kvm compiled. the panic looked like this: http://memic.paniert.org/screen.png The impi log looks like this 2d | 02/19/2009 | 02:54:31 | Processor #0x0d | Transition to Non-recoverable

[PATCH 0/3] ioapic/lapic/msi cleanup

2009-03-04 Thread Gleb Natapov
There are many code/logic duplications throughout ioapic/lapic/msi device emulation. Try to consolidate as much code as possible. The patch series is on top of Sheng Yang's patch: KVM: Merge kvm_ioapic_get_delivery_bitmask into kvm_get_intr_delivery_bitmask --- Gleb Natapov (3):

[PATCH 2/3] ioapic/msi interrupt delivery consolidation.

2009-03-04 Thread Gleb Natapov
ioapic_deliver() and kvm_set_msi() have code duplication. Move the code into ioapic_deliver_entry() function and call it from both places. Signed-off-by: Gleb Natapov g...@redhat.com --- virt/kvm/ioapic.c | 60 +++ virt/kvm/ioapic.h |4

[PATCH 3/3] Consolidate ioapic/ipi interrupt delivery logic.

2009-03-04 Thread Gleb Natapov
Use kvm_apic_match_dest() in kvm_get_intr_delivery_bitmask() instead of duplicating the same code. Use kvm_get_intr_delivery_bitmask() in apic_send_ipi() to figure out ipi destination instead of reimplementing the logic. Signed-off-by: Gleb Natapov g...@redhat.com --- arch/ia64/kvm/kvm-ia64.c

Re: [PATCH 04/10] Support for device capability

2009-03-04 Thread Marcelo Tosatti
On Wed, Mar 04, 2009 at 10:56:36AM +0800, Sheng Yang wrote: On Tuesday 03 March 2009 20:42:07 Marcelo Tosatti wrote: Hi Sheng, On Mon, Mar 02, 2009 at 04:29:27PM +0800, Sheng Yang wrote: This framework can be easily extended to support device capability, like MSI/MSI-x.

Re: Problem with KVM-84 and more than 4 processors

2009-03-04 Thread Matthias Hovestadt
Hi Charles! This mail seems to be somehow related to the bug report of Justin Keogh from 30th Dec 2008. He was able to fix his problem by simply setting CONFIG_KVM=m, but I'm already using modules, so his workaround can't be applied here. Quick question -- does your dmesg output include

Re: [PATCH 1/3] Make kvm_apic_set_irq() deliver all kinds of interrupts.

2009-03-04 Thread Marcelo Tosatti
Hi Gleb, On Wed, Mar 04, 2009 at 03:30:11PM +0200, Gleb Natapov wrote: Get rid of ioapic_inj_irq() and ioapic_inj_nmi() functions. Signed-off-by: Gleb Natapov g...@redhat.com --- index afc59b2..d58268f 100644 --- a/arch/x86/kvm/lapic.c +++ b/arch/x86/kvm/lapic.c @@ -196,20 +196,30 @@

Re: [PATCH 3/3] Consolidate ioapic/ipi interrupt delivery logic.

2009-03-04 Thread Marcelo Tosatti
On Wed, Mar 04, 2009 at 03:30:22PM +0200, Gleb Natapov wrote: Use kvm_apic_match_dest() in kvm_get_intr_delivery_bitmask() instead of duplicating the same code. Use kvm_get_intr_delivery_bitmask() in apic_send_ipi() to figure out ipi destination instead of reimplementing the logic.

Re: kvm-autotest -- introducing kvm_runtest_2

2009-03-04 Thread Ryan Harper
* Uri Lublin u...@redhat.com [2009-03-04 02:59]: - it seems like the definition and rules ought to be separate from the last section which defines which tests to run (the fc8_quick area), so adding something as simple as include support to kvm_config.py would be sufficient to support

Re: [PATCH 1/1] KVM: Merge kvm_ioapic_get_delivery_bitmask into kvm_get_intr_delivery_bitmask

2009-03-04 Thread Marcelo Tosatti
On Wed, Mar 04, 2009 at 01:33:02PM +0800, Sheng Yang wrote: Gleb fixed bitmap ops usage in kvm_ioapic_get_delivery_bitmask. Sheng merged two functions, as well as fixed several issues in kvm_get_intr_delivery_bitmask 1. deliver_bitmask is a bitmap rather than a unsigned long intereger. 2.

Re: kvm-autotest -- introducing kvm_runtest_2

2009-03-04 Thread sudhir kumar
On Wed, Mar 4, 2009 at 11:45 PM, Ryan Harper ry...@us.ibm.com wrote: * Uri Lublin u...@redhat.com [2009-03-04 02:59]:  - it seems like the definition and rules ought to be separate from the  last section which defines which tests to run (the fc8_quick area), so  adding something as simple

Re: [PATCH 1/3] kvm: fix irq 0 assignment

2009-03-04 Thread Marcelo Tosatti
On Wed, Mar 04, 2009 at 03:54:27PM +0800, Sheng Yang wrote: Shouldn't update assigned irq if host irq is 0, which means uninitialized or don't support INTx. Signed-off-by: Sheng Yang sh...@linux.intel.com --- qemu/hw/device-assignment.c |4 1 files changed, 4 insertions(+), 0

Re: [PATCH 1/3] kvm: fix irq 0 assignment

2009-03-04 Thread Chris Wright
* Marcelo Tosatti (mtosa...@redhat.com) wrote: Please do not special case irq 0. The fact that only x86/IA64 are supported at the moment does not mean other architectures can't use it. Seems logical to use a flag instead of overloading the irq value. Also, the kernel code to handle dev/irq

Re: kvm-84 kernel panic

2009-03-04 Thread Johannes Baumann
update: the kernel panics just seem to happen with smp guests, right now i have only non smp guest and everything is fine, even under load. is this a known problem? Johannes Baumann schrieb: Hi, i had serveral kernel panics in the last days on a DELL R300. First i had ubuntu 8.10 running

Re: kvm-autotest -- introducing kvm_runtest_2

2009-03-04 Thread Dor Laor
sudhir kumar wrote: On Wed, Mar 4, 2009 at 11:45 PM, Ryan Harper ry...@us.ibm.com wrote: * Uri Lublin u...@redhat.com [2009-03-04 02:59]: - it seems like the definition and rules ought to be separate from the last section which defines which tests to run (the fc8_quick area), so

Re: kvm-83 write performance raw

2009-03-04 Thread Paolo Pedaletti
Ciao Mark, RAID1 has *much* better write performance. With striping RAIDs, alignment is important. RAID controllers sometimes introduce hidden alignment offsets. Excessive read-ahead is a waste of time with a lot of small random I/O, which is what I see mostly with guests on flat disk

Re: kvm-84 kernel panic

2009-03-04 Thread Marcelo Tosatti
On Wed, Mar 04, 2009 at 02:10:30PM +0100, Johannes Baumann wrote: Hi, i had serveral kernel panics in the last days on a DELL R300. First i had ubuntu 8.10 running with latest kvm compiled. the panic looked like this: http://memic.paniert.org/screen.png The impi log looks like this 2d

Re: kvm-autotest -- introducing kvm_runtest_2

2009-03-04 Thread Uri Lublin
From: Ryan Harper ry...@us.ibm.com Uri Lublin u...@redhat.com [2009-03-04 02:59]: - it seems like the definition and rules ought to be separate from the last section which defines which tests to run (the fc8_quick area), so adding something as simple as include support to kvm_config.py

Re: kvm-autotest -- introducing kvm_runtest_2

2009-03-04 Thread Uri Lublin
From: sudhir kumar smalik...@gmail.com On Wed, Mar 4, 2009 at 11:45 PM, Ryan Harper ry...@us.ibm.com wrote: * Uri Lublin u...@redhat.com [2009-03-04 02:59]: - guest install wizard using md5sum region matching ... ouch.  This is quite fickle.  I've seen different kvms generate different md5sum

RE: [PATCH] kvm : qemu : fix compilation error in kvm-userspace for ia64

2009-03-04 Thread Zhang, Yang
Marcelo Tosatti wrote: On Tue, Mar 03, 2009 at 11:38:17AM +0800, Zhang, Yang wrote: diff --git a/qemu/hw/i8259.c b/qemu/hw/i8259.c index 9cb3941..025f993 100644 --- a/qemu/hw/i8259.c +++ b/qemu/hw/i8259.c @@ -189,8 +189,10 @@ static void i8259_set_irq(void *opaque, int irq, int

Re: [PATCH] kvm : qemu : fix compilation error in kvm-userspace for ia64

2009-03-04 Thread Marcelo Tosatti
On Thu, Mar 05, 2009 at 09:36:13AM +0800, Zhang, Yang wrote: Marcelo Tosatti wrote: On Tue, Mar 03, 2009 at 11:38:17AM +0800, Zhang, Yang wrote: diff --git a/qemu/hw/i8259.c b/qemu/hw/i8259.c index 9cb3941..025f993 100644 --- a/qemu/hw/i8259.c +++ b/qemu/hw/i8259.c @@ -189,8 +189,10

RE: [PATCH] kvm : qemu : fix compilation error in kvm-userspace for ia64

2009-03-04 Thread Zhang, Yang
Marcelo Tosatti wrote: On Thu, Mar 05, 2009 at 09:36:13AM +0800, Zhang, Yang wrote: seems right, But i cannot find the proper place to define it. And i think we can do this at the time of we need to use it. The thinking is avoid code from piling in kvm-userspace when it belongs in upstream

Re: kvm-83 write performance raw

2009-03-04 Thread Mark van Walraven
Hi Paolo, Sorry, list - getting a bit off-topic but I'll include because it might be of general interest for kvm users ... On Wed, Mar 04, 2009 at 11:28:18PM +0100, Paolo Pedaletti wrote: ok, I can understand this but on a big multimedia-file partition an opportune read-ahead could be

Re: [PATCH 1/3] kvm: fix irq 0 assignment

2009-03-04 Thread Sheng Yang
On Thursday 05 March 2009 03:41:41 Chris Wright wrote: * Marcelo Tosatti (mtosa...@redhat.com) wrote: Please do not special case irq 0. The fact that only x86/IA64 are supported at the moment does not mean other architectures can't use it. Seems logical to use a flag instead of

RE: compile problems userspace-tools from git on ubuntu intrepid 8.10 x64

2009-03-04 Thread Zhang, Xiantao
Did you do make sync LINUX=/usr/local/src/linux-kvm-git in kvm-userspace.git before your ./configure ? Xiantao freisei wrote: Hi @ll, I want to update my kvm-84 to the latest git releaste due to an IOMMU-feature. git clone git://git.kernel.org/pub/scm/virt/kvm/kvm-userspace.git

Re: compile problems userspace-tools from git on ubuntu intrepid 8.10 x64

2009-03-04 Thread Freisei
Thank you! Solved my problem! How could i know it without asking the list? greets freisei Zhang, Xiantao schrieb: Did you do make sync LINUX=/usr/local/src/linux-kvm-git in kvm-userspace.git before your ./configure ? Xiantao freisei wrote: Hi @ll, I want to update my kvm-84 to the

[PATCH] kvm : qemu: fix compilation error with old kernel

2009-03-04 Thread Zhang, Yang
The function hrtimer_start_expires() in the kvm-ia64.c is not supported before the kernel 2.6.28. So we need to hack it. please review it. thanks Signed-off-by: Yang Zhang yang.zh...@intel.com --- kernel/Makefile |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git