Re: How many threads should a kvm vm be starting?

2011-09-27 Thread Avi Kivity
On 09/27/2011 03:29 AM, Thomas Fjellstrom wrote: I just noticed something interesting, a virtual machine on one of my servers seems to have 69 threads (including the main thread). Other guests on the machine only have a couple threads. Is this normal? or has something gone horribly wrong?

Re: [PATCH 1/2] virtio-net: Verify page list size before fitting into skb

2011-09-27 Thread Sasha Levin
On Mon, 2011-09-26 at 22:55 +0300, Michael S. Tsirkin wrote: On Mon, Sep 26, 2011 at 10:37:22PM +0300, Sasha Levin wrote: On Mon, 2011-09-26 at 21:44 +0300, Michael S. Tsirkin wrote: On Mon, Sep 26, 2011 at 08:41:08PM +0300, Sasha Levin wrote: This patch verifies that the length of a

Re: [PATCH 1/2] virtio-net: Verify page list size before fitting into skb

2011-09-27 Thread Michael S. Tsirkin
On Tue, Sep 27, 2011 at 09:44:02AM +0300, Sasha Levin wrote: On Mon, 2011-09-26 at 22:55 +0300, Michael S. Tsirkin wrote: On Mon, Sep 26, 2011 at 10:37:22PM +0300, Sasha Levin wrote: On Mon, 2011-09-26 at 21:44 +0300, Michael S. Tsirkin wrote: On Mon, Sep 26, 2011 at 08:41:08PM +0300,

Re: [PATCH v3] kvm tools: Support multiple net devices

2011-09-27 Thread Ingo Molnar
* Sasha Levin levinsasha...@gmail.com wrote: --- a/tools/kvm/builtin-run.c +++ b/tools/kvm/builtin-run.c @@ -87,9 +87,12 @@ static bool sdl; static bool balloon; static bool using_rootfs; static bool custom_rootfs; +static bool no_net; extern bool ioport_debug; extern int

Re: [kvm] Re: [kvm] Re: Questions about duplicate memory work

2011-09-27 Thread Robin Lee Powell
On Mon, Sep 26, 2011 at 04:15:37PM +0800, Emmanuel Noobadmin wrote: On 9/26/11, Robin Lee Powell rlpow...@digitalkingdom.org wrote: On Mon, Sep 26, 2011 at 01:49:18PM +0800, Emmanuel Noobadmin wrote: On 9/25/11, Robin Lee Powell rlpow...@digitalkingdom.org wrote: OK, so I've got a Linux

Re: [kvm] Re: [kvm] Re: Questions about duplicate memory work

2011-09-27 Thread Emmanuel Noobadmin
On 9/27/11, Robin Lee Powell rlpow...@digitalkingdom.org wrote: On Mon, Sep 26, 2011 at 04:15:37PM +0800, Emmanuel Noobadmin wrote: It's unrelated to what you're actually using as the disks, whether file or block devices like LVs. I think it just makes KVM tell the host not to cache I/O done

Re: [kvm] Re: [kvm] Re: Questions about duplicate memory work

2011-09-27 Thread Robin Lee Powell
On Tue, Sep 27, 2011 at 04:41:33PM +0800, Emmanuel Noobadmin wrote: On 9/27/11, Robin Lee Powell rlpow...@digitalkingdom.org wrote: On Mon, Sep 26, 2011 at 04:15:37PM +0800, Emmanuel Noobadmin wrote: It's unrelated to what you're actually using as the disks, whether file or block devices

Re: [kvm] Re: [kvm] Re: Questions about duplicate memory work

2011-09-27 Thread Robin Lee Powell
On Tue, Sep 27, 2011 at 01:48:43AM -0700, Robin Lee Powell wrote: On Tue, Sep 27, 2011 at 04:41:33PM +0800, Emmanuel Noobadmin wrote: On 9/27/11, Robin Lee Powell rlpow...@digitalkingdom.org wrote: On Mon, Sep 26, 2011 at 04:15:37PM +0800, Emmanuel Noobadmin wrote: It's unrelated to

Re: [PATCH] qemu-kvm: Switch POSIX compat AIO implementation to upstream

2011-09-27 Thread Avi Kivity
On 09/26/2011 09:09 PM, Anthony Liguori wrote: On 09/26/2011 12:24 PM, Avi Kivity wrote: On 09/26/2011 08:23 PM, Jan Kiszka wrote: Perhaps qemu_eventfd() can be used in the future instead of an explicit pipe. Then Linux will do eventfd while other OSes will fall back to pipes. Basically

Re: LPC2011 Virtualization Micro Conf

