Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Amit Shah
On (Thu) 16 Jun 2011 [13:38:49], Luiz Capitulino wrote: If I start qemu with: # qemu -hda disks/test.img -enable-kvm -m 1G -snapshot \ -device virtio-serial \ -chardev socket,host=localhost,port=1234,server,nowait,id=foo \ -device

[Qemu-devel] [PATCH] multiboot: Support commas in module parameters

2011-06-17 Thread Adam Lackorzynski
Support commas in the parameter list of multiboot modules, by using double commas (via get_opt_value()). Signed-off-by: Adam Lackorzynski a...@os.inf.tu-dresden.de --- hw/multiboot.c | 18 +- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git a/hw/multiboot.c

Re: [Qemu-devel] [PATCH] CPU consumption optimization of 'qemu-img convert' using bdrv_is_allocated()

2011-06-17 Thread Stefan Hajnoczi
On Thu, Jun 16, 2011 at 9:10 AM, Dmitry Konishchev konishc...@gmail.com wrote: On Wed, Jun 15, 2011 at 5:57 PM, Stefan Hajnoczi stefa...@gmail.com wrote: Anyway, bdrv_getlength() will return the total_sectors value instead of calling into raw-posix.c .bdrv_getlength().  That's why it should be

[Qemu-devel] [PATCH] Fix ioapic vmstate

2011-06-17 Thread Pavel Dovgaluk
This patch fixes save/restore vmstate of IOAPIC. When irr member of IOAPICState is not saved and loaded, restoring becomes non-deterministic, because irr is kept from state of VM that was before loading. Signed-off-by: Pavel Dovgalyuk pavel.dovga...@gmail.com --- hw/ioapic.c |1 + 1

Re: [Qemu-devel] [PATCH] Fix ioapic vmstate

2011-06-17 Thread Jan Kiszka
On 2011-06-17 09:30, Pavel Dovgaluk wrote: This patch fixes save/restore vmstate of IOAPIC. When irr member of IOAPICState is not saved and loaded, restoring becomes non-deterministic, because irr is kept from state of VM that was before loading. Signed-off-by: Pavel Dovgalyuk

Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Markus Armbruster
Amit Shah amit.s...@redhat.com writes: On (Thu) 16 Jun 2011 [13:38:49], Luiz Capitulino wrote: If I start qemu with: # qemu -hda disks/test.img -enable-kvm -m 1G -snapshot \ -device virtio-serial \ -chardev socket,host=localhost,port=1234,server,nowait,id=foo \

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Kevin Wolf
Am 16.06.2011 16:38, schrieb Marcelo Tosatti: On Thu, Jun 16, 2011 at 02:35:37PM +0200, Kevin Wolf wrote: Am 14.06.2011 20:18, schrieb Stefan Hajnoczi: Overview This patch series adds image streaming support for QED image files. Other image formats can also be supported in the

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Kevin Wolf
Am 16.06.2011 16:52, schrieb Marcelo Tosatti: On Thu, Jun 16, 2011 at 03:08:30PM +0200, Kevin Wolf wrote: Am 16.06.2011 14:49, schrieb Avi Kivity: On 06/16/2011 03:35 PM, Kevin Wolf wrote: * Image streaming is a normal image file plus copy-on-read plus a background task that copies data from

[Qemu-devel] [Bug 754591] Re: NIC doesn't work when it had been used before

2011-06-17 Thread Yongjie Ren
It was fixed in kvm.git. This bug is fixed in the kvm upstream. It was fixed in kvm.git f8fcfd7 by alex.william...@redhat.com. commit f8fcfd775523347afe460dc3a0f45d0479e784a2 Author: Alex Williamson alex.william...@redhat.com Date: Tue May 10 10:02:39 2011 -0600 KVM: Use

Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Amit Shah
On (Fri) 17 Jun 2011 [09:47:31], Markus Armbruster wrote: Amit Shah amit.s...@redhat.com writes: On (Thu) 16 Jun 2011 [13:38:49], Luiz Capitulino wrote: If I start qemu with: # qemu -hda disks/test.img -enable-kvm -m 1G -snapshot \ -device virtio-serial \ -chardev

