Re: [Qemu-devel] [PATCH v3 2/3] qdev: interface for SysBusDevice to change property on requirement

2013-09-02 Thread liu ping fan
On Fri, Aug 30, 2013 at 4:17 PM, Paolo Bonzini pbonz...@redhat.com wrote: Il 30/08/2013 09:53, Liu Ping Fan ha scritto: qdev's property can not be set after realized, but there is a requirement of adjusting device's behavior on different mother boards. So introducing a callback in

[Qemu-devel] [Bug 1024248] Re: qemu -M isapc displays blank screen

2013-09-02 Thread Michael Tokarev
It looks like this issue is fixed in 1.6.0 version ** Changed in: qemu Status: Confirmed = Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1024248 Title: qemu -M isapc displays

Re: [Qemu-devel] [PATCH v3 2/3] qdev: interface for SysBusDevice to change property on requirement

2013-09-02 Thread liu ping fan
On Fri, Aug 30, 2013 at 8:32 PM, Andreas Färber afaer...@suse.de wrote: Am 30.08.2013 10:17, schrieb Paolo Bonzini: Il 30/08/2013 09:53, Liu Ping Fan ha scritto: qdev's property can not be set after realized, but there is a requirement of adjusting device's behavior on different mother

Re: [Qemu-devel] [Qemu-trivial] [PATCH v3] slirp: Port redirection option behave differently on Linux and Windows

2013-09-02 Thread Jan Kiszka
On 2013-09-01 18:13, Stefan Weil wrote: Am 01.09.2013 17:46, schrieb Michael Tokarev: 30.08.2013 15:04, Jan Kiszka wrote: On 2013-08-15 21:25, Taimoor wrote: From: Taimoor Mirza tmi...@codesourcery.com port redirection code uses SO_REUSEADDR socket option before binding to host port.

[Qemu-devel] [Bug 1187121] Re: segfault with -vga vmware and -display gtk

2013-09-02 Thread Michael Tokarev
This has been fixed in 1.6.0. ** Changed in: qemu Status: Confirmed = Fix Released -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1187121 Title: segfault with -vga vmware and -display gtk

[Qemu-devel] [PATCH v4 0/3] bugs fix for hpet

2013-09-02 Thread Liu Ping Fan
note: I rebase it onto Stefan's net-next tree, since pc-1.7 has already been defined there. v4: use standard compat property to set hpet's interrupt compatibility v3: change hpet interrupt capablity on board's demand Liu Ping Fan (3): hpet: inverse polarity when pin above ISA_NUM_IRQS

[Qemu-devel] [PATCH v4 1/3] hpet: inverse polarity when pin above ISA_NUM_IRQS

2013-09-02 Thread Liu Ping Fan
According to hpet spec, hpet irq is high active. But according to ICH spec, there is inversion before the input of ioapic. So the OS will expect low active on this IRQ line.(And this is observed on bare metal). We fold the emulation of this inversion inside the hpet logic. Signed-off-by: Liu

[Qemu-devel] [PATCH v4 2/3] hpet: entitle more irq pins for hpet

2013-09-02 Thread Liu Ping Fan
On PC, IRQ2/8 can be reserved for hpet timer 0/1. And pin 16~23 of ioapic can be dynamically assigned to hpet as guest chooses. (Will enable them after introducing pc 1.6 compat) Signed-off-by: Liu Ping Fan pingf...@linux.vnet.ibm.com --- hw/timer/hpet.c | 13 +++-- 1 file changed, 11

[Qemu-devel] [PATCH v4 3/3] pc-1.6: add compatibility for hpet intcap on pc-*-1.6

2013-09-02 Thread Liu Ping Fan
For guest bug compat, we limit hpet's interrupt compatibility on ioapic's IRQ2 for pc-*-1.6. As to pc-*-1.7 and newer, IRQ2, IRQ8, and IRQ16~23 are allowed. Signed-off-by: Liu Ping Fan pingf...@linux.vnet.ibm.com --- hw/timer/hpet.c | 6 +- include/hw/i386/pc.h | 4 2 files

Re: [Qemu-devel] [PATCH v3 04/19] block: update bs-total_sectors on writes

2013-09-02 Thread Peter Lieven
On 17.08.2013 08:27, Paolo Bonzini wrote: Il 02/08/2013 09:05, Peter Lieven ha scritto: can you give an update what are to current plans/schedule to merge this series? I have a few patches in the queue that in their current version depend on this series being merged. It should go in soon,

[Qemu-devel] [PATCH v5 3/8] qcow2: Employ metadata overlap checks

2013-09-02 Thread Max Reitz
The pre-write overlap check function is now called before most of the qcow2 writes (aborting it on collision or other error). Signed-off-by: Max Reitz mre...@redhat.com --- block/qcow2-cache.c| 17 + block/qcow2-cluster.c | 21 + block/qcow2-snapshot.c |

[Qemu-devel] [PATCH v5 1/8] qcow2: Add corrupt bit

2013-09-02 Thread Max Reitz
This adds an incompatible bit indicating corruption to qcow2. Any image with this bit set may not be written to unless for repairing (and subsequently clearing the bit if the repair has been successful). Signed-off-by: Max Reitz mre...@redhat.com --- block/qcow2.c | 47

