[PATCH] kvm: qemu: revert fix power management timer overflow handling

2008-05-29 Thread Avi Kivity
From: Avi Kivity [EMAIL PROTECTED] The fix is bogus, causing sci to be deferred for longer and longer. Noticed by Alex Williamson. Signed-off-by: Avi Kivity [EMAIL PROTECTED] diff --git a/qemu/hw/acpi.c b/qemu/hw/acpi.c index c305702..e3cd8d7 100644 --- a/qemu/hw/acpi.c +++ b/qemu/hw/acpi.c @@

[PATCH] KVM: x86 emulator: add support for jmp far 0xea

2008-05-29 Thread Avi Kivity
From: Guillaume Thouvenin [EMAIL PROTECTED] Add support for jmp far (opcode 0xea) instruction. Signed-off-by: Guillaume Thouvenin [EMAIL PROTECTED] Signed-off-by: Laurent Vivier [EMAIL PROTECTED] Signed-off-by: Avi Kivity [EMAIL PROTECTED] diff --git a/arch/x86/kvm/x86_emulate.c

[PATCH] KVM: migrate PIT timer

2008-05-29 Thread Avi Kivity
From: Marcelo Tosatti [EMAIL PROTECTED] Migrate the PIT timer to the physical CPU which vcpu0 is scheduled on, similarly to what is done for the LAPIC timers, otherwise PIT interrupts will be delayed until an unrelated event causes an exit. Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED]

[PATCH] KVM: Prefixes segment functions that will be exported with kvm_

2008-05-29 Thread Avi Kivity
From: Guillaume Thouvenin [EMAIL PROTECTED] Prefixes functions that will be exported with kvm_. We also prefixed set_segment() even if it still static to be coherent. signed-off-by: Guillaume Thouvenin [EMAIL PROTECTED] Signed-off-by: Laurent Vivier [EMAIL PROTECTED] Signed-off-by: Avi Kivity

[PATCH] KVM: x86 emulator: implement 'push imm' (opcode 0x68)

2008-05-29 Thread Avi Kivity
From: Avi Kivity [EMAIL PROTECTED] Encountered in FC6 boot sequence, now that we don't force ss.rpl = 0 during the protected mode transition. Not really necessary, but nice to have. Signed-off-by: Avi Kivity [EMAIL PROTECTED] diff --git a/arch/x86/kvm/x86_emulate.c b/arch/x86/kvm/x86_emulate.c

Re: [PATCH 1/3] Fix partial csum rx handling

2008-05-29 Thread Herbert Xu
On Tue, May 27, 2008 at 12:36:06PM +0100, Mark McLoughlin wrote: When we receive a packet with a partial csum, just set ip_summed to CHECKSUM_UNNECESSARY. Signed-off-by: Mark McLoughlin [EMAIL PROTECTED] Cc: Herbert Xu [EMAIL PROTECTED] I'm not sure I follow the context of this patch.

[ kvm-Bugs-1970154 ] AMD CPU + KVM BUG.. Can't Share CPU..

2008-05-29 Thread SourceForge.net
Bugs item #1970154, was opened at 2008-05-23 15:36 Message generated for change (Comment added) made by call518 You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=893831aid=1970154group_id=180599 Please note that this message will contain a full copy of the comment

Re: [PATCH/RFC] virtio_blk: check for hardsector size from host

2008-05-29 Thread Rusty Russell
On Tuesday 27 May 2008 19:04:59 Christian Borntraeger wrote: Rusty, Jens, I need your opinion on the following patch. It seems to work, but I would like to get some feedback if this patch is the right approach: Looks like the right approach to me. Don't know about the block side of it...

Re: [kvm-devel] performance with guests running 2.4 kernels (specifically RHEL3)

2008-05-29 Thread Avi Kivity
Andrea Arcangeli wrote: - set up kmap to point at pte - test_and_clear_bit(pte) - kunmap From kvm's point of view this looks like - several accesses to set up the kmap Hmm, the kmap establishment takes a single guest operation in the fixmap area. That's a single write to the pte, to

[PATCH] virtio_blk: fix endianess annotations

