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

2017-07-07 Thread sundeep subbaraya
Hi Alistair, On Wed, Jul 5, 2017 at 11:28 PM, Alistair Francis wrote: > On Wed, Jul 5, 2017 at 10:56 AM, Alistair Francis > wrote: > > On Sun, Jul 2, 2017 at 9:45 PM, Subbaraya Sundeep > > wrote: > >> Modelled System Timer in Microsemi's Smartfusion2 Soc. > >> Timer has two 32bit down counters

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

2017-07-07 Thread sundeep subbaraya
Hi Alistair, On Wed, Jul 5, 2017 at 11:36 PM, Alistair Francis wrote: > On Sun, Jul 2, 2017 at 9:45 PM, Subbaraya Sundeep > wrote: > > Added Sytem register block of Smartfusion2. > > This block has PLL registers which are accessed by guest. > > > > Signed-off-by: Subbaraya Sundeep > > --- > >

Re: [Qemu-devel] [Qemu devel v6 PATCH 4/5] msf2: Add Smartfusion2 SoC.

2017-07-07 Thread sundeep subbaraya
Hi Alistair, On Wed, Jul 5, 2017 at 11:55 PM, Alistair Francis wrote: > On Sun, Jul 2, 2017 at 9:45 PM, Subbaraya Sundeep > wrote: > > The patch title shouldn't end in a full stop. > Ok will remove . > > > Smartfusion2 SoC has hardened Microcontroller subsystem > > and flash based FPGA fabric

Re: [Qemu-devel] [Qemu-ppc] [PATCH 0/5] spapr: DRC cleanups (part VI)

2017-07-07 Thread David Gibson
On Thu, Jul 06, 2017 at 11:31:35AM -0300, Daniel Henrique Barboza wrote: > > > On 07/05/2017 06:53 PM, Daniel Henrique Barboza wrote: > > Worth a try. I'll see if I can make a POC of this DRC reset at incoming > > migration time. > > > Just played a little with the idea of manual reset during m

Re: [Qemu-devel] [PATCH v2 08/27] linux-user/sh4: Notice gUSA regions during signal delivery

2017-07-07 Thread John Paul Adrian Glaubitz
On 07/07/2017 04:20 AM, Richard Henderson wrote: > We translate gUSA regions atomically in a parallel context. > But in a serial context a gUSA region may be interrupted. > In that case, restart the region as the kernel would. This patch is still causing random segfaults, unfortunately: Setting u

Re: [Qemu-devel] [RFC v2 0/8] VIRTIO-IOMMU device

2017-07-07 Thread Auger Eric
On 07/07/2017 08:25, Bharat Bhushan wrote: > Hi Eric, > >> -Original Message- >> From: Auger Eric [mailto:eric.au...@redhat.com] >> Sent: Friday, July 07, 2017 2:47 AM >> To: Bharat Bhushan ; Jean-Philippe Brucker >> ; eric.auger@gmail.com; >> peter.mayd...@linaro.org; alex.william..