[Qemu-devel] [PATCH v5 0/8] Add metadata overlap checks

2013-09-02 Thread Max Reitz
If a qcow2 image file becomes corrupted, any write may inadvertently overwrite important metadata structures such as the L1 table. This series adds functionality for detecting, preventing and (to some extent) repairing such collisions. v5: - fixed patch 6 (forgot to update the event_names array

[Qemu-devel] [PATCH v5 2/8] qcow2: Metadata overlap checks

2013-09-02 Thread Max Reitz
Two new functions are added; the first one checks a given range in the image file for overlaps with metadata (main header, L1 tables, L2 tables, refcount table and blocks). The second one should be used immediately before writing to the image file as it calls the first function and, upon

[Qemu-devel] [PATCH v5 5/8] qcow2-refcount: Repair OFLAG_COPIED errors

2013-09-02 Thread Max Reitz
Since the OFLAG_COPIED checks are now executed after the refcounts have been repaired (if repairing), it is safe to assume that they are correct but the OFLAG_COPIED flag may be not. Therefore, if its value differs from what it should be (considering the according refcount), that discrepancy can

[Qemu-devel] [PATCH v5 4/8] qcow2-refcount: Move OFLAG_COPIED checks

2013-09-02 Thread Max Reitz
Move the OFLAG_COPIED checks out of check_refcounts_l1 and check_refcounts_l2 and after the actual refcount checks/fixes (since the refcounts might actually change there). Signed-off-by: Max Reitz mre...@redhat.com --- block/qcow2-refcount.c | 115

[Qemu-devel] [PATCH v5 8/8] qemu-iotests: Overlapping cluster allocations

2013-09-02 Thread Max Reitz
A new test on corrupted images with overlapping cluster allocations. Signed-off-by: Max Reitz mre...@redhat.com --- tests/qemu-iotests/060 | 111 + tests/qemu-iotests/060.out | 44 ++ tests/qemu-iotests/group | 1 + 3 files

[Qemu-devel] [PATCH v5 7/8] qcow2_check: Mark image consistent

2013-09-02 Thread Max Reitz
If no corruptions remain after an image repair (and no errors have been encountered), clear the corrupt flag in qcow2_check. Signed-off-by: Max Reitz mre...@redhat.com --- block/qcow2.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/block/qcow2.c b/block/qcow2.c index

[Qemu-devel] [PATCH v5 6/8] qcow2-refcount: Repair shared refcount blocks

2013-09-02 Thread Max Reitz
If the refcount of a refcount block is greater than one, we can at least try to repair that problem by duplicating the affected block. Signed-off-by: Max Reitz mre...@redhat.com --- block/blkdebug.c | 1 + block/qcow2-refcount.c | 148 -

Re: [Qemu-devel] [PATCH v3 2/3] qcow2: Implement bdrv_amend_options

2013-09-02 Thread Max Reitz
Am 02.09.2013 05:43, schrieb Fam Zheng: On Fri, 08/30 12:27, Max Reitz wrote: Implement bdrv_amend_options for compat, size, backing_file, backing_fmt and lazy_refcounts. Downgrading images from compat=1.1 to compat=0.10 is achieved through handling all incompatible flags accordingly, clearing

Re: [Qemu-devel] [PATCH v2 2/2] milkymist-uart: use Device::realize instead of SysBusDevice::init

2013-09-02 Thread Andreas Färber
Am 01.09.2013 20:28, schrieb Antony Pavlov: On Sat, 31 Aug 2013 21:09:20 +0200 Andreas Färber afaer...@suse.de wrote: Am 31.08.2013 19:22, schrieb Antony Pavlov: Use of SysBusDevice::init is deprecated. Use Device::realize instead of SysBusDevice::init. Check dma/pl330.c for an example of

Re: [Qemu-devel] [PATCH v3 2/3] qcow2: Implement bdrv_amend_options

2013-09-02 Thread Kevin Wolf
Am 02.09.2013 um 05:43 hat Fam Zheng geschrieben: On Fri, 08/30 12:27, Max Reitz wrote: Implement bdrv_amend_options for compat, size, backing_file, backing_fmt and lazy_refcounts. Downgrading images from compat=1.1 to compat=0.10 is achieved through handling all incompatible flags

Re: [Qemu-devel] [PATCH] target-ppc: Fix include for qemu_notify_event

2013-09-02 Thread Andreas Färber
Am 01.09.2013 18:00, schrieb Richard Henderson: Signed-off-by: Richard Henderson r...@twiddle.net --- target-ppc/kvm_ppc.c | 1 + 1 file changed, 1 insertion(+) The PowerPC target has been broken for more than a week. Déjà vu, Alex already sent a fix on ~Friday. Andreas r~ diff

[Qemu-devel] [PULL 1/6] pci: Introduce helper to retrieve a PCI device's DMA address space

2013-09-02 Thread Michael S. Tsirkin
From: Alexey Kardashevskiy a...@ozlabs.ru A PCI device's DMA address space (possibly an IOMMU) is returned by a method on the PCIBus. At the moment that only has one caller, so the method is simply open coded. We'll need another caller for VFIO, so this patch introduces a helper/wrapper

[Qemu-devel] [PULL 2/6] pc: fix regression for 64 bit PCI memory

2013-09-02 Thread Michael S. Tsirkin
commit 398489018183d613306ab022653552247d93919f pc: limit 64 bit hole to 2G by default introduced a way for management to control the window allocated to the 64 bit PCI hole. This is useful, but existing management tools do not know how to set this property. As a result, e.g. specifying a

[Qemu-devel] [PULL 5/6] pc: reduce duplication, fix PIIX descriptions

2013-09-02 Thread Michael S. Tsirkin
We have a lot of code duplication between machine types, this increases with each new machine type and each new field. This has already introduced a minor bug: description for pc-1.3 says Standard PC while description for pc-1.4 is Standard PC (i440FX + PIIX, 1996) which makes you think 1.3 is

[Qemu-devel] [PULL 0/6] pc,pci,virtio fixes and cleanups

2013-09-02 Thread Michael S. Tsirkin
The following changes since commit 1ae2757c6c4525c9b42f408c86818f843bad7418: virtio: virtqueue_get_avail_bytes: fix desc_pa when loop over the indirect descriptor table (2013-08-25 12:52:33 +0300) are available in the git repository at: git://git.kernel.org/pub/scm/virt/kvm/mst/qemu.git

[Qemu-devel] [PULL 4/6] hw: Clean up bogus default boot order

2013-09-02 Thread Michael S. Tsirkin
From: Markus Armbruster arm...@redhat.com We set default boot order cad in every single machine definition except pseries and moxiesim, even though very few boards actually care for boot order, and cad makes sense for even fewer. Machines that care: * pc and its variants Accept up to three

[Qemu-devel] [PULL 3/6] pci: add config space access traces

2013-09-02 Thread Michael S. Tsirkin
From: Alexey Kardashevskiy a...@ozlabs.ru This adds pci_cfg_read and pci_cfg_write traces for config spaces accesses. Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Signed-off-by: Michael S. Tsirkin m...@redhat.com --- hw/pci/pci_host.c | 11 ++- trace-events | 4 2 files

[Qemu-devel] [PULL 6/6] virtio_pci: fix level interrupts with irqfd

2013-09-02 Thread Michael S. Tsirkin
commit 62c96360ae7f2c7a8b029277fbb7cb082fdef7fd virtio-pci: fix level interrupts only helps systems without irqfd: on systems with irqfd support we passed in flag requesting irqfd even when msix is disabled. As a result, for level interrupts we didn't install an fd handler so unmasking an fd

Re: [Qemu-devel] [PATCH] target-ppc: Fix include for qemu_notify_event

2013-09-02 Thread Alexander Graf
On 02.09.2013, at 09:56, Andreas Färber wrote: Am 01.09.2013 18:00, schrieb Richard Henderson: Signed-off-by: Richard Henderson r...@twiddle.net --- target-ppc/kvm_ppc.c | 1 + 1 file changed, 1 insertion(+) The PowerPC target has been broken for more than a week. Déjà vu, Alex already

[Qemu-devel] [PULL 10/18] target-ppc: fix bit extraction for FPBF and FPL

2013-09-02 Thread Alexander Graf
From: Aurelien Jarno aurel...@aurel32.net Bit extraction for the FP BF and L field of the MTFSFI and MTFSF instructions is wrong and doesn't match the reference manual (which explain the bit number in big endian format). It has been broken in commit 7d08d85645def18eac2a9d672c1868a35e0bcf79. This

[Qemu-devel] [PULL 16/18] PPC: KVM: Compile fix for qemu_notify_event

2013-09-02 Thread Alexander Graf
The function qemu_notify_event is defined by a header that we don't include in the PPC KVM code. Include it to get the code building again. target-ppc/kvm_ppc.c: In function 'kvmppc_timer_hack': target-ppc/kvm_ppc.c:26:5: error: implicit declaration of function 'qemu_notify_event'

[Qemu-devel] [PULL 00/18] ppc patch queue 2013-09-02

2013-09-02 Thread Alexander Graf
Hi Blue / Aurelien / Anthony, This is my current patch queue for ppc. Please pull. Alex The following changes since commit 4ff78e0dbcd5c795962567fdc1b31e9e03c55b07: Merge remote-tracking branch 'luiz/queue/qmp' into staging (2013-08-30 12:26:04 -0500) are available in the git repository

[Qemu-devel] [PULL 05/18] disas/ppc.c: Fix little endian disassembly

2013-09-02 Thread Alexander Graf
From: Anton Blanchard an...@samba.org Use info-endian to select the endian of the instruction to be disassembled. Signed-off-by: Anton Blanchard an...@samba.org Reviewed-by: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Alexander Graf ag...@suse.de --- disas/ppc.c | 3 ++- 1 file changed,

[Qemu-devel] [PULL 08/18] ppc405_uc: Disable debug output

2013-09-02 Thread Alexander Graf
From: Andreas Färber afaer...@suse.de Signed-off-by: Andreas Färber afaer...@suse.de Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/ppc405_uc.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/hw/ppc/ppc405_uc.c b/hw/ppc/ppc405_uc.c index

[Qemu-devel] [PULL 06/18] ppc: virtex_ml507: QEMU_OPTION_dtb support for this machine.

2013-09-02 Thread Alexander Graf
From: Efimov Vasily r...@ispras.ru QEMU has 'dtb' option for specifing the device tree file for the kernel. The patch adds support for this option to the 'virtex_ml507' machine implementation. Signed-off-by: Efimov Vasily r...@ispras.ru Signed-off-by: Alexander Graf ag...@suse.de ---

[Qemu-devel] [PULL 07/18] ppc405_boards: Disable debug output

2013-09-02 Thread Alexander Graf
From: Andreas Färber afaer...@suse.de Also move one stray debug output into an #ifdef. Signed-off-by: Andreas Färber afaer...@suse.de Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/ppc405_boards.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

[Qemu-devel] [PULL 12/18] target-ppc: Use #define instead of opencoding SLB valid bit

2013-09-02 Thread Alexander Graf
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com Use SLB_ESID_V instead of (1 27) in the code Reviewed-by: Andreas Färber afaer...@suse.de Signed-off-by: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com Signed-off-by: Alexander Graf ag...@suse.de --- target-ppc/mmu_helper.c | 2 +- 1

[Qemu-devel] [PULL 01/18] PPC: E500: Generate device tree on reset

2013-09-02 Thread Alexander Graf
Today we generate the device tree once on machine initialization and then store the finalized blob in memory to reload it on reset. This is bad for 2 reasons. First we potentially waste a bunch of RAM for no good reason, as we have all information required to regenerate the device tree available

[Qemu-devel] [PULL 04/18] target-ppc: POWER7 supports the MSR_LE bit

2013-09-02 Thread Alexander Graf
From: Anton Blanchard an...@samba.org Add MSR_LE to the msr_mask for POWER7. Signed-off-by: Anton Blanchard an...@samba.org Reviewed-by: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Alexander Graf ag...@suse.de --- target-ppc/translate_init.c | 2 +- 1 file changed, 1 insertion(+), 1

[Qemu-devel] [PULL 02/18] pseries: Fix stalls on hypervisor virtual console

2013-09-02 Thread Alexander Graf
From: Anton Blanchard an...@samba.org A number of users are reporting stalls when using the pseries hypervisor virtual console. A simple test case is to paste 15 or 17 characters at a time into the console. Pasting 15 characters at a time works fine but pasting 17 characters hangs for a random

[Qemu-devel] [PULL 13/18] spapr-pci: rework MSI/MSIX

2013-09-02 Thread Alexander Graf
From: Alexey Kardashevskiy a...@ozlabs.ru On the sPAPR platform a guest allocates MSI/MSIX vectors via RTAS hypercalls which return global IRQ numbers to a guest so it only operates with those and never touches MSIMessage. Therefore MSIMessage handling is completely hidden in QEMU. Previously

[Qemu-devel] [PULL 15/18] pseries: Add H_SET_MODE hcall to change guest exception endianness

2013-09-02 Thread Alexander Graf
From: Anton Blanchard an...@samba.org H_SET_MODE is used for controlling various partition settings. One of these settings is the endianness a guest takes its exceptions in. Signed-off-by: Anton Blanchard an...@samba.org [agraf: fix whitespace] Signed-off-by: Alexander Graf ag...@suse.de ---

[Qemu-devel] [PULL 17/18] spapr: add stop-self RTAS call required to support hot CPU unplug

2013-09-02 Thread Alexander Graf
From: Alexey Kardashevskiy a...@ozlabs.ru PAPR+ requires two RTAS calls to be supported by the hypervisor in order to allow hotplugging VCPUs from the guest. The start-cpu RTAS call was already there but stop-self was not. This adds the stop-self RTAS call. Signed-off-by: Alexey Kardashevskiy

[Qemu-devel] [PULL 03/18] target-ppc: USE LPCR_ILE to control exception endian on POWER7

2013-09-02 Thread Alexander Graf
From: Anton Blanchard an...@samba.org On POWER7, LPCR_ILE is used to control what endian guests take their exceptions in so use it instead of MSR_ILE. Signed-off-by: Anton Blanchard an...@samba.org Reviewed-by: Anthony Liguori aligu...@us.ibm.com Signed-off-by: Alexander Graf ag...@suse.de ---

[Qemu-devel] [PULL 09/18] ppc405_boards: Don't enforce presence of firmware for qtest

2013-09-02 Thread Alexander Graf
From: Andreas Färber afaer...@suse.de Adopt error_report() while at it. Signed-off-by: Andreas Färber afaer...@suse.de Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/ppc405_boards.c | 35 --- 1 file changed, 20 insertions(+), 15 deletions(-) diff --git

[Qemu-devel] [PULL 14/18] xics: move registration of global state to realize()

2013-09-02 Thread Alexander Graf
From: Alexey Kardashevskiy a...@ozlabs.ru Registration of global state belongs into realize so move it there. Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru Reviewed-by: Andreas Färber afaer...@suse.de Signed-off-by: Alexander Graf ag...@suse.de --- hw/intc/xics.c | 21 +++--

[Qemu-devel] [PULL 11/18] spapr-pci: fix config space access to support bridges

2013-09-02 Thread Alexander Graf
From: Alexey Kardashevskiy a...@ozlabs.ru spapr-pci config space accessors use find_dev() to find a PCI device. However find_dev() only searched on a primary bus and did not do recursive search through secondary buses so config space access was not possible for devices other that on a primary

[Qemu-devel] [PULL 18/18] PPC: spapr: iommu: rework traces

2013-09-02 Thread Alexander Graf
From: Alexey Kardashevskiy a...@ozlabs.ru This converts old style fprintf to traces. Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru [agraf: change patch subject] Signed-off-by: Alexander Graf ag...@suse.de --- hw/ppc/spapr_iommu.c | 71 ++--

Re: [Qemu-devel] [PULL 00/26] Block patches

2013-09-02 Thread Kevin Wolf
Am 30.08.2013 um 19:14 hat Anthony Liguori geschrieben: This pull request breaks make check, qemu-system-i386 segvs when running qtest. bisect blames the following commit. I confirmed this commit introduces the breakage too. commit 19abade25242079f4b5582de17b2302fe185be2b Author: Max

Re: [Qemu-devel] [PATCH] target-i386: Only provide CMOV and friends if feature bit set

2013-09-02 Thread Peter Maydell
Ping^3! thanks -- PMM On 20 August 2013 13:59, Peter Maydell peter.mayd...@linaro.org wrote: Ping^2! This has been reviewed and I've checked that the patch still applies to master. thanks -- PMM On 25 July 2013 17:54, Peter Maydell peter.mayd...@linaro.org wrote: Ping! (patchwork url:

Re: [Qemu-devel] [PATCH v2] pseries: Update SLOF firmware image

2013-09-02 Thread Alexander Graf
On 28.08.2013, at 09:42, Alexey Kardashevskiy wrote: On 08/27/2013 06:33 PM, Alexander Graf wrote: On 27.08.2013, at 05:42, Alexey Kardashevskiy wrote: This has reworked USB OHCI and adds support of USB EHCI, VIRTIO-SCSI and various fixes (IBM VSCSI, VGA and more). The full list of

[Qemu-devel] [PULL v2 00/26] Block patches

2013-09-02 Thread Kevin Wolf
The following changes since commit b5d54bd42158b90b239bb6ce9c13072eb3a53fd2: Merge remote-tracking branch 'qemu-kvm/uq/master' into stable-1.5 (2013-08-29 17:21:51 -0500) are available in the git repository at: git://repo.or.cz/qemu/kevin.git for-anthony for you to fetch changes up to

[Qemu-devel] [PULL v2 01/26] qcow2: Change default for new images to compat=1.1

2013-09-02 Thread Kevin Wolf
By the time that qemu 1.7 will be released, enough time will have passed since qemu 1.1, which is the first version to understand version 3 images, that changing the default shouldn't hurt many people any more and the benefits of using the new format outweigh the pain. qemu-iotests already runs

[Qemu-devel] [PULL v2 02/26] block: Remove redundant assertion

2013-09-02 Thread Kevin Wolf
The failing condition is checked immediately before the assertion, so keeping the assertion is kind of redundant. Signed-off-by: Kevin Wolf kw...@redhat.com --- block.c | 1 - 1 file changed, 1 deletion(-) diff --git a/block.c b/block.c index a387c1a..26639e8 100644 --- a/block.c +++ b/block.c

[Qemu-devel] [PULL v2 11/26] raw_bsd: add raw_create_options

2013-09-02 Thread Kevin Wolf
From: Laszlo Ersek ler...@redhat.com On 08/05/13 15:03, Paolo Bonzini wrote: [...] 4) There is another member, .create_options, which is an array of QEMUOptionParameter structs, terminated by an all-zero item. The only option you need is for the virtual disk size. You will find something