2011-09-27 Thread Jes Sorensen
Hi, For those who are interested, I have posted the notes from the 2011 Linux Plumbers Virtualization micro conference here: http://wiki.linuxplumbersconf.org/2011:virtualization Slides can be found by clicking on the presentation and going onto the Plumbers abstracts. Cheers, Jes -- To

Re: [Xen-devel] [PATCH 00/10] [PATCH RFC V2] Paravirtualized ticketlocks

2011-09-27 Thread Stephan Diestelhorst
On Wednesday 14 September 2011, 17:31:32 Jeremy Fitzhardinge wrote: This series replaces the existing paravirtualized spinlock mechanism with a paravirtualized ticketlock mechanism. [...] The unlock code is very straightforward: prev = *lock; __ticket_unlock_release(lock);

Re: [kvm] Re: [kvm] Re: Questions about duplicate memory work

2011-09-27 Thread Avi Kivity
On 09/27/2011 12:00 PM, Robin Lee Powell wrote: On Tue, Sep 27, 2011 at 01:48:43AM -0700, Robin Lee Powell wrote: On Tue, Sep 27, 2011 at 04:41:33PM +0800, Emmanuel Noobadmin wrote: On 9/27/11, Robin Lee Powellrlpow...@digitalkingdom.org wrote: On Mon, Sep 26, 2011 at 04:15:37PM

Re: [PATCH v3 1/6] iommu/core: split mapping to page sizes as supported by the hardware

2011-09-27 Thread Roedel, Joerg
On Fri, Sep 16, 2011 at 01:51:41PM -0400, Ohad Ben-Cohen wrote: drivers/iommu/iommu.c | 158 +--- drivers/iommu/omap-iovmm.c | 12 +--- include/linux/iommu.h |6 +- virt/kvm/iommu.c |4 +- 4 files changed, 157

kvm linux-next tree

2011-09-27 Thread Avi Kivity
Please add git://github.com/avikivity/kvm.git kvm-updates/3.2 to the linux-next collection, until kernel.org is back up. Thanks. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to

Re: [PATCH v3 1/6] iommu/core: split mapping to page sizes as supported by the hardware

2011-09-27 Thread Ohad Ben-Cohen
On Tue, Sep 27, 2011 at 1:05 PM, Roedel, Joerg joerg.roe...@amd.com wrote: On Fri, Sep 16, 2011 at 01:51:41PM -0400, Ohad Ben-Cohen wrote:  int iommu_map(struct iommu_domain *domain, unsigned long iova, -             phys_addr_t paddr, int gfp_order, int prot) +             phys_addr_t paddr,

Re: [PATCH 1/2] virtio-net: Verify page list size before fitting into skb

2011-09-27 Thread Sasha Levin
On Tue, 2011-09-27 at 10:00 +0300, Michael S. Tsirkin wrote: skb = page_to_skb(vi, page, len); ... Sorry I don't get it yet. Where is mergeable ignored here? The NULL deref happens in page_to_skb(), before merge buffers are handled. I'll test it and see if it's

Re: [PATCH 1/2] virtio-net: Verify page list size before fitting into skb

2011-09-27 Thread Michael S. Tsirkin
On Tue, Sep 27, 2011 at 02:20:29PM +0300, Sasha Levin wrote: On Tue, 2011-09-27 at 10:00 +0300, Michael S. Tsirkin wrote: skb = page_to_skb(vi, page, len); ... Sorry I don't get it yet. Where is mergeable ignored here? The NULL deref happens in

[PATCH] kvm tools: Set active console before running /bin/sh

