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

2019-03-14 Thread Cédric Le Goater
On 3/15/19 1:23 AM, David Gibson wrote: > On Thu, Mar 14, 2019 at 08:56:25AM +0100, Cédric Le Goater wrote: >> 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:

[Qemu-devel] [PATCH V4 0/4] Migration/colo: Fix upstream bugs when occur failover

2019-03-14 Thread Zhang Chen
From: Zhang Chen This series focus on COLO failover bug fix and optimization. V4: - Remove merged patch. - Fix grammer issues in patch 2/4 to address markus's comments. - Rebased on upstream code. V3: - Fix grammer issues in patch 4/7. - Add more information in commit log of patch 5/7. V2

Re: [Qemu-devel] [PATCH v7 6/7] vhost-user-blk: Add support to reconnect backend

2019-03-14 Thread Yongji Xie
On Fri, 15 Mar 2019 at 11:09, Michael S. Tsirkin wrote: > > On Fri, Mar 15, 2019 at 10:46:34AM +0800, Yongji Xie wrote: > > On Thu, 14 Mar 2019 at 19:18, Michael S. Tsirkin wrote: > > > > > > On Wed, Mar 13, 2019 at 10:47:08AM +0800, Yongji Xie wrote: > > > > On Wed, 13 Mar 2019 at 09:16, Michael

[Qemu-devel] [PATCH V4 3/4] Migration/colo.c: Add the necessary checks for colo_do_failover

2019-03-14 Thread Zhang Chen
From: Zhang Chen Signed-off-by: Zhang Chen Reviewed-by: Dr. David Alan Gilbert --- migration/colo.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/migration/colo.c b/migration/colo.c index dbe2b88807..d1ae2e6d11 100644 --- a/migration/colo.c +++ b/migration/colo

Re: [Qemu-devel] [PATCH v3 16/23] aspeed/scu: Use qemu_guest_getrandom_nofail

2019-03-14 Thread Cédric Le Goater
On 3/15/19 4:26 AM, Richard Henderson wrote: > The random number is intended for use by the guest. As such, we should > honor the -seed argument for reproducibility. Use the *_nofail routine > instead of rolling our own error handling locally. > > Cc: qemu-...@nongnu.org > Cc: Cédric Le Goater

[Qemu-devel] [PATCH V4 1/4] Migration/colo.c: Fix COLO failover status error

2019-03-14 Thread Zhang Chen
From: Zhang Chen When finished COLO failover, the status is FAILOVER_STATUS_COMPLETED. The origin codes misunderstand the FAILOVER_STATUS_REQUIRE. Signed-off-by: Zhang Chen Reviewed-by: Dr. David Alan Gilbert --- migration/colo.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) d

[Qemu-devel] [PATCH V4 4/4] Migration/colo.c: Make user obtain the COLO mode info after failover

2019-03-14 Thread Zhang Chen
From: Zhang Chen Add the last_colo_mode to save the status after failover. This patch can solve the issue that user got nothing to call query_colo_status after failover. Signed-off-by: Zhang Chen --- migration/colo.c | 28 +++- 1 file changed, 23 insertions(+), 5 deleti

[Qemu-devel] [PATCH V4 2/4] Migration/colo.c: Add new COLOExitReason to handle all failover state

2019-03-14 Thread Zhang Chen
From: Zhang Chen In this patch we add the processing state for COLOExitReason, because we have to identify COLO in the failover processing state or failover error state. In the way, we can handle all the failover state. We have improved the description of the COLOExitReason by the way. Signed-of

Re: [Qemu-devel] [PATCH V2 1/3] audio/paaudio: fix ignored buffer_length setting