[Qemu-devel] [PULL v2 07/26] add skeleton for BSD licensed raw BlockDriver

2013-09-02 Thread Kevin Wolf
From: Laszlo Ersek ler...@redhat.com On 08/05/13 15:03, Paolo Bonzini wrote: - Original Message - From: Laszlo Ersek ler...@redhat.com To: Paolo Bonzini pbonz...@redhat.com Sent: Monday, August 5, 2013 2:43:46 PM Subject: Re: [PATCH 1/2] raw: add license header On 08/02/13 00:27,

[Qemu-devel] [PULL v2 06/26] block/qcow2.h: Avoid 1LL 63 (shifts into sign bit)

2013-09-02 Thread Kevin Wolf
From: Peter Maydell peter.mayd...@linaro.org The expression 1LL 63 tries to shift the 1 into the sign bit of a 'long long', which provokes a clang sanitizer warning: runtime error: left shift of 1 by 63 places cannot be represented in type 'long long' Use 1ULL 63 as the definition of

[Qemu-devel] [PULL v2 03/26] qapi-types.py: Split off generate_struct_fields()

2013-09-02 Thread Kevin Wolf
Signed-off-by: Kevin Wolf kw...@redhat.com Reviewed-by: Eric Blake ebl...@redhat.com --- scripts/qapi-types.py | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) diff --git a/scripts/qapi-types.py b/scripts/qapi-types.py index 5ee46ea..86de980 100644 ---

