Re: [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Peter Xu
On Wed, Sep 27, 2017 at 06:46:18AM +, Bharat Bhushan wrote: > Hi Peter, Hi, Bharat! > > While vfio with virtio-iommu I observed one issue, When virtio-iommu device > exists but guest kernel does not have virtio-iommu driver (not enabled in > Config) then IOMMU faults are reported on host.

Re: [Qemu-devel] [PATCH v2 1/2] s390x/ais: enable ais when migration is available

2017-09-27 Thread Christian Borntraeger
On 09/27/2017 07:45 AM, Yi Min Zhao wrote: > > > 在 2017/9/26 下午9:36, Christian Borntraeger 写道: >> @@ -557,6 +557,12 @@ static void kvm_s390_flic_realize(DeviceState *dev, >> Error **errp) >> test_attr.group = KVM_DEV_FLIC_CLEAR_IO_IRQ; >> flic_state->clear_io_supported = !ioctl(fli

Re: [Qemu-devel] [PATCH v2 2/2] s390x/ais: disable ais for compat machines

2017-09-27 Thread Christian Borntraeger
On 09/26/2017 03:51 PM, David Hildenbrand wrote: > On 26.09.2017 15:36, Christian Borntraeger wrote: >> With newer kernels that do support the ais feature (4.13) a qemu 2.11 >> will not only enable the ais feature for the 2.11 machine, but also >> for a <=2.10 compat machine. As this feature is n

Re: [Qemu-devel] [PATCH v3 2/5] ppc: spapr: Handle "ibm, nmi-register" and "ibm, nmi-interlock" RTAS calls

2017-09-27 Thread David Gibson
On Thu, Sep 21, 2017 at 02:39:06PM +0530, Aravinda Prasad wrote: > > > On Tuesday 22 August 2017 07:38 AM, David Gibson wrote: > > [ . . . ] > > diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h > index 46012b3..eee8d33 100644 > --- a/include/hw/ppc/spapr.h > +++

Re: [Qemu-devel] [PATCH 0/1] qemu-firmware repo

2017-09-27 Thread Paolo Bonzini
On 26/09/2017 13:17, Gerd Hoffmann wrote: > > Ok, we want for varios reasons separate the firmware from the main > qemu repo. Here is my attempt to create such a repo. For starters > only seabios has been added here. > > You can find the repo with currently three patches here: > https://www

[Qemu-devel] [PATCH v3 3/3] s390x/ais: disable ais for compat machines

2017-09-27 Thread Christian Borntraeger
With newer kernels that do support the ais feature (4.13) a qemu 2.11 will not only enable the ais feature for the 2.11 machine, but also for a <=2.10 compat machine. As this feature is not available in QEMU <=2.9 (and QEMU 2.10.1), this guest will fail to migrate back to an older qemu like 2.9 wit

[Qemu-devel] [PATCH v3 0/3] more ais fixups

2017-09-27 Thread Christian Borntraeger
I think Patches 1 and 2 are undisputed. Patch 1 needs to go into 2.10.1 as well. Patch 2 is for 2.11 Patch 3 is still under discussion if this is the right solution v2->v3: - add the ais removal from the z14 cpu model - fix indent Christian Borntraeger (3): s390x/cpumodel: remove ais from z1

[Qemu-devel] [PATCH v3 1/3] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Christian Borntraeger
We disabled ais for 2.10 lets also remove it from the z14 default model. Fixes: 3f2d07b3b01e s390x/ais: for 2.10 stable: disable ais facility Signed-off-by: Christian Borntraeger --- target/s390x/gen-features.c | 1 - 1 file changed, 1 deletion(-) diff --git a/target/s390x/gen-features.c b/tar

[Qemu-devel] [PATCH v3 2/3] s390x/ais: enable ais when migration is available

2017-09-27 Thread Christian Borntraeger
Instead of unconditionally enabling the KVM AIS capability in the kvm arch init function, do this in the flic realize function when we know if migration is available. This requires to initialize flic before the CPUs (and cpu models). Signed-off-by: Christian Borntraeger Reviewed-by: David Hildenb

Re: [Qemu-devel] [RFC v2 10/33] migration: allow dst vm pause on postcopy

2017-09-27 Thread Peter Xu
On Thu, Sep 21, 2017 at 08:29:03PM +0100, Dr. David Alan Gilbert wrote: > * Peter Xu (pet...@redhat.com) wrote: > > When there is IO error on the incoming channel (e.g., network down), > > instead of bailing out immediately, we allow the dst vm to switch to the > > new POSTCOPY_PAUSE state. Current

Re: [Qemu-devel] [PATCH v1 0/6] s390x/tcg: fix some accesses using low address

2017-09-27 Thread Thomas Huth
On 27.09.2017 01:10, Richard Henderson wrote: > On 09/26/2017 11:33 AM, David Hildenbrand wrote: >> David Hildenbrand (6): >> s390x/tcg: fix checking for invalid memory check >> s390x/tcg: add MMU for real addresses >> s390x/tcg: make lura(g) use the new _real mmu. >> s390x/tcg: make stora(

Re: [Qemu-devel] [PATCH] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Cornelia Huck
On Tue, 26 Sep 2017 20:26:31 +0200 David Hildenbrand wrote: > On 26.09.2017 20:05, Christian Borntraeger wrote: > > We disabled ais for 2.10 lets also remove it from the z14 default model. > > > > Fixes: 3f2d07b3b01e s390x/ais: for 2.10 stable: disable ais facility > > Signed-off-by: Christian

Re: [Qemu-devel] [PATCH] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Christian Borntraeger
On 09/27/2017 09:39 AM, Cornelia Huck wrote: > On Tue, 26 Sep 2017 20:26:31 +0200 > David Hildenbrand wrote: > >> On 26.09.2017 20:05, Christian Borntraeger wrote: >>> We disabled ais for 2.10 lets also remove it from the z14 default model. >>> >>> Fixes: 3f2d07b3b01e s390x/ais: for 2.10 stabl

Re: [Qemu-devel] [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Linu Cherian
Hi, On Wed Sep 27, 2017 at 12:03:15PM +0530, Bharat Bhushan wrote: > This patch series integrates VFIO/VHOST with virtio-iommu. > > This version is mainly about rebasing on v4 version on > virtio-iommu device framework from Eric Augur and > addresing review comments. > > This patch series allows

[Qemu-devel] [PULL 02/26] ppc/kvm: drop kvmppc_has_cap_htab_fd()

2017-09-27 Thread David Gibson
From: Greg Kurz It never got used since its introduction (commit 7c43bca004af). Signed-off-by: Greg Kurz Signed-off-by: David Gibson --- target/ppc/kvm.c | 5 - target/ppc/kvm_ppc.h | 6 -- 2 files changed, 11 deletions(-) diff --git a/target/ppc/kvm.c b/target/ppc/kvm.c index 20

[Qemu-devel] [PULL 00/26] ppc-for-2.11 queue 20170927

2017-09-27 Thread David Gibson
The following changes since commit 31bc1d8481af414cfa2857f905e40f7d8e6d5b2e: Merge remote-tracking branch 'remotes/mjt/tags/trivial-patches-fetch' into staging (2017-09-26 19:49:08 +0100) are available in the git repository at: git://github.com/dgibson/qemu.git tags/ppc-for-2.1

[Qemu-devel] [PULL 04/26] ehci: Add ppc4xx-ehci for the USB 2.0 controller in embedded PPC SoCs

2017-09-27 Thread David Gibson
From: BALATON Zoltan Some PPC SoCs have an EHCI with OHCI companion USB controller. Add a new type for this similar to types used for other embedded SoCs. Signed-off-by: BALATON Zoltan Reviewed-by: David Gibson Reviewed-by: Gerd Hoffmann Signed-off-by: David Gibson --- hw/usb/hcd-ehci-sysbu

[Qemu-devel] [PULL 01/26] ppc/kvm: check some capabilities with kvm_vm_check_extension()

2017-09-27 Thread David Gibson
From: Greg Kurz The following capabilities are VM specific: - KVM_CAP_PPC_SMT_POSSIBLE - KVM_CAP_PPC_HTAB_FD - KVM_CAP_PPC_ALLOC_HTAB If both KVM HV and KVM PR are present, checking them always return the HV value, even if we explicitely requested to use PR. This has no visible effect for KVM_C

[Qemu-devel] [PULL 17/26] spapr_pci: make index property mandatory

2017-09-27 Thread David Gibson
From: Greg Kurz PHBs can be created with an index property, in which case the machine code automatically sets all the MMIO windows at addresses derived from the index. Alternatively, they can be manually created without index, but the user has to provide addresses for all MMIO windows. The non-i

[Qemu-devel] [PULL 05/26] ppc: Add 460EX embedded CPU

2017-09-27 Thread David Gibson
From: BALATON Zoltan Despite its name it is a 440 core CPU Signed-off-by: BALATON Zoltan Reviewed-by: David Gibson Signed-off-by: David Gibson --- target/ppc/cpu-models.c | 3 +++ target/ppc/cpu-models.h | 1 + target/ppc/translate_init.c | 38 ++

[Qemu-devel] [PULL 06/26] ppc4xx: Add more PLB registers

2017-09-27 Thread David Gibson
From: BALATON Zoltan These registers are present in 440 SoCs (and maybe in others too) and U-Boot accesses them when printing register info. We don't emulate these but add them to avoid crashing when they are read or written. Signed-off-by: BALATON Zoltan Signed-off-by: David Gibson --- hw/pp

[Qemu-devel] [PULL 03/26] ohci: Allow sysbus version to be used as a companion

2017-09-27 Thread David Gibson
From: BALATON Zoltan Some PPC SoCs have an EHCI with OHCI companion USB controller. To emulate this allow the sysbus version of OHCI to be used as a companion. Signed-off-by: BALATON Zoltan Reviewed-by: David Gibson Reviewed-by: Gerd Hoffmann Signed-off-by: David Gibson --- hw/usb/hcd-ohci.

[Qemu-devel] [PULL 21/26] spapr: fix the value of SDR1 in kvmppc_put_books_sregs()

2017-09-27 Thread David Gibson
From: Greg Kurz When running with KVM PR, if a new HPT is allocated we need to inform KVM about the HPT address and size. This is currently done by hacking the value of SDR1 and pushing it to KVM in several places. Also, migration breaks the guest since it is very unlikely the HPT has the same a

[Qemu-devel] [PULL 10/26] ppc/ide/macio: Add missing registers

2017-09-27 Thread David Gibson
From: Benjamin Herrenschmidt The timing register exists on all variants of MacIO IDE, we just store and return its value. The interrupts register only exists on KeyLargo but it doesn't hurt to have it. The lack of this register causes MacOS X to hangs under some circumstances. Both are 32-bit o

[Qemu-devel] [PULL 08/26] ppc/mac: Advertise a high clock frequency for NewWorld Macs

2017-09-27 Thread David Gibson
From: Benjamin Herrenschmidt We use 900Mhz, otherwise MacOS X 10.5 refuses to install. Signed-off-by: Benjamin Herrenschmidt Signed-off-by: David Gibson --- hw/ppc/mac_newworld.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/ppc/mac_newworld.c b/hw/ppc/mac_newworld.c

[Qemu-devel] [PULL 18/26] ppc: remove unused CPU definitions

2017-09-27 Thread David Gibson
From: John Snow Following commit aef77960, remove now-unused definitions from cpu-models.h. Signed-off-by: John Snow Reviewed-by: Thomas Huth Signed-off-by: David Gibson --- target/ppc/cpu-models.h | 32 1 file changed, 32 deletions(-) diff --git a/target/pp

[Qemu-devel] [PULL 26/26] macio: use object link between MACIO_IDE and MAC_DBDMA object

2017-09-27 Thread David Gibson
From: Mark Cave-Ayland Using a standard QOM object link we can pass a reference to the MAC_DBDMA controller to the MACIO_IDE object which removes the last external parameter to macio_ide_register_dma(). Signed-off-by: Mark Cave-Ayland Signed-off-by: David Gibson --- hw/ide/macio.c| 9

[Qemu-devel] [PULL 11/26] ppc: Fix OpenPIC model

2017-09-27 Thread David Gibson
From: Benjamin Herrenschmidt Apple uses an IBM MPIC2A without timers, it has 64 sources. Signed-off-by: Benjamin Herrenschmidt Signed-off-by: David Gibson --- hw/intc/openpic.c| 35 +++ hw/ppc/mac_newworld.c| 2 +- include/hw/ppc/openpic.h | 1 +

[Qemu-devel] [PULL 13/26] ppc/kvm: generalize the use of kvmppc_get_htab_fd()

2017-09-27 Thread David Gibson
From: Greg Kurz The use of KVM_PPC_GET_HTAB_FD is open-coded in kvmppc_read_hptes() and kvmppc_write_hpte(). This patch modifies kvmppc_get_htab_fd() so that it can be used everywhere we need to access the in-kernel htab: - add an index argument => only kvmppc_read_hptes() passes an actual ind

[Qemu-devel] [PULL 12/26] ppc/kvm: change kvmppc_get_htab_fd() to return -errno on error

2017-09-27 Thread David Gibson
From: Greg Kurz When kvmppc_get_htab_fd() fails, its return value is propagated up to qemu_savevm_state_iterate() or to qemu_savevm_state_complete_precopy(). All savevm handlers expect to receive a negative errno on error. Let's patch kvmppc_get_htab_fd() accordingly. While here, let's change h

[Qemu-devel] [PULL 15/26] ppc/pnv: Improve macro parenthesization

2017-09-27 Thread David Gibson
From: Eric Blake Although none of the existing macro call-sites were broken, it's always better to write macros that properly parenthesize arguments that can be complex expressions, so that the intended order of operations is not broken. Signed-off-by: Eric Blake Reviewed-by: Cédric Le Goater

[Qemu-devel] [PULL 07/26] ppc: QOMify g3beige machine

2017-09-27 Thread David Gibson
From: Mark Cave-Ayland Signed-off-by: Mark Cave-Ayland Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: David Gibson --- hw/ppc/mac_oldworld.c | 17 +++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/hw/ppc/mac_oldworld.c b/hw/ppc/mac_oldworld.c index 61838c3e6

[Qemu-devel] [PULL 09/26] ppc/mac: More rework of the DBDMA emulation

2017-09-27 Thread David Gibson
From: Benjamin Herrenschmidt This completely reworks the handling of the control register according to my understanding of the HW and the spec. It should (hopefully ... still testing) fix a number of issues most notably cases of MacOS hanging. Also update dbdma_unassigned_rw() and dbdma_unassig

[Qemu-devel] [PULL 23/26] mac_dbdma: QOMify

2017-09-27 Thread David Gibson
From: Mark Cave-Ayland Signed-off-by: Mark Cave-Ayland Signed-off-by: David Gibson --- hw/misc/macio/mac_dbdma.c | 59 ++ include/hw/ppc/mac_dbdma.h | 6 + 2 files changed, 55 insertions(+), 10 deletions(-) diff --git a/hw/misc/macio/mac_dbdma

[Qemu-devel] [PULL 14/26] spapr: introduce helpers to migrate HPT chunks and the end marker

2017-09-27 Thread David Gibson
From: Greg Kurz This consolidates some duplicated code in a dedicated helpers. Signed-off-by: Greg Kurz Signed-off-by: David Gibson --- hw/ppc/spapr.c | 38 +- 1 file changed, 21 insertions(+), 17 deletions(-) diff --git a/hw/ppc/spapr.c b/hw/ppc/spapr.c i

[Qemu-devel] [PULL 19/26] ppc: remove all unused CPU definitions

2017-09-27 Thread David Gibson
From: John Snow Remove *all* unused CPU definitions as indicated by compile-time `#if 0` constructs. Signed-off-by: John Snow [dwg: Removed some additional now-useless comments] Reviewed-by: Thomas Huth Signed-off-by: David Gibson --- target/ppc/cpu-models.h | 225 ---

[Qemu-devel] [PULL 25/26] macio: pass channel into MACIOIDEState via qdev property

2017-09-27 Thread David Gibson
From: Mark Cave-Ayland One of the reasons macio_ide_register_dma() needs to exist is because the channel id isn't passed into the MACIO_IDE object. Pass in the channel id using a qdev property to remove this requirement. Signed-off-by: Mark Cave-Ayland Signed-off-by: David Gibson --- hw/ide/m

[Qemu-devel] [PULL 16/26] macio: convert pmac_ide_ops from old_mmio

2017-09-27 Thread David Gibson
From: Mark Cave-Ayland Signed-off-by: Mark Cave-Ayland Signed-off-by: David Gibson --- hw/ide/macio.c | 181 - 1 file changed, 75 insertions(+), 106 deletions(-) diff --git a/hw/ide/macio.c b/hw/ide/macio.c index db5db390e7..18ae952934 1

[Qemu-devel] [PULL 20/26] ppc/pnv: check for OPAL firmware file presence

2017-09-27 Thread David Gibson
From: Cédric Le Goater and exit before uselessly trying to load it if the file does not exists. Issue discovered by Coverity Scan. Signed-off-by: Cédric Le Goater Reviewed-by: Thomas Huth Signed-off-by: David Gibson --- hw/ppc/pnv.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-)

Re: [Qemu-devel] [PATCH v2 2/2] s390x/ais: disable ais for compat machines

2017-09-27 Thread Christian Borntraeger
On 09/27/2017 09:12 AM, Christian Borntraeger wrote: > > > On 09/26/2017 03:51 PM, David Hildenbrand wrote: >> On 26.09.2017 15:36, Christian Borntraeger wrote: >>> With newer kernels that do support the ais feature (4.13) a qemu 2.11 >>> will not only enable the ais feature for the 2.11 machin

[Qemu-devel] [PULL 24/26] mac_dbdma: remove DBDMA_init() function

2017-09-27 Thread David Gibson
From: Mark Cave-Ayland Instead we can now instantiate the MAC_DBDMA object directly within the macio device. We also add the DBDMA device as a child property so that it is possible to retrieve later. Signed-off-by: Mark Cave-Ayland Signed-off-by: David Gibson --- hw/misc/macio/mac_dbdma.c |

Re: [Qemu-devel] [PATCH 4/9] s390x: refactor error handling for SSCH and RSCH

2017-09-27 Thread Dong Jia Shi
* Halil Pasic [2017-09-25 12:57:31 +0200]: [restored Cc:] > > > On 09/25/2017 09:31 AM, Dong Jia Shi wrote: > > * Cornelia Huck [2017-09-08 11:59:50 +0200]: > > > >> On Fri, 8 Sep 2017 11:21:57 +0200 > >> Halil Pasic wrote: > >> > >>> On 09/08/2017 05:41 AM, Dong Jia Shi wrote: > Let'

[Qemu-devel] [PULL 22/26] mac_dbdma: remove unused IO fields from DBDMAState

2017-09-27 Thread David Gibson
From: Mark Cave-Ayland These fields were used to manually handle IO requests that weren't aligned to a sector boundary before this feature was supported by the block API. Once the block API changed to support byte-aligned IO requests, the macio controller was switched over to use it in commit be

Re: [Qemu-devel] [PATCH] spapr: move registration of "host" CPU core type to machine code

2017-09-27 Thread Igor Mammedov
On Wed, 27 Sep 2017 16:21:43 +1000 David Gibson wrote: > On Tue, Sep 26, 2017 at 09:19:28AM +0200, Greg Kurz wrote: > > On Tue, 26 Sep 2017 12:57:39 +1000 > > David Gibson wrote: > > > > > On Mon, Sep 25, 2017 at 11:47:33AM +0200, Greg Kurz wrote: > > > > The CPU core abstraction belongs to

Re: [Qemu-devel] [PATCH] spapr: select default vty

2017-09-27 Thread David Gibson
On Tue, Sep 26, 2017 at 02:17:50PM +0200, Laurent Vivier wrote: > SLOF uses the "/chosen/linux,stdout-path" variable to > choose its console. This variable is provided by QEMU. > QEMU selects the spapr-vty using the "reg" property: > it takes the vty with the lowest reg number. > This patch allows

Re: [Qemu-devel] [PATCH] PCI: PCIe access should always be little endian

2017-09-27 Thread Matt Redfearn
ping? On 16/08/17 11:25, Marcel Apfelbaum wrote: On 15/08/2017 17:44, Matt Redfearn wrote: PCIe busses are always little endian, so set the endianness of the memory region to little endian rather than native such that operations work as expected on big endian targets. Signed-off-by: Matt Redf

Re: [Qemu-devel] [PATCH] spapr: select default vty

2017-09-27 Thread Laurent Vivier
On 27/09/2017 09:59, David Gibson wrote: > On Tue, Sep 26, 2017 at 02:17:50PM +0200, Laurent Vivier wrote: >> SLOF uses the "/chosen/linux,stdout-path" variable to >> choose its console. This variable is provided by QEMU. >> QEMU selects the spapr-vty using the "reg" property: >> it takes the vty w

Re: [Qemu-devel] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Bharat Bhushan
Hi Peter, > -Original Message- > From: Peter Xu [mailto:pet...@redhat.com] > Sent: Wednesday, September 27, 2017 12:32 PM > To: Bharat Bhushan > Cc: eric.au...@redhat.com; eric.auger@gmail.com; > peter.mayd...@linaro.org; alex.william...@redhat.com; m...@redhat.com; > qemu-...@nongnu.

Re: [Qemu-devel] [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Bharat Bhushan
Hi, > -Original Message- > From: Linu Cherian [mailto:linuc.dec...@gmail.com] > Sent: Wednesday, September 27, 2017 1:11 PM > To: Bharat Bhushan > Cc: eric.au...@redhat.com; eric.auger@gmail.com; > peter.mayd...@linaro.org; alex.william...@redhat.com; m...@redhat.com; > qemu-...@nongn

[Qemu-devel] [PATCH] console: purge curses bits from console.h

2017-09-27 Thread Gerd Hoffmann
Handle the translation from vga chars to curses chars in curses_update() instead of console_write_ch(). Purge any curses support bits from ui/console.h include file. Signed-off-by: Gerd Hoffmann --- include/ui/console.h | 23 ++- ui/curses.c | 25 +++

Re: [Qemu-devel] [PATCH] virtio: introduce `info virtio' hmp command

2017-09-27 Thread Cornelia Huck
On Tue, 26 Sep 2017 19:20:20 +0200 Kevin Wolf wrote: > Am 26.09.2017 um 18:13 hat Jan Dakinevich geschrieben: > > The command is intended for exposing device specific virtio feature bits > > and their negotiation status. It is convenient and useful for debug > > purpose. > > > > Names of feature

Re: [Qemu-devel] [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Auger Eric
Hi Linu, On 27/09/2017 10:30, Bharat Bhushan wrote: > Hi, > >> -Original Message- >> From: Linu Cherian [mailto:linuc.dec...@gmail.com] >> Sent: Wednesday, September 27, 2017 1:11 PM >> To: Bharat Bhushan >> Cc: eric.au...@redhat.com; eric.auger@gmail.com; >> peter.mayd...@linaro.org

Re: [Qemu-devel] [PATCH v2 1/6] migration: pre_save return int

2017-09-27 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > Modify the pre_save method on VMStateDescription to return an int > rather than void so that it potentially can fail. > > Changed zillions of devices to make them return 0; the only > case I've made it return non-0 is hw/i

Re: [Qemu-devel] [PATCH] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Cornelia Huck
On Wed, 27 Sep 2017 09:41:21 +0200 Christian Borntraeger wrote: > On 09/27/2017 09:39 AM, Cornelia Huck wrote: > > On Tue, 26 Sep 2017 20:26:31 +0200 > > David Hildenbrand wrote: > > > >> On 26.09.2017 20:05, Christian Borntraeger wrote: > >>> We disabled ais for 2.10 lets also remove it fr

Re: [Qemu-devel] [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Linu Cherian
On Wed Sep 27, 2017 at 08:30:51AM +, Bharat Bhushan wrote: > Hi, > > > -Original Message- > > From: Linu Cherian [mailto:linuc.dec...@gmail.com] > > Sent: Wednesday, September 27, 2017 1:11 PM > > To: Bharat Bhushan > > Cc: eric.au...@redhat.com; eric.auger@gmail.com; > > peter.ma

Re: [Qemu-devel] [PATCH v2 2/6] migration: check pre_save return in vmstate_save_state

2017-09-27 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > Check the return value of pre_save state and fail vmstate_save_state > if the pre_save failed. > > Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->name

2017-09-27 Thread Auger Eric
Hi Wei, On 26/09/2017 18:52, Wei Huang wrote: > > > On 09/26/2017 10:37 AM, Andrew Jones wrote: >> On Tue, Sep 26, 2017 at 04:28:03PM +0100, Peter Maydell wrote: >>> On 26 September 2017 at 11:54, Andrew Jones wrote: On Fri, Sep 22, 2017 at 08:51:15AM -0500, Wei Huang wrote: > Instead

Re: [Qemu-devel] [PATCH v2 4/6] migration: wire vmstate_save_state errors up to vmstate_subsection_save

2017-09-27 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > Route the errors from vmstate_subsection_save up through > vmstate_subsection_save (and back down, all rather recursive). > > Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH v2 3/6] migration: Check field save returns

2017-09-27 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > Check the return values from vmstate_save_state for fields and also the > return values from 'put' for fields that use that. > > Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

Re: [Qemu-devel] [PATCH V3] migration: disable auto-converge during bulk block migration

2017-09-27 Thread Juan Quintela
Peter Lieven wrote: > auto-converge and block migration currently do not play well together. > During block migration the auto-converge logic detects that ram > migration makes no progress and thus throttles down the vm until > it nearly stalls completely. Avoid this by disabling the throttling >

Re: [Qemu-devel] [PATCH v2 5/6] migration: Route errors up through vmstate_save

2017-09-27 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > Route the errors from vsmtate_save_state back up through > vmstate_save and out to the normal device state path. > That's the normal error path done. > > Signed-off-by: Dr. David Alan Gilbert Reviewed-by: Juan Quintela

Re: [Qemu-devel] [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Linu Cherian
On Wed Sep 27, 2017 at 10:55:07AM +0200, Auger Eric wrote: > Hi Linu, > > On 27/09/2017 10:30, Bharat Bhushan wrote: > > Hi, > > > >> -Original Message- > >> From: Linu Cherian [mailto:linuc.dec...@gmail.com] > >> Sent: Wednesday, September 27, 2017 1:11 PM > >> To: Bharat Bhushan > >> C

Re: [Qemu-devel] [PATCH v2 6/6] migration: Route more error paths

2017-09-27 Thread Juan Quintela
"Dr. David Alan Gilbert (git)" wrote: > From: "Dr. David Alan Gilbert" > > vmstate_save is called in a few places, and vmstate_save_state is > called in lots of places. > > Route error returns from the easier cases back up; there are lots > of more complex cases where their own error paths need

Re: [Qemu-devel] [PATCH] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Christian Borntraeger
On 09/27/2017 10:57 AM, Cornelia Huck wrote: > On Wed, 27 Sep 2017 09:41:21 +0200 > Christian Borntraeger wrote: > >> On 09/27/2017 09:39 AM, Cornelia Huck wrote: >>> On Tue, 26 Sep 2017 20:26:31 +0200 >>> David Hildenbrand wrote: >>> On 26.09.2017 20:05, Christian Borntraeger wrote:

Re: [Qemu-devel] [PATCH 0/1] qemu-firmware repo

2017-09-27 Thread Gerd Hoffmann
On Wed, 2017-09-27 at 09:19 +0200, Paolo Bonzini wrote: > On 26/09/2017 13:17, Gerd Hoffmann wrote: > > > > Ok, we want for varios reasons separate the firmware from the main > > qemu repo.  Here is my attempt to create such a repo.  For starters > > only seabios has been added here. > > > > You

Re: [Qemu-devel] [PATCH v3 1/3] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Cornelia Huck
On Wed, 27 Sep 2017 09:20:28 +0200 Christian Borntraeger wrote: > We disabled ais for 2.10 lets also remove it from the z14 default model. s/lets/, so let's/ > > Fixes: 3f2d07b3b01e s390x/ais: for 2.10 stable: disable ais facility > Signed-off-by: Christian Borntraeger > --- > target/s390x/

Re: [Qemu-devel] [PATCH] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Cornelia Huck
On Wed, 27 Sep 2017 11:09:28 +0200 Christian Borntraeger wrote: > On 09/27/2017 10:57 AM, Cornelia Huck wrote: > > On Wed, 27 Sep 2017 09:41:21 +0200 > > Christian Borntraeger wrote: > > > >> On 09/27/2017 09:39 AM, Cornelia Huck wrote: > >>> On Tue, 26 Sep 2017 20:26:31 +0200 > >>> David H

Re: [Qemu-devel] [PATCH 0/1] qemu-firmware repo

2017-09-27 Thread Daniel P. Berrange
On Wed, Sep 27, 2017 at 09:19:22AM +0200, Paolo Bonzini wrote: > On 26/09/2017 13:17, Gerd Hoffmann wrote: > > > > Ok, we want for varios reasons separate the firmware from the main > > qemu repo. Here is my attempt to create such a repo. For starters > > only seabios has been added here. > > >

Re: [Qemu-devel] [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Linu Cherian
On Wed Sep 27, 2017 at 10:55:07AM +0200, Auger Eric wrote: > Hi Linu, > > On 27/09/2017 10:30, Bharat Bhushan wrote: > > Hi, > > > >> -Original Message- > >> From: Linu Cherian [mailto:linuc.dec...@gmail.com] > >> Sent: Wednesday, September 27, 2017 1:11 PM > >> To: Bharat Bhushan > >> C

Re: [Qemu-devel] [Qemu-arm] [PATCH v4 0/5] virtio-iommu: VFIO integration

2017-09-27 Thread Auger Eric
Hi Linu, On 27/09/2017 11:21, Linu Cherian wrote: > On Wed Sep 27, 2017 at 10:55:07AM +0200, Auger Eric wrote: >> Hi Linu, >> >> On 27/09/2017 10:30, Bharat Bhushan wrote: >>> Hi, >>> -Original Message- From: Linu Cherian [mailto:linuc.dec...@gmail.com] Sent: Wednesday, Sept

[Qemu-devel] [PULL URGENT 0/1] another s390x compat fix

2017-09-27 Thread Cornelia Huck
The following changes since commit 31bc1d8481af414cfa2857f905e40f7d8e6d5b2e: Merge remote-tracking branch 'remotes/mjt/tags/trivial-patches-fetch' into staging (2017-09-26 19:49:08 +0100) are available in the git repository at: git://github.com/cohuck/qemu tags/s390x-20170927

[Qemu-devel] [PULL URGENT 1/1] s390x/cpumodel: remove ais from z14 default model-> also for 2.10.1

2017-09-27 Thread Cornelia Huck
From: Christian Borntraeger We disabled ais for 2.10, so let's also remove it from the z14 default model. Fixes: 3f2d07b3b01e ("s390x/ais: for 2.10 stable: disable ais facility") CC: qemu-sta...@nongnu.org Signed-off-by: Christian Borntraeger Message-Id: <20170927072030.35737-2-borntrae...@de.i

Re: [Qemu-devel] [PATCH 5/5] s390x/ccs: add ccw-tester emulated device

2017-09-27 Thread Dong Jia Shi
* Dong Jia Shi [2017-09-26 15:48:56 +0800]: [...] > > > > > > Tried to test with the following method: > > > 1. Start g1 (first level guest on kvm a host) with a virtio blk device > > >defined: > > > -drive > > > file=/dev/disk/by-path/ccw-0.0.3f3e,if=none,id=drive-virtio-disk1,format=raw

Re: [Qemu-devel] [PATCH 1/1] s390x: create a compat s390 phb for <=2.10

2017-09-27 Thread Cornelia Huck
On Tue, 26 Sep 2017 20:40:25 +0200 David Hildenbrand wrote: > On 26.09.2017 18:20, Cornelia Huck wrote: > > d32bd032d8 ("s390x/ccw: create s390 phb conditionally") made > > creating the s390 phb dependant on the zpci facility. This broke > > migration from pre-cpu model machines which was fixed w

Re: [Qemu-devel] [RFC v2 20/33] migration: new message MIG_RP_MSG_RECV_BITMAP

2017-09-27 Thread Peter Xu
On Fri, Sep 22, 2017 at 12:05:42PM +0100, Dr. David Alan Gilbert wrote: [...] > > diff --git a/migration/ram.c b/migration/ram.c > > index 7e20097..5d938e3 100644 > > --- a/migration/ram.c > > +++ b/migration/ram.c > > @@ -182,6 +182,70 @@ void ramblock_recv_bitmap_clear(RAMBlock *rb, void > > *

Re: [Qemu-devel] [PATCH 2/2] hw/pcie: disable IO port fwd by default for pcie-root-port

2017-09-27 Thread Marcel Apfelbaum
Hi Laszlo, On 20/09/2017 14:01, Laszlo Ersek wrote: We now have if (mem_pref_32_reserve != (uint32_t)-1 && mem_pref_64_reserve != (uint64_t)-1) { error_setg(errp, but after introducing the zero value, this is going to be too strict. Consider all the possible combinatio

Re: [Qemu-devel] [PATCH 8/8] nbd: Minimal structured read for client

2017-09-27 Thread Vladimir Sementsov-Ogievskiy
26.09.2017 01:19, Eric Blake wrote: On 09/25/2017 08:58 AM, Vladimir Sementsov-Ogievskiy wrote: Minimal implementation: drop most of additional error information. Signed-off-by: Vladimir Sementsov-Ogievskiy --- block/nbd-client.h | 2 + include/block/nbd.h | 15 - block/nbd-client.

Re: [Qemu-devel] [RFC v2 21/33] migration: new cmd MIG_CMD_POSTCOPY_RESUME

2017-09-27 Thread Peter Xu
On Fri, Sep 22, 2017 at 12:08:06PM +0100, Dr. David Alan Gilbert wrote: > * Peter Xu (pet...@redhat.com) wrote: > > Introducing this new command to be sent when the source VM is ready to > > resume the paused migration. What the destination does here is > > basically release the fault thread to co

Re: [Qemu-devel] [PATCH 1/1] s390x: create a compat s390 phb for <=2.10

2017-09-27 Thread Yi Min Zhao
在 2017/9/27 下午5:47, Cornelia Huck 写道: On Tue, 26 Sep 2017 20:40:25 +0200 David Hildenbrand wrote: On 26.09.2017 18:20, Cornelia Huck wrote: d32bd032d8 ("s390x/ccw: create s390 phb conditionally") made creating the s390 phb dependant on the zpci facility. This broke migration from pre-cpu mo

Re: [Qemu-devel] [PATCH V3] migration: disable auto-converge during bulk block migration

2017-09-27 Thread Dr. David Alan Gilbert
* Peter Lieven (p...@kamp.de) wrote: > auto-converge and block migration currently do not play well together. > During block migration the auto-converge logic detects that ram > migration makes no progress and thus throttles down the vm until > it nearly stalls completely. Avoid this by disabling t

[Qemu-devel] [PATCH v2] console: purge curses bits from console.h

2017-09-27 Thread Gerd Hoffmann
Handle the translation from vga chars to curses chars in curses_update() instead of console_write_ch(). Purge any curses support bits from ui/console.h include file. Signed-off-by: Gerd Hoffmann --- include/ui/console.h | 23 ++- target/unicore32/helper.c | 6 ++ u

Re: [Qemu-devel] [PATCH 0/1] qemu-firmware repo

2017-09-27 Thread Gerd Hoffmann
Hi, > The other bits of mention are all built normally as part of QEMU and > not subject to these problems, Sure? As far I know no firmware is build automatically as cross compilers might be needed to do that for guest arch != host arch. But yes, we don't have licensing issues (we ship the so

Re: [Qemu-devel] [PATCH v2 0/6] migration: let pre_save fail

2017-09-27 Thread Dr. David Alan Gilbert
* Dr. David Alan Gilbert (git) (dgilb...@redhat.com) wrote: > From: "Dr. David Alan Gilbert" > > At the moment the pre-save method on VMStateDescription returns void > which means there's no sane way of a pre_save routine failing. There > are quite a few pre_save's that abort or do other nasty t

Re: [Qemu-devel] [RFC v4 10/16] virtio-iommu: Implement probe request

2017-09-27 Thread Tomasz Nowicki
Hi Eric, On 19.09.2017 09:46, Eric Auger wrote: This patch implements the PROBE request. At the moment, no reserved regions are returned. At the moment reserved regions are stored per device. Signed-off-by: Eric Auger --- [...] + +static int virtio_iommu_fill_property(int devid, int typ

Re: [Qemu-devel] [PATCH 1/1] s390x: create a compat s390 phb for <=2.10

2017-09-27 Thread Cornelia Huck
On Wed, 27 Sep 2017 18:25:00 +0800 Yi Min Zhao wrote: > 在 2017/9/27 下午5:47, Cornelia Huck 写道: > > On Tue, 26 Sep 2017 20:40:25 +0200 > > David Hildenbrand wrote: > >> I'd really really really (did I mention really?) favor something like a > >> dummy device, because we could easily handle the !C

Re: [Qemu-devel] [PATCH 1/1] s390x: create a compat s390 phb for <=2.10

2017-09-27 Thread Christian Borntraeger
On 09/27/2017 12:56 PM, Cornelia Huck wrote: > On Wed, 27 Sep 2017 18:25:00 +0800 > Yi Min Zhao wrote: > >> 在 2017/9/27 下午5:47, Cornelia Huck 写道: >>> On Tue, 26 Sep 2017 20:40:25 +0200 >>> David Hildenbrand wrote: > I'd really really really (did I mention really?) favor something like a

Re: [Qemu-devel] [RFC v4 10/16] virtio-iommu: Implement probe request

2017-09-27 Thread Bharat Bhushan
> -Original Message- > From: Tomasz Nowicki [mailto:tnowi...@caviumnetworks.com] > Sent: Wednesday, September 27, 2017 4:23 PM > To: Eric Auger ; eric.auger@gmail.com; > peter.mayd...@linaro.org; alex.william...@redhat.com; m...@redhat.com; > qemu-...@nongnu.org; qemu-devel@nongnu.org

Re: [Qemu-devel] libvirt/QEMU/SEV interaction

2017-09-27 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > On Fri, Sep 08, 2017 at 06:57:30AM -0500, Brijesh Singh wrote: > > Hi All, > > Sorry if below comment doesn't make sense, I might be misunderstanding > something basic about SEV. Also sorry about the delay, I've been on > vacation. > > > > (sorry f

Re: [Qemu-devel] [RFC v4 00/16] VIRTIO-IOMMU device

2017-09-27 Thread Tomasz Nowicki
Hi Eric, Out of curiosity, I compared your SMMUv3 full emulation against virtio-iommu. For virtio-net device behind the virtio-iommu I get 50% performance of what I can get for SMMUv3 emulation. Do you have similar observations? Since there is no need to emulate HW behaviour in QEMU I expect

Re: [Qemu-devel] [RFC 6/6] device-crash-test: Multi-device device_add test

2017-09-27 Thread Thomas Huth
On 27.09.2017 01:07, Eduardo Habkost wrote: > When running device_add tests, test all devices in a single run > instead of restarting QEMU every time. > > There's a plug_all testcase argument that can be used to make the > test code plug all devices at once. I'm adding this mode because > there's

Re: [Qemu-devel] [PATCH 0/1] qemu-firmware repo

2017-09-27 Thread Paolo Bonzini
On 27/09/2017 11:15, Daniel P. Berrange wrote: > On Wed, Sep 27, 2017 at 09:19:22AM +0200, Paolo Bonzini wrote: >> Are you planning to include only submodules, or also "QEMU-native" >> firmware such as linuxboot, kvmvapic, s390-ccw, spapr-rtas, etc.? > > The submodules make sense to split out beca

Re: [Qemu-devel] ppce500 usb-ehci-pci issue on last uboot

2017-09-27 Thread Gerd Hoffmann
Hi, > "actually, I tried it yesterday and it seems to be a bug in qemu in > case you reset the EHCI controller, it doesn't come out of reset for > whatever reason ... so you probably want to talk to the qemu guys, > it's when U-Boot sets USBCMD RESET bit and then does handshake, the > controller

[Qemu-devel] Something wrong in usb_try_create_simple()

2017-09-27 Thread Thomas Huth
Hi Gerd, today, I've run into this issue reported by valgrind: $ valgrind x86_64-softmmu/qemu-system-x86_64 -device usb-ehci -nographic -S ==2043== Memcheck, a memory error detector ==2043== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al. ==2043== Using Valgrind-3.12.0 and LibVEX

[Qemu-devel] [Bug 1719282] Re: Unable to boot after drive-mirror

2017-09-27 Thread Farid Zarazvand
In last try, vm shutdown before completing blockjob. So i tried again and these are the exact qmp commands which i used: Sequence of qmp commands: socat UNIX-CONNECT:/var/run/qemu-server/48016.qmp STDIO {"QMP": {"version": {"qemu": {"micro": 0, "minor": 7, "major": 2}, "package": "pve-qemu-kvm_2

Re: [Qemu-devel] [PATCH v1 0/6] s390x/tcg: fix some accesses using low address

2017-09-27 Thread Cornelia Huck
On Tue, 26 Sep 2017 20:33:12 +0200 David Hildenbrand wrote: > Conny reported that a guest Linux currently spits out: > "The s390-virtio transport is deprecated. Please switch to a modern host > providing virtio-ccw." > > This is strange, because we have no legacy virtio transport in QEMU :)

[Qemu-devel] [RFC] ppc: define spapr core types statically

2017-09-27 Thread Igor Mammedov
-- patch does 3 things at the same time and should be split but it has 'host' consolidation is it as well to demonstrate idea -- spapr core type definition doesn't have any fields that require it to be defined at runtime. So replace code that fills in TypeInfo at runtime with static TypeInfo array

[Qemu-devel] [PATCH] egl: misc framebuffer helper improvements.

2017-09-27 Thread Gerd Hoffmann
Rename the functions to to say "setup" instead of "create" because they support being called multiple times on the same egl framebuffer. Properly delete unused textures, update function interfaces to support this. Signed-off-by: Gerd Hoffmann --- include/ui/egl-helpers.h | 5 +++-- ui/egl-head

Re: [Qemu-devel] [PATCH v3 2/5] ppc: spapr: Handle "ibm, nmi-register" and "ibm, nmi-interlock" RTAS calls

2017-09-27 Thread Aravinda Prasad
On Wednesday 27 September 2017 12:45 PM, David Gibson wrote: > On Thu, Sep 21, 2017 at 02:39:06PM +0530, Aravinda Prasad wrote: >> >> >> On Tuesday 22 August 2017 07:38 AM, David Gibson wrote: >> >> [ . . . ] >> >> diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h >> index 4601

[Qemu-devel] KVM call for agenda for 2017-10-10

2017-09-27 Thread Juan Quintela
Hi Please, send any topic that you are interested in covering. At the end of Monday I will send an email with the agenda or the cancellation of the call, so hurry up. After discussions on the QEMU Summit, we are going to have always open a KVM call where you can add topics. Call details: By

[Qemu-devel] [PATCH v8 8/8] tpm: Added support for TPM emulator

2017-09-27 Thread Amarnath Valluri
This change introduces a new TPM backend driver that can communicate with swtpm(software TPM emulator) using unix domain socket interface. QEMU talks to TPM emulator using QEMU's socket-based chardev backend device. Swtpm uses two Unix sockets for communications, one for plain TPM commands and res

  1   2   3   >