[Qemu-devel] [PATCH v4 4/6] rename memory_region_init_ram_may_fail to memory_region_init_ram

2014-08-05 Thread Hu Tao
Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- backends/hostmem-ram.c | 4 ++-- include/exec/memory.h | 15 +++ memory.c | 12 ++-- 3 files changed, 15 insertions(+), 16 deletions(-) diff --git a/backends/hostmem-ram.c b/backends/hostmem-ram.c index

[Qemu-devel] [PATCH v4 1/6] memory: add memory_region_init_ram_may_fail() and memory_region_init_ram_ptr_may_fail()

2014-08-05 Thread Hu Tao
These two are almost the same as memory_region_init_ram() and memory_region_init_ram_ptr() except that they have an extra errp parameter to let callers handle error. The purpose is to fix the bug described below. We should have added errp directly to memory_region_ram(), but that mixes updates to

[Qemu-devel] [PATCH v4 0/6] bug fixs for memory backend

2014-08-05 Thread Hu Tao
This series includes: 1. two patches to fix bugs of memory backend. See patch 1-2. 2. Add errp to memory_region_init_ram and memory_region_init_ram_ptr, and add corresponding nofail versions. See patch 3-4. changes to v3: - fix build on 32bit host - add memory API renaming - rebase on latest

[Qemu-devel] [PATCH v4 2/6] exec: improve error handling and reporting in file_ram_alloc() and gethugepagesize()

2014-08-05 Thread Hu Tao
This patch fixes two problems of memory-backend-file: 1. If user adds a memory-backend-file object using object_add command, specifying a non-existing directory for property mem-path, qemu will core dump with message: /nonexistingdir: No such file or directory Bad ram offset

[Qemu-devel] [PATCH v4 5/6] rename memory_region_init_ram_ptr to memory_region_init_ram_ptr_nofail

2014-08-05 Thread Hu Tao
Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- hw/display/g364fb.c | 4 ++-- hw/i386/kvm/pci-assign.c | 3 ++- hw/misc/ivshmem.c| 9 + hw/misc/vfio.c | 3 ++- hw/ppc/spapr.c | 4 ++-- include/exec/memory.h| 16 memory.c

[Qemu-devel] [PATCH v4 6/6] rename memory_region_init_ram_ptr_may_fail to memory_region_init_ram_ptr

2014-08-05 Thread Hu Tao
Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- include/exec/memory.h | 18 +- memory.c | 14 +++--- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/include/exec/memory.h b/include/exec/memory.h index 8bebae8..9107fa5 100644 ---

[Qemu-devel] [PATCH v4 3/6] rename memory_region_init_ram to memory_region_init_ram_nofail

2014-08-05 Thread Hu Tao
Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- hw/alpha/typhoon.c | 2 +- hw/arm/armv7m.c | 6 +++--- hw/arm/cubieboard.c | 2 +- hw/arm/digic_boards.c| 2 +- hw/arm/exynos4210.c | 9

Re: [Qemu-devel] [PATCH 7/7] qapi: drop support for inline subtypes

2014-08-05 Thread Fam Zheng
On Mon, 08/04 22:36, Eric Blake wrote: diff --git a/tests/Makefile b/tests/Makefile index a5d82e1..c54d802 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -193,7 +193,8 @@ check-qapi-schema-y := $(addprefix tests/qapi-schema/, \ comments.json empty.json enum-empty.json

Re: [Qemu-devel] [PATCH] /proc/self/maps content is not correct for a guest

2014-08-05 Thread Riku Voipio
Hi, The patch in patchwork doesn't apply, even with a bit of editing. You need to send the patch to list with git send-email instead of including the patch inline. We have docs at: http://wiki.qemu.org/Contribute/SubmitAPatch Also, please include the description of the patch in the commit

Re: [Qemu-devel] Cc'ing emails [

2014-08-05 Thread Michael Tokarev
05.08.2014 08:41, Chen Gang wrote: Every members have their own tastes, and one working flow may be not suitable for all members. I can understand, and hope other members understand too. At least for me, next, I shall send patch to the members which I can get from 'get_maintainers.pl' and

Re: [Qemu-devel] [PATCH v4 3/6] rename memory_region_init_ram to memory_region_init_ram_nofail

2014-08-05 Thread Hu Tao
On Tue, Aug 05, 2014 at 04:42:37PM +1000, Peter Crosthwaite wrote: On Tue, Aug 5, 2014 at 3:56 PM, Hu Tao hu...@cn.fujitsu.com wrote: Signed-off-by: Hu Tao hu...@cn.fujitsu.com --- hw/alpha/typhoon.c | 2 +- hw/arm/armv7m.c | 6 +++---

Re: [Qemu-devel] [PATCH 4/4] ivshmem: check the value returned by fstat()

2014-08-05 Thread zhanghailiang
On 2014/8/5 14:50, Levente Kurusa wrote: - Original Message - Hi Michael, Thanks for your review of this patch! On Mon, Aug 04, 2014 at 04:25:44PM +0800, zhanghailiang wrote: The function fstat() may fail, so check its return value. Signed-off-by:

[Qemu-devel] [Bug 1350435] Re: tcg.c:1693: tcg fatal error

2014-08-05 Thread Colin Watson
Peter: While this is true, is it actually likely to happen? I thought in our conversations in the past (when I previously attempted to escalate this class of problems via Linaro) it had been fairly clear that this was a very difficult task that isn't likely to be scheduled for the foreseeable

Re: [Qemu-devel] Cc'ing emails [

2014-08-05 Thread Peter Maydell
n 5 August 2014 08:08, Michael Tokarev m...@tls.msk.ru wrote: 05.08.2014 08:41, Chen Gang wrote: Every members have their own tastes, and one working flow may be not suitable for all members. I can understand, and hope other members understand too. At least for me, next, I shall send patch

Re: [Qemu-devel] [PATCH v8 1/5] block: Support Archipelago as a QEMU block backend

2014-08-05 Thread Stefan Hajnoczi
On Mon, Aug 04, 2014 at 05:35:32PM +0300, Chrysostomos Nanakos wrote: VM Image on Archipelago volume is specified like this: file.driver=archipelago,file.volume=volumename[,file.mport=mapperd_port[, file.vport=vlmcd_port][,file.segment=segment_name]] 'archipelago' is the protocol.

[Qemu-devel] [Bug 1350435] Re: tcg.c:1693: tcg fatal error

2014-08-05 Thread Peter Maydell
I think it's likely to happen eventually; it depends rather on the balance between this and other work priorities (at least if it's going to be Linaro doing the work). Regardless, I'm not taking hacky workarounds like this into mainline (hacks are hard to get out once you let them in, and they