[Qemu-devel] [PULL v2 13/26] switch raw block driver from raw.o to raw_bsd.o

2013-09-02 Thread Kevin Wolf
From: Laszlo Ersek ler...@redhat.com Incoming function prototypes and outgoing function calls must match reality. Implemented using the struct BlockDriver definition in include/block/block_int.h, and gcc errors warnings. v1-v2: On 08/20/13 09:51, Kevin Wolf wrote: Am 18.08.2013 um 16:29 hat

[Qemu-devel] [PULL v2 09/26] raw_bsd: add raw_create()

2013-09-02 Thread Kevin Wolf
From: Laszlo Ersek ler...@redhat.com On 08/05/13 15:03, Paolo Bonzini wrote: [...] 2) This is also a simple forwarder function: .bdrv_create but there is no BlockDriverState argument so the forwarded-to function does not have a bs-file argument either. The forwarded-to function is

[Qemu-devel] [PULL v2 05/26] qemu-iotests: Update reference output for 051

2013-09-02 Thread Kevin Wolf
Signed-off-by: Kevin Wolf kw...@redhat.com --- tests/qemu-iotests/051.out | 1 - 1 file changed, 1 deletion(-) diff --git a/tests/qemu-iotests/051.out b/tests/qemu-iotests/051.out index 5582ed3..86e989c 100644 --- a/tests/qemu-iotests/051.out +++ b/tests/qemu-iotests/051.out @@ -85,7 +85,6 @@