[Qemu-devel] [PATCH] usb-storage: Turn drive serial into a qdev property usb-storage.serial

2011-06-17 Thread Markus Armbruster
It needs to be a qdev property, because it belongs to the drive's guest part. Precedence: commit a0fef654 and 6ced55a5. Bonus: info qtree now shows the serial number. Signed-off-by: Markus Armbruster arm...@redhat.com --- hw/usb-msd.c | 14 +++--- 1 files changed, 11 insertions(+), 3

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Stefan Hajnoczi
On Fri, Jun 17, 2011 at 9:36 AM, Kevin Wolf kw...@redhat.com wrote: By the way, we'll get problems with the colon syntax. Without -blockdev we'll have to invent a new syntax, maybe with brackets: blkstream:[nbd:localhost]:out.qcow2 Embedding block driver options in filenames is getting worse

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Kevin Wolf
Am 17.06.2011 10:57, schrieb Stefan Hajnoczi: On Fri, Jun 17, 2011 at 9:36 AM, Kevin Wolf kw...@redhat.com wrote: By the way, we'll get problems with the colon syntax. Without -blockdev we'll have to invent a new syntax, maybe with brackets: blkstream:[nbd:localhost]:out.qcow2 Embedding

[Qemu-devel] [PATCH v2] Register Linux dyntick timer as per-thread signal

2011-06-17 Thread Jan Kiszka
Derived from kvm-tool patch http://thread.gmane.org/gmane.comp.emulators.kvm.devel/74309 Ingo Molnar pointed out that sending the timer signal to the whole process, just blocking it everywhere, is suboptimal with an increasing number of threads. QEMU is also using this pattern so far. Linux

Re: [Qemu-devel] [RFC][PATCH] Register Linux dyntick timer as per-thread signal

2011-06-17 Thread Jan Kiszka
On 2011-06-16 17:24, Alexandre Raymond wrote: Hi Jan, On Thu, Jun 16, 2011 at 5:31 AM, Jan Kiszka jan.kis...@siemens.com wrote: Ingo Molnar pointed out that sending the timer signal to the whole process, just blocking it everywhere, is suboptimal with an increasing number of threads. QEMU

[Qemu-devel] [PATCH] fix network interface tap backend

2011-06-17 Thread Christoph Egger
Fix network interface tap backend work on NetBSD. It uses an ioctl to get the tap name. From Manuel Bouyer bou...@netbsd.org Signed-off-by: Christoph Egger christoph.eg...@amd.com diff --git a/net/tap-bsd.c b/net/tap-bsd.c index 2f3efde..577aafe 100644 --- a/net/tap-bsd.c +++ b/net/tap-bsd.c

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Anthony Liguori
On 06/17/2011 03:36 AM, Kevin Wolf wrote: Am 16.06.2011 16:52, schrieb Marcelo Tosatti: On Thu, Jun 16, 2011 at 03:08:30PM +0200, Kevin Wolf wrote: Over this scheme, you'd have: 1) Block copy. Reopen image to be copied with blkstream:/path/to/current-image:/path/to/destination-image,

Re: [Qemu-devel] [Xen-devel] [PATCH 3/3] xen: implement unplug protocol in xen_platform

2011-06-17 Thread Anthony PERARD
On Thu, Jun 16, 2011 at 17:05, stefano.stabell...@eu.citrix.com wrote: From: Stefano Stabellini stefano.stabell...@eu.citrix.com The unplug protocol is necessary to support PV drivers in the guest: the drivers expect to be able to unplug emulated disks and nics before initializing the Xen PV

Re: [Qemu-devel] Aboriginal Linux 1.0.2: linux-2.6.39 system images for a dozen targets.

2011-06-17 Thread Rob Landley
On 06/16/2011 12:45 PM, Mulyadi Santosa wrote: On Thu, Jun 16, 2011 at 19:34, Rob Landley r...@landley.net wrote: Aboriginal Linux's motto is we cross compile so you don't have to. Hi Rob Thanks for your kind work I began to learn about building cross compiler and found your

