[RFC v6 05/13] hw/arm/smmu: Fill IOMMUTLBEntry notifier type

2020-08-26 Thread Eugenio Pérez
Signed-off-by: Eugenio Pérez --- hw/arm/smmu-common.c | 2 ++ hw/arm/smmuv3.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/hw/arm/smmu-common.c b/hw/arm/smmu-common.c index b02ffb8822..88cf1b86ea 100644 --- a/hw/arm/smmu-common.c +++ b/hw/arm/smmu-common.c @@ -181,6 +181,7 @@

[RFC v6 00/13] memory: Delete assertion in memory_region_unregister_iommu_notifier

2020-08-26 Thread Eugenio Pérez
I am able to hit this assertion when a Red Hat 7 guest virtio_net device raises an "Invalidation" of all the TLB entries. This happens in the guest's startup if 'intel_iommu=on' argument is passed to the guest kernel and right IOMMU/ATS devices are declared in qemu's command line. Command line:

[RFC v6 01/13] memory: Rename memory_region_notify_one to memory_region_notify_iommu_one

2020-08-26 Thread Eugenio Pérez
Previous name didn't reflect the iommu operation. Signed-off-by: Eugenio Pérez --- hw/arm/smmu-common.c | 2 +- hw/arm/smmuv3.c | 2 +- hw/i386/intel_iommu.c | 4 ++-- include/exec/memory.h | 6 +++--- softmmu/memory.c | 6 +++--- 5 files changed, 10 insertions(+), 10 deletions(-)

Re: [RFC PATCH v3 13/34] Hexagon (target/hexagon) register map

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > +#ifndef HEXAGON_REGMAP_H > +#define HEXAGON_REGMAP_H > + > +/* Name Num Table */ > +DEF_REGMAP(R_16, 16, 0, 1, 2, 3, 4, 5, 6, 7, 16, 17, 18, 19, 20, 21, 22, 23) > +DEF_REGMAP(R__8, 8, 0, 2, 4, 6, 16, 18, 20, 22) > +DEF_REGMAP(R__4, 4, 0,

Re: [PULL v2 00/34] Block patches

2020-08-26 Thread Peter Maydell
On Wed, 26 Aug 2020 at 08:51, Max Reitz wrote: > > The following changes since commit 30aa19446d82358a30eac3b556b4d6641e00b7c1: > > Merge remote-tracking branch 'remotes/cschoenebeck/tags/pull-9p-20200812' > into staging (2020-08-24 16:39:53 +0100) > > are available in the Git repository at: >

Re: [RFC PATCH v3 12/34] Hexagon (target/hexagon) instruction attributes

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > +#define ATTRIB_WIDTH 32 > +#define GET_ATTRIB(opcode, attrib) \ > +(((opcode_attribs[opcode][attrib / ATTRIB_WIDTH])\ > +>> (attrib % ATTRIB_WIDTH)) & 0x1) Can you define GET_ATTRIB in terms of qemu/bitops.h? I'm leery of ATTRIB_WIDTH being

Re: [RFC PATCH v3 11/34] Hexagon (target/hexagon) register fields

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > Declare bitfields within registers such as user status register (USR) > > Signed-off-by: Taylor Simpson > --- > target/hexagon/reg_fields.h | 40 + > target/hexagon/reg_fields_def.h | 78 >

