Re: [Qemu-devel] [PATCH v2 04/13] spapr/xive: add state synchronization with KVM

2019-03-13 Thread Cédric Le Goater
On 3/14/19 3:10 AM, David Gibson wrote: > On Mon, Mar 11, 2019 at 09:41:12PM +0100, Cédric Le Goater wrote: >> On 2/26/19 1:01 AM, David Gibson wrote: >>> On Fri, Feb 22, 2019 at 02:13:13PM +0100, Cédric Le Goater wrote: This extends the KVM XIVE device backend with 'synchronize_state' me

Re: [Qemu-devel] [PATCH] ppc/xics/spapr: Fix H_IPOLL implementation

2019-03-13 Thread Greg Kurz
On Thu, 14 Mar 2019 07:38:55 +0100 Cédric Le Goater wrote: > From: Benjamin Herrenschmidt > > H_IPOLL takes the CPU# of the processor to poll as an argument, > it doesn't operate on self. > True. > Signed-off-by: Benjamin Herrenschmidt > Signed-off-by: Cédric Le Goater > --- Reviewed-by:

Re: [Qemu-devel] [PATCH RESEND v4] drive-mirror: add incremental mode

2019-03-13 Thread Vladimir Sementsov-Ogievskiy
On 13.03.2019 19:52, John Snow wrote: > > > On 3/12/19 12:32 PM, Vladimir Sementsov-Ogievskiy wrote: >> 12.03.2019 18:58, John Snow wrote: >>> >>> >>> On 3/12/19 9:43 AM, Eric Blake wrote: On 3/12/19 3:19 AM, Vladimir Sementsov-Ogievskiy wrote: >> So one important point about incr

[Qemu-devel] [PATCH] ppc/xics/spapr: Fix H_IPOLL implementation

2019-03-13 Thread Cédric Le Goater
From: Benjamin Herrenschmidt H_IPOLL takes the CPU# of the processor to poll as an argument, it doesn't operate on self. Signed-off-by: Benjamin Herrenschmidt Signed-off-by: Cédric Le Goater --- hw/intc/xics_spapr.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/h

Re: [Qemu-devel] Maintainers, please tell us how to boot your machines!

2019-03-13 Thread Mark Cave-Ayland
On 12/03/2019 17:36, Markus Armbruster wrote: > = hw/ppc/prep.c = > "Hervé Poussineau" (maintainer:PReP) > David Gibson (maintainer:PowerPC) > qemu-...@nongnu.org (open list:PReP) Note that -M prep has been formally deprecated and so we're really only interested in -M 40p going

Re: [Qemu-devel] [PATCH] Clean up includes

2019-03-13 Thread Markus Armbruster
Eric Blake writes: > On 3/13/19 11:28 AM, Markus Armbruster wrote: >> Clean up includes so that osdep.h is included first and headers >> which it implies are not included manually. >> >> This commit was created with scripts/clean-includes, with the changes >> to the following files manually reve

Re: [Qemu-devel] Maintainers, please tell us how to boot your machines!

2019-03-13 Thread Gerd Hoffmann
> I could not yet boot Linux on it but I'm interested in this machine because > I want to use it to cross-check my patches so don't drop it just yet. > Currently it seems to panic because of my newly added radeon emulation that > I need to fix: Doesn't look like an issue in ati-vga. It doesn't co

Re: [Qemu-devel] [PATCH] Clean up includes

2019-03-13 Thread Markus Armbruster
BALATON Zoltan writes: > On Wed, 13 Mar 2019, Markus Armbruster wrote: >> BALATON Zoltan writes: >>> On Wed, 13 Mar 2019, Markus Armbruster wrote: Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was creat

[Qemu-devel] [PATCH 2/2] virtio-gpu: clear command queue on reset

2019-03-13 Thread Gerd Hoffmann
It was never correct to not clear them. Due to commit "3912e66a3feb virtio-vga: fix reset." this became more obvious though. The virtio rings get properly reset now, and trying to process the stale commands will trigger an assert in the virtio core. Signed-off-by: Gerd Hoffmann --- hw/display/

[Qemu-devel] [PATCH 1/2] virtio-gpu: delay virglrenderer reset when blocked.

2019-03-13 Thread Gerd Hoffmann
If renderer_blocked is set do not call virtio_gpu_virgl_reset(). Instead set a flag indicating that virglrenderer needs a reset. When renderer_blocked gets cleared do the actual reset call. Without this we can trigger an assert in spice due to calling spice_qxl_gl_scanout() while another operation

[Qemu-devel] [PATCH 0/2] virtio-gpu: more reset fixes.

2019-03-13 Thread Gerd Hoffmann
Gerd Hoffmann (2): virtio-gpu: delay virglrenderer reset when blocked. virtio-gpu: clear command queue on reset include/hw/virtio/virtio-gpu.h | 1 + hw/display/virtio-gpu.c| 19 ++- 2 files changed, 19 insertions(+), 1 deletion(-) -- 2.18.1

[Qemu-devel] [PATCH v2 14/14] target/arm: Implement ARMv8.5-RNG

2019-03-13 Thread Richard Henderson
Cc: Peter Maydell Signed-off-by: Richard Henderson --- target/arm/cpu.h| 5 + target/arm/cpu64.c | 1 + target/arm/helper.c | 32 3 files changed, 38 insertions(+) diff --git a/target/arm/cpu.h b/target/arm/cpu.h index 5f23c62132..aaa9e02e78 100644 --

