Re: [PATCH 1/2] configure: avx2 and avx512f detection for clang

2020-07-23 Thread Thomas Huth
On 23/07/2020 08.04, Shu-Chun Weng wrote: > Do we have the flexibility to do that for util/bufferiszero.c as well? > Otherwise, we are using different mechanisms to detect (compile test.c > with -mavx2) and actually use (GCC pragma & __attribute__((target(*) > the feature in production.

Re: [PATCH 1/2] configure: avx2 and avx512f detection for clang

2020-07-23 Thread Shu-Chun Weng
Do we have the flexibility to do that for util/bufferiszero.c as well? Otherwise, we are using different mechanisms to detect (compile test.c with -mavx2) and actually use (GCC pragma & __attribute__((target(*) the feature in production. Shu-Chun On Wed, Jul 22, 2020 at 9:55 PM Thomas Huth

Re: [PATCH v4 3/8] s390/sclp: rework sclp boundary and length checks

2020-07-23 Thread Cornelia Huck
On Tue, 21 Jul 2020 14:40:14 -0400 Collin Walling wrote: > On 7/21/20 4:41 AM, David Hildenbrand wrote: > > The options I would support are > > > > 1. "sccb_boundary_is_valid" which returns "true" if valid > > 2. "sccb_boundary_is_invalid" which returns "true" if invalid > > 3.

Re: [PATCH 0/2] virtio: non-legacy device handling

2020-07-23 Thread Cornelia Huck
On Mon, 20 Jul 2020 11:07:51 +0200 David Hildenbrand wrote: > On 20.07.20 11:03, Michael S. Tsirkin wrote: > > On Mon, Jul 20, 2020 at 10:09:57AM +0200, David Hildenbrand wrote: > >> On 07.07.20 12:54, Cornelia Huck wrote: > >>> As discussed in "virtio-fs: force virtio 1.x usage", it seems

Re: [PATCH-for-5.1] hw/nvram/fw_cfg: Let fw_cfg_add_from_generator() return boolean value

2020-07-23 Thread Markus Armbruster
Cc: Vladimir Laszlo Ersek writes: > On 07/21/20 10:33, Markus Armbruster wrote: >> Laszlo Ersek writes: >> >>> On 07/20/20 14:35, Philippe Mathieu-Daudé wrote: Commits b6d7e9b66f..a43770df5d simplified the error propagation. Similarly to commit 6fd5bef10b "qom: Make functions taking

Re: [PATCH v2 15/20] iotests: 219: prepare for backup over block-copy

2020-07-23 Thread Max Reitz
On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: > The further change of moving backup to be a on block-copy call will -on? > make copying chunk-size and cluster-size a separate things. So, even s/a/two/ > with 64k cluster sized qcow2 image, default chunk would be 1M. > Test 219 depends

Re: [PATCH v2 16/20] iotests: 257: prepare for backup over block-copy

2020-07-23 Thread Max Reitz
On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: > Iotest 257 dumps a lot of in-progress information of backup job, such > as offset and bitmap dirtiness. Further commit will move backup to be > one block-copy call, which will introduce async parallel requests > instead of plain

Re: What is TYPE_TPM_TIS_ISA? (Not an ISA Device)

2020-07-23 Thread Markus Armbruster
Stefan Berger writes: > On 7/22/20 1:55 AM, Markus Armbruster wrote: >> pm socket --tpmstate dir=tpm --ctrl type=unixio,path=tpm/swtpm-soc >> running in another terminal. >> 3/ no machine plug it using isa_register_ioport() (it is not registered to the ISA memory space) >>> There's

Re: [PATCH for-5.1] sd/milkymist-memcard: Fix format string

2020-07-23 Thread Markus Armbruster
Stefan Weil writes: Let's add Fixes: b98e8d1230ff7023bb34ddeb7194424dfcbaf789 > Signed-off-by: Stefan Weil > --- > hw/sd/milkymist-memcard.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/sd/milkymist-memcard.c b/hw/sd/milkymist-memcard.c > index

Re: Replacing existing kernel with new on qemuriscv64

2020-07-23 Thread Pankaj Vinadrao Joshi
Hi, yes am building it natively for riscv (sifive's u540 hardware ) and in /boot i am able to find System.map-5.7.2 and vmlinux-5.7.2 root@exaleapsemi:~/linux-stable# make install make: Warning: File '.vmlinux.cmd' has modification time 1221719 s in the future make[1]: Warning: File

Re: 5.1.0-rc1 regression: reset fails with kvm and -cpu host

2020-07-23 Thread Philippe Mathieu-Daudé
+Vitaly On 7/23/20 10:40 AM, Dr. David Alan Gilbert wrote: > * Eduardo Habkost (ehabk...@redhat.com) wrote: >> On Wed, Jul 22, 2020 at 04:47:32PM -0400, Eduardo Habkost wrote: >>> On Wed, Jul 22, 2020 at 08:05:01PM +0200, Jan Kiszka wrote: On 22.07.20 19:35, Eduardo Habkost wrote: > Hi

Re: [PULL 0/2] fw_cfg patches for 2020-07-21

2020-07-23 Thread Peter Maydell
On Tue, 21 Jul 2020 at 18:54, Philippe Mathieu-Daudé wrote: > > The following changes since commit 90218a9a393c7925f330e7dcc08658e2a01d3bd4: > > Merge remote-tracking branch 'remotes/huth-gitlab/tags/pull-request-2020-07= > -21' into staging (2020-07-21 10:24:38 +0100) > > are available in the

Re: [PATCH v2 12/20] iotests: 56: prepare for backup over block-copy

2020-07-23 Thread Max Reitz
On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: > After introducing parallel async copy requests instead of plain > cluster-by-cluster copying loop, we'll have to wait for paused status, > as we need to wait for several parallel request. So, let's gently wait > instead of just asserting

Re: [PATCH for-5.1] sd/milkymist-memcard: Fix format string

2020-07-23 Thread Philippe Mathieu-Daudé
On 7/22/20 10:40 PM, Stefan Weil wrote: Fixes: b98e8d1230 ("sd/milkymist-memcard: Plug minor memory leak in realize") Reviewed-by: Philippe Mathieu-Daudé > Signed-off-by: Stefan Weil > --- > hw/sd/milkymist-memcard.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH for-5.1] sd/milkymist-memcard: Fix format string

2020-07-23 Thread Li Qiang
Stefan Weil 于2020年7月23日周四 上午4:41写道: > > Signed-off-by: Stefan Weil Reviewed-by: Li Qiang > --- > hw/sd/milkymist-memcard.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/sd/milkymist-memcard.c b/hw/sd/milkymist-memcard.c > index afdb8aa0c0..11f61294fc 100644 >

Re: [PATCH v2] hw/misc/edu: support pci device state migration

2020-07-23 Thread Peter Maydell
On Thu, 23 Jul 2020 at 10:01, Zeng Guang wrote: > > Currently edu device doesn't support live migration. Part of PCI > configuration information would be lost after migration. > > PCI device state in source VM: > Bus 0, device 3, function 0: > Class 0255: PCI device 1234:11e8 >

Re: [RFC v2 19/76] target/riscv: rvv-0.9: add narrower_nanbox_fpr helper

2020-07-23 Thread Frank Chang
On Thu, Jul 23, 2020 at 3:15 AM Richard Henderson < richard.hender...@linaro.org> wrote: > On 7/22/20 2:15 AM, frank.ch...@sifive.com wrote: > > From: Frank Chang > > > > For floating-point operations, the scalar can be taken from a scalar > > f register. If FLEN > SEW, the value in the f

[PATCH] linux-user: Use getcwd syscall directly

2020-07-23 Thread Andreas Schwab
The glibc getcwd function returns different errors than the getcwd syscall, which triggers an assertion failure in the glibc getcwd function when running under the emulation. Signed-off-by: Andreas Schwab --- linux-user/syscall.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-)

Re: please try to avoid sending pullreqs late on release-candidate day

2020-07-23 Thread Philippe Mathieu-Daudé
On 7/23/20 8:28 AM, Markus Armbruster wrote: > Alex Bennée writes: > >> Kevin Wolf writes: >> >>> Am 21.07.2020 um 17:56 hat Peter Maydell geschrieben: It is not helpful if everybody sends their pullrequests late on the Tuesday afternoon, as there just isn't enough time in the

Re: [PATCH-for-5.2] qapi/error: Make error_vprepend() static

2020-07-23 Thread Li Qiang
Philippe Mathieu-Daudé 于2020年7月23日周四 下午6:15写道: > > error_vprepend() is only used by util/error.c where it is > defined. Make it static to reduce its scope. > > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Li Qiang > --- > include/qapi/error.h | 6 -- > util/error.c | 6

[PATCH] hw/pci-host: save/restore pci host config register

2020-07-23 Thread Wang King
From: Hogan Wang The pci host config register is used to save PCI address for read/write config data. If guest write a value to config register, and then pause the vcpu to migrate, After the migration, the guest continue to write pci config data, and the write data will be ignored because of new

Re: [PATCH-for-5.1 v2 2/2] tpm: List the available TPM backends

2020-07-23 Thread Philippe Mathieu-Daudé
On 7/23/20 12:39 PM, Philippe Mathieu-Daudé wrote: > When an incorrect backend is selected, tpm_display_backend_drivers() > is supposed to list the available backends. However the error is > directly propagated, and we never display the list. The user only > gets "Parameter 'type' expects a TPM

[PATCH] qapi: enable use of g_autoptr with QAPI types

2020-07-23 Thread Daniel P . Berrangé
Currently QAPI generates a type and function for free'ing it: typedef struct QCryptoBlockCreateOptions QCryptoBlockCreateOptions; void qapi_free_QCryptoBlockCreateOptions(QCryptoBlockCreateOptions *obj); This is used in the traditional manner: QCryptoBlockCreateOptions *opts = NULL;

Re: [virtio-comment] [RFC] ivshmem v2: Shared memory device specification

2020-07-23 Thread Stefan Hajnoczi
On Fri, Jul 17, 2020 at 06:15:58PM +0200, Jan Kiszka wrote: > On 15.07.20 15:27, Stefan Hajnoczi wrote: > > On Mon, May 25, 2020 at 09:58:28AM +0200, Jan Kiszka wrote: Thanks for the responses. It would be great to update the spec with these clarifications. > > > If BAR 2 is not present, the

Re: [PATCH] virtio: Drop broken and superfluous object_property_set_link()

2020-07-23 Thread Cornelia Huck
On Tue, 21 Jul 2020 14:11:53 +0200 Markus Armbruster wrote: > virtio_crypto_pci_realize() and copies the value of vcrypto->vdev's > property "cryptodev" to vcrypto's property: > > object_property_set_link(OBJECT(vrng), "rng", OBJECT(vrng->vdev.conf.rng), >

Re: [PATCH v0 0/4] background snapshot

2020-07-23 Thread Denis Plotnikov
On 22.07.2020 19:30, Peter Xu wrote: On Wed, Jul 22, 2020 at 06:47:44PM +0300, Denis Plotnikov wrote: On 22.07.2020 18:42, Denis Plotnikov wrote: On 22.07.2020 17:50, Peter Xu wrote: Hi, Denis, Hi, Peter ... How to use: 1. enable background snapshot capability     virsh

Re: 5.1.0-rc1 regression: reset fails with kvm and -cpu host

2020-07-23 Thread Dr. David Alan Gilbert
* Eduardo Habkost (ehabk...@redhat.com) wrote: > On Wed, Jul 22, 2020 at 04:47:32PM -0400, Eduardo Habkost wrote: > > On Wed, Jul 22, 2020 at 08:05:01PM +0200, Jan Kiszka wrote: > > > On 22.07.20 19:35, Eduardo Habkost wrote: > > > > Hi Jan, > > > > > > > > What was the last version where it

[PATCH v2] hw/misc/edu: support pci device state migration

2020-07-23 Thread Zeng Guang
Currently edu device doesn't support live migration. Part of PCI configuration information would be lost after migration. PCI device state in source VM: Bus 0, device 3, function 0: Class 0255: PCI device 1234:11e8 PCI subsystem 1af4:1100 IRQ 11, pin A BAR0: 32 bit

Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems

2020-07-23 Thread Daniel P . Berrangé
On Wed, Jul 22, 2020 at 12:02:59PM -0700, Richard Henderson wrote: > On 7/22/20 9:44 AM, Daniel P. Berrangé wrote: > > OpenStack uses TCG in alot of their CI infrastructure for example > > and runs multiple VMs. If there's 4 VMs, that's another 4 GB of > > RAM usage just silently added on top of

Re: [PATCH 3/4] error: Remove NULL checks on error_propagate() calls (again)

2020-07-23 Thread Markus Armbruster
Eric Blake writes: > On 7/22/20 3:40 AM, Markus Armbruster wrote: >> Patch created mechanically by rerunning: >> >> $ spatch --sp-file scripts/coccinelle/error_propagate_null.cocci \ >> --macro-file scripts/cocci-macro-file.h \ >> --use-gitgrep . >> >> Cc: Jens

Re: [RFC PATCH-for-5.1 2/2] tpm: List the available TPM backends

2020-07-23 Thread Philippe Mathieu-Daudé
On 7/22/20 11:44 PM, Stefan Berger wrote: > On 7/22/20 7:23 AM, Philippe Mathieu-Daudé wrote: >> When an incorrect backend is selected, tpm_display_backend_drivers() >> is supposed to list the available backends. However the error is >> directly propagated, and we never display the list. The user

Re: [PATCH 2/2] e1000e: make TX reentrant

2020-07-23 Thread Peter Maydell
On Wed, 22 Jul 2020 at 10:00, Jason Wang wrote: > > In loopback mode, e1000e RX can DMA into TX doorbell which requires > TX to be reentrant. This patch make e1000e's TX routine reentrant by > introducing a per device boolean for recording whether or not a TX > rountine is being called and return

Re: [PATCH 1/2] ppc: Rename current DAWR macros

2020-07-23 Thread Cornelia Huck
On Thu, 23 Jul 2020 16:12:19 +0530 Ravi Bangoria wrote: > Power10 is introducing second DAWR. Use real register names (with > suffix 0) from ISA for current macros. > > Signed-off-by: Ravi Bangoria > --- > include/hw/ppc/spapr.h | 2 +- > linux-headers/asm-powerpc/kvm.h | 4 ++-- >

Re: please try to avoid sending pullreqs late on release-candidate day

2020-07-23 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > On 7/23/20 8:28 AM, Markus Armbruster wrote: >> Alex Bennée writes: >> >>> Kevin Wolf writes: >>> Am 21.07.2020 um 17:56 hat Peter Maydell geschrieben: > It is not helpful if everybody sends their pullrequests late > on the Tuesday afternoon, as

Re: [PATCH v2 13/20] iotests: 129: prepare for backup over block-copy

2020-07-23 Thread Max Reitz
On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: > After introducing parallel async copy requests instead of plain > cluster-by-cluster copying loop, backup job may finish earlier than > final assertion in do_test_stop. Let's require slow backup explicitly > by specifying speed parameter.

Re: [PATCH v2 14/20] iotests: 185: prepare for backup over block-copy

2020-07-23 Thread Max Reitz
On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: > The further change of moving backup to be a on block-copy call will -on? > make copying chunk-size and cluster-size a separate things. So, even s/a/two/ > with 64k cluster sized qcow2 image, default chunk would be 1M. > 185 test however

Re: [PATCH] spice: simplify chardev setup

2020-07-23 Thread Christophe de Dinechin
On 2020-07-22 at 13:18 CEST, Gerd Hoffmann wrote... > On Wed, Jul 22, 2020 at 12:19:43PM +0200, Christophe de Dinechin wrote: >> >> On 2020-07-22 at 11:20 CEST, Christophe de Dinechin wrote... >> > On 2020-07-22 at 10:49 CEST, Gerd Hoffmann wrote... >> >> Initialize spice before chardevs. That

Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems

2020-07-23 Thread Alex Bennée
Daniel P. Berrangé writes: > On Wed, Jul 22, 2020 at 12:02:59PM -0700, Richard Henderson wrote: >> On 7/22/20 9:44 AM, Daniel P. Berrangé wrote: >> > OpenStack uses TCG in alot of their CI infrastructure for example >> > and runs multiple VMs. If there's 4 VMs, that's another 4 GB of >> > RAM

[PATCH-for-5.2] qapi/error: Make error_vprepend() static

2020-07-23 Thread Philippe Mathieu-Daudé
error_vprepend() is only used by util/error.c where it is defined. Make it static to reduce its scope. Signed-off-by: Philippe Mathieu-Daudé --- include/qapi/error.h | 6 -- util/error.c | 6 +- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/include/qapi/error.h

Re: [PATCH for-5.1] Fix grammar in documentation

2020-07-23 Thread Peter Maydell
On Wed, 22 Jul 2020 at 22:06, Stefan Weil wrote: > > Signed-off-by: Stefan Weil > --- > docs/system/build-platforms.rst | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/docs/system/build-platforms.rst b/docs/system/build-platforms.rst > index c2b92a9698..9734eba2f1

Re: [PATCH 0/2] virtio: non-legacy device handling

2020-07-23 Thread Cornelia Huck
On Mon, 20 Jul 2020 11:54:06 +0200 Halil Pasic wrote: > On Tue, 7 Jul 2020 12:54:44 +0200 > Cornelia Huck wrote: > > > As discussed in "virtio-fs: force virtio 1.x usage", it seems like > > a good idea to make sure that any new virtio device (which does not > > support legacy virtio) is

Re: [virtio-comment] [RFC] ivshmem v2: Shared memory device specification

2020-07-23 Thread Jan Kiszka
On 23.07.20 08:54, Stefan Hajnoczi wrote: On Fri, Jul 17, 2020 at 06:15:58PM +0200, Jan Kiszka wrote: On 15.07.20 15:27, Stefan Hajnoczi wrote: On Mon, May 25, 2020 at 09:58:28AM +0200, Jan Kiszka wrote: Thanks for the responses. It would be great to update the spec with these

roms/seabios-hppa can't be built with gcc-10: -fno-ipa-sra

2020-07-23 Thread Michael Tokarev
Switching to gcc-10 makes seabios-hppa unbuildable. It fails at the final linking step with a lot of missing references to memcpy & memcmp all over the places. The notable difference between gcc-10 and previous gcc is that ccode32flat.o does _not_ have the text for these two functions but have

Re: [PATCH for-5.1] nbd: Fix large trim/zero requests

2020-07-23 Thread Vladimir Sementsov-Ogievskiy
23.07.2020 00:22, Eric Blake wrote: Although qemu as NBD client limits requests to <2G, the NBD protocol allows clients to send requests almost all the way up to 4G. But because our block layer is not yet 64-bit clean, we accidentally wrap such requests into a negative size, and fail with EIO

Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems

2020-07-23 Thread Daniel P . Berrangé
On Thu, Jul 23, 2020 at 10:22:25AM +0100, Alex Bennée wrote: > > Daniel P. Berrangé writes: > > > On Wed, Jul 22, 2020 at 12:02:59PM -0700, Richard Henderson wrote: > >> On 7/22/20 9:44 AM, Daniel P. Berrangé wrote: > >> > OpenStack uses TCG in alot of their CI infrastructure for example > >> >

Re: [PATCH-for-5.1?] qapi/error: Check format string argument in error_propagate_prepend()

2020-07-23 Thread Stefan Weil
Am 23.07.20 um 11:13 schrieb Philippe Mathieu-Daudé: > error_propagate_prepend() "behaves like error_prepend()", and > error_prepend() uses "formatting @fmt, ... like printf()". > error_prepend() checks its format string argument, but > error_propagate_prepend() does not. Fix that. > > This would

Re: [PATCH-for-5.1 1/2] tpm: Display when no backend is available

2020-07-23 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > Display "No TPM backend available in this binary." error when > no backend is available. > > Signed-off-by: Philippe Mathieu-Daudé > --- > tpm.c | 21 + > 1 file changed, 21 insertions(+) > > diff --git a/tpm.c b/tpm.c > index

Re: [PATCH v2 11/20] qapi: backup: add x-max-chunk and x-max-workers parameters

2020-07-23 Thread Max Reitz
On 22.07.20 14:22, Max Reitz wrote: > On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: >> Add new parameters to configure future backup features. The patch >> doesn't introduce aio backup requests (so we actually have only one >> worker) neither requests larger than one cluster. Still,

[PATCH-for-5.1?] qapi/error: Check format string argument in error_propagate_prepend()

2020-07-23 Thread Philippe Mathieu-Daudé
error_propagate_prepend() "behaves like error_prepend()", and error_prepend() uses "formatting @fmt, ... like printf()". error_prepend() checks its format string argument, but error_propagate_prepend() does not. Fix that. This would have catched the invalid format introduced in commit

Re: [PATCH v2 06/12] accel/tcg: better handle memory constrained systems

2020-07-23 Thread Alex Bennée
Daniel P. Berrangé writes: > On Thu, Jul 23, 2020 at 10:22:25AM +0100, Alex Bennée wrote: >> >> Daniel P. Berrangé writes: >> >> > On Wed, Jul 22, 2020 at 12:02:59PM -0700, Richard Henderson wrote: >> >> On 7/22/20 9:44 AM, Daniel P. Berrangé wrote: >> >> > OpenStack uses TCG in alot of

Re: [PATCH-for-5.1] hw/nvram/fw_cfg: Let fw_cfg_add_from_generator() return boolean value

2020-07-23 Thread Laszlo Ersek
+Igor, and question below On 07/23/20 09:37, Markus Armbruster wrote: > You must use ERRP_GUARD() in functions that dereference their @errp > parameter (so that works even when the argument is null) or pass it to > error_prepend() or error_append_hint() (so they get reached even when > the

[Bug 1886793] Re: "go install" command fails while running inside s390x docker container on x86_64 host using qemu

2020-07-23 Thread Nirman Narang
I ran the following commands: #apt install debootstrap #debootstrap_dir=debootstrap #debootstrap --arch=s390x --foreign sid "$debootstrap_dir" #sudo mkdir -p "${debootstrap_dir}/usr/bin" #sudo cp "$(which qemu-s390x-static)" "${debootstrap_dir}/usr/bin" #sudo cp "$(which qemu-s390x)"

Re: [PATCH 0/2] virtio: non-legacy device handling

2020-07-23 Thread Cornelia Huck
On Thu, 23 Jul 2020 13:57:08 +0200 David Hildenbrand wrote: > On 23.07.20 08:33, Cornelia Huck wrote: > > On Mon, 20 Jul 2020 11:07:51 +0200 > > David Hildenbrand wrote: > > > >> On 20.07.20 11:03, Michael S. Tsirkin wrote: > >>> On Mon, Jul 20, 2020 at 10:09:57AM +0200, David Hildenbrand

Re: 5.1.0-rc1 regression: reset fails with kvm and -cpu host

2020-07-23 Thread Vitaly Kuznetsov
Philippe Mathieu-Daudé writes: > +Vitaly > > On 7/23/20 10:40 AM, Dr. David Alan Gilbert wrote: >> * Eduardo Habkost (ehabk...@redhat.com) wrote: >>> On Wed, Jul 22, 2020 at 04:47:32PM -0400, Eduardo Habkost wrote: On Wed, Jul 22, 2020 at 08:05:01PM +0200, Jan Kiszka wrote: > On

[PATCH 4/4] gitlab-ci.yml: Add build-system-debian and build-system-centos jobs

2020-07-23 Thread Thomas Huth
We were missing the two new targets avr-softmmu and rx-softmmu in the gitlab-CI so far, and did not add some of the "other endianess" targets like sh4eb-softmmu yet. Since the current build-system-* jobs run already for a very long time, let's do not add these missing targets there, but introduce

[PATCH 3/4] tests/acceptance: Disable the rx sash and arm cubieboard replay test on Gitlab

2020-07-23 Thread Thomas Huth
These tests always time out on Gitlab, not sure what's happening here. Let's disable them until somebody has enough spare time to debug the issues. Signed-off-by: Thomas Huth --- tests/acceptance/machine_rx_gdbsim.py | 4 tests/acceptance/replay_kernel.py | 1 + 2 files changed, 5

Re: [Virtio-fs] [PATCH for-5.1 2/3] virtiofsd: add container-friendly -o chroot sandboxing option

2020-07-23 Thread Stefan Hajnoczi
On Wed, Jul 22, 2020 at 02:17:10PM -0400, Vivek Goyal wrote: > On Wed, Jul 22, 2020 at 02:02:05PM +0100, Stefan Hajnoczi wrote: > > virtiofsd cannot run in an unprivileged container because CAP_SYS_ADMIN > > is required to create namespaces. > > > > Introduce a weaker sandbox that is sufficient

Re: [PATCH] acpi: Fix access to PM1 control and status registers

2020-07-23 Thread Michael S. Tsirkin
On Thu, Jul 16, 2020 at 11:05:06AM +0200, Cédric Le Goater wrote: > On 7/2/20 1:12 PM, Michael S. Tsirkin wrote: > > On Wed, Jul 01, 2020 at 01:48:36PM +0100, Anthony PERARD wrote: > >> On Wed, Jul 01, 2020 at 08:01:55AM -0400, Michael S. Tsirkin wrote: > >>> On Wed, Jul 01, 2020 at 12:05:49PM

Re: [PATCH v2] hw/pci-host: save/restore pci host config register

2020-07-23 Thread Michael S. Tsirkin
On Thu, Jul 23, 2020 at 08:23:01PM +0800, Wang King wrote: > From: Hogan Wang > > The pci host config register is used to save PCI address for > read/write config data. If guest write a value to config register, > and then pause the vcpu to migrate, After the migration, the guest > continue to

Re: [PATCH for-5.1] nbd: Fix large trim/zero requests

2020-07-23 Thread Vladimir Sementsov-Ogievskiy
23.07.2020 14:47, Eric Blake wrote: On 7/23/20 2:23 AM, Vladimir Sementsov-Ogievskiy wrote: 23.07.2020 00:22, Eric Blake wrote: Although qemu as NBD client limits requests to <2G, the NBD protocol allows clients to send requests almost all the way up to 4G.  But because our block layer is not

Re: 5.1.0-rc1 regression: reset fails with kvm and -cpu host

2020-07-23 Thread Paolo Bonzini
Yes, that seems correct. Paolo Il gio 23 lug 2020, 15:26 Vitaly Kuznetsov ha scritto: > This depends on whether the guest has performed VMXON or not I believe. > > Anyways, I *think* the fix will be: > > diff --git a/target/i386/kvm.c b/target/i386/kvm.c > index 2b6b744..75c2e68 100644 > ---

Re: [PATCH 3/4] error: Remove NULL checks on error_propagate() calls (again)

2020-07-23 Thread Eric Blake
On 7/23/20 8:38 AM, Markus Armbruster wrote: +++ b/migration/colo.c @@ -798,9 +798,7 @@ static void colo_incoming_process_checkpoint(MigrationIncomingState *mis, colo_send_message(mis->to_src_file, COLO_MESSAGE_VMSTATE_LOADED, _err); -if (local_err) { -

[PATCH 2/2] ppc: Enable 2nd DAWR support on p10

2020-07-23 Thread Ravi Bangoria
As per the PAPR, bit 0 of byte 64 in pa-features property indicates availability of 2nd DAWR registers. i.e. If this bit is set, 2nd DAWR is present, otherwise not. Use KVM_CAP_PPC_DAWR1 capability to find whether kvm supports 2nd DAWR or nor. If it's supported, set the pa-feature bit in guest DT

[Bug 1888601] Re: QEMU v5.1.0-rc0/rc1 hang with nested virtualization

2020-07-23 Thread Simon Kaegi
** Description changed: We're running Kata Containers using QEMU and with v5.1.0rc0 and rc1 have noticed a problem at startup where QEMu appears to hang. We are not seeing this problem on our bare metal nodes and only on a VSI that supports nested virtualization. We unfortunately see

Re: please try to avoid sending pullreqs late on release-candidate day

2020-07-23 Thread Markus Armbruster
Alex Bennée writes: > Kevin Wolf writes: > >> Am 21.07.2020 um 17:56 hat Peter Maydell geschrieben: >>> It is not helpful if everybody sends their pullrequests late >>> on the Tuesday afternoon, as there just isn't enough time in the >>> day to merge test and apply them all before I have to cut

Re: [PATCH] Fix vhost-user buffer over-read on ram hot-unplug

2020-07-23 Thread Marc-André Lureau
On Fri, Jul 17, 2020 at 8:21 AM Raphael Norwitz wrote: > > The VHOST_USER_PROTOCOL_F_CONFIGURE_MEM_SLOTS vhost-user protocol > feature introduced a shadow-table, used by the backend to dynamically > determine how a vdev's memory regions have changed since the last > vhost_user_set_mem_table()

[PATCH] trace/simple: Allow enabling simple traces from command line

2020-07-23 Thread Josh DuBois
The simple trace backend is enabled / disabled with a call to st_set_trace_file_enabled(). When initializing tracing from the command-line, this must be enabled on startup. (Prior to db25d56c014aa1a9, command-line initialization of simple trace worked because every call to st_set_trace_file

Re: [PATCH v2 17/20] backup: move to block-copy

2020-07-23 Thread Max Reitz
On 01.06.20 20:11, Vladimir Sementsov-Ogievskiy wrote: > This brings async request handling and block-status driven chunk sizes > to backup out of the box, which improves backup performance. > > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > include/block/block-copy.h | 9 +-- >

Re: [PATCH-for-5.1?] qapi/error: Check format string argument in error_propagate_prepend()

2020-07-23 Thread Philippe Mathieu-Daudé
On 7/23/20 11:44 AM, Stefan Weil wrote: > Am 23.07.20 um 11:13 schrieb Philippe Mathieu-Daudé: > >> error_propagate_prepend() "behaves like error_prepend()", and >> error_prepend() uses "formatting @fmt, ... like printf()". >> error_prepend() checks its format string argument, but >>

[PATCH-for-5.1 v2 0/2] tpm: Improve error reporting

2020-07-23 Thread Philippe Mathieu-Daudé
Improve error reporting by listing TPM backends. Philippe Mathieu-Daudé (2): tpm: Display when no backend is available tpm: List the available TPM backends tpm.c | 40 ++-- 1 file changed, 30 insertions(+), 10 deletions(-) -- 2.21.3

[PATCH-for-5.1 v2 1/2] tpm: Display when no backend is available

2020-07-23 Thread Philippe Mathieu-Daudé
Display "No TPM backend available in this binary." error when no backend is available. Reviewed-by: Stefan Berger Signed-off-by: Philippe Mathieu-Daudé --- tpm.c | 21 + 1 file changed, 21 insertions(+) diff --git a/tpm.c b/tpm.c index fe03b24858..e36803a64d 100644 ---

[PATCH-for-5.1 v2 2/2] tpm: List the available TPM backends

2020-07-23 Thread Philippe Mathieu-Daudé
When an incorrect backend is selected, tpm_display_backend_drivers() is supposed to list the available backends. However the error is directly propagated, and we never display the list. The user only gets "Parameter 'type' expects a TPM backend type" error. Convert the fprintf(stderr,) calls to

Re: [RFC PATCH-for-5.1 2/2] tpm: List the available TPM backends

2020-07-23 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > When an incorrect backend is selected, tpm_display_backend_drivers() > is supposed to list the available backends. However the error is > directly propagated, and we never display the list. The user only > gets "Parameter 'type' expects a TPM backend type" error.

Re: [PATCH 2/2] ppc: Enable 2nd DAWR support on p10

2020-07-23 Thread Cornelia Huck
On Thu, 23 Jul 2020 16:12:20 +0530 Ravi Bangoria wrote: > As per the PAPR, bit 0 of byte 64 in pa-features property indicates > availability of 2nd DAWR registers. i.e. If this bit is set, 2nd > DAWR is present, otherwise not. Use KVM_CAP_PPC_DAWR1 capability to > find whether kvm supports 2nd

Re: [PATCH] hw/pci-host: save/restore pci host config register

2020-07-23 Thread Laszlo Ersek
On 07/23/20 12:49, Wang King wrote: > From: Hogan Wang > > The pci host config register is used to save PCI address for > read/write config data. If guest write a value to config register, > and then pause the vcpu to migrate, After the migration, the guest > continue to write pci config data,

Re: [PATCH 2/2] e1000e: make TX reentrant

2020-07-23 Thread Stefan Hajnoczi
On Thu, Jul 23, 2020 at 10:25:35AM +0800, Jason Wang wrote: > > On 2020/7/22 下午8:53, Michael Tokarev wrote: > > FWIW, this is not "making TX reentrant", it is about forbidding > > reentrancy instead :) > > > > /mjt > > > Indeed, I will rename the title. Please also include a comment

Re: [PATCH for-5.1 2/3] virtiofsd: add container-friendly -o chroot sandboxing option

2020-07-23 Thread Stefan Hajnoczi
On Wed, Jul 22, 2020 at 05:58:11PM +0100, Daniel P. Berrangé wrote: > On Wed, Jul 22, 2020 at 02:02:05PM +0100, Stefan Hajnoczi wrote: > > virtiofsd cannot run in an unprivileged container because CAP_SYS_ADMIN > > is required to create namespaces. > > > > Introduce a weaker sandbox that is

Re: [PATCH for-5.1 2/3] virtiofsd: add container-friendly -o chroot sandboxing option

2020-07-23 Thread Stefan Hajnoczi
On Wed, Jul 22, 2020 at 08:03:18PM +0100, Dr. David Alan Gilbert wrote: > * Stefan Hajnoczi (stefa...@redhat.com) wrote: > > +/* > > + * Make the shared directory the file system root so that FUSE_OPEN > > + * (lo_open()) cannot escape the shared directory by opening a symlink. > > +

Re: [PATCH for-5.1 2/3] virtiofsd: add container-friendly -o chroot sandboxing option

2020-07-23 Thread Stefan Hajnoczi
On Wed, Jul 22, 2020 at 06:58:20PM +0100, Dr. David Alan Gilbert wrote: > * Stefan Hajnoczi (stefa...@redhat.com) wrote: > > virtiofsd cannot run in an unprivileged container because CAP_SYS_ADMIN > > is required to create namespaces. > > > > Introduce a weaker sandbox that is sufficient in

Re: [PATCH] acpi: Fix access to PM1 control and status registers

2020-07-23 Thread Michael S. Tsirkin
On Fri, Jul 10, 2020 at 10:42:58AM +0100, Anthony PERARD wrote: > On Thu, Jul 02, 2020 at 07:12:08AM -0400, Michael S. Tsirkin wrote: > > memory: align to min access size > > > > If impl.min_access_size > valid.min_access_size access callbacks > > can get a misaligned access as size is increased.

[PATCH v1] hw/pci-host: save/restore pci host config register

2020-07-23 Thread Wang King
From: Hogan Wang The pci host config register is used to save PCI address for read/write config data. If guest write a value to config register, and then pause the vcpu to migrate, After the migration, the guest continue to write pci config data, and the write data will be ignored because of new

Re: [PATCH 3/4] tests/acceptance: Disable the rx sash and arm cubieboard replay test on Gitlab

2020-07-23 Thread Philippe Mathieu-Daudé
On 7/23/20 2:27 PM, Thomas Huth wrote: > These tests always time out on Gitlab, not sure what's happening here. > Let's disable them until somebody has enough spare time to debug the > issues. Is the Avocado cache working? Is it failing with an empty cache? If so, maybe we need to run avocado

Re: [PATCH] hw/pci-host: save/restore pci host config register

2020-07-23 Thread Michael S. Tsirkin
On Thu, Jul 23, 2020 at 01:48:47PM +0200, Laszlo Ersek wrote: > On 07/23/20 12:49, Wang King wrote: > > From: Hogan Wang > > > > The pci host config register is used to save PCI address for > > read/write config data. If guest write a value to config register, > > and then pause the vcpu to

Re: [PATCH] qapi: enable use of g_autoptr with QAPI types

2020-07-23 Thread Daniel P . Berrangé
On Thu, Jul 23, 2020 at 02:50:51PM +0200, Markus Armbruster wrote: > Daniel P. Berrangé writes: > > > On Thu, Jul 23, 2020 at 06:49:44AM -0500, Eric Blake wrote: > >> On 7/23/20 6:12 AM, Daniel P. Berrangé wrote: > >> > Currently QAPI generates a type and function for free'ing it: > >> > > >> >

Re: [PATCH 3/4] tests/acceptance: Disable the rx sash and arm cubieboard replay test on Gitlab

2020-07-23 Thread Thomas Huth
On 23/07/2020 14.53, Philippe Mathieu-Daudé wrote: > On 7/23/20 2:27 PM, Thomas Huth wrote: >> These tests always time out on Gitlab, not sure what's happening here. >> Let's disable them until somebody has enough spare time to debug the >> issues. > > Is the Avocado cache working? Is it failing

Re: [PATCH v2 0/2] linux-user: fix clock_nanosleep()

2020-07-23 Thread Alex Bennée
Laurent Vivier writes: > Update the "remain" time only if errno is EINTR and flags is TIMER_ABSTIME. > > The v2 restores the get_errno() as our safe_clock_nanosleep() uses > errno to return the error value (and not ret). > > As we use errno, we don't need the special case for ppc here, the CRF

[PATCH] target/unicore32: Remove CURSES stuff from the Makefile.objs

2020-07-23 Thread Thomas Huth
The dependency on curses has been removed in commit c7a856b42e403e2b ("target/unicore32: Prefer qemu_semihosting_log_out() over curses"). So we can remove the related lines in the Makefile now, too. Signed-off-by: Thomas Huth --- target/unicore32/Makefile.objs | 4 1 file changed, 4

Re: [Virtio-fs] [PATCH for-5.1 3/3] virtiofsd: probe unshare(CLONE_FS) and print an error

2020-07-23 Thread Vivek Goyal
On Thu, Jul 23, 2020 at 01:50:35PM +0100, Daniel P. Berrangé wrote: > On Thu, Jul 23, 2020 at 01:46:11PM +0100, Stefan Hajnoczi wrote: > > On Wed, Jul 22, 2020 at 06:03:28PM +0100, Daniel P. Berrangé wrote: > > > On Wed, Jul 22, 2020 at 02:02:06PM +0100, Stefan Hajnoczi wrote: > > > > An

[PATCH] configure: Allow to build tools without pixman

2020-07-23 Thread Thomas Huth
If pixman is not installed, it is currently not possible to run: .../configure --disable-system --enable-tools Seems like there was a dependency from one of the required source files to pixman in the past, but since commit 1ac0206b2ae1ffaeec56 ("qemu-timer.c: Trim list of included headers"),

[PATCH for-5.1 0/2] tpm: Fix error reporting, improve help

2020-07-23 Thread Markus Armbruster
The alternative to PATCH 1 is a proper conversion to Error, as discussed in Subject: Re: What is TYPE_TPM_TIS_ISA? (Not an ISA Device) Message-ID: <87tuxyoauy@dusky.pond.sub.org> Such a conversion would be much too invasive for 5.1. Going the other way, like PATCH 1 does, is simple

[PATCH for-5.1 2/2] tpm: Improve help on TPM types when none are available

2020-07-23 Thread Markus Armbruster
Help is a bit awkward when no TPM types are built into QEMU: $ upstream-qemu -tpmdev nonexistent,id=tpm0 upstream-qemu: -tpmdev nonexistent,id=tpm0: Parameter 'type' expects a TPM backend type Supported TPM types (choose only one): Improve to upstream-qemu: -tpmdev

[PATCH v2] hw/pci-host: save/restore pci host config register

2020-07-23 Thread Wang King
From: Hogan Wang The pci host config register is used to save PCI address for read/write config data. If guest write a value to config register, and then pause the vcpu to migrate, After the migration, the guest continue to write pci config data, and the write data will be ignored because of new

[PATCH 2/4] iotests: Select a default machine for the rx and avr targets

2020-07-23 Thread Thomas Huth
If you are building only with either the new rx-softmmu or avr-softmmu target, "make check-block" fails a couple of tests since there is no default machine defined in these new targets. We have to select a machine in the "check" script for these, just like we already do for the arm- and

[PATCH 0/4] Test more in less time in the Gitlab-CI

2020-07-23 Thread Thomas Huth
This patch series adds two new "build-system" pipelines to the Gitlab-CI, one based on Debian and one on CentOS. We then use these build pipelines to test the targets that were missing so far (e.g. the two new targets rx-softmmu and avr-softmmu), and move some of the targets from the other

[PATCH 1/4] tests/docker: Add python3-venv and netcat to the debian-amd64 container

2020-07-23 Thread Thomas Huth
Without python3-venv, I get the following message when trying to run the acceptance tests within the debian container: The virtual environment was not created successfully because ensurepip is not available. On Debian/Ubuntu systems, you need to install the python3-venv package using the

Re: [PATCH 3/4] tests/acceptance: Disable the rx sash and arm cubieboard replay test on Gitlab

2020-07-23 Thread Wainer dos Santos Moschetta
On 7/23/20 9:27 AM, Thomas Huth wrote: These tests always time out on Gitlab, not sure what's happening here. Let's disable them until somebody has enough spare time to debug the issues. It's fair to me. Signed-off-by: Thomas Huth --- tests/acceptance/machine_rx_gdbsim.py | 4

Re: [PATCH] hw/pci-host: save/restore pci host config register

2020-07-23 Thread Michael S. Tsirkin
On Thu, Jul 23, 2020 at 06:49:35PM +0800, Wang King wrote: > From: Hogan Wang > > The pci host config register is used to save PCI address for > read/write config data. If guest write a value to config register, > and then pause the vcpu to migrate, After the migration, the guest > continue to

Re: [PATCH for-5.1 3/3] virtiofsd: probe unshare(CLONE_FS) and print an error

2020-07-23 Thread Daniel P . Berrangé
On Thu, Jul 23, 2020 at 01:46:11PM +0100, Stefan Hajnoczi wrote: > On Wed, Jul 22, 2020 at 06:03:28PM +0100, Daniel P. Berrangé wrote: > > On Wed, Jul 22, 2020 at 02:02:06PM +0100, Stefan Hajnoczi wrote: > > > An assertion failure is raised during request processing if > > > unshare(CLONE_FS)

Re: 5.1.0-rc1 regression: reset fails with kvm and -cpu host

2020-07-23 Thread Dr. David Alan Gilbert
* Vitaly Kuznetsov (vkuzn...@redhat.com) wrote: > Philippe Mathieu-Daudé writes: > > > +Vitaly > > > > On 7/23/20 10:40 AM, Dr. David Alan Gilbert wrote: > >> * Eduardo Habkost (ehabk...@redhat.com) wrote: > >>> On Wed, Jul 22, 2020 at 04:47:32PM -0400, Eduardo Habkost wrote: > On Wed, Jul

Re: [PATCH] qapi: enable use of g_autoptr with QAPI types

2020-07-23 Thread Markus Armbruster
Daniel P. Berrangé writes: > On Thu, Jul 23, 2020 at 06:49:44AM -0500, Eric Blake wrote: >> On 7/23/20 6:12 AM, Daniel P. Berrangé wrote: >> > Currently QAPI generates a type and function for free'ing it: >> > >> >typedef struct QCryptoBlockCreateOptions QCryptoBlockCreateOptions; >> >

  1   2   3   4   >