Re: [RFC PATCH v3 10/34] Hexagon (target/hexagon) instruction and packet types

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > +struct Instruction { > +semantic_insn_t generate;/* pointer to genptr routine */ > +size1u_t regno[REG_OPERANDS_MAX];/* reg operands including > predicates */ > +size2u_t opcode; > + > +size4u_t iclass:6; > +size4u_t

[PATCH v2 1/2] hw/misc/sbsa_ec : Add an embedded controller for sbsa-ref

2020-08-26 Thread Graeme Gregory
A difference between sbsa platform and the virt platform is PSCI is handled by ARM-TF in the sbsa platform. This means that the PSCI code there needs to communicate some of the platform power changes down to the qemu code for things like shutdown/reset control. Space has been left to extend the

[PATCH v2 2/2] hw/arm/sbsa-ref : Add embedded controller in secure memory

2020-08-26 Thread Graeme Gregory
Add the previously created sbsa-ec device to the sbsa-ref machine in secure memory so the PSCI implementation in ARM-TF can access it, but not expose it to non secure firmware or OS except by via ARM-TF. Signed-off-by: Graeme Gregory --- hw/arm/sbsa-ref.c | 14 ++ 1 file changed, 14

[PATCH v2 0/2] Add an embedded controller to sbsa-ref machine

2020-08-26 Thread Graeme Gregory
This series is to an add embedded controller to the sbsa-ref machine so that PSCI can communicate platform power states to the platform which in this case is QEMU. v1->v2 - broke out the EC itself as hw/misc/sbsa_ec.c as seperate patch - applied review comments to date

Re: [RFC PATCH v3 09/34] Hexagon (target/hexagon) architecture types

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > +#ifndef HEXAGON_ARCH_TYPES_H > +#define HEXAGON_ARCH_TYPES_H > + > +#include Do you really need to re-include this? This was done in "qemu/osdep.h". In general, osdep.h must be included first, and it takes care of all of the basic system includes.

Re: [PATCH] numa: hmat: fix cache size check

2020-08-26 Thread Eduardo Habkost
On Fri, Aug 21, 2020 at 06:05:19AM -0400, Igor Mammedov wrote: > when QEMU is started like: > > qemu-system-x86_64 -smp 2 -machine hmat=on \ > -m 2G \ > -object memory-backend-ram,size=1G,id=m0 \ > -object memory-backend-ram,size=1G,id=m1 \ > -numa node,nodeid=0,memdev=m0 \ > -numa

Re: [RFC PATCH v3 08/34] Hexagon (target/hexagon) GDB Stub

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > GDB register read and write routines > > Signed-off-by: Taylor Simpson > --- > target/hexagon/internal.h | 2 ++ > target/hexagon/cpu.c | 2 ++ > target/hexagon/gdbstub.c | 49 > +++ > 3 files

Re: [RFC PATCH v3 07/34] Hexagon (target/hexagon) scalar core helpers

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > The majority of helpers are generated. Define the helper functions needed > then include the generated file > > Signed-off-by: Taylor Simpson > --- > target/hexagon/helper.h| 33 > target/hexagon/op_helper.c | 368 >

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Dr. David Alan Gilbert
* Igor Mammedov (imamm...@redhat.com) wrote: > On Tue, 25 Aug 2020 16:25:21 +0100 > "Dr. David Alan Gilbert" wrote: > > > * Igor Mammedov (imamm...@redhat.com) wrote: > > > On Tue, 25 Aug 2020 09:15:04 +0100 > > > "Dr. David Alan Gilbert" wrote: > > > > > > > * Babu Moger

Re: [PATCH v2 6/7] x68: acpi: trigger SMI before sending hotplug Notify event to OSPM

2020-08-26 Thread Igor Mammedov
On Wed, 26 Aug 2020 15:32:07 +0200 Laszlo Ersek wrote: > On 08/26/20 11:24, Laszlo Ersek wrote: > > Hi Igor, > > > > On 08/25/20 19:25, Laszlo Ersek wrote: > > > >> So I would suggest fetching the CNEW array element back into "uid" > >> first, then using "uid" for both the NOTIFY call, and

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Eduardo Habkost
On Wed, Aug 26, 2020 at 02:38:49PM +0200, Igor Mammedov wrote: > On Fri, 21 Aug 2020 17:12:19 -0500 > Babu Moger wrote: > > > To support some of the complex topology, we introduced EPYC mode apicid > > decode. > > But, EPYC mode decode is running into problems. Also it can become quite a > >

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Igor Mammedov
On Wed, 26 Aug 2020 14:36:38 +0100 Daniel P. Berrangé wrote: > On Wed, Aug 26, 2020 at 03:30:34PM +0200, Igor Mammedov wrote: > > On Wed, 26 Aug 2020 13:50:59 +0100 > > Daniel P. Berrangé wrote: > > > > > On Wed, Aug 26, 2020 at 02:38:49PM +0200, Igor Mammedov wrote: > > > > On Fri, 21 Aug

Re: [PATCH v2 2/2] linux-user: Add support for utimensat_time64() and semtimedop_time64()

2020-08-26 Thread Laurent Vivier
Le 25/08/2020 à 16:23, Laurent Vivier a écrit : > Le 25/08/2020 à 09:18, Laurent Vivier a écrit : >> Le 25/08/2020 à 00:30, Filip Bozuta a écrit : >>> This patch introduces functionality for following time64 syscalls: >>> >>> *utimensat_time64() >>> >>> int utimensat(int dirfd, const char

Re: [Fwd] Issue 25164 in oss-fuzz: qemu: Fuzzing build failure

2020-08-26 Thread Alexander Bulekov
On 200825 0958, Paolo Bonzini wrote: > Il lun 24 ago 2020, 00:58 Alexander Bulekov ha scritto: > > > Hi Paolo, > > Our oss-fuzz builds started failing, after the meson merge. I think I > > tracked down the issues: > > 1.) Looking at the build-log here: > > > >

Re: Issue with submodules on macOS & meson

2020-08-26 Thread Emmanuel Blot
On 26 Aug 2020, at 15:50, Marc-André Lureau wrote: H Marc-André, >> You need to run 'make' (at least once - but still by preference for now). > Submodule handling is done by Makefile. Ok, thanks a lot for this tip.

Re: [RFC PATCH v3 06/34] Hexagon (disas) disassembler

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > +len = disassemble_hexagon(words, i, buf, PACKET_BUFFER_LEN); > +slen = strlen(buf); > +if (buf[slen - 1] == '\n') { > +buf[slen - 1] = '\0'; > +} > +(*info->fprintf_func)(info->stream, "%s", buf); Normally our disassemblers

Re: [PATCH v5 07/10] block: introduce preallocate filter

2020-08-26 Thread David Edmondson
On Tuesday, 2020-08-25 at 17:11:34 +02, Max Reitz wrote: > On 21.08.20 16:11, Vladimir Sementsov-Ogievskiy wrote: >> It's intended to be inserted between format and protocol nodes to >> preallocate additional space (expanding protocol file) on writes >> crossing EOF. It improves performance for

[Bug 1893040] Re: External modules retreval using Go1.15 on s390x appears to have checksum and ECDSA verification issues

2020-08-26 Thread Daniel Berrange
Can you provide a *simple* way to demonstrate the problem. ie some simple Go demo program, that doens't involve building kubernetes. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1893040 Title:

Re: Issue with submodules on macOS & meson

2020-08-26 Thread Marc-André Lureau
Hi On Wed, Aug 26, 2020 at 5:40 PM Emmanuel Blot wrote: > Hi, > > Using current master 78dca230 w/ Meson/ninja, build fails with capstone > dependency. > > * ../configure --target-list=riscv64-softmmu && ninja > You need to run 'make' (at least once - but still by preference for now).

Re: [PATCH] Coroutine-aware monitor_cur() with coroutine-specific data

2020-08-26 Thread Kevin Wolf
Am 26.08.2020 um 14:40 hat Markus Armbruster geschrieben: > Kevin Wolf writes: > > > Am 07.08.2020 um 15:29 hat Markus Armbruster geschrieben: > >> This is just a sketch. It needs comments and a real commit message. > >> > >> As is, it goes on top of Kevin's series. It is meant to be squashed

PTHREAD_MUTEX_ERRORCHECK and fork()

2020-08-26 Thread Christian Schoenebeck
On Freitag, 21. August 2020 15:13:35 CEST Paolo Bonzini wrote: > On 20/08/20 14:00, Christian Schoenebeck wrote: > > One way would be a recursive type and logging a warning, which you > > obviously don't like; another option would be an assertion fault instead > > to make developers immediately

Re: [PATCH v3 0/8] meson: mingw installation fixes & nsis conversion

2020-08-26 Thread Paolo Bonzini
We should document the difference in the 5.2 changelog then (it's okay to introduce it since the configure option is named differently). Il mer 26 ago 2020, 13:50 Marc-André Lureau ha scritto: > Hi > > On Wed, Aug 26, 2020 at 3:38 PM Paolo Bonzini wrote: > >> Since behavioral changes should be

[PATCH] os-posix: Restore firmware location ../share/qemu

2020-08-26 Thread azoff
Prior to commit 6dd2dacedd83d12328afa8559bffb2b9ec5c89ed (v5.0.0), the binary relative path ../share/qemu was searched for firmware, but in that commit, this path got lost. Consider the following use-case: * QEMU is built in a docker image on one system. * QEMU is supposed to be executed on a

Issue with submodules on macOS & meson

2020-08-26 Thread Emmanuel Blot
Hi, Using current master 78dca230 w/ Meson/ninja, build fails with capstone dependency. * ../configure --target-list=riscv64-softmmu && ninja fails because capstone is automatically enabled @ configure stage, but capstone is not installed on the host, and the local capstone/ submodule is

Re: [RFC PATCH v3 05/34] Hexagon (target/hexagon) register names

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > Signed-off-by: Taylor Simpson > --- > target/hexagon/hex_regs.h | 83 > +++ > 1 file changed, 83 insertions(+) > create mode 100644 target/hexagon/hex_regs.h Noting for the record that this apparent

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Daniel P . Berrangé
On Wed, Aug 26, 2020 at 03:30:34PM +0200, Igor Mammedov wrote: > On Wed, 26 Aug 2020 13:50:59 +0100 > Daniel P. Berrangé wrote: > > > On Wed, Aug 26, 2020 at 02:38:49PM +0200, Igor Mammedov wrote: > > > On Fri, 21 Aug 2020 17:12:19 -0500 > > > Babu Moger wrote: > > > > > > > To support some

Re: [RFC PATCH v3 04/34] Hexagon (target/hexagon) scalar core definition

2020-08-26 Thread Richard Henderson
On 8/18/20 8:50 AM, Taylor Simpson wrote: > +#include This should not be in cpu.h. What's up? > +#define TARGET_PAGE_BITS 16 /* 64K pages */ > +#define TARGET_LONG_BITS 32 Belongs in cpu-param.h > +#ifdef CONFIG_USER_ONLY > +#define TOTAL_PER_THREAD_REGS 64 > +#else ... > +

Re: [PATCH v2 6/7] x68: acpi: trigger SMI before sending hotplug Notify event to OSPM

2020-08-26 Thread Laszlo Ersek
On 08/26/20 15:32, Laszlo Ersek wrote: > On 08/26/20 11:24, Laszlo Ersek wrote: >> Hi Igor, >> >> On 08/25/20 19:25, Laszlo Ersek wrote: >> >>> So I would suggest fetching the CNEW array element back into "uid" >>> first, then using "uid" for both the NOTIFY call, and the (currently >>> missing)

Re: [PATCH v2 6/7] x68: acpi: trigger SMI before sending hotplug Notify event to OSPM

2020-08-26 Thread Laszlo Ersek
On 08/26/20 11:24, Laszlo Ersek wrote: > Hi Igor, > > On 08/25/20 19:25, Laszlo Ersek wrote: > >> So I would suggest fetching the CNEW array element back into "uid" >> first, then using "uid" for both the NOTIFY call, and the (currently >> missing) restoration of CSEL. Then we can write 1 to

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Igor Mammedov
On Wed, 26 Aug 2020 13:50:59 +0100 Daniel P. Berrangé wrote: > On Wed, Aug 26, 2020 at 02:38:49PM +0200, Igor Mammedov wrote: > > On Fri, 21 Aug 2020 17:12:19 -0500 > > Babu Moger wrote: > > > > > To support some of the complex topology, we introduced EPYC mode apicid > > > decode. > > >

[PATCH] block/mirror: fix core when using iothreads

2020-08-26 Thread Peng Liang
We found an issue when doing block-commit with iothreads, which tries to dereference a NULL pointer. | | mirror_start_job | 1. bdrv_ref(mirror_top_bs); | bdrv_drained_begin(bs);|

Re: [PATCH v3] block: Raise an error when backing file parameter is an empty string

2020-08-26 Thread Kevin Wolf
Am 13.08.2020 um 15:47 hat Connor Kuehl geschrieben: > Providing an empty string for the backing file parameter like so: > > qemu-img create -f qcow2 -b '' /tmp/foo > > allows the flow of control to reach and subsequently fail an assert > statement because passing an empty string to > >

[Bug 1893040] Re: External modules retreval using Go1.15 on s390x appears to have checksum and ECDSA verification issues

2020-08-26 Thread Guirish Salgaonkar
Yes we have observed that the issue persist in later QEMU version too. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1893040 Title: External modules retreval using Go1.15 on s390x appears to have

Meson build on macOS: undefined symbol treatment

2020-08-26 Thread Emmanuel Blot
Hi, I’ve noticed that since meson builds have been enabled, on macOS, the build outcome with undefined symbols has changed: - prior to meson introduction (v5.1.0): * referencing an undeclared symbol in source code led to a warning at compile stage * referencing an undeclared symbol at

Re: [PATCH] meson: install pc-bios blobs

2020-08-26 Thread Marc-André Lureau
Hi On Wed, Aug 26, 2020 at 3:34 PM wrote: > From: Marc-André Lureau > > Signed-off-by: Marc-André Lureau > --- > Makefile| 31 -- > pc-bios/meson.build | 63 + > 2 files changed, 63 insertions(+), 31 deletions(-) > >

[PATCH 6/6] build-sys: remove install target from Makefile

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Now covered by meson Signed-off-by: Marc-André Lureau --- Makefile | 5 - 1 file changed, 5 deletions(-) diff --git a/Makefile b/Makefile index 9a9e7c0301..75db8be52e 100644 --- a/Makefile +++ b/Makefile @@ -229,13 +229,8 @@ distclean: clean ninja-distclean

[PATCH 1/6] meson: install pc-bios blobs

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile| 31 -- pc-bios/meson.build | 63 + 2 files changed, 63 insertions(+), 31 deletions(-) diff --git a/Makefile b/Makefile index

[PATCH 4/6] meson: install desktop file

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile | 3 --- ui/meson.build | 2 ++ 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/Makefile b/Makefile index d1a3cd77a5..dc3c20dd5e 100644 --- a/Makefile +++ b/Makefile @@ -243,9 +243,6 @@ endif # Needed

[PATCH 5/6] meson: install $localstatedir/run for qga

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile| 10 +- qga/meson.build | 2 ++ 2 files changed, 3 insertions(+), 9 deletions(-) diff --git a/Makefile b/Makefile index dc3c20dd5e..9a9e7c0301 100644 --- a/Makefile +++ b/Makefile @@ -232,17 +232,9 @@

[PATCH 2/6] meson: install scripts/qemu-trace-stap

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile| 3 --- meson.build | 1 + scripts/meson.build | 3 +++ 3 files changed, 4 insertions(+), 3 deletions(-) create mode 100644 scripts/meson.build diff --git a/Makefile b/Makefile index

[PATCH 0/6] meson: convert the remaining of install target

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Hi, A few patches to finish replacing the Makefile install target by meson. Marc-André Lureau (6): meson: install pc-bios blobs meson: install scripts/qemu-trace-stap meson: install icons meson: install desktop file meson: install $localstatedir/run for qga

[PATCH 3/6] meson: install icons

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile | 12 ui/icons/meson.build | 13 + ui/meson.build | 1 + 3 files changed, 14 insertions(+), 12 deletions(-) create mode 100644 ui/icons/meson.build diff --git a/Makefile

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Daniel P . Berrangé
On Wed, Aug 26, 2020 at 02:38:49PM +0200, Igor Mammedov wrote: > On Fri, 21 Aug 2020 17:12:19 -0500 > Babu Moger wrote: > > > To support some of the complex topology, we introduced EPYC mode apicid > > decode. > > But, EPYC mode decode is running into problems. Also it can become quite a > >

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Igor Mammedov
On Tue, 25 Aug 2020 16:25:21 +0100 "Dr. David Alan Gilbert" wrote: > * Igor Mammedov (imamm...@redhat.com) wrote: > > On Tue, 25 Aug 2020 09:15:04 +0100 > > "Dr. David Alan Gilbert" wrote: > > > > > * Babu Moger (babu.mo...@amd.com) wrote: > > > > Hi Dave, > > > > > > > > On 8/24/20 1:41

Re: [PATCH v5 0/8] Remove EPYC mode apicid decode and use generic decode

2020-08-26 Thread Igor Mammedov
On Fri, 21 Aug 2020 17:12:19 -0500 Babu Moger wrote: > To support some of the complex topology, we introduced EPYC mode apicid > decode. > But, EPYC mode decode is running into problems. Also it can become quite a > maintenance problem in the future. So, it was decided to remove that code and >

Re: [PATCH] Coroutine-aware monitor_cur() with coroutine-specific data

2020-08-26 Thread Markus Armbruster
Kevin Wolf writes: > Am 07.08.2020 um 15:29 hat Markus Armbruster geschrieben: >> This is just a sketch. It needs comments and a real commit message. >> >> As is, it goes on top of Kevin's series. It is meant to be squashed >> into PATCH 06. >> >> Signed-off-by: Markus Armbruster >> --- >>

Re: [PATCH] Simple & stupid coroutine-aware monitor_cur()

2020-08-26 Thread Markus Armbruster
Kevin Wolf writes: > Am 07.08.2020 um 15:27 hat Markus Armbruster geschrieben: >> This is just a sketch. It's incomplete, needs comments and a real >> commit message. >> >> Support for "[PATCH v6 09/12] hmp: Add support for coroutine command >> handlers" is missing. Marked FIXME. >> >> As

Re: [PATCH 08/12] migration/colo: Plug memleaks in colo_process_incoming_thread

2020-08-26 Thread Li Qiang
Pan Nengyuan 于2020年8月14日周五 下午6:52写道: > > 'local_err' forgot to free in colo_process_incoming_thread error path. > Fix that. > > Reported-by: Euler Robot > Signed-off-by: Pan Nengyuan > --- > Cc: Hailiang Zhang > Cc: Juan Quintela > Cc: "Dr. David Alan Gilbert" > --- > migration/colo.c | 2

Re: [PATCH v5 06/12] migration/dirtyrate: Record hash results for each sampled page

2020-08-26 Thread Dr. David Alan Gilbert
* Chuan Zheng (zhengch...@huawei.com) wrote: > Record hash results for each sampled page, crc32 is taken to calculate > hash results for each sampled 4K-page. > > Signed-off-by: Chuan Zheng > Signed-off-by: YanYing Zhuang > --- > migration/dirtyrate.c | 136 >

Re: [PATCH v5 06/12] migration/dirtyrate: Record hash results for each sampled page

2020-08-26 Thread David Edmondson
On Wednesday, 2020-08-26 at 13:30:16 +01, Dr. David Alan Gilbert wrote: > * David Edmondson (d...@dme.org) wrote: >> On Monday, 2020-08-24 at 17:14:34 +08, Chuan Zheng wrote: >> >> > Record hash results for each sampled page, crc32 is taken to calculate >> > hash results for each sampled

[Bug 1883984] Re: QEMU S/390x sqxbr (128-bit IEEE 754 square root) crashes qemu-system-s390x

2020-08-26 Thread Christian Ehrhardt 
old version sudo apt install qemu-system-s390x=1:4.2-3ubuntu6.4 ...test as listed in the test instructions ... ubuntu@focal-sqxbr:~$ ./a.out Segmentation fault (qemu is dead at this point) $ sudo apt install qemu-system-s390x=1:4.2-3ubuntu6.5 Reading package lists... Done Building dependency

Re: [PATCH v5 06/12] migration/dirtyrate: Record hash results for each sampled page

2020-08-26 Thread Dr. David Alan Gilbert
* David Edmondson (d...@dme.org) wrote: > On Monday, 2020-08-24 at 17:14:34 +08, Chuan Zheng wrote: > > > Record hash results for each sampled page, crc32 is taken to calculate > > hash results for each sampled 4K-page. > > > > Signed-off-by: Chuan Zheng > > Signed-off-by: YanYing Zhuang > >

Re: [PATCH v3] virtio-rng: return available data with O_NONBLOCK

2020-08-26 Thread Michael S. Tsirkin
On Tue, Aug 11, 2020 at 04:42:32PM +0200, Laurent Vivier wrote: > On 11/08/2020 16:28, mwi...@suse.com wrote: > > From: Martin Wilck > > > > If a program opens /dev/hwrng with O_NONBLOCK and uses poll() and > > non-blocking read() to retrieve random data, it ends up in a tight > > loop with

Re: [PATCH 07/12] target/i386/cpu: Fix memleak in x86_cpu_class_check_missing_features

2020-08-26 Thread Li Qiang
Pan Nengyuan 于2020年8月14日周五 下午6:38写道: > > 'err' forgot to free in x86_cpu_class_check_missing_features error path. > Fix that. > > Reported-by: Euler Robot > Signed-off-by: Pan Nengyuan Reviewed-by: Li Qiang > --- > Cc: Paolo Bonzini > Cc: Richard Henderson > Cc: Eduardo Habkost > --- >

Re: [PATCH 06/12] ui/gtk-gl-area: Plug memleak in gd_gl_area_create_context()

2020-08-26 Thread Li Qiang
Pan Nengyuan 于2020年8月14日周五 下午6:15写道: > > Receiving error in local variable err, and forgot to free it. > Considering that there is no place to deal with it. Clean up. > > Reported-by: Euler Robot > Signed-off-by: Pan Nengyuan > --- > Cc: Gerd Hoffmann > --- > ui/gtk-gl-area.c | 5 ++--- > 1

Re: [PATCH v5 8/8] i386: Simplify CPUID_8000_001E for AMD

2020-08-26 Thread Igor Mammedov
On Fri, 21 Aug 2020 17:13:09 -0500 Babu Moger wrote: > apic_id contains all the information required to build > CPUID_8000_001E. core_id and node_id is already part of > apic_id generated by x86_topo_ids_from_apicid_epyc. > Also remove the restriction on number bits on core_id and > node_id. >

[PATCH 2/3] blockdev: add sequential mode to *-backup transactions

2020-08-26 Thread Stefan Reiter
Only supported with completion-mode 'grouped', since it relies on a JobTxn to exist. This means that for now it is only available for {drive,blockdev}-backup transactions. Since only one job will be running at a time, bandwidth-limits can be applied effectively. It can also prevent overloading a

[PATCH 1/3] job: add sequential transaction support

2020-08-26 Thread Stefan Reiter
Jobs in a sequential transaction should never be started with job_start manually. job_txn_start_seq and the sequentially called job_start will take care of it, 'assert'ing in case a job is already running or has finished. Signed-off-by: Stefan Reiter --- include/qemu/job.h | 12

[PATCH 0/3] Add support for sequential backups

2020-08-26 Thread Stefan Reiter
Backups can already be done for multiple drives in a transaction. However, these jobs will start all at once, potentially hogging a lot of disk IO all at once. This problem is made worse, since IO throttling is only available on a per-job basis. Add a flag to QMP to support sequential

[PATCH 3/3] backup: initialize bcs bitmap on job create, not start

2020-08-26 Thread Stefan Reiter
After backup_init_bcs_bitmap the copy-before-write behaviour is active. This way, multiple backup jobs created at once but running in a sequential transaction will still represent the same point in time. Signed-off-by: Stefan Reiter --- I'd imagine this was done on job start for a purpose, so

[Bug 1893040] Re: External modules retreval using Go1.15 on s390x appears to have checksum and ECDSA verification issues

2020-08-26 Thread Daniel Berrange
> We are observing issue while building go-runner image and we suspect it is > due to QEMU version > being used. As referred in below issue: > https://github.com/golang/go/issues/40949 This issue says the problem was due to https://bugs.launchpad.net/qemu/+bug/1847232/ which was fixed in QEMU

Re: [PATCH v5 04/12] migration/dirtyrate: Add dirtyrate statistics series functions

2020-08-26 Thread Dr. David Alan Gilbert
* Chuan Zheng (zhengch...@huawei.com) wrote: > Add dirtyrate statistics to record/update dirtyrate info. > > Signed-off-by: Chuan Zheng > --- > migration/dirtyrate.c | 29 + > migration/dirtyrate.h | 10 ++ > 2 files changed, 39 insertions(+) > > diff --git

Re: [PATCH V8] Introduce a new flag for i440fx to disable PCI hotplug on the root bus

2020-08-26 Thread Igor Mammedov
On Mon, 24 Aug 2020 20:32:04 +0530 Ani Sinha wrote: > Reminder to kindly review this patch. I'm a bit occupied elsewhere right now, will look into it later when more urgent tasks are dealt with. Maybe Julia can help with review (CCed) (she should be familiar with the pci hotplug code atm) >

Re: [PATCH v5 03/12] migration/dirtyrate: Add RamlockDirtyInfo to store sampled page info

2020-08-26 Thread Dr. David Alan Gilbert
* Chuan Zheng (zhengch...@huawei.com) wrote: > Add RamlockDirtyInfo to store sampled page info of each ramblock. Note typo 'RAM*B*lockDirtyInfo' (and in the title. > Signed-off-by: Chuan Zheng > --- > migration/dirtyrate.h | 18 ++ > 1 file changed, 18 insertions(+) > > diff

Re: [PATCH 01/12] qga/channel-posix: Plug memory leak in ga_channel_write_all()

2020-08-26 Thread Li Qiang
Pan Nengyuan 于2020年8月14日周五 下午6:30写道: > > Missing g_error_free on error path in ga_channel_write_all(). Fix that. > > Reported-by: Euler Robot > Signed-off-by: Pan Nengyuan Reviewed-by: Li Qiang > --- > Cc: Michael Roth > --- > qga/channel-posix.c | 6 +- > 1 file changed, 5

Re: [PATCH v2 6/7] x68: acpi: trigger SMI before sending hotplug Notify event to OSPM

2020-08-26 Thread Igor Mammedov
On Wed, 26 Aug 2020 11:24:14 +0200 Laszlo Ersek wrote: > Hi Igor, > > On 08/25/20 19:25, Laszlo Ersek wrote: > > > So I would suggest fetching the CNEW array element back into "uid" > > first, then using "uid" for both the NOTIFY call, and the (currently > > missing) restoration of CSEL. Then

Re: [PATCH 02/12] hw/vfio/ap: Plug memleak in vfio_ap_get_group()

2020-08-26 Thread Li Qiang
Cornelia Huck 于2020年8月26日周三 下午7:41写道: > > On Wed, 26 Aug 2020 19:03:37 +0800 > Li Qiang wrote: > > > Pan Nengyuan 于2020年8月14日周五 下午6:29写道: > > > > > > Missing g_error_free() in vfio_ap_get_group() error path. Fix that. > > > > > > Reported-by: Euler Robot > > > Signed-off-by: Pan Nengyuan > >

Re: [PATCH v3 0/8] meson: mingw installation fixes & nsis conversion

2020-08-26 Thread Marc-André Lureau
Hi On Wed, Aug 26, 2020 at 3:38 PM Paolo Bonzini wrote: > Since behavioral changes should be reviewed separately let's just include > v2 first. > > I am more confident that this version doesn't introduce regressions though.. The use of + qemu_confsuffix was problematic, and I noticed some

[Bug 1893040] [NEW] External modules retreval using Go1.15 on s390x appears to have checksum and ECDSA verification issues

2020-08-26 Thread Guirish Salgaonkar
Public bug reported: We are observing issue while building go-runner image and we suspect it is due to QEMU version being used. As referred in below issue: https://github.com/golang/go/issues/40949 We tried to build go-runner image using go1.15 and register QEMU (docker run --rm --privileged

Re: [PATCH v5 02/12] migration/dirtyrate: add DirtyRateStatus to denote calculation status

2020-08-26 Thread Dr. David Alan Gilbert
* Chuan Zheng (zhengch...@huawei.com) wrote: > add DirtyRateStatus to denote calculating status. > > Signed-off-by: Chuan Zheng With the minor wording changes from David Edmondson: Reviewed-by: Dr. David Alan Gilbert > --- > migration/dirtyrate.c | 22 ++ >

[PATCH v3 12/12] hw/usb: Add U2F device autoscan to passthru mode

2020-08-26 Thread César Belley
This patch adds an autoscan to let u2f-passthru choose the first U2F device it finds. The autoscan is performed using libudev with an enumeration of all the hidraw devices present on the host. The first device which happens to be a U2F device is taken to do the passtru. Signed-off-by: César

[PATCH v3 09/12] docs/qdev-device-use.txt: Add USB U2F key to the QDEV devices examples

2020-08-26 Thread César Belley
Signed-off-by: César Belley --- docs/qdev-device-use.txt | 1 + 1 file changed, 1 insertion(+) diff --git a/docs/qdev-device-use.txt b/docs/qdev-device-use.txt index f8d0d2fe29..9889521e3c 100644 --- a/docs/qdev-device-use.txt +++ b/docs/qdev-device-use.txt @@ -325,6 +325,7 @@ The new way is

[PATCH v3 06/12] hw/usb: Add U2F key emulated mode

2020-08-26 Thread César Belley
This patch adds the U2F key emulated mode. The emulated mode consists of completely emulating the behavior of a U2F device through software part. Libu2f-emu is used for that. The emulated mode is associated with a device inheriting from u2f-key base. To work, an emulated U2F device must have

[PATCH v3 07/12] meson: Add U2F key to meson

2020-08-26 Thread César Belley
Signed-off-by: César Belley --- configure | 8 +++- hw/usb/Kconfig | 5 + hw/usb/meson.build | 7 +++ meson.build| 7 +++ meson_options.txt | 1 + 5 files changed, 27 insertions(+), 1 deletion(-) diff --git a/configure b/configure index

[PATCH v3 10/12] scripts: Add u2f-setup-gen script

2020-08-26 Thread César Belley
This patch adds the script used to generate setup directories, needed for the device u2f-emulated configuration in directory mode: python u2f-setup-gen.py $DIR qemu -usb -device u2f-emulated,dir=$DIR Signed-off-by: César Belley --- scripts/u2f-setup-gen.py | 170

[PATCH v3 08/12] docs/system: Add U2F key to the USB devices examples

2020-08-26 Thread César Belley
Signed-off-by: César Belley --- docs/system/usb.rst | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/system/usb.rst b/docs/system/usb.rst index ddfa828d74..9a2f1927c4 100644 --- a/docs/system/usb.rst +++ b/docs/system/usb.rst @@ -81,6 +81,9 @@ option or the ``device_add`` monitor

[PATCH v3 05/12] hw/usb: Add U2F key passthru mode

2020-08-26 Thread César Belley
This patch adds the U2F key pass-through mode. The pass-through mode consists of passing all requests made from the guest to the physical security key connected to the host machine and vice versa. In addition, the dedicated pass-through allows to have a U2F security key shared on several guests

[PATCH v3 04/12] hw/usb: Add U2F key base class implementation

2020-08-26 Thread César Belley
This patch adds the U2F key base class implementation. The U2F key base mainly takes care of the HID interfacing with guest. On the one hand, it retrieves the guest U2FHID packets and transmits them to the variant associated according to the mode: pass-through or emulated. On the other hand, it

[PATCH v3 02/12] docs: Add USB U2F key device documentation

2020-08-26 Thread César Belley
Add USB U2F key device documentation: - USB U2F key device - Building - Using u2f-emulated - Using u2f-passthru - Libu2f-emu Signed-off-by: César Belley --- docs/u2f.txt | 101 +++ 1 file changed, 101 insertions(+) create mode 100644 docs/u2f.txt

[PATCH v3 00/12] Introduce USB U2F key device

2020-08-26 Thread César Belley
This patch series adds the U2F dedicated support to Qemu through a USB U2F key device that can operate in two modes: pass-through and emulated. This work is also part of the GSoC program of this year and follows the proposal I made for the 'Virtual FIDO2/U2F security key' subject, suggested and

[PATCH v3 11/12] hw/usb: Add U2F device check to passthru mode

2020-08-26 Thread César Belley
This patchs adds a check to verify that the device passed through the hidraw property is a U2F device. The check is done by ensuring that the first values of the report descriptor (USAGE PAGE and USAGE) correspond to those of a U2F device. Signed-off-by: César Belley --- hw/usb/u2f-passthru.c

[PATCH v3 01/12] hw/usb: Regroup USB HID protocol values

2020-08-26 Thread César Belley
Group some HID values that are used pretty much everywhere when dealing with HID devices. Signed-off-by: César Belley --- hw/usb/dev-hid.c | 26 +++--- hw/usb/dev-wacom.c | 12 +++- include/hw/usb/hid.h | 17 + 3 files changed, 27 insertions(+),

[PATCH v3 03/12] hw/usb: Add U2F key base class

2020-08-26 Thread César Belley
This patch adds the specification for the U2F key base class. Used to group the common characteristics, this device class will be inherited by its two variants, corresponding to the two modes: passthrough and emulated This prepares the U2F devices hierarchy which is as follow: USB device ->

Re: [PATCH 02/12] hw/vfio/ap: Plug memleak in vfio_ap_get_group()

2020-08-26 Thread Cornelia Huck
On Wed, 26 Aug 2020 19:03:37 +0800 Li Qiang wrote: > Pan Nengyuan 于2020年8月14日周五 下午6:29写道: > > > > Missing g_error_free() in vfio_ap_get_group() error path. Fix that. > > > > Reported-by: Euler Robot > > Signed-off-by: Pan Nengyuan > > Reviewed-by: Li Qiang > > I see Cornelia Huck has

Re: [PATCH v3 0/8] meson: mingw installation fixes & nsis conversion

2020-08-26 Thread Paolo Bonzini
Since behavioral changes should be reviewed separately let's just include v2 first. Paolo Il mer 26 ago 2020, 13:04 ha scritto: > From: Marc-André Lureau > > Hi, > > The following patches fix installation path when cross-compiling Windows > version, and move the NSIS build rule to meson. > >

[PATCH v4 4/5] configure: Fix include and linkage issue on msys2

2020-08-26 Thread luoyonggang
From: Yonggang Luo On msys2, the -I/e/path/to/qemu -L/e/path/to/qemu are not recognized by the compiler Cause $PWD are result posix style path such as /e/path/to/qemu that can not be recognized by mingw gcc, and `pwd -W` are result Windows style path such as E:/path/to/qemu that can be

[PATCH v4 5/5] meson: Fixes ninjatool can not be recognized as script under Window/MSYS2

2020-08-26 Thread luoyonggang
From: Yonggang Luo use ninja instead ${build_path}/ninjatool Signed-off-by: Yonggang Luo --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 3b9e79923d..2ad0c58492 100755 --- a/configure +++ b/configure @@ -8222,7 +8222,7 @@ fi mv

Re: linux-user static build broken

2020-08-26 Thread Paolo Bonzini
Let me take a look tonight as I plan to prepare a pull request. Paolo Il mer 26 ago 2020, 12:53 Laurent Vivier ha scritto: > Le 26/08/2020 à 11:08, Paolo Bonzini a écrit : > > $ORIGIN is a special literal used by ld.so. It's probably fixed by the > > same patch that was posted for msys. > > If

[PATCH v4 3/5] meson: Mingw64 gcc doesn't recognize system include_type for sdl2

2020-08-26 Thread luoyonggang
From: Yonggang Luo Fixes this for msys2/mingw64 by remove the include_type for sdl2 discovery in meson Signed-off-by: Yonggang Luo --- meson.build | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/meson.build b/meson.build index f0fe5f8799..1644bbd83c 100644 ---

[PATCH v4 1/5] meson: Fixes the ninjatool issue that E$$: are generated in Makefile.ninja

2020-08-26 Thread luoyonggang
From: Yonggang Luo SIMPLE_PATH_RE should match the full path token. Or the $ and : contained in path would not matched if the path are start with C:/ and E:/ Signed-off-by: Yonggang Luo --- scripts/ninjatool.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v4 2/5] meson: fixes relpath may fail on win32.

2020-08-26 Thread luoyonggang
From: Yonggang Luo On win32, os.path.relpath would raise exception when do the following relpath: C:/msys64/mingw64/x.exe relative to E:/path/qemu-build would fail. So we try catch it for stopping it from raise exception on msys2 Signed-off-by: Yonggang Luo --- scripts/mtest2make.py | 11

[PATCH] meson: install pc-bios blobs

2020-08-26 Thread marcandre . lureau
From: Marc-André Lureau Signed-off-by: Marc-André Lureau --- Makefile| 31 -- pc-bios/meson.build | 63 + 2 files changed, 63 insertions(+), 31 deletions(-) diff --git a/Makefile b/Makefile index

[[PATCH v3] 3/5] meson: Mingw64 gcc doesn't recognize system include_type for sdl2

2020-08-26 Thread luoyonggang
From: Yonggang Luo Fixes this for msys2/mingw64 by remove the include_type for sdl2 discovery in meson Signed-off-by: Yonggang Luo --- meson.build | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/meson.build b/meson.build index f0fe5f8799..1644bbd83c 100644 ---

<    1   2   3   4   5   >