2008-05-29 Thread Christian Borntraeger
Since commit 72e61eb40b55dd57031ec5971e810649f82b0259 (virtio: change config to guest endian) config space is no longer fixed endian. Lets change the virtio_blk_config variables. Signed-off-by: Christian Borntraeger [EMAIL PROTECTED] --- include/linux/virtio_blk.h |8 1 file

[PATCH]: Fake emulate Intel perfctr MSRs

2008-05-29 Thread Chris Lalancette
Attached is a patch for fake emulating Intel perfctr MSRs, similar to the recent patch to fake emulate the AMD perfctr MSRs. This is needed for a reason similar for the previous patch; older linux guests (in this case, 2.6.9) can attempt to access the MSR's without a fixup section, and injecting

Re: [kvm-devel] performance with guests running 2.4 kernels (specifically RHEL3)

2008-05-29 Thread Avi Kivity
David S. Ahern wrote: I've been instrumenting the guest kernel as well. It's the scanning of the active lists that triggers a lot of calls to paging64_prefetch_page, and, as you guys know, correlates with the number of direct pages in the list. Earlier in this thread I traced the kvm cycles to

Re: [Patch 0/7] RealMode: Support for big real mode

2008-05-29 Thread Anthony Liguori
Avi Kivity wrote: Anthony Liguori wrote: Avi Kivity wrote: Guillaume Thouvenin wrote: Hello, Those patches allow to boot OpenSuse-10.3. The problem with Opensuse 10.3 is it uses a version of gfxboot that reads SS after switching from real to protected mode, where SS contains an invalid

Re: [kvm-devel] performance with guests running 2.4 kernels (specifically RHEL3)

2008-05-29 Thread David S. Ahern
This is 2.4/RHEL3, so HZ=100. 848 jiffies = 8.48 seconds -- and that's just the one age bucket and this is just one example pulled randomly (well after boot). During that time kscand does get scheduled out, but ultimately guest time is at 100% during the scans. david Avi Kivity wrote: David S.

Re: [kvm-devel] performance with guests running 2.4 kernels (specifically RHEL3)

2008-05-29 Thread Avi Kivity
David S. Ahern wrote: This is 2.4/RHEL3, so HZ=100. 848 jiffies = 8.48 seconds -- and that's just the one age bucket and this is just one example pulled randomly (well after boot). During that time kscand does get scheduled out, but ultimately guest time is at 100% during the scans. Er,

Re: [kvm-devel] performance with guests running 2.4 kernels (specifically RHEL3)

2008-05-29 Thread Andrea Arcangeli
On Thu, May 29, 2008 at 01:01:06PM +0300, Avi Kivity wrote: No, two: static inline void set_pte(pte_t *ptep, pte_t pte) { ptep-pte_high = pte.pte_high; smp_wmb(); ptep-pte_low = pte.pte_low; } Right, that can be 2 or 1 depending on PAE non-PAE, other 2.4 enterprise

Re: [kvm-devel] performance with guests running 2.4 kernels (specifically RHEL3)

2008-05-29 Thread David S. Ahern
Andrea Arcangeli wrote: On Thu, May 29, 2008 at 01:01:06PM +0300, Avi Kivity wrote: No, two: static inline void set_pte(pte_t *ptep, pte_t pte) { ptep-pte_high = pte.pte_high; smp_wmb(); ptep-pte_low = pte.pte_low; } Right, that can be 2 or 1 depending on PAE

[PATCH 1/5] remove REGWPTR

2008-05-29 Thread Glauber Costa
Blue Swirl points out that it is not used anymore. Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- cpu-exec.c | 13 - target-sparc/exec.h |4 2 files changed, 0 insertions(+), 17 deletions(-) diff --git a/cpu-exec.c b/cpu-exec.c index b660954..ea0e5b1 100644

[PATCH 3/5] cpu_info_ip

2008-05-29 Thread Glauber Costa
This commit makes the code for info_cpus simpler in monitor.c Signed-off-by: Glauber Costa [EMAIL PROTECTED] --- exec-all.h|2 ++ monitor.c | 17 + target-alpha/helper.c |2 ++ target-arm/helper.c |2 ++ target-cris/helper.c |2 ++

[PATCH 2/5] simplify cpu_exec