[Qemu-devel] [PULL v2 19/26] qcow2: Add corrupt bit

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com This adds an incompatible bit indicating corruption to qcow2. Any image with this bit set may not be written to unless for repairing (and subsequently clearing the bit if the repair has been successful). Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by:

[Qemu-devel] [PULL v2 04/26] Revert block: Disable driver-specific options for 1.6

2013-09-02 Thread Kevin Wolf
This reverts commit 8afaefb8919dc8746a57c450a758717c516c7b0a. Signed-off-by: Kevin Wolf kw...@redhat.com --- blockdev.c | 143 --- tests/qemu-iotests/group | 2 +- 2 files changed, 1 insertion(+), 144 deletions(-) diff --git

[Qemu-devel] [PULL v2 08/26] raw_bsd: emit debug events in bdrv_co_readv() and bdrv_co_writev()

2013-09-02 Thread Kevin Wolf
From: Laszlo Ersek ler...@redhat.com On 08/05/13 15:03, Paolo Bonzini wrote: [...] 1) BlockDriver is a struct in which these function members are interesting: .bdrv_reopen_prepare .bdrv_co_readv .bdrv_co_writev .bdrv_co_is_allocated .bdrv_co_write_zeroes

[Qemu-devel] [PULL v2 10/26] raw_bsd: introduce special members