2019-03-14 Thread Martin Schrodt
On 3/14/19 10:34 PM, Philippe Mathieu-Daudé wrote: >> +static int qpa_validate_per_direction_opts (Audiodev *dev, >> AudiodevPaPerDirectionOptions *pdo) > > If this function is expected to return a boolean value, please use the > 'bool' C type and true/false. > >> +{ >> +if (!pdo->has_buffer

[Qemu-devel] [PATCH] MAINTAINERS: Update the latest email address

2019-03-14 Thread Zhang Chen
From: Zhang Chen Signed-off-by: Zhang Chen --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 0e7baa9aa2..f127b1356c 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2167,7 +2167,7 @@ F: include/migration/failover.h F: docs/COLO-FT

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

2019-03-14 Thread Peter Xu
On Thu, Mar 14, 2019 at 11:36:39AM +0100, Sergio Lopez wrote: > > Paolo Bonzini writes: > > > 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

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

2019-03-14 Thread Peter Xu
On Wed, Mar 13, 2019 at 12:21:34PM +0100, Paolo Bonzini wrote: > On 13/03/19 10:43, Peter Xu wrote: > > 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, becau

Re: [Qemu-devel] [PULL 17/29] target/riscv: Convert quadrant 1 of RVXC insns to decodetree

2019-03-14 Thread Palmer Dabbelt
On Thu, 14 Mar 2019 21:57:37 PDT (-0700), alistai...@gmail.com wrote: On Thu, Mar 14, 2019 at 8:59 PM Palmer Dabbelt wrote: On Thu, 14 Mar 2019 13:28:37 PDT (-0700), alistai...@gmail.com wrote: > On Wed, Mar 13, 2019 at 7:53 AM Palmer Dabbelt wrote: >> >> From: Bastian Koppelmann >> >> Revie

Re: [Qemu-devel] [PULL 17/29] target/riscv: Convert quadrant 1 of RVXC insns to decodetree

2019-03-14 Thread Alistair Francis
On Thu, Mar 14, 2019 at 8:59 PM Palmer Dabbelt wrote: > > On Thu, 14 Mar 2019 13:28:37 PDT (-0700), alistai...@gmail.com wrote: > > On Wed, Mar 13, 2019 at 7:53 AM Palmer Dabbelt wrote: > >> > >> From: Bastian Koppelmann > >> > >> Reviewed-by: Richard Henderson > >> Signed-off-by: Bastian Koppe

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

2019-03-14 Thread David Gibson
On Thu, Mar 14, 2019 at 08:56:25AM +0100, Cédric Le Goater wrote: > 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: > >>

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

2019-03-14 Thread David Gibson
On Thu, Mar 14, 2019 at 10:24:49PM +0100, Cédric Le Goater wrote: > On 3/14/19 3:11 AM, David Gibson wrote: > > 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: > >

Re: [Qemu-devel] [PULL 17/29] target/riscv: Convert quadrant 1 of RVXC insns to decodetree

2019-03-14 Thread Palmer Dabbelt
On Thu, 14 Mar 2019 13:28:37 PDT (-0700), alistai...@gmail.com wrote: On Wed, Mar 13, 2019 at 7:53 AM Palmer Dabbelt wrote: From: Bastian Koppelmann Reviewed-by: Richard Henderson Signed-off-by: Bastian Koppelmann Signed-off-by: Peer Adelt This commit is the first bad commit in breaking

[Qemu-devel] [PATCH v3 05/23] crypto: Use getrandom for qcrypto_random_bytes

2019-03-14 Thread Richard Henderson
Prefer it to direct use of /dev/urandom. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- v3: If getrandom is not present, fall back on /dev/(u)random. --- crypto/random-platform.c | 37 - configure| 18 +- 2 files c

Re: [Qemu-devel] [PATCH v3 17/23] hw/misc/nrf51_rng: Use qemu_guest_getrandom_nofail

2019-03-14 Thread Joel Stanley
On Fri, 15 Mar 2019 at 03:26, Richard Henderson wrote: > > The random number is intended for use by the guest. As such, we should > honor the -seed argument for reproducibility. Use the *_nofail routine > instead of error_abort directly. > > Cc: qemu-...@nongnu.org > Cc: Joel Stanley > Signed-o

[Qemu-devel] [PATCH v3 04/23] crypto: Use O_CLOEXEC in qcrypto_random_init

2019-03-14 Thread Richard Henderson
Avoids leaking the /dev/urandom fd into any child processes. Reviewed-by: 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 260b64564d

Re: [Qemu-devel] [PATCH v7 6/7] vhost-user-blk: Add support to reconnect backend

2019-03-14 Thread Yongji Xie
On Thu, 14 Mar 2019 at 19:43, Daniel P. Berrangé wrote: > > On Thu, Mar 14, 2019 at 07:34:03AM -0400, Michael S. Tsirkin wrote: > > On Thu, Mar 14, 2019 at 11:24:22AM +, Daniel P. Berrangé wrote: > > > On Tue, Mar 12, 2019 at 12:49:35PM -0400, Michael S. Tsirkin wrote: > > > > On Thu, Feb 28,

[Qemu-devel] [PATCH v3 07/23] ui/vnc: Split out authentication_failure

2019-03-14 Thread Richard Henderson
There were 3 copies of this code, one of which used the wrong data size for the failure indicator. Cc: Gerd Hoffmann Signed-off-by: Richard Henderson --- ui/vnc.c | 37 +++-- 1 file changed, 15 insertions(+), 22 deletions(-) diff --git a/ui/vnc.c b/ui/vnc.c inde

Re: [Qemu-devel] [PATCH v3 16/23] aspeed/scu: Use qemu_guest_getrandom_nofail

2019-03-14 Thread Joel Stanley
On Fri, 15 Mar 2019 at 03:26, Richard Henderson wrote: > > The random number is intended for use by the guest. As such, we should > honor the -seed argument for reproducibility. Use the *_nofail routine > instead of rolling our own error handling locally. Nice! Reviewed-by: Joel Stanley > >

[Qemu-devel] [PATCH v3 00/23] Add qemu_getrandom and ARMv8.5-RNG etc

2019-03-14 Thread Richard Henderson
Changes since v2: * Changes from review. - getrandom is not exclusive of /dev/urandom fallback. - vnc fails gracefully on crypto failure. - a great renaming. * Drop the "nonblock" argument, as it's not deliverable from the backend. * Propagate Error back through qemu_guest_getrand

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

2019-03-14 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 v3 06/23] crypto: Change the qcrypto_random_bytes buffer type to void*

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

[Qemu-devel] [PATCH v3 10/23] cpus: Initialize pseudo-random seeds for all guest cpus

2019-03-14 Thread Richard Henderson
When the -seed option is given, call qemu_guest_random_seed_main, putting the subsystem into deterministic mode. Pass derived seeds to each cpu created; which is a no-op unless the subsystem is in deterministic mode. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson --- include/qom/cpu.h

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

2019-03-14 Thread Richard Henderson
We can always get EINTR for read; /dev/urandom is no exception. Rearrange the order of tests for likelihood; allow degenerate buflen==0 case to perform a no-op zero-length read. This means that the normal success path is a straight line with a single test for success. Reviewed-by: Daniel P. Berr

[Qemu-devel] [PATCH v3 11/23] linux-user: Initialize pseudo-random seeds for all guest cpus

2019-03-14 Thread Richard Henderson
When the -seed option is given, call qemu_guest_random_seed_main, putting the subsystem into deterministic mode. Pass derived seeds to each cpu created during clone; which is a no-op unless the subsystem is in deterministic mode. Cc: Laurent Vivier Signed-off-by: Richard Henderson --- linux-us

[Qemu-devel] [PATCH v3 08/23] ui/vnc: Use gcrypto_random_bytes for start_auth_vnc

2019-03-14 Thread Richard Henderson
Use a better interface for random numbers than rand(). Fail gracefully if for some reason we cannot use the crypto system. 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

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

2019-03-14 Thread Richard Henderson
Use #ifdef _WIN32 instead of #ifndef _WIN32. This will make other tests easier to sequence. Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Daniel P. Berrangé Signed-off-by: Richard Henderson --- crypto/random-platform.c | 35 +-- 1 file changed, 17 insertions(

[Qemu-devel] [PATCH v3 13/23] linux-user: Use qemu_guest_getrandom_nofail for AT_RANDOM

2019-03-14 Thread Richard Henderson
Use a better interface for random numbers than rand * 16. Cc: Laurent Vivier Reviewed-by: Philippe Mathieu-Daudé 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 ind

[Qemu-devel] [PATCH v3 12/23] linux-user: Call qcrypto_init if not using -seed

2019-03-14 Thread Richard Henderson
Cc: Laurent Vivier Signed-off-by: Richard Henderson --- linux-user/main.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/linux-user/main.c b/linux-user/main.c index cf7095bdaf..8478306eef 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@ -38,6 +38,7

[Qemu-devel] [PATCH v3 16/23] aspeed/scu: Use qemu_guest_getrandom_nofail

2019-03-14 Thread Richard Henderson
The random number is intended for use by the guest. As such, we should honor the -seed argument for reproducibility. Use the *_nofail routine instead of rolling our own error handling locally. Cc: qemu-...@nongnu.org Cc: Cédric Le Goater Cc: Andrew Jeffery Cc: Joel Stanley Signed-off-by: Rich

[Qemu-devel] [PATCH v3 09/23] util: Add qemu_guest_getrandom and associated routines

2019-03-14 Thread Richard Henderson
This routine is intended to produce high-quality random numbers to the guest. Normally, such numbers are crypto quality from the host, but a command-line option can force the use of a fully deterministic sequence for use while debugging. Cc: Daniel P. Berrangé Signed-off-by: Richard Henderson -

[Qemu-devel] [PATCH v3 15/23] linux-user: Remove srand call

2019-03-14 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 8478306eef..80cb9fdfbe 100644 --- a/linux-user/main.c +++ b/linux-user/main.c @@

[Qemu-devel] [PATCH v3 17/23] hw/misc/nrf51_rng: Use qemu_guest_getrandom_nofail

2019-03-14 Thread Richard Henderson
The random number is intended for use by the guest. As such, we should honor the -seed argument for reproducibility. Use the *_nofail routine instead of error_abort directly. Cc: qemu-...@nongnu.org Cc: Joel Stanley Signed-off-by: Richard Henderson --- hw/misc/nrf51_rng.c | 4 ++-- 1 file cha

[Qemu-devel] [PATCH v3 20/23] target/arm: Put all PAC keys into a structure

2019-03-14 Thread Richard Henderson
This allows us to use a single syscall to initialize them all. Signed-off-by: Richard Henderson --- target/arm/cpu.h | 12 +++- linux-user/aarch64/cpu_loop.c | 6 +- linux-user/syscall.c | 10 +- target/arm/helper.c | 20 ++--

[Qemu-devel] [PATCH v3 14/23] linux-user/aarch64: Use qemu_guest_getrandom for PAUTH keys

2019-03-14 Thread Richard Henderson
Use a better interface for random numbers than rand() * 3. Cc: Laurent Vivier Signed-off-by: Richard Henderson --- linux-user/aarch64/target_syscall.h | 2 -- linux-user/aarch64/cpu_loop.c | 29 ++- linux-user/syscall.c| 31

[Qemu-devel] [PATCH v3 19/23] hw/misc/exynos4210_rng: Use qemu_guest_getrandom

2019-03-14 Thread Richard Henderson
The random number is intended for use by the guest. As such, we should honor the -seed argument for reproducibility. Cc: qemu-...@nongnu.org Cc: Igor Mitsyanko Signed-off-by: Richard Henderson --- hw/misc/exynos4210_rng.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff

[Qemu-devel] [PATCH v3 21/23] target/arm: Implement ARMv8.5-RNG

2019-03-14 Thread Richard Henderson
Cc: qemu-...@nongnu.org Cc: Peter Maydell Signed-off-by: Richard Henderson --- v3: Log errors with -d unimp, for lack of a better flag. --- target/arm/cpu.h| 5 + target/arm/cpu64.c | 1 + target/arm/helper.c | 44 3 files changed, 50 inser

[Qemu-devel] [PATCH v3 18/23] hw/misc/bcm2835_rng: Use qemu_guest_getrandom_nofail

2019-03-14 Thread Richard Henderson
The random number is intended for use by the guest. As such, we should honor the -seed argument for reproducibility. Use the *_nofail routine instead of rolling our own error handling locally. Cc: qemu-...@nongnu.org Cc: Andrew Baumann Cc: Philippe Mathieu-Daudé Signed-off-by: Richard Henderso

[Qemu-devel] [PATCH v3 23/23] target/i386: Implement CPUID_EXT_RDRAND

2019-03-14 Thread Richard Henderson
We now have an interface for guest visible random numbers. Cc: Paolo Bonzini Cc: Eduardo Habkost Signed-off-by: Richard Henderson --- target/i386/helper.h | 2 ++ target/i386/cpu.c| 5 ++-- target/i386/int_helper.c | 21 +++ target/i386/translate.c | 55 +

[Qemu-devel] [PATCH v3 22/23] target/ppc: Use qemu_guest_getrandom for DARN

2019-03-14 Thread Richard Henderson
We now have an interface for guest visible random numbers. Cc: qemu-...@nongnu.org Cc: David Gibson Signed-off-by: Richard Henderson --- target/ppc/int_helper.c | 38 +++--- 1 file changed, 27 insertions(+), 11 deletions(-) diff --git a/target/ppc/int_helper.c b

Re: [Qemu-devel] [PATCH v7 6/7] vhost-user-blk: Add support to reconnect backend

2019-03-14 Thread Yongji Xie
On Thu, 14 Mar 2019 at 20:31, Yury Kotov wrote: > > Hi, > > 14.03.2019, 14:44, "Daniel P. Berrangé" : > > On Thu, Mar 14, 2019 at 07:34:03AM -0400, Michael S. Tsirkin wrote: > >> On Thu, Mar 14, 2019 at 11:24:22AM +, Daniel P. Berrangé wrote: > >> > On Tue, Mar 12, 2019 at 12:49:35PM -0400,

Re: [Qemu-devel] [PATCH v7 6/7] vhost-user-blk: Add support to reconnect backend

2019-03-14 Thread Michael S. Tsirkin
On Fri, Mar 15, 2019 at 10:46:34AM +0800, Yongji Xie wrote: > On Thu, 14 Mar 2019 at 19:18, Michael S. Tsirkin wrote: > > > > On Wed, Mar 13, 2019 at 10:47:08AM +0800, Yongji Xie wrote: > > > On Wed, 13 Mar 2019 at 09:16, Michael S. Tsirkin wrote: > > > > > > > > On Thu, Feb 28, 2019 at 04:53:54P

Re: [Qemu-devel] State of QEMU CI as we enter 4.0

2019-03-14 Thread Fam Zheng
> On Mar 15, 2019, at 02:22, Peter Maydell wrote: > > On Thu, 14 Mar 2019 at 15:57, Alex Bennée wrote: >> Testing in the Cloud >> >> >> After BuildBot went out-of-service we have been relying heavily on Travis >> as our primary CI platform. This has been creaking somewha

Re: [Qemu-devel] [PATCH v7 6/7] vhost-user-blk: Add support to reconnect backend

2019-03-14 Thread Yongji Xie
On Thu, 14 Mar 2019 at 19:18, Michael S. Tsirkin wrote: > > On Wed, Mar 13, 2019 at 10:47:08AM +0800, Yongji Xie wrote: > > On Wed, 13 Mar 2019 at 09:16, Michael S. Tsirkin wrote: > > > > > > On Thu, Feb 28, 2019 at 04:53:54PM +0800, elohi...@gmail.com wrote: > > > > From: Xie Yongji > > > > > >

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

2019-03-14 Thread Alex Williamson
On Thu, 14 Mar 2019 19:05:06 -0400 Zhao Yan wrote: > On Fri, Mar 15, 2019 at 06:44:58AM +0800, Alex Williamson wrote: > > On Wed, 13 Mar 2019 21:12:22 -0400 > > Zhao Yan wrote: > > > > > On Thu, Mar 14, 2019 at 03:14:54AM +0800, Alex Williamson wrote: > > > > On Tue, 12 Mar 2019 21:13:01 -0

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

2019-03-14 Thread Philippe Mathieu-Daudé
Hi Zoltan, On 3/14/19 11:08 PM, BALATON Zoltan wrote: > On Thu, 14 Mar 2019, Aleksandar Markovic wrote: >>> From: BALATON Zoltan >>> Subject: Data bus error with redeonfb on mips_fulong2e >>> >>> Hello, >>> >>> Trying to debug the Linux kernel oops with radeonfb I've added some more >>> debug log

Re: [Qemu-devel] [PATCH] hw/riscv/virt: re-add machine-specific compatible string to /soc/ node

2019-03-14 Thread Bin Meng
Hi Lukas, On Fri, Mar 15, 2019 at 5:01 AM Auer, Lukas wrote: > > Hi Bin, > > On Wed, 2019-03-13 at 09:51 +0800, Bin Meng wrote: > > Hi Lukas, > > > > On Tue, Mar 12, 2019 at 10:39 PM Auer, Lukas > > wrote: > > > Hi Bin, > > > > > > On Mon, 2019-03-11 at 23:28 +0800, Bin Meng wrote: > > > > Hi Lu

Re: [Qemu-devel] [PATCH v6 0/2] CODING_STYLE: trivial update

2019-03-14 Thread Wei Yang
Is this patch set picked up by someone? On Mon, Mar 04, 2019 at 03:16:29PM +0800, Wei Yang wrote: >The first one is suggested by Igor Mammedov to provide rule for multiline >code. > >The second is a trivial fix to make example code all indented with 4 spaces. > >v6: > * add ) for last example of

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