Re: [Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-07-07 Thread Markus Armbruster
Sorry for the late review, got a bit overwhelmed... Vadim Galitsyn writes: > Commands above provide the following memory information in bytes: > > * base-memory - amount of unremovable memory specified > with '-m' option at the start of the QEMU process. > > * hotpluggable-memory - amoun

Re: [Qemu-devel] [RFC v2 0/8] VIRTIO-IOMMU device

2017-07-07 Thread Auger Eric
Hi, On 06/07/2017 23:11, Auger Eric wrote: > Hello Bharat, Jean-Philippe, > On 06/07/2017 12:02, Jean-Philippe Brucker wrote: >> On 05/07/17 09:49, Bharat Bhushan wrote:>>> Also when setup msi-route >> kvm_irqchip_add_msi_route() we needed to provide the translated address. > According to

Re: [Qemu-devel] Managing architectural restrictions with -device and libvirt

2017-07-07 Thread Mark Cave-Ayland
On 06/07/17 15:36, Markus Armbruster wrote: I see two ways to continue: 1. A new kind of pci-bridge should be created with a "special" secondary bus that has less slots. (harder to implement) 2. Add the limitation of the number of slots to the PCIBus class, (

Re: [Qemu-devel] [PATCH v11 6/6] trace: [trivial] Statically enable all guest events

2017-07-07 Thread Lluís Vilanova
Eric Blake writes: > On 07/04/2017 03:54 AM, Lluís Vilanova wrote: >> The existing optimizations makes it feasible to have them available on all >> builds. > While this change may feel trivial, I think it is a misnomer to include > "[trivial]" in the subject line, and I also think it should not g

Re: [Qemu-devel] [PATCH v11 06/29] target/i386: [tcg] Refactor init_disas_context

2017-07-07 Thread Lluís Vilanova
Emilio G Cota writes: > On Wed, Jun 28, 2017 at 15:40:52 +0300, Lluís Vilanova wrote: [,,,] >> @@ -8425,7 +8415,7 @@ void gen_intermediate_code(CPUState *cpu, >> TranslationBlock *tb) >> dc-> code64 = (flags >> HF_CS64_SHIFT) & 1; >> #endif >> dc-> flags = flags; >> -dc->jmp_opt = !

Re: [Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-07-07 Thread Markus Armbruster
Markus Armbruster writes: > Sorry for the late review, got a bit overwhelmed... > > Vadim Galitsyn writes: > >> Commands above provide the following memory information in bytes: >> >> * base-memory - amount of unremovable memory specified >> with '-m' option at the start of the QEMU proces

[Qemu-devel] [PATCH 0/3] pci: allow PCI bus slots to be marked as reserved

2017-07-07 Thread Mark Cave-Ayland
For some machines it is impossible to plug devices into a particular PCI bus slot, e.g. for a real Ultra 5 there are 2 PCI bridges attached to the root bus behind which all devices must be plugged. Ignoring this rule will cause problems with interrupt routing since the interrupt numbers are calcula

[Qemu-devel] [PATCH 2/3] pci: add dev_reserved_mask property to PCIBus

2017-07-07 Thread Mark Cave-Ayland
This is just a simple bitmask indicating whether or not each PCI slot on the bus is reserved. Ensure that it is initialised to zero so that all bus slots are available by default. Signed-off-by: Mark Cave-Ayland --- hw/pci/pci.c |1 + include/hw/pci/pci_bus.h |1 + 2 files ch

[Qemu-devel] [PATCH 1/3] pci: move check for existing devfn into new pci_bus_devfn_available() helper

2017-07-07 Thread Mark Cave-Ayland
Also touch up the logic in do_pci_register_device() accordingly. Signed-off-by: Mark Cave-Ayland --- hw/pci/pci.c | 14 -- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index 0c6f74a..04e6edb 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c

[Qemu-devel] [PATCH 3/3] pci: add reserved slot check to do_pci_register_device()

2017-07-07 Thread Mark Cave-Ayland
Signed-off-by: Mark Cave-Ayland --- hw/pci/pci.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index 239161e..9dece2a 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c @@ -961,6 +961,15 @@ static bool pci_bus_devfn_available(PC

Re: [Qemu-devel] [PATCH v3] xenfb: remove xen_init_display "temporary" hack

2017-07-07 Thread Paul Durrant
> -Original Message- > From: Stefano Stabellini [mailto:sstabell...@kernel.org] > Sent: 07 July 2017 00:48 > To: xen-de...@lists.xenproject.org; qemu-devel@nongnu.org > Cc: sstabell...@kernel.org; peter.mayd...@linaro.org; Anthony Perard > ; kra...@redhat.com; Paul Durrant > ; Owen Smith >

Re: [Qemu-devel] [PATCH v22 23/30] qmp: add persistent flag to block-dirty-bitmap-add

2017-07-07 Thread Markus Armbruster
QAPI schema review only... I apologize for its lateness. Vladimir Sementsov-Ogievskiy writes: > Add optional 'persistent' flag to qmp command block-dirty-bitmap-add. > Default is false. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > Signed-off-by: Denis V. Lunev > Reviewed-by: Max Reitz >

Re: [Qemu-devel] [PATCH v22 24/30] qmp: add autoload parameter to block-dirty-bitmap-add

2017-07-07 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > Optional. Default is false. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > Signed-off-by: Denis V. Lunev > Reviewed-by: Max Reitz > Reviewed-by: John Snow > --- > blockdev.c | 18 -- > qapi/block-core.json | 6 +- > 2 f

Re: [Qemu-devel] [PATCH v22 25/30] qmp: add x-debug-block-dirty-bitmap-sha256

2017-07-07 Thread Markus Armbruster
Vladimir Sementsov-Ogievskiy writes: > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > block/dirty-bitmap.c | 5 + > blockdev.c | 25 + > include/block/dirty-bitmap.h | 1 + > include/qemu/hbitmap.h | 8 > qapi/block-co

Re: [Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-07-07 Thread Dr. David Alan Gilbert
* Markus Armbruster (arm...@redhat.com) wrote: > Sorry for the late review, got a bit overwhelmed... > > Vadim Galitsyn writes: > > > Commands above provide the following memory information in bytes: > > > > * base-memory - amount of unremovable memory specified > > with '-m' option at the

Re: [Qemu-devel] [PATCH v22 23/30] qmp: add persistent flag to block-dirty-bitmap-add

2017-07-07 Thread Vladimir Sementsov-Ogievskiy
07.07.2017 10:54, Markus Armbruster wrote: QAPI schema review only... I apologize for its lateness. Vladimir Sementsov-Ogievskiy writes: Add optional 'persistent' flag to qmp command block-dirty-bitmap-add. Default is false. Signed-off-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Denis

Re: [Qemu-devel] [PATCH v22 25/30] qmp: add x-debug-block-dirty-bitmap-sha256

2017-07-07 Thread Daniel P. Berrange
On Fri, Jul 07, 2017 at 10:05:22AM +0200, Markus Armbruster wrote: > Vladimir Sementsov-Ogievskiy writes: > > > Signed-off-by: Vladimir Sementsov-Ogievskiy > > --- > > block/dirty-bitmap.c | 5 + > > blockdev.c | 25 + > > include/block/dir

Re: [Qemu-devel] [PATCH v2 08/27] linux-user/sh4: Notice gUSA regions during signal delivery

2017-07-07 Thread Richard Henderson
On 07/06/2017 09:25 PM, John Paul Adrian Glaubitz wrote: On 07/07/2017 04:20 AM, Richard Henderson wrote: We translate gUSA regions atomically in a parallel context. But in a serial context a gUSA region may be interrupted. In that case, restart the region as the kernel would. This patch is st

Re: [Qemu-devel] [PATCH v2 3/3] xen-disk: use an IOThread per instance

2017-07-07 Thread Paul Durrant
> -Original Message- > From: Stefano Stabellini [mailto:sstabell...@kernel.org] > Sent: 22 June 2017 23:15 > To: Paul Durrant > Cc: xen-de...@lists.xenproject.org; qemu-devel@nongnu.org; qemu- > bl...@nongnu.org; Stefano Stabellini ; Anthony > Perard ; Kevin Wolf ; > Max Reitz ; afaer...@s

Re: [Qemu-devel] [PATCH v2 08/27] linux-user/sh4: Notice gUSA regions during signal delivery

2017-07-07 Thread John Paul Adrian Glaubitz
On 07/07/2017 10:20 AM, Richard Henderson wrote: > How do I reproduce this from the filesystem image you linked earlier? So, the problem happens in the tzdata package when it's being configured: (sid-sh4-sbuild)root@nofan:/# dpkg-reconfigure tzdata perl: warning: Setting locale failed. perl: warn

Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report()

2017-07-07 Thread Cornelia Huck
On Thu, 6 Jul 2017 16:49:44 -0700 Alistair Francis wrote: > Convert all uses of error_report("[Ww]arning:"... to use warn_report() > instead. This helps standardise on a single method of printing warnings > to the user. > > All of the warnings were found using this regex expression: > error_

Re: [Qemu-devel] [PATCH v3 1/4] ACPI: Add APEI GHES Table Generation support

2017-07-07 Thread gengdongjiu
Hi Laszlo, sorry for my late response. On 2017/6/3 20:01, Laszlo Ersek wrote: > On 05/22/17 16:23, Laszlo Ersek wrote: >> Keeping some context: >> >> On 05/12/17 23:00, Laszlo Ersek wrote: >>> On 04/30/17 07:35, Dongjiu Geng wrote: > >> (68) In the code below, you are not taking an "OVMF heade

Re: [Qemu-devel] [PATCH v2 08/27] linux-user/sh4: Notice gUSA regions during signal delivery

2017-07-07 Thread John Paul Adrian Glaubitz
On 07/07/2017 10:30 AM, John Paul Adrian Glaubitz wrote: > The scripts which are run here can be found as > /var/lib/dpkg/info/tzdata.{config,postinst). > I don't know yet which command in particular triggers the crash. Interesting. It crashes for me immediately after resizing the terminal window

Re: [Qemu-devel] [PATCH v4 04/17] dirty-bitmap: Drop unused functions

2017-07-07 Thread Vladimir Sementsov-Ogievskiy
07.07.2017 02:43, John Snow wrote: On 07/03/2017 11:10 AM, Eric Blake wrote: We had several functions that no one is currently using, and which use sector-based interfaces. I'm trying to convert towards byte-based interfaces, so it's easier to just drop the unused functions: bdrv_dirty_bitmap

Re: [Qemu-devel] [PATCH v4] hmp, qmp: introduce memory-size-summary commands

2017-07-07 Thread Markus Armbruster
"Dr. David Alan Gilbert" writes: > * Markus Armbruster (arm...@redhat.com) wrote: >> Sorry for the late review, got a bit overwhelmed... >> >> Vadim Galitsyn writes: >> >> > Commands above provide the following memory information in bytes: >> > >> > * base-memory - amount of unremovable memo

Re: [Qemu-devel] [PATCH v22 25/30] qmp: add x-debug-block-dirty-bitmap-sha256

2017-07-07 Thread Markus Armbruster
"Daniel P. Berrange" writes: > On Fri, Jul 07, 2017 at 10:05:22AM +0200, Markus Armbruster wrote: >> Vladimir Sementsov-Ogievskiy writes: >> >> > Signed-off-by: Vladimir Sementsov-Ogievskiy >> > --- >> > block/dirty-bitmap.c | 5 + >> > blockdev.c | 25 +

Re: [Qemu-devel] [PATCH 12/17] migration: add postcopy migration of dirty bitmaps

2017-07-07 Thread Denis V. Lunev
On 07/06/2017 08:53 PM, John Snow wrote: > > On 07/06/2017 04:05 AM, Vladimir Sementsov-Ogievskiy wrote: >> 06.07.2017 00:46, John Snow wrote: >>> On 07/05/2017 05:24 AM, Vladimir Sementsov-Ogievskiy wrote: 16.02.2017 16:04, Fam Zheng wrote: >> +dbms->node_name = bdrv_get_node_

Re: [Qemu-devel] [PATCH v2 08/27] linux-user/sh4: Notice gUSA regions during signal delivery

2017-07-07 Thread Laurent Vivier
Le 07/07/2017 à 04:20, Richard Henderson a écrit : > We translate gUSA regions atomically in a parallel context. > But in a serial context a gUSA region may be interrupted. > In that case, restart the region as the kernel would. > > Signed-off-by: Richard Henderson > --- > linux-user/signal.c |

Re: [Qemu-devel] [PATCH v2 08/27] linux-user/sh4: Notice gUSA regions during signal delivery

2017-07-07 Thread Laurent Vivier
Le 07/07/2017 à 11:05, Laurent Vivier a écrit : > Le 07/07/2017 à 04:20, Richard Henderson a écrit : >> We translate gUSA regions atomically in a parallel context. >> But in a serial context a gUSA region may be interrupted. >> In that case, restart the region as the kernel would. >> >> Signed-off-

Re: [Qemu-devel] [PATCH v2 08/27] linux-user/sh4: Notice gUSA regions during signal delivery

2017-07-07 Thread John Paul Adrian Glaubitz
On 07/07/2017 11:05 AM, Laurent Vivier wrote:> I think unwind_gusa() should be moved after the get_sigfram() (in both> cases), because r15 can be updated and the sigframe base lost. Tried the change. "dpkg-reconfigure tzdata" now crashes differently: Current default time zone: 'Europe/Berlin' Lo

Re: [Qemu-devel] [PATCH 12/17] migration: add postcopy migration of dirty bitmaps

2017-07-07 Thread Vladimir Sementsov-Ogievskiy
06.07.2017 20:53, John Snow wrote: On 07/06/2017 04:05 AM, Vladimir Sementsov-Ogievskiy wrote: 06.07.2017 00:46, John Snow wrote: On 07/05/2017 05:24 AM, Vladimir Sementsov-Ogievskiy wrote: 16.02.2017 16:04, Fam Zheng wrote: +dbms->node_name = bdrv_get_node_name(bs); +

Re: [Qemu-devel] [PATCH v4 19/21] block: Make bdrv_is_allocated() byte-based

2017-07-07 Thread Kevin Wolf
Am 07.07.2017 um 04:55 hat Eric Blake geschrieben: > On 07/06/2017 11:02 AM, Kevin Wolf wrote: > > >> +++ b/qemu-img.c > >> @@ -3229,6 +3229,7 @@ static int img_rebase(int argc, char **argv) > >> int64_t new_backing_num_sectors = 0; > >> uint64_t sector; > >> int n; > >>

Re: [Qemu-devel] [PATCH v11 10/29] target/i386: [tcg] Refactor translate_insn

2017-07-07 Thread Lluís Vilanova
Emilio G Cota writes: > On Wed, Jun 28, 2017 at 15:57:00 +0300, Lluís Vilanova wrote: >> Incrementally paves the way towards using the generic instruction translation >> loop. >> >> Signed-off-by: Lluís Vilanova >> --- >> target/i386/translate.c | 72 >> +++

Re: [Qemu-devel] [PATCH] block: fix bs->file leak in bdrv_new_open_driver()

2017-07-07 Thread Kevin Wolf
Am 29.06.2017 um 22:06 hat Manos Pitsidianakis geschrieben: > On Thu, Jun 29, 2017 at 03:57:49PM +0200, Kevin Wolf wrote: > >Am 29.06.2017 um 14:07 hat Manos Pitsidianakis geschrieben: > >>On Thu, Jun 29, 2017 at 01:18:24PM +0200, Kevin Wolf wrote: > >>>Am 29.06.2017 um 08:03 hat Manos Pitsidianaki

Re: [Qemu-devel] [PATCH v3 1/4] ACPI: Add APEI GHES Table Generation support

2017-07-07 Thread Laszlo Ersek
On 07/07/17 10:32, gengdongjiu wrote: > Hi Laszlo, >sorry for my late response. > > On 2017/6/3 20:01, Laszlo Ersek wrote: >> On 05/22/17 16:23, Laszlo Ersek wrote: >>> Keeping some context: >>> >>> On 05/12/17 23:00, Laszlo Ersek wrote: On 04/30/17 07:35, Dongjiu Geng wrote: >> >>> (68)

Re: [Qemu-devel] [PATCH] block: fix bs->file leak in bdrv_new_open_driver()

2017-07-07 Thread Manos Pitsidianakis
On Fri, Jul 07, 2017 at 11:28:15AM +0200, Kevin Wolf wrote: Am 29.06.2017 um 22:06 hat Manos Pitsidianakis geschrieben: On Thu, Jun 29, 2017 at 03:57:49PM +0200, Kevin Wolf wrote: >Am 29.06.2017 um 14:07 hat Manos Pitsidianakis geschrieben: >>On Thu, Jun 29, 2017 at 01:18:24PM +0200, Kevin Wolf

Re: [Qemu-devel] [PATCH v3 2/2] mttcg/i386: Patch instruction using async_safe_* framework

2017-07-07 Thread Alex Bennée
Pranith Kumar writes: > In mttcg, calling pause_all_vcpus() during execution from the > generated TBs causes a deadlock if some vCPU is waiting for exclusive > execution in start_exclusive(). Fix this by using the aync_safe_* > framework instead of pausing vcpus for patching instructions. It lo

Re: [Qemu-devel] [Qemu-block] [PATCH v3 2/6] block: Add VFIO based NVMe driver

2017-07-07 Thread Paolo Bonzini
On 06/07/2017 19:38, Keith Busch wrote: > On Wed, Jul 05, 2017 at 09:36:31PM +0800, Fam Zheng wrote: >> This is a new protocol driver that exclusively opens a host NVMe >> controller through VFIO. It achieves better latency than linux-aio by >> completely bypassing host kernel vfs/block layer. >> >

[Qemu-devel] [RFC PATCH v2 0/8] Implement network booting directly into the s390-ccw BIOS

2017-07-07 Thread Thomas Huth
It's already possible to do a network boot of an s390x guest with an external netboot image based on a Linux installation, but it would be much more convenient if the s390-ccw firmware supported network booting right out of the box, without the need to assemble such an external image first. This p

[Qemu-devel] [RFC PATCH v2 1/8] pc-bios/s390-ccw: Move libc functions to separate header

2017-07-07 Thread Thomas Huth
The upcoming netboot code will use the libc from SLOF. To be able to still use s390-ccw.h there, the libc related functions in this header have to be moved to a different location. And while we're at it, remove the duplicate memcpy() function from sclp.c. Signed-off-by: Thomas Huth --- pc-bios/s

[Qemu-devel] [RFC PATCH v2 2/8] pc-bios/s390-ccw: Move ebc2asc to sclp.c

2017-07-07 Thread Thomas Huth
We will later need this array in a file that we will link to the netboot code, too. Since there is some ebcdic convertion done in sclp_get_loadparm_ascii(), the sclp.c file seems to be a good candidate. Signed-off-by: Thomas Huth --- pc-bios/s390-ccw/main.c | 11 --- pc-bios/s390-ccw/scl

[Qemu-devel] [RFC PATCH v2 3/8] pc-bios/s390-ccw: Move virtio-block related functions into a separate file

2017-07-07 Thread Thomas Huth
The netboot code is going to link against the code from virtio.c, too, so we've got to move the virtio-block related code out of the way. Signed-off-by: Thomas Huth --- pc-bios/s390-ccw/Makefile | 2 +- pc-bios/s390-ccw/main.c | 2 +- pc-bios/s390-ccw/s390-ccw.h | 2 +- pc-bios

[Qemu-devel] [RFC PATCH v2 8/8] pc-bios/s390-ccw: Link libnet into the netboot image and do the TFTP load

2017-07-07 Thread Thomas Huth
Most of the code has been taken from SLOF's netload.c file. Now we can finally load an image via TFTP and execute the downloaded kernel. Signed-off-by: Thomas Huth --- pc-bios/s390-ccw/netboot.mak | 12 ++- pc-bios/s390-ccw/netmain.c | 225 +++ 2 files

Re: [Qemu-devel] [PATCH v3 2/2] mttcg/i386: Patch instruction using async_safe_* framework

2017-07-07 Thread Paolo Bonzini
On 29/06/2017 09:11, Pranith Kumar wrote: > In mttcg, calling pause_all_vcpus() during execution from the > generated TBs causes a deadlock if some vCPU is waiting for exclusive > execution in start_exclusive(). Fix this by using the aync_safe_* > framework instead of pausing vcpus for patching i

[Qemu-devel] [RFC PATCH v2 4/8] pc-bios/s390-ccw: Add a write() function for stdio

2017-07-07 Thread Thomas Huth
The stdio functions from the SLOF libc need a write() function for printing text to stdout/stderr. Let's implement this function by refactoring the code from sclp_print(). Signed-off-by: Thomas Huth --- pc-bios/s390-ccw/sclp.c | 16 ++-- 1 file changed, 14 insertions(+), 2 deletions(

[Qemu-devel] virtio-gpu: bug report : vnc connect can not display when migrate to destination.

2017-07-07 Thread wanghaibin
Hi, all: I met a problem. I boot a aarch64 kernel with virtio-gpu device on aarch64 board. The cmdline just like: qemu-system-aarch64 -name guest=vm,debug-threads=on -S -machine virt-2.6,accel=kvm,usb=off,gic-version=3 -cpu host -m 8192 -realtime mlock=off -smp 4,sockets=4,cores=1,threads=1 -

Re: [Qemu-devel] [Qemu-ppc] [PATCH 0/5] spapr: DRC cleanups (part VI)

2017-07-07 Thread Daniel Henrique Barboza
On 07/07/2017 04:14 AM, David Gibson wrote: On Thu, Jul 06, 2017 at 11:31:35AM -0300, Daniel Henrique Barboza wrote: On 07/05/2017 06:53 PM, Daniel Henrique Barboza wrote: Worth a try. I'll see if I can make a POC of this DRC reset at incoming migration time. Just played a little with the

[Qemu-devel] [RFC PATCH v2 5/8] roms/SLOF: Update submodule to latest status

2017-07-07 Thread Thomas Huth
We need the latest fixes for building the libc and libnet of SLOF for the s390-ccw network bootloader firmware. Signed-off-by: Thomas Huth --- roms/SLOF | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/roms/SLOF b/roms/SLOF index 66d250e..64215f4 16 --- a/roms/SLOF +++ b/r

[Qemu-devel] [RFC PATCH v2 6/8] pc-bios/s390-ccw: Add core files for the network bootloading program

2017-07-07 Thread Thomas Huth
This is just a preparation for the next steps: Add a makefile and a stripped down copy of pc-bios/s390-ccw/main.c as a basis for the network bootloader program, linked against the libc from SLOF already (which we will need for SLOF's libnet). The networking code is not included yet. Signed-off-by:

[Qemu-devel] [RFC PATCH v2 7/8] pc-bios/s390-ccw: Add virtio-net driver code

2017-07-07 Thread Thomas Huth
The driver provides the recv() and send() functions which will be required by SLOF's libnet code for receiving and sending packets. Signed-off-by: Thomas Huth --- pc-bios/s390-ccw/netboot.mak | 2 +- pc-bios/s390-ccw/virtio-net.c | 130 ++ pc-bios/s390-

Re: [Qemu-devel] [PATCH v2] Python3 Support for qmp.py

2017-07-07 Thread Stefan Hajnoczi
On Fri, Jul 07, 2017 at 12:38:47AM +0530, Ishani Chugh wrote: Commit messages usually begin with a prefix indicating the affected component. This makes it easier for code reviewers and maintainers to browse email subjects and the git log. I recommend the following commit message: scripts/qmp:

Re: [Qemu-devel] [PATCH v11 13/29] target/i386: [tcg] Port to generic translation framework

2017-07-07 Thread Lluís Vilanova
Emilio G Cota writes: > On Wed, Jun 28, 2017 at 16:09:06 +0300, Lluís Vilanova wrote: [...] >> diff --git a/target/i386/translate.c b/target/i386/translate.c >> index 3950fe95a4..295be26a95 100644 >> --- a/target/i386/translate.c >> +++ b/target/i386/translate.c > (snip) >> @@ -8544,111 +8548,23 @

Re: [Qemu-devel] [PATCH v11 13/29] target/i386: [tcg] Port to generic translation framework

2017-07-07 Thread Lluís Vilanova
Emilio G Cota writes: > On Wed, Jun 28, 2017 at 16:09:06 +0300, Lluís Vilanova wrote: >> Signed-off-by: Lluís Vilanova >> --- > Reviewed-by: Emilio G. Cota > Tested-by: Emilio G. Cota > BTW have you tested icount mode? Yes. Regular, -singlestep and -icount 1. Thanks, Lluis

Re: [Qemu-devel] [PATCH 0/3] Add new utility function memory_region_allocate_aux_memory()

2017-07-07 Thread Peter Maydell
On 6 July 2017 at 18:47, Peter Maydell wrote: > On 6 July 2017 at 18:26, Paolo Bonzini wrote: >> Maybe, for memory_region_init_ram only, the owner argument can be made a >> DeviceState (or NULL)? > > Something that might influence things here -- of the twelve > callers of vmstate_register_ram(),

Re: [Qemu-devel] [PATCH v5 2/2] live-block-ops.txt: Rename, rewrite, and improve it

2017-07-07 Thread Kashyap Chamarthy
On Thu, Jul 06, 2017 at 06:41:07PM -0400, John Snow wrote: > On 07/06/2017 10:36 AM, Kashyap Chamarthy wrote: [...] > > +disk image chains by merging data from overlays into backing files; live > > +synchronize data from a disk image chain (including current active disk) > > +to another target im

Re: [Qemu-devel] [RFC 00/29] postcopy+vhost-user/shared ram

2017-07-07 Thread Dr. David Alan Gilbert
* Marc-André Lureau (marcandre.lur...@gmail.com) wrote: > Hi > > On Thu, Jun 29, 2017 at 8:56 PM Dr. David Alan Gilbert > wrote: > > > * Dr. David Alan Gilbert (git) (dgilb...@redhat.com) wrote: > > > From: "Dr. David Alan Gilbert" > > > > > > Hi, > > > This is a RFC/WIP series that enables p

Re: [Qemu-devel] [PATCH 0/3] Add new utility function memory_region_allocate_aux_memory()

2017-07-07 Thread Paolo Bonzini
On 07/07/2017 12:43, Peter Maydell wrote: > On 6 July 2017 at 18:47, Peter Maydell wrote: >> On 6 July 2017 at 18:26, Paolo Bonzini wrote: >>> Maybe, for memory_region_init_ram only, the owner argument can be made a >>> DeviceState (or NULL)? >> >> Something that might influence things here --

Re: [Qemu-devel] [PATCH] chardev: block during sync read

2017-07-07 Thread Paolo Bonzini
On 06/07/2017 19:03, Marc-André Lureau wrote: > A sync read should block until all requested data is > available (instead of retrying in qemu_chr_fe_read_all). Change the > channel to blocking during sync_read. > > Signed-off-by: Marc-André Lureau > --- > chardev/char-socket.c | 2 ++ > 1 file

Re: [Qemu-devel] [PATCH v2] Python3 Support for qmp.py

2017-07-07 Thread Daniel P. Berrange
On Fri, Jul 07, 2017 at 12:38:47AM +0530, Ishani Chugh wrote: > This patch intends to make qmp.py compatible with both python2 and python3. > > * Python 3 does not have dict.has_key(key), use key in dict instead > * Avoid line-based I/O since Python 2/3 have different character >encoding beh

Re: [Qemu-devel] [PATCH v11 23/29] target/arm: [tcg] Port to translate_insn

2017-07-07 Thread Lluís Vilanova
Richard Henderson writes: > On 06/28/2017 06:49 AM, Lluís Vilanova wrote: >> +/* We want to stop the TB if the next insn starts in a new page, >> + * or if it spans between this page and the next. This means that >> + * if we're looking at the last halfword in the page we n

Re: [Qemu-devel] [PATCH v11 24/29] target/arm: [tcg, a64] Port to translate_insn

2017-07-07 Thread Lluís Vilanova
Richard Henderson writes: > On 06/28/2017 06:53 AM, Lluís Vilanova wrote: >> Incrementally paves the way towards using the generic instruction translation >> loop. >> >> Signed-off-by: Lluís Vilanova >> --- >> target/arm/translate-a64.c | 74 >> +++- >>

Re: [Qemu-devel] [PATCH v11 29/29] target/arm: [tcg] Port to generic translation framework

2017-07-07 Thread Lluís Vilanova
Richard Henderson writes: > On 06/28/2017 07:13 AM, Lluís Vilanova wrote: >> @@ -11377,6 +11385,9 @@ static void aarch64_trblock_tb_stop(DisasContextBase >> *dcbase, CPUState *cpu) >> break; >> } >> } >> + >> +/* Functions above can change dc->pc, so re-align db->pc_next */ >> +dc->base.p

Re: [Qemu-devel] [PATCH v1 4/6] char-socket: Report TCP socket waiting as information

2017-07-07 Thread Philippe Mathieu-Daudé
On 07/06/2017 08:49 PM, Alistair Francis wrote: When QEMU is waiting for a TCP socket connection it reports that message as an error. This isn't an error it is just information so let's change the report to use info_report() instead. Signed-off-by: Alistair Francis Reviewed-by: Thomas Huth R

Re: [Qemu-devel] [PATCH v11 6/6] trace: [trivial] Statically enable all guest events

2017-07-07 Thread Eric Blake
On 07/07/2017 02:34 AM, Lluís Vilanova wrote: > Eric Blake writes: > >> On 07/04/2017 03:54 AM, Lluís Vilanova wrote: >>> The existing optimizations makes it feasible to have them available on all >>> builds. > >> While this change may feel trivial, I think it is a misnomer to include >> "[trivia

Re: [Qemu-devel] [PATCHv7 5/6] fw_cfg: move qdev_init_nofail() from fw_cfg_init1() to callers

2017-07-07 Thread Igor Mammedov
On Tue, 4 Jul 2017 19:08:44 +0100 Mark Cave-Ayland wrote: > On 03/07/17 10:39, Igor Mammedov wrote: > > > On Thu, 29 Jun 2017 15:07:19 +0100 > > Mark Cave-Ayland wrote: > > > >> When looking to instantiate a TYPE_FW_CFG_MEM or TYPE_FW_CFG_IO device to > >> be > >> able to wire it up differe

Re: [Qemu-devel] [RFC v2 0/8] VIRTIO-IOMMU device

2017-07-07 Thread Bharat Bhushan
Hi Eric, > -Original Message- > From: Jean-Philippe Brucker [mailto:jean-philippe.bruc...@arm.com] > Sent: Thursday, July 06, 2017 3:33 PM > To: Bharat Bhushan ; Auger Eric > ; eric.auger@gmail.com; > peter.mayd...@linaro.org; alex.william...@redhat.com; >

Re: [Qemu-devel] [RFC 29/29] vhost-user: Claim support for postcopy

2017-07-07 Thread Dr. David Alan Gilbert
* Maxime Coquelin (maxime.coque...@redhat.com) wrote: > > > On 06/28/2017 09:00 PM, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > Tell QEMU we understand the protocol features needed for postcopy. > > > > Signed-off-by: Dr. David Alan Gilbert > > --- > > con

[Qemu-devel] [PATCH v12 00/27] translate: [tcg] Generic translation framework

2017-07-07 Thread Lluís Vilanova
This series proposes a generic (target-agnostic) instruction translation framework. It basically provides a generic main loop for instruction disassembly, which calls target-specific functions when necessary. This generalization makes inserting new code in the main loop easier, and helps in keepin

Re: [Qemu-devel] [PATCH v1 6/6] Convert error_report*_err() to warn_report*_err()

2017-07-07 Thread Philippe Mathieu-Daudé
Hi Francis, On 07/06/2017 08:49 PM, Alistair Francis wrote: Convert all uses of error_report*_err("[Ww]arning:"... to use this is not the regex you used, maybe just use "Warning:"... warn_report*_err() instead. This helps standardise on a single standardize? "Enforce single method ..."?

[Qemu-devel] [PATCH v12 01/27] Pass generic CPUState to gen_intermediate_code()

2017-07-07 Thread Lluís Vilanova
Needed to implement a target-agnostic gen_intermediate_code() in the future. Signed-off-by: Lluís Vilanova Reviewed-by: David Gibson Reviewed-by: Richard Henderson --- accel/tcg/translate-all.c |2 +- include/exec/exec-all.h |2 +- target/alpha/translate.c |5 ++---

[Qemu-devel] [PATCH v12 02/27] cpu-exec: Avoid global variables in icount-related functions

2017-07-07 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- include/exec/gen-icount.h | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/include/exec/gen-icount.h b/include/exec/gen-icount.h index 62d462e494..6c28ef59c3 100644 --- a/include/exec/gen-icount.h +++ b/include/exec/gen-icount.h @@

[Qemu-devel] [PATCH v12 03/27] target: [tcg] Use a generic enum for DISAS_ values

2017-07-07 Thread Lluís Vilanova
Used later. An enum makes expected values explicit and bounds the value space of switches. Signed-off-by: Lluís Vilanova Reviewed-by: Emilio G. Cota Reviewed-by: Richard Henderson --- include/exec/exec-all.h |6 -- include/exec/translator.h | 39

Re: [Qemu-devel] [RFC 24/29] vhost+postcopy: Lock around set_mem_table

2017-07-07 Thread Dr. David Alan Gilbert
* Maxime Coquelin (maxime.coque...@redhat.com) wrote: > > > On 06/28/2017 09:00 PM, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > **HACK - better solution needed ** > > We have the situation where: > > > > qemu bridge > > > > se

[Qemu-devel] [PATCH v12 04/27] target: [tcg] Add generic translation framework

2017-07-07 Thread Lluís Vilanova
Signed-off-by: Lluís Vilanova --- accel/tcg/Makefile.objs |1 accel/tcg/translator.c| 152 + include/exec/gen-icount.h |2 - include/exec/translator.h | 99 + 4 files changed, 253 insertions(+), 1 deletion(-

Re: [Qemu-devel] [PATCH 0/3] Add new utility function memory_region_allocate_aux_memory()

2017-07-07 Thread Peter Maydell
On 6 July 2017 at 18:26, Paolo Bonzini wrote: > > > On 06/07/2017 19:13, Peter Maydell wrote: >> Slightly awkward because owner is an Object but vmstate_register_ram() >> needs a DeviceState. Is this OK, or too much magic? >> >> DeviceState *owner_dev; >> Error *err = NULL; >> >> memor

Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report()

2017-07-07 Thread Eduardo Habkost
On Fri, Jul 07, 2017 at 08:33:19AM +0200, Thomas Huth wrote: > On 07.07.2017 01:49, Alistair Francis wrote: > > Convert all uses of error_report("[Ww]arning:"... to use warn_report() > > instead. This helps standardise on a single method of printing warnings > > to the user. > > > > All of the war

[Qemu-devel] [PATCH v12 05/27] target/i386: [tcg] Port to DisasContextBase

2017-07-07 Thread Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation loop. Signed-off-by: Lluís Vilanova Reviewed-by: Emilio G. Cota Reviewed-by: Richard Henderson --- target/i386/translate.c | 142 +++ 1 file changed, 70 insertions(+), 72

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

2017-07-07 Thread sundeep subbaraya
Hi Alistair, On Wed, Jul 5, 2017 at 11:48 PM, Alistair Francis wrote: > On Sun, Jul 2, 2017 at 9:45 PM, Subbaraya Sundeep > wrote: > > Modelled Microsemi's Smartfusion2 SPI controller. > > > > Signed-off-by: Subbaraya Sundeep > > --- > > hw/ssi/Makefile.objs | 1 + > > hw/ssi/mss-spi.c

Re: [Qemu-devel] [PATCH 0/3] Add new utility function memory_region_allocate_aux_memory()

2017-07-07 Thread Paolo Bonzini
On 07/07/2017 13:58, Peter Maydell wrote: >>> >>> memory_region_init_ram(mr, owner, name, ram_size, &err); >>> if (err) { >>> error_propagate(errp, err); >>> return; >>> } >>> /* Note that owner_dev may be NULL if owner is not a DeviceState; >>> * in that case

Re: [Qemu-devel] [RFC 00/29] postcopy+vhost-user/shared ram

2017-07-07 Thread Dr. David Alan Gilbert
* Michael S. Tsirkin (m...@redhat.com) wrote: > On Wed, Jun 28, 2017 at 08:00:18PM +0100, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > Hi, > > This is a RFC/WIP series that enables postcopy migration > > with shared memory to a vhost-user process. > > It's base

[Qemu-devel] [PATCH v12 06/27] target/i386: [tcg] Port to init_disas_context

2017-07-07 Thread Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation loop. Signed-off-by: Lluís Vilanova Reviewed-by: Richard Henderson --- target/i386/translate.c | 41 +++-- 1 file changed, 23 insertions(+), 18 deletions(-) diff --git a/target/i

Re: [Qemu-devel] [PATCH v1 6/6] Convert error_report*_err() to warn_report*_err()

2017-07-07 Thread Eduardo Habkost
On Thu, Jul 06, 2017 at 04:49:54PM -0700, Alistair Francis wrote: > Convert all uses of error_report*_err("[Ww]arning:"... to use > warn_report*_err() instead. This helps standardise on a single > method of printing warnings to the user. > > Signed-off-by: Alistair Francis > Cc: Paolo Bonzini >

Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report()

2017-07-07 Thread Eduardo Habkost
On Thu, Jul 06, 2017 at 04:49:44PM -0700, Alistair Francis wrote: > Convert all uses of error_report("[Ww]arning:"... to use warn_report() > instead. This helps standardise on a single method of printing warnings > to the user. > > All of the warnings were found using this regex expression: >

Re: [Qemu-devel] [PATCH v1 3/6] Convert error_report() to warn_report()

2017-07-07 Thread Thomas Huth
On 07.07.2017 13:58, Eduardo Habkost wrote: > On Fri, Jul 07, 2017 at 08:33:19AM +0200, Thomas Huth wrote: >> On 07.07.2017 01:49, Alistair Francis wrote: >>> Convert all uses of error_report("[Ww]arning:"... to use warn_report() >>> instead. This helps standardise on a single method of printing wa

[Qemu-devel] [PATCH v12 07/27] target/i386: [tcg] Port to insn_start

2017-07-07 Thread Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation loop. Signed-off-by: Lluís Vilanova Reviewed-by: Emilio G. Cota Reviewed-by: Richard Henderson --- target/i386/translate.c |9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/target/i386/tr

Re: [Qemu-devel] [PATCH v2.2 3/4] doc: add item for "-M enforce-config-section"

2017-07-07 Thread Eduardo Habkost
On Fri, Jul 07, 2017 at 10:54:08AM +0800, Peter Xu wrote: > It's never documented, and now we have one more parameter for it (which > obsoletes this one). Document it properly. > > Although now when enforce-config-section is set, it'll override the > other "-global" parameter, that is not necessar

[Qemu-devel] [PATCH v12 08/27] target/i386: [tcg] Port to breakpoint_check

2017-07-07 Thread Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation loop. Signed-off-by: Lluís Vilanova Reviewed-by: Richard Henderson Reviewed-by: Emilio G. Cota --- target/i386/translate.c | 58 ++- 1 file changed, 47 insertions(+), 11

[Qemu-devel] [PATCH] migration: Make analyze-migration script target-page-size aware

2017-07-07 Thread Alexander Graf
The configuration section has a new subsection to transmit the target page size along with the migration stream. The analyze migration script needs to learn about that to read configuration streams that were triggering this subsection to get transmitted. With this patch applied, I can successfully

Re: [Qemu-devel] [PATCH v2.2 3/4] doc: add item for "-M enforce-config-section"

2017-07-07 Thread Dr. David Alan Gilbert
* Eduardo Habkost (ehabk...@redhat.com) wrote: > On Fri, Jul 07, 2017 at 10:54:08AM +0800, Peter Xu wrote: > > It's never documented, and now we have one more parameter for it (which > > obsoletes this one). Document it properly. > > > > Although now when enforce-config-section is set, it'll overr

[Qemu-devel] [PATCH v12 09/27] target/i386: [tcg] Port to translate_insn

2017-07-07 Thread Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation loop. Signed-off-by: Lluís Vilanova Reviewed-by: Richard Henderson Reviewed-by: Emilio G. Cota --- target/i386/translate.c | 72 +++ 1 file changed, 48 insertions(+), 24

[Qemu-devel] [PATCH RFC 2/7] s390x: chsc nt2 events are pci-only

2017-07-07 Thread Cornelia Huck
The nt2 event class is pci-only and therefore implemented in the s390x pci code. Properly stub it out for non-pci builds. Signed-off-by: Cornelia Huck --- hw/s390x/s390-pci-bus.c | 4 ++-- hw/s390x/s390-pci-bus.h | 4 ++-- target/s390x/ioinst.c | 18 ++ 3 files changed, 22 in

[Qemu-devel] [PATCH v12 10/27] target/i386: [tcg] Port to tb_stop

2017-07-07 Thread Lluís Vilanova
Incrementally paves the way towards using the generic instruction translation loop. Signed-off-by: Lluís Vilanova Reviewed-by: Emilio G. Cota Reviewed-by: Richard Henderson --- target/i386/translate.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) diff --

Re: [Qemu-devel] [PATCH v22 23/30] qmp: add persistent flag to block-dirty-bitmap-add

2017-07-07 Thread Vladimir Sementsov-Ogievskiy
07.07.2017 10:54, Markus Armbruster wrote: QAPI schema review only... I apologize for its lateness. Vladimir Sementsov-Ogievskiy writes: Add optional 'persistent' flag to qmp command block-dirty-bitmap-add. Default is false. Signed-off-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Denis

[Qemu-devel] [PATCH RFC 3/7] s390x/sclp: properly guard pci-specific functions

2017-07-07 Thread Cornelia Huck
For non-pci builds, pci reconfiguration via sclp is not available. Don't indicate it in the sclp facilities and return an invalid command if the guest tries to issue pci configure/deconfigure. Signed-off-by: Cornelia Huck --- hw/s390x/sclp.c | 17 ++--- 1 file changed, 14 insertions(

  1   2   3   4   5   >