2011-09-27 Thread Sasha Levin
This patch sets active console to the serial console before running the shell. Doing so fixes two issues: * Fix job control. * Set all env vars. The user visible issues are less warnings (no more of this: sh: cannot set terminal process group (-1): Inappropriate ioctl for device sh: no job

Re: [PATCH 0/4] acpi: fix up EJ0 in DSDT

2011-09-27 Thread Michael S. Tsirkin
On Mon, Sep 26, 2011 at 08:04:24PM -0400, Kevin O'Connor wrote: On Mon, Sep 26, 2011 at 10:04:13AM +0300, Michael S. Tsirkin wrote: On Mon, Sep 26, 2011 at 12:40:18AM -0400, Kevin O'Connor wrote: On Thu, Sep 22, 2011 at 09:09:49AM +0300, Michael S. Tsirkin wrote: On Thu, Sep 22, 2011 at

[PATCH v2] kvm tools: Set active console before running /bin/sh

2011-09-27 Thread Sasha Levin
This patch sets active console to the serial console before running the shell. Doing so fixes two issues: * Fix job control. * Set all env vars. The user visible issues are less warnings (no more of this: sh: cannot set terminal process group (-1): Inappropriate ioctl for device sh: no job

Re: [PATCH v3 1/6] iommu/core: split mapping to page sizes as supported by the hardware

2011-09-27 Thread Roedel, Joerg
On Tue, Sep 27, 2011 at 08:26:29AM -0400, Ohad Ben-Cohen wrote: With an unsigned long you can use plain and fast bit_ops instead of the full bitmap functions. Not sure I follow; the only bit operation I'm using while mapping is find_next_bit() (which is a bitops.h method). What other

Re: [PATCH v3 1/6] iommu/core: split mapping to page sizes as supported by the hardware

2011-09-27 Thread Ohad Ben-Cohen
On Tue, Sep 27, 2011 at 4:12 PM, Roedel, Joerg joerg.roe...@amd.com wrote: You pass a pointer to an unsigned long for the page-size bitmap. This allows to use an array of unsigned long. But a single unsigned long is sufficient This is fine; I can change that if you like it better (though

Re: KVM call agenda for September 26

2011-09-27 Thread Juan Quintela
Juan Quintela quint...@redhat.com wrote: Hi Please send in any agenda items you are interested in covering. Thanks, Juan. As there are no agenda for today, call gets cancelled. Happy hacking, Juan. -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to

Re: kvm linux-next tree

2011-09-27 Thread Stephen Rothwell
Hi Avi, On Tue, 27 Sep 2011 15:25:02 +0300 Avi Kivity a...@redhat.com wrote: Please add git://github.com/avikivity/kvm.git kvm-updates/3.2 to the linux-next collection, until kernel.org is back up. Thanks. I will switch to this tomorrow. -- Cheers, Stephen Rothwell

Re: [PATCH 0/4] acpi: fix up EJ0 in DSDT

2011-09-27 Thread Paolo Bonzini
On 09/27/2011 03:04 PM, Michael S. Tsirkin wrote: There was a concern raised with two-pass PCI initialization that I need to follow up on before tagging. The isa bridge? I thought that got fixed ... Daniel Berrange reported a regression with virtio-9p. Paolo -- To unsubscribe from

Re: [Xen-devel] [PATCH 00/10] [PATCH RFC V2] Paravirtualized ticketlocks

2011-09-27 Thread Jeremy Fitzhardinge
On 09/27/2011 02:34 AM, Stephan Diestelhorst wrote: On Wednesday 14 September 2011, 17:31:32 Jeremy Fitzhardinge wrote: This series replaces the existing paravirtualized spinlock mechanism with a paravirtualized ticketlock mechanism. [...] The unlock code is very straightforward: prev

[PATCH 0/4] KVM test: Unattended install fixes

2011-09-27 Thread Lucas Meneghel Rodrigues
Here we have a handful of fixes for the unattended install code. In special, with the last patch we recover the ability to run ping pong migration in the background during the unattended install for linux guests. This patchset was also sent as a pull request on gitub. Feel free to make comments

[PATCH 1/4] KVM test: unattended_install: Fix unused variable warning

2011-09-27 Thread Lucas Meneghel Rodrigues
Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/tests/unattended_install.py |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/client/tests/kvm/tests/unattended_install.py b/client/tests/kvm/tests/unattended_install.py index 1ff73af..7d90a91

[PATCH 3/4] KVM test: unattended_install: Remove unused variable post_install_delay

2011-09-27 Thread Lucas Meneghel Rodrigues
Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/tests/unattended_install.py |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/client/tests/kvm/tests/unattended_install.py b/client/tests/kvm/tests/unattended_install.py index 7655bc1..023d00f

[PATCH 4/4] KVM test: unattended_install: Fix the background migration code

2011-09-27 Thread Lucas Meneghel Rodrigues
Currently we use initrd and kernel params to perform unattended install in all supported linux guests. Before that, we used to remove those params from the qemu params because they alledgedly 'broke' the migration. Well, turns out this code renders our migrate in background code path useless, and

[PATCH 2/4] KVM test: unattended_install: Remove unused variable images_dir

2011-09-27 Thread Lucas Meneghel Rodrigues
Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com --- client/tests/kvm/tests/unattended_install.py |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/client/tests/kvm/tests/unattended_install.py b/client/tests/kvm/tests/unattended_install.py index 7d90a91..7655bc1

Qemu/KVM is 3x slower under libvirt

2011-09-27 Thread Reeted
I repost this, this time by also including the libvirt mailing list. Info on my libvirt: it's the version in Ubuntu 11.04 Natty which is 0.8.8-1ubuntu6.5 . I didn't recompile this one, while Kernel and qemu-kvm are vanilla and compiled by hand as described below. My original message follows:

Re: [PATCH v3 1/6] iommu/core: split mapping to page sizes as supported by the hardware

2011-09-27 Thread Roedel, Joerg
On Tue, Sep 27, 2011 at 09:28:37AM -0400, Ohad Ben-Cohen wrote: So you're suggesting to re-implement find_next_bit() using ffs()/fls() and shifting ? No. I suggest a simpler and shorter algorithm using the bit helpers. Something like that: min_idx = __ffs(iommu_page_sizes);

Re: [KVM-autotest][PATCH] cgroup test with KVM guest +first subtests

2011-09-27 Thread Lucas Meneghel Rodrigues
On 09/22/2011 01:29 PM, Lukas Doktor wrote: Hi guys, Do you remember the discussion about cgroup testing in autotest vs. LTP? I hope there won't be any doubts about this one as ground_test (+ first 2 subtests) are strictly focused on cgroups features enforced on KVM guest systems. Also more

Re: [Qemu-devel] RFC [v2]: vfio / device assignment -- layout of device fd files

2011-09-27 Thread Scott Wood
On 09/26/2011 07:45 PM, Alex Williamson wrote: On Mon, 2011-09-26 at 18:59 -0500, Scott Wood wrote: On 09/26/2011 01:34 PM, Alex Williamson wrote: /* Reset the device */ #define VFIO_DEVICE_RESET _IO(, ,) What generic way do we have to do this? We should probably have a

Re: How many threads should a kvm vm be starting?

2011-09-27 Thread Thomas Fjellstrom
On September 27, 2011, Avi Kivity wrote: On 09/27/2011 03:29 AM, Thomas Fjellstrom wrote: I just noticed something interesting, a virtual machine on one of my servers seems to have 69 threads (including the main thread). Other guests on the machine only have a couple threads. Is this

order 1 page allocation failures

2011-09-27 Thread Thomas Fjellstrom
Hi, I've been having some issues with KVM recently where one or more vms will cause page allocation failure messages, usually with the backtrace including networking functions, example follows: [362409.429944] kvm: page allocation failure: order:1, mode:0x20 [362409.429957] Pid: 3453, comm:

Re: [kvm] Weird console issue.

2011-09-27 Thread Robin Lee Powell
On Mon, Sep 26, 2011 at 01:45:07AM -0700, Robin Lee Powell wrote: Everything seems to work until it asks for a password. As soon as that happens, any character I type causes it to respond as though I'd hit enter. This continues until /bin/login completes and a new getty starts, at which

tcpdump locks up kvm host for a while.

2011-09-27 Thread Robin Lee Powell
I can essentially test this at will, so feel free to ask for debugging steps. My host and VMs are all Fedora 15. Every time I run tcpdump on a VM, it hangs for a while. It uses all available CPU, virsh list hangs trying to show it, and I can't shut it down except by killing the qemu-kvm

Re: [Qemu-devel] RFC [v2]: vfio / device assignment -- layout of device fd files

2011-09-27 Thread Alex Williamson
On Tue, 2011-09-27 at 16:28 -0500, Scott Wood wrote: On 09/26/2011 07:45 PM, Alex Williamson wrote: On Mon, 2011-09-26 at 18:59 -0500, Scott Wood wrote: On 09/26/2011 01:34 PM, Alex Williamson wrote: /* Reset the device */ #define VFIO_DEVICE_RESET _IO(, ,) What

[PATCH v4] kvm tools: Support multiple net devices

2011-09-27 Thread Sasha Levin
This patch adds support for multiple network devices. The command line syntax changes to the following: --network/-n [mode=tap/user/none][,guest_ip=ip][,host_ip= ip][,guest_mac=mac][,script=file] Each of the parameters is optional, and the config defaults to a TAP based networking with a

Re: [PATCH 5/5] KVM: PPC: booke: Improve timer register emulation

2011-09-27 Thread Alexander Graf
On 27.09.2011, at 18:01, Bhushan Bharat-R65777 wrote: -Original Message- From: kvm-ppc-ow...@vger.kernel.org [mailto:kvm-ppc- ow...@vger.kernel.org] On Behalf Of Alexander Graf Sent: Tuesday, September 27, 2011 1:45 PM To: Wood Scott-B07421 Cc: Liu Yu-B13201; Wood Scott-B07421;

RE: [PATCH 5/5] KVM: PPC: booke: Improve timer register emulation

2011-09-27 Thread Bhushan Bharat-R65777
-Original Message- From: kvm-ppc-ow...@vger.kernel.org [mailto:kvm-ppc- ow...@vger.kernel.org] On Behalf Of Alexander Graf Sent: Tuesday, September 27, 2011 9:39 PM To: Bhushan Bharat-R65777 Cc: Wood Scott-B07421; Liu Yu-B13201; kvm-ppc@vger.kernel.org Subject: Re: [PATCH 5/5]