2013-09-02 Thread Kevin Wolf
From: Laszlo Ersek ler...@redhat.com On 08/05/13 15:03, Paolo Bonzini wrote: [...] 3) These members are special .format_name is the string raw .bdrv_open raw_open should set bs-sg to bs-file-sg and return 0 .bdrv_closeraw_close should do nothing .bdrv_probe

[Qemu-devel] [PULL v2 24/26] qcow2-refcount: Repair shared refcount blocks

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com If the refcount of a refcount block is greater than one, we can at least try to repair that problem by duplicating the affected block. Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by: Kevin Wolf kw...@redhat.com --- block/blkdebug.c | 1 +

[Qemu-devel] [PULL v2 14/26] block: Remove old raw driver

2013-09-02 Thread Kevin Wolf
This is unused code now. Signed-off-by: Kevin Wolf kw...@redhat.com --- block/raw.c | 192 1 file changed, 192 deletions(-) delete mode 100644 block/raw.c diff --git a/block/raw.c b/block/raw.c deleted file mode 100644 index

[Qemu-devel] [PULL v2 12/26] raw_bsd: register bdrv_raw

2013-09-02 Thread Kevin Wolf
From: Laszlo Ersek ler...@redhat.com On 08/05/13 15:03, Paolo Bonzini wrote: [...] 5) Formats are registered with bdrv_register (takes a BlockDriver*). You also need to pass the caller of bdrv_register to block_init. Fill in the BlockDriver structure with the raw_*() functions that have

[Qemu-devel] [PULL v2 18/26] qemu-iotests: Snapshotting zero clusters

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com This test creates an image with unallocated zero clusters, then creates a snapshot. Afterwards, there should be neither any errors nor leaks. Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by: Kevin Wolf kw...@redhat.com --- tests/qemu-iotests/062 |

[Qemu-devel] [PULL v2 16/26] option: Add assigned flag to QEMUOptionParameter

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com Adds an assigned flag to QEMUOptionParameter which is cleared at the beginning of parse_option_parameters and set on (successful) set_option_parameter and set_option_parameter_int. Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by: Kevin Wolf

[Qemu-devel] [PULL v2 21/26] qcow2: Employ metadata overlap checks

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com The pre-write overlap check function is now called before most of the qcow2 writes (aborting it on collision or other error). Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by: Kevin Wolf kw...@redhat.com --- block/qcow2-cache.c| 17 +

[Qemu-devel] [PULL v2 22/26] qcow2-refcount: Move OFLAG_COPIED checks

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com Move the OFLAG_COPIED checks out of check_refcounts_l1 and check_refcounts_l2 and after the actual refcount checks/fixes (since the refcounts might actually change there). Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by: Kevin Wolf kw...@redhat.com ---

