Re: [Qemu-devel] [PATCH] tests/prom-env: Bump the timeout, and test pseries only in slow mode

2017-09-20 Thread Stefan Weil
Am 21.09.2017 um 07:39 schrieb Thomas Huth: > If QEMU has been compiled with the flags --enable-tcg-interpreter and > --enable-debug, the guest is running incredibly slow. The prom-env > test is approximately 10 times slower than normal in this case, and > it takes up to 500 seconds until the test

[Qemu-devel] [PATCH] tests/prom-env: Bump the timeout, and test pseries only in slow mode

2017-09-20 Thread Thomas Huth
If QEMU has been compiled with the flags --enable-tcg-interpreter and --enable-debug, the guest is running incredibly slow. The prom-env test is approximately 10 times slower than normal in this case, and it takes up to 500 seconds until the test with the pseries machine finishs. While we should

Re: [Qemu-devel] [PATCH qemu v4 09/18] memory: Store physical root MR in FlatView

2017-09-20 Thread Alexey Kardashevskiy
On 21/09/17 10:02, Alexey Kardashevskiy wrote: > On 21/09/17 03:15, Paolo Bonzini wrote: >> On 20/09/2017 13:46, Alexey Kardashevskiy wrote: >>> Address spaces get to keep a root MR (alias or not) but FlatView stores >>> the actual MR as this is going to be used later on to decide whether to >>>

Re: [Qemu-devel] [PULL 00/16] Migration PULL request

2017-09-20 Thread Peter Xu
On Thu, Sep 21, 2017 at 12:10:39PM +0800, Peter Xu wrote: > On Wed, Sep 20, 2017 at 08:32:46PM +0100, Peter Maydell wrote: > > On 20 September 2017 at 12:42, Juan Quintela wrote: > > > Juan Quintela wrote: > > >> Hi > > >> > > >> To make merges easier,

Re: [Qemu-devel] [RFC PATCH] NUMA: Enable adding NUMA node implicitly

2017-09-20 Thread Dou Liyang
Hi Igor, I am sorry I missed some comments you gave to me. my reply is below. At 09/18/2017 05:24 PM, Dou Liyang wrote: [...] ranges where *the guest will attempt to probe for a device that QEMU doesn't *implement and a stub device is required. + * @numa_implicit_add_node0: + *

Re: [Qemu-devel] [PULL 00/16] Migration PULL request

2017-09-20 Thread Peter Xu
On Wed, Sep 20, 2017 at 08:32:46PM +0100, Peter Maydell wrote: > On 20 September 2017 at 12:42, Juan Quintela wrote: > > Juan Quintela wrote: > >> Hi > >> > >> To make merges easier, this includes: > >> - Peter Xu reviewed patches from Postocpy recovery

Re: [Qemu-devel] [PATCH] ppc/pnv: fix cores per chip for multiple cpus

2017-09-20 Thread Nikunj A Dadhania
David Gibson writes: > On Wed, Sep 20, 2017 at 12:48:55PM +0530, Nikunj A Dadhania wrote: >> David Gibson writes: >> >> > On Wed, Sep 20, 2017 at 12:10:48PM +0530, Nikunj A Dadhania wrote: >> >> David Gibson

[Qemu-devel] [Bug 1673976] Re: core dump

2017-09-20 Thread Thomas Huth
Could you please check whether the problem also occurs with QEMU v2.10? ** Changed in: qemu Status: New => Incomplete -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1673976 Title: core dump

Re: [Qemu-devel] [PATCH v3] spapr_pci: make index property mandatory

2017-09-20 Thread David Gibson
On Wed, Sep 20, 2017 at 04:46:20PM +0200, Greg Kurz wrote: > 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

Re: [Qemu-devel] [RFC 01/15] char-io: fix possible race on IOWatchPoll

2017-09-20 Thread Peter Xu
On Wed, Sep 20, 2017 at 12:29:21PM +0100, Daniel P. Berrange wrote: > On Wed, Sep 20, 2017 at 07:18:49PM +0800, Peter Xu wrote: > > On Wed, Sep 20, 2017 at 12:03:09PM +0100, Daniel P. Berrange wrote: > > > On Wed, Sep 20, 2017 at 06:49:58PM +0800, Peter Xu wrote: > > > > On Wed, Sep 20, 2017 at

Re: [Qemu-devel] [PATCH V2] add migration capability to bypass the shared memory

2017-09-20 Thread Zhang Haoyu
Hi Jiangshan, Any update from this patch? Thanks, Zhang Haoyu On 2016/8/11 22:45, Lai Jiangshan wrote: > Note, the old local migration patchset: > https://lists.gnu.org/archive/html/qemu-devel/2013-12/msg00073.html > > this patch can be considered as a new local migration implementation, > but

Re: [Qemu-devel] block ais migration for machines <= 2.9

2017-09-20 Thread Yi Min Zhao
在 2017/9/21 上午12:04, Dr. David Alan Gilbert 写道: * Christian Borntraeger (borntrae...@de.ibm.com) wrote: Something like the following seems to do the tricks. Needs proper patch description, review, full test with different kernel versions. Without knowing anything about 'ais' - will this

[Qemu-devel] [PULL 2/2] xen/pt: allow QEMU to request MSI unmasking at bind time