[Qemu-devel] [PATCH v2 09/14] util: Use qcrypto_random_bytes for qemu_getrandom

2019-03-13 Thread Richard Henderson
When not requesting deterministic bytes, use our existing crypto. Signed-off-by: Richard Henderson --- util/random.c | 61 --- 1 file changed, 48 insertions(+), 13 deletions(-) diff --git a/util/random.c b/util/random.c index 467c987a66..c2628ace8

[Qemu-devel] [PATCH v2 06/14] crypto: Change the qcrypto_random_bytes buffer type to void*

2019-03-13 Thread Richard Henderson
Using uint8_t* merely requires useless casts for use with other types to be filled with randomness. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- include/crypto/random.h | 2 +- crypto/random-gcrypt.c | 2 +- crypto/random-gnutls.c | 2 +- crypto/random-platform.c | 4 ++-- 4

[Qemu-devel] [PATCH v2 12/14] linux-user/aarch64: Use qemu_getrandom for arm_init_pauth_key

2019-03-13 Thread Richard Henderson
Use a better interface for random numbers than rand * 3. Cc: Laurent Vivier Cc: Peter Maydell Signed-off-by: Richard Henderson --- linux-user/aarch64/cpu_loop.c | 16 ++-- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/linux-user/aarch64/cpu_loop.c b/linux-user/aarc

[Qemu-devel] [PATCH v2 08/14] util: Add qemu_getrandom and support functions

2019-03-13 Thread Richard Henderson
Initialize the system from system and linux-user arguments. Propagate deterministic seeds when creating new cpu threads. Signed-off-by: Richard Henderson --- include/qemu/random.h | 58 +++ include/qom/cpu.h | 1 + cpus.c| 9 ++ linux-use

[Qemu-devel] [PATCH v2 05/14] crypto: Use getrandom for qcrypto_random_bytes

2019-03-13 Thread Richard Henderson
Prefer it to direct use of /dev/urandom. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- crypto/random-platform.c | 21 + configure| 18 +- 2 files changed, 38 insertions(+), 1 deletion(-) diff --git a/crypto/random-platform.c b/c

[Qemu-devel] [PATCH v2 04/14] crypto: Use O_CLOEXEC in qcrypto_random_init

2019-03-13 Thread Richard Henderson
Avoids leaking the /dev/urandom fd into any child processes. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- crypto/random-platform.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crypto/random-platform.c b/crypto/random-platform.c index 0866f216dc..8bfce99

[Qemu-devel] [PATCH v2 13/14] linux-user: Remove srand call

2019-03-13 Thread Richard Henderson
We no longer use rand() within linux-user. Cc: Laurent Vivier Signed-off-by: Richard Henderson --- linux-user/main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/linux-user/main.c b/linux-user/main.c index 192bf759b8..11a37f7f9b 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@

[Qemu-devel] [PATCH v2 02/14] crypto: Reverse code blocks in random-platform.c

2019-03-13 Thread Richard Henderson
Use #ifdef _WIN32 instead of #ifndef _WIN32. This will make other tests easier to sequence. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- crypto/random-platform.c | 35 +-- 1 file changed, 17 insertions(+), 18 deletions(-) diff --git a/crypto/rando

[Qemu-devel] [PATCH v2 03/14] crypto: Do not fail for EINTR during qcrypto_random_bytes

2019-03-13 Thread Richard Henderson
We can always get EINTR for read; /dev/urandom is no exception. Clean up return paths and avoid unnecessary goto. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- crypto/random-platform.c | 33 + 1 file changed, 13 insertions(+), 20 deletions(-) diff

[Qemu-devel] [PATCH v2 07/14] ui/vnc: Use qcrypto_random_bytes for make_challenge

2019-03-13 Thread Richard Henderson
Use a better interface for random numbers than rand, plus some useless floating point arithmetic. Cc: Gerd Hoffmann Signed-off-by: Richard Henderson --- v2: Use qcrypto_random_bytes, not qemu_getrandom, as there is no need for deterministic results for this interface. --- ui/vnc.c | 8 ++---

[Qemu-devel] [PATCH v2 11/14] linux-user: Use qemu_getrandom for AT_RANDOM

2019-03-13 Thread Richard Henderson
Use a better interface for random numbers than rand * 16. Cc: Laurent Vivier Signed-off-by: Richard Henderson --- linux-user/elfload.c | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/linux-user/elfload.c b/linux-user/elfload.c index c1a26021f8..f33bf4fb85 100644 ---

[Qemu-devel] [PATCH v2 01/14] crypto: Merge crypto-obj-y into libqemuutil.a

2019-03-13 Thread Richard Henderson
We will shortly need this in the user-only binaries, so drop the split into system and tools binaries. This also means that crypto-aes-obj-y can be merged back into crypto-obj-y. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- Makefile | 12 +--- Makefile.objs

[Qemu-devel] [PATCH v2 10/14] linux-user: Call qcrypto_init if not using -seed

2019-03-13 Thread Richard Henderson
This is required before using qcrypto_random_bytes by way of qemu_getrandom. Cc: Laurent Vivier Signed-off-by: Richard Henderson --- linux-user/main.c | 8 1 file changed, 8 insertions(+) diff --git a/linux-user/main.c b/linux-user/main.c index 9682e81610..192bf759b8 100644 --- a/linu

[Qemu-devel] [PATCH v2 00/14] Add qemu_getrandom and ARMv8.5-RNG