[Qemu-devel] [PULL v2 26/26] qemu-iotests: Overlapping cluster allocations

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com A new test on corrupted images with overlapping cluster allocations. Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by: Kevin Wolf kw...@redhat.com --- tests/qemu-iotests/060 | 111 +

[Qemu-devel] [PULL v2 23/26] qcow2-refcount: Repair OFLAG_COPIED errors

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com Since the OFLAG_COPIED checks are now executed after the refcounts have been repaired (if repairing), it is safe to assume that they are correct but the OFLAG_COPIED flag may be not. Therefore, if its value differs from what it should be (considering the

[Qemu-devel] [PULL v2 15/26] gluster: Abort on AIO completion failure

2013-09-02 Thread Kevin Wolf
From: Bharata B Rao bhar...@linux.vnet.ibm.com Currently if gluster AIO callback thread fails to notify the QEMU thread about AIO completion, we try graceful recovery by marking the disk drive as inaccessible. This error recovery code is race-prone as found by Asias and Stefan. However as found

[Qemu-devel] [PULL v2 17/26] qcow2-refcount: Snapshot update for zero clusters

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com Account for all cluster types in qcow2_update_snapshot_refcounts; this prevents this function from updating the refcount of unallocated zero clusters which effectively led to wrong adjustments of the refcount of cluster 0 (the main qcow2 header). This in turn

[Qemu-devel] [PULL v2 20/26] qcow2: Metadata overlap checks

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com Two new functions are added; the first one checks a given range in the image file for overlaps with metadata (main header, L1 tables, L2 tables, refcount table and blocks). The second one should be used immediately before writing to the image file as it calls

[Qemu-devel] [PATCH 1/4] monitor: introduce monitor_read_console

2013-09-02 Thread Lei Li
This patch introduces monitor_read_console(), which will drop into interactive mode with chardev ringbuf backend, and install a readline handler. When the handler is invoked, the given data will be written to ringbuf backend. Signed-off-by: Lei Li li...@linux.vnet.ibm.com ---

[Qemu-devel] [PATCH 0/4 RFC] Introduce console for ringbuf backend

2013-09-02 Thread Lei Li
This patch series aims at adding HMP console feature for ringbuf backend. It behaves like a serial console, which drops user into an interactive mode with ringbuf backend and takes user back to the monitor by the 'ctrl-]' escape sequence. It would suspend the monitor, output the data that

[Qemu-devel] [PULL v2 25/26] qcow2_check: Mark image consistent

2013-09-02 Thread Kevin Wolf
From: Max Reitz mre...@redhat.com If no corruptions remain after an image repair (and no errors have been encountered), clear the corrupt flag in qcow2_check. Signed-off-by: Max Reitz mre...@redhat.com Signed-off-by: Kevin Wolf kw...@redhat.com --- block/qcow2.c | 6 +- 1 file changed, 5

[Qemu-devel] [PATCH 3/4] qemu-char: export ringbuf_count

2013-09-02 Thread Lei Li
Signed-off-by: Lei Li li...@linux.vnet.ibm.com --- include/sysemu/char.h |2 ++ qemu-char.c |2 +- 2 files changed, 3 insertions(+), 1 deletions(-) diff --git a/include/sysemu/char.h b/include/sysemu/char.h index 8053130..ed20fe3 100644 --- a/include/sysemu/char.h +++

[Qemu-devel] [PATCH 2/4] hmp: factor out ringbuf_print_help()

2013-09-02 Thread Lei Li
Factor out ringbuf_print_help(), which will be called in hmp_read_ringbuf_cb() reading data that can be written with monitor_printf() to the console from ringbuf backend. Signed-off-by: Lei Li li...@linux.vnet.ibm.com --- hmp.c | 31 +++ 1 files changed, 19

[Qemu-devel] [PATCH 4/4] hmp: add console support for ringbuf backend

2013-09-02 Thread Lei Li
This patch add console command which would suspend the monitor, output the data that backed in the ringbuf backend to console first, and install a new readline handler to get input back to the ringbuf backend. Take back to the monitor once escape sequence 'Ctrl-]' is detected. Signed-off-by: Lei

Re: [Qemu-devel] [PATCH] seccomp: adding a second whitelist

2013-09-02 Thread Stefan Hajnoczi
On Fri, Aug 30, 2013 at 11:42:34AM -0400, Paul Moore wrote: On Friday, August 30, 2013 05:23:45 PM Stefan Hajnoczi wrote: On Fri, Aug 30, 2013 at 4:21 PM, Eduardo Otubo ot...@linux.vnet.ibm.com wrote: On 08/29/2013 05:34 AM, Stefan Hajnoczi wrote: On Wed, Aug 28, 2013 at 10:04:32PM

Re: [Qemu-devel] About coroutine inside qemu

2013-09-02 Thread Stefan Hajnoczi
On Fri, Aug 30, 2013 at 08:59:37PM +, Yaodong Yang wrote: When a new work thread, which is not inside coroutine,  enter a coroutine(named as A), what happened to the work thread? Is it true that the work thread also become a coroutine, or it just stop? Later if the coroutine A call yield.

Re: [Qemu-devel] Capture SIGSEGV to track pc.ram page access

2013-09-02 Thread Stefan Hajnoczi
On Sun, Sep 01, 2013 at 11:38:34AM +0200, Thomas Knauth wrote: My question is where do I have to touch qemu to call my code for handling SIGSEGVs? Is this possible at all? Can anyone suggest alternative ways of tracking which pages of pc.ram are accessed? Maybe you can use the dirty page

[Qemu-devel] [PATCH V3 0/3] qemu-iotests: add test for fd passing via SCM rights

2013-09-02 Thread Wenchao Xia
This series add test case for fd passing with unix socket at runtime. Since getfd and closefd interface will interact with monitor's data, so it will help to do regression test for monitor patches. Since python2 do not support sendmsg(), so a C helper program is added to do the job. v2: 1: add

[Qemu-devel] [PATCH V3 1/3] qemu-iotests: add unix socket help program

2013-09-02 Thread Wenchao Xia
This program can do a sendmsg call to transfer fd with unix socket, which is not supported in python2. The built binary will not be deleted in clean, but it is a existing issue in ./tests, which should be solved in another patch. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com ---

[Qemu-devel] [PATCH V3 2/3] qemu-iotests: add infrastructure of fd passing via SCM

2013-09-02 Thread Wenchao Xia
This patch make use of the compiled scm helper program to transfer fd via unix socket at runtime. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- QMP/qmp.py|6 ++ tests/qemu-iotests/check |1 + tests/qemu-iotests/iotests.py | 23

[Qemu-devel] [PATCH V3 3/3] qemu-iotests: add tests for runtime fd passing via SCM rights

2013-09-02 Thread Wenchao Xia
This case will test whether the monitor can receive fd at runtime. To verify better, additional monitor is created to see if qemu can handler two monitor instances correctly. Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com --- tests/qemu-iotests/045 | 36

Re: [Qemu-devel] [PATCH 0/6][RFC] AArch64 support for Versatile Express using KVM

2013-09-02 Thread Andreas Färber
Alex, Am 28.06.2013 15:04, schrieb Andreas Färber: Am 28.06.2013 14:44, schrieb Peter Maydell: On 28 June 2013 13:20, Mian M. Hamayun m.hama...@virtualopensystems.com wrote: This patch series implements KVM support in QEMU for the ARMv8 Cortex A57 CPU. It depends on the previously

[Qemu-devel] [PATCH] spice-core: Use g_strdup_printf instead of snprintf

2013-09-02 Thread Christophe Fergeau
Several places in spice-core.c were using either g_malloc+snprintf or snprintf+g_strdup to achieve the same result as g_strdup_printf. Signed-off-by: Christophe Fergeau cferg...@redhat.com --- ui/spice-core.c | 22 -- 1 file changed, 8 insertions(+), 14 deletions(-) diff

Re: [Qemu-devel] [PATCH 0/6][RFC] AArch64 support for Versatile Express using KVM

2013-09-02 Thread Peter Maydell
On 2 September 2013 10:46, Andreas Färber afaer...@suse.de wrote: Am 28.06.2013 15:04, schrieb Andreas Färber: Is there a 32-bit aarch64 register in ARMv8 to be accessed by TCG? If not, the field should be in ARMCPU and a bool rather than uint32_t. Care to comment on the latter? I don't

[Qemu-devel] [PATCH v4 1/5] block: Image file option amendment

2013-09-02 Thread Max Reitz
This patch adds the amend option to qemu-img which allows changing image options on existing image files. It also adds the generic bdrv implementation which is basically just a wrapper for the image format specific function. Signed-off-by: Max Reitz mre...@redhat.com --- block.c

[Qemu-devel] [PATCH v4 2/5] qcow2-cluster: Expand zero clusters

2013-09-02 Thread Max Reitz
Add functionality for expanding zero clusters. This is necessary for downgrading the image version to one without zero cluster support. For non-backed images, this function may also just discard zero clusters instead of truly expanding them. Signed-off-by: Max Reitz mre...@redhat.com ---

[Qemu-devel] [PATCH v4 0/5] block/qcow2: Image file option amendment

2013-09-02 Thread Max Reitz
This series adds support to qemu-img, block and qcow2 for amending image options on existing image files. Depends on: - option: Add assigned flag to QEMUOptionParameter - qcow2-refcount: Snapshot update for zero clusters (series, v3) - Add metadata overlap checks (series, v5) v4: - rebased

[Qemu-devel] [PATCH v4 3/5] qcow2: Save refcount order in BDRVQcowState

2013-09-02 Thread Max Reitz
Save the image refcount order in BDRVQcowState. This will be relevant for future code supporting different refcount orders than four and also for code that needs to verify a certain refcount order for an opened image. Signed-off-by: Max Reitz mre...@redhat.com --- block/qcow2.c | 3 ++-

[Qemu-devel] [PATCH v4 5/5] qemu-iotest: qcow2 image option amendment

2013-09-02 Thread Max Reitz
Add tests for qemu-img amend on qcow2 image files. Signed-off-by: Max Reitz mre...@redhat.com --- tests/qemu-iotests/061 | 178 +++ tests/qemu-iotests/061.out | 349 + tests/qemu-iotests/group | 1 + 3 files changed, 528

  1   2   3   4   >