2008-05-29 Thread Glauber Costa
This is a first attempt to simplify cpu_exec(): it has simply too many ifdefs, which is not a very good practice at all. Following some work I've already posted in the past, I'm moving the target-b ifdefs to target-xxx/helper.c, encapsuled into relevant functions. Signed-off-by: Glauber Costa

[ kvm-Bugs-1978070 ] Launching KVM freezes the system

2008-05-29 Thread SourceForge.net
Bugs item #1978070, was opened at 2008-05-29 18:59 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=893831aid=1978070group_id=180599 Please note that this message will contain a full copy of

PCI-passthrough: interrupt work structure per device

2008-05-29 Thread Ben-Ami Yassour
Amit, Attached is a patch for the pci-passthrough tree. This patch changes the workq structure of the interrupt handling to be per device. This also requires that all locks are locked with irqsave, since the interrupt handler is also using the pt device list. I tested it with an e1000 NIC and

Re: PCI-passthrough: interrupt work structure per device

2008-05-29 Thread Mark McLoughlin
On Thu, 2008-05-29 at 20:17 +0300, Ben-Ami Yassour wrote: Attached is a patch for the pci-passthrough tree. This patch changes the workq structure of the interrupt handling to be per device. Heh, I just happened to notice this myself yesterday. ... From

[patch 09/12] libkvm: in-kernel ACPI pmtimer interface

2008-05-29 Thread Marcelo Tosatti
Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: kvm-userspace.realtip/libkvm/kvm-common.h === --- kvm-userspace.realtip.orig/libkvm/kvm-common.h +++ kvm-userspace.realtip/libkvm/kvm-common.h @@ -49,6 +49,10 @@ struct

[patch 04/12] KVM: move muldiv64 to x86.c, export

2008-05-29 Thread Marcelo Tosatti
This should probably to go lib/ Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: kvm/arch/x86/kvm/i8254.c === --- kvm.orig/arch/x86/kvm/i8254.c +++ kvm/arch/x86/kvm/i8254.c @@ -45,26 +45,6 @@ #define RW_STATE_WORD0 3

[patch 12/12] QEMU/KVM: in-kernel pmtimer save/restore support

2008-05-29 Thread Marcelo Tosatti
Signed-off-by: Marcelo Tosatti [EMAIL PROTECTED] Index: kvm-userspace.realtip/qemu/hw/acpi.c === --- kvm-userspace.realtip.orig/qemu/hw/acpi.c +++ kvm-userspace.realtip/qemu/hw/acpi.c @@ -86,8 +86,16 @@ static uint32_t

[patch 06/12] QEMU/KVM: self-disabling C2 emulation

2008-05-29 Thread Marcelo Tosatti
Inform C2 state support via ACPI's CST per-processor package's, but write an invalid latency value the first time the guest attempts to idle via P_LVL2 port. This way the TSC is considered unreliable, and we get away with the costs relative to APIC timer broadcasts on enter/exit necessary for

Re: PCI-passthrough: interrupt work structure per device

2008-05-29 Thread Amit Shah
On Thursday 29 May 2008 23:39:37 Mark McLoughlin wrote: @@ -184,16 +190,27 @@ out: static irqreturn_t kvm_pci_pt_dev_intr(int irq, void *dev_id) { struct kvm *kvm = (struct kvm *) dev_id; + struct kvm_pci_pt_dev_list *pci_pt_dev; if (!test_bit(irq, pt_irq_handled))

Re: [PATCH] virtio_config: fix len calculation of config elements

2008-05-29 Thread Rusty Russell
On Thursday 29 May 2008 19:08:01 Christian Borntraeger wrote: v is a pointer, to we have to use sizeof(*v) instead of sizeof(v). How embarassing. Applied, thanks. Rusty. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to [EMAIL PROTECTED] More majordomo

[ kvm-Bugs-1970154 ] AMD CPU + KVM BUG.. Can't Share CPU..

2008-05-29 Thread SourceForge.net
Bugs item #1970154, was opened at 2008-05-23 15:36 Message generated for change (Comment added) made by call518 You can respond by visiting: https://sourceforge.net/tracker/?func=detailatid=893831aid=1970154group_id=180599 Please note that this message will contain a full copy of the comment