[Qemu-devel] [Bug 1352179] Re: could not open disk image

2014-08-05 Thread MS
** Description changed: After restart the server it's show this error: Error starting domain: internal error process exited while connecting to monitor: char device redirected to /dev/pts/1 qemu-kvm: -drive

Re: [Qemu-devel] Are -cdrom/-hda (or -drive if=ide) supposed to work in q35?

2014-08-05 Thread Kevin Wolf
Am 01.08.2014 um 22:10 hat John Snow geschrieben: On 06/12/2014 05:03 AM, Markus Armbruster wrote: Michael Tokarev m...@tls.msk.ru writes: 10.06.2014 10:34, Paolo Bonzini wrote: Il 10/06/2014 08:30, Michael Tokarev ha scritto: Hello. The question is: are the drive shortcuts - -cdrom,

Re: [Qemu-devel] [PATCH] block: add watermark event

2014-08-05 Thread Kevin Wolf
Am 01.08.2014 um 13:39 hat Stefan Hajnoczi geschrieben: On Tue, Jul 08, 2014 at 04:49:24PM +0200, Francesco Romani wrote: @@ -5813,3 +5815,57 @@ void bdrv_flush_io_queue(BlockDriverState *bs) bdrv_flush_io_queue(bs-file); } } + +static bool

Re: [Qemu-devel] [PATCH v8 1/5] block: Support Archipelago as a QEMU block backend

