Re: [PATCH 2/4] KVM: Add accessor for reading cr4 (or some bits of cr4)

2009-12-08 Thread Avi Kivity
On 12/08/2009 09:57 AM, Sheng Yang wrote: vcpu-arch.cr0 = cr0; - vmx_set_cr4(vcpu, vcpu-arch.cr4); + vmx_set_cr4(vcpu, kvm_read_cr4(vcpu)); } Another place accessed cr4 directly, in ept_update_paging_mode_cr4() That one is called from

Re: [PATCH] virtio spec: add virtio-blk max sectors feature

2009-12-08 Thread Avishay Traeger1
Rusty Russell ru...@rustcorp.com.au wrote on 12/08/2009 07:48:00 AM: Avishay; this would be the total sectors in an I/O, as separate from SIZE_MAX (maximum size of any single scatterlist entry) and SEG_MAX (maximum number of scatterlist entries)? Correct. In the guest virtblk driver, it

Re: [PATCH] virtio spec: add virtio-blk max sectors feature

2009-12-08 Thread Avi Kivity
On 12/08/2009 11:44 AM, Avishay Traeger1 wrote: Thanks! Rusty. Sure. Avi - do you want me to resubmit the kvm and qemu patches? You mean the virtio and qemu patches. That's up to their maintainers (Rusty and Anthony). -- error compiling committee.c: too many arguments to

[AUTOTEST PATCH 1/2 - V3] Add a server-side test - kvm_migration

2009-12-08 Thread Yolkfull Chow
This patch will add a server-side test namely kvm_migration. Currently, it will use existing KVM client test framework and add a new file kvm_migration.py to help judge executing routine: source machine or dest machine. Improvement based on Version #2: * Log into migrated guest from source

[AUTOTEST PATCH 2/2] KVM test: subtest migration: Add rem_host and rem_port for migrate()

2009-12-08 Thread Yolkfull Chow
Since kvm_test_utils.migrate() adds two arguments to adopt server-side migration. This client side test also needs update. Signed-off-by: Yolkfull Chow yz...@redhat.com --- client/tests/kvm/tests/migration.py |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git

Re: KVM: MMU: remove prefault from invlpg handler

2009-12-08 Thread Avi Kivity
On 12/07/2009 10:51 PM, Marcelo Tosatti wrote: The BAD_POOL_HEADER BSOD happens at address 0xF8A000DDD000 (complaining it contains 00, Arg4). Walking the pagetables takes to 0x18996 as the pte page: (qemu) xp 0x18996ee8 (vaddr 0xF8A000DDD000) 18996ee8: 0x153c9963 (qemu)

Re: [PATCH 12/12] Kick appropriate CPUs when signalling interrupts.

2009-12-08 Thread Chris Lalancette
On 12/02/2009 04:44 PM, Gleb Natapov wrote: On Tue, Dec 01, 2009 at 03:36:41PM +0100, Chris Lalancette wrote: Make sure that we kick the appropriate vcpu when delivering an interrupt. This makes sure that we wake any idle cpus to cause a vcpu_run and an interrupt injection to occur.

Re: [PATCH 0/1] Defer skb allocation for both mergeable buffers and big packets in virtio_net

2009-12-08 Thread Michael S. Tsirkin
On Mon, Nov 23, 2009 at 11:23:18AM +1030, Rusty Russell wrote: On Fri, 20 Nov 2009 04:39:19 pm Shirley Ma wrote: Guest virtio_net receives packets from its pre-allocated vring buffers, then it delivers these packets to upper layer protocols as skb buffs. So it's not necessary to

Re: Test failures during git daily testing

2009-12-08 Thread Avi Kivity
On 12/07/2009 07:54 PM, Lucas Meneghel Rodrigues wrote: Hi Folks: Today pretty much all install tests for kvm and qemu upstream git failed. the vm screen says something along the lines: Starting SeaBIOS [version-string] No bootable device. Screenshot attached. The command line the test used:

Re: [PATCH v2] KVM: x86: Extend KVM_SET_VCPU_EVENTS with selective updates

2009-12-08 Thread Marcelo Tosatti
On Sun, Dec 06, 2009 at 06:24:15PM +0100, Jan Kiszka wrote: User space may not want to overwrite asynchronously changing VCPU event states on write-back. So allow to skip nmi.pending and sipi_vector by setting corresponding bits in the flags field of kvm_vcpu_events. Signed-off-by: Jan

Re: [PATCH v2] KVM: x86: Extend KVM_SET_VCPU_EVENTS with selective updates

2009-12-08 Thread Avi Kivity
On 12/08/2009 04:02 PM, Marcelo Tosatti wrote: On Sun, Dec 06, 2009 at 06:24:15PM +0100, Jan Kiszka wrote: User space may not want to overwrite asynchronously changing VCPU event states on write-back. So allow to skip nmi.pending and sipi_vector by setting corresponding bits in the flags

Re: [PATCH 2/4] KVM: Add accessor for reading cr4 (or some bits of cr4)