2017-09-20 Thread Stefano Stabellini
From: Roger Pau Monne When a MSI interrupt is bound to a guest using xc_domain_update_msi_irq (XEN_DOMCTL_bind_pt_irq) the interrupt is left masked by default. This causes problems with guests that first configure interrupts and clean the per-entry MSIX table mask bit and

[Qemu-devel] [PULL 1/2] xen-disk: use g_new0 to fix build

2017-09-20 Thread Stefano Stabellini
From: Olaf Hering g_malloc0_n is available since glib-2.24. To allow build with older glib versions use the generic g_new0, which is already used in many other places in the code. Fixes commit 3284fad728 ("xen-disk: add support for multi-page shared rings") Signed-off-by: Olaf

[Qemu-devel] [PULL 0/2] xen-20170920-tag

2017-09-20 Thread Stefano Stabellini
-20170920-tag for you to fetch changes up to a8036336609d2e184fc3543a4c439c0ba7d7f3a2: xen/pt: allow QEMU to request MSI unmasking at bind time (2017-09-20 19:05:27 -0700) Xen 2017/09/20

Re: [Qemu-devel] [PATCH] virtio/vhost: reset dev->log after syncing

2017-09-20 Thread Jason Wang
On 2017年09月21日 02:53, Felipe Franciosi wrote: vhost_log_put() is called to decomission the dirty log between qemu and a vhost device when stopping the device. Such a call can happen from migration_completion(). Present code sets dev->log_size to zero too early in vhost_log_put(), causing the

[Qemu-devel] [PATCH v2] ide: fix enum comparison for gcc 4.7

2017-09-20 Thread John Snow
Apparently GCC gets bent over comparing enum values against zero. Replace the conditional with something less readable. Tested-by: Mark Cave-Ayland Signed-off-by: John Snow --- v2: Second verse, same as the first. Signed-off-by: John Snow

Re: [Qemu-devel] [RFC PATCH v2 18/21] ppc/xive: add device tree support

2017-09-20 Thread David Gibson
On Wed, Sep 20, 2017 at 02:26:32PM +0200, Cédric Le Goater wrote: > On 09/19/2017 10:44 AM, David Gibson wrote: > > On Mon, Sep 11, 2017 at 07:12:32PM +0200, Cédric Le Goater wrote: > >> Like for XICS, the XIVE interface for the guest is described in the > >> device tree under the

Re: [Qemu-devel] [RFC PATCH v2 00/21] Guest exploitation of the XIVE interrupt controller (POWER9)

2017-09-20 Thread David Gibson
On Wed, Sep 20, 2017 at 02:33:37PM +0200, Cédric Le Goater wrote: > On 09/19/2017 10:46 AM, David Gibson wrote: > > On Tue, Sep 19, 2017 at 06:20:20PM +1000, David Gibson wrote: > >> On Mon, Sep 11, 2017 at 07:12:14PM +0200, Cédric Le Goater wrote: > >>> On a POWER9 sPAPR machine, the Client

[Qemu-devel] [PATCH 1/2] ide: generic ide_data_read

2017-09-20 Thread John Snow
Signed-off-by: John Snow --- hw/ide/core.c | 99 +-- hw/ide/trace-events | 4 +- include/hw/ide/internal.h | 3 +- 3 files changed, 49 insertions(+), 57 deletions(-) diff --git a/hw/ide/core.c b/hw/ide/core.c

[Qemu-devel] [PATCH 0/2] IDE: combine portio r/w functions

2017-09-20 Thread John Snow
Mark, here's a quick sketch for you. There are two things I don't like, but didn't care enough to fix: (1) Restricting nbytes to 2 or 4 means some extra boilerplate to quiet compilers who don't know it will only ever be 2 or 4 (2) the address value is all-but-ignored, it carries over from the

[Qemu-devel] [PATCH 2/2] ide: generic ide_data_write

2017-09-20 Thread John Snow
Signed-off-by: John Snow --- hw/ide/core.c | 86 +-- hw/ide/trace-events | 3 +- include/hw/ide/internal.h | 1 + 3 files changed, 40 insertions(+), 50 deletions(-) diff --git a/hw/ide/core.c b/hw/ide/core.c index

Re: [Qemu-devel] [PATCH] xen: use vMSI related #define-s from public interface

2017-09-20 Thread Stefano Stabellini
On Fri, 1 Sep 2017, Jan Beulich wrote: > Xen and qemu having identical #define-s (with different names) is a > strong hint that these should have been part of the public interface > from the very start. Use them if they're available, falling back to > privately defined values only when using older

Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA

2017-09-20 Thread Dong Jia Shi
* Halil Pasic [2017-09-20 13:13:01 +0200]: > > > On 09/20/2017 10:33 AM, Cornelia Huck wrote: > > On Wed, 20 Sep 2017 15:42:38 +0800 > > Dong Jia Shi wrote: > > > >> * Halil Pasic [2017-09-19 20:27:45 +0200]: >

Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA

2017-09-20 Thread Dong Jia Shi
* Halil Pasic [2017-09-20 18:46:57 +0200]: > > > On 09/20/2017 01:18 PM, Cornelia Huck wrote: > > On Wed, 20 Sep 2017 13:13:01 +0200 > > Halil Pasic wrote: > > > >> On 09/20/2017 10:33 AM, Cornelia Huck wrote: > >>> On Wed, 20 Sep 2017

Re: [Qemu-devel] [PATCH v10 03/13] scripts: Add archive-source.sh