2014-08-05 Thread Kevin Wolf
Am 05.08.2014 um 10:20 hat Stefan Hajnoczi geschrieben: On Mon, Aug 04, 2014 at 05:35:32PM +0300, Chrysostomos Nanakos wrote: VM Image on Archipelago volume is specified like this: file.driver=archipelago,file.volume=volumename[,file.mport=mapperd_port[,

[Qemu-devel] [PATCH v4 00/10] target-arm: Parts of the AArch64 EL2/3 exception model

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Hi, This is a second round of AArch64 EL2/3 patches working on the exception model. Among other things adding HVC/SMC, interrupt routing to EL2/3 and Virtual IRQs/FIQs. The VIRQ/VFIQ support only adds the external signal delivery method. Patch 8

[Qemu-devel] [PATCH v4 01/10] target-arm: Add HCR_EL2

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-arm/cpu.h| 36 target-arm/helper.c | 34 ++ 2 files changed, 70 insertions(+) diff --git

[Qemu-devel] [PATCH v4 02/10] target-arm: Add SCR_EL3

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-arm/cpu.h| 16 +++- target-arm/helper.c | 36 +--- 2 files changed, 48 insertions(+), 4 deletions(-) diff --git

[Qemu-devel] [PATCH v4 03/10] target-arm: A64: Refactor aarch64_cpu_do_interrupt

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Introduce new_el and new_mode in preparation for future patches that add support for taking exceptions to and from EL2 and 3. No functional change. Reviewed-by: Peter Maydell peter.mayd...@linaro.org Signed-off-by: Edgar E. Iglesias

[Qemu-devel] [PATCH v4 04/10] target-arm: Break out exception masking to a separate func

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Reviewed-by: Greg Bellows greg.bell...@linaro.org Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- cpu-exec.c | 5 ++--- target-arm/cpu.h | 15 +++ 2 files changed, 17 insertions(+), 3 deletions(-) diff --git

[Qemu-devel] [PATCH v4 05/10] target-arm: Don't take interrupts targeting lower ELs

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Reviewed-by: Alex Bennée alex.ben...@linaro.org Reviewed-by: Greg Bellows greg.bell...@linaro.org Reviewed-by: Peter Maydell peter.mayd...@linaro.org Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-arm/cpu.h | 7 +++ 1

Re: [Qemu-devel] [PATCH v1 00/16] target-arm: Parts of the AArch64 EL2/3 exception model

2014-08-05 Thread Edgar E. Iglesias
On Fri, Aug 01, 2014 at 03:35:55PM +0100, Peter Maydell wrote: On 30 May 2014 08:28, Edgar E. Iglesias edgar.igles...@gmail.com wrote: This is a second round of AArch64 EL2/3 patches working on the exception model. Among other things adding HVC/SMC, interrupt routing to EL2/3 and Virtual

[Qemu-devel] [PATCH v4 06/10] target-arm: A64: Correct updates to FAR and ESR on exceptions

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Not all exception types update both FAR and ESR. Reviewed-by: Alex Bennée alex.ben...@linaro.org Reviewed-by: Greg Bellows greg.bell...@linaro.org Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-arm/helper-a64.c | 7 +++

[Qemu-devel] [PATCH v4 07/10] target-arm: A64: Emulate the HVC insn

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-arm/cpu.h | 1 + target-arm/helper-a64.c| 1 + target-arm/helper.c| 28 +++- target-arm/helper.h| 1 +

[Qemu-devel] [PATCH v4 08/10] target-arm: A64: Emulate the SMC insn

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-arm/cpu.h | 1 + target-arm/helper-a64.c| 1 + target-arm/helper.c| 6 ++ target-arm/helper.h| 1 + target-arm/internals.h | 6 ++

[Qemu-devel] [PATCH v4 09/10] target-arm: Add IRQ and FIQ routing to EL2 and 3

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Reviewed-by: Greg Bellows greg.bell...@linaro.org Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- target-arm/cpu.h| 12 target-arm/helper.c | 14 ++ 2 files changed, 26 insertions(+) diff --git

[Qemu-devel] [PATCH v4 10/10] target-arm: Add support for VIRQ and VFIQ

2014-08-05 Thread Edgar E. Iglesias
From: Edgar E. Iglesias edgar.igles...@xilinx.com Acked-by: Greg Bellows greg.bell...@linaro.org Signed-off-by: Edgar E. Iglesias edgar.igles...@xilinx.com --- cpu-exec.c | 12 target-arm/cpu.c| 25 +++-- target-arm/cpu.h| 36

[Qemu-devel] [PATCH 1/2] block: Pass errp in blkconf_geometry

2014-08-05 Thread Fam Zheng
This allows us to pass error information to caller. Signed-off-by: Fam Zheng f...@redhat.com --- hw/block/block.c | 18 +- hw/block/virtio-blk.c| 7 +++ hw/ide/qdev.c| 11 --- hw/scsi/scsi-disk.c | 11 --- include/hw/block/block.h

[Qemu-devel] [PATCH 0/2] scsi: Change device init to realize

2014-08-05 Thread Fam Zheng
DeviceClass-init is the old interface, let's convert scsi devices to the new -realize API. A user visible change is the error message shown in qemu-iotests reference output, but I don't know what's the best way to retain it. So please review if and how this could be improved. Fam Fam Zheng (2):

[Qemu-devel] [PATCH 2/2] scsi-bus: Convert DeviceClass init to realize

2014-08-05 Thread Fam Zheng
Replace init/destroy with realize/unrealize in SCSIDeviceClass, which has errp as a parameter. So all the implementations now uses error_setg instead of error_report for reporting error. Also in lsi53c895a, report the error when initializing the if=scsi devices, before dropping it, because the

Re: [Qemu-devel] Are -cdrom/-hda (or -drive if=ide) supposed to work in q35?

2014-08-05 Thread Markus Armbruster
Kevin Wolf kw...@redhat.com writes: Am 01.08.2014 um 22:10 hat John Snow geschrieben: On 06/12/2014 05:03 AM, Markus Armbruster wrote: Michael Tokarev m...@tls.msk.ru writes: 10.06.2014 10:34, Paolo Bonzini wrote: Il 10/06/2014 08:30, Michael Tokarev ha scritto: Hello. The question

Re: [Qemu-devel] [RFC 0/3] QMP: extend BLOCK_IO_ERROR event

2014-08-05 Thread Kevin Wolf
Am 23.07.2014 um 15:17 hat Luiz Capitulino geschrieben: Management software, such as OpenStack and RHEV's vdsm, wants to be able to allocate VM disk space on demand. The basic use case is to start a VM with a small disk and then the disk is enlarged when QEMU hits a ENOSPC condition. To

Re: [Qemu-devel] [PATCH 2/2] hw/arm/vexpress: add SP810 to the vexpress

2014-08-05 Thread Aggeler Fabian
On 17 Jul 2014, at 01:29, Peter Crosthwaite peter.crosthwa...@xilinx.com wrote: On Thu, Jul 17, 2014 at 1:05 AM, Alex Bennée alex.ben...@linaro.org wrote: Fabian Aggeler writes: The SP810, which is present in the Versatile Express motherboards, allows to set the timing reference to

Re: [Qemu-devel] [PATCH v3 00/21] target-mips: add MIPS64R6 Instruction Set support

2014-08-05 Thread Leon Alrae
ping http://patchwork.ozlabs.org/patch/365066/ http://patchwork.ozlabs.org/patch/365042/ http://patchwork.ozlabs.org/patch/365046/ http://patchwork.ozlabs.org/patch/365056/ http://patchwork.ozlabs.org/patch/365059/ On 27/06/2014 16:21, Leon Alrae wrote: The following patchset implements MIPS64

Re: [Qemu-devel] Are -cdrom/-hda (or -drive if=ide) supposed to work in q35?

2014-08-05 Thread Kevin Wolf
Am 05.08.2014 um 11:13 hat Markus Armbruster geschrieben: Kevin Wolf kw...@redhat.com writes: Am 01.08.2014 um 22:10 hat John Snow geschrieben: On 06/12/2014 05:03 AM, Markus Armbruster wrote: Michael Tokarev m...@tls.msk.ru writes: 10.06.2014 10:34, Paolo Bonzini wrote: Il

Re: [Qemu-devel] Are -cdrom/-hda (or -drive if=ide) supposed to work in q35?

2014-08-05 Thread Michael Tokarev
05.08.2014 13:30, Kevin Wolf wrote: [] However, I don't think we should keep if=ide broken on Q35 just to motivate such work. I agree for the moment. It only becomes a strict requirement once we attempt to change the default machine type. BTW, maybe, at the very least, produce a warning

Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support

2014-08-05 Thread Stefan Hajnoczi
On Tue, Aug 05, 2014 at 11:33:01AM +0800, Ming Lei wrote: These patches bring up below 4 changes: - introduce object allocation pool and apply it to virtio-blk dataplane for improving its performance - introduce selective coroutine bypass mechanism for

Re: [Qemu-devel] Cc'ing emails [

2014-08-05 Thread Markus Armbruster
Michael Tokarev m...@tls.msk.ru writes: 05.08.2014 08:41, Chen Gang wrote: Every members have their own tastes, and one working flow may be not suitable for all members. I can understand, and hope other members understand too. At least for me, next, I shall send patch to the members

[Qemu-devel] [Bug 1352555] Re: iproute2 incompatibility

2014-08-05 Thread Michael Tokarev
** Changed in: qemu Status: New = Incomplete ** Changed in: qemu Status: Incomplete = Invalid -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1352555 Title: iproute2 incompatibility

Re: [Qemu-devel] [PATCH v3 0/4] libqos: add a simple first-fit memory allocator

2014-08-05 Thread Stefan Hajnoczi
On Fri, Aug 01, 2014 at 11:38:55AM -0400, John Snow wrote: This set collects two patches by Marc Marí already on the mailing list, but goes further by adding a simple memory allocator that allows us to track and debug freed memory, and optionally keep track of any leaks. For convenience:

Re: [Qemu-devel] [ANNOUNCE] QEMU 2.1.0 is now available

2014-08-05 Thread Michael S. Tsirkin
On Mon, Aug 04, 2014 at 06:33:08PM -0700, Nishank Trivedi wrote: Hi Michael, I'm trying to send mail to qemu-devel, but it seems to be getting filtering out as I dont see it come over mailing list. Is there a keyword needed in subject for qemu-devel? I'm trying to report issue of x86 host

Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support

2014-08-05 Thread Kevin Wolf
Am 05.08.2014 um 05:33 hat Ming Lei geschrieben: Hi, These patches bring up below 4 changes: - introduce object allocation pool and apply it to virtio-blk dataplane for improving its performance - introduce selective coroutine bypass mechanism for

Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support

2014-08-05 Thread Ming Lei
On Tue, Aug 5, 2014 at 5:38 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Tue, Aug 05, 2014 at 11:33:01AM +0800, Ming Lei wrote: These patches bring up below 4 changes: - introduce object allocation pool and apply it to virtio-blk dataplane for improving its performance

[Qemu-devel] [Bug 1352555] Re: iproute2 incompatibility

2014-08-05 Thread Michael Tokarev
This is a Very Useful BugReport (NOT!). Qemu does not use neither ifupdown nor iproute. Instead, it runs a script, /etc/qemu-ifup by default, which should use whatever commands needed on the given operating system to do whatever configuration of the tap devices which is necessary. But qemu

Re: [Qemu-devel] [PATCH v5 1/2] loader: Add load_image_gzipped function.

2014-08-05 Thread Alex Bennée
Richard W.M. Jones writes: As the name suggests this lets you load a ROM/disk image that is gzipped. It is uncompressed before storing it in guest memory. snip + +/* Is it a gzip-compressed file? */ +if (len 2 || +compressed_data[0] != '\x1f' || +

Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support

2014-08-05 Thread Kevin Wolf
Am 05.08.2014 um 11:50 hat Ming Lei geschrieben: On Tue, Aug 5, 2014 at 5:38 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Tue, Aug 05, 2014 at 11:33:01AM +0800, Ming Lei wrote: These patches bring up below 4 changes: - introduce object allocation pool and apply it to

Re: [Qemu-devel] [PATCH v4 0/2] a few simple trace fixes

2014-08-05 Thread Stefan Hajnoczi
On Fri, Aug 01, 2014 at 05:08:55PM +0100, Alex Bennée wrote: As v3 posted earlier today but with a format string fix which didn't show up in the ust build I tested it on Alex Bennée (2): trace: teach lttng backend to use format strings trace: add some tcg tracing support

Re: [Qemu-devel] [PATCH v5 2/2] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-05 Thread Alex Bennée
Richard W.M. Jones writes: On aarch64 it is the bootloader's job to uncompress the kernel. UEFI and u-boot bootloaders do this automatically when the kernel is gzip-compressed. However the qemu -kernel option does not do this. The following command does not work: qemu-system-aarch64

Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support

2014-08-05 Thread Ming Lei
On Tue, Aug 5, 2014 at 5:48 PM, Kevin Wolf kw...@redhat.com wrote: Am 05.08.2014 um 05:33 hat Ming Lei geschrieben: Hi, These patches bring up below 4 changes: - introduce object allocation pool and apply it to virtio-blk dataplane for improving its performance -

Re: [Qemu-devel] [PATCH 4/4] ivshmem: check the value returned by fstat()

2014-08-05 Thread Michael S. Tsirkin
On Tue, Aug 05, 2014 at 10:00:41AM +0800, zhanghailiang wrote: Hi Michael, Thanks for your review of this patch! On Mon, Aug 04, 2014 at 04:25:44PM +0800, zhanghailiang wrote: The function fstat() may fail, so check its return value. Signed-off-by:

Re: [Qemu-devel] [PATCH v5 1/2] loader: Add load_image_gzipped function.

2014-08-05 Thread Richard W.M. Jones
On Tue, Aug 05, 2014 at 10:57:26AM +0100, Alex Bennée wrote: Richard W.M. Jones writes: As the name suggests this lets you load a ROM/disk image that is gzipped. It is uncompressed before storing it in guest memory. Signed-off-by: Richard W.M. Jones rjo...@redhat.com snip +/*

Re: [Qemu-devel] [PATCH v5 1/2] loader: Add load_image_gzipped function.

2014-08-05 Thread Alex Bennée
Richard W.M. Jones writes: As the name suggests this lets you load a ROM/disk image that is gzipped. It is uncompressed before storing it in guest memory. Signed-off-by: Richard W.M. Jones rjo...@redhat.com snip +/* Is it a gzip-compressed file? */ +if (len 2 || +

[Qemu-devel] [PATCH v6 0/2] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-05 Thread Richard W.M. Jones
Changes from v5: - Fixed warning/error about comparison of uint8_t and char literal. - Compile-tested on x86-64. - Compiled and re-tested on aarch64. Rich.

[Qemu-devel] [PATCH v6 2/2] aarch64: Allow -kernel option to take a gzip-compressed kernel.

2014-08-05 Thread Richard W.M. Jones
On aarch64 it is the bootloader's job to uncompress the kernel. UEFI and u-boot bootloaders do this automatically when the kernel is gzip-compressed. However the qemu -kernel option does not do this. The following command does not work: qemu-system-aarch64 [...] -kernel /boot/vmlinuz

[Qemu-devel] [PATCH v6 1/2] loader: Add load_image_gzipped function.

2014-08-05 Thread Richard W.M. Jones
As the name suggests this lets you load a ROM/disk image that is gzipped. It is uncompressed before storing it in guest memory. Signed-off-by: Richard W.M. Jones rjo...@redhat.com --- hw/core/loader.c| 48 include/hw/loader.h | 1 + 2 files

Re: [Qemu-devel] [PATCH] linux-aio: avoid deadlock in nested aio_poll() calls

2014-08-05 Thread Ming Lei
On Mon, Aug 4, 2014 at 11:56 PM, Stefan Hajnoczi stefa...@redhat.com wrote: If two Linux AIO request completions are fetched in the same io_getevents() call, QEMU will deadlock if request A's callback waits for request B to complete using an aio_poll() loop. This was reported to happen with

Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support

2014-08-05 Thread Ming Lei
On Tue, Aug 5, 2014 at 5:56 PM, Kevin Wolf kw...@redhat.com wrote: Am 05.08.2014 um 11:50 hat Ming Lei geschrieben: On Tue, Aug 5, 2014 at 5:38 PM, Stefan Hajnoczi stefa...@redhat.com wrote: On Tue, Aug 05, 2014 at 11:33:01AM +0800, Ming Lei wrote: These patches bring up below 4 changes:

[Qemu-devel] [PATCH v2 2/2] hw/arm/vexpress: add SP810 to the vexpress

2014-08-05 Thread Fabian Aggeler
The SP810, which is present in the Versatile Express motherboards, allows to set the timing reference to either REFCLK or TIMCLK. QEMU currently sets the SP804 timer to 1MHz by default. To reflect this, we set the TimerEn0Sel, TimerEn1Sel, TimerEn2Sel, and TimerEn3Sel of the system control

[Qemu-devel] [PATCH v2 0/2] Add SP810 to Versatile Express boards

2014-08-05 Thread Fabian Aggeler
Hi, The Versatile Express emulation in QEMU currently does not have a model of the SP810 used in real hardware. The registers provided by this System Controller can be used to set the frequency of the SP804 timers. On newer releases of the board the SP804 is set to 32kHz by default and has to be

[Qemu-devel] [PATCH v2 1/2] hw/misc/arm_sp810: Create SP810 device

2014-08-05 Thread Fabian Aggeler
This adds a device model for the PrimeXsys System Controller (SP810) which is present in the Versatile Express motherboards. It is so far read-only but allows to set the SCCTRL register. Signed-off-by: Fabian Aggeler aggel...@ethz.ch --- default-configs/arm-softmmu.mak | 1 +

Re: [Qemu-devel] Are -cdrom/-hda (or -drive if=ide) supposed to work in q35?

2014-08-05 Thread Markus Armbruster
Kevin Wolf kw...@redhat.com writes: Am 05.08.2014 um 11:13 hat Markus Armbruster geschrieben: Kevin Wolf kw...@redhat.com writes: Am 01.08.2014 um 22:10 hat John Snow geschrieben: On 06/12/2014 05:03 AM, Markus Armbruster wrote: Michael Tokarev m...@tls.msk.ru writes: 10.06.2014

Re: [Qemu-devel] Are -cdrom/-hda (or -drive if=ide) supposed to work in q35?

2014-08-05 Thread Markus Armbruster
Michael Tokarev m...@tls.msk.ru writes: 05.08.2014 13:30, Kevin Wolf wrote: [] However, I don't think we should keep if=ide broken on Q35 just to motivate such work. I agree for the moment. It only becomes a strict requirement once we attempt to change the default machine type. BTW,

[Qemu-devel] [PATCH] linux-user: /proc/self/maps content

2014-08-05 Thread Mikhail Ilyin
Build /proc/self/maps doing a match against guest memory translation table. Output only that map records which are valid for guest memory layout. Signed-off-by: Mikhail Ilyin m.i...@samsung.com --- include/exec/cpu-all.h | 2 ++ linux-user/syscall.c | 25 ++--- 2 files

Re: [Qemu-devel] [PATCH 0/2] scsi: Change device init to realize

2014-08-05 Thread Paolo Bonzini
Il 05/08/2014 11:11, Fam Zheng ha scritto: DeviceClass-init is the old interface, let's convert scsi devices to the new -realize API. A user visible change is the error message shown in qemu-iotests reference output, but I don't know what's the best way to retain it. So please review if and

Re: [Qemu-devel] [PATCH v2 00/16] linux-user fixes improvements

2014-08-05 Thread Riku Voipio
Hi Paul, On Fri, Jun 27, 2014 at 04:30:34PM +0300, Riku Voipio wrote: On Sat, Jun 21, 2014 at 11:52:55PM +0100, Paul Burton wrote: From: Paul Burton paul.bur...@imgtec.com This series fixes a number of bugs in QEMUs linux-user support, some specific to targetting the MIPS architecture

Re: [Qemu-devel] [PATCH 2/2] scsi-bus: Convert DeviceClass init to realize

2014-08-05 Thread Fam Zheng
On Tue, 08/05 17:11, Fam Zheng wrote: Replace init/destroy with realize/unrealize in SCSIDeviceClass, which has errp as a parameter. So all the implementations now uses s/uses/use/ error_setg instead of error_report for reporting error. Also in lsi53c895a, report the error when

Re: [Qemu-devel] [PATCH v1 00/17] dataplane: optimization and multi virtqueue support

2014-08-05 Thread Paolo Bonzini
Il 05/08/2014 12:00, Ming Lei ha scritto: I also think that the device emulation has no business in deciding whether the bypass is used (it depends solely on conditions outside of the device) and that leaking the fd number out of raw-posix is wrong. Both of them are layering violations

Re: [Qemu-devel] [PATCH v4 1/6] memory: add memory_region_init_ram_may_fail() and memory_region_init_ram_ptr_may_fail()

2014-08-05 Thread Eric Blake
On 08/04/2014 11:56 PM, Hu Tao wrote: These two are almost the same as memory_region_init_ram() and memory_region_init_ram_ptr() except that they have an extra errp parameter to let callers handle error. The purpose is to fix the bug described below. We should have added errp directly to

Re: [Qemu-devel] [PATCH] linux-user: /proc/self/maps content

2014-08-05 Thread Riku Voipio
Hi, On Tue, Aug 05, 2014 at 03:10:07PM +0400, Mikhail Ilyin wrote: Build /proc/self/maps doing a match against guest memory translation table. Output only that map records which are valid for guest memory layout. This is clear improvement, for most archs. But seems aarch64, openrisc still leak

[Qemu-devel] [Bug 498035] Re: qemu hangs on shutdown or reboot (XP guest)

2014-08-05 Thread Michael Tokarev
So, is this issue still relevant with current qemu (which happens to be 2.1.0? I remember seeing hangs on reboot/halt like this before too, but on my side they're long gone, I don't observe these hangs anymore. Maybe this bugreport can be closed finally? ** Changed in: qemu Status:

Re: [Qemu-devel] [PATCH v1 01/17] qemu/obj_pool.h: introduce object allocation pool

2014-08-05 Thread Eric Blake
On 08/04/2014 09:33 PM, Ming Lei wrote: This patch introduces object allocation pool for speeding up object allocation in fast path. Signed-off-by: Ming Lei ming@canonical.com --- include/qemu/obj_pool.h | 64 +++ 1 file changed, 64

Re: [Qemu-devel] [PATCH v1 01/17] qemu/obj_pool.h: introduce object allocation pool

2014-08-05 Thread Michael S. Tsirkin
On Tue, Aug 05, 2014 at 05:55:49AM -0600, Eric Blake wrote: On 08/04/2014 09:33 PM, Ming Lei wrote: This patch introduces object allocation pool for speeding up object allocation in fast path. Signed-off-by: Ming Lei ming@canonical.com --- include/qemu/obj_pool.h | 64

Re: [Qemu-devel] [RFC 1/3] qapi: block-core.json: improve query-block doc

2014-08-05 Thread Eric Blake
On 07/23/2014 07:17 AM, Luiz Capitulino wrote: List device models supporting the io-status key. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- qapi/block-core.json | 1 + 1 file changed, 1 insertion(+) diff --git a/qapi/block-core.json b/qapi/block-core.json index

Re: [Qemu-devel] [RFC 2/3] QMP: rate limit BLOCK_IO_ERROR

2014-08-05 Thread Eric Blake
On 07/23/2014 07:17 AM, Luiz Capitulino wrote: This event has the same characteristics of the other rate-limited events, mainly we can emit dozens of it. Rate limit it then. Signed-off-by: Luiz Capitulino lcapitul...@redhat.com --- monitor.c | 1 + 1 file changed, 1 insertion(+) It's not

Re: [Qemu-devel] [RFC 3/3] QMP: extend BLOCK_IO_ERROR event with no-space indicator

2014-08-05 Thread Eric Blake
On 07/23/2014 07:17 AM, Luiz Capitulino wrote: Management software, such as OpenStack and RHEV's vdsm, want to be able to allocate disk space on demand. The basic use case is to start a VM with a small disk and then the disk is enlarged when QEMU hits a ENOSPC condition. I'd still like

Re: [Qemu-devel] Cc'ing emails [

2014-08-05 Thread Chen Gang
On 08/05/2014 04:07 PM, Peter Maydell wrote: n 5 August 2014 08:08, Michael Tokarev m...@tls.msk.ru wrote: 05.08.2014 08:41, Chen Gang wrote: Every members have their own tastes, and one working flow may be not suitable for all members. I can understand, and hope other members understand

Re: [Qemu-devel] [PATCH v1 01/17] qemu/obj_pool.h: introduce object allocation pool

2014-08-05 Thread Eric Blake
On 08/05/2014 06:05 AM, Michael S. Tsirkin wrote: On Tue, Aug 05, 2014 at 05:55:49AM -0600, Eric Blake wrote: On 08/04/2014 09:33 PM, Ming Lei wrote: This patch introduces object allocation pool for speeding up object allocation in fast path. Signed-off-by: Ming Lei ming@canonical.com

Re: [Qemu-devel] [PATCH v1 02/17] dataplane: use object pool to speed up allocation for virtio blk request

2014-08-05 Thread Eric Blake
On 08/04/2014 09:33 PM, Ming Lei wrote: g_slice_new(VirtIOBlockReq), its free pair and access the instance Took me a while to read this. Maybe: Calling g_slice_new(VirtIOBlockReq) and its free pair, and accessing the instance, are a bit slow... is a bit slow since sizeof(VirtIOBlockReq)

Re: [Qemu-devel] [PATCH v1 01/24] qmp: Extract system emulation related code from qmp.c into qmp-system.c

2014-08-05 Thread Eric Blake
On 07/31/2014 11:26 PM, Benoît Canet wrote: This patch will allow to link qmp.o with utility binaries without dragging too s/allow to link/allow linking/ much unrelated object files and externals dependencies. s/dragging too much/dragging in too many/ s/externals/external/ Signed-off-by:

Re: [Qemu-devel] [PATCH v1 02/24] monitor: Make some function public

2014-08-05 Thread Eric Blake
On 07/31/2014 11:27 PM, Benoît Canet wrote: Next commits will split monitor.c in monitor.c and monitor-system.c. s/in/to/ Change some function from static to public in order to prepare this. s/function/functions/ s/this/for this/ Signed-off-by: Benoit Canet ben...@irqsave.net ---

Re: [Qemu-devel] [PATCH v1 05/17] garbage collector: introduced for support of bypass coroutine

2014-08-05 Thread Eric Blake
On 08/04/2014 09:33 PM, Ming Lei wrote: In case of bypass coroutine, some buffers in stack have to convert to survive in the whole I/O submit completion cycle. Garbase collector is one of the best data structure for this purpose, s/Garbase/A garbage/ as I thought of. Signed-off-by:

Re: [Qemu-devel] [PATCH v1 03/24] monitor: Extract monitor-system.h header

2014-08-05 Thread Eric Blake
On 07/31/2014 11:27 PM, Benoît Canet wrote: This header will allow to split monitor in two parts. s/allow to split monitor in/allow splitting the monitor into/ Signed-off-by: Benoit Canet ben...@irqsave.net --- include/monitor/monitor-system.h | 99

Re: [Qemu-devel] [PATCH v1 04/24] monitor: Make monitor_fprintf public before extracting it

2014-08-05 Thread Eric Blake
On 07/31/2014 11:27 PM, Benoît Canet wrote: Signed-off-by: Benoit Canet ben...@irqsave.net --- disas.c | 10 -- include/monitor/monitor.h | 2 ++ monitor.c | 4 ++-- 3 files changed, 4 insertions(+), 12 deletions(-) Reviewed-by: Eric Blake

Re: [Qemu-devel] [PATCH v1 01/17] qemu/obj_pool.h: introduce object allocation pool

2014-08-05 Thread Michael S. Tsirkin
On Tue, Aug 05, 2014 at 06:21:55AM -0600, Eric Blake wrote: On 08/05/2014 06:05 AM, Michael S. Tsirkin wrote: On Tue, Aug 05, 2014 at 05:55:49AM -0600, Eric Blake wrote: On 08/04/2014 09:33 PM, Ming Lei wrote: This patch introduces object allocation pool for speeding up object allocation

Re: [Qemu-devel] [PATCH v1 23/24] qapi: Add a script to filter qmp-commands-old.h to generate a subset of it.

2014-08-05 Thread Eric Blake
On 07/31/2014 11:27 PM, Benoît Canet wrote: Since qmp-command-olds.h is generated from qmp-commands.hx we will sometime want to include only a subset of it. For example when linking qapi block commands with qemu-nbd. Signed-off-by: Benoit Canet ben...@irqsave.net --- Makefile

[Qemu-devel] About the VT-d features that q35 chipset supports

2014-08-05 Thread Le Tan
Hi, I am now adding features to the VT-d emulation for q35 chipset. I think it is good to know what features q35 chipset supports exactly. However I can't find materials about this. Does anyone know this? Or it will be fine if someone can tell me the value of the Capability Register and Extended

Re: [Qemu-devel] [PATCH v4 0/8] don't use Yoda conditions

2014-08-05 Thread Eric Blake
On 08/04/2014 03:19 AM, arei.gong...@huawei.com wrote: From: Gonglei arei.gong...@huawei.com $WHATEVER: don't use 'Yoda conditions' 'Yoda conditions' are not part of idiomatic QEMU coding style, so rewrite them in the more usual order. v4: - trivial typo for patch 1/8 suggested by

Re: [Qemu-devel] [PATCH v2 1/7] block: Add status callback to bdrv_amend_options()

2014-08-05 Thread Eric Blake
On 08/01/2014 05:49 PM, Max Reitz wrote: Depending on the changed options and the image format, bdrv_amend_options() may take a significant amount of time. In these cases, a way to be informed about the operation's status is desirable. Since the operation is rather complex and may

Re: [Qemu-devel] [PATCH v2 2/7] qemu-img: Add progress output for amend

2014-08-05 Thread Eric Blake
On 08/01/2014 05:49 PM, Max Reitz wrote: Now that bdrv_amend_options() supports a status callback, use it to display a progress report. Signed-off-by: Max Reitz mre...@redhat.com --- qemu-img-cmds.hx | 4 ++-- qemu-img.c | 25 ++--- qemu-img.texi| 2 +- 3

Re: [Qemu-devel] [PATCH] block: add watermark event

2014-08-05 Thread Stefan Hajnoczi
On Tue, Aug 05, 2014 at 10:47:57AM +0200, Kevin Wolf wrote: Am 01.08.2014 um 13:39 hat Stefan Hajnoczi geschrieben: On Tue, Jul 08, 2014 at 04:49:24PM +0200, Francesco Romani wrote: @@ -5813,3 +5815,57 @@ void bdrv_flush_io_queue(BlockDriverState *bs)

Re: [Qemu-devel] KVM call for agenda for 2014-08-05

2014-08-05 Thread Juan Quintela
Juan Quintela quint...@redhat.com wrote: Reset, this time with the right mailing lists. Thanks to Markus for noticing. Later, Juan. Juan Quintela quint...@redhat.com wrote: No agenda, no call. Sorry for the late advice :-( Later, Juan.

Re: [Qemu-devel] [PATCH v2 4/7] block/qcow2: Implement status CB for amend

2014-08-05 Thread Eric Blake
On 08/01/2014 05:49 PM, Max Reitz wrote: The only really time-consuming operation potentially performed by qcow2_amend_options() is zero cluster expansion when downgrading qcow2 images from compat=1.1 to compat=0.10, so report status of that operation and that operation only through the status

Re: [Qemu-devel] [PATCH v2 5/7] block/qcow2: Make get_refcount() global

2014-08-05 Thread Eric Blake
On 08/01/2014 05:49 PM, Max Reitz wrote: Reading the refcount of a cluster is an operation which can be useful in all of the qcow2 code, so make that function globally available. While touching this function, amend the comment describing the addend parameter: It is (no longer, if it ever

Re: [Qemu-devel] [PATCH] linux-user: /proc/self/maps content

2014-08-05 Thread Mikhail Ilin
Thanks, I've tested the sample for Aarch64 myself and found that the approach should also work fine. Translation layout: $ qemu-aarch64 -strace /tmp/busybox-static cat /proc/self/maps startend size prot 0040-005ba000 001ba000

Re: [Qemu-devel] Cc'ing emails [

2014-08-05 Thread Anthony Liguori
On Aug 5, 2014 2:42 AM, Markus Armbruster arm...@redhat.com wrote: Michael Tokarev m...@tls.msk.ru writes: 05.08.2014 08:41, Chen Gang wrote: Every members have their own tastes, and one working flow may be not suitable for all members. I can understand, and hope other members

  1   2   3   >