2019-03-13 Thread Richard Henderson
Changes since v1: * Build crypto-obj-y for linux-user as well. * Several patches to tidy crypto/random-platform.c. * Use getrandom(2) in crypto/random-platform.c. * Use qcrypto_random_bytes in ui/vnc.c. * In qemu_getrandom: - Use g_rand_int instead of srand48. - Use qcrypto_random

Re: [Qemu-devel] [PATCH v2 04/13] spapr/xive: add state synchronization with KVM

2019-03-13 Thread David Gibson
On Mon, Mar 11, 2019 at 09:41:12PM +0100, Cédric Le Goater wrote: > On 2/26/19 1:01 AM, David Gibson wrote: > > On Fri, Feb 22, 2019 at 02:13:13PM +0100, Cédric Le Goater wrote: > >> This extends the KVM XIVE device backend with 'synchronize_state' > >> methods used to retrieve the state from KVM.

Re: [Qemu-devel] [PATCH v2 02/13] spapr/xive: add hcall support when under KVM

2019-03-13 Thread David Gibson
On Wed, Mar 13, 2019 at 11:43:54AM +0100, Cédric Le Goater wrote: > On 3/12/19 11:26 AM, David Gibson wrote: > > On Mon, Mar 11, 2019 at 06:32:05PM +0100, Cédric Le Goater wrote: > >> On 2/26/19 12:22 AM, David Gibson wrote: > >>> On Fri, Feb 22, 2019 at 02:13:11PM +0100, Cédric Le Goater wrote: >

Re: [Qemu-devel] [PATCH v2] MAINTAINERS: PPC: add a PowerNV machine entry

2019-03-13 Thread David Gibson
On Wed, Mar 13, 2019 at 05:24:23PM +0100, Cédric Le Goater wrote: > and declare David and myself as maintainers of the PPC PowerNV > (Non-Virtualized) machine using the OPAL (skiboot) firmware. Applied to ppc-for-4.0, thanks. > > Signed-off-by: Cédric Le Goater > Reviewed-by: Greg Kurz > --- >

Re: [Qemu-devel] [PATCH] spapr: Correctly set LPCR[GTSE] in H_REGISTER_PROCESS_TABLE

2019-03-13 Thread David Gibson
On Wed, Mar 13, 2019 at 08:17:42AM +0100, Cédric Le Goater wrote: > On 3/13/19 4:20 AM, David Gibson wrote: > > 176dccee "target/ppc/spapr: Clear partition table entry when allocating > > hash table" reworked the H_REGISTER_PROCESS_TABLE hypercall, but > > unfortunately due to a small error no long

Re: [Qemu-devel] [PULL 00/60] ppc-for-4.0 queue 20190310

2019-03-13 Thread David Gibson
On Wed, Mar 13, 2019 at 09:30:54AM +, Alex Bennée wrote: > > David Gibson writes: > > > On Tue, Mar 12, 2019 at 07:20:22PM +, Alex Bennée wrote: > >> > >> David Gibson writes: > >> > >> > On Mon, Mar 11, 2019 at 10:40:54AM +, Alex Bennée wrote: > >> >> > > > > [snip] > >> >> > MALLO

Re: [Qemu-devel] [PATCH 1/2] ppc/pnv: Use local_err variable in pnv_chip_power9_intc_create()

2019-03-13 Thread David Gibson
On Wed, Mar 13, 2019 at 03:54:01PM +0100, Greg Kurz wrote: > Detected by Coverity: CID 1399702 Series applied to ppc-for-4.0, thanks. > > Signed-off-by: Greg Kurz > --- > hw/ppc/pnv.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/ppc/pnv.c b/hw/ppc/pnv.c > ind

Re: [Qemu-devel] [PATCH] ppc/pnv: update skiboot to commit 261ca8e779e5.

2019-03-13 Thread David Gibson
On Wed, Mar 13, 2019 at 08:20:40AM +0100, Cédric Le Goater wrote: > On 3/13/19 5:23 AM, David Gibson wrote: > > On Sun, Mar 10, 2019 at 06:53:37PM +0100, Cédric Le Goater wrote: > >> It includes better support for POWER9 processor and the QEMU platform. > >> DD1.0 workarounds have been removed whic

Re: [Qemu-devel] [PATCH 5/7] tcg/i386: add support for IBT