2017-09-20 Thread Fam Zheng
On Wed, 09/20 08:20, Eric Blake wrote: > On 09/19/2017 10:25 PM, Fam Zheng wrote: > > Signed-off-by: Fam Zheng > > --- > > scripts/archive-source.sh | 51 > > +++ > > 1 file changed, 51 insertions(+) > > create mode 100755

Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Dong Jia Shi
* Halil Pasic [2017-09-20 13:02:59 +0200]: > > > On 09/20/2017 10:25 AM, Cornelia Huck wrote: > > On Wed, 20 Sep 2017 15:47:51 +0800 > > Dong Jia Shi wrote: > > > >> * Halil Pasic [2017-09-19 20:27:44 +0200]: >

Re: [Qemu-devel] [PATCH v10 03/13] scripts: Add archive-source.sh

2017-09-20 Thread Fam Zheng
On Wed, 09/20 08:20, Eric Blake wrote: > On 09/19/2017 10:25 PM, Fam Zheng wrote: > > Signed-off-by: Fam Zheng > > --- > > scripts/archive-source.sh | 51 > > +++ > > 1 file changed, 51 insertions(+) > > create mode 100755

Re: [Qemu-devel] [PATCH qemu v4 09/18] memory: Store physical root MR in FlatView

2017-09-20 Thread Alexey Kardashevskiy
On 21/09/17 03:15, Paolo Bonzini wrote: > On 20/09/2017 13:46, Alexey Kardashevskiy wrote: >> Address spaces get to keep a root MR (alias or not) but FlatView stores >> the actual MR as this is going to be used later on to decide whether to >> share a particular FlatView or not. >> >>

Re: [Qemu-devel] [PATCH qemu v4 18/18] memory: Give memory_region_transaction_commit a hint

2017-09-20 Thread Alexey Kardashevskiy
On 21/09/17 03:14, Paolo Bonzini wrote: > On 20/09/2017 13:46, Alexey Kardashevskiy wrote: >> This extends memory_region_transaction_commit() to receive a MR as >> if it is a root MR or its topmost parent is, then we can only rebuild >> its FlatView and update it for address spaces sharing it. >>

Re: [Qemu-devel] [PATCH qemu v4 12/18] memory: Share FlatView's and dispatch trees between address spaces

