[Bug 1860920] Re: qemu-s390x-softmmu: crash

2020-01-30 Thread Thomas Huth
There is definitely something wrong here ;-) According to the "Principles of Operations" ISA document, opcode 0xEC54 is the RNSBG instruction (ROTATE THEN AND SELECTED BITS). But op_rosbg() apparently currently handles 0xEC55, 0xEC56 and 0xEC57. 0xEC55 seems wrong there, since this opcode should

Re: [PATCH v5 1/8] multifd: Add multifd-method parameter

2020-01-30 Thread Juan Quintela
Markus Armbruster wrote: > Juan Quintela writes: > >> This will store the compression method to use. We start with none. >> >> Signed-off-by: Juan Quintela >> Reviewed-by: Markus Armbruster > > I don't remember giving my R-by. I gave my Acked-by for > [PATCH v2 06/10] migration: Add

Re: [PATCH v5 5/8] multifd: Add zlib compression multifd support

2020-01-30 Thread Markus Armbruster
Juan Quintela writes: > Signed-off-by: Juan Quintela For QAPI: Acked-by: Markus Armbruster

Re: [PATCH v5 4/8] multifd: Add multifd-zlib-level parameter

2020-01-30 Thread Juan Quintela
Markus Armbruster wrote: > Juan Quintela writes: > >> It will indicate which level use for compression. >> >> Signed-off-by: Juan Quintela > > This is slightly confusing (there is no zlib compression), unless you > peek at the next patch (which adds zlib compression). > > Three ways to make it

[Bug 1805913] Re: readdir() returns NULL (errno=EOVERFLOW) for 32-bit user-static qemu on 64-bit host

2020-01-30 Thread Philippe Vaucher
Sorry, meant `< 2.28` above. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1805913 Title: readdir() returns NULL (errno=EOVERFLOW) for 32-bit user-static qemu on 64-bit host Status in QEMU:

Re: [PATCH v2 29/29] MAINTAINERS: Add Acceptance tests reviewers

2020-01-30 Thread Thomas Huth
On 29/01/2020 22.23, Philippe Mathieu-Daudé wrote: > Acceptance tests can test any piece of the QEMU codebase. > As such, the directory holding them does not belong to a specific > subsystem with designated maintainers. > > Each subsystem covered by a test is welcomed to add the test path > to

Re: [PATCH] spapr: Enable DD2.3 accelerated count cache flush in pseries-5.0 machine

2020-01-30 Thread Greg Kurz
On Thu, 30 Jan 2020 12:26:22 +1100 David Gibson wrote: > For POWER9 DD2.2 cpus, the best current Spectre v2 indirect branch > mitigation is "count cache disabled", which is configured with: > -machine cap-ibs=fixed-ccd > However, this option isn't available on DD2.3 CPUs with KVM, because

Re: [PATCH] spapr: Enable DD2.3 accelerated count cache flush in pseries-5.0 machine

2020-01-30 Thread Greg Kurz
On Thu, 30 Jan 2020 09:09:18 +0100 Laurent Vivier wrote: > On 30/01/2020 02:26, David Gibson wrote: > > For POWER9 DD2.2 cpus, the best current Spectre v2 indirect branch > > mitigation is "count cache disabled", which is configured with: > > -machine cap-ibs=fixed-ccd > > However, this

Re: [PATCH 01/10] scripts: Explicit usage of Python 3

2020-01-30 Thread Juan Quintela
Philippe Mathieu-Daudé wrote: > Use the program search path to find the Python 3 interpreter. > > Patch created mechanically by running: > > $ sed -i "s,^#\!/usr/bin/\(env\ \)\?python$,#\!/usr/bin/env python3," \ >$(git grep -l 'if __name__.*__main__') > > Reported-by: Vladimir

Re: [PATCH v2 2/4] virtio-scsi: default num_queues to -smp N

2020-01-30 Thread Stefan Hajnoczi
On Thu, Jan 30, 2020 at 01:29:16AM +0100, Paolo Bonzini wrote: > On 29/01/20 16:44, Stefan Hajnoczi wrote: > > On Mon, Jan 27, 2020 at 02:10:31PM +0100, Cornelia Huck wrote: > >> On Fri, 24 Jan 2020 10:01:57 + > >> Stefan Hajnoczi wrote: > >>> @@ -47,10 +48,15 @@ static void

Re: [GSoC/Outreachy QEMU proposal] Extend support for ioctls in QEMU linux-user mode

2020-01-30 Thread Aleksandar Markovic
14:34 Čet, 23.01.2020. Aleksandar Markovic је написао/ла: > > Extend support for ioctls in QEMU linux-user mode > > > PLANNED ACTIVITIES > > BACKGROUND > > There is currently 2500+ ioctls defined in Linux kernel. QEMU linux-user currently supports only several hundred. There is a constant need

[Bug 1861394] Re: qemu-system-riscv64 hangs after poweroff linux command

2020-01-30 Thread Wojtek Lewandowski
** Description changed: QEMU Version : v4.2.0-773-g43d1455-dirty (commit 43d1455cf84283466e5c22a217db5ef4b8197b14) Command: qemu-system-riscv64 -machine virt -kernel ./bbl -nographic -initrd rootfs.cpio.gz -append "root=/dev/ram console=ttyS0" Host:LSB Version:

[kvm-unit-tests PATCH v2 4/9] arm: pmu: Check Required Event Support

2020-01-30 Thread Eric Auger
If event counters are implemented check the common events required by the PMUv3 are implemented. Some are unconditionally required (SW_INCR, CPU_CYCLES, either INST_RETIRED or INST_SPEC). Some others only are required if the implementation implements some other features. Check those wich are

Re: [PATCH 3/6] tests/boot_linux_console: fix extract_from_deb() comment

2020-01-30 Thread Stefano Garzarella
On Mon, Jan 27, 2020 at 04:36:31PM +, Liam Merwick wrote: > The second param in extract_from_deb() is 'path' not 'file' > > Signed-off-by: Liam Merwick > --- > tests/acceptance/boot_linux_console.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Stefano Garzarella >

Re: [PATCH 1/6] tests/boot_linux_console: add microvm acceptance test

2020-01-30 Thread Stefano Garzarella
On Mon, Jan 27, 2020 at 04:36:29PM +, Liam Merwick wrote: > Refactor test_x86_64_pc() to test_x86_64_machine() so that separate > functions which specify the Avocado tag of ':avocado: tags=machine:' > as being either 'pc' or 'microvm' can be used to test booting a > compressed kernel using

Re: [PATCH 5/6] tests/boot_linux_console: add extract_from_rpm method

2020-01-30 Thread Stefano Garzarella
On Mon, Jan 27, 2020 at 04:36:33PM +, Liam Merwick wrote: > Add a method to extract a specified file from an RPM to the test's > working directory and return the path to the extracted file. > > Signed-off-by: Liam Merwick > --- > tests/acceptance/boot_linux_console.py | 14 ++ >

Re: [PATCH v5 1/8] multifd: Add multifd-method parameter

2020-01-30 Thread Markus Armbruster
Juan Quintela writes: > Markus Armbruster wrote: >> Juan Quintela writes: >> >>> This will store the compression method to use. We start with none. >>> >>> Signed-off-by: Juan Quintela >>> Reviewed-by: Markus Armbruster >> >> I don't remember giving my R-by. I gave my Acked-by for >>

Re: [PATCH v2 01/29] hw/core: Allow setting 'virtio-blk-device.scsi' property on OSX host

2020-01-30 Thread Cornelia Huck
On Wed, 29 Jan 2020 22:23:17 +0100 Philippe Mathieu-Daudé wrote: > Commit ed65fd1a27 restricted the 'scsi' property to Linux hosts: It was restricted to Linux hosts before that commit; that only became problematic because we set it unconditionally in the compat code. > > 1261 static

Re: [PATCH v2 04/29] tests/acceptance: Use 'version-min' tag to verify QEMU binary version

2020-01-30 Thread Cornelia Huck
On Wed, 29 Jan 2020 22:23:20 +0100 Philippe Mathieu-Daudé wrote: > Introduce the 'version-min' tag. > Tests can set it to the minimum version of QEMU they require. > If QEMU is older, the tests will be cancelled (skipped): > > $ python -m avocado --show=app run >

[PATCH v4 07/11] monitor/hmp: move hmp_snapshot_* to block-hmp-cmds.c hmp_snapshot_blkdev is from GPLv2 version of the hmp-cmds.c thus have to change the licence to GPLv2

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 56 -- include/block/block-hmp-cmds.h | 4 +++ include/monitor/hmp.h | 3 -- monitor/hmp-cmds.c | 47 4

[PATCH v4 01/11] usb/dev-storage: remove unused include

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Philippe Mathieu-Daudé --- hw/usb/dev-storage.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw/usb/dev-storage.c b/hw/usb/dev-storage.c index 90da008df1..5629213d55 100644 --- a/hw/usb/dev-storage.c +++ b/hw/usb/dev-storage.c @@ -19,7 +19,6 @@

[PATCH v4 02/11] monitor/hmp: uninline add_init_drive

2020-01-30 Thread Maxim Levitsky
This is only used by hmp_drive_add. The code is just a bit shorter this way. No functional changes Signed-off-by: Maxim Levitsky Reviewed-by: Markus Armbruster --- device-hotplug.c | 31 --- 1 file changed, 12 insertions(+), 19 deletions(-) diff --git

Re: [PATCH v2 15/29] tests: rename virtio_seg_max_adjust to virtio_check_params

2020-01-30 Thread Cornelia Huck
On Wed, 29 Jan 2020 22:23:31 +0100 Philippe Mathieu-Daudé wrote: > From: Denis Plotnikov > > Since, virtio_seg_max_adjust checks not only seg_max, but also > virtqueue_size parameter, let's make the test more general and > add new parameters to be checked there in the future. > >

Re: [PATCH v2 04/29] tests/acceptance: Use 'version-min' tag to verify QEMU binary version

2020-01-30 Thread Philippe Mathieu-Daudé
On 1/30/20 1:26 PM, Cornelia Huck wrote: On Wed, 29 Jan 2020 22:23:20 +0100 Philippe Mathieu-Daudé wrote: Introduce the 'version-min' tag. Tests can set it to the minimum version of QEMU they require. If QEMU is older, the tests will be cancelled (skipped): $ python -m avocado --show=app

Re: request_alignment vs file size, how to fix crash?

2020-01-30 Thread Vladimir Sementsov-Ogievskiy
29.01.2020 21:01, Vladimir Sementsov-Ogievskiy wrote: Hi! I found a crash, which may be simply triggered for images unaligned to request_alignment: # ./qemu-io --image-opts -c 'write 0 512' driver=blkdebug,align=4096,image.driver=null-co,image.size=512 qemu-io: block/io.c:1505:

Re: [PATCH] riscv: Add semihosting support [v4]

2020-01-30 Thread Peter Maydell
On Wed, 29 Jan 2020 at 16:45, Keith Packard wrote: > > Peter Maydell writes: > > > True but irrelevant. You need to refer to a proper > > risc-v specification for your semihosting. > > The RISC-V Foundation defined semihosting as relative to the existing > ARM specification, so using a link to

[Bug 1805913] Re: readdir() returns NULL (errno=EOVERFLOW) for 32-bit user-static qemu on 64-bit host

2020-01-30 Thread diddly
There has been some motion on this by Aladjev Andrew. I will butcher the explanation of his approach if I try, but it is described in the following bugs. I have no idea of the schedule, or even possibility of adoption; it seems to still be in proof-of-concept phase. GLIBC bug (see last several

Re: [PATCH v2 2/4] virtio-scsi: default num_queues to -smp N

2020-01-30 Thread Cornelia Huck
On Thu, 30 Jan 2020 10:52:35 + Stefan Hajnoczi wrote: > On Thu, Jan 30, 2020 at 01:29:16AM +0100, Paolo Bonzini wrote: > > On 29/01/20 16:44, Stefan Hajnoczi wrote: > > > On Mon, Jan 27, 2020 at 02:10:31PM +0100, Cornelia Huck wrote: > > >> On Fri, 24 Jan 2020 10:01:57 + > > >>

Re: [GSoC/Outreachy QEMU proposal] Extend support for ioctls in QEMU linux-user mode

2020-01-30 Thread Laurent Vivier
Le 30/01/2020 à 12:09, Aleksandar Markovic a écrit : > 14:34 Čet, 23.01.2020. Aleksandar Markovic > је написао/ла: >> >> Extend support for ioctls in QEMU linux-user mode >> >> >> PLANNED ACTIVITIES >> >> BACKGROUND >> >> There is currently 2500+ ioctls defined

Re: [PATCH 2/6] tests/boot_linux_console: add BIOS acceptance test

2020-01-30 Thread Stefano Garzarella
Hi Liam, On Mon, Jan 27, 2020 at 04:36:30PM +, Liam Merwick wrote: > Add tests to use qboot with the 'pc' and 'microvm' machine classes > by adding the '-bios' option via self.vm.add_args() before calling > do_test_x86_64_machine(). > > Signed-off-by: Liam Merwick > --- >

[kvm-unit-tests PATCH v2 9/9] arm: pmu: Test overflow interrupts

2020-01-30 Thread Eric Auger
Test overflows for MEM_ACCESS and SW_INCR events. Also tests overflows with 64-bit events. Signed-off-by: Eric Auger --- v1 -> v2: - inline setup_irq() code --- arm/pmu.c | 137 ++ arm/unittests.cfg | 6 ++ 2 files changed, 143

[kvm-unit-tests PATCH v2 7/9] arm: pmu: test 32-bit <-> 64-bit transitions

2020-01-30 Thread Eric Auger
Test configurations where we transit from 32b to 64b counters and conversely. Also tests configuration where chain counters are configured but only one counter is enabled. Signed-off-by: Eric Auger --- arm/pmu.c | 136 ++ arm/unittests.cfg |

[kvm-unit-tests PATCH v2 8/9] arm: gic: Provide per-IRQ helper functions

2020-01-30 Thread Eric Auger
From: Andre Przywara A common theme when accessing per-IRQ parameters in the GIC distributor is to set fields of a certain bit width in a range of MMIO registers. Examples are the enabled status (one bit per IRQ), the level/edge configuration (2 bits per IRQ) or the priority (8 bits per IRQ).

[PATCH v2 03/12] gitlab-ci: Refresh the list of iotests

2020-01-30 Thread Alex Bennée
From: Thomas Huth iotest 147 and 205 have recently been marked as "NBD-only", so they are currently simply skipped and thus can be removed. iotest 129 occasionally fails in the gitlab-CI, and according to Max, there are some known issues with this test (see for example this URL:

[PATCH v2 09/12] .travis.yml: build documents under bionic

2020-01-30 Thread Alex Bennée
It looks like the xenial tooling doesn't like something in our setup. We should probably be moving to bionic for everything soon anyway (libssh aside). Signed-off-by: Alex Bennée --- .travis.yml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml

[PATCH v2 06/12] hw/hppa/Kconfig: LASI chipset requires PARALLEL port

2020-01-30 Thread Alex Bennée
From: Philippe Mathieu-Daudé The PARISC Lasi chipset emulation requires some of the common parallel support and fails to build on a --without-default-devices: LINKhppa-softmmu/qemu-system-hppa /usr/bin/ld: hw/hppa/lasi.o: in function `lasi_init': hw/hppa/lasi.c:324: undefined

Re: [PATCH v4 3/4] spapr: Add NVDIMM device support

2020-01-30 Thread Shivaprasad G Bhat
On 01/03/2020 06:50 AM, David Gibson wrote: On Tue, Dec 17, 2019 at 02:49:14AM -0600, Shivaprasad G Bhat wrote: Add support for NVDIMM devices for sPAPR. Piggyback on existing nvdimm device interface in QEMU to support virtual NVDIMM devices for Power. Create the required DT entries for the

[PATCH v5 0/4] ppc: spapr: virtual NVDIMM support

2020-01-30 Thread Shivaprasad G Bhat
The patchset attempts to implement the virtual NVDIMM for pseries. PAPR semantics is such that each NVDIMM device is comprising of multiple SCM(Storage Class Memory) blocks. The hypervisor is expected to prepare the FDT for the NVDIMM device and send guest a hotplug interrupt with new type

[PATCH v5 2/4] nvdimm: add uuid property to nvdimm

2020-01-30 Thread Shivaprasad G Bhat
For ppc64, PAPR requires the nvdimm device to have UUID property set in the device tree. Add an option to get it from the user. Signed-off-by: Shivaprasad G Bhat Reviewed-by: David Gibson --- hw/mem/nvdimm.c | 40 include/hw/mem/nvdimm.h |

Re: [PATCH] riscv: Add semihosting support [v4]

2020-01-30 Thread Peter Maydell
On Thu, 30 Jan 2020 at 11:38, Palmer Dabbelt wrote: > * The semihosting comment doesn't define the semihosting call numbers, just > the > sequence to get to a call. That said, we haven't written down the Linux ABI > either -- though there's a much larger breadth of software out there that >

Re: [PATCH 4/6] travis.yml: install rpm2cpio for acceptance tests

2020-01-30 Thread Stefano Garzarella
On Mon, Jan 27, 2020 at 04:36:32PM +, Liam Merwick wrote: > The extract_from_rpm() method added for the PVH acceptance tests needs > rpm2cpio to extract a vmlinux binary from an RPM. > > Signed-off-by: Liam Merwick > --- > .travis.yml | 1 + > 1 file changed, 1 insertion(+) Reviewed-by:

[PATCH v4 00/11] RFC: [for 5.0]: HMP monitor handlers refactoring

2020-01-30 Thread Maxim Levitsky
This patch series is bunch of cleanups to the hmp monitor code. It mostly moves the blockdev related hmp handlers to its own file, and does some minor refactoring. No functional changes expected. Changes from V1: * move the handlers to block/monitor/block-hmp-cmds.c * tiny cleanup for the

[PATCH v4 09/11] monitor/hmp: move remaining hmp_block* functions to block-hmp-cmds.c

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 140 + include/block/block-hmp-cmds.h | 9 +++ include/monitor/hmp.h | 6 -- monitor/hmp-cmds.c | 137

[PATCH v4 03/11] monitor/hmp: rename device-hotplug.c to block/monitor/block-hmp-cmds.c

2020-01-30 Thread Maxim Levitsky
These days device-hotplug.c only contains the hmp_drive_add In the next patch, rest of hmp_drive* functions will be moved there. Also add block-hmp-cmds.h to contain prototypes of these functions License for block-hmp-cmds.h since it contains the code moved from sysemu.h which lacks license and

Re: [PATCH v6 00/11] error: auto propagated local_err part I

2020-01-30 Thread Vladimir Sementsov-Ogievskiy
Markus, what about this? Should I respin? 10.01.2020 22:41, Vladimir Sementsov-Ogievskiy wrote: Hi all! Now, when preparations from [RFC v5 000/126] error: auto propagated local_err https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg02771.html

[PATCH v4 05/11] monitor/hmp: move hmp_drive_mirror and hmp_drive_backup to block-hmp-cmds.c Moved code was added after 2012-01-13, thus under GPLv2+

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 60 ++ include/block/block-hmp-cmds.h | 12 +-- include/monitor/hmp.h | 2 -- monitor/hmp-cmds.c | 58

[Bug 1861404] [NEW] AVX instruction VMOVDQU implementation error for YMM registers

2020-01-30 Thread Stevie Lavern
Public bug reported: Hi, Tested with Qemu 4.2.0, and with git version bddff6f6787c916b0e9d63ef9e4d442114257739. The x86 AVX instruction VMOVDQU doesn't work properly with YMM registers (32 bytes). It works with XMM registers (16 bytes) though. See the attached test case `ymm.c`: when copying

Re: [PATCH] Handling SIGSETXID used by glibc NPTL setuid/setgid

2020-01-30 Thread Laurent Vivier
Le 29/01/2020 à 17:12, Laurent Vivier a écrit : > Le 28/01/2020 à 10:26, Peter Maydell a écrit : >> On Thu, 16 Jan 2020 at 11:58, Matus Kysel wrote: >>> >>> Used same style to handle another glibc reserved signal SIGSETXID (33), >>> that is used by glibc NPTL setuid/setgid functions. This should

Re: [RFC PATCH] hw/arm/virt: Support NMI injection

2020-01-30 Thread Marc Zyngier
Gavin, On 2020-01-29 21:54, Gavin Shan wrote: On 1/29/20 6:57 PM, Julien Thierry wrote: On 1/29/20 3:46 AM, Gavin Shan wrote: On 1/28/20 7:29 PM, Julien Thierry wrote: .../... Julien, thanks for the explanation. The question we're not sure if NMI should be injected on receiving HMP/QMP

Re: [PATCH 10/10] tests/qemu-iotests/check: Update to match Python 3 interpreter

2020-01-30 Thread Kevin Wolf
Am 30.01.2020 um 00:14 hat Philippe Mathieu-Daudé geschrieben: > All the iotests Python scripts have been converted to search for > the Python 3 interpreter. Update the ./check script accordingly. > > Signed-off-by: Philippe Mathieu-Daudé > diff --git a/tests/qemu-iotests/check

[kvm-unit-tests PATCH v2 5/9] arm: pmu: Basic event counter Tests

2020-01-30 Thread Eric Auger
Adds the following tests: - event-counter-config: test event counter configuration - basic-event-count: - programs counters #0 and #1 to count 2 required events (resp. CPU_CYCLES and INST_RETIRED). Counter #0 is preset to a value close enough to the 32b overflow limit so that we check the

[kvm-unit-tests PATCH v2 1/9] arm64: Provide read/write_sysreg_s

2020-01-30 Thread Eric Auger
From: Andrew Jones Sometimes we need to test access to system registers which are missing assembler mnemonics. Signed-off-by: Andrew Jones Reviewed-by: Alexandru Elisei --- lib/arm64/asm/sysreg.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/lib/arm64/asm/sysreg.h

[kvm-unit-tests PATCH v2 3/9] arm: pmu: Add a pmu struct

2020-01-30 Thread Eric Auger
This struct aims at storing information potentially used by all tests such as the pmu version, the read-only part of the PMCR, the number of implemented event counters, ... Signed-off-by: Eric Auger --- arm/pmu.c | 30 +- 1 file changed, 25 insertions(+), 5

[kvm-unit-tests PATCH v2 6/9] arm: pmu: Test chained counter

2020-01-30 Thread Eric Auger
Add 2 tests exercising chained counters. The first one uses CPU_CYCLES and the second one uses SW_INCR. Signed-off-by: Eric Auger --- arm/pmu.c | 128 ++ arm/unittests.cfg | 12 + 2 files changed, 140 insertions(+) diff --git a/arm/pmu.c

[kvm-unit-tests PATCH v2 2/9] arm: pmu: Let pmu tests take a sub-test parameter

2020-01-30 Thread Eric Auger
As we intend to introduce more PMU tests, let's add a sub-test parameter that will allow to categorize them. Existing tests are in the cycle-counter category. Signed-off-by: Eric Auger --- arm/pmu.c | 24 +++- arm/unittests.cfg | 7 --- 2 files changed, 19

[PATCH v2 10/12] .travis.yml: move cache flushing to early common phase

2020-01-30 Thread Alex Bennée
We shall be adding more common early setup in a future commit. Signed-off-by: Alex Bennée --- .travis.yml | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/.travis.yml b/.travis.yml index f245765cfe..1b92f40eab 100644 --- a/.travis.yml +++ b/.travis.yml @@ -82,11

[PATCH v2 05/12] .shippable: --disable-docs for cross-compile tests

2020-01-30 Thread Alex Bennée
The sphinx support is fairly new and we don't seem to have all the bugs worked out for cross development environments right now. Signed-off-by: Alex Bennée --- .shippable.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.shippable.yml b/.shippable.yml index

[PATCH v2 02/12] tests/docker: better handle symlinked libs

2020-01-30 Thread Alex Bennée
When we are copying we want to ensure we grab the first resolution (the found in path section). However even that binary might be a symlink so lets make sure we chase the symlinks to copy the right binary to where it can be found. Signed-off-by: Alex Bennée --- tests/docker/docker.py | 5 +++--

[PATCH v2 07/12] .travis.yml: Drop superfluous use of --python=python3 parameter

2020-01-30 Thread Alex Bennée
From: Philippe Mathieu-Daudé As we require Python3 since commit ddf9069963, we don't need to explicit it with the --python=/usr/bin/python3 configure option. Reported-by: Thomas Huth Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20200125184217.30034-1-f4...@amsat.org> --- .travis.yml |

Re: [PATCH] riscv: Add semihosting support [v4]

2020-01-30 Thread Palmer Dabbelt
On Thu, 30 Jan 2020 10:54:37 GMT (+), Peter Maydell wrote: On Wed, 29 Jan 2020 at 16:45, Keith Packard wrote: Peter Maydell writes: > True but irrelevant. You need to refer to a proper > risc-v specification for your semihosting. The RISC-V Foundation defined semihosting as relative to

[PATCH v2 12/12] .travis.yml: limit the arm64 target list

2020-01-30 Thread Alex Bennée
Even though we can do a good multi-threaded build we are still exceeding the 50 minute build time while running the tests. Signed-off-by: Alex Bennée --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index a600f508b0..71bf650b1d 100644

[PATCH v2 08/12] .travis.yml: Add description to each job

2020-01-30 Thread Alex Bennée
From: Philippe Mathieu-Daudé The NAME variable can be used to describe nicely a job (see [*]). As we currently have 32 jobs, use it. This helps for quickly finding a particular job. before: https://travis-ci.org/qemu/qemu/builds/639887646 after:

[PATCH v2 04/12] travis.yml: Install genisoimage package

2020-01-30 Thread Alex Bennée
From: Wainer dos Santos Moschetta The genisoimage program is required for tests/cdrom-test tests, otherwise they are skipped. The current Travis environments do not provide it by default, so let's explicitly require the genisoimage package. Signed-off-by: Wainer dos Santos Moschetta

[PATCH v5 3/4] spapr: Add NVDIMM device support

2020-01-30 Thread Shivaprasad G Bhat
Add support for NVDIMM devices for sPAPR. Piggyback on existing nvdimm device interface in QEMU to support virtual NVDIMM devices for Power. Create the required DT entries for the device (some entries have dummy values right now). The patch creates the required DT node and sends a hotplug

[PATCH v5 1/4] mem: move nvdimm_device_list to utilities

2020-01-30 Thread Shivaprasad G Bhat
nvdimm_device_list is required for parsing the list for devices in subsequent patches. Move it to common utility area. Signed-off-by: Shivaprasad G Bhat Reviewed-by: Igor Mammedov Reviewed-by: David Gibson --- hw/acpi/nvdimm.c| 28 +---

Re: [PATCH v4 4/4] spapr: Add Hcalls to support PAPR NVDIMM device

2020-01-30 Thread Shivaprasad G Bhat
On 01/03/2020 07:14 AM, David Gibson wrote: On Tue, Dec 17, 2019 at 02:49:36AM -0600, Shivaprasad G Bhat wrote: This patch implements few of the necessary hcalls for the nvdimm support. Fixing all the comments. Of course, we're not *actually* unbinding anything. But I guess the idea here

Re: [PATCH 6/6] tests/boot_linux_console: add PVH acceptance tests

2020-01-30 Thread Stefano Garzarella
On Mon, Jan 27, 2020 at 04:36:34PM +, Liam Merwick wrote: > Add tests to boot an uncompressed kernel using the x86/HVM direct boot ABI. > The vmlinux binary is obtained from a small RPM for Kata containers and > extracted using the new extract_from_rpm() method. > > Signed-off-by: Liam

Re: request_alignment vs file size, how to fix crash?

2020-01-30 Thread Kevin Wolf
Am 30.01.2020 um 12:30 hat Vladimir Sementsov-Ogievskiy geschrieben: > 30.01.2020 14:11, Kevin Wolf wrote: > > Am 30.01.2020 um 11:40 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > 29.01.2020 21:01, Vladimir Sementsov-Ogievskiy wrote: > > > > Hi! > > > > > > > > I found a crash, which may be

[PATCH v4 06/11] monitor/hmp: move hmp_block_job* to block-hmp-cmds.c

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 52 ++ include/block/block-hmp-cmds.h | 6 include/monitor/hmp.h | 5 monitor/hmp-cmds.c | 52

[PATCH v4 04/11] monitor/hmp: move hmp_drive_del and hmp_commit to block-hmp-cmds.c

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 108 - blockdev.c | 95 - include/block/block-hmp-cmds.h | 4 ++ include/sysemu/blockdev.h | 4 -- 4

Re: [PATCH 02/13] qcrypto-luks: implement encryption key management

2020-01-30 Thread Kevin Wolf
Am 28.01.2020 um 18:32 hat Daniel P. Berrangé geschrieben: > On Tue, Jan 28, 2020 at 05:11:16PM +, Daniel P. Berrangé wrote: > > On Tue, Jan 21, 2020 at 03:13:01PM +0200, Maxim Levitsky wrote: > > > On Tue, 2020-01-21 at 08:54 +0100, Markus Armbruster wrote: > > > > > > > > > > > > > > +##

Re: [PATCH v2 18/29] tests/acceptance/virtio_check_params: List machine being tested

2020-01-30 Thread Cornelia Huck
On Wed, 29 Jan 2020 22:23:34 +0100 Philippe Mathieu-Daudé wrote: > Add logging for easier debugging of failures: > > $ avocado --show=machine run tests/acceptance/virtio_check_params.py >(1/1) > tests/acceptance/virtio_check_params.py:VirtioMaxSegSettingsCheck.test_machine_types: >

Re: [PATCH v2 19/29] tests/acceptance/virtio_check_params: Only test Xen as superuser

2020-01-30 Thread Cornelia Huck
On Wed, 29 Jan 2020 22:23:35 +0100 Philippe Mathieu-Daudé wrote: > When running the test unprivileged, we get: > > $ avocado --show=app,machine run tests/acceptance/virtio_check_params.py > JOB ID : b631d5d692e49b791b211d33b80730315d561d45 > JOB LOG:

Re: [PATCH 02/13] qcrypto-luks: implement encryption key management

2020-01-30 Thread Maxim Levitsky
On Tue, 2020-01-28 at 17:21 +, Daniel P. Berrangé wrote: > On Tue, Jan 14, 2020 at 09:33:39PM +0200, Maxim Levitsky wrote: > > Next few patches will expose that functionality > > to the user. > > > > Signed-off-by: Maxim Levitsky > > --- > > crypto/block-luks.c | 374

Re: [PATCH v2 01/29] hw/core: Allow setting 'virtio-blk-device.scsi' property on OSX host

2020-01-30 Thread Philippe Mathieu-Daudé
On 1/30/20 1:20 PM, Cornelia Huck wrote: On Wed, 29 Jan 2020 22:23:17 +0100 Philippe Mathieu-Daudé wrote: Commit ed65fd1a27 restricted the 'scsi' property to Linux hosts: It was restricted to Linux hosts before that commit; that only became problematic because we set it unconditionally in

[Bug 1861404] Re: AVX instruction VMOVDQU implementation error for YMM registers

2020-01-30 Thread Stevie Lavern
Note: Qemu has been built with the following commands: ``` % ./configure --target-list=x86_64-linux-user && make OR % ./configure --target-list=x86_64-linux-user --enable-avx2 && make ``` -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to

[Bug 1861394] [NEW] qemu-system-riscv64 hangs after poweroff linux command

2020-01-30 Thread Wojtek Lewandowski
Public bug reported: QEMU Version : v4.2.0-773-g43d1455-dirty (commit 43d1455cf84283466e5c22a217db5ef4b8197b14) Command: qemu-system-riscv64 -machine virt -kernel ./bbl -nographic -initrd rootfs.cpio.gz -append "root=/dev/ram console=ttyS0" Host:LSB Version::core-4.1-amd64:core-4.1-noarch

Re: request_alignment vs file size, how to fix crash?

2020-01-30 Thread Kevin Wolf
Am 30.01.2020 um 11:40 hat Vladimir Sementsov-Ogievskiy geschrieben: > 29.01.2020 21:01, Vladimir Sementsov-Ogievskiy wrote: > > Hi! > > > > I found a crash, which may be simply triggered for images unaligned to > > request_alignment: > > > > # ./qemu-io --image-opts -c 'write 0 512' > >

[kvm-unit-tests PATCH v2 0/9] KVM: arm64: PMUv3 Event Counter Tests

2020-01-30 Thread Eric Auger
This series implements tests exercising the PMUv3 event counters. It tests both the 32-bit and 64-bit versions. Overflow interrupts also are checked. Those tests only are written for arm64. It allowed to reveal some issues related to SW_INCR implementation (esp. related to 64-bit implementation),

Re: request_alignment vs file size, how to fix crash?

2020-01-30 Thread Vladimir Sementsov-Ogievskiy
30.01.2020 14:11, Kevin Wolf wrote: Am 30.01.2020 um 11:40 hat Vladimir Sementsov-Ogievskiy geschrieben: 29.01.2020 21:01, Vladimir Sementsov-Ogievskiy wrote: Hi! I found a crash, which may be simply triggered for images unaligned to request_alignment: # ./qemu-io --image-opts -c 'write 0

[PATCH v2 01/12] tests/docker: move most cross compilers to buster base

2020-01-30 Thread Alex Bennée
This includes fixing up the dependencies (Which were already wrong for one of the mips variants). Signed-off-by: Alex Bennée --- tests/docker/Makefile.include| 16 tests/docker/dockerfiles/debian-amd64.docker | 2 +-

[PATCH v2 11/12] .travis.yml: probe for number of available processors

2020-01-30 Thread Alex Bennée
The arm64 hardware was especially hit by only building on 3 of the 32 available cores. Introduce a JOBS environment variable which we use for all parallel builds. We still run the main checks single threaded though so to make it easier to spot hangs. Signed-off-by: Alex Bennée --- .travis.yml |

Re: [PULL 0/2] Ide patches

2020-01-30 Thread Peter Maydell
On Tue, 28 Jan 2020 at 01:07, John Snow wrote: > > The following changes since commit 105b07f1ba462ec48b27e5cb74ddf81c6a79364c: > > Merge remote-tracking branch 'remotes/cohuck/tags/s390x-20200127' into > staging (2020-01-27 13:02:36 +) > > are available in the Git repository at: > >

[PATCH v5 4/4] spapr: Add Hcalls to support PAPR NVDIMM device

2020-01-30 Thread Shivaprasad G Bhat
This patch implements few of the necessary hcalls for the nvdimm support. PAPR semantics is such that each NVDIMM device is comprising of multiple SCM(Storage Class Memory) blocks. The guest requests the hypervisor to bind each of the SCM blocks of the NVDIMM device using hcalls. There can be SCM

Re: [GSoC/Outreachy QEMU proposal] Extend support for ioctls in QEMU linux-user mode

2020-01-30 Thread Peter Maydell
On Thu, 30 Jan 2020 at 11:20, Laurent Vivier wrote: > > Le 30/01/2020 à 12:09, Aleksandar Markovic a écrit : > > Laurent, all this seems to be dependant on whether you are ready to > > mentor the project. Are you? > > Yes, of course. > > > The deadline for submitting GSoC/Outreachy projects

[PATCH v2 00/12] testing/next (with build fixes!)

2020-01-30 Thread Alex Bennée
Hi, It seemed the CI degraded while I wasn't looking so I've rolled up a bunch of fixes to try and get things working again. There is some messing around for documentation builds (disabling on shippable, moving to bionic on Travis), a build fix for hppa + no-default-configs and some attempts to

Re: [PATCH v2 02/29] accel/accel: Make TYPE_ACCEL abstract

2020-01-30 Thread Cornelia Huck
On Wed, 29 Jan 2020 22:23:18 +0100 Philippe Mathieu-Daudé wrote: > There is no generic acceleration, we have to use specific > implementations. Make the base class abstract. > > Fixes: b14a0b7469f Fixes: b14a0b7469fa ("accel: Use QOM classes for accel types") > Signed-off-by: Philippe

[PATCH v4 10/11] monitor/hmp: move hmp_info_block* to block-hmp-cmds.c

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 390 + include/block/block-hmp-cmds.h | 4 + include/monitor/hmp.h | 4 - monitor/hmp-cmds.c | 388 4

[PATCH v4 11/11] monitor/hmp: Move hmp_drive_add_node to block-hmp-cmds.c

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 30 blockdev.c | 42 +++--- include/block/block_int.h | 5 ++-- 3 files changed, 41 insertions(+), 36

[PATCH v4 08/11] monitor/hmp: move hmp_nbd_server* to block-hmp-cmds.c

2020-01-30 Thread Maxim Levitsky
Signed-off-by: Maxim Levitsky Reviewed-by: Dr. David Alan Gilbert --- block/monitor/block-hmp-cmds.c | 89 ++ include/block/block-hmp-cmds.h | 5 ++ include/monitor/hmp.h | 4 -- monitor/hmp-cmds.c | 87 - 4

Re: [PATCH 02/13] qcrypto-luks: implement encryption key management

2020-01-30 Thread Daniel P . Berrangé
On Thu, Jan 30, 2020 at 01:38:47PM +0100, Kevin Wolf wrote: > Am 28.01.2020 um 18:32 hat Daniel P. Berrangé geschrieben: > > On Tue, Jan 28, 2020 at 05:11:16PM +, Daniel P. Berrangé wrote: > > > On Tue, Jan 21, 2020 at 03:13:01PM +0200, Maxim Levitsky wrote: > > > > On Tue, 2020-01-21 at 08:54

Re: [PULL 00/18] Pull migration patches

2020-01-30 Thread Peter Maydell
On Wed, 29 Jan 2020 at 11:16, Juan Quintela wrote: > > The following changes since commit 4c60e3289875ae6c516a37523bcecb87f68ce67c: > > Merge remote-tracking branch 'remotes/rth/tags/pull-pa-20200127' into > staging (2020-01-28 15:11:04 +) > > are available in the Git repository at: > >

Re: [PATCH v5 7/8] multifd: Add multifd-zstd-level parameter

2020-01-30 Thread Markus Armbruster
Juan Quintela writes: > Signed-off-by: Juan Quintela Same confusion as in PATCH 4; my proposal there applies here, too. For QAPI: Acked-by: Markus Armbruster

Re: [PATCH v5 4/8] multifd: Add multifd-zlib-level parameter

2020-01-30 Thread Markus Armbruster
Juan Quintela writes: > It will indicate which level use for compression. > > Signed-off-by: Juan Quintela This is slightly confusing (there is no zlib compression), unless you peek at the next patch (which adds zlib compression). Three ways to make it less confusing: * Squash the two

Re: [PATCH] spapr: Enable DD2.3 accelerated count cache flush in pseries-5.0 machine

2020-01-30 Thread Laurent Vivier
On 30/01/2020 02:26, David Gibson wrote: > For POWER9 DD2.2 cpus, the best current Spectre v2 indirect branch > mitigation is "count cache disabled", which is configured with: > -machine cap-ibs=fixed-ccd > However, this option isn't available on DD2.3 CPUs with KVM, because they > don't have

Re: [PATCH v5 8/8] multifd: Add zstd compression multifd support

2020-01-30 Thread Markus Armbruster
Juan Quintela writes: > Signed-off-by: Juan Quintela For QAPI: Acked-by: Markus Armbruster

Re: [PATCH] tests/acceptance: Add boot tests for some of the QEMU advent calendar images

2020-01-30 Thread Thomas Huth
On 29/01/2020 18.48, BALATON Zoltan wrote: > On Wed, 29 Jan 2020, Thomas Huth wrote: >> On 27/01/2020 18.31, Wainer dos Santos Moschetta wrote: >>> On 1/24/20 3:03 PM, Thomas Huth wrote: The 2018 edition of the QEMU advent calendar 2018 featured Linux images for various non-x86 machines.

[Bug 1805913] Re: readdir() returns NULL (errno=EOVERFLOW) for 32-bit user-static qemu on 64-bit host

2020-01-30 Thread Philippe Vaucher
Is there a workaround for this? I tried: - Building on an XFS partition. - Building from ubuntu:16.04 so the host has glib <2.27. It looks like the only way is to have the chroot with glib <2.27, and in alpine images glib is at minimum 2.56. If the bug is fixed in glib maybe I can install glib

[Bug 1860920] Re: qemu-s390x-softmmu: crash

2020-01-30 Thread Thomas Huth
** Information type changed from Private Security to Public Security -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1860920 Title: qemu-s390x-softmmu: crash Status in QEMU: New Bug description:

[PULL 05/18] qapi/block-core: add option for io_uring

2020-01-30 Thread Stefan Hajnoczi
From: Aarushi Mehta Since io_uring is the actual name of the Linux API, we use it as enum value even though the QAPI schema conventions would prefer io-uring. Signed-off-by: Aarushi Mehta Acked-by: Markus Armbruster Acked-by: Stefano Garzarella Signed-off-by: Stefan Hajnoczi Message-id:

  1   2   3   4   5   >