[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: [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(, ,) >

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 proce

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

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: kvm

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. > > > >

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 probab

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 s

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); w

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:

[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 --- 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 100644 --- a/client

[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 i

[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 --- 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 100644 --- a/client

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

2011-09-27 Thread Lucas Meneghel Rodrigues
Signed-off-by: Lucas Meneghel Rodrigues --- 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 100644 --- a/client

[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 th

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: >>

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: kvm linux-next tree

2011-09-27 Thread Stephen Rothwell
Hi Avi, On Tue, 27 Sep 2011 15:25:02 +0300 Avi Kivity 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 Rothwells...@ca

Re: KVM call agenda for September 26

2011-09-27 Thread Juan Quintela
Juan Quintela 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 majord...@vger

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 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 without doing the chang

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

[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 c

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,

[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 c

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 i

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

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 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, size_t size, i

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 major

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 inse

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 Powell wrote: > > > On Mon, Sep 26, 2011 at 04:15:37PM +0800, Emmanuel Noobadmin

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: 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 unsubsc

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. Basica

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 wrote: > > > On Mon, Sep 26, 2011 at 04:15:37PM +0800, Emmanuel Noobadmin > > > wrote: > > >> It's unrelated to what you're actua

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 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

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

2011-09-27 Thread Emmanuel Noobadmin
On 9/27/11, Robin Lee Powell 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 on the storage device.

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 wrote: > > On Mon, Sep 26, 2011 at 01:49:18PM +0800, Emmanuel Noobadmin wrote: > >> On 9/25/11, Robin Lee Powell wrote: > >> > > >> > OK, so I've got a Linux host, and a bunch of Linux VMs. > >> > >

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

2011-09-27 Thread Ingo Molnar
* Sasha Levin 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 active_console; > exte

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