2017-09-20 Thread Alexey Kardashevskiy
On 21/09/17 03:18, Paolo Bonzini wrote: > On 20/09/2017 13:46, Alexey Kardashevskiy wrote: >> +QTAILQ_FOREACH(as, _spaces, address_spaces_link) { >> +MemoryRegion *physmr = memory_region_get_flatview_root(as->root); >> +FlatView *new_view = g_hash_table_lookup(flat_views,

Re: [Qemu-devel] [PATCH qemu v4 15/18] memory: Share special empty FlatView

2017-09-20 Thread Alexey Kardashevskiy
On 21/09/17 03:13, Paolo Bonzini wrote: > On 20/09/2017 13:46, Alexey Kardashevskiy wrote: >> This shares an cached empty FlatView among address spaces. The empty >> FV is used every time when a root MR renders into a FV without memory >> sections which happens when MR or its children are not

Re: [Qemu-devel] [PATCH] virtio/vhost: reset dev->log after syncing

2017-09-20 Thread Felipe Franciosi
Heya, > On 20 Sep 2017, at 13:33, Marc-André Lureau > wrote: > > Hi > > - Original Message - >> vhost_log_put() is called to decomission the dirty log between qemu and >> a vhost device when stopping the device. Such a call can happen from >>

Re: [Qemu-devel] [PATCH 2/3] linux-user: add SO_LINGER to setsockopt

2017-09-20 Thread Carlo Arenas
On Wed, Sep 20, 2017 at 11:53 AM, Laurent Vivier wrote: > > the test for optlen is replaced by passing optlen to the underlying > > setsockopt call directly, who would do the test and return the right > error. > > You can't do that, because sizeof(struct linger) may be

Re: [Qemu-devel] [PATCH 00/12] Patch Round-up for stable 2.10.1, freeze on 2017-09-27

2017-09-20 Thread Greg Kurz
On Tue, 19 Sep 2017 19:45:09 -0500 Michael Roth wrote: > Hi everyone, > > The following new patches are queued for QEMU stable v2.10.1: > > https://github.com/mdroth/qemu/commits/stable-2.10-staging > > The release is planned for 2017-10-02: > >

Re: [Qemu-devel] qemu-arm SIGSEGV for self-modifying code

2017-09-20 Thread John Reiser
I don't really know why we use 0xf700 as our reserved_va value here, though. Alex, you added that years ago, can you remember why you used that value? IIRC I wanted to map the full 32 bits of address space possibly in use by a 32bit application, but leave some room for something, but I

[Qemu-devel] [PATCH v3 6/8] xlnx-zynqmp-ipi: Initial version of the Xilinx IPI device

2017-09-20 Thread Alistair Francis
This is the initial version of the Inter Processor Interrupt device. Signed-off-by: Alistair Francis --- hw/intc/Makefile.objs | 1 + hw/intc/xlnx-zynqmp-ipi.c | 377 ++ include/hw/intc/xlnx-zynqmp-ipi.h |

[Qemu-devel] [PATCH v3 8/8] xlnx-zynqmp: Connect the IPI device to the ZynqMP SoC

2017-09-20 Thread Alistair Francis
Signed-off-by: Alistair Francis --- hw/arm/xlnx-zynqmp.c | 14 ++ include/hw/arm/xlnx-zynqmp.h | 2 ++ 2 files changed, 16 insertions(+) diff --git a/hw/arm/xlnx-zynqmp.c b/hw/arm/xlnx-zynqmp.c index 2b27daf51d..8aa1f02c62 100644 ---

[Qemu-devel] [PATCH v3 4/8] xlnx-pmu-iomod-intc: Add the PMU Interrupt controller

2017-09-20 Thread Alistair Francis
Add the PMU IO Module Interrupt controller device. Signed-off-by: Alistair Francis --- default-configs/microblaze-softmmu.mak | 1 + hw/intc/Makefile.objs | 1 + hw/intc/xlnx-pmu-iomod-intc.c | 554 +

[Qemu-devel] [PATCH v3 2/8] xlnx-zynqmp-pmu: Add the CPU and memory

2017-09-20 Thread Alistair Francis
Connect the MicroBlaze CPU and the ROM and RAM memory regions. Signed-off-by: Alistair Francis --- V2: - Fix the pmu-cpu name - Use err and errp for CPU realise instead of error_fatal hw/microblaze/xlnx-zynqmp-pmu.c | 70 +++--

[Qemu-devel] [PATCH v3 7/8] xlnx-zynqmp-pmu: Connect the IPI device to the PMU

2017-09-20 Thread Alistair Francis
Signed-off-by: Alistair Francis --- hw/microblaze/xlnx-zynqmp-pmu.c | 40 ++-- 1 file changed, 34 insertions(+), 6 deletions(-) diff --git a/hw/microblaze/xlnx-zynqmp-pmu.c b/hw/microblaze/xlnx-zynqmp-pmu.c index

[Qemu-devel] [PATCH v3 3/8] aarch64-softmmu.mak: Use an ARM specific config

2017-09-20 Thread Alistair Francis
In preperation for having an ARM and MicroBlaze ZynqMP machine let's split out the current ARM specific config options. Signed-off-by: Alistair Francis Acked-by: Peter Maydell --- default-configs/aarch64-softmmu.mak | 1 +

[Qemu-devel] [PATCH v3 0/8] Add the ZynqMP PMU and IPI

2017-09-20 Thread Alistair Francis
This series adds the ZynqMP Power Management Unit (PMU) machine with basic functionality. The machine only has the - CPU - Memory - Interrupt controller - IPI device connected, but that is enough to run some of the ROM and firmware code on the machine The series also adds the IPI device

[Qemu-devel] [PATCH v3 1/8] xlnx-zynqmp-pmu: Initial commit of the ZynqMP PMU

2017-09-20 Thread Alistair Francis
The Xilinx ZynqMP SoC has two main processing systems in it. The ARM processing system (which is already modeled in QEMU) and the MicroBlaze Power Management Unit (PMU). This is the inital work for adding support for the PMU. The PMU susbsystem runs along side the ARM system on hardware, but due

Re: [Qemu-devel] [PATCH] ide: fix enum comparison for gcc 4.7

2017-09-20 Thread Mark Cave-Ayland
On 20/09/17 22:33, John Snow wrote: > On 09/20/2017 05:28 PM, Mark Cave-Ayland wrote: >> On 20/09/17 20:41, John Snow wrote: >> >>> Apparently GCC gets bent over comparing enum values against zero. >>> Replace the conditional with something less readable. >>> >>> Signed-off-by: John Snow

Re: [Qemu-devel] [Qemu-block] [PATCH 2/2] qcow2: truncate the tail of the image file after shrinking the image

2017-09-20 Thread John Snow
On 09/20/2017 09:58 AM, Pavel Butsykin wrote: > Now after shrinking the image, at the end of the image file, there might be a > tail that probably will never be used. So we can find the last used cluster > and > cut the tail. > > Signed-off-by: Pavel Butsykin > --- >

Re: [Qemu-devel] [PATCH] ide: fix enum comparison for gcc 4.7

2017-09-20 Thread John Snow
On 09/20/2017 05:28 PM, Mark Cave-Ayland wrote: > On 20/09/17 20:41, John Snow wrote: > >> Apparently GCC gets bent over comparing enum values against zero. >> Replace the conditional with something less readable. >> >> Signed-off-by: John Snow >> --- >> hw/ide/core.c

Re: [Qemu-devel] qemu-arm SIGSEGV for self-modifying code

2017-09-20 Thread Alexander Graf
On 20.09.17 20:04, Peter Maydell wrote: On 20 September 2017 at 18:05, John Reiser wrote: Yes, the SEGV occurs on the store, "long" before the re-written instruction ever is executed OK, I've identified the immediate cause for this SEGV. (1) when the guest initially

Re: [Qemu-devel] [PATCH] ide: fix enum comparison for gcc 4.7

2017-09-20 Thread Mark Cave-Ayland
On 20/09/17 20:41, John Snow wrote: > Apparently GCC gets bent over comparing enum values against zero. > Replace the conditional with something less readable. > > Signed-off-by: John Snow > --- > hw/ide/core.c | 2 +- > include/hw/ide/internal.h | 3 +-- > 2

[Qemu-devel] [PATCH] hw/pci-bridge/pcie_pci_bridge: properly handle MSI unavailability case

2017-09-20 Thread Aleksandr Bezzubikov
Signed-off-by: Aleksandr Bezzubikov --- hw/pci-bridge/pcie_pci_bridge.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/hw/pci-bridge/pcie_pci_bridge.c b/hw/pci-bridge/pcie_pci_bridge.c index 9aa5cc3..da562fe 100644 ---

Re: [Qemu-devel] [PATCH v2 4/8] disas: Support the Capstone disassembler library

2017-09-20 Thread Richard Henderson
On 09/20/2017 06:17 AM, Alex Bennée wrote: > Hmm I get a link failure: > > LINKtilegx-linux-user/qemu-tilegx > disas.o: In function `cap_disas_start': > /home/alex/lsrc/qemu/qemu.git/disas.c:196: undefined reference to `cs_open' As discussed on IRC, this turned out to be wrong link

Re: [Qemu-devel] [PATCH 6/6] hw/arm/omap2.c: Don't use old_mmio

2017-09-20 Thread Richard Henderson
On 09/16/2017 11:46 AM, Peter Maydell wrote: > Don't use old_mmio in the memory region ops struct. > > Signed-off-by: Peter Maydell > --- > hw/arm/omap2.c | 49 + > 1 file changed, 37 insertions(+), 12 deletions(-)

Re: [Qemu-devel] [PULL 0/3] 9pfs fixes for 2.11 20170920

2017-09-20 Thread Peter Maydell
On 20 September 2017 at 14:50, Greg Kurz wrote: > The following changes since commit c51700273ad9802a21c19f8d2b4bcb67c38e74ac: > > Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20170919-v2' into > staging (2017-09-19 18:08:48 +0100) > > are available in the git

Re: [Qemu-devel] [PATCH 5/6] hw/i2c/omap_i2c.c: Don't use old_mmio

2017-09-20 Thread Richard Henderson
On 09/16/2017 11:46 AM, Peter Maydell wrote: > Don't use old_mmio in the memory region ops struct. > > Signed-off-by: Peter Maydell > --- > hw/i2c/omap_i2c.c | 44 > 1 file changed, 32 insertions(+), 12 deletions(-)

Re: [Qemu-devel] [PATCH 4/6] hw/timer/omap_gptimer: Don't use old_mmio

2017-09-20 Thread Richard Henderson
On 09/16/2017 11:46 AM, Peter Maydell wrote: > Don't use the old_mmio struct in memory region ops. > > Signed-off-by: Peter Maydell > --- > hw/timer/omap_gptimer.c | 49 > + > 1 file changed, 37 insertions(+), 12

Re: [Qemu-devel] [PATCH 3/6] hw/timer/omap_synctimer.c: Don't use old_mmio

2017-09-20 Thread Richard Henderson
On 09/16/2017 11:46 AM, Peter Maydell wrote: > Don't use the old_mmio in the memory region ops struct. > > Signed-off-by: Peter Maydell > --- > hw/timer/omap_synctimer.c | 35 +-- > 1 file changed, 21 insertions(+), 14 deletions(-)

Re: [Qemu-devel] [PATCH 2/6] hw/gpio/omap_gpio.c: Don't use old_mmio

2017-09-20 Thread Richard Henderson
On 09/16/2017 11:46 AM, Peter Maydell wrote: > Drop the use of old_mmio in the omap2_gpio memory ops. > > Signed-off-by: Peter Maydell > --- > hw/gpio/omap_gpio.c | 26 -- > 1 file changed, 12 insertions(+), 14 deletions(-) Reviewed-by: Richard

Re: [Qemu-devel] [PATCH 1/6] hw/arm/palm.c: Don't use old_mmio for static_ops

2017-09-20 Thread Richard Henderson
On 09/16/2017 11:46 AM, Peter Maydell wrote: > Update the static_ops functions to use new-style mmio > rather than the legacy old_mmio functions. > > Signed-off-by: Peter Maydell > --- > hw/arm/palm.c | 30 ++ > 1 file changed, 10

[Qemu-devel] [PATCH 1/2] slirp: Fix intermittent send queue hangs on a socket

2017-09-20 Thread Kevin Cernekee
if_output() originally sent one mbuf per call and used the slirp->next_m variable to keep track of where it left off. But nowadays it tries to send all of the mbufs from the fastq, and one mbuf from each session on the batchq. The next_m variable is both redundant and harmful: there is a case[0]

[Qemu-devel] [PATCH 2/2] slirp: Add a special case for the NULL socket

2017-09-20 Thread Kevin Cernekee
NULL sockets are used for NDP, BOOTP, and other critical operations. If the topmost mbuf in a NULL session is blocked pending resolution, it may cause problems if it blocks other packets with a NULL socket. So do not add mbufs with a NULL socket field to the same session. Signed-off-by: Kevin

Re: [Qemu-devel] [PATCH] virtio/vhost: reset dev->log after syncing

2017-09-20 Thread Marc-André Lureau
Hi - Original Message - > vhost_log_put() is called to decomission the dirty log between qemu and > a vhost device when stopping the device. Such a call can happen from > migration_completion(). > > Present code sets dev->log_size to zero too early in vhost_log_put(), > causing the sync

[Qemu-devel] [Bug 1673976] Re: core dump

2017-09-20 Thread Matheus Izvekov
I can confirm this. The ninja build system is also affected. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1673976 Title: core dump Status in QEMU: New Bug description: I'm running a command

[Qemu-devel] [PATCH v11 4/5] msf2: Add Smartfusion2 SoC

2017-09-20 Thread Philippe Mathieu-Daudé
From: Subbaraya Sundeep Smartfusion2 SoC has hardened Microcontroller subsystem and flash based FPGA fabric. This patch adds support for Microcontroller subsystem in the SoC. Signed-off-by: Subbaraya Sundeep Reviewed-by: Alistair Francis

[Qemu-devel] [PATCH v11 5/5] msf2: Add Emcraft's Smartfusion2 SOM kit

2017-09-20 Thread Philippe Mathieu-Daudé
From: Subbaraya Sundeep Emulated Emcraft's Smartfusion2 System On Module starter kit. Signed-off-by: Subbaraya Sundeep Signed-off-by: Philippe Mathieu-Daudé [PMD: drop cpu_model to directly use cpu type] --- hw/arm/msf2-som.c

[Qemu-devel] [PATCH v11 1/5] msf2: Add Smartfusion2 System timer

2017-09-20 Thread Philippe Mathieu-Daudé
From: Subbaraya Sundeep Modelled System Timer in Microsemi's Smartfusion2 Soc. Timer has two 32bit down counters and two interrupts. Signed-off-by: Subbaraya Sundeep Reviewed-by: Alistair Francis Acked-by: Philippe

[Qemu-devel] [PATCH v11 3/5] msf2: Add Smartfusion2 SPI controller

2017-09-20 Thread Philippe Mathieu-Daudé
From: Subbaraya Sundeep Modelled Microsemi's Smartfusion2 SPI controller. Signed-off-by: Subbaraya Sundeep Reviewed-by: Alistair Francis Tested-by: Philippe Mathieu-Daudé ---

[Qemu-devel] [PATCH v11 2/5] msf2: Microsemi Smartfusion2 System Register block

2017-09-20 Thread Philippe Mathieu-Daudé
From: Subbaraya Sundeep Added Sytem register block of Smartfusion2. This block has PLL registers which are accessed by guest. Signed-off-by: Subbaraya Sundeep Reviewed-by: Alistair Francis Acked-by: Philippe

[Qemu-devel] [PATCH v11 0/5] Add support for Smartfusion2 SoC

2017-09-20 Thread Philippe Mathieu-Daudé
Hi Peter, Now than Igor's patch landed, I respin Sundeep's series updating it to work after the "arm: drop intermediate cpu_model -> cpu type parsing and use cpu type directly" patch. v11: - msf2-soc.c: add a check for null m3clk - msf2-soc.c, msf2-som.c: drop cpu_model to directly use cpu type

Re: [Qemu-devel] [Qemu-block] [PATCH 1/2] qcow2: fix return error code in qcow2_truncate()

2017-09-20 Thread John Snow
On 09/20/2017 09:58 AM, Pavel Butsykin wrote: > Signed-off-by: Pavel Butsykin > --- > block/qcow2.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/block/qcow2.c b/block/qcow2.c > index 2174a84d1f..8a4311d338 100644 > --- a/block/qcow2.c >

Re: [Qemu-devel] [PATCH v10 07/13] tests: Add FreeBSD image

2017-09-20 Thread Eric Blake
On 09/19/2017 10:25 PM, Fam Zheng wrote: > The image is prepared following instructions as in: > > https://wiki.qemu.org/Hosts/BSD > > Signed-off-by: Fam Zheng > --- > tests/vm/freebsd | 42 ++ > 1 file changed, 42 insertions(+) >

Re: [Qemu-devel] [PATCH 2/2] block/block-backend.c: remove blk_pread_unthrottled()

2017-09-20 Thread John Snow
On 09/20/2017 07:43 AM, Manos Pitsidianakis wrote: > blk_pread_unthrottled was used to bypass I/O throttling on the BlockBackend in > the case of async I/O. This is not needed anymore and we can just call > blk_pread() directly. > > Signed-off-by: Manos Pitsidianakis >

Re: [Qemu-devel] [PATCH v7 1/4] hw/pci: introduce pcie-pci-bridge device

2017-09-20 Thread Aleksandr Bezzubikov
2017-09-20 17:02 GMT+03:00 Marcel Apfelbaum : > On 20/09/2017 16:57, Eduardo Habkost wrote: >> >> On Wed, Sep 20, 2017 at 09:52:01AM +, Aleksandr Bezzubikov wrote: >>> >>> ср, 20 сент. 2017 г. в 10:13, Marcel Apfelbaum : >>> On 19/09/2017 23:34,

Re: [Qemu-devel] [PATCH v4 0/6] QOMify MIPS cpu

2017-09-20 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH v4 0/6] QOMify MIPS cpu Message-id: 20170920194934.23071-1-f4...@amsat.org Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline

[Qemu-devel] [PATCH v4 6/6] mips: replace cpu_mips_init() with cpu_generic_init()

2017-09-20 Thread Philippe Mathieu-Daudé
From: Igor Mammedov now cpu_mips_init() reimplements subset of cpu_generic_init() tasks, so just drop it and use cpu_generic_init() directly. Signed-off-by: Igor Mammedov Reviewed-by: Hervé Poussineau Signed-off-by: Philippe

[Qemu-devel] [PATCH v4 2/6] mips: introduce internal.h and cleanup cpu.h

2017-09-20 Thread Philippe Mathieu-Daudé
no logical change, only code movement (and fix a comment typo). Signed-off-by: Philippe Mathieu-Daudé Tested-by: Igor Mammedov Tested-by: James Hogan Acked-by: Eduardo Habkost --- This patch triggers 3 positive

[Qemu-devel] [PATCH v4 5/6] mips: MIPSCPU model subclasses

2017-09-20 Thread Philippe Mathieu-Daudé
From: Igor Mammedov Register separate QOM types for each mips cpu model, so it would be possible to reuse generic CPU creation routines. Signed-off-by: Igor Mammedov Signed-off-by: Philippe Mathieu-Daudé [PMD: use internal.h, use

[Qemu-devel] [PATCH v4 3/6] mips: split cpu_mips_realize_env() out of cpu_mips_init()

2017-09-20 Thread Philippe Mathieu-Daudé
so it can be used in mips_cpu_realizefn() in the next commit Signed-off-by: Philippe Mathieu-Daudé Tested-by: Igor Mammedov Tested-by: James Hogan Reviewed-by: Eduardo Habkost --- target/mips/internal.h | 1

[Qemu-devel] [PATCH v4 4/6] mips: call cpu_mips_realize_env() from mips_cpu_realizefn()

2017-09-20 Thread Philippe Mathieu-Daudé
This changes the order between cpu_mips_realize_env() and cpu_exec_initfn(), but cpu_exec_initfn() don't have anything that depends on cpu_mips_realize_env() being called first. Signed-off-by: Philippe Mathieu-Daudé Tested-by: Igor Mammedov Tested-by: James

Re: [Qemu-devel] [PATCH v9 05/20] dirty-bitmap: Avoid size query failure during truncate

2017-09-20 Thread John Snow
On 09/20/2017 09:11 AM, Eric Blake wrote: > On 09/19/2017 09:10 PM, Fam Zheng wrote: > >>> >>> Do you suspect that almost certainly if bdrv_truncate() fails overall >>> that the image format driver will either unmount the image or become >>> read-only? > > Uggh - it feels like I've bitten off

[Qemu-devel] [PATCH v4 0/6] QOMify MIPS cpu

2017-09-20 Thread Philippe Mathieu-Daudé
Hi, This series is based on Igor's "complete cpu QOMification" [1] but only modify the MIPS part. Igor posted an updated series [2]. Yongbok: this series is now ready to apply to your MIPS tree :) Regards, Phil. [1]: http://lists.nongnu.org/archive/html/qemu-devel/2017-07/msg04414.html [2]:

[Qemu-devel] [PATCH v4 1/6] mips: move hw/mips/cputimer.c to target/mips/

2017-09-20 Thread Philippe Mathieu-Daudé
This timer is a required part of the MIPS32/MIPS64 System Control coprocessor (CP0). Moving it with the other architecture related files will allow an opaque use of CPUMIPSState* in the next commit (introduce "internal.h"). also remove it from 'user' targets, remove an unnecessary include.

Re: [Qemu-devel] [PATCH] ide: fix enum comparison for gcc 4.7

2017-09-20 Thread Eric Blake
On 09/20/2017 02:41 PM, John Snow wrote: > Apparently GCC gets bent over comparing enum values against zero. > Replace the conditional with something less readable. > > Signed-off-by: John Snow > --- > hw/ide/core.c | 2 +- > include/hw/ide/internal.h | 3 +-- > 2

[Qemu-devel] [PATCH] ide: fix enum comparison for gcc 4.7

2017-09-20 Thread John Snow
Apparently GCC gets bent over comparing enum values against zero. Replace the conditional with something less readable. Signed-off-by: John Snow --- hw/ide/core.c | 2 +- include/hw/ide/internal.h | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff

Re: [Qemu-devel] [PULL 00/16] Migration PULL request

2017-09-20 Thread Peter Maydell
On 20 September 2017 at 12:42, Juan Quintela wrote: > Juan Quintela wrote: >> Hi >> >> To make merges easier, this includes: >> - Peter Xu reviewed patches from Postocpy recovery (3) >> - Alexey reviewed pages from block postcopy (4) > > I meaned here to

Re: [Qemu-devel] Block Migration and CPU throttling

2017-09-20 Thread Peter Lieven
Am 19.09.2017 um 16:41 schrieb Paolo Bonzini: > On 19/09/2017 15:36, Peter Lieven wrote: >> Hi, >> >> I just noticed that CPU throttling and Block Migration don't work >> together very well. >> During block migration the throttling heuristic detects that we >> obviously make no progress >> in ram

Re: [Qemu-devel] Block Migration and CPU throttling

2017-09-20 Thread Peter Lieven
Am 19.09.2017 um 16:41 schrieb Dr. David Alan Gilbert: > * Peter Lieven (p...@kamp.de) wrote: >> Am 19.09.2017 um 16:38 schrieb Dr. David Alan Gilbert: >>> * Peter Lieven (p...@kamp.de) wrote: Hi, I just noticed that CPU throttling and Block Migration don't work together very

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-20 Thread Mark Cave-Ayland
On 20/09/17 18:55, John Snow wrote: > Guh. From which distro does your GCC 4.7 hail? > > Regardless, I suppose I will revert to Eric's workaround, though I like > the way it reads an awful lot less. Thanks John - it's just a standard Debian Wheezy installation on amd64. ATB, Mark.

Re: [Qemu-devel] [PATCH 2/3] linux-user: add SO_LINGER to setsockopt

2017-09-20 Thread Laurent Vivier
Le 20/09/2017 à 19:29, Carlo Arenas a écrit : > On Wed, Sep 20, 2017 at 1:39 AM, Laurent Vivier > wrote: > > Why did you remove "optname = SO_LINGER" and "if (optlen != > sizeof(struct target_linger))"? > > > the optname assignment is not

[Qemu-devel] [PATCH] virtio/vhost: reset dev->log after syncing

2017-09-20 Thread Felipe Franciosi
vhost_log_put() is called to decomission the dirty log between qemu and a vhost device when stopping the device. Such a call can happen from migration_completion(). Present code sets dev->log_size to zero too early in vhost_log_put(), causing the sync check to always return false. As a

Re: [Qemu-devel] [PATCH 2/2] ppc: remove all unused CPU definitions

2017-09-20 Thread Thomas Huth
On 19.09.2017 23:36, John Snow wrote: > Remove *all* unused CPU definitions as indicated by compile-time > `#if 0` constructs. > > Signed-off-by: John Snow > --- > target/ppc/cpu-models.h | 223 > > 1 file changed, 223

[Qemu-devel] SunOS support

2017-09-20 Thread Peter Tribble
Hi, To introduce myself: I'm a member of the illumos community (the successor to OpenSolaris, to those unfamiliar with us), and I maintain my own illumos distribution. Having seen the scary 'SUPPORT FOR THIS HOST OS WILL GO AWAY' message, I'm reaching out to see what needs to be done so that

Re: [Qemu-devel] [PATCH 1/2] ppc: remove unused CPU definitions

2017-09-20 Thread Thomas Huth
On 19.09.2017 23:36, John Snow wrote: > Following commit aef77960, remove now-unused definitions from > cpu-models.h. > > Signed-off-by: John Snow > --- > target/ppc/cpu-models.h | 32 > 1 file changed, 32 deletions(-) > > diff --git

Re: [Qemu-devel] [PATCH v10 01/10] userfault: update kernel header for UFFD_FEATURE_*

2017-09-20 Thread Dr. David Alan Gilbert
* Alexey Perevalov (a.pereva...@samsung.com) wrote: > This commit adds modification for UFFD_FEATURE_SIGBUS and > UFFD_FEATURE_THREAD_ID. > > Signed-off-by: Alexey Perevalov This should be replaced with just running the scripts/update-linux-headers.sh against a

Re: [Qemu-devel] [PATCH 19/19] nvic: Support banked exceptions in acknowledge and complete

2017-09-20 Thread Richard Henderson
On 09/12/2017 01:14 PM, Peter Maydell wrote: > Update armv7m_nvic_acknowledge_irq() and armv7m_nvic_complete_irq() > to handle banked exceptions: > * acknowledge needs to use the correct vector, which may be >in sec_vectors[] > * acknowledge needs to return to its caller whether the >

Re: [Qemu-devel] [PATCH 18/19] nvic: Make SHCSR banked for v8M

2017-09-20 Thread Richard Henderson
On 09/12/2017 01:14 PM, Peter Maydell wrote: > Handle banking of SHCSR: some register bits are banked between > Secure and Non-Secure, and some are only accessible to Secure. > > Signed-off-by: Peter Maydell > --- > hw/intc/armv7m_nvic.c | 221 >

[Qemu-devel] [Bug 1715700] Re: Windows 7 guest won't boot on qemu 2.10 (works on 2.9)

2017-09-20 Thread Laszlo Ersek (Red Hat)
edk2 commit range: b68c793144e8..947f3737abf6. ** Changed in: qemu Status: New => Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1715700 Title: Windows 7 guest won't boot on

Re: [Qemu-devel] [PULL 00/12] Machine/CPU/NUMA queue, 2017-09-19

2017-09-20 Thread Peter Maydell
On 19 September 2017 at 21:18, Eduardo Habkost wrote: > The following changes since commit a9158a5cba955b79d580a252cc58ff44d154e370: > > Merge remote-tracking branch > 'remotes/kraxel/tags/audio-20170918-pull-request' into staging (2017-09-18 > 12:40:54 +0100) > > are

Re: [Qemu-devel] qemu-arm SIGSEGV for self-modifying code

2017-09-20 Thread Peter Maydell
On 20 September 2017 at 18:05, John Reiser wrote: > Yes, the SEGV occurs on the store, "long" before the re-written > instruction ever is executed OK, I've identified the immediate cause for this SEGV. (1) when the guest initially mmap()s at 0xf700 and above we pass

Re: [Qemu-devel] [PULL 00/11] Ide patches

2017-09-20 Thread John Snow
On 09/20/2017 01:02 PM, Mark Cave-Ayland wrote: > On 18/09/17 19:14, Peter Maydell wrote: > >> On 18 September 2017 at 19:00, Peter Maydell >> wrote: >>> On 18 September 2017 at 18:55, John Snow wrote: On 09/16/2017 10:34 AM, Peter Maydell

Re: [Qemu-devel] [PATCH 17/19] nvic: Make ICSR banked for v8M

2017-09-20 Thread Richard Henderson
On 09/12/2017 01:14 PM, Peter Maydell wrote: > The ICSR NVIC register is banked for v8M. This doesn't > require any new state, but it does mean that some bits > are controlled by BFHNFNMINS and some bits must work > with the correct banked exception. There is also a new > in v8M PENDNMICLR bit. >

  1   2   3   4   >