[Qemu-devel] [PATCH v4 1/2] pxa2xx_lcd: add proper rotation support

2011-06-17 Thread Vasily Khoruzhick
Until now, pxa2xx_lcd only supported 90deg rotation, but some machines (for example Zipit Z2) needs 270deg rotation. Signed-off-by: Vasily Khoruzhick anars...@gmail.com --- v2: codestyle fixes v3: fix dpy_update calls for 180 and 360 deg. rotation. v4: codestyle fixes; replace atoi with strtol

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Marcelo Tosatti
On Thu, Jun 16, 2011 at 04:30:18PM +0100, Stefan Hajnoczi wrote: On Thu, Jun 16, 2011 at 11:52:43AM -0300, Marcelo Tosatti wrote: This approach does not use the backing file feature? Not directly. For block copy, the destination (or source) images might share the same backing file (this is a

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Marcelo Tosatti
On Fri, Jun 17, 2011 at 10:36:21AM +0200, Kevin Wolf wrote: Am 16.06.2011 16:52, schrieb Marcelo Tosatti: On Thu, Jun 16, 2011 at 03:08:30PM +0200, Kevin Wolf wrote: Am 16.06.2011 14:49, schrieb Avi Kivity: On 06/16/2011 03:35 PM, Kevin Wolf wrote: * Image streaming is a normal image file

[Qemu-devel] [PATCH 1/2] qemu-x86: Add tsc_khz option to -cpu

2011-06-17 Thread Joerg Roedel
To let the user configure the desired tsc frequency for the guest if running in KVM. Signed-off-by: Joerg Roedel joerg.roe...@amd.com --- target-i386/cpu.h |1 + target-i386/cpuid.c | 10 ++ 2 files changed, 11 insertions(+), 0 deletions(-) diff --git a/target-i386/cpu.h

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Stefan Hajnoczi
On Fri, Jun 17, 2011 at 10:22 AM, Kevin Wolf kw...@redhat.com wrote: Am 17.06.2011 10:57, schrieb Stefan Hajnoczi: On Fri, Jun 17, 2011 at 9:36 AM, Kevin Wolf kw...@redhat.com wrote: By the way, we'll get problems with the colon syntax. Without -blockdev we'll have to invent a new syntax,

[Qemu-devel] [PATCH] target-i386: fix cmpxchg

2011-06-17 Thread Christoph Egger
Correct emulation of i386 cmpxchg instruction in the case where the comparison outcome is unequal and the memory write causes a page fault. From: Andreas Gustafsson g...@netbsd.org Signed-off-by: Christoph Egger christoph.eg...@amd.com diff --git a/target-i386/translate.c

[Qemu-devel] [PATCH] use mmap to allocate execute memory

2011-06-17 Thread Christoph Egger
Use mmap to allocate executable memory on NetBSD as well. From: Tobias Nygren t...@netbsd.org Signed-off-by: Christoph Egger christoph.eg...@amd.com diff --git a/exec.c b/exec.c index 09928a3..1954a1c 100644 --- a/exec.c +++ b/exec.c @@ -520,7 +520,8 @@ static void code_gen_alloc(unsigned long

Re: [Qemu-devel] how to verify virtio is being used?

2011-06-17 Thread al pat
Hi Kevin, Thanks! Yes, one disk is visible in guest as sdb (partitioned to sdb1), mounted and I write to it. The virtio disk is visible as /dev/vda, (partitioned to vda1), mounted and I write to it. Kernel log on guest - do you mean dmesg? I was trying to trace through the virt io calls to

[Qemu-devel] [PATCH 2/2] qemu-x86: Set tsc_khz in kvm when supported

2011-06-17 Thread Joerg Roedel
Make use of the KVM_TSC_CONTROL feature if available. Signed-off-by: Joerg Roedel joerg.roe...@amd.com --- target-i386/kvm.c | 11 +++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/target-i386/kvm.c b/target-i386/kvm.c index 1ae2d61..2d89544 100644 ---

Re: [Qemu-devel] [PATCH] target-i386: fix cmpxchg

2011-06-17 Thread malc
On Fri, 17 Jun 2011, Christoph Egger wrote: Correct emulation of i386 cmpxchg instruction in the case where the comparison outcome is unequal and the memory write causes a page fault. From: Andreas Gustafsson g...@netbsd.org Signed-off-by: Christoph Egger christoph.eg...@amd.com We've

Re: [Qemu-devel] how to verify virtio is being used?

2011-06-17 Thread Kevin Wolf
Am 16.06.2011 20:57, schrieb al pat: I have posted this on kvm alias, but have not heard back. seeing some inputs. seeking some pointers/guidance as to how to determine virtio is being used... I configured a VM to use block device with if=virtio (create a 1GB disk using dd I exported this

Re: [Qemu-devel] [PATCH] multiboot: Support commas in module parameters

2011-06-17 Thread Kevin Wolf
Am 17.06.2011 08:51, schrieb Adam Lackorzynski: Support commas in the parameter list of multiboot modules, by using double commas (via get_opt_value()). Signed-off-by: Adam Lackorzynski a...@os.inf.tu-dresden.de The one thing the commit message doesn't mention is that this change requires

[Qemu-devel] [PATCH 0/2] Initial TSC-Scaling support for uq/master

2011-06-17 Thread Joerg Roedel
Hi, these two patches implement initial tsc-scaling support for qemu. The TSC frequency for the guest can be configured with -cpu ,tsc_khz=x. If supported by the kernel module qemu will pass the frequency to KVM. The patches apply against current uq/master branch. Regards, Joerg

[Qemu-devel] [PATCH 1/9] PPC: E500: Implement reboot controller

2011-06-17 Thread Alexander Graf
When Linux reboots an e500 VM, it writes to a magic register in the global-utilities device indicated by the device tree. We were not emulating that device so far, rendering the VM reboot-less. This patch implements that device with only the reboot functionality implemented and adds it to the

Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Luiz Capitulino
On Fri, 17 Jun 2011 12:17:36 +0530 Amit Shah amit.s...@redhat.com wrote: On (Thu) 16 Jun 2011 [13:38:49], Luiz Capitulino wrote: If I start qemu with: # qemu -hda disks/test.img -enable-kvm -m 1G -snapshot \ -device virtio-serial \ -chardev

Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Luiz Capitulino
On Fri, 17 Jun 2011 10:16:44 -0300 Luiz Capitulino lcapitul...@redhat.com wrote: On Fri, 17 Jun 2011 12:17:36 +0530 Amit Shah amit.s...@redhat.com wrote: On (Thu) 16 Jun 2011 [13:38:49], Luiz Capitulino wrote: If I start qemu with: # qemu -hda disks/test.img -enable-kvm -m 1G

[Qemu-devel] [PATCH 6/9] PPC: Only set lower 32bits with mtmsr

2011-06-17 Thread Alexander Graf
As Nathan pointed out correctly, the mtmsr instruction does not modify the high 32 bits of MSR. It also doesn't matter if SF is set or not, the instruction always behaves the same. This patch moves it a bit closer to the spec. Reported-by: Nathan Whitehorn nwhiteh...@freebsd.org Signed-off-by:

[Qemu-devel] [PATCH 0/9] PPC patch queue 2011-06-17

2011-06-17 Thread Alexander Graf
The following changes since commit eb47d7c5d96060040931c42773ee07e61e547af9: Peter Maydell (1): hw/9118.c: Implement active-low interrupt support are available in the git repository at: git://repo.or.cz/qemu/agraf.git ppc-next Alexander Graf (8): PPC: E500: Implement reboot

[Qemu-devel] [PATCH 2/9] target-ppc: Handle memory-forced I/O controller access

2011-06-17 Thread Alexander Graf
From: Hervé Poussineau hpous...@reactos.org On at least the PowerPC 601, a direct-store (T=1) with bus unit ID 0x07F is special-cased as memory-forced I/O controller access. It is supposed to be checked immediately if T=1, bypassing all protection mechanisms and acting cache-inhibited and global.

[Qemu-devel] [PATCH v4 2/2] Add support for Zipit Z2 machine

2011-06-17 Thread Vasily Khoruzhick
Zipit Z2 is small PXA270 based handheld. Signed-off-by: Vasily Khoruzhick anars...@gmail.com --- v2: codestyle fixes, added VMStateDescription for LCD device and AER915, traces clean up. v3: no changes v4: no changes Makefile.target |1 + hw/z2.c | 352

[Qemu-devel] [PATCH 9/9] PPC: move TLBs to their own arrays

2011-06-17 Thread Alexander Graf
Until now, we've created a union over multiple different TLB types and allocated that union. While it's a waste of memory (and cache) to allocate TLB information for a TLB type with much information when you only need little, it also inflicts another issue. With the new KVM API, we can now share

[Qemu-devel] [PATCH 4/9] PPC: mpc8544ds: Add hypervisor node

2011-06-17 Thread Alexander Graf
When running a PPC guest with KVM that can do PV operations, we need to indicate the guest which instructions to use for a hypercall and that it is running as KVM guest. This logic was available on openbios based machines already. This patch also adds said functionality to the mpc8544ds machine.

Re: [Qemu-devel] [PATCH 09/14] exec: last_first_tb was only used in !ONLY_USER case

2011-06-17 Thread Peter Maydell
On 2 June 2011 12:53, Juan Quintela quint...@redhat.com wrote: Once there, use a better variable name. Signed-off-by: Juan Quintela quint...@redhat.com Reviewed-by: Peter Maydell peter.mayd...@linaro.org ---  exec.c |   10 +++---  1 files changed, 7 insertions(+), 3 deletions(-) diff

[Qemu-devel] [PATCH 3/9] PPC: calculate kernel, initrd, cmdline locations dynamically

2011-06-17 Thread Alexander Graf
During testing, I was generating a vmlinux binary that easily occupied more than 20MB of RAM. Since the current -kernel code loads the initrd at a fixed address behind the kernel, we were overwriting kernel data when the kernel got too big. To finally get rid of the issue, let's calculate the

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Marcelo Tosatti
On Thu, Jun 16, 2011 at 04:30:18PM +0100, Stefan Hajnoczi wrote: On Thu, Jun 16, 2011 at 11:52:43AM -0300, Marcelo Tosatti wrote: This approach does not use the backing file feature? blkstream block driver: - Maintain in memory whether given block is allocated in local image, if not,

[Qemu-devel] [PATCH 7/9] PPC: E500: Use MAS registers instead of internal TLB representation

2011-06-17 Thread Alexander Graf
The natural format for e500 cores to do TLB manipulation with are the MAS registers. Instead of converting them into some internal representation and back again when the guest reads them, we can just keep the data identical to the way the guest passed it to us. The main advantage of this approach

Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Amit Shah
On (Fri) 17 Jun 2011 [10:16:44], Luiz Capitulino wrote: On Fri, 17 Jun 2011 12:17:36 +0530 Amit Shah amit.s...@redhat.com wrote: On (Thu) 16 Jun 2011 [13:38:49], Luiz Capitulino wrote: If I start qemu with: # qemu -hda disks/test.img -enable-kvm -m 1G -snapshot \

Re: [Qemu-devel] Image streaming and live block copy

2011-06-17 Thread Marcelo Tosatti
On Fri, Jun 17, 2011 at 10:36:21AM +0200, Kevin Wolf wrote: Am 16.06.2011 16:52, schrieb Marcelo Tosatti: On Thu, Jun 16, 2011 at 03:08:30PM +0200, Kevin Wolf wrote: Am 16.06.2011 14:49, schrieb Avi Kivity: On 06/16/2011 03:35 PM, Kevin Wolf wrote: * Image streaming is a normal image file

[Qemu-devel] [V3 1/3] Enhance info block to display hostcache setting

2011-06-17 Thread Supriya Kannery
Enhance info block to display hostcache setting for each block device. Example: (qemu) info block ide0-hd0: type=hd removable=0 file=../rhel6-32.qcow2 ro=0 drv=qcow2 encrypted=0 Enhanced to display hostcache setting: (qemu) info block ide0-hd0: type=hd removable=0 hostcache=true

Re: [Qemu-devel] how to verify virtio is being used?

2011-06-17 Thread Kevin Wolf
Am 17.06.2011 13:39, schrieb al pat: Hi Kevin, Thanks! Yes, one disk is visible in guest as sdb (partitioned to sdb1), mounted and I write to it. The virtio disk is visible as /dev/vda, (partitioned to vda1), mounted and I write to it. Then it's using virtio-blk. Kernel log on guest

[Qemu-devel] [V3 0/3]block: Dynamically change hostcache setting using new block_set command

2011-06-17 Thread Supriya Kannery
Currently host page cache setting for a block device cannot be changed without restarting a running VM. Following patchset [V3] is for enabling dynamic change of hostcache setting for block devices through qemu monitor and QMP. Code changes are based on patches from Christoph Hellwig and Prerna

[Qemu-devel] [PATCH 8/9] PPC: 440: Use 440 style MMU as default, so Qemu knows the MMU type

2011-06-17 Thread Alexander Graf
We have some KVM interaction code in Qemu that tries to be clever and ignore some capabilities when running on BookE style MMUs. Unfortunately, the default CPU bamboo was defaulting to was not a BookE-style MMU, resulting in the check to fail. With this patch, guests can run again on 440 with

[Qemu-devel] [V2 3/3] Command block_set for dynamic block params change

2011-06-17 Thread Supriya Kannery
New command block_set added for dynamically changing any of the block device parameters. For now, dynamic setting of hostcache params using this command is implemented. Other block device parameters, can be integrated in similar lines. Signed-off-by: Supriya Kannery supri...@in.ibm.com ---

[Qemu-devel] [V3 2/3] Error classes for file reopen and device insertion

2011-06-17 Thread Supriya Kannery
New error classes defined for cases where device not inserted and file reopen failed. Signed-off-by: Supriya Kannery supri...@in.ibm.com --- qerror.c |8 qerror.h |6 ++ 2 files changed, 14 insertions(+) Index: qemu/qerror.c

Re: [Qemu-devel] [V2 3/3] Resend Command block_set for dynamic block params change

2011-06-17 Thread Supriya Kannery
Resending block_set command patch to avoid mismatch between paramname defined for human monitor and name used in do_block_set for reading name of block device parameter. --- New command block_set added for dynamically changing any of the block device parameters. For now, dynamic setting of

Re: [Qemu-devel] [PATCH v2] Register Linux dyntick timer as per-thread signal

2011-06-17 Thread Richard Henderson
On 06/17/2011 02:25 AM, Jan Kiszka wrote: Derived from kvm-tool patch http://thread.gmane.org/gmane.comp.emulators.kvm.devel/74309 Ingo Molnar pointed out that sending the timer signal to the whole process, just blocking it everywhere, is suboptimal with an increasing number of threads.

Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Luiz Capitulino
On Fri, 17 Jun 2011 21:39:26 +0530 Amit Shah amit.s...@redhat.com wrote: On (Fri) 17 Jun 2011 [10:16:44], Luiz Capitulino wrote: On Fri, 17 Jun 2011 12:17:36 +0530 Amit Shah amit.s...@redhat.com wrote: On (Thu) 16 Jun 2011 [13:38:49], Luiz Capitulino wrote: If I start qemu with:

Re: [Qemu-devel] QEMU API for modelling of user's hardware

2011-06-17 Thread wzab
I've tried to collect some information from the QEMU sources (I've built my local LXR service with QEMU sources tom make it easier ;-) ), and from different web sources. As a result I've prepared a model of Bus Mastering PCI device which is a simple AES256 accelerator. Sources are available on

[Qemu-devel] [PATCH v2] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Luiz Capitulino
If I start qemu with: # qemu -hda disks/test.img -enable-kvm -m 1G -snapshot \ -device virtio-serial \ -chardev socket,host=localhost,port=1234,server,nowait,id=foo \ -device virtserialport,chardev=foo,name=org.qemu.guest_agent I get a segfault when booting a Fedora

Re: [Qemu-devel] [PATCH v5 2/5] guest agent: qemu-ga daemon

2011-06-17 Thread Michael Roth
On 06/16/2011 01:42 PM, Luiz Capitulino wrote: On Tue, 14 Jun 2011 15:06:22 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: This is the actual guest daemon, it listens for requests over a virtio-serial/isa-serial/unix socket channel and routes them through to dispatch routines, and writes

Re: [Qemu-devel] [PATCH v5 3/5] guest agent: add guest agent RPCs/commands

2011-06-17 Thread Michael Roth
On 06/16/2011 01:52 PM, Luiz Capitulino wrote: On Tue, 14 Jun 2011 15:06:23 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: This adds the initial set of QMP/QAPI commands provided by the guest agent: guest-sync guest-ping guest-info guest-shutdown guest-file-open guest-file-read

Re: [Qemu-devel] [PATCH v5 2/5] guest agent: qemu-ga daemon

2011-06-17 Thread Luiz Capitulino
On Fri, 17 Jun 2011 14:21:31 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: On 06/16/2011 01:42 PM, Luiz Capitulino wrote: On Tue, 14 Jun 2011 15:06:22 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: This is the actual guest daemon, it listens for requests over a

Re: [Qemu-devel] [PATCH 2/9] target-ppc: Handle memory-forced I/O controller access

2011-06-17 Thread Andreas Färber
Am 17.06.2011 um 16:43 schrieb Alexander Graf: From: Hervé Poussineau hpous...@reactos.org On at least the PowerPC 601, a direct-store (T=1) with bus unit ID 0x07F is special-cased as memory-forced I/O controller access. It is supposed to be checked immediately if T=1, bypassing all

[Qemu-devel] [PATCH 0/3] ppc: booke206: KVM MMU API and info tlb

2011-06-17 Thread Scott Wood
This depends on these qemu patches: http://patchwork.ozlabs.org/patch/100826/ (PPC: E500: Use MAS registers instead of internal TLB representation) http://patchwork.ozlabs.org/patch/100821/ (PPC: move TLBs to their own arrays) For this functionality to work with KVM, this kernel patch is

[Qemu-devel] [PATCH 3/3] ppc: booke206: add info tlb support

2011-06-17 Thread Scott Wood
Signed-off-by: Scott Wood scottw...@freescale.com --- hmp-commands.hx |2 +- monitor.c |5 ++- target-ppc/cpu.h|2 + target-ppc/helper.c | 89 +++ 4 files changed, 95 insertions(+), 3 deletions(-) diff --git

Re: [Qemu-devel] [PATCH] Fix ioapic vmstate

2011-06-17 Thread Andreas Färber
Am 17.06.2011 um 09:30 schrieb Pavel Dovgaluk: This patch fixes save/restore vmstate of IOAPIC. When irr member of IOAPICState is not saved and loaded, restoring becomes non-deterministic, because irr is kept from state of VM that was before loading. Signed-off-by: Pavel Dovgalyuk

[Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-17 Thread Scott Wood
Share the TLB array with KVM. This allows us to set the initial TLB both on initial boot and reset, is useful for debugging, and could eventually be used to support migration. Signed-off-by: Scott Wood scottw...@freescale.com --- hw/ppce500_mpc8544ds.c |2 + target-ppc/cpu.h |2 +

[Qemu-devel] [PATCH 2/3] ppc: booke206: use MAV=2.0 TSIZE definition, fix 4G pages

2011-06-17 Thread Scott Wood
This definition is backward compatible with MAV=1.0 as long as the guest does not set reserved bits in MAS1/MAS4. Also, fix the shift in booke206_tlb_to_page_size -- it's the base that should be able to hold a 4G page size, not the shift count. Signed-off-by: Scott Wood scottw...@freescale.com

Re: [Qemu-devel] [PATCH v5 2/5] guest agent: qemu-ga daemon

2011-06-17 Thread Michael Roth
On 06/17/2011 03:13 PM, Luiz Capitulino wrote: On Fri, 17 Jun 2011 14:21:31 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: On 06/16/2011 01:42 PM, Luiz Capitulino wrote: On Tue, 14 Jun 2011 15:06:22 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: This is the actual guest daemon,

Re: [Qemu-devel] [PATCH 2/9] target-ppc: Handle memory-forced I/O controller access

2011-06-17 Thread Alexander Graf
On 17.06.2011, at 21:34, Andreas Färber wrote: Am 17.06.2011 um 16:43 schrieb Alexander Graf: From: Hervé Poussineau hpous...@reactos.org On at least the PowerPC 601, a direct-store (T=1) with bus unit ID 0x07F is special-cased as memory-forced I/O controller access. It is supposed to

Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-17 Thread Alexander Graf
On 17.06.2011, at 22:39, Scott Wood wrote: Share the TLB array with KVM. This allows us to set the initial TLB both on initial boot and reset, is useful for debugging, and could eventually be used to support migration. Signed-off-by: Scott Wood scottw...@freescale.com ---

Re: [Qemu-devel] [PATCH 3/3] ppc: booke206: add info tlb support

2011-06-17 Thread Alexander Graf
On 17.06.2011, at 22:39, Scott Wood wrote: Signed-off-by: Scott Wood scottw...@freescale.com --- hmp-commands.hx |2 +- monitor.c |5 ++- target-ppc/cpu.h|2 + target-ppc/helper.c | 89 +++ 4 files changed, 95

Re: [Qemu-devel] [PATCH v5 3/5] guest agent: add guest agent RPCs/commands

2011-06-17 Thread Luiz Capitulino
On Fri, 17 Jun 2011 15:19:56 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: On 06/16/2011 01:52 PM, Luiz Capitulino wrote: On Tue, 14 Jun 2011 15:06:23 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: This adds the initial set of QMP/QAPI commands provided by the guest agent:

Re: [Qemu-devel] [PATCH v5 2/5] guest agent: qemu-ga daemon

2011-06-17 Thread Luiz Capitulino
On Fri, 17 Jun 2011 16:25:32 -0500 Michael Roth mdr...@linux.vnet.ibm.com wrote: On 06/17/2011 03:13 PM, Luiz Capitulino wrote: On Fri, 17 Jun 2011 14:21:31 -0500 Michael Rothmdr...@linux.vnet.ibm.com wrote: On 06/16/2011 01:42 PM, Luiz Capitulino wrote: On Tue, 14 Jun 2011 15:06:22

Re: [Qemu-devel] [PATCH] virtio-serial: Fix segfault on guest boot

2011-06-17 Thread Amit Shah
On (Fri) 17 Jun 2011 [15:08:11], Luiz Capitulino wrote: if (!cpkt.value) { -error_report(virtio-serial-bus: Guest failure in adding device %s\n, - vser-bus.qbus.name); -break; +error_report(virtio-serial-bus:

[Qemu-devel] [Bug 799036] [NEW] NIC assignment order in command line make some NIC can't work

2011-06-17 Thread Yongjie Ren
Public bug reported: Environment: Host OS (ia32/ia32e/IA64):All Guest OS (ia32/ia32e/IA64):ia32e Guest OS Type (Linux/Windows):Linux kvm.git Commit:681fb677ace0754589792c92b8dbee9884d07158 qemu-kvm Commit:05f1737582ab6c075476bde931c5eafbc62a9349 Host Kernel Version:3.0.0-rc2+

[Qemu-devel] [PATCH] fix MinGW compilation when --enable-vnc-jpeg is specified

2011-06-17 Thread Roy Tam
This patch fix conflicting types for 'INT32' in basetsd.h in including qemu-common.h first. Sign-off-by: Roy Tam roy...@gmail.com -- diff --git a/ui/vnc-enc-tight.c b/ui/vnc-enc-tight.c index 87fdf35..1591df0 100644 --- a/ui/vnc-enc-tight.c +++ b/ui/vnc-enc-tight.c @@ -28,6 +28,8 @@ #include