2009-12-08 Thread Sheng Yang
On Tue, Dec 08, 2009 at 11:36:59AM +0200, Avi Kivity wrote: On 12/08/2009 09:57 AM, Sheng Yang wrote: vcpu-arch.cr0 = cr0; - vmx_set_cr4(vcpu, vcpu-arch.cr4); + vmx_set_cr4(vcpu, kvm_read_cr4(vcpu)); } Another place accessed cr4 directly, in

Re: [PATCH 2/4] KVM: Add accessor for reading cr4 (or some bits of cr4)

2009-12-08 Thread Avi Kivity
On 12/08/2009 04:40 PM, Sheng Yang wrote: But you are right, I should have placed a comment. I'll add a patch that inlines ept_update_paging_mode_cr4 into its caller so it can access the cr4 parameter directly instead of vcpu-arch.cr4. Just notice another thing, seems the

Re: [Qemu-devel] Re: Endless loop in qcow2_alloc_cluster_offset

2009-12-08 Thread Kevin Wolf
Am 07.12.2009 16:00, schrieb Kevin Wolf: Am 07.12.2009 15:16, schrieb Jan Kiszka: Likely not. What I did was nothing special, and I did not noticed such a crash in the last months. And now it happened again (qemu-kvm head, during kernel installation from network onto local qcow2-disk). Any

[PATCH] Make S390x work in qemu-kvm

2009-12-08 Thread Alexander Graf
We now have S390x KVM support in qemu upstream. Unfortunately it doesn't work in qemu-kvm, because that has its own main loop and slightly different calling conventions for the KVM helpers. So let's hack in some small compat ifdefs that make qemu-kvm work on S390x! Signed-off-by: Alexander Graf

Re: Test failures during git daily testing

2009-12-08 Thread Jan Kiszka
Avi Kivity wrote: On 12/07/2009 07:54 PM, Lucas Meneghel Rodrigues wrote: Hi Folks: Today pretty much all install tests for kvm and qemu upstream git failed. the vm screen says something along the lines: Starting SeaBIOS [version-string] No bootable device. Screenshot attached. The

[PATCH] MAINTAINERS: Remove file pattern from KERNEL VIRTUAL MACHINE (KVM) FOR AMD-V

2009-12-08 Thread Joe Perches
Commit 6c8166a77c98f473eb91e96a61c3cf78ac617278 folded this file away. Signed-off-by: Joe Perches j...@perches.com diff --git a/MAINTAINERS b/MAINTAINERS index ea781c1..fda3eec 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3109,7 +3109,6 @@ L:kvm@vger.kernel.org W:

[PATCH 1/2] KVM test: autotest subtest - avoid dependencies on symlinks

2009-12-08 Thread Lucas Meneghel Rodrigues
Currently the autotest subtest relies on symbolic links placed in the kvm test. During the control file cleanup discussions, we decided we don't want to rely on symlinks too heavily, then make the autotest subtest a bit more robust, avoiding such dependencies. Note: If the control file cleanup is

[PATCH 2/2] KVM test: Fixing empty stress control file

2009-12-08 Thread Lucas Meneghel Rodrigues
For some reason, when the stress test was addeed to the KVM tests, the control file for it came without the actual statement to execute the stress test. Let's fix that. Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/autotest_control/stress.control |1 + 1 files

[PATCH] KVM test: Change the default file name used by kvm_config.py

2009-12-08 Thread Lucas Meneghel Rodrigues
In order to reflect the new schema of config files. Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/kvm_config.py |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/client/tests/kvm/kvm_config.py b/client/tests/kvm/kvm_config.py index

[PATCH] KVM test: build subtest (koji): turn lists into strings and vice versa

2009-12-08 Thread Lucas Meneghel Rodrigues
Since the KVM config system deliver strings for the user, let's make a bit of eval trickery to make the old syntax for the koji method work. Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/tests/build.py | 17 ++--- 1 files changed, 10 insertions(+), 7

KVM autotest patch queue report 12-09-2009

2009-12-08 Thread Lucas Meneghel Rodrigues
Patch: Major control file cleanup URL: http://patchwork.test.kernel.org/patch/1600/ Comments: Stage 1 of the planned work done Status: Need review (patchset made by the maintainer, need review by other people) Patch: KSM ovecommit test v.2 URL: http://patchwork.test.kernel.org/patch/1529/

Re: [PATCH] qemu-kvm: fix ia64 build breakage

2009-12-08 Thread Alexander Graf
Am 09.12.2009 um 05:37 schrieb Zhang, Xiantao xiantao.zh...@intel.com: I don't think how much effort needs to make it work again. So maybe need to evaluate it first, and give you the answer later. By the way, are you using it now ? No, but sles11 ships with it and I want to make

Re: Test failures during git daily testing

2009-12-08 Thread Avi Kivity
On 12/08/2009 11:34 PM, Lucas Meneghel Rodrigues wrote: (-boot n actually overwrites -boot d.) Isn't PXE booting temporarily disabled in qemu-kvm? So there is no device remaining and the tests fail. Avi, can you confirm that? If that's the case, I will have to switch over step file