2019-03-14 Thread Eric Blake
On 3/13/19 11:55 PM, Richard Henderson wrote: > Initialize the system from system and linux-user arguments. > Propagate deterministic seeds when creating new cpu threads. > > Signed-off-by: Richard Henderson > --- > +/** > + * qemu_seedrandom_thread_part1(void) > + * > + * If qemu_getrandom is i

Re: [Qemu-devel] [PULL 04/14] audio: -audiodev command line option basic implementation

2019-03-14 Thread Zoltán Kővágó
On 2019-03-14 10:46, Peter Maydell wrote: > On Tue, 12 Mar 2019 at 07:13, Gerd Hoffmann wrote: >> >> From: Kővágó, Zoltán >> >> Audio drivers now get an Audiodev * as config paramters, instead of the >> global audio_option structs. There is some code in audio/audio_legacy.c >> that converts the

[Qemu-devel] [PATCH v3] hw/acpi: extract acpi_add_rom_blob()

2019-03-14 Thread Wei Yang
arm and i386 has almost the same function acpi_add_rom_blob(), except giving different FWCfgCallback function. This patch extract acpi_add_rom_blob() to utils.c by passing FWCfgCallback to it. Signed-off-by: Wei Yang --- v3: * put acpi_add_rom_blob() to hw/acpi/utils.c v2: * remove unused h

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

2019-03-14 Thread Zhao Yan
On Fri, Mar 15, 2019 at 06:44:58AM +0800, Alex Williamson wrote: > On Wed, 13 Mar 2019 21:12:22 -0400 > Zhao Yan wrote: > > > 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 comme

Re: [Qemu-devel] [Xen-devel] Commit 331b51 breaks live migration on FreeBSD/Xen dom0

2019-03-14 Thread Marek Marczykowski-Górecki
On Thu, Mar 14, 2019 at 04:07:55PM +, Igor Druzhinin wrote: > The change was supposed to be platform independent - it relies on the > fact addr argument to mmap() works as expected - mmaps at the specified > address. You have this guarantee only with MAP_FIXED flag, otherwise the addr argumen

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

2019-03-14 Thread Alex Williamson
On Wed, 13 Mar 2019 21:12:22 -0400 Zhao Yan wrote: > 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 suggesti

Re: [Qemu-devel] [PATCH 1/1] Update seabios-hppa to latest upstream

2019-03-14 Thread Richard Henderson
On 3/14/19 12:41 PM, Helge Deller wrote: > Hi Richard, > > On 14.03.19 16:26, Richard Henderson wrote: >> On 3/13/19 1:00 PM, Sven Schnelle wrote: >>> Signed-off-by: Sven Schnelle >>> --- >>> pc-bios/hppa-firmware.img | Bin 760040 -> 763944 bytes >>> roms/seabios-hppa | 2 +- >>> 2 fi

Re: [Qemu-devel] [multiprocess RFC PATCH 35/37] multi-process: QMP/HMP commands to resize block device on remote process

2019-03-14 Thread Elena Ufimtseva
On Thu, Mar 07, 2019 at 10:15:36AM -0600, Eric Blake wrote: > On 3/7/19 1:22 AM, elena.ufimts...@oracle.com wrote: > > From: Jagannathan Raman > > > > Adds rblock_resize QMP/HMP commands to resize block devices on the remote > > process. > > > > Signed-off-by: John G Johnson > > Signed-off-by:

Re: [Qemu-devel] [PATCH 1/2] kconfig: add CONFIG_MSI

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 3:30 PM, Paolo Bonzini wrote: > Not all interrupt controllers support message-signalled interrupts, > and some devices *only* support message-signalled interrupts. > > In QEMU this is represented by the "msi_nonbroken" variable. This > patch adds a new configuration symbol enabled when

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

2019-03-14 Thread Richard Henderson
On 3/14/19 2:43 PM, Philippe Mathieu-Daudé wrote: > On 3/14/19 4:39 PM, Daniel P. Berrangé wrote: >> On Wed, Mar 13, 2019 at 09:55:19PM -0700, Richard Henderson wrote: >>> static void make_challenge(VncState *vs) >>> { >>> -int i; >>> - >>> -srand(time(NULL)+getpid()+getpid()*987654+rand(

Re: [Qemu-devel] [PATCH 2/2] kconfig: add dependencies on CONFIG_MSI

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 3:30 PM, Paolo Bonzini wrote: > For devices that require msi_init/msix_init to succeed, add a > dependency on CONFIG_MSI. This will prevent those devices from > appearing in a binary that cannot instantiate them. > > Signed-off-by: Paolo Bonzini > --- > Kconfig.host | 3 +++

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

2019-03-14 Thread BALATON Zoltan
On Thu, 14 Mar 2019, Aleksandar Markovic wrote: From: BALATON Zoltan Subject: Data bus error with redeonfb on mips_fulong2e 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-vg

Re: [Qemu-devel] [PATCH 2/5] trace-events: Shorten file names in comments

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 7:09 PM, Markus Armbruster wrote: > We spell out sub/dir/ in sub/dir/trace-events' comments pointing to > source files. That's because when trace-events got split up, the > comments were moved verbatim. > > Delete the sub/dir/ part from these comments. Gets rid of several > misspel

Re: [Qemu-devel] [PATCH 1/5] trace-events: Consistently point to docs/devel/tracing.txt

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 7:09 PM, Markus Armbruster wrote: > Almost all trace-events do so in a comment right at the beginning. It was not obvious at first that "do so" means "consistently point to docs/devel/tracing.txt". > Touch up the ones that don't. > > Signed-off-by: Markus Armbruster > --- > accel/k

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 5:55 AM, Richard Henderson wrote: > 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(-)

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

2019-03-14 Thread Aleksandar Markovic
Hello, Paul, Zoltan > From: BALATON Zoltan > Subject: Data bus error with redeonfb on mips_fulong2e > > 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=

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 5:55 AM, Richard Henderson wrote: > Use #ifdef _WIN32 instead of #ifndef _WIN32. > This will make other tests easier to sequence. > > Cc: Daniel P. Berrangé > Signed-off-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé > --- > crypto/random-platform.c | 35 +++

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 5:55 AM, Richard Henderson wrote: > 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

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 4:39 PM, Daniel P. Berrangé wrote: > On Wed, Mar 13, 2019 at 09:55:19PM -0700, Richard Henderson wrote: >> 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

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 5:55 AM, Richard Henderson wrote: > 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 Reviewed-by: Philippe Mathieu-Daudé > --- > include/crypto/random.h | 2 +- > cr

Re: [Qemu-devel] [PATCH V2 1/3] audio/paaudio: fix ignored buffer_length setting

2019-03-14 Thread Philippe Mathieu-Daudé
Hi Martin, On 3/14/19 3:26 PM, Martin Schrodt wrote: > Audiodev configuration allows to set the length of the buffered data. The > setting was ignored and a constant value used instead. This patch makes the > code apply the setting properly. > > Signed-off-by: Martin Schrodt > --- > audio/paa

Re: [Qemu-devel] [PATCH] scripts/make-release: Stop shipping u-boot source as a tarball

2019-03-14 Thread Philippe Mathieu-Daudé
Hi Peter, On 3/14/19 4:56 PM, Peter Maydell wrote: > In commit d0dead3b6df7f6cd970e we changed to shipping the u-boot > sources as a tarball, to work around a problem where they > contained a file and directory that had the same name except > for case, which was preventing QEMU's source tarball be

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

2019-03-14 Thread Cédric Le Goater
On 3/14/19 3:11 AM, David Gibson wrote: > 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 0

Re: [Qemu-devel] [PATCH for-4.0] maint: Ignore built elf2dmp

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/13/19 3:45 PM, Eric Blake wrote: > Commit 3fa2d384 added a binary 'elf2dmp' but did not ignore it > during an in-tree build. I'd use ".gitignore" or "elf2dmp" prefix in subject rather than "maint". > > Signed-off-by: Eric Blake Reviewed-by: Philippe Mathieu-Daudé > --- > > Yes, I know

Re: [Qemu-devel] [PATCH] hw/riscv/virt: re-add machine-specific compatible string to /soc/ node

2019-03-14 Thread Auer, Lukas
Hi Bin, On Wed, 2019-03-13 at 09:51 +0800, Bin Meng wrote: > Hi Lukas, > > On Tue, Mar 12, 2019 at 10:39 PM Auer, Lukas > wrote: > > Hi Bin, > > > > On Mon, 2019-03-11 at 23:28 +0800, Bin Meng wrote: > > > Hi Lukas, > > > > > > On Mon, Mar 11, 2019 at 2:03 AM Auer, Lukas > > > wrote: > > > >

Re: [Qemu-devel] [PATCH 6/6] xen-pt: Round pci regions sizes to XEN_PAGE_SIZE

2019-03-14 Thread Simon Gaiser
Jason Andryuk: > On Wed, Mar 13, 2019 at 11:09 AM Paul Durrant wrote: >> >>> -Original Message- >>> From: Jason Andryuk [mailto:jandr...@gmail.com] >>> Sent: 11 March 2019 18:02 >>> To: qemu-devel@nongnu.org >>> Cc: xen-de...@lists.xenproject.org; marma...@invisiblethingslab.com; Simon >>

Re: [Qemu-devel] [PULL 17/29] target/riscv: Convert quadrant 1 of RVXC insns to decodetree

2019-03-14 Thread Alistair Francis
On Wed, Mar 13, 2019 at 7:53 AM Palmer Dabbelt wrote: > > From: Bastian Koppelmann > > Reviewed-by: Richard Henderson > Signed-off-by: Bastian Koppelmann > Signed-off-by: Peer Adelt This commit is the first bad commit in breaking 32-bit boot. It looks like the jal doesn't jump to the correct

[Qemu-devel] [PATCH 0/5] trace-events: Clean up

2019-03-14 Thread Markus Armbruster
Markus Armbruster (5): trace-events: Consistently point to docs/devel/tracing.txt trace-events: Shorten file names in comments scripts/cleanup-trace-events: Update for current practice trace-events: Delete unused trace points trace-events: Fix attribution of trace points to source accel

Re: [Qemu-devel] State of QEMU CI as we enter 4.0

2019-03-14 Thread Peter Maydell
On Thu, 14 Mar 2019 at 15:57, Alex Bennée wrote: > Testing in the Cloud > > > After BuildBot went out-of-service we have been relying heavily on Travis > as our primary CI platform. This has been creaking somewhat under the > strain and while we have a large test matrix its co

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

2019-03-14 Thread Michael S. Tsirkin
On Wed, Mar 13, 2019 at 10:00:45PM +0100, Laszlo Ersek wrote: > 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/20

Re: [Qemu-devel] [PATCH v3 12/12] docs/interop/firmware.json: Prefer -machine to if=pflash

2019-03-14 Thread Markus Armbruster
Michal Privoznik writes: > On 3/8/19 2:14 PM, Markus Armbruster wrote: >> The previous commit added a way to configure firmware with -blockdev >> rather than -drive if=pflash. Document it as the preferred way. >> >> Signed-off-by: Markus Armbruster >> --- >> docs/interop/firmware.json | 20 ++

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/14/19 7:38 PM, Philippe Mathieu-Daudé wrote: > Hi Laszlo, > > On 3/13/19 10:00 PM, Laszlo Ersek wrote: >> 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

Re: [Qemu-devel] [PATCH v5 10/15] s390-bios: Support for running format-0/1 channel programs

2019-03-14 Thread Farhan Ali
On 03/13/2019 12:31 PM, Jason J. Herne wrote: Introduce a library function for executing format-0 and format-1 channel programs and waiting for their completion before continuing execution. Add cu_type() to channel io library. This will be used to query control unit type which is used to dete

Re: [Qemu-devel] [PATCH 0/2] kconfig: add fine-grained dependencies for MSI

2019-03-14 Thread Palmer Dabbelt
On Thu, 14 Mar 2019 10:30:20 PDT (-0700), alistai...@gmail.com wrote: On Thu, Mar 14, 2019 at 10:09 AM Michael S. Tsirkin wrote: On Thu, Mar 14, 2019 at 03:30:30PM +0100, Paolo Bonzini wrote: > RISC-V targets did not include PCIe ports before the Kconfig transition, > and grew them afterwards,

Re: [Qemu-devel] [PATCH 6/6] xen-pt: Round pci regions sizes to XEN_PAGE_SIZE

2019-03-14 Thread Simon Gaiser
Jason Andryuk: > On Wed, Mar 13, 2019 at 11:09 AM Paul Durrant wrote: >> >>> -Original Message- >>> From: Jason Andryuk [mailto:jandr...@gmail.com] >>> Sent: 11 March 2019 18:02 >>> To: qemu-devel@nongnu.org >>> Cc: xen-de...@lists.xenproject.org; marma...@invisiblethingslab.com; Simon >>

Re: [Qemu-devel] [PULL 15/54] build: convert pci.mak to Kconfig

2019-03-14 Thread Markus Armbruster
Paolo Bonzini writes: > On 14/03/19 14:03, David Abdurachmanov wrote: >> Would it be enough to modify ./default-configs/riscv32-softmmu.mak by >> adding: >> >> CONFIG_PCIE_PORT=n > > The missing dependency is for any board that doesn't support MSI, not > just RISC-V. > > Note that technically wh

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

2019-03-14 Thread BALATON Zoltan
On Thu, 14 Mar 2019, Gerd Hoffmann wrote: 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 lik

[Qemu-devel] [PATCH] test-thread-pool: be more reliable

2019-03-14 Thread Paolo Bonzini
There is a rare race between the atomic_cmpxchg and bdrv_aio_cancel/bdrv_aio_cancel_async invocations. Detect it, the only sensible we can do about it is to exit long_cb immediately. Signed-off-by: Paolo Bonzini --- tests/test-thread-pool.c | 32 +--- 1 file changed,

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/13/19 10:00 PM, Laszlo Ersek wrote: > 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 -- thi

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

2019-03-14 Thread Markus Armbruster
Kevin Wolf writes: > Am 13.03.2019 um 18:44 hat Markus Armbruster geschrieben: >> Patch created mechanically by rerunning: >> >> $ spatch --sp-file scripts/coccinelle/qobject.cocci \ >> --macro-file scripts/cocci-macro-file.h \ >> --dir hw/block --in-place >> >> Si

Re: [Qemu-devel] [PATCH 1/1] Update seabios-hppa to latest upstream

2019-03-14 Thread Helge Deller
Hi Richard, On 14.03.19 16:26, Richard Henderson wrote: > On 3/13/19 1:00 PM, Sven Schnelle wrote: >> Signed-off-by: Sven Schnelle >> --- >> pc-bios/hppa-firmware.img | Bin 760040 -> 763944 bytes >> roms/seabios-hppa | 2 +- >> 2 files changed, 1 insertion(+), 1 deletion(-) > > Thank

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/13/19 10:00 PM, Laszlo Ersek wrote: > 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

Re: [Qemu-devel] [PATCH 6/6] xen-pt: Round pci regions sizes to XEN_PAGE_SIZE

2019-03-14 Thread Andrew Cooper
On 14/03/2019 19:22, Simon Gaiser wrote: > Jason Andryuk: >> On Wed, Mar 13, 2019 at 11:09 AM Paul Durrant >> wrote: -Original Message- From: Jason Andryuk [mailto:jandr...@gmail.com] Sent: 11 March 2019 18:02 To: qemu-devel@nongnu.org Cc: xen-de...@lists.xenproje

Re: [Qemu-devel] [PATCH] ati-vga: i2c fix

2019-03-14 Thread BALATON Zoltan
On Thu, 14 Mar 2019, Gerd Hoffmann wrote: gets radeonfb going for me, on top of your i2c patches. --- hw/display/ati_int.h | 1 + hw/display/ati_regs.h | 1 + hw/display/ati.c | 35 +++ 3 files changed, 29 insertions(+), 8 deletions(-) diff --git a/hw/display

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

2019-03-14 Thread Eric Blake
On 3/14/19 1:02 PM, Philippe Mathieu-Daudé wrote: > On 3/13/19 10:00 PM, Laszlo Ersek wrote: >> Update the roms/edk2 submodule hash from edk2-stable201811 to >> edk2-stable201903. The release notes are available at >> . >> >> $ git s

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

2019-03-14 Thread Philippe Mathieu-Daudé
Hi Laszlo, On 3/13/19 10:00 PM, Laszlo Ersek wrote: > 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:

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

2019-03-14 Thread Philippe Mathieu-Daudé
On 3/13/19 10:00 PM, Laszlo Ersek wrote: > 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. > > S

Re: [Qemu-devel] [build-error] possible build error at the tip of the trunk? (dtc-related?)

2019-03-14 Thread Aleksandar Markovic
> From: Peter Maydell > Subject: Re: [Qemu-devel] [build-error] possible build error at the tip of > > the trunk? (dtc-related?) > > On Mon, 11 Mar 2019 at 16:59, Aleksandar Markovic > wrote: > > > > Hello, all > > > > All of the sudden, the latest code doesn't build on my host: (the code > >

  1   2   3   4   >