2019-03-13 Thread Richard Henderson
On 3/13/19 5:40 AM, Paolo Bonzini wrote: > +static void tcg_out_endbr(TCGContext *s) > +{ > +#if defined __CET__ && (__CET__ & 1) > +#ifdef __x86_64__ > +tcg_out32(s, 0xfa1e0ff3); > +#else > +tcg_out32(s, 0xfb1e0ff3); > +#endif > +#endif > +} Normally we'd use a runtime test for the featur

Re: [Qemu-devel] [PATCH 0/5] QEMU VFIO live migration

2019-03-13 Thread Zhao Yan
On Thu, Mar 14, 2019 at 03:14:54AM +0800, Alex Williamson wrote: > On Tue, 12 Mar 2019 21:13:01 -0400 > Zhao Yan wrote: > > > hi Alex > > Any comments to the sequence below? > > > > Actaully we have some concerns and suggestions to userspace-opaque migration > > data. > > > > 1. if data is opaq

Re: [Qemu-devel] [PATCH] ati-vga: Implement DDC and EDID info from monitor

2019-03-13 Thread BALATON Zoltan
On Mon, 11 Mar 2019, BALATON Zoltan wrote: > On Mon, 11 Mar 2019, Gerd Hoffmann wrote: >> On Sun, Mar 10, 2019 at 12:22:17AM +0100, BALATON Zoltan wrote: >>> This adds DDC support to ati-vga and connects i2c-ddc to provide EDID >>> info that is read by guests to find available screen modes. Not sur

Re: [Qemu-devel] [PATCH 3/7] configure: add CET support

2019-03-13 Thread Richard Henderson
On 3/13/19 5:40 AM, Paolo Bonzini wrote: > +## > +# detect CET support in the toolchain > + > +if test "$cet" != no; then > + write_c_skeleton; > + if ! compile_prog "-fcf-protection" "" ; then > +if test "$cet" = yes; then > + feature_not_found "c

[Qemu-devel] Data bus error with redeonfb on mips_fulong2e

2019-03-13 Thread BALATON Zoltan
Hello, Trying to debug the Linux kernel oops with radeonfb I've added some more debug logs and got this: radeonfb_pci_register BEGIN pci_host_data: pci_data_read: ati-vga: addr=04 val= len=4 PCI: Enabling device :00:06.0 ( -> 0003) pci_host_data: pci_data_read: ati-vga: addr=04

Re: [Qemu-devel] [PATCH 7/7] coroutine-x86: add CET shadow stack support

2019-03-13 Thread Richard Henderson
On 3/13/19 5:40 AM, Paolo Bonzini wrote: > +static bool have_cet(void) > +{ > +#if defined CONFIG_CET > +uint64_t ssp; > +asm ("xor %0, %0; rdsspq %0\n" : "=rm" (ssp)); The xor is incompatible with a memory output. I don't think you really wanted that in the first place. Just use "=r". Th

Re: [Qemu-devel] [PATCH 6/7] linux-user: add IBT support to x86 safe-syscall.S

2019-03-13 Thread Richard Henderson
On 3/13/19 5:40 AM, Paolo Bonzini wrote: > Because safe-syscall.S does not go through the C compiler, the > .note.gnu.property note has to be added manually. Safe syscalls do not > involve any indirect branch or stack unwinding, so they are trivially > safe for IBT or shadow stacks. > > Signed-of

Re: [Qemu-devel] [PATCH 5/7] tcg/i386: add support for IBT

2019-03-13 Thread Richard Henderson
On 3/13/19 5:40 AM, Paolo Bonzini wrote: > Add endbr annotations before indirect branch targets. This lets QEMU enable > IBT even for TCG-enabled builds. > @@ -3514,6 +3526,7 @@ static void tcg_target_qemu_prologue(TCGContext *s) >CPU_TEMP_BUF_NLONGS * sizeof(long)); > >

Re: [Qemu-devel] Maintainers, please tell us how to boot your machines!

2019-03-13 Thread Peter.Chubb
> "Markus" == Markus Armbruster writes: Markus> = hw/arm/kzm.c = Peter Chubb Markus> (odd fixer:i.MX31 (kzm)) Peter Maydell Markus> (odd fixer:i.MX31 (kzm)) Markus> qemu-...@nongnu.org (open list:i.MX31 (kzm)) qemu-system-arm -m kzm -nographic -kernel zImage -initrd rootfs.cpio.gz Ker

Re: [Qemu-devel] [PATCH 4/7] tcg: add tcg_out_start

2019-03-13 Thread Richard Henderson
On 3/13/19 5:40 AM, Paolo Bonzini wrote: > This function is called at the beginning of any translation block. We will > use it to emit ENDBR32 or ENDBR64 annotations for x86 CET. > > Signed-off-by: Paolo Bonzini > --- > tcg/aarch64/tcg-target.inc.c | 4 > tcg/arm/tcg-target.inc.c | 4 +

Re: [Qemu-devel] [PULL 0/2] Ui 20190313 patches

2019-03-13 Thread Samuel Thibault
pstream' into > > staging (2019-03-11 18:26:37 +) > > > > are available in the Git repository at: > > > > git://git.kraxel.org/qemu tags/ui-20190313-pull-request > > > > for you to fetch changes up to 2f8b7cd587558944532f587abb5203ce54badba9:

Re: [Qemu-devel] [PULL 0/2] Ui 20190313 patches

2019-03-13 Thread Peter Maydell
ilable in the Git repository at: > > git://git.kraxel.org/qemu tags/ui-20190313-pull-request > > for you to fetch changes up to 2f8b7cd587558944532f587abb5203ce54badba9: > > curses: add option to specify VGA font encoding (2019-03-13 08:29:06 +0100) > > --

Re: [Qemu-devel] Maintainers, please tell us how to boot your machines!

2019-03-13 Thread Alistair Francis
On Tue, Mar 12, 2019 at 11:22 AM Markus Armbruster wrote: > > Dear board code maintainers, ... > > > Machines with at least one maintainer: > ... > > = hw/arm/netduino2.c = > Alistair Francis (maintainer:Netduino 2) > Peter Maydell (maintainer:Netduino 2) > qemu-...@nongnu.or

Re: [Qemu-devel] [PATCH v2 1/4] slirp: relicense GPL files to BSD-3

2019-03-13 Thread Samuel Thibault
Hello, Marc-André Lureau, le mer. 13 mars 2019 18:35:09 +0100, a ecrit: > On Wed, Mar 13, 2019 at 6:32 PM Eric Blake wrote: > > > > On 3/13/19 11:42 AM, Marc-André Lureau wrote: > > > In order to make slirp a standalone project, the project must have a > > > clear license, and be compatible with

Re: [Qemu-devel] [RFC PATCH 3/3] hw/acpi: Extract build_mcfg

2019-03-13 Thread Wei Yang
On Wed, Mar 13, 2019 at 05:09:43PM +0100, Igor Mammedov wrote: >On Wed, 13 Mar 2019 13:33:59 + >Wei Yang wrote: > >> On Wed, Mar 13, 2019 at 01:23:00PM +0100, Igor Mammedov wrote: >> >On Wed, 13 Mar 2019 12:42:53 +0800 >> >Wei Yang wrote: >> > >> >> Now we have two identical build_mcfg func

[Qemu-devel] [PULL 2/2] configure: remove slirp submodule support that doesn't exist yet

2019-03-13 Thread Samuel Thibault
From: Daniel P. Berrangé The slirp code is not yet split off into a separate repository, so configuring QEMU to use slirp as a submodule is premature. This causes the non-existant "slirp" to be requested from git when syncing submodules. This in turn appears to be cause of non-deterministic failu

[Qemu-devel] [PULL 1/1] configure: remove slirp submodule support that doesn't exist yet

2019-03-13 Thread Samuel Thibault
From: Daniel P. Berrangé The slirp code is not yet split off into a separate repository, so configuring QEMU to use slirp as a submodule is premature. This causes the non-existant "slirp" to be requested from git when syncing submodules. This in turn appears to be cause of non-deterministic failu

[Qemu-devel] [PULL 1/2] slirp: remove empty state.h

2019-03-13 Thread Samuel Thibault
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Message-Id: <20190313173949.2369-1-marcandre.lur...@redhat.com> Reviewed-by: Eric Blake Signed-off-by: Samuel Thibault --- slirp/src/state.c | 1 - slirp/src/state.h | 0 2 files changed, 1 deletion(-) delete mode 100644 slirp/src/stat

Re: [Qemu-devel] [PATCH] slirp: remove empty state.h

2019-03-13 Thread Samuel Thibault
Eric Blake, le mer. 13 mars 2019 13:21:23 -0500, a ecrit: > On 3/13/19 12:39 PM, Marc-André Lureau wrote: > > Signed-off-by: Marc-André Lureau > > --- > > slirp/src/state.h | 0 > > slirp/src/state.c | 1 - > > 2 files changed, 1 deletion(-) > > delete mode 100644 slirp/src/state.h > > Made emp

[Qemu-devel] [PULL 0/2] Slirp updates

2019-03-13 Thread Samuel Thibault
The following changes since commit cd82b1e170019c4b722ed53116ee9346315d7791: slirp: remove empty state.h (2019-03-13 22:12:23 +0100) are available in the Git repository at: https://people.debian.org/~sthibault/qemu.git tags/samuel-thibault for you to fetch changes up to 1f773d9dd3187281b29c

[Qemu-devel] [PATCH v2 01/12] roms: lift "edk2-funcs.sh" from "tests/uefi-test-tools/build.sh"

2019-03-13 Thread Laszlo Ersek
Extract the dense logic for architecture and toolchain massaging from "tests/uefi-test-tools/build.sh", to a set of small functions. We'll reuse these functions for building full platform firmware images. Signed-off-by: Laszlo Ersek Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu

Re: [Qemu-devel] [PATCH v5 00/15] s390: vfio-ccw dasd ipl support

2019-03-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com Subject: [Qemu-devel

[Qemu-devel] [PULL 0/1] slirp update

2019-03-13 Thread Samuel Thibault
The following changes since commit cd82b1e170019c4b722ed53116ee9346315d7791: slirp: remove empty state.h (2019-03-13 22:12:23 +0100) are available in the Git repository at: https://people.debian.org/~sthibault/qemu.git tags/samuel-thibault for you to fetch changes up to cd82b1e170019c4b722e

Re: [Qemu-devel] [PULL 0/1] slirp update

2019-03-13 Thread Samuel Thibault
Samuel Thibault, le mer. 13 mars 2019 22:14:42 +0100, a ecrit: > The following changes since commit cd82b1e170019c4b722ed53116ee9346315d7791: > > slirp: remove empty state.h (2019-03-13 22:12:23 +0100) Oops, sorry, there was another patch to commit, you can ignore this pull request. Samuel

[Qemu-devel] [PATCH v2 03/12] tests/uefi-test-tools/build.sh: work around TianoCore#1607

2019-03-13 Thread Laszlo Ersek
The edk2-stabe201903 release introduced Python3 support to edk2's BaseTools; however the Python3 enablement breaks in a corner case (which is nevertheless supported by the edk2 community), namely the in-module parallelization that we utilize. This is tracked under

[Qemu-devel] [PATCH v2 02/12] roms/edk2-funcs.sh: require gcc-4.8+ for building i386 and x86_64

2019-03-13 Thread Laszlo Ersek
Adapt the qemu_edk2_get_toolchain() function in "roms/edk2-funcs.sh" in advance to edk2 commit 8d7cdfae8cb8 ("OvmfPkg: require GCC48 or later", 2019-01-08), which is part of the "edk2-stable201903" tag. Signed-off-by: Laszlo Ersek Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-D

[Qemu-devel] [PULL 1/1] slirp: remove empty state.h

2019-03-13 Thread Samuel Thibault
From: Marc-André Lureau Signed-off-by: Marc-André Lureau Message-Id: <20190313173949.2369-1-marcandre.lur...@redhat.com> Reviewed-by: Eric Blake Signed-off-by: Samuel Thibault --- slirp/src/state.c | 1 - slirp/src/state.h | 0 2 files changed, 1 deletion(-) delete mode 100644 slirp/src/stat

[Qemu-devel] [PATCH v2 07/12] roms: build edk2 firmware binaries and variable store templates

2019-03-13 Thread Laszlo Ersek
Add the "efi" target to "Makefile". Introduce "Makefile.edk2" for building and cleaning the firmware images and varstore templates. Collect the common bits from the recipes in the helper script "edk2-build.sh". Signed-off-by: Laszlo Ersek --- Notes: v2: - drop comma after copyrigh

[Qemu-devel] [PATCH v2 04/12] roms/edk2: advance to tag edk2-stable201903

2019-03-13 Thread Laszlo Ersek
Update the roms/edk2 submodule hash from edk2-stable201811 to edk2-stable201903. The release notes are available at . $ git shortlog edk2-stable201811..edk2-stable201903 Achin Gupta (9): ArmPkg: Add PCDs needed for MM communi

Re: [Qemu-devel] [Qemu-ppc] [PATCH v5 1/2] spapr: helper functions to get valid host fields

2019-03-13 Thread Maxiwell S. Garcia
On Tue, Mar 12, 2019 at 11:52:24AM +0100, Greg Kurz wrote: Hi Greg, > On Mon, 11 Mar 2019 19:57:08 -0300 > "Maxiwell S. Garcia" wrote: > > > The pseries options 'host-serial' and 'host-model' accepts > > 'none', 'passthrough', or content. The helper > > functions in this commit return a valid

[Qemu-devel] [PATCH v2 00/12] bundle edk2 platform firmware with QEMU

2019-03-13 Thread Laszlo Ersek
Repo: https://github.com/lersek/qemu.git Branch: edk2_build_v2 Version 1, that is: [Qemu-devel] [PATCH 00/10] bundle edk2 platform firmware with QEMU was posted at: https://lists.gnu.org/archive/html/qemu-devel/2019-03/msg02846.html 20190309004826.9027-1-lersek@redhat.com">http://mid.mail

[Qemu-devel] [PATCH v2 09/12] pc-bios: document the edk2 firmware images; add firmware descriptors

2019-03-13 Thread Laszlo Ersek
Update the README file with information on the images added previously, and provide firmware descriptor documents that conform to "docs/interop/firmware.json". Signed-off-by: Laszlo Ersek Reviewed-by: Michal Privoznik Reviewed-by: Michael S. Tsirkin --- Notes: v2: - refer to "edk2

[Qemu-devel] [PATCH v2 05/12] roms/edk2-funcs.sh: add the qemu_edk2_get_thread_count() function

2019-03-13 Thread Laszlo Ersek
The edk2 "build" utility natively supports building modules (that is, INF files) in parallel. The feature is not useful when building a single module (with the "-m" option), but it is useful for platform firmware builds (which include many modules). Add a function that determines the "-n" option ar

[Qemu-devel] [PATCH v2 11/12] Makefile: install the edk2 firmware images and their descriptors

2019-03-13 Thread Laszlo Ersek
Decompress and install the edk2 firmware blobs as part of "make install", unless blob installation was disabled with configure's "--disable-blobs" option. Additionally, decompress the blobs as a pre-requisite for building softmmu binaries -- this is helpful for both "make check" and other ad-hoc t

[Qemu-devel] [PATCH v2 08/12] pc-bios: add edk2 firmware binaries and variable store templates

2019-03-13 Thread Laszlo Ersek
Add the files built by the last patch: (compressed) binaries, and the cumulative license text that covers them. Signed-off-by: Laszlo Ersek --- Notes: v2: - capture the compressed build outputs of the last patch; slightly update the commit message [Dan, Michael, Phil]

Re: [Qemu-devel] Maintainers, please tell us how to boot your machines!

2019-03-13 Thread Stafford Horne
On Tue, Mar 12, 2019 at 06:36:05PM +0100, Markus Armbruster wrote: > = hw/openrisc/openrisc_sim.c = > Jia Liu (maintainer:or1k-sim) > Stafford Horne (odd fixer:OpenRISC) For OpenRISC the main test I do is booting linux. The steps and a link to a system image are described here: h

[Qemu-devel] [PATCH v2 06/12] roms/Makefile: replace the $(EFIROM) target with "edk2-basetools"

2019-03-13 Thread Laszlo Ersek
We don't (can't) have a recipe for building just $(EFIROM); therefore, while we call the target $(EFIROM), we actually build all of the edk2 BaseTools. Rename the target to edk2-basetools, and update the iPXE prerequisite accordingly. This will let other targets depend on "edk2-basetools", where an

[Qemu-devel] [PATCH v2 10/12] tests: add missing dependency to build QTEST_QEMU_BINARY, round 2

2019-03-13 Thread Laszlo Ersek
In commit b94b330e2333 ("tests: add missing dependency to build QTEST_QEMU_BINARY", 2017-07-31), Phil fixed the dependency list of make target "check-qtest-%". Namely, the recipe would set QTEST_QEMU_BINARY to the softmmu emulator for the emulation target, but the prerequisites didn't include the e

[Qemu-devel] [PATCH v2 12/12] MAINTAINERS: add the "EDK2 Firmware" subsystem

2019-03-13 Thread Laszlo Ersek
We now have the edk2 submodule, somewhat elaborate build helpers for it, and even a UEFI application written against edk2 whose genuine home is the QEMU repository. Add the "EDK2 Firmware" subsystem such that all relevant pathnames be covered. Suggested-by: Daniel P. Berrangé Suggested-by: Michae

Re: [Qemu-devel] [PATCH] Clean up includes

2019-03-13 Thread BALATON Zoltan
On Wed, 13 Mar 2019, Markus Armbruster wrote: BALATON Zoltan writes: On Wed, 13 Mar 2019, Markus Armbruster wrote: Clean up includes so that osdep.h is included first and headers which it implies are not included manually. This commit was created with scripts/clean-includes, with the changes

Re: [Qemu-devel] [PATCH v5 0/5] Add "boot_linux" acceptance test

2019-03-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20190313204611.21041-1-cr...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20190313204611.21041-1-cr...@redhat.com Subject: [Qemu-devel] [PATCH v5 0/5] Add "boot_linux"

[Qemu-devel] [PATCH v5 3/5] Acceptance tests: add the build directory to the system PATH

2019-03-13 Thread Cleber Rosa
So that when binaries such as qemu-img are searched for, those in the build tree will be favored. As a clarification, SRC_ROOT_DIR is dependent on the location from where tests are executed, so they are equal to the build directory if one is being used. The original motivation is that Avocado lib

[Qemu-devel] [PATCH v5 0/5] Add "boot_linux" acceptance test

2019-03-13 Thread Cleber Rosa
This adds an acceptance test that validates that a full blown Linux guest can successfully boot in QEMU. Changes from v4: * New commit "Acceptance tests: use relative location for tests" * New commit "Acceptance tests: keep a stable reference to the QEMU build dir" * Pinned

[Qemu-devel] [PATCH v5 4/5] Acceptance tests: depend on qemu-img

2019-03-13 Thread Cleber Rosa
Tests using the avocado.utils.vmimage library make use of qemu-img, and because it makes sense to use the version matching the rest of the source code, let's make sure it gets built. Its selection, instead of a possible qemu-img binary installed system wide, is already dealt with by the change tha

[Qemu-devel] [PATCH v5 5/5] Add "boot_linux" acceptance test for x86_64 and pc machine type

2019-03-13 Thread Cleber Rosa
This acceptance test, validates that a full blown Linux guest can successfully boot in QEMU. In this specific case, the guest chosen is Fedora version 29. The method for checking the successful boot is based on "cloudinit" and its "phone home" feature. The guest is given an ISO image with the lo

[Qemu-devel] [PATCH v5 2/5] Acceptance tests: keep a stable reference to the QEMU build dir

2019-03-13 Thread Cleber Rosa
This is related to the the differences in in-tree and out-of-tree builds in QEMU. For simplification, means my build directory. Currently, by running a `make check-acceptance` one gets (in tests/acceptance/avocado_qemu/__init__.py): SRC_ROOT_DIR: /tests/acceptance/avocado_qemu/../../.. This

[Qemu-devel] [PATCH v5 1/5] Acceptance tests: use relative location for tests

2019-03-13 Thread Cleber Rosa
An Avocado Test ID[1] is composed by a number of components, but it starts with the Test Name, usually a file system location that was given to the loader. Because the source directory is being given as a prefix to the "tests/acceptance" directory containing the acceptance tests, the test names wi

Re: [Qemu-devel] [PATCH v2 4/4] RFC: slirp: is not maintained by Kelly Price for a long time

2019-03-13 Thread Kelly Price
You have my sign-off. Take it away! > On Mar 13, 2019, at 1:59 PM, Eric Blake wrote: > >> On 3/13/19 11:42 AM, Marc-André Lureau wrote: >> slirp has been maintained by the QEMU maintainers and will be >> maintained under an independent project soon. >> >> Cc: Kelly Price >> Cc: Kelly Price >>

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

2019-03-13 Thread Peter Maydell
On Wed, 13 Mar 2019 at 02:54, Michael S. Tsirkin wrote: > > The following changes since commit 9d867123659ac32edd2a0b64cabe2dca544f1d0e: > > Merge remote-tracking branch > 'remotes/ehabkost/tags/python-next-pull-request' into staging (2019-03-12 > 18:31:56 +) > > are available in the Git r

Re: [Qemu-devel] [PATCH 0/5] QEMU VFIO live migration

2019-03-13 Thread Alex Williamson
On Tue, 12 Mar 2019 02:48:39 + "Tian, Kevin" wrote: > > From: Alex Williamson [mailto:alex.william...@redhat.com] > > Sent: Tuesday, March 12, 2019 4:19 AM > > On Mon, 11 Mar 2019 02:33:11 + > > "Tian, Kevin" wrote: > > > [...] > > > > I think I've fully conceded any notion of securi

[Qemu-devel] Seccomp profile for swtpm as default

2019-03-13 Thread Stefan Berger
Hello!  If you have some feedback regarding a seccomp profile extension for swtpm for v0.2, please let me know. I created this github issue here: https://github.com/stefanberger/swtpm/issues/115 Basically the choice is whether to make the creation of the seccomp profile a default behavior

Re: [Qemu-devel] [PATCH 0/5] QEMU VFIO live migration

2019-03-13 Thread Alex Williamson
On Tue, 12 Mar 2019 21:13:01 -0400 Zhao Yan wrote: > hi Alex > Any comments to the sequence below? > > Actaully we have some concerns and suggestions to userspace-opaque migration > data. > > 1. if data is opaque to userspace, kernel interface must be tightly bound to > migration. >e.g. ve

Re: [Qemu-devel] [PATCH for-4.1 0/7] Add qemu_getrandom and ARMv8.5-RNG

2019-03-13 Thread Richard Henderson
On 3/13/19 11:49 AM, Richard Henderson wrote: > On 3/13/19 10:38 AM, Markus Armbruster wrote: >> Have you considered GLib's? Mersenne Twister under the hood. >> >> https://developer.gnome.org/glib/stable/glib-Random-Numbers.html > > I hadn't considered glib's, but I am considering MT19337-64. Th

Re: [Qemu-devel] [PATCH v5 00/15] s390: vfio-ccw dasd ipl support

2019-03-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com Subject: [Qemu-devel

Re: [Qemu-devel] [PULL 00/22] Bitmaps patches

2019-03-13 Thread Peter Maydell
On Tue, 12 Mar 2019 at 20:23, John Snow wrote: > > The following changes since commit 46316f1dfffc6be72e94e89f7b0e9162e7dcdcf1: > > Merge remote-tracking branch > 'remotes/awilliam/tags/vfio-updates-20190311.0' into staging (2019-03-12 > 13:37:29 +) > > are available in the Git repository

Re: [Qemu-devel] [PATCH for-4.1 0/7] Add qemu_getrandom and ARMv8.5-RNG

2019-03-13 Thread Richard Henderson
On 3/13/19 10:38 AM, Markus Armbruster wrote: > Have you considered GLib's? Mersenne Twister under the hood. > > https://developer.gnome.org/glib/stable/glib-Random-Numbers.html I hadn't considered glib's, but I am considering MT19337-64. r~

Re: [Qemu-devel] [PATCH v5 00/15] s390: vfio-ccw dasd ipl support

2019-03-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com Subject: [Qemu-devel

Re: [Qemu-devel] [PATCH v5 00/15] s390: vfio-ccw dasd ipl support

2019-03-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com Subject: [Qemu-devel

Re: [Qemu-devel] [PATCH v2 3/4] slirp: remove reference to COPYRIGHT file

2019-03-13 Thread Eric Blake
On 3/13/19 1:07 PM, Marc-André Lureau wrote: > Hi > > On Wed, Mar 13, 2019 at 6:57 PM Eric Blake wrote: >> >> On 3/13/19 11:42 AM, Marc-André Lureau wrote: >>> The slirp COPYRIGHT file is a BSD-3 license. >> >> Is it? I only see 2 clauses listed there. > > In commit 2f5f89963186d42a7ded253bc6cf5

Re: [Qemu-devel] [PATCH v5 00/15] s390: vfio-ccw dasd ipl support

2019-03-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com Subject: [Qemu-devel

Re: [Qemu-devel] [PATCH for-4.1 0/7] Add qemu_getrandom and ARMv8.5-RNG

2019-03-13 Thread Richard Henderson
On 3/13/19 10:57 AM, Daniel P. Berrangé wrote: > We already have an internal API for providing strong random bytes in > QEMU qcrypto_random_bytes. It is preferentially backed by gnutls or > gcrypt, but if those aren't built-in it falls back to a platform > native API like /dev/random. I've got a to

Re: [Qemu-devel] [PATCH v5 00/15] s390: vfio-ccw dasd ipl support

2019-03-13 Thread no-reply
Patchew URL: https://patchew.org/QEMU/1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 1552494682-16788-1-git-send-email-jjhe...@linux.ibm.com Subject: [Qemu-devel

Re: [Qemu-devel] [PATCH] xen-block: Replace qdict_put_obj() by qdict_put() where appropriate

2019-03-13 Thread Eric Blake
On 3/13/19 12:44 PM, Markus Armbruster wrote: > Patch created mechanically by rerunning: > > $ spatch --sp-file scripts/coccinelle/qobject.cocci \ > --macro-file scripts/cocci-macro-file.h \ > --dir hw/block --in-place > > Signed-off-by: Markus Armbruster > --- > h

Re: [Qemu-devel] [PATCH] slirp: remove empty state.h

2019-03-13 Thread Eric Blake
On 3/13/19 12:39 PM, Marc-André Lureau wrote: > Signed-off-by: Marc-André Lureau > --- > slirp/src/state.h | 0 > slirp/src/state.c | 1 - > 2 files changed, 1 deletion(-) > delete mode 100644 slirp/src/state.h Made empty in commit d890344, before moving to its current location in c2d63650. Re

Re: [Qemu-devel] [PATCH] Clean up includes

2019-03-13 Thread Eric Blake
On 3/13/19 11:28 AM, Markus Armbruster wrote: > Clean up includes so that osdep.h is included first and headers > which it implies are not included manually. > > This commit was created with scripts/clean-includes, with the changes > to the following files manually reverted: Is it worth tweaking

Re: [Qemu-devel] [PATCH v4 01/17] util: add helper APIs for dealing with inotify in portable manner

2019-03-13 Thread Daniel P . Berrangé
On Wed, Mar 13, 2019 at 01:19:58PM -0400, Bandan Das wrote: > Daniel P. Berrangé writes: > > > On Tue, Mar 12, 2019 at 07:07:42PM -0400, Bandan Das wrote: > >> Daniel P. Berrangé writes: > >> ... > >> > + > >> > +int > >> > +qemu_file_monitor_add_watch(QFileMonitor *mon, > >> > +

Re: [Qemu-devel] [PATCH] intel-iommu: optimize nodmar memory regions

2019-03-13 Thread Paolo Bonzini
On 13/03/19 12:45, Sergio Lopez wrote: > > Peter Xu writes: > >> Previously we have per-device system memory aliases when DMAR is >> disabled by the system. It will slow the system down if there are >> lots of devices especially when DMAR is disabled, because each of the >> aliased system addres

  1   2   3   4   >