Re: [Qemu-devel] [PATCH v3 22/29] vhost+postcopy: Call wakeups

2018-03-02 Thread Peter Xu
On Fri, Feb 16, 2018 at 01:16:18PM +, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" > > Cause the vhost-user client to be woken up whenever: > a) We place a page in postcopy mode > b) We get a fault and the page has already been received > > Signed-off-by: Dr. Davi

[Qemu-devel] [PATCH] checkpatch: add a warning for basename/dirname

2018-03-02 Thread Julia Suvorova via Qemu-devel
Signed-off-by: Julia Suvorova --- scripts/checkpatch.pl | 5 + 1 file changed, 5 insertions(+) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 1b4b812..6c4fb42 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2584,6 +2584,11 @@ sub process {

Re: [Qemu-devel] [PATCH] checkpatch: add a warning for basename/dirname

2018-03-02 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1519978965-16865-1-git-send-email-jus...@mail.ru Subject: [Qemu-devel] [PATCH] checkpatch: add a warning for basename/dirname === TEST SCRIPT BEGIN === #!/bin/bash BASE=base

Re: [Qemu-devel] [PATCH v4] tap: setting error appropriately when calling net_init_tap_one()

2018-03-02 Thread Zhoujian (jay)
> -Original Message- > From: Jason Wang [mailto:jasow...@redhat.com] > Sent: Friday, March 02, 2018 2:30 PM > To: Zhoujian (jay) ; qemu-devel@nongnu.org > Cc: Huangweidong (C) ; m...@redhat.com; Gonglei > (Arei) ; imamm...@redhat.com; wangxin (U) > > Subject: Re: [Qemu-devel] [PATCH v4]

[Qemu-devel] [PATCH v3] vmgenid: allow VM Generation ID modification via QMP/HMP

2018-03-02 Thread Or Idgar
From: Or Idgar This patch allow changing the Virtual Machine Generation ID through QMP/HMP while the vm guest is running. the spec (http://go.microsoft.com/fwlink/?LinkId=260709) mentions that "when the generation ID changes, execute an ACPI Notify operation on the generation ID device". To test

Re: [Qemu-devel] [PATCH v3] vmgenid: allow VM Generation ID modification via QMP/HMP

2018-03-02 Thread no-reply
Hi, This series failed docker-mingw@fedora build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. Type: series Message-id: 20180302083720.25137-1-id...@virtualoco.com Subject: [Qemu-devel] [PATCH v3] vmgenid: allo

Re: [Qemu-devel] [PATCH v2] s390/ipl: only print boot menu error if -boot menu=on was specified

2018-03-02 Thread Christian Borntraeger
On 02/27/2018 08:35 PM, Collin L. Walling wrote: > It is possible that certain QEMU configurations may not > create an IPLB (such as when -kernel is provided). In > this case, a misleading error message will be printed > stating that the "boot menu is not supported for this > device type". > > To

Re: [Qemu-devel] [PATCH v3] vmgenid: allow VM Generation ID modification via QMP/HMP

2018-03-02 Thread no-reply
Hi, This series failed docker-quick@centos6 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. Type: series Message-id: 20180302083720.25137-1-id...@virtualoco.com Subject: [Qemu-devel] [PATCH v3] vmgenid: all

Re: [Qemu-devel] [PATCH v3] PPC: e500: Fix duplicate kernel load and device tree overlap

2018-03-02 Thread David Engraf
Am 02.03.2018 um 02:45 schrieb David Gibson: On Thu, Feb 15, 2018 at 10:36:00AM +0100, David Engraf wrote: This patch fixes an incorrect behavior when the -kernel argument has been specified without -bios. In this case the kernel was loaded twice. At address 32M as a raw image and afterwards by

Re: [Qemu-devel] [PATCH v3] vmgenid: allow VM Generation ID modification via QMP/HMP

2018-03-02 Thread no-reply
Hi, This series failed docker-build@min-glib build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. Type: series Message-id: 20180302083720.25137-1-id...@virtualoco.com Subject: [Qemu-devel] [PATCH v3] vmgenid: al

Re: [Qemu-devel] [PATCH] checkpatch: add a warning for basename/dirname

2018-03-02 Thread Paolo Bonzini
On 02/03/2018 09:22, Julia Suvorova wrote: > Signed-off-by: Julia Suvorova > --- > scripts/checkpatch.pl | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > index 1b4b812..6c4fb42 100755 > --- a/scripts/checkpatch.pl > +++ b/scripts/check

[Qemu-devel] [PATCH v5] tap: setting error appropriately when calling net_init_tap_one()

2018-03-02 Thread Jay Zhou
If netdev_add tap,id=net0,...,vhost=on failed in net_init_tap_one(), the followed up device_add virtio-net-pci,netdev=net0 will fail too, prints: TUNSETOFFLOAD ioctl() failed: Bad file descriptor TUNSETOFFLOAD ioctl() failed: Bad file descriptor The reason is that the fd of tap is closed wh

[Qemu-devel] [PATCH v3 0/3] virtio-balloon: free page hint reporting support

2018-03-02 Thread Wei Wang
This is the deivce part implementation to add a new feature, VIRTIO_BALLOON_F_FREE_PAGE_HINT to the virtio-balloon device. The device receives the guest free page hints from the driver and clears the corresponding bits in the dirty bitmap, so that those free pages are not transferred by the migrati

[Qemu-devel] [PATCH v3 2/3] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT

2018-03-02 Thread Wei Wang
The new feature enables the virtio-balloon device to receive hints of guest free pages from the free page vq. Callers call the free_page_start API to start the reporting, which creates a thread to poll for free page hints. The free_page_stop API stops the reporting and makes the thread exit. Signe

[Qemu-devel] [PATCH v3 3/3] migration: use the free page hint feature from balloon

2018-03-02 Thread Wei Wang
Start the free page optimization when the bulk stage starts. In case the guest is slow in reporting, actively stops it when the bulk stage ends. The optimization avoids sending guest free pages during the bulk stage. Currently, the optimization is added to precopy only. Signed-off-by: Wei Wang CC

[Qemu-devel] [PATCH v3 1/3] migration: API to clear bits of guest free pages from the dirty bitmap

2018-03-02 Thread Wei Wang
This patch adds an API to clear bits corresponding to guest free pages from the dirty bitmap. Spilt the free page block if it crosses the QEMU RAMBlock boundary. Signed-off-by: Wei Wang CC: Dr. David Alan Gilbert CC: Juan Quintela CC: Michael S. Tsirkin --- include/migration/misc.h | 2 ++ m

Re: [Qemu-devel] [qemu-s390x] [PATCH v4 1/3] s390x/sclp: proper support of larger send and receive masks

2018-03-02 Thread Christian Borntraeger
On 02/23/2018 06:42 PM, Claudio Imbrenda wrote: > Until 67915de9f0383ccf4a ("s390x/event-facility: variable-length event masks") > we only supported sclp event masks with a size of exactly 4 bytes, even > though the architecture allows the guests to set up sclp event masks > from 1 to 1021 bytes

Re: [Qemu-devel] [PATCH] hw/s390x/ipl: Bail out if the network bootloader can not be found

2018-03-02 Thread Cornelia Huck
On Tue, 27 Feb 2018 11:05:13 +0100 Thomas Huth wrote: > If QEMU fails to load 's390-netboot.img', the guest firmware currently > loops forever and just floods the console with "Network boot device > detected" messages. The code in ipl.c apparently already tried to stop > the VM with vm_stop() in

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3] PPC: e500: Fix duplicate kernel load and device tree overlap

2018-03-02 Thread Mark Cave-Ayland
On 02/03/18 08:53, David Engraf wrote: Am 02.03.2018 um 02:45 schrieb David Gibson: On Thu, Feb 15, 2018 at 10:36:00AM +0100, David Engraf wrote: This patch fixes an incorrect behavior when the -kernel argument has been specified without -bios. In this case the kernel was loaded twice. At add

Re: [Qemu-devel] [qemu-s390x] [PATCH v4 2/3] s390x/sclp: clean up sclp masks

2018-03-02 Thread Christian Borntraeger
On 02/23/2018 06:42 PM, Claudio Imbrenda wrote: > Introduce an sccb_mask_t to be used for SCLP event masks instead of just > unsigned int or uint32_t. This will allow later to extend the mask with > more ease. > > Signed-off-by: Claudio Imbrenda Certainly a sane cleanup, especially the use if t

Re: [Qemu-devel] [BUG] I/O thread segfault for QEMU on s390x

2018-03-02 Thread Stefan Hajnoczi
On Thu, Mar 01, 2018 at 09:33:35AM -0500, Farhan Ali wrote: > Hi, > > I have been noticing some segfaults for QEMU on s390x, and I have been > hitting this issue quite reliably (at least once in 10 runs of a test case). > The qemu version is 2.11.50, and I have systemd created coredumps > when thi

Re: [Qemu-devel] [PATCH v2 0/4] vl: introduce vm_shutdown()

2018-03-02 Thread Stefan Hajnoczi
On Fri, Mar 02, 2018 at 08:58:26AM +0800, Fam Zheng wrote: > On Thu, 03/01 14:54, Stefan Hajnoczi wrote: > > On Thu, Mar 01, 2018 at 09:15:17AM +0800, Fam Zheng wrote: > > > On Wed, 02/28 18:19, Stefan Hajnoczi wrote: > > > > v2: > > > > * Tackle the .ioeventfd_stop() vs vq handler race by removin

Re: [Qemu-devel] [PATCH v4 1/2] tpm: extend TPM emulator with state migration support

2018-03-02 Thread Marc-André Lureau
Hi Stefan On Thu, Mar 1, 2018 at 8:59 PM, Stefan Berger wrote: > Extend the TPM emulator backend device with state migration support. > > The external TPM emulator 'swtpm' provides a protocol over > its control channel to retrieve its state blobs. We implement > functions for getting and setting

Re: [Qemu-devel] [PATCH] checkpatch: add a warning for basename/dirname

2018-03-02 Thread Julia Suvorova via Qemu-devel
On 02.03.2018 11:56, Paolo Bonzini wrote: > On 02/03/2018 09:22, Julia Suvorova wrote: >> Signed-off-by: Julia Suvorova >> --- >> scripts/checkpatch.pl | 5 + >> 1 file changed, 5 insertions(+) >> >> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl >> index 1b4b812..6c4fb42 100755

Re: [Qemu-devel] [PATCH v3] vmgenid: allow VM Generation ID modification via QMP/HMP

2018-03-02 Thread Dr. David Alan Gilbert
* Or Idgar (id...@virtualoco.com) wrote: > From: Or Idgar Hi Or, > This patch allow changing the Virtual Machine Generation > ID through QMP/HMP while the vm guest is running. > the spec (http://go.microsoft.com/fwlink/?LinkId=260709) > mentions that "when the generation ID changes, execute an >

Re: [Qemu-devel] [PATCH v2 1/3] virtio-balloon: VIRTIO_BALLOON_F_FREE_PAGE_HINT

2018-03-02 Thread Wei Wang
On 02/07/2018 09:04 AM, Michael S. Tsirkin wrote: On Tue, Feb 06, 2018 at 07:08:17PM +0800, Wei Wang wrote: The new feature enables the virtio-balloon device to receive the hint of guest free pages from the free page vq, and clears the corresponding bits of the free page from the dirty bitmap, s

Re: [Qemu-devel] [PATCH v2] s390/ipl: only print boot menu error if -boot menu=on was specified

2018-03-02 Thread Cornelia Huck
On Tue, 27 Feb 2018 14:35:21 -0500 "Collin L. Walling" wrote: > It is possible that certain QEMU configurations may not > create an IPLB (such as when -kernel is provided). In > this case, a misleading error message will be printed > stating that the "boot menu is not supported for this > device

Re: [Qemu-devel] [PATCHv1 11/14] target/s390x: convert to DisasContextBase

2018-03-02 Thread David Hildenbrand
On 01.03.2018 23:53, Emilio G. Cota wrote: > Notes: > > - Did not convert {num,max}_insns and is_jmp, since the corresponding > code will go away in the next patch. > > - Avoided a checkpatch error in use_exit_tb. > > - As suggested by David, (1) Drop ctx.pc and use > ctx.base.pc_next instea

Re: [Qemu-devel] [PATCH v4 2/2] tpm: extend TPM TIS with state migration support

2018-03-02 Thread Marc-André Lureau
Hi On Thu, Mar 1, 2018 at 8:59 PM, Stefan Berger wrote: > Extend the TPM TIS interface with state migration support. > > We need to synchronize with the backend thread to make sure that a command > being processed by the external TPM emulator has completed and its > response been received. > > Si

Re: [Qemu-devel] [qemu-s390x] [PATCH v4 3/3] s390x/sclp: extend SCLP event masks to 64 bits

2018-03-02 Thread Christian Borntraeger
On 02/23/2018 06:42 PM, Claudio Imbrenda wrote: > Extend the SCLP event masks to 64 bits. > > Notice that using any of the new bits results in a state that cannot be > migrated to an older version. > > Signed-off-by: Claudio Imbrenda > --- > hw/s390x/event-facility.c | 56 > +

Re: [Qemu-devel] [PATCH v5] tap: setting error appropriately when calling net_init_tap_one()

2018-03-02 Thread Jason Wang
On 2018年03月02日 17:04, Jay Zhou wrote: If netdev_add tap,id=net0,...,vhost=on failed in net_init_tap_one(), the followed up device_add virtio-net-pci,netdev=net0 will fail too, prints: TUNSETOFFLOAD ioctl() failed: Bad file descriptor TUNSETOFFLOAD ioctl() failed: Bad file descriptor T

Re: [Qemu-devel] [PATCHv1 12/14] target/s390x: convert to TranslatorOps

2018-03-02 Thread David Hildenbrand
On 01.03.2018 23:53, Emilio G. Cota wrote: > Note: I looked into dropping dc->do_debug. However, I don't see > an easy way to do it given that TOO_MANY is also valid > when we just translate more than max_insns. Thus, the check > for do_debug in "case DISAS_PC_CC_UPDATED" would still need > additio

Re: [Qemu-devel] [PATCH qemu v2] qmp: Add qom-list-properties to list QOM object properties

2018-03-02 Thread Andrea Bolognani
On Wed, 2018-02-28 at 15:42 +0100, Andrea Bolognani wrote: > I'm going to implement support for this new command in libvirt > and report back if I run into any issue with the current design, > but it looks very good so far. I've posted the RFC implementation to libvir-list: https://www.redhat.c

Re: [Qemu-devel] [PATCHv1 00/14] Translation loop conversion for sh4/sparc/mips/s390x/openrisc targets

2018-03-02 Thread Cornelia Huck
On Thu, 1 Mar 2018 17:53:44 -0500 "Emilio G. Cota" wrote: > [ What is this all about? See this message: > http://lists.gnu.org/archive/html/qemu-devel/2018-02/msg04785.html ] > > Merged the separate patchsets I sent in the last couple of weeks into > one set. This will be easier to merge sinc

Re: [Qemu-devel] [PATCHv1 10/14] target/s390x: convert to DisasJumpType

2018-03-02 Thread Cornelia Huck
On Thu, 1 Mar 2018 17:53:54 -0500 "Emilio G. Cota" wrote: > The only non-trivial modification is the use of DISAS_TOO_MANY > in the same way is used by the generic translation loop. > > Reviewed-by: David Hildenbrand > Reviewed-by: Richard Henderson > Cc: David Hildenbrand > Cc: Cornelia Huc

Re: [Qemu-devel] [PATCHv1 11/14] target/s390x: convert to DisasContextBase

2018-03-02 Thread Cornelia Huck
On Thu, 1 Mar 2018 17:53:55 -0500 "Emilio G. Cota" wrote: > Notes: > > - Did not convert {num,max}_insns and is_jmp, since the corresponding > code will go away in the next patch. > > - Avoided a checkpatch error in use_exit_tb. > > - As suggested by David, (1) Drop ctx.pc and use > ctx.b

Re: [Qemu-devel] [PATCHv1 12/14] target/s390x: convert to TranslatorOps

2018-03-02 Thread Cornelia Huck
On Thu, 1 Mar 2018 17:53:56 -0500 "Emilio G. Cota" wrote: > Note: I looked into dropping dc->do_debug. However, I don't see > an easy way to do it given that TOO_MANY is also valid > when we just translate more than max_insns. Thus, the check > for do_debug in "case DISAS_PC_CC_UPDATED" would st

Re: [Qemu-devel] [PATCH v4 1/2] tpm: extend TPM emulator with state migration support

2018-03-02 Thread Dr. David Alan Gilbert
* Marc-André Lureau (marcandre.lur...@gmail.com) wrote: > Hi Stefan > > On Thu, Mar 1, 2018 at 8:59 PM, Stefan Berger > wrote: > > Extend the TPM emulator backend device with state migration support. > > > > The external TPM emulator 'swtpm' provides a protocol over > > its control channel to ret

[Qemu-devel] ping Re: [PATCH v2 0/2] block latency histogram

2018-03-02 Thread Vladimir Sementsov-Ogievskiy
ping 07.02.2018 15:50, Vladimir Sementsov-Ogievskiy wrote: v2: 01: add block_latency_histogram_clear() 02: fix spelling (sorry =() some rewordings remove histogram if latency parameter unspecified Vladimir Sementsov-Ogievskiy (2): block/accounting: introduce latency histogram qa

Re: [Qemu-devel] [PATCH v1 1/1] target/arm: Fix the A53 L2CTLR typo

2018-03-02 Thread Peter Maydell
On 2 March 2018 at 04:34, Alistair Francis wrote: > On Thu, Mar 1, 2018 at 4:20 PM, Alistair Francis > wrote: >> The cortex A53 TRM specifices that bits 24 and 25 of the L2CTLR register >> specify the number of cores present and not the number of processors. We >> have correctly been reporting th

Re: [Qemu-devel] [PATCH v1 1/1] target/arm: Fix the A53 L2CTLR typo

2018-03-02 Thread Peter Maydell
On 2 March 2018 at 10:29, Peter Maydell wrote: > On 2 March 2018 at 04:34, Alistair Francis wrote: >> target/arm: Report the number of cores in the cluster >> >> Previously we assumed that we only has a single cluster, which meant we >> could get away with reporting smp_cpus to the gu

Re: [Qemu-devel] [PATCHv5 4/5] qemu-doc: deprecate query-cpus

2018-03-02 Thread Cornelia Huck
On Tue, 27 Feb 2018 20:06:55 +0100 Cornelia Huck wrote: > On Tue, 27 Feb 2018 14:41:38 +0100 > Viktor Mihajlovski wrote: > > > On 19.02.2018 15:57, Cornelia Huck wrote: > > > On Fri, 16 Feb 2018 17:08:40 +0100 > > > Viktor Mihajlovski wrote: > > > > > >> Start the deprecation period for

Re: [Qemu-devel] [PATCH v2 0/4] tricore: added small features + fixed wrong masks

2018-03-02 Thread Bastian Koppelmann
Hi David, On 03/01/2018 04:56 PM, David Brenken wrote: > From: David Brenken > > Hi Bastian, > > thank you for your feedback and sorry for the late reply. > > Changes from v1: > * Removed OPC1_16_SB_JNE instruction. > * Added CPU feature checks to new instructions. > * Renamed ICR.IE and PC

[Qemu-devel] [PATCH v2] checkpatch: add a warning for basename/dirname

2018-03-02 Thread Julia Suvorova via Qemu-devel
g_path_get_* do the same as g_strdup(basename/dirname(...)) but without modifying the argument. Signed-off-by: Julia Suvorova --- scripts/checkpatch.pl | 5 + 1 file changed, 5 insertions(+) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 1b4b812..a88af61 100755 --- a/scrip

Re: [Qemu-devel] [PATCH v2] checkpatch: add a warning for basename/dirname

2018-03-02 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1519987399-19160-1-git-send-email-jus...@mail.ru Subject: [Qemu-devel] [PATCH v2] checkpatch: add a warning for basename/dirname === TEST SCRIPT BEGIN === #!/bin/bash BASE=b

Re: [Qemu-devel] [PATCH v2 1/4] tricore: added some missing cpu instructions

2018-03-02 Thread Bastian Koppelmann
On 03/01/2018 04:56 PM, David Brenken wrote: > From: David Brenken > > Signed-off-by: David Brenken > Signed-off-by: Florian Artmeier > Signed-off-by: Georg Hofstetter > --- > target/tricore/translate.c | 27 +++ > target/tricore/tricore-opcodes.h | 3 +++ > 2 f

Re: [Qemu-devel] [PATCH v2 3/4] tricore: renamed masking of IE

2018-03-02 Thread Bastian Koppelmann
On 03/01/2018 04:56 PM, David Brenken wrote: > From: David Brenken > > Signed-off-by: David Brenken > Signed-off-by: Florian Artmeier > Signed-off-by: Georg Hofstetter > --- > target/tricore/cpu.h | 3 ++- > target/tricore/op_helper.c | 17 + > target/tricore/translate.

Re: [Qemu-devel] [PATCH v2 2/4] tricore: added CORE_ID

2018-03-02 Thread Bastian Koppelmann
On 03/01/2018 04:56 PM, David Brenken wrote: > From: David Brenken > > Signed-off-by: David Brenken > Signed-off-by: Florian Artmeier > Signed-off-by: Georg Hofstetter > --- > target/tricore/cpu.h| 1 + > target/tricore/csfr.def | 1 + > 2 files changed, 2 insertions(+) > I gave you a R

Re: [Qemu-devel] [edk2] [PATCH 2/7] ovmf: link with Tcg2ConfigPei module

2018-03-02 Thread Laszlo Ersek
On 03/01/18 15:59, Marc-André Lureau wrote: > Hi > > On Fri, Feb 23, 2018 at 6:31 PM, Laszlo Ersek wrote: >> (6) Now, I realize Tcg2Pei *apparently* depends on >> gEfiPeiReadOnlyVariable2PpiGuid (i.e., read-only variable access in the >> PEI phase) as well. That's a bug in the INF file (the [dep

Re: [Qemu-devel] [PATCH v2 05/15] qio: refactor net listener source operations

2018-03-02 Thread Daniel P . Berrangé
On Fri, Mar 02, 2018 at 11:58:52AM +0800, Peter Xu wrote: > On Thu, Mar 01, 2018 at 10:47:17AM +, Daniel P. Berrangé wrote: > > On Thu, Mar 01, 2018 at 04:44:28PM +0800, Peter Xu wrote: > > > Three functions are abstracted from the old code: > > > > > > - qio_net_listener_source_add(): create

Re: [Qemu-devel] [edk2] [PATCH 4/7] ovmf: link with Tcg2Pei module

2018-03-02 Thread Laszlo Ersek
On 03/01/18 16:08, Marc-André Lureau wrote: > Hi > > On Mon, Feb 26, 2018 at 10:38 AM, Laszlo Ersek wrote: >> On 02/23/18 14:23, marcandre.lur...@redhat.com wrote: >>> From: Marc-André Lureau >>> >>> This module will initialize TPM device, measure reported FVs and BIOS >>> version. >>> >>> CC: L

Re: [Qemu-devel] [PATCH] use g_path_get_basename instead of basename

2018-03-02 Thread Paolo Bonzini
On 01/03/2018 17:20, Alex Williamson wrote: >> basename(3) and dirname(3) modify their argument and may return >> pointers to statically allocated memory which may be overwritten by >> subsequent calls. >> g_path_get_basename and g_path_get_dirname have no such issues, and >> therefore more prefera

Re: [Qemu-devel] [PATCH v3] vmgenid: allow VM Generation ID modification via QMP/HMP

2018-03-02 Thread Daniel P . Berrangé
On Fri, Mar 02, 2018 at 10:37:20AM +0200, Or Idgar wrote: > From: Or Idgar > > This patch allow changing the Virtual Machine Generation > ID through QMP/HMP while the vm guest is running. > the spec (http://go.microsoft.com/fwlink/?LinkId=260709) > mentions that "when the generation ID changes, e

Re: [Qemu-devel] [PATCH v2 08/15] chardev: allow telnet gsource to switch gcontext

2018-03-02 Thread Daniel P . Berrangé
On Thu, Mar 01, 2018 at 06:16:53PM +0100, Paolo Bonzini wrote: > On 01/03/2018 16:46, Daniel P. Berrangé wrote: > > On Thu, Mar 01, 2018 at 04:44:31PM +0800, Peter Xu wrote: > >> It was originally created by qio_channel_add_watch() so it's always > >> assigning the task to main context. Now we use

[Qemu-devel] [PULL 01/39] xlnx-zynqmp-rtc: Initial commit

2018-03-02 Thread Peter Maydell
From: Alistair Francis Initial commit of the ZynqMP RTC device. Signed-off-by: Alistair Francis Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Peter Maydell --- hw/timer/Makefile.objs | 1 + include/hw/timer/xlnx-zynqmp-rtc.h | 84 +++ hw/timer/xlnx-zynqmp-rtc.

[Qemu-devel] [PULL 07/39] hw/arm/armv7m: Honour CPU's address space for image loads

2018-03-02 Thread Peter Maydell
Instead of loading guest images to the system address space, use the CPU's address space. This is important if we're trying to load the file to memory or via an alias memory region that is provided by an SoC object and thus not mapped into the system address space. Signed-off-by: Peter Maydell R

[Qemu-devel] [PULL 00/39] target-arm queue

2018-03-02 Thread Peter Maydell
aging (2018-03-01 18:46:41 +) are available in the Git repository at: git://git.linaro.org/people/pmaydell/qemu-arm.git tags/pull-target-arm-20180302 for you to fetch changes up to e66a67bf28e1b4fce2e3d72a2610dbd48d9d3078: target/arm: Enable ARM_FEATURE_V8_FCMA (2018-03-02

[Qemu-devel] [PULL 03/39] xlnx-zynqmp: Connect the RTC device

2018-03-02 Thread Peter Maydell
From: Alistair Francis Signed-off-by: Alistair Francis Reviewed-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Peter Maydell --- include/hw/arm/xlnx-zynqmp.h | 2 ++ hw/arm/xlnx-zynqmp.c | 14 ++ 2 files changed, 16 insertions(+) diff --git a/inclu

[Qemu-devel] [PULL 06/39] hw/arm/boot: Honour CPU's address space for image loads

2018-03-02 Thread Peter Maydell
Instead of loading kernels, device trees, and the like to the system address space, use the CPU's address space. This is important if we're trying to load the file to memory or via an alias memory region that is provided by an SoC object and thus not mapped into the system address space. Signed-of

[Qemu-devel] [PULL 04/39] decodetree: Propagate return value from translate subroutines

2018-03-02 Thread Peter Maydell
From: Richard Henderson Allow the translate subroutines to return false for invalid insns. At present we can of course invoke an invalid insn exception from within the translate subroutine, but in the short term this consolidates code. In the long term it would allow the decodetree language to s

[Qemu-devel] [PULL 02/39] xlnx-zynqmp-rtc: Add basic time support

2018-03-02 Thread Peter Maydell
From: Alistair Francis Allow the guest to determine the time set from the QEMU command line. This includes adding a trace event to debug the new time. Signed-off-by: Alistair Francis Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- include/hw/

[Qemu-devel] [PULL 12/39] target/arm: Add Cortex-M33

2018-03-02 Thread Peter Maydell
Add a Cortex-M33 definition. The M33 is an M profile CPU which implements the ARM v8M architecture, including the M profile Security Extension. Signed-off-by: Peter Maydell Reviewed-by: Richard Henderson Message-id: 20180220180325.29818-9-peter.mayd...@linaro.org --- target/arm/cpu.c | 31 +

[Qemu-devel] [PULL 20/39] hw/misc/iotkit-secctl: Add handling for PPCs

2018-03-02 Thread Peter Maydell
The IoTKit Security Controller includes various registers that expose to software the controls for the Peripheral Protection Controllers in the system. Implement these. Signed-off-by: Peter Maydell Reviewed-by: Richard Henderson Message-id: 20180220180325.29818-17-peter.mayd...@linaro.org --- i

[Qemu-devel] [PULL 15/39] qdev: Add new qdev_init_gpio_in_named_with_opaque()

2018-03-02 Thread Peter Maydell
The function qdev_init_gpio_in_named() passes the DeviceState pointer as the opaque data pointor for the irq handler function. Usually this is what you want, but in some cases it would be helpful to use some other data pointer. Add a new function qdev_init_gpio_in_named_with_opaque() which allows

[Qemu-devel] [PULL 09/39] armv7m: Forward idau property to CPU object

2018-03-02 Thread Peter Maydell
Create an "idau" property on the armv7m container object which we can forward to the CPU object. Annoyingly, we can't use object_property_add_alias() because the CPU object we want to forward to doesn't exist until the armv7m container is realized. Signed-off-by: Peter Maydell Reviewed-by: Richar

[Qemu-devel] [PULL 05/39] loader: Add new load_ramdisk_as()

2018-03-02 Thread Peter Maydell
Add a function load_ramdisk_as() which behaves like the existing load_ramdisk() but allows the caller to specify the AddressSpace to use. This matches the pattern we have already for various other loader functions. Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Ric

[Qemu-devel] [PULL 10/39] target/arm: Define init-svtor property for the reset secure VTOR value

2018-03-02 Thread Peter Maydell
The Cortex-M33 allows the system to specify the reset value of the secure Vector Table Offset Register (VTOR) by asserting config signals. In particular, guest images for the MPS2 AN505 board rely on the MPS2's initial VTOR being correct for that board. Implement a QEMU property so board and SoC co

[Qemu-devel] [PULL 25/39] target/arm: Refactor disas_simd_indexed decode

2018-03-02 Thread Peter Maydell
From: Richard Henderson Include the U bit in the switches rather than testing separately. Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell Message-id: 20180228193125.20577-3-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate-a64.c | 129

[Qemu-devel] [PULL 14/39] include/hw/or-irq.h: Add missing include guard

2018-03-02 Thread Peter Maydell
The or-irq.h header file is missing the customary guard against multiple inclusion, which means compilation fails if it gets included twice. Fix the omission. Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-id: 20180220180325.29818-11-pete

[Qemu-devel] [PULL 08/39] target/arm: Define an IDAU interface

2018-03-02 Thread Peter Maydell
In v8M, the Implementation Defined Attribution Unit (IDAU) is a small piece of hardware typically implemented in the SoC which provides board or SoC specific security attribution information for each address that the CPU performs MPU/SAU checks on. For QEMU, we model this with a QOM interface which

[Qemu-devel] [PULL 18/39] hw/misc/tz-ppc: Model TrustZone peripheral protection controller

2018-03-02 Thread Peter Maydell
Add a model of the TrustZone peripheral protection controller (PPC), which is used to gate transactions to non-TZ-aware peripherals so that secure software can configure them to not be accessible to non-secure software. Signed-off-by: Peter Maydell Reviewed-by: Richard Henderson Message-id: 2018

[Qemu-devel] [PULL 11/39] armv7m: Forward init-svtor property to CPU object

2018-03-02 Thread Peter Maydell
Create an "init-svtor" property on the armv7m container object which we can forward to the CPU object. Signed-off-by: Peter Maydell Reviewed-by: Richard Henderson Message-id: 20180220180325.29818-8-peter.mayd...@linaro.org --- include/hw/arm/armv7m.h | 2 ++ hw/arm/armv7m.c | 9

[Qemu-devel] [PULL 29/39] target/arm: Decode aa64 armv8.1 scalar/vector x indexed element

2018-03-02 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Alex Bennée Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-7-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate-a64.c | 29 + 1 file changed, 29 insertions(+) diff --git a/targ

[Qemu-devel] [PULL 21/39] hw/misc/iotkit-secctl: Add remaining simple registers

2018-03-02 Thread Peter Maydell
Add remaining easy registers to iotkit-secctl: * NSCCFG just routes its two bits out to external GPIO lines * BRGINSTAT/BRGINTCLR/BRGINTEN can be dummies, because QEMU's bus fabric can never report errors Signed-off-by: Peter Maydell Message-id: 20180220180325.29818-18-peter.mayd...@linaro.o

[Qemu-devel] [PULL 13/39] hw/misc/unimp: Move struct to header file

2018-03-02 Thread Peter Maydell
Move the definition of the struct for the unimplemented-device from unimp.c to unimp.h, so that users can embed the struct in their own device structs if they prefer. Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-id: 20180220180325.29818

[Qemu-devel] [PULL 26/39] target/arm: Refactor disas_simd_indexed size checks

2018-03-02 Thread Peter Maydell
From: Richard Henderson The integer size check was already outside of the opcode switch; move the floating-point size check outside as well. Unify the size vs index adjustment between fp and integer paths. Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell Message-id: 20180228193125.

[Qemu-devel] [PULL 27/39] target/arm: Decode aa64 armv8.1 scalar three same extra

2018-03-02 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Alex Bennée Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-5-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/Makefile.objs | 2 +- target/arm/helper.h| 4 ++ target/arm/translate-a64.c | 84 +++

[Qemu-devel] [PULL 19/39] hw/misc/iotkit-secctl: Arm IoT Kit security controller initial skeleton

2018-03-02 Thread Peter Maydell
The Arm IoT Kit includes a "security controller" which is largely a collection of registers for controlling the PPCs and other bits of glue in the system. This commit provides the initial skeleton of the device, implementing just the ID registers, and a couple of read-only read-as-zero registers.

[Qemu-devel] [PULL 17/39] hw/misc/mps2-fpgaio: FPGA control block for MPS2 AN505

2018-03-02 Thread Peter Maydell
The MPS2 AN505 FPGA image includes a "FPGA control block" which is a small set of registers handling LEDs, buttons and some counters. Signed-off-by: Peter Maydell Reviewed-by: Richard Henderson Message-id: 20180220180325.29818-14-peter.mayd...@linaro.org --- hw/misc/Makefile.objs |

[Qemu-devel] [PULL 23/39] mps2-an505: New board model: MPS2 with AN505 Cortex-M33 FPGA image

2018-03-02 Thread Peter Maydell
Define a new board model for the MPS2 with an AN505 FPGA image containing a Cortex-M33. Since the FPGA images for TrustZone cores (AN505, and the similar AN519 for Cortex-M23) have a significantly different layout of devices to the non-TrustZone images, we use a new source file rather than shoehorn

[Qemu-devel] [PULL 22/39] hw/arm/iotkit: Model Arm IOT Kit

2018-03-02 Thread Peter Maydell
Model the Arm IoT Kit documented in http://infocenter.arm.com/help/index.jsp?topic=/com.arm.doc.ecm0601256/index.html The Arm IoT Kit is a subsystem which includes a CPU and some devices, and is intended be extended by adding extra devices to form a complete system. It is used in the MPS2 board's

[Qemu-devel] [PULL 16/39] hw/core/split-irq: Device that splits IRQ lines

2018-03-02 Thread Peter Maydell
In some board or SoC models it is necessary to split a qemu_irq line so that one input can feed multiple outputs. We currently have qemu_irq_split() for this, but that has several deficiencies: * it can only handle splitting a line into two * it unavoidably leaks memory, so it can't be used i

[Qemu-devel] [PULL 28/39] target/arm: Decode aa64 armv8.1 three same extra

2018-03-02 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-6-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/helper.h| 9 + target/arm/translate-a64.c | 83 +

[Qemu-devel] [PULL 33/39] target/arm: Add ARM_FEATURE_V8_FCMA

2018-03-02 Thread Peter Maydell
From: Richard Henderson Not enabled anywhere yet. Reviewed-by: Alex Bennée Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-11-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/cpu.h | 1 + linux-user/elfload.c | 1 + 2 files changed, 2 insertions(

[Qemu-devel] [PULL 31/39] target/arm: Decode aa32 armv8.1 two reg and a scalar

2018-03-02 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-9-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate.c | 46 ++ 1 file changed, 42 insertio

[Qemu-devel] [PULL 24/39] target/arm: Add ARM_FEATURE_V8_RDM

2018-03-02 Thread Peter Maydell
From: Richard Henderson Not enabled anywhere yet. Reviewed-by: Alex Bennée Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-2-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/cpu.h | 1 + linux-user/elfload.c | 1 + 2 f

[Qemu-devel] [PULL 34/39] target/arm: Decode aa64 armv8.3 fcadd

2018-03-02 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Alex Bennée Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-12-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/helper.h| 7 target/arm/translate-a64.c | 48 ++- target/arm/vec_hel

[Qemu-devel] [PULL 36/39] target/arm: Decode aa32 armv8.3 3-same

2018-03-02 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell Message-id: 20180228193125.20577-14-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate.c | 68 ++ 1 file changed, 68 insertions(+

[Qemu-devel] [PULL 30/39] target/arm: Decode aa32 armv8.1 three same

2018-03-02 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-8-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate.c | 86 +++--- 1 file changed, 67 insertions(+)

[Qemu-devel] [PULL 35/39] target/arm: Decode aa64 armv8.3 fcmla

2018-03-02 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-13-richard.hender...@linaro.org Signed-off-by: Peter Maydell [PMM: renamed e1/e2/e3/e4 to use the same naming as the version of the pseudocode in the Arm ARM] Reviewed-by: Peter Maydell --- target/arm/h

[Qemu-devel] [PULL 39/39] target/arm: Enable ARM_FEATURE_V8_FCMA

2018-03-02 Thread Peter Maydell
From: Richard Henderson Enable it for the "any" CPU used by *-linux-user. Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell Message-id: 20180228193125.20577-17-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/cpu.c | 1 + target/arm/cpu64.c | 1 + 2 files

[Qemu-devel] [PULL 32/39] target/arm: Enable ARM_FEATURE_V8_RDM

2018-03-02 Thread Peter Maydell
From: Richard Henderson Enable it for the "any" CPU used by *-linux-user. Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell Message-id: 20180228193125.20577-10-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/cpu.c | 1 + target/arm/cpu64.c | 1 + 2 files

[Qemu-devel] [PULL 38/39] target/arm: Decode t32 simd 3reg and 2reg_scalar extension

2018-03-02 Thread Peter Maydell
From: Richard Henderson Happily, the bits are in the same places compared to a32. Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-16-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/translate.c | 14 +- 1 file c

Re: [Qemu-devel] [PATCH v10 14/28] hmp: add 'info sev' command

2018-03-02 Thread Dr. David Alan Gilbert
* Brijesh Singh (brijesh.si...@amd.com) wrote: > The command can be used to show the SEV information when memory > encryption is enabled on AMD platform. > > Cc: Eric Blake > Cc: "Daniel P. Berrangé" > Cc: "Dr. David Alan Gilbert" > Cc: Markus Armbruster > Reviewed-by: "Dr. David Alan Gilbert"

Re: [Qemu-devel] [Qemu-ppc] [PATCH v3] PPC: e500: Fix duplicate kernel load and device tree overlap

2018-03-02 Thread David Engraf
Am 02.03.2018 um 10:11 schrieb Mark Cave-Ayland: On 02/03/18 08:53, David Engraf wrote: Am 02.03.2018 um 02:45 schrieb David Gibson: On Thu, Feb 15, 2018 at 10:36:00AM +0100, David Engraf wrote: This patch fixes an incorrect behavior when the -kernel argument has been specified without -bios.

Re: [Qemu-devel] [edk2] [PATCH 6/7] ovmf: link with Tcg2ConfigDxe module

2018-03-02 Thread Laszlo Ersek
On 03/01/18 17:59, Stefan Berger wrote: > On 02/26/2018 04:58 AM, Laszlo Ersek wrote: >> On 02/23/18 14:23, marcandre.lur...@redhat.com wrote: >>> From: Marc-André Lureau >>> >>> The module allows to tweak and interact with the TPM. Note that many >>> actions are broken due to implementation of qe

[Qemu-devel] [PULL 37/39] target/arm: Decode aa32 armv8.3 2-reg-index

2018-03-02 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20180228193125.20577-15-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/translate.c | 61 ++ 1 file changed, 61 insertions(+

Re: [Qemu-devel] [PATCH v2 03/15] qio: introduce qio_channel_add_watch_{full|source}

2018-03-02 Thread Paolo Bonzini
On 02/03/2018 04:54, Peter Xu wrote: > On Thu, Mar 01, 2018 at 06:13:06PM +0100, Paolo Bonzini wrote: >> On 01/03/2018 09:44, Peter Xu wrote: >>> + * qio_channel_add_watch_source: >>> + * @ioc: the channel object >>> + * @condition: the I/O condition to monitor >>> + * @func: callback to invoke whe

Re: [Qemu-devel] [PATCH v2 07/15] qio/chardev: update net listener gcontext

2018-03-02 Thread Paolo Bonzini
On 02/03/2018 05:26, Peter Xu wrote: > Frankly speaking I was a bit confused when I started to read > chardev/qio codes with so many hooks, e.g., when I saw: > > qio_net_listener_set_client_func(s->listener, tcp_chr_accept, > chr, NULL); > > I totally ha

Re: [Qemu-devel] [PULL 00/13] virtio, vhost, pci, pc: features, fixes and cleanups

2018-03-02 Thread Peter Maydell
On 1 March 2018 at 16:45, Michael S. Tsirkin wrote: > The following changes since commit 8cb340c613ee3e626b070e0429c589f8a60ac657: > > Merge remote-tracking branch 'remotes/famz/tags/staging-pull-request' into > staging (2018-03-01 12:32:31 +) > > are available in the git repository at: > >

  1   2   3   4   >