Re: [Qemu-devel] [PATCH 0/8] ivshmem: test msi=off, remove CharDriver

2016-01-29 Thread Markus Armbruster
Marc-André Lureau writes: > Hi > > On Mon, Dec 21, 2015 at 12:30 PM, wrote: >> From: Marc-André Lureau >> >> This is a ivshmem series with various bits: >> - add a test for a recently introduced regression

[Qemu-devel] [PATCH v9 10/16] block: Make bdrv_close() static

2016-01-29 Thread Max Reitz
There are no users of bdrv_close() left, except for one of bdrv_open()'s failure paths, bdrv_close_all() and bdrv_delete(), and that is good. Make bdrv_close() static so nobody makes the mistake of directly using bdrv_close() again. Signed-off-by: Max Reitz Reviewed-by: Eric

[Qemu-devel] [PATCH v9 11/16] block: Add list of all BlockDriverStates

2016-01-29 Thread Max Reitz
We need this list so that bdrv_close_all() can keep track of which BDSs are still open after having removed the BDSs from all of the BBs and having released all monitor BDS references. Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf Reviewed-by: Fam Zheng

Re: [Qemu-devel] [PATCH] arm: virt-acpi: each MADT.GICC entry as enabled unconditionally

2016-01-29 Thread Shannon Zhao
On 2016/1/29 23:26, Andrew Jones wrote: On Fri, Jan 29, 2016 at 10:59:32PM +0800, Shannon Zhao wrote: > > >On 2016/1/29 22:24, Igor Mammedov wrote: > >in current impl. condition > > > >build_madt() { > > ... > > if (test_bit(i, cpuinfo->found_cpus)) > > > >is always true since loop

[Qemu-devel] [PATCH v3] linux-user: Fix qemu-binfmt-conf.h to store config across reboot

2016-01-29 Thread Laurent Vivier
Original qemu-binfmt-conf.h is only able to write configuration into /proc/sys/fs/binfmt_misc, and the configuration is lost on reboot. This script can configure debian and systemd services to restore configuration on reboot. Moreover, it is able to manage binfmt credential and to configure the

Re: [Qemu-devel] [Qemu-arm] [PATCH 6/8] target-arm: Handle exception return from AArch64 to non-EL0 AArch32

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > Remove the assumptions that the AArch64 exception return code was > making about a return to AArch32 always being a return to EL0. > This includes pulling out the illegal-SPSR checks so we can apply > them for return to 32 bit as well as return to

Re: [Qemu-devel] [PATCH v7 01/13] machine: Don't allow CPU toplogies with partially filled cores

2016-01-29 Thread Igor Mammedov
On Fri, 29 Jan 2016 13:36:05 -0200 Eduardo Habkost wrote: > On Fri, Jan 29, 2016 at 04:10:47PM +0100, Igor Mammedov wrote: > > On Fri, 29 Jan 2016 12:24:18 -0200 > > Eduardo Habkost wrote: > > > > > On Fri, Jan 29, 2016 at 02:52:30PM +1100, David

Re: [Qemu-devel] [PATCH] arm: virt-acpi: each MADT.GICC entry as enabled unconditionally

2016-01-29 Thread Andrew Jones
On Fri, Jan 29, 2016 at 10:59:32PM +0800, Shannon Zhao wrote: > > > On 2016/1/29 22:24, Igor Mammedov wrote: > >in current impl. condition > > > >build_madt() { > > ... > > if (test_bit(i, cpuinfo->found_cpus)) > > > >is always true since loop handles only present CPUs > >in range

Re: [Qemu-devel] [PATCH v1 2/5] configure: ensure ldflags propagated to config_host

2016-01-29 Thread Alex Bennée
Paolo Bonzini writes: > On 28/01/2016 11:15, Alex Bennée wrote: >> diff --git a/configure b/configure >> index bd29ba7..148b79a 100755 >> --- a/configure >> +++ b/configure >> @@ -5871,7 +5871,7 @@ if test "$target_linux_user" = "yes" -o >> "$target_bsd_user" = "yes" ;

Re: [Qemu-devel] [PATCH v5] qom, qmp, hmp, qapi: create qom-type-prop-list for class properties

2016-01-29 Thread Eduardo Habkost
On Fri, Jan 29, 2016 at 01:03:38PM +0300, Valentin Rakush wrote: > Hi Eduardo, hi Daniel, > > I checked most of the classes that are used for x86_64 qemu simulation with > this command line: > x86_64-softmmu/qemu-system-x86_64 -qmp tcp:localhost:,server,nowait > -machine pc -cpu core2duo > >

[Qemu-devel] [PATCH v9 07/16] block: Remove BDS close notifier

2016-01-29 Thread Max Reitz
It is unused now, so we can remove it. Signed-off-by: Max Reitz Reviewed-by: Fam Zheng Reviewed-by: Kevin Wolf --- block.c| 8 block/block-backend.c | 7 --- include/block/block.h | 1

[Qemu-devel] [PATCH v9 02/16] iotests: Add test for eject under NBD server

2016-01-29 Thread Max Reitz
This patch adds a test for ejecting the BlockBackend an NBD server is connected to (the NBD server is supposed to stop). Signed-off-by: Max Reitz Reviewed-by: Fam Zheng Reviewed-by: Kevin Wolf --- tests/qemu-iotests/140 | 92

[Qemu-devel] [PATCH v9 03/16] block: Add BB-BDS remove/insert notifiers

2016-01-29 Thread Max Reitz
bdrv_close() no longer signifies ejection of a medium, this is now done by removing the BDS from the BB. Therefore, we want to have a notifier for that in the BB instead of a close notifier in the BDS. The former is added now, the latter is removed later. Symmetrically, another notifier list is

[Qemu-devel] [PATCH v9 13/16] block: Add blk_remove_all_bs()

2016-01-29 Thread Max Reitz
When bdrv_close_all() is called, instead of force-closing all root BlockDriverStates, it is better to just drop the reference from all BlockBackends and let them be closed automatically. This prevents BDS from getting closed that are still referenced by other BDS, which may result in loss of

Re: [Qemu-devel] [PATCH 6/8] ivshmem: generalize ivshmem_setup_interrupts

2016-01-29 Thread Markus Armbruster
marcandre.lur...@redhat.com writes: > From: Marc-André Lureau > > Call ivshmem_setup_interrupts() with or without MSI, always allocate > msi_vectors that is going to be used in all case in the following patch. > > Signed-off-by: Marc-André Lureau

Re: [Qemu-devel] [PATCH 7/8] ivshmem: use a single eventfd callback, get rid of CharDriver

2016-01-29 Thread Markus Armbruster
marcandre.lur...@redhat.com writes: > From: Marc-André Lureau > > Simplify the interrupt handling by having a single callback on irq > cases. Remove usage of CharDriver, replace it with > qemu_set_fd_handler(). Use event_notifier_test_and_clear() to read the >

Re: [Qemu-devel] [PATCH] arm: virt-acpi: each MADT.GICC entry as enabled unconditionally

2016-01-29 Thread Igor Mammedov
On Fri, 29 Jan 2016 22:59:32 +0800 Shannon Zhao wrote: > On 2016/1/29 22:24, Igor Mammedov wrote: > > in current impl. condition > > > > build_madt() { > >... > >if (test_bit(i, cpuinfo->found_cpus)) > > > > is always true since loop handles only present CPUs > >

Re: [Qemu-devel] [PATCH v4] Add optionrom compatible with fw_cfg DMA version

2016-01-29 Thread Stefan Hajnoczi
On Fri, Jan 29, 2016 at 12:12:24PM +0100, Marc Marí wrote: > This optionrom is based on linuxboot.S. > > Added changes proposed by Gerd Hoffman, Stefan Hajnoczi and Kevin O'Connor. > > All optionroms are now compiled in 32 bits. This also forces to not use any > standard C header because this

[Qemu-devel] [PATCH v9 09/16] blockdev: Use blk_remove_bs() in do_drive_del()

2016-01-29 Thread Max Reitz
Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf Reviewed-by: Fam Zheng --- blockdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/blockdev.c b/blockdev.c index 1044a6a..09d4621 100644 --- a/blockdev.c +++

[Qemu-devel] [PATCH v9 05/16] virtio-scsi: Catch BDS-BB removal/insertion

2016-01-29 Thread Max Reitz
Make use of the BDS-BB removal and insertion notifiers to remove or set up, respectively, virtio-scsi's op blockers. Signed-off-by: Max Reitz --- hw/scsi/virtio-scsi.c | 55 + include/hw/virtio/virtio-scsi.h | 10 2

[Qemu-devel] [PATCH v9 16/16] iotests: Add test for block jobs and BDS ejection

2016-01-29 Thread Max Reitz
Suggested-by: Paolo Bonzini Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf --- tests/qemu-iotests/141 | 186 + tests/qemu-iotests/141.out | 59 ++ tests/qemu-iotests/group

[Qemu-devel] [PATCH v9 01/16] block: Release named dirty bitmaps in bdrv_close()

2016-01-29 Thread Max Reitz
bdrv_delete() is not very happy about deleting BlockDriverStates with dirty bitmaps still attached to them. In the past, we got around that very easily by relying on bdrv_close_all() bypassing bdrv_delete(), and bdrv_close() simply ignoring that condition. We should fix that by releasing all named

Re: [Qemu-devel] [PATCH v7 12/13] qmp: Add query-ppc-cpu-cores command

2016-01-29 Thread Igor Mammedov
On Thu, 28 Jan 2016 11:19:54 +0530 Bharata B Rao wrote: > Show the details of PPC CPU cores via a new QMP command. > > TODO: update qmp-commands.hx with example > > Signed-off-by: Bharata B Rao > --- > hw/ppc/cpu-core.c |

Re: [Qemu-devel] [PATCH v1 3/5] include/qemu/atomic.h: default to __atomic functions

2016-01-29 Thread Alex Bennée
Paolo Bonzini writes: > On 28/01/2016 11:15, Alex Bennée wrote: >> +/* atomic_mb_read/set semantics map Java volatile variables. They are >> + * less expensive on some platforms (notably POWER & ARM) than fully >> + * sequentially consistent operations. >> + * >> + * As

[Qemu-devel] [PATCH] coroutine: drop GThread coroutine backend

2016-01-29 Thread Stefan Hajnoczi
The GThread coroutine backend was a portable coroutine implementation. Over the years all platforms got their own optimized coroutine implementations and nothing uses the GThread backend anymore. In fact, ./configure mentions the GThread backend doesn't work but might be useful for debugging.

Re: [Qemu-devel] [PATCH] coroutine: drop GThread coroutine backend

2016-01-29 Thread Alex Bennée
Stefan Hajnoczi writes: > The GThread coroutine backend was a portable coroutine implementation. > Over the years all platforms got their own optimized coroutine > implementations and nothing uses the GThread backend anymore. > > In fact, ./configure mentions the GThread

Re: [Qemu-devel] [Qemu-arm] [PATCH 8/8] target-arm: ignore ELR_ELx[1] for exception return to 32-bit ARM mode

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > The architecture requires that for an exception return to AArch32 the > low bits of ELR_ELx are ignored when the PC is set from them: > * if returning to Thumb mode, ignore ELR_ELx[0] > * if returning to ARM mode, ignore ELR_ELx[1:0] > > We were only

[Qemu-devel] [PATCH v9 08/16] block: Use blk_remove_bs() in blk_delete()

2016-01-29 Thread Max Reitz
Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf Reviewed-by: Fam Zheng --- block/block-backend.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/block/block-backend.c b/block/block-backend.c index

[Qemu-devel] [PATCH v9 04/16] virtio-blk: Functions for op blocker management

2016-01-29 Thread Max Reitz
Put the code for setting up and removing op blockers into an own function, respectively. Then, we can invoke those functions whenever a BDS is removed from an virtio-blk BB or inserted into it. Signed-off-by: Max Reitz Reviewed-by: Fam Zheng Reviewed-by:

Re: [Qemu-devel] [PATCH v7 01/13] machine: Don't allow CPU toplogies with partially filled cores

2016-01-29 Thread Eduardo Habkost
On Fri, Jan 29, 2016 at 04:10:47PM +0100, Igor Mammedov wrote: > On Fri, 29 Jan 2016 12:24:18 -0200 > Eduardo Habkost wrote: > > > On Fri, Jan 29, 2016 at 02:52:30PM +1100, David Gibson wrote: > > > On Thu, Jan 28, 2016 at 11:19:43AM +0530, Bharata B Rao wrote: > > > >

Re: [Qemu-devel] [PATCH] arm: virt-acpi: each MADT.GICC entry as enabled unconditionally

2016-01-29 Thread Andrew Jones
On Fri, Jan 29, 2016 at 11:44:24PM +0800, Shannon Zhao wrote: > > > On 2016/1/29 23:26, Andrew Jones wrote: > >On Fri, Jan 29, 2016 at 10:59:32PM +0800, Shannon Zhao wrote: > >>> > >>> > >>>On 2016/1/29 22:24, Igor Mammedov wrote: > >in current impl. condition > > > >build_madt()

Re: [Qemu-devel] [PATCH 00/37] clean include files to use osdep.h

2016-01-29 Thread Peter Maydell
On 27 January 2016 at 08:03, Paolo Bonzini wrote: > > > On 26/01/2016 19:16, Peter Maydell wrote: >> I propose to apply this directly to master in the not too >> distant future, since the consensus appears to be that that's >> overall less painful than trying to split it all

[Qemu-devel] [PATCH] ui/cocoa.m: Include qemu/osdep.h

2016-01-29 Thread Peter Maydell
Include "qemu/osdep.h". (This is a manual commit equivalent to what the clean-includes script would do, because that script can't handle ObjectiveC source files.) Signed-off-by: Peter Maydell --- ui/cocoa.m | 2 ++ 1 file changed, 2 insertions(+) diff --git

[Qemu-devel] [PATCH v5 0/2] trace: Show vCPU info in guest code events

2016-01-29 Thread Lluís Vilanova
NOTE: This series should complete the framework for guest code tracing. From here on, other series can concurrently add actual events and improve the guest code tracing features and performance (e.g., control tracing independently on each vCPU). This series introduces the "vcpu"

[Qemu-devel] [PATCH v5 2/2] trace: Add 'vcpu' event property to trace guest vCPU

2016-01-29 Thread Lluís Vilanova
This property identifies events that trace vCPU-specific information. It adds a "CPUState*" argument to events with the property, identifying the vCPU raising the event. TCG translation events also have a "TCGv_cpu" implicit argument that is later used as the "CPUState*" argument at execution

Re: [Qemu-devel] [PATCH 2/8] target-arm: Move aarch64_cpu_do_interrupt() to helper.c

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > Move the aarch64_cpu_do_interrupt() function to helper.c. We want > to be able to call this from code that isn't AArch64-only, and > the move allows us to avoid awkward #ifdeffery at the callsite. Reviewed-by: Sergey Fedorov >

Re: [Qemu-devel] [Qemu-arm] [PATCH 3/8] target-arm: Use a single entry point for AArch64 and AArch32 exceptions

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > If EL2 or EL3 is present on an AArch64 CPU, then exceptions can be > taken to an exception level which is running AArch32 (if only EL0 > and EL1 are present then EL1 must be AArch64 and all exceptions are > taken to AArch64). To support this we need to

Re: [Qemu-devel] [PATCH 4/8] target-arm: Pull semihosting handling out to arm_cpu_do_interrupt()

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > Handling of semihosting calls should depend on the register width > of the calling code, not on that of any higher exception level, > so we need to identify and handle semihosting calls before we > decide whether to deliver the exception as an entry to

[Qemu-devel] [RFC v2 3/8] Generic PCIe host bridge INTx determination for INTx routing

2016-01-29 Thread Eric Auger
This patch stores information about assigned legacy interrupt numbers in GPEX host structure. This is used during GPEX INTx number determination from a pin during INTx routing. Signed-off-by: Pranavkumar Sawargaonkar Signed-off-by: Tushar Jagad --- hw/arm/virt.c

[Qemu-devel] [RFC v2 8/8] hw: vfio: common: adapt vfio_listeners for reserved_iova region

2016-01-29 Thread Eric Auger
In case of reserved iova region, let's declare this region to the kernel so that it can use it for IOVA/PA bindings. Signed-off-by: Eric Auger --- hw/vfio/common.c | 46 -- 1 file changed, 28 insertions(+), 18 deletions(-) diff

Re: [Qemu-devel] [PATCH RFC V5 0/9] Implement GIC-500 from GICv3 family for arm64

2016-01-29 Thread Christopher Covington
On 10/20/2015 01:22 PM, Shlomo Pongratz wrote: > From: Shlomo Pongratz > > This patch is a first step multicores support for arm64. > > This implemntation was tested up to 100 cores. > > Things left to do: > > Support SPI, ITS and ITS CONTROL, note that this patch

Re: [Qemu-devel] [PATCH 1/1] arm: virt: change GPIO trigger interrupt to pulse

2016-01-29 Thread Peter Maydell
On 29 January 2016 at 15:13, Wei Huang wrote: > > > On 01/29/2016 08:50 AM, Peter Maydell wrote: >> I wonder if we should be asserting the GPIO pin in the powerdown-request >> hook and then deasserting it on system reset somewhere... > > This is another possibility. We can try to

[Qemu-devel] [PATCH v9 00/16] block: Rework bdrv_close_all()

2016-01-29 Thread Max Reitz
Currently, bdrv_close_all() force-closes all BDSs with a BlockBackend, which can lead to data corruption (see the iotest added in the final patch of this series) and is most certainly very ugly. This series reworks bdrv_close_all() to instead eject the BDS trees from all BlockBackends and then

[Qemu-devel] [PATCH v9 06/16] nbd: Switch from close to eject notifier

2016-01-29 Thread Max Reitz
The NBD code uses the BDS close notifier to determine when a medium is ejected. However, now it should use the BB's BDS removal notifier for that instead of the BDS's close notifier. Signed-off-by: Max Reitz Reviewed-by: Fam Zheng Reviewed-by: Kevin Wolf

[Qemu-devel] [PATCH v9 12/16] blockdev: Keep track of monitor-owned BDS

2016-01-29 Thread Max Reitz
As a side effect, we can now make x-blockdev-del's check whether a BDS is actually owned by the monitor explicit. Signed-off-by: Max Reitz --- blockdev.c | 28 +++- include/block/block_int.h | 4

Re: [Qemu-devel] [PATCH v14 7/8] Implement new driver for block replication

2016-01-29 Thread Stefan Hajnoczi
On Fri, Jan 29, 2016 at 11:13:42AM +0800, Changlong Xie wrote: > On 01/28/2016 11:15 PM, Stefan Hajnoczi wrote: > >On Thu, Jan 28, 2016 at 09:13:24AM +0800, Wen Congyang wrote: > >>On 01/27/2016 10:46 PM, Stefan Hajnoczi wrote: > >>>On Wed, Jan 13, 2016 at 05:18:31PM +0800, Changlong Xie wrote: >

Re: [Qemu-devel] [PATCH 4/8] libqos: remove some leaks

2016-01-29 Thread Markus Armbruster
marcandre.lur...@redhat.com writes: > From: Marc-André Lureau > > qpci_device_find() returns allocated data, don't leak it. > > Signed-off-by: Marc-André Lureau > --- > tests/libqos/pci.c | 2 ++ > 1 file changed, 2 insertions(+) > >

[Qemu-devel] [PATCH v5 1/2] trace: Add support for vCPU pointers in trace events

2016-01-29 Thread Lluís Vilanova
The tracing infrastructure later needs to differentiate between regular pointers and pointers to vCPUs. Also changes all targets to use the new 'TCGv_cpu' type instead of the generic 'TCGv_ptr'. As of now, the change is merely cosmetic ('TCGv_cpu' translates into 'TCGv_ptr'), but that could

Re: [Qemu-devel] [PATCH 1/8] target-arm: Properly support EL2 and EL3 in arm_el_is_aa64()

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > Support EL2 and EL3 in arm_el_is_aa64() by implementing the > logic for checking the SCR_EL3 and HCR_EL2 register-width bits > as appropriate to determine the register width of lower exception > levels. Reviewed-by: Sergey Fedorov

Re: [Qemu-devel] [PATCH 5/8] target-arm: Fix wrong AArch64 entry offset for EL2/EL3 target

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > The entry offset when taking an exception to AArch64 from a lower > exception level may be 0x400 or 0x600. 0x400 is used if the > implemented exception level immediately lower than the target level > is using AArch64, and 0x600 if it is using AArch32. We

Re: [Qemu-devel] [PATCH 7/8] target-arm: Implement remaining illegal return event checks

2016-01-29 Thread Sergey Fedorov
On 14.01.2016 21:34, Peter Maydell wrote: > We already implement almost all the checks for the illegal > return events from AArch64 state described in the ARM ARM section > D1.11.2. Add the two missing ones: > * return to EL2 when EL3 is implemented and SCR_EL3.NS is 0 > * return to Non-secure

[Qemu-devel] [RFC v2 1/8] linux-headers: partial update for VFIO reserved IOVA registration

2016-01-29 Thread Eric Auger
This is a partial update aiming at enhancing the VFIO user API according to not yet upstreamed kernel developments available at: https://git.linaro.org/people/eric.auger/linux.git/shortlog/refs/heads/v4.5-rc1-pcie-passthrough-v1 See https://lkml.org/lkml/2016/1/26/371 for more details.

[Qemu-devel] [RFC v2 5/8] memory: add reserved_iova region type

2016-01-29 Thread Eric Auger
Introduce a new reserved_iova region type. This type of iova region is bound to be used by the kernel to map some host physical addresses. A new initializer, memory_region_init_reserved_iova is introduced, as well as a test function, memory_region_is_reserved_iova. Signed-off-by: Eric Auger

[Qemu-devel] [RFC v2 7/8] hw: arm: virt: register reserved IOVA region

2016-01-29 Thread Eric Auger
Registers a 16x64kB reserved iova region. Currently this iova region is used by the host kernel to map host MSI controller frames (GICv2m, GITS_TRANSLATER). The host kernel needs this iova window since it cannot program the PCIe device with MSI frame physical address (as opposed to x86) since the

[Qemu-devel] [PATCH v9 14/16] block: Rewrite bdrv_close_all()

2016-01-29 Thread Max Reitz
This patch rewrites bdrv_close_all(): Until now, all root BDSs have been force-closed. This is bad because it can lead to cached data not being flushed to disk. Instead, try to make all reference holders relinquish their reference voluntarily: 1. All BlockBackend users are handled by making all

[Qemu-devel] [PATCH v9 15/16] iotests: Add test for multiple BB on BDS tree

2016-01-29 Thread Max Reitz
This adds a test for having multiple BlockBackends in one BDS tree. In this case, there is one BB for the protocol BDS and one BB for the format BDS in a simple two-BDS tree (with the protocol BDS and BB added first). When bdrv_close_all() is executed, no cached data from any BDS should be lost;

Re: [Qemu-devel] [PATCH] usb: ehci: add capability mmio write function

2016-01-29 Thread P J P
Hello Gerd, +-- On Fri, 29 Jan 2016, Gerd Hoffmann wrote --+ | On Fr, 2016-01-29 at 18:30 +0530, P J P wrote: | > pointer dereference issue. Add a do nothing 'ehci_caps_write' | > definition to avoid it; Do nothing because capability registers | > are Read Only(RO). | | Surely makes sense,

Re: [Qemu-devel] [PATCH 1/8] target-arm: Properly support EL2 and EL3 in arm_el_is_aa64()

2016-01-29 Thread Sergey Fedorov
On 29.01.2016 19:45, Sergey Fedorov wrote: > On 14.01.2016 21:34, Peter Maydell wrote: >> > Support EL2 and EL3 in arm_el_is_aa64() by implementing the >> > logic for checking the SCR_EL3 and HCR_EL2 register-width bits >> > as appropriate to determine the register width of lower exception >> >

[Qemu-devel] [RFC v2 2/8] Add a function to determine interrupt number for INTx routing

2016-01-29 Thread Eric Auger
This patch adds a PCI bus specific function pointer "route_intx_to_irq" for GPEX. This is used in detemining PCI INTx number from pin. Signed-off-by: Pranavkumar Sawargaonkar Signed-off-by: Tushar Jagad --- hw/pci-host/gpex.c | 12

[Qemu-devel] [RFC v2 4/8] hw: vfio: common: introduce vfio_register_reserved_iova

2016-01-29 Thread Eric Auger
vfio_register_reserved_iova allows to register the reserved IOVA region, typically for MSI frame binding purpose. The kernel allows registering a single reserved IOVA region. Unregistration is handled through legacy vfio_dma_unmap. The function will become static in subsequent patches. However,

[Qemu-devel] [RFC v2 6/8] hw: platform-bus: enable to map any memory region onto the platform-bus

2016-01-29 Thread Eric Auger
The platform bus currently is used to map dynamically instantiable platform device MMIO regions. The platform bus also can be seen as a pool of free guest physical addresses. We would like to use that pool to allocate a contiguous reserved IOVA region usable for MSI message address IOMMU mapping.

[Qemu-devel] [RFC v2 0/8] KVM PCI/MSI passthrough with mach-virt

2016-01-29 Thread Eric Auger
This series enables KVM PCI/MSI passthrough with mach-virt. A new memory region type is introduced (reserved iova). On vfio_listener_region_add this IOVA region is registered to the kernel with VFIO_IOMMU_MAP_DMA (using the new VFIO_DMA_MAP_FLAG_MSI_RESERVED_IOVA flag). The host VFIO PCI driver

Re: [Qemu-devel] [PATCH v7 01/13] machine: Don't allow CPU toplogies with partially filled cores

2016-01-29 Thread Eduardo Habkost
On Fri, Jan 29, 2016 at 05:52:15PM +0100, Igor Mammedov wrote: > On Fri, 29 Jan 2016 13:36:05 -0200 > Eduardo Habkost wrote: > > > On Fri, Jan 29, 2016 at 04:10:47PM +0100, Igor Mammedov wrote: > > > On Fri, 29 Jan 2016 12:24:18 -0200 > > > Eduardo Habkost

[Qemu-devel] [PULL 01/48] block: Add blk_dev_has_tray()

2016-01-29 Thread Kevin Wolf
From: Max Reitz Pull out the check whether a block device has a tray from blk_dev_is_tray_open() into its own function so both attributes (whether there is a tray vs. whether that tray is open) can be queried independently. Cc: qemu-stable

[Qemu-devel] [PULL 09/48] iotests: Change coding style of _filter_nbd in 083

2016-01-29 Thread Kevin Wolf
From: Max Reitz In order to be able to move _filter_nbd to common.filter in the next patch, its coding style needs to be adapted to that of common.filter. That means, we have to convert tabs to four spaces, adjust the alignment of the last line (done with spaces already,

[Qemu-devel] [PULL 06/48] iotests: Limit supported formats for 118

2016-01-29 Thread Kevin Wolf
From: Max Reitz Image formats used in test 118 need to support image creation. Reported-by: Markus Armbruster Signed-off-by: Max Reitz Reviewed-by: Markus Armbruster Signed-off-by: Kevin Wolf ---

[Qemu-devel] [PULL 36/48] raw: Assign bs to file in raw_co_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Signed-off-by: Fam Zheng Message-id: 1453780743-16806-5-git-send-email-f...@redhat.com Reviewed-by: Max Reitz Signed-off-by: Max Reitz --- block/raw-posix.c | 1 + block/raw_bsd.c | 1 + 2 files

[Qemu-devel] [PULL 21/48] nbd: Switch from close to eject notifier

2016-01-29 Thread Kevin Wolf
From: Max Reitz The NBD code uses the BDS close notifier to determine when a medium is ejected. However, now it should use the BB's BDS removal notifier for that instead of the BDS's close notifier. Signed-off-by: Max Reitz Reviewed-by: Fam Zheng

[Qemu-devel] [PULL 32/48] block: acquire in bdrv_query_image_info

2016-01-29 Thread Kevin Wolf
From: Paolo Bonzini NFS calls aio_poll inside bdrv_get_allocated_size. This requires acquiring the AioContext. Signed-off-by: Paolo Bonzini Message-id: 1450867706-19860-1-git-send-email-pbonz...@redhat.com Reviewed-by: Fam Zheng

[Qemu-devel] [PULL 46/48] qemu-img: In "map", use the returned "file" from bdrv_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Now all drivers should return a correct "file", we can make use of it, even with the recursion into backing chain above. Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng

[Qemu-devel] [PULL 37/48] iscsi: Assign bs to file in iscsi_co_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng Message-id: 1453780743-16806-6-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- block/iscsi.c

[Qemu-devel] [PULL 31/48] iotests: Add test for block jobs and BDS ejection

2016-01-29 Thread Kevin Wolf
From: Max Reitz Suggested-by: Paolo Bonzini Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf Signed-off-by: Kevin Wolf --- tests/qemu-iotests/141 | 186

[Qemu-devel] [PATCH 14/15] contrib: Clean up includes

2016-01-29 Thread Peter Maydell
Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes. Signed-off-by: Peter Maydell --- contrib/ivshmem-client/ivshmem-client.c | 2 +-

[Qemu-devel] [PULL 48/48] iotests: Add "qemu-img map" test for VMDK extents

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Signed-off-by: Fam Zheng Message-id: 1453780743-16806-17-git-send-email-f...@redhat.com Reviewed-by: Max Reitz Signed-off-by: Max Reitz --- tests/qemu-iotests/059 | 10 ++

[Qemu-devel] [PATCH 04/15] backends: Clean up includes

2016-01-29 Thread Peter Maydell
Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes. Signed-off-by: Peter Maydell --- backends/baum.c | 1 + backends/hostmem-file.c | 1 +

Re: [Qemu-devel] [PATCH 1/8] target-arm: Properly support EL2 and EL3 in arm_el_is_aa64()

2016-01-29 Thread Sergey Fedorov
On 29.01.2016 20:05, Peter Maydell wrote: > On 29 January 2016 at 16:45, Sergey Fedorov wrote: >> > On 14.01.2016 21:34, Peter Maydell wrote: >>> >> Support EL2 and EL3 in arm_el_is_aa64() by implementing the >>> >> logic for checking the SCR_EL3 and HCR_EL2 register-width

[Qemu-devel] [PULL 00/48] Block patches

2016-01-29 Thread Kevin Wolf
The following changes since commit 047e363b05679724d6b784c6ec6310697fe48ba0: Merge remote-tracking branch 'remotes/pmaydell/tags/pull-softfloat-20160122' into staging (2016-01-22 15:19:21 +) are available in the git repository at: git://repo.or.cz/qemu/kevin.git tags/for-upstream for

[Qemu-devel] [PULL 05/48] vmdk: Fix converting to streamOptimized

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Commit d62d9dc4b8 lifted streamOptimized images's version to 3, but we now refuse to open version 3 images read-write. We need to make streamOptimized an exception to allow converting to it. This fixes the accidentally broken iotests case 059 for the same

[Qemu-devel] [PULL 18/48] block: Add BB-BDS remove/insert notifiers

2016-01-29 Thread Kevin Wolf
From: Max Reitz bdrv_close() no longer signifies ejection of a medium, this is now done by removing the BDS from the BB. Therefore, we want to have a notifier for that in the BB instead of a close notifier in the BDS. The former is added now, the latter is removed later.

[Qemu-devel] [PULL 14/48] iotests: Add test for a nonexistent NBD export

2016-01-29 Thread Kevin Wolf
From: Max Reitz Trying to connect to a nonexistent NBD export should not crash the server. Signed-off-by: Max Reitz Reviewed-by: Eric Blake Signed-off-by: Kevin Wolf --- tests/qemu-iotests/143 | 73

[Qemu-devel] [PULL 07/48] nbd: client_close on error in nbd_co_client_start

2016-01-29 Thread Kevin Wolf
From: Max Reitz Use client_close() if an error in nbd_co_client_start() occurs instead of manually inlining parts of it. This fixes an assertion error on the server side if nbd_negotiate() fails. Signed-off-by: Max Reitz Acked-by: Paolo Bonzini

[Qemu-devel] [PULL 44/48] vmdk: Return extent's file in bdrv_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Signed-off-by: Fam Zheng Message-id: 1453780743-16806-13-git-send-email-f...@redhat.com Reviewed-by: Max Reitz Signed-off-by: Max Reitz --- block/vmdk.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[Qemu-devel] [PULL 22/48] block: Remove BDS close notifier

2016-01-29 Thread Kevin Wolf
From: Max Reitz It is unused now, so we can remove it. Signed-off-by: Max Reitz Reviewed-by: Fam Zheng Reviewed-by: Kevin Wolf Signed-off-by: Kevin Wolf --- block.c| 8

[Qemu-devel] [PATCH 09/15] slirp: Clean up includes

2016-01-29 Thread Peter Maydell
Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes. Signed-off-by: Peter Maydell --- slirp/arp_table.c | 1 + slirp/bootp.c | 1 + slirp/cksum.c | 1

[Qemu-devel] [PULL 41/48] vdi: Assign bs->file->bs to file in vdi_co_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng Message-id: 1453780743-16806-10-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- block/vdi.c | 1 + 1 file changed, 1 insertion(+) diff

[Qemu-devel] [PATCH 11/15] qobject: Clean up includes

2016-01-29 Thread Peter Maydell
Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes. Signed-off-by: Peter Maydell --- qobject/json-lexer.c| 2 +- qobject/json-parser.c | 2 +-

[Qemu-devel] [PATCH 10/15] net: Clean up includes

2016-01-29 Thread Peter Maydell
Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes. Signed-off-by: Peter Maydell --- net/checksum.c | 1 + net/dump.c | 1 + net/eth.c

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

2016-01-29 Thread Peter Maydell
On 29 January 2016 at 17:37, Kevin Wolf wrote: > The following changes since commit 047e363b05679724d6b784c6ec6310697fe48ba0: > > Merge remote-tracking branch > 'remotes/pmaydell/tags/pull-softfloat-20160122' into staging (2016-01-22 > 15:19:21 +) > > are available in

Re: [Qemu-devel] [PATCH 1/8] target-arm: Properly support EL2 and EL3 in arm_el_is_aa64()

2016-01-29 Thread Peter Maydell
On 29 January 2016 at 16:45, Sergey Fedorov wrote: > On 14.01.2016 21:34, Peter Maydell wrote: >> Support EL2 and EL3 in arm_el_is_aa64() by implementing the >> logic for checking the SCR_EL3 and HCR_EL2 register-width bits >> as appropriate to determine the register width

[Qemu-devel] [PULL 10/48] iotests: Move _filter_nbd into common.filter

2016-01-29 Thread Kevin Wolf
From: Max Reitz _filter_nbd can be useful for other NBD tests, too, therefore it should reside in common.filter. Signed-off-by: Max Reitz Reviewed-by: Eric Blake Signed-off-by: Kevin Wolf --- tests/qemu-iotests/083

[Qemu-devel] [PULL 19/48] virtio-blk: Functions for op blocker management

2016-01-29 Thread Kevin Wolf
From: Max Reitz Put the code for setting up and removing op blockers into an own function, respectively. Then, we can invoke those functions whenever a BDS is removed from an virtio-blk BB or inserted into it. Signed-off-by: Max Reitz Reviewed-by: Fam

[Qemu-devel] [PULL 15/48] block: Remove unused struct definition BlockFinishData

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Unused since 94db6d2d3. Signed-off-by: Fam Zheng Reviewed-by: Stefan Hajnoczi Signed-off-by: Kevin Wolf --- blockjob.c | 8 1 file changed, 8 deletions(-) diff --git a/blockjob.c b/blockjob.c

[Qemu-devel] [PULL 08/48] iotests: Rename filter_nbd to _filter_nbd in 083

2016-01-29 Thread Kevin Wolf
From: Max Reitz In the patch after the next, this function is moved to common.filter. Therefore, its name should be preceded by an underscore to signify its global availability. To keep the code motion patch clean, we cannot rename it in the same patch, so we need to choose

[Qemu-devel] [PULL 23/48] block: Use blk_remove_bs() in blk_delete()

2016-01-29 Thread Kevin Wolf
From: Max Reitz Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf Reviewed-by: Fam Zheng Signed-off-by: Kevin Wolf --- block/block-backend.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-)

[Qemu-devel] [PULL 34/48] qcow: Assign bs->file->bs to file in qcow_co_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Reviewed-by: Eric Blake Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng Message-id: 1453780743-16806-3-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- block/qcow.c

[Qemu-devel] [PULL 45/48] block: Use returned *file in bdrv_co_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Now that all drivers return the right "file" pointer, we can use it. Signed-off-by: Fam Zheng Reviewed-by: Max Reitz Message-id: 1453780743-16806-14-git-send-email-f...@redhat.com Signed-off-by: Max Reitz

[Qemu-devel] [PULL 24/48] blockdev: Use blk_remove_bs() in do_drive_del()

2016-01-29 Thread Kevin Wolf
From: Max Reitz Signed-off-by: Max Reitz Reviewed-by: Kevin Wolf Reviewed-by: Fam Zheng Signed-off-by: Kevin Wolf --- blockdev.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Qemu-devel] [PATCH 01/15] ui: Clean up includes

2016-01-29 Thread Peter Maydell
Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes. Signed-off-by: Peter Maydell --- ui/console-gl.c | 1 + ui/console.c | 1 +

[Qemu-devel] [PULL 42/48] vpc: Assign bs->file->bs to file in vpc_co_get_block_status

2016-01-29 Thread Kevin Wolf
From: Fam Zheng Reviewed-by: Stefan Hajnoczi Signed-off-by: Fam Zheng Message-id: 1453780743-16806-11-git-send-email-f...@redhat.com Signed-off-by: Max Reitz --- block/vpc.c | 2 ++ 1 file changed, 2 insertions(+)

[Qemu-devel] [PULL 43/48] vmdk: Fix calculation of block status's offset

2016-01-29 Thread Kevin Wolf
From: Fam Zheng "offset" is the offset of cluster and sector_num doesn't necessarily refer to the start of it, it should add index_in_cluster. Signed-off-by: Fam Zheng Message-id: 1453780743-16806-12-git-send-email-f...@redhat.com Reviewed-by: Max Reitz

<    1   2   3   4   >