[PATCH v2] Remove VXHS block device

2020-07-11 Thread Marc-André Lureau
The vxhs code doesn't compile since v2.12.0. There's no point in fixing and then adding CI for a config that our users have demonstrated that they do not use; better to just remove it. Signed-off-by: Marc-André Lureau Reviewed-by: Markus Armbruster --- configure| 39 --

Re: [PATCH v3 1/1] scripts/performance: Add dissect.py script

2020-07-11 Thread Philippe Mathieu-Daudé
On 7/9/20 7:20 AM, Ahmed Karaman wrote: > Python script that dissects QEMU execution into three main phases: > code generation, JIT execution and helpers execution. > > Syntax: > dissect.py [-h] -- [] \ > [] > > [-h] - Print the script arguments help message. > > Example of us

Re: [PATCH 0/2] module fixes

2020-07-11 Thread Alex Bennée
Gerd Hoffmann writes: > Gerd Hoffmann (2): > tests: fix "make check-qtest" for modular builds > Revert "vga: build virtio-gpu as module" Queued to pr/100720-testing-and-misc-2, thanks. > > util/module.c| 6 -- > hw/display/Makefile.objs | 23 ++---

Re: [PATCH 2/2] iotests: Set LC_ALL=C for sort

2020-07-11 Thread Alex Bennée
Max Reitz writes: > Otherwise the result is basically unpredictable. > > (Note that the precise environment variable to control sorting order is > LC_COLLATE, but LC_ALL overrides LC_COLLATE, and we do not want the > sorting order to be messed up if LC_ALL is set in the environment.) > > Report

Re: [PULL v2 00/32] AVR port

2020-07-11 Thread Philippe Mathieu-Daudé
Peter, please disregard this pullreq as I committed an incorrect line while rebasing the fix :/ I'll send v3 shortly. On 7/10/20 7:04 PM, Philippe Mathieu-Daudé wrote: > Since v1: > > Fixed issue on big-endian host reported by Peter Maydell. > > Possible false-positives from checkpatch: > > W

[PULL v3 00/32] AVR port

2020-07-11 Thread Philippe Mathieu-Daudé
f2a1cf9180f63e88bb38ff21c169da97c3f2bad5: Merge remote-tracking branch 'remotes/armbru/tags/pull-error-2020-07-07-v2'= into staging (2020-07-10 14:41:23 +0100) are available in the Git repository at: https://gitlab.com/philmd/qemu.git tags/avr-port-20200711 for you to fetch cha

[PULL v3 09/32] target/avr: Add instruction helpers

2020-07-11 Thread Philippe Mathieu-Daudé
From: Michael Rolnik Add helpers for instructions that need to interact with QEMU. Also, add stubs for unimplemented instructions. Instructions SPM and WDR are left unimplemented because they require emulation of complex peripherals. The implementation of instruction SLEEP is very limited due to

Re: [PATCH 3/3] cpu-timers, icount: new modules

2020-07-11 Thread Claudio Fontana
On 7/11/20 12:45 AM, Paolo Bonzini wrote: > On 10/07/20 06:36, Thomas Huth wrote: >> >> In short this goes away if I again set icount to enabled for qtest, >> basically ensuring that --enable-tcg is there and then reenabling icount. >> >> qtest was forcing icount and shift=0 by creating qemu option

Re: [PATCH 3/3] cpu-timers, icount: new modules

2020-07-11 Thread Paolo Bonzini
On 11/07/20 11:14, Claudio Fontana wrote: > On 7/11/20 12:45 AM, Paolo Bonzini wrote: >> On 10/07/20 06:36, Thomas Huth wrote: >>> >>> In short this goes away if I again set icount to enabled for qtest, >>> basically ensuring that --enable-tcg is there and then reenabling icount. >>> >>> qtest was

Re: [PATCH 1/2] block/block-backend: add converter from BlockAcctStats to BlockBackend

2020-07-11 Thread Vladimir Sementsov-Ogievskiy
10.07.2020 20:27, Denis V. Lunev wrote: Right now BlockAcctStats is always reside on BlockBackend. This structure is not used in any other place. Thus we are able to create a converter from one pointer to another. Signed-off-by: Denis V. Lunev CC: Vladimir Sementsov-Ogievskiy CC: Kevin Wolf C

[PATCH 1/3] hw/arm/virt: Move post cpu realize check into its own function

2020-07-11 Thread Andrew Jones
We'll add more to this new function in coming patches so we also state the gic must be created and call it below create_gic(). No functional change intended. Signed-off-by: Andrew Jones --- hw/arm/virt.c | 43 +++ 1 file changed, 27 insertions(+), 16 dele

[PATCH 2/3] hw/arm/virt: Move kvm pmu setup to virt_cpu_post_init

2020-07-11 Thread Andrew Jones
Move the KVM PMU setup part of fdt_add_pmu_nodes() to virt_cpu_post_init(), which is a more appropriate location. Now fdt_add_pmu_nodes() is also named more appropriately, because it no longer does anything but fdt node creation. No functional change intended. Signed-off-by: Andrew Jones --- hw

[PATCH 0/3] hw/arm/virt: Introduce kvm-steal-time

2020-07-11 Thread Andrew Jones
KVM supports the ability to publish the amount of time that VCPUs were runnable, but not running due to other host threads running instead, to the guest. The guest scheduler may use that information when making decisions and the guest may expose it to its userspace (Linux publishes this information

[PATCH 3/3] hw/arm/virt: Implement kvm-steal-time

2020-07-11 Thread Andrew Jones
We add the kvm-steal-time CPU property and implement it for machvirt. A tiny bit of refactoring was also done to allow pmu and pvtime to use the same vcpu device helper functions. Signed-off-by: Andrew Jones --- docs/system/arm/cpu-features.rst | 11 + hw/arm/virt.c| 33 +

Re: [PATCH 2/2] block: add logging facility for long standing IO requests

2020-07-11 Thread Vladimir Sementsov-Ogievskiy
10.07.2020 20:27, Denis V. Lunev wrote: There are severe delays with IO requests processing if QEMU is running in virtual machine or over software defined storage. Such delays potentially results in unpredictable guest behavior. For example, guests over IDE or SATA drive could remount filesystem

[Bug 1886318] Re: Qemu after v5.0.0 breaks macos guests

2020-07-11 Thread Simon John
Is this not the place to report qemu bugs? -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1886318 Title: Qemu after v5.0.0 breaks macos guests Status in QEMU: New Bug description: The Debian S

[PATCH v4 00/22] target/xtensa: implement double precision FPU

2020-07-11 Thread Max Filippov
Hello, this series implements double precision floating point unit option for target/xtensa, updates FPU tests and adds two new CPU cores, one with FPU2000 option and one with DFPU option. It is tagged xtensa-5.1-dfp-v4 in the qemu-xtensa tree at git://github.com/OSLL/qemu-xtensa.git I don't pos

[PATCH v4 03/22] softfloat: add xtensa specialization for pickNaNMulAdd

2020-07-11 Thread Max Filippov
pickNaNMulAdd logic on Xtensa is to apply pickNaN to the inputs of the expression (a * b) + c. However if default NaN is produces as a result of (a * b) calculation it is not considered when c is NaN. So with two pickNaN variants there must be two pickNaNMulAdd variants. In addition the invalid fla

[PATCH v4 01/22] softfloat: make NO_SIGNALING_NANS runtime property

2020-07-11 Thread Max Filippov
target/xtensa, the only user of NO_SIGNALING_NANS macro has FPU implementations with and without the corresponding property. With NO_SIGNALING_NANS being a macro they cannot be a part of the same QEMU executable. Replace macro with new property in float_status to allow cores with different FPU impl

[PATCH v4 02/22] softfloat: pass float_status pointer to pickNaN

2020-07-11 Thread Max Filippov
Pass float_status structure pointer to the pickNaN so that machine-specific settings are available to NaN selection code. Add use_first_nan property to float_status and use it in Xtensa-specific pickNaN. Cc: Peter Maydell Cc: "Alex Bennée" Reviewed-by: Alex Bennée Reviewed-by: Richard Henderson

[PATCH v4 04/22] target/xtensa: add geometry to xtensa_get_regfile_by_name

2020-07-11 Thread Max Filippov
Register file name may not uniquely identify a register file in the set of configurations. E.g. floating point registers may have different size in different configurations. Use register file geometry as additional identifier. Signed-off-by: Max Filippov --- target/xtensa/cpu.h | 2 +- ta

[PATCH v4 06/22] target/xtensa: rename FPU2000 translators and helpers

2020-07-11 Thread Max Filippov
Add _s suffix to all FPU2000 opcode translators and helpers that also have double-precision variant to unify naming and allow adding DFPU implementations. Add _fpu2k_ to the names of helpers that will have different implementation for the DFPU . Reviewed-by: Richard Henderson Signed-off-by: Max F

[PATCH v4 07/22] target/xtensa: move FSR/FCR register accessors

2020-07-11 Thread Max Filippov
Move FSR/FCR register accessors from core opcodes to FPU2000 opcodes as they are FPU2000-specific. Reviewed-by: Richard Henderson Signed-off-by: Max Filippov --- target/xtensa/translate.c | 64 +++ 1 file changed, 32 insertions(+), 32 deletions(-) diff --git

[PATCH v4 05/22] target/xtensa: support copying registers up to 64 bits wide

2020-07-11 Thread Max Filippov
FLIX dependency breaking code assumes that all registers are 32 bit wide. This may not always be correct. Extract actual register width from the associated register file and use it to create temporaries of correct width and generate correct data movement instructions. Signed-off-by: Max Filippov

[PATCH v4 11/22] target/xtensa: implement FPU division and square root

2020-07-11 Thread Max Filippov
This does not implement all opcodes related to div/sqrt as specified in the xtensa ISA, partly because the official specification is not complete and partly because precise implementation is unnecessarily complex. Instead instructions specific to the div/sqrt sequences are implemented differently,

[PATCH v4 09/22] target/xtensa: add DFPU option

2020-07-11 Thread Max Filippov
Double precision floating point unit is a FPU implementation different from the FPU2000 in the following ways: - it may be configured with only single or with both single and double precision operations support; - it may be configured with division and square root opcodes; - FSR register accumula

[PATCH v4 08/22] target/xtensa: don't access BR regfile directly

2020-07-11 Thread Max Filippov
BR registers used in FPU comparison opcodes are available as opcode arguments for translators. Use them. This simplifies comparison helpers interface and makes them usable in FLIX bundles. Reviewed-by: Richard Henderson Signed-off-by: Max Filippov --- target/xtensa/fpu_helper.c | 42 +++

[PATCH v4 20/22] tests/tcg/xtensa: add DFP0 arithmetic tests

2020-07-11 Thread Max Filippov
Add test for basic double precision opcode properties. Signed-off-by: Max Filippov --- Changes v2->v3: - add more infzero tests for DFPU - fix test names in test_dfp0_arith.S tests/tcg/xtensa/test_dfp0_arith.S | 162 + 1 file changed, 162 insertions(+) create mode 1

[PATCH v4 12/22] tests/tcg/xtensa: fix test execution on ISS

2020-07-11 Thread Max Filippov
Space for test results may be allocated in IRAM which is only word-accessible. Use full 32-bit words to access test results. Signed-off-by: Max Filippov --- tests/tcg/xtensa/macros.inc | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/tcg/xtensa/macros.inc b/te

[PATCH v4 10/22] target/xtensa: add DFPU registers and opcodes

2020-07-11 Thread Max Filippov
DFPU may be configured with 32-bit or with 64-bit registers. Xtensa ISA does not specify how single-precision values are stored in 64-bit registers. Existing implementations store them in the low half of the registers. Add value extraction and write back to single-precision opcodes. Add new double

[PATCH v4 14/22] tests/tcg/xtensa: expand madd tests

2020-07-11 Thread Max Filippov
Test that madd doesn't do rounding after multiplication. Test NaN propagation rules for FPU2000 and DFPU madd opcode. Signed-off-by: Max Filippov --- Changes v2->v3: - add more infzero tests for FPU2000 and DFPU tests/tcg/xtensa/test_fp0_arith.S | 104 ++ 1 file chan

[PATCH v4 13/22] tests/tcg/xtensa: update test_fp0_arith for DFPU

2020-07-11 Thread Max Filippov
DFPU arithmetic opcodes update FSR flags. Add FSR parameters and expected FSR register values for the arithmetic tests. Signed-off-by: Max Filippov --- tests/tcg/xtensa/fpu.h| 142 tests/tcg/xtensa/test_fp0_arith.S | 178 ++ 2 file

[PATCH v4 16/22] tests/tcg/xtensa: update test_fp1 for DFPU

2020-07-11 Thread Max Filippov
DFPU sets Invalid flag in FSR when at least one argument of FP comparison opcodes is NaN, SNaN for most opcodes, any NaN for olt/ole. Add checks for FSR and expected FSR values. Signed-off-by: Max Filippov --- tests/tcg/xtensa/test_fp1.S | 62 - 1 file changed

[PATCH v4 18/22] tests/tcg/xtensa: add fp0 div and sqrt tests

2020-07-11 Thread Max Filippov
Test exact division/sqrt DFPU sequences. Signed-off-by: Max Filippov --- tests/tcg/xtensa/test_fp0_div.S | 82 tests/tcg/xtensa/test_fp0_sqrt.S | 76 + 2 files changed, 158 insertions(+) create mode 100644 tests/tcg/xtensa/test_fp0_d

[PATCH v4 15/22] tests/tcg/xtensa: update test_fp0_conv for DFPU

2020-07-11 Thread Max Filippov
DFPU conversion opcodes update FSR flags. Add FSR parameters and expected FSR register values for the conversion tests. Signed-off-by: Max Filippov --- tests/tcg/xtensa/test_fp0_conv.S | 299 --- 1 file changed, 155 insertions(+), 144 deletions(-) diff --git a/tests/

[PATCH v4 17/22] tests/tcg/xtensa: update test_lsc for DFPU

2020-07-11 Thread Max Filippov
DFPU doesn't have pre-increment FP load/store opcodes, it has post-increment opcodes instead. Test increment opcodes present in the current config. Signed-off-by: Max Filippov --- tests/tcg/xtensa/test_lsc.S | 47 +++-- 1 file changed, 34 insertions(+), 13 deletio

[PATCH v4 19/22] tests/tcg/xtensa: test double precision load/store

2020-07-11 Thread Max Filippov
Add ldi[p]/sdi[p]/ldx[p]/sdx[p] opcode tests to test_lsc. Signed-off-by: Max Filippov --- tests/tcg/xtensa/test_lsc.S | 123 1 file changed, 123 insertions(+) diff --git a/tests/tcg/xtensa/test_lsc.S b/tests/tcg/xtensa/test_lsc.S index 9d59c1815a9e..348822bd

[Bug 1886318] Re: Qemu after v5.0.0 breaks macos guests

2020-07-11 Thread Simon John
qemu console screenshot, this is as far as it gets after clover: https://i.imgur.com/HWY96Kq.png same result with or without usb/pci passthrough, qxl/vnc, git master HEAD or debian 5.0-6 -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU

Re: [PATCH 3/3] cpu-timers, icount: new modules

2020-07-11 Thread Claudio Fontana
On 7/10/20 8:33 AM, Cornelia Huck wrote: > On Thu, 9 Jul 2020 20:46:56 +0200 > Claudio Fontana wrote: > >> On 7/9/20 8:38 PM, Claudio Fontana wrote: >>> On 7/8/20 5:05 PM, Paolo Bonzini wrote: On 08/07/20 17:00, Claudio Fontana wrote: >> Bisectable, 100% failure rate, etc. :( Can yo

Re: [PATCH 3/3] cpu-timers, icount: new modules

2020-07-11 Thread Claudio Fontana
On 7/11/20 11:39 AM, Paolo Bonzini wrote: > On 11/07/20 11:14, Claudio Fontana wrote: >> On 7/11/20 12:45 AM, Paolo Bonzini wrote: >>> On 10/07/20 06:36, Thomas Huth wrote: In short this goes away if I again set icount to enabled for qtest, basically ensuring that --enable-tcg is the

[Bug 1886318] Re: Qemu after v5.0.0 breaks macos guests

2020-07-11 Thread Mark Cave-Ayland
Indeed it is, but bear in mind it was QEMU 5.1 release feature freeze this week so most developers are busy rebasing and fixing up bugs from the resulting merge. Given that you have already built QEMU from source, what would help enormously is if you can do a "git bisect" between the v5.0.0 tag (w

[Bug 1885332] Re: Error in user-mode calculation of ELF aux vector's AT_PHDR

2020-07-11 Thread Dmitry
When I switch to armv7 the issue goes away $ cat Dockerfile.armv7 FROM arm32v7/ubuntu RUN apt-get update && \ apt-get install -y \ gcc make libpcre3-dev libreadline-dev git RUN cd /home && git clone https://github.com/nginx/njs RUN cd /home/njs && ./configure --cc-opt='-O0 -static -lm -

Re: [PATCH 3/3] cpu-timers, icount: new modules

2020-07-11 Thread Paolo Bonzini
On 11/07/20 13:49, Claudio Fontana wrote: >> Apart from the name, icount is more like deterministic execution than > > Maybe we should start choosing names more carefully in a way to express what > we mean? I don't disagree. For icount in particular however we're about 12 years too late. >> q

Re: [PATCH 3/3] cpu-timers, icount: new modules

2020-07-11 Thread Claudio Fontana
On 7/11/20 2:19 PM, Paolo Bonzini wrote: > On 11/07/20 13:49, Claudio Fontana wrote: >>> Apart from the name, icount is more like deterministic execution than >> >> Maybe we should start choosing names more carefully in a way to express what >> we mean? > > I don't disagree. For icount in partic

Re: [PULL 0/1] virtio: bugfix

2020-07-11 Thread Peter Maydell
On Fri, 10 Jul 2020 at 00:04, Michael S. Tsirkin wrote: > > Fixes a single bug in vdpa. > > The following changes since commit eb2c66b10efd2b914b56b20ae90655914310c925: > > Merge remote-tracking branch 'remotes/maxreitz/tags/pull-block-2020-07-06' > into staging (2020-07-07 19:47:26 +0100) > >

cve patch wanted

2020-07-11 Thread 林奕帆
Hello I am a student from Fudan University in China. I am doing research on CVE patch recently. But i can not find the PATCH COMMIT of CVE-2019-12247 cve-2019-12155 cve-2019-6778.Can you give me the commit fix this cve?

Re: [PATCH v2] scripts/simplebench: compare write request performance

2020-07-11 Thread Vladimir Sementsov-Ogievskiy
26.06.2020 17:31, Andrey Shinkevich wrote: The script 'bench_write_req.py' allows comparing performances of write request for two qemu-img binary files. An example with (qemu-img binary 1) and without (qemu-img binary 2) the applied patch "qcow2: skip writing zero buffers to empty COW areas" (git

Re: [PULL 0/2] xen queue 2020-07-10

2020-07-11 Thread Peter Maydell
On Fri, 10 Jul 2020 at 14:11, Anthony PERARD wrote: > > The following changes since commit b6d7e9b66f59ca6ebc6e9b830cd5e7bf849d31cf: > > Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-request' > into staging (2020-07-10 09:01:28 +0100) > > are available in the Git repository a

[PATCH for-5.1 3/3] qdev: Document GPIO related functions

2020-07-11 Thread Peter Maydell
Add documentation comments for the various qdev functions related to creating and connecting GPIO lines. Signed-off-by: Peter Maydell --- include/hw/qdev-core.h | 191 - 1 file changed, 189 insertions(+), 2 deletions(-) diff --git a/include/hw/qdev-core.h

[PATCH for-5.1 2/3] qdev: Document qdev_unrealize()

2020-07-11 Thread Peter Maydell
Add a doc comment for qdev_unrealize(), to go with the new documentation for the realize part of the qdev lifecycle. Signed-off-by: Peter Maydell --- include/hw/qdev-core.h | 19 +++ 1 file changed, 19 insertions(+) diff --git a/include/hw/qdev-core.h b/include/hw/qdev-core.h in

[PATCH for-5.1 0/3] Move and improve qdev API doc comments

2020-07-11 Thread Peter Maydell
This patchset adds documentation comments to qdev-core.h (and in a few cases qdev-properties.h) to functions (but not all functions!) which were missing them. The prompt for this was Markus' recent welcome cleanup and correction of the qdev realize functions to get reference count and bus parentin

[PATCH for-5.1 1/3] qdev: Move doc comments from qdev.c to qdev-core.h

2020-07-11 Thread Peter Maydell
The doc-comments which document the qdev API are split between the header file and the C source files, because as a project we haven't been consistent about where we put them. Move all the doc-comments in qdev.c to the header files, so that users of the APIs don't have to look at the implementatio

usb-wacom-tablet failing to register

2020-07-11 Thread Michael Nazzareno Trimarchi
Hi all On my 4.17.0-rc1 linux kernel i386 running on qemu, I can't register the wacom driver emulation QEMU emulator version 4.2.0 (Debian 1:4.2-3ubuntu6.3) Copyright (c) 2003-2019 Fabrice Bellard and the QEMU Project developers [0.395368] ata2.00: configured for MWDMA2 [0.397049] scsi 1:

Re: [PATCH] linux-user: Add several IFTUN ioctls

2020-07-11 Thread Laurent Vivier
Le 10/07/2020 à 21:53, Josh Kunz a écrit : > This change includes most widely-available if_tun ioctls that are > integer typed. > > Tested by compiling all linux-user emulators. This patch has also been > used successfully to run several binaries that utilize these ioctls for > several months. >

QEMU | Pipeline #165534352 has failed for master | 82793715

2020-07-11 Thread GitLab via
Your pipeline has failed. Project: QEMU ( https://gitlab.com/qemu-project/qemu ) Branch: master ( https://gitlab.com/qemu-project/qemu/-/commits/master ) Commit: 82793715 ( https://gitlab.com/qemu-project/qemu/-/commit/827937158b72ce2265841ff528bba3c44a1bfbc8 ) Commit Message: Merge remote-tr

[PATCH] hw/timer/renesas_tmr.c cleanup read operation.

2020-07-11 Thread Yoshinori Sato
Cleanup read operation. This module different return of access size. Signed-off-by: Yoshinori Sato --- hw/timer/renesas_tmr.c | 106 ++--- 1 file changed, 57 insertions(+), 49 deletions(-) diff --git a/hw/timer/renesas_tmr.c b/hw/timer/renesas_tmr.c index 446

Re: [PULL 0/1] 9p fixes for 5.1 2020-07-10

2020-07-11 Thread Peter Maydell
On Fri, 10 Jul 2020 at 17:35, Greg Kurz wrote: > > The following changes since commit b6d7e9b66f59ca6ebc6e9b830cd5e7bf849d31cf: > > Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-request' > into staging (2020-07-10 09:01:28 +0100) > > are available in the Git repository at: >

Re: [PATCH v2 2/2] hw/riscv: sifive_u: Provide a reliable way for bootloader to detect whether it is running in QEMU

2020-07-11 Thread Alistair Francis
On Thu, Jul 9, 2020 at 5:48 PM Bin Meng wrote: > > Hi Alistair, > > On Fri, Jul 10, 2020 at 6:19 AM Alistair Francis wrote: > > > > On Thu, Jul 9, 2020 at 3:07 AM Bin Meng wrote: > > > > > > From: Bin Meng > > > > > > The reset vector codes are subject to change, e.g.: with recent > > > fw_dyna

Re: [PATCH v2 2/2] hw/riscv: sifive_u: Provide a reliable way for bootloader to detect whether it is running in QEMU

2020-07-11 Thread Alistair Francis
On Thu, Jul 9, 2020 at 5:50 PM Bin Meng wrote: > > Hi Palmer, > > On Fri, Jul 10, 2020 at 8:45 AM Palmer Dabbelt > wrote: > > > > On Thu, 09 Jul 2020 15:09:18 PDT (-0700), alistai...@gmail.com wrote: > > > On Thu, Jul 9, 2020 at 3:07 AM Bin Meng wrote: > > >> > > >> From: Bin Meng > > >> > > >

Re: [PULL 10/41] python/qemu: Add ConsoleSocket for optional use in QEMUMachine

2020-07-11 Thread Robert Foley
Hi, Thanks for the detailed feedback! I will look at making these changes. On Fri, 10 Jul 2020 at 15:20, John Snow wrote: > > > > On 7/7/20 3:08 AM, Alex Bennée wrote: > > From: Robert Foley > > > > > +def recv(self, n=1, sleep_delay_s=0.1): > > +"""Return chars from in memory buff

[PATCH 04/11] riscv: Add RV64F instructions description

2020-07-11 Thread LIU Zhiwei
For supporting multi-precision, split all 32 fp registers into two groups. The RV64F instructions will use only 16 fp registers selected by gfp32(). Signed-off-by: LIU Zhiwei --- rv64.risu | 94 +++ 1 file changed, 94 insertions(+) diff --git

[PATCH 00/11] RISC-V risu porting

2020-07-11 Thread LIU Zhiwei
In contrast to the RFC, add more instructions description. Now it supports RV64IMACFD. Some cross verifications have been done, such as comparison between QEMU and TinyEMU, and comparison between QEMU and C906 FPGA. Now it has some productive. Features: * support RV64IMACFD. * support multi-preci

[PATCH 10/11] riscv: Implement payload load interfaces

2020-07-11 Thread LIU Zhiwei
When a risu op emits, the signal handler wll take over execution before running the payload again. The signal handler need some interfaces, such as setting struct reginfo and the comparison of struct reginfo. Signed-off-by: LIU Zhiwei --- risu_reginfo_riscv64.c | 132 +++

[PATCH 06/11] riscv: Add RV64C instructions description

2020-07-11 Thread LIU Zhiwei
Make it a separate file, so that we can get subarch to recgonize the instrcution length. Signed-off-by: LIU Zhiwei --- rv64c.risu | 97 ++ 1 file changed, 97 insertions(+) create mode 100644 rv64c.risu diff --git a/rv64c.risu b/rv64c.risu new

[PATCH 11/11] riscv: Add configure script

2020-07-11 Thread LIU Zhiwei
For RV64 risu, make CFLAGS="-march=rv64g" Signed-off-by: LIU Zhiwei --- configure | 4 +- upstream/configure | 204 + 2 files changed, 207 insertions(+), 1 deletion(-) create mode 100644 upstream/configure diff --git a/configure b/configur

[PATCH 07/11] riscv: Generate payload scripts

2020-07-11 Thread LIU Zhiwei
In general, generate payload. 1. Setup memory for load/store instructions. 2. Initialize the registers. 3. Emit instructions according to instruction format description. Specially, modify according to RISC-V ISA. 1. Support multi-precision by dividing fp registers into two groups. 2. Use $bytecoun

[PATCH 03/11] riscv: Add RV64A instructions description

2020-07-11 Thread LIU Zhiwei
Ensure $rs2 != $rs1, so that the $rs2 register's value will not be covered when setting the $rs1 register's value to get a valid address. Signed-off-by: LIU Zhiwei --- rv64.risu | 90 +++ 1 file changed, 90 insertions(+) diff --git a/rv64.risu

[PATCH 02/11] riscv: Add RV64M instructions description

2020-07-11 Thread LIU Zhiwei
Signed-off-by: LIU Zhiwei --- rv64.risu | 41 + 1 file changed, 41 insertions(+) diff --git a/rv64.risu b/rv64.risu index edf0d1f..2c4154e 100644 --- a/rv64.risu +++ b/rv64.risu @@ -139,3 +139,44 @@ SRLW RISCV 000 rs2:5 rs1:5 101 rd:5 0011011 \ SRAW

[PATCH 01/11] riscv: Add RV64I instructions description

2020-07-11 Thread LIU Zhiwei
Avoid using stack pointer(x2), thread pointer(x3), global pointer(x4), as they are not under control of risu. Besides, avoid using x0 as base address register, because we can't emit a valid random address by setting x0. Signed-off-by: LIU Zhiwei --- rv64.risu | 141 ++

[PATCH 09/11] riscv: Define riscv struct reginfo

2020-07-11 Thread LIU Zhiwei
Signed-off-by: LIU Zhiwei --- risu_reginfo_riscv64.h | 28 1 file changed, 28 insertions(+) create mode 100644 risu_reginfo_riscv64.h diff --git a/risu_reginfo_riscv64.h b/risu_reginfo_riscv64.h new file mode 100644 index 000..4536480 --- /dev/null +++ b/risu_re

[PATCH 08/11] riscv: Add standard test case

2020-07-11 Thread LIU Zhiwei
Signed-off-by: LIU Zhiwei Reviewed-by: Richard Henderson --- test_riscv64.s | 85 ++ 1 file changed, 85 insertions(+) create mode 100644 test_riscv64.s diff --git a/test_riscv64.s b/test_riscv64.s new file mode 100644 index 000..22a22b6 --- /

[PATCH 05/11] riscv: Add RV64D instructions description

2020-07-11 Thread LIU Zhiwei
For supporting multi-precison, split all 32 fp registers into two groups. The RV64D instructions will use only the 16 fp registers selected by gfp64(). Signed-off-by: LIU Zhiwei --- rv64.risu | 100 ++ 1 file changed, 100 insertions(+) diff --

Re: [PATCH v8 03/10] qcow2_format.py: change Qcow2BitmapExt initialization method

2020-07-11 Thread Vladimir Sementsov-Ogievskiy
03.07.2020 16:13, Andrey Shinkevich wrote: There are two ways to initialize a class derived from Qcow2Struct: 1. Pass a block of binary data to the constructor. 2. Pass the file descriptor to allow reading the file from constructor. Let's change the Qcow2BitmapExt initialization method from 1 to

Re: [PATCH v8 04/10] qcow2_format.py: dump bitmap flags in human readable way.

2020-07-11 Thread Vladimir Sementsov-Ogievskiy
03.07.2020 16:13, Andrey Shinkevich wrote: Introduce the class BitmapFlags that parses a bitmap flags mask. Suggested-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/qcow2_format.py | 16 1 file changed, 16 insertions(+) diff --git

QEMU | Pipeline #165544924 has failed for master | d4a6bab1

2020-07-11 Thread GitLab via
Your pipeline has failed. Project: QEMU ( https://gitlab.com/qemu-project/qemu ) Branch: master ( https://gitlab.com/qemu-project/qemu/-/commits/master ) Commit: d4a6bab1 ( https://gitlab.com/qemu-project/qemu/-/commit/d4a6bab1ecf23d86e20e2f08a144e4e48643a6e4 ) Commit Message: Merge remote-tr

[PULL v2 00/50] testing updates (vm, gitlab, misc build fixes)

2020-07-11 Thread Alex Bennée
Fixed a few, dropped a few, added a few --- The following changes since commit 827937158b72ce2265841ff528bba3c44a1bfbc8: Merge remote-tracking branch 'remotes/aperard/tags/pull-xen-20200710' into staging (2020-07-11 13:56:03 +0100) are available in the Git repository at: https://github.co

[PULL v2 43/50] docs/devel: add some notes on tcg-icount for developers

2020-07-11 Thread Alex Bennée
This attempts to bring together my understanding of the requirements for icount behaviour into one reference document for our developer notes. Signed-off-by: Alex Bennée Reviewed-by: Richard Henderson Cc: Paolo Bonzini Cc: Pavel Dovgalyuk Cc: Peter Maydell Message-Id: <20200709141327.14631-3-

[PULL v2 47/50] .cirrus.yml: add bash to the brew packages

2020-07-11 Thread Alex Bennée
Like the sed we include earlier we want something more recent for iotests to work. Fixes: 57ee95ed Cc: Max Reitz Signed-off-by: Alex Bennée Message-Id: <20200710182238.10675-1-alex.ben...@linaro.org> diff --git a/.cirrus.yml b/.cirrus.yml index 69342ae031..f287d23c5b 100644 --- a/.cirrus.yml ++

[PULL v2 04/50] tests/vm: Add configuration to basevm.py

2020-07-11 Thread Alex Bennée
From: Robert Foley Added use of a configuration to tests/vm/basevm.py. The configuration provides parameters used to configure a VM. This allows for providing alternate configurations to the VM being created/launched. cpu, machine, memory, and NUMA configuration are all examples of configuration

[PULL v2 44/50] docs: Add to gdbstub documentation the PhyMemMode

2020-07-11 Thread Alex Bennée
From: Jon Doron The PhyMemMode gdb extension command was missing from the gdb.rst document. Signed-off-by: Jon Doron Signed-off-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20200601171609.1665397-1-ari...@gmail.com> Message-Id: <20200709141327.14631-4-alex.ben...@linaro.or

[PULL v2 42/50] docs/devel: convert and update MTTCG design document

2020-07-11 Thread Alex Bennée
Do a light conversion to .rst and clean-up some of the language at the start now MTTCG has been merged for a while. Signed-off-by: Alex Bennée Reviewed-by: Richard Henderson Message-Id: <20200709141327.14631-2-alex.ben...@linaro.org> diff --git a/docs/devel/index.rst b/docs/devel/index.rst inde

[PULL v2 48/50] tests: fix "make check-qtest" for modular builds

2020-07-11 Thread Alex Bennée
From: Gerd Hoffmann Signed-off-by: Gerd Hoffmann Signed-off-by: Alex Bennée Message-Id: <20200710203652.9708-2-kra...@redhat.com> diff --git a/tests/qtest/Makefile.include b/tests/qtest/Makefile.include index 98af2c2d93..6a0276fd42 100644 --- a/tests/qtest/Makefile.include +++ b/tests/qtest/Ma

[PULL v2 45/50] tests/docker: fall back more gracefully when pull fails

2020-07-11 Thread Alex Bennée
I only spotted this in the small window between my testing with my registry while waiting for the gitlab PR to go in. As we pre-pull the registry image we know if that fails there isn't any point attempting to use the cache. Fall back to the way we used to do it at that point. Signed-off-by: Alex

[PULL v2 50/50] iotests: Set LC_ALL=C for sort

2020-07-11 Thread Alex Bennée
From: Max Reitz Otherwise the result is basically unpredictable. (Note that the precise environment variable to control sorting order is LC_COLLATE, but LC_ALL overrides LC_COLLATE, and we do not want the sorting order to be messed up if LC_ALL is set in the environment.) Reported-by: John Snow

[PULL v2 46/50] tests/docker: update toolchain set in debian-xtensa-cross

2020-07-11 Thread Alex Bennée
From: Max Filippov Switch to the prebuilt xtensa toolchains release 2020.07. Drop csp toolchain as the csp core is not a part of QEMU. Add de233_fpu and dsp3400 toolchains to enable DFPU and FPU2000 tests. Signed-off-by: Max Filippov Signed-off-by: Alex Bennée Reviewed-by: Philippe Mathieu-Dau

[PULL v2 49/50] Revert "vga: build virtio-gpu as module"

2020-07-11 Thread Alex Bennée
From: Gerd Hoffmann This reverts commit 8d5a24c83dba90b08ef163bbf166d6dfbad9019b. Compiling all virtio-gpu objects into a single module isn't a good plan because the individual objects have different CONFIG_* dependencies. Leads to module load failures on s390x due to vga support being disabled,

Re: [PATCH v4 7/7] Makefile: Ship the generic platform bios images for RISC-V

2020-07-11 Thread Alistair Francis
On Fri, Jul 10, 2020 at 11:36 AM Alistair Francis wrote: > > On Thu, Jul 9, 2020 at 10:11 PM Bin Meng wrote: > > > > From: Bin Meng > > > > Update the install blob list to include the generic platform > > fw_dynamic bios images. > > > > Signed-off-by: Bin Meng > > You didn't address the comment

Re: [PATCH v4 4/7] hw/riscv: Use pre-built bios image of generic platform for virt & sifive_u

2020-07-11 Thread Alistair Francis
On Thu, Jul 9, 2020 at 10:07 PM Bin Meng wrote: > > From: Bin Meng > > Update virt and sifive_u machines to use the opensbi fw_dynamic bios > image built for the generic FDT platform. > > Remove the out-of-date no longer used bios images. > > Signed-off-by: Bin Meng > Reviewed-by: Anup Patel >

Re: [PULL 10/41] python/qemu: Add ConsoleSocket for optional use in QEMUMachine

2020-07-11 Thread Alex Bennée
Robert Foley writes: > Hi, > Thanks for the detailed feedback! I will look at making these > changes. In the interest of getting the CI green I've submitted v2 as is but I'll roll up Robert's cleanups in my rc0 series (which is hopefully a lot smaller!). -- Alex Bennée

Re: [PATCH] cpu: Add starts_halted() method

2020-07-11 Thread Alex Bennée
Thiago Jung Bauermann writes: > Alex Bennée writes: > >> Thiago Jung Bauermann writes: >> >>> Eduardo Habkost writes: >>> On Wed, Jul 08, 2020 at 09:11:55PM +0100, Peter Maydell wrote: > On Wed, 8 Jul 2020 at 18:36, Eduardo Habkost wrote: > > > > On Wed, Jul 08, 2020 at 06:

Re: [PULL v3 00/47] Misc patches for QEMU 5.1 soft freeze

2020-07-11 Thread Peter Maydell
On Sat, 11 Jul 2020 at 00:50, Paolo Bonzini wrote: > > The following changes since commit 45db94cc90c286a9965a285ba19450f448760a09: > > Merge remote-tracking branch 'remotes/mcayland/tags/qemu-openbios-20200707' > into staging (2020-07-10 16:43:40 +0100) > > are available in the Git repository

Re: [PATCH 0/2] hw/arm/palm.c: Fix Coverity issue CID 1421944

2020-07-11 Thread Peter Maydell
On Sun, 28 Jun 2020 at 22:42, Peter Maydell wrote: > > As for spitz and tosa, fix the Coverity issue CID 1421944 which > points out that memory returned from qemu_allocate_irqs() is leaked > by encapsulating the GPIO handling into a simple device. > As with the other series, detabify the file firs

[Bug 1681439] Re: qemu-system-x86_64: hw/ide/core.c:685: ide_cancel_dma_sync: Assertion `s->bus->dma->aiocb == NULL' failed.

2020-07-11 Thread Alexander Bulekov
Here's a qtest reproducer cat << EOF | ./i386-softmmu/qemu-system-i386 \ -M pc,accel=qtest -qtest null -nographic -vga qxl -qtest stdio -nodefaults \ -drive if=none,id=drive0,file=null-co://,file.read-zeroes=on,format=raw \ -drive if=none,id=drive1,file=null-co://,file.read-zeroes=on,format=raw \

Re: [PATCH v3 0/9] Build fixes for Haiku

2020-07-11 Thread Peter Maydell
On Fri, 3 Jul 2020 at 15:56, Peter Maydell wrote: > > This patchset is essentially a resend of David Carlier's build fixes > for the Haiku platform. I've taken David's patches and put them together > into a set of emails threaded in the way our CI tools expect, as the > easiest way to get the patc

Re: [PATCH v2 1/4] target/nios2: add DISAS_NORETURN case for nothing more to generate

2020-07-11 Thread Peter Maydell
On Fri, 10 Jul 2020 at 16:46, Wentong Wu wrote: > > Add DISAS_NORETURN case for nothing more to generate because at runtime > execution will never return from some helper call. And at the same time > replace DISAS_UPDATE in t_gen_helper_raise_exception and gen_exception > with the newly added DISA

Re: [PATCH v3 0/9] Build fixes for Haiku

2020-07-11 Thread David CARLIER
Alright thanks for the assistance. Regards. On Sat, 11 Jul 2020 at 19:44, Peter Maydell wrote: > On Fri, 3 Jul 2020 at 15:56, Peter Maydell > wrote: > > > > This patchset is essentially a resend of David Carlier's build fixes > > for the Haiku platform. I've taken David's patches and put them t

Re: [PATCH v8 05/10] qcow2_format.py: Dump bitmap directory information

2020-07-11 Thread Vladimir Sementsov-Ogievskiy
03.07.2020 16:13, Andrey Shinkevich wrote: Read and dump entries from the bitmap directory of QCOW2 image. It extends the output in the test case #291. Header extension: magic 0x23852875 (Bitmaps) ... Bitmap name bitmap-1 bitmap_table_offset 0xf bitma

[Bug 1777315] Re: Denial of service

2020-07-11 Thread Alexander Bulekov
Here's a qtest reproducer: ./i386-softmmu/qemu-system-i386 -M pc,accel=qtest \ -qtest null -nographic -vga qxl -qtest stdio \ -drive if=none,id=drive0,file=null-co://,file.read-zeroes=on,format=raw \ -drive if=none,id=drive1,file=null-co://,file.read-zeroes=on,format=raw \ -device ide-cd,drive=dri

Re: [PATCH v2 07/17] linux-user: Fix guest_addr_valid vs reserved_va

2020-07-11 Thread Richard Henderson
On 6/25/20 9:37 AM, Peter Maydell wrote: > On Fri, 5 Jun 2020 at 05:17, Richard Henderson > wrote: >> >> We must always use GUEST_ADDR_MAX, because even 32-bit hosts can >> use -R to restrict the memory address of the guest. >> >> Signed-off-by: Richard Henderson >> --- >> include/exec/cpu_ldst

Re: [PATCH v2 06/17] linux-user: Do not use guest_addr_valid for h2g_valid

2020-07-11 Thread Richard Henderson
On 6/25/20 9:34 AM, Peter Maydell wrote: > On Fri, 5 Jun 2020 at 05:17, Richard Henderson > wrote: >> >> This is the only use of guest_addr_valid that does not begin >> with a guest address, but a host address being transformed to >> a guest address. >> >> We will shortly adjust guest_addr_valid t

  1   2   >