Re: [Qemu-devel] [PATCH v3 01/39] virtio-pci: get config on init

2011-08-07 Thread Avi Kivity
On 08/05/2011 04:52 PM, Anthony Liguori wrote: static void virtio_write_config(PCIDevice *pci_dev, uint32_t address, @@ -689,6 +686,10 @@ void virtio_init_pci(VirtIOPCIProxy *proxy, VirtIODevice *vdev) proxy-host_features |= 0x1 VIRTIO_F_NOTIFY_ON_EMPTY; proxy-host_features |=

Re: [Qemu-devel] [PATCH v3 02/39] pci: add API to get a BAR's mapped address

2011-08-07 Thread Avi Kivity
On 08/05/2011 04:53 PM, Anthony Liguori wrote: On 08/04/2011 08:05 AM, Avi Kivity wrote: This is a hack, for devices that have a back-channel to read this address back outside the normal configuration mechanisms, such as VMware svga. Reviewed-by: Richard Hendersonr...@twiddle.net

Re: [Qemu-devel] [PATCH v3 05/39] cirrus: simplify mmio BAR access functions

2011-08-07 Thread Avi Kivity
On 08/05/2011 05:08 PM, Anthony Liguori wrote: .read = cirrus_mmio_read, .write = cirrus_mmio_write, .endianness = DEVICE_LITTLE_ENDIAN, +.impl = { +.min_access_size = 1, +.max_access_size = 1, +}, }; static const MemoryRegionOps cirrus_mmio_io_ops

Re: [Qemu-devel] [PATCH v3 10/39] Integrate I/O memory regions into qemu

2011-08-07 Thread Avi Kivity
On 08/05/2011 05:15 PM, Anthony Liguori wrote: */ MemoryRegion *get_system_memory(void); +MemoryRegion *get_system_io(void); + @@ -28,6 +28,8 @@ Could you add a doc comment for this function? Done. -- error compiling committee.c: too many arguments to function -- To unsubscribe from

Re: [Qemu-devel] [PATCH v3 14/39] ac97: convert to memory API

2011-08-07 Thread Avi Kivity
On 08/05/2011 07:47 PM, malc wrote: On Fri, 5 Aug 2011, Anthony Liguori wrote: On 08/04/2011 08:06 AM, Avi Kivity wrote: fixes BAR sizing as well. Reviewed-by: Richard Hendersonr...@twiddle.net Signed-off-by: Avi Kivitya...@redhat.com Reviewed-by: Anthony

Re: [Qemu-devel] [PATCH v3 13/39] rtl8139: convert to memory API

2011-08-07 Thread Avi Kivity
On 08/05/2011 05:21 PM, Anthony Liguori wrote: +{ 0, 0x100, 4, .write = rtl8139_ioport_writel, }, +PORTIO_END Hrm, I missed this #define when it was introduced. Elsewhere we use: VMSTATE_END_OF_LIST() DEFINE_PROP_END_OF_LIST() For consistency, we ought to use PORTIO_END_OF_LIST()

Re: [Qemu-devel] [PATCH v3 25/39] ne2000: convert to memory API

2011-08-07 Thread Avi Kivity
On 08/05/2011 06:28 PM, Anthony Liguori wrote: diff --git a/hw/ne2000-isa.c b/hw/ne2000-isa.c index e41dbba..ce7b365 100644 --- a/hw/ne2000-isa.c +++ b/hw/ne2000-isa.c @@ -61,24 +61,18 @@ static const VMStateDescription vmstate_isa_ne2000 = { } }; +#include exec-memory.h + Should

Re: [Qemu-devel] [PATCH v3 28/39] isa-mmio: concert to memory API

2011-08-07 Thread Avi Kivity
On 08/05/2011 06:29 PM, Anthony Liguori wrote: On 08/04/2011 08:06 AM, Avi Kivity wrote: Reviewed-by: Richard Hendersonr...@twiddle.net Signed-off-by: Avi Kivitya...@redhat.com For the subject, s:concert:convert:g Fixed -- error compiling committee.c: too many arguments to function -- To

Re: [Qemu-devel] [PATCH v3 29/39] sun4u: convert to memory API

2011-08-07 Thread Avi Kivity
On 08/05/2011 06:30 PM, Anthony Liguori wrote: -isa_bus_new(s-qdev); +EbusState *s = container_of(pci_dev, EbusState, pci_dev); DO_UPCAST() is the qdev macro for this. Fixed. -- error compiling committee.c: too many arguments to function -- To unsubscribe from this list: send the

Re: [PATCH 3/3] Fix TSC MSR read in nested SVM

2011-08-07 Thread Joerg Roedel
On Fri, Aug 05, 2011 at 12:32:24PM -0300, Marcelo Tosatti wrote: So to keep consistency between the remaining MSRs, i agree with Nadav's patch. Apparently SVM's original patches to return L1 TSC were aimed at fixing the problem VMX is facing now, which is fixed by introduction read_l1_tsc

Re: [Bug 40542] overflow/panic on KVM hipervizor

2011-08-07 Thread Brad Campbell
On 05/08/11 01:33, bugzilla-dae...@bugzilla.kernel.org wrote: https://bugzilla.kernel.org/show_bug.cgi?id=40542 --- Comment #2 from Slawek Rozbickisla...@rozbicki.eu 2011-08-04 17:33:52 --- http port is DNATed by iptables to gentoo guest on tap virtio adapter. This looks like the bug

kvm linux guest hanging for minutes at a time

2011-08-07 Thread Thomas Fjellstrom
Occasionally when there's heavy cpu and/or io load, a kvm guest will lock up for minutes at a time, last occurrence was for about 12 minutes or so, and the guest itself reported: [1992982.639514] Clocksource tsc unstable (delta = -747307707123 ns) in dmesg after it came back. The only other

[Bug 40542] overflow/panic on KVM hipervizor

2011-08-07 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=40542 --- Comment #9 from Anonymous Emailer anonym...@kernel-bugs.osdl.org 2011-08-07 14:55:13 --- Reply-To: b...@fnarfbargle.com On 05/08/11 01:33, bugzilla-dae...@bugzilla.kernel.org wrote: https://bugzilla.kernel.org/show_bug.cgi?id=40542

Re: [Bug 40542] overflow/panic on KVM hipervizor

2011-08-07 Thread Avi Kivity
On 08/07/2011 04:39 PM, Brad Campbell wrote: This looks like the bug I've been fighting with on and off. What's the bugzilla number for that? (unfortunately, no great insight except for CLOSED DUPLICATE) hopefully someone from netdev can take a look, DNAT is seriously broken. -- error

[Bug 40542] overflow/panic on KVM hipervizor

2011-08-07 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=40542 --- Comment #10 from Slawek Rozbicki sla...@rozbicki.eu 2011-08-07 15:10:09 --- My environment needs apache port mapping from kvm guest to public IP address. I am able to run apache benchmark from KVM host just to investigate further this

percpu crash on NetBurst

2011-08-07 Thread Avi Kivity
qemu, under some conditions (-cpu host or -cpu kvm64), erroneously passes family=15 as the virtual cpuid. This causes a BUG() in percpu code during late boot: [ cut here ] kernel BUG at mm/percpu.c:577! invalid opcode: [#1] SMP CPU 0 Modules linked in: stp llc

[Bug 40542] overflow/panic on KVM hipervizor

2011-08-07 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=40542 --- Comment #11 from Avi Kivity a...@redhat.com 2011-08-07 16:01:51 --- On 08/07/2011 04:39 PM, Brad Campbell wrote: This looks like the bug I've been fighting with on and off. What's the bugzilla number for that? (unfortunately, no great

Re: [PATCH] kvm tools: Split 'kvm pause' and add 'kvm resume'

2011-08-07 Thread Pekka Enberg
On Sat, 6 Aug 2011, Sasha Levin wrote: This patch seperates 'kvm pause' into 'kvm pause' and 'kvm resume', 'kvm pause' no longer acts as a switch between running and paused state but instead just switches the instance into paused and reume is handled by the new 'kvm resume'. Signed-off-by:

[PATCH resend] kvm tools: Split 'kvm pause' and add 'kvm resume'

2011-08-07 Thread Sasha Levin
This patch seperates 'kvm pause' into 'kvm pause' and 'kvm resume', 'kvm pause' no longer acts as a switch between running and paused state but instead just switches the instance into paused and reume is handled by the new 'kvm resume'. Signed-off-by: Sasha Levin levinsasha...@gmail.com ---

[PATCH 1/4] kvm tools: Make --kvm-dev option shorter

2011-08-07 Thread Asias He
This patch changes --kvm-dev to --dev. Signed-off-by: Asias He asias.he...@gmail.com --- tools/kvm/Documentation/kvm-run.txt |3 +-- tools/kvm/builtin-run.c | 10 +- 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/tools/kvm/Documentation/kvm-run.txt

[PATCH 2/4] kvm tools: Make --virtio-9p option shorter

2011-08-07 Thread Asias He
This patch changes --virtio-9p to --9p because: 1. It's shorter 2. No need to add virtio prior to 9p since we use virtio by default Signed-off-by: Asias He asias.he...@gmail.com --- tools/kvm/builtin-run.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 3/4] kvm tools: Sort options of kvm run command

2011-08-07 Thread Asias He
This patch sort options of kvm run command to make it look tidier and cleaner. Signed-off-by: Asias He asias.he...@gmail.com --- tools/kvm/builtin-run.c | 13 ++--- 1 files changed, 6 insertions(+), 7 deletions(-) diff --git a/tools/kvm/builtin-run.c b/tools/kvm/builtin-run.c index

[PATCH 4/4] kvm tools: Tune --disk option usage information

2011-08-07 Thread Asias He
This patch makes --disk option usage information include rootfs direcotry, since --disk supports both iamges and rootfs dir. Signed-off-by: Asias He asias.he...@gmail.com --- tools/kvm/builtin-run.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/kvm/builtin-run.c

Re: [RFC PATCH]vhost-blk: In-kernel accelerator for virtio block device

2011-08-07 Thread Liu Yuan
On 08/06/2011 02:02 AM, Badari Pulavarty wrote: On 8/5/2011 4:04 AM, Liu Yuan wrote: On 08/05/2011 05:58 AM, Badari Pulavarty wrote: Hi Liu Yuan, I started testing your patches. I applied your kernel patch to 3.0 and applied QEMU to latest git. I passed 6 blockdevices from the host to guest

Re: [Bug 40542] overflow/panic on KVM hipervizor

2011-08-07 Thread Brad Campbell
On 07/08/11 23:08, Avi Kivity wrote: On 08/07/2011 04:39 PM, Brad Campbell wrote: This looks like the bug I've been fighting with on and off. What's the bugzilla number for that? (unfortunately, no great insight except for CLOSED DUPLICATE) hopefully someone from netdev can take a look,

[Bug 40542] overflow/panic on KVM hipervizor

2011-08-07 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=40542 --- Comment #12 from Anonymous Emailer anonym...@kernel-bugs.osdl.org 2011-08-08 01:41:09 --- Reply-To: b...@fnarfbargle.com On 07/08/11 23:08, Avi Kivity wrote: On 08/07/2011 04:39 PM, Brad Campbell wrote: This looks like the bug I've

[RFC] postcopy livemigration proposal

2011-08-07 Thread Isaku Yamahata
This mail is on Yabusame: Postcopy Live Migration for Qemu/KVM on which we'll give a talk at KVM-forum. The purpose of this mail is to letting developers know it in advance so that we can get better feedback on its design/implementation approach early before our starting to implement it.

Re: [RFC PATCH]vhost-blk: In-kernel accelerator for virtio block device

2011-08-07 Thread Badari Pulavarty
On 8/7/2011 6:35 PM, Liu Yuan wrote: On 08/06/2011 02:02 AM, Badari Pulavarty wrote: On 8/5/2011 4:04 AM, Liu Yuan wrote: On 08/05/2011 05:58 AM, Badari Pulavarty wrote: Hi Liu Yuan, I started testing your patches. I applied your kernel patch to 3.0 and applied QEMU to latest git. I passed

Network available but no network connectivity

2011-08-07 Thread Satz Klauer
Hi, currently I fail with a problem that I already had (and never could solve) with QNX 6.4.1, now I tried again with QNX 6.5 but still don't get it working. I start qemu with qemu-system-x86_64 -hda qnx.img -m 512 -net nic -net user (brain-attritional simple and well-working with all other