Re: [for-5.2 v4 10/10] s390: Recognize host-trust-limitation option

2020-09-10 Thread Greg Kurz
On Fri, 11 Sep 2020 10:07:18 +1000 David Gibson wrote: > On Thu, Sep 10, 2020 at 08:29:24PM +0200, Halil Pasic wrote: > > On Thu, 10 Sep 2020 13:36:09 +0200 > > Cornelia Huck wrote: > > > > > On Mon, 7 Sep 2020 17:22:53 +0200 > > > Halil Pasic wrote: > > > > > > > On Fri, 24 Jul 2020 12:57:44

Re: [PATCH v2 1/2] Introduce (x86) CPU model deprecation API

2020-09-10 Thread Robert Hoo
On Wed, 2020-09-09 at 14:15 -0400, Eduardo Habkost wrote: > Hi, > > Thanks for the patch, and sorry for taking so long to review > this. I'm finally getting to the patches that were postponed to > 5.2. > > Comments and questions below: > > On Thu, Jun 11, 2020 at 10:47:55AM +0800, Robert Hoo wr

Re: [PATCH 07/14] block/blklogwrites: drop error propagation

2020-09-10 Thread Greg Kurz
On Fri, 11 Sep 2020 07:30:37 +0200 Markus Armbruster wrote: > Markus Armbruster writes: > > > Greg Kurz writes: > > > >> On Wed, 9 Sep 2020 21:59:23 +0300 > >> Vladimir Sementsov-Ogievskiy wrote: > >> > >>> It's simple to avoid error propagation in blk_log_writes_open(), we > >>> just need t

Re: [Bug 1895080] [NEW] pgb_reserved_va: Assertion `addr == test' failed

2020-09-10 Thread Hansni Bu
> This problem occurs on CentOS-7.5 (64-bit) with qemu-5.1.0, qemu head > > (commit 9435a8b3dd35f1f926f1b9127e8a906217a5518a) for riscv32-linux- > > user. > > I tried to build qemu-5.1 on CentOS-7.5.1 but as python 3.5 is not > available, I gave up. > Thank you for your effort. I installed python3

Re: [PATCH 07/14] block/blklogwrites: drop error propagation

2020-09-10 Thread Markus Armbruster
Markus Armbruster writes: > Greg Kurz writes: > >> On Wed, 9 Sep 2020 21:59:23 +0300 >> Vladimir Sementsov-Ogievskiy wrote: >> >>> It's simple to avoid error propagation in blk_log_writes_open(), we >>> just need to refactor blk_log_writes_find_cur_log_sector() a bit. >>> >>> Signed-off-by: V

[PATCH v9 14/14] tests/acceptance: console boot tests for quanta-gsj

2020-09-10 Thread Havard Skinnemoen via
This adds two acceptance tests for the quanta-gsj machine. One test downloads a lightly patched openbmc flash image from github and verifies that it boots all the way to the login prompt. The other test downloads a kernel, initrd and dtb built from the same openbmc source and verifies that the ke

[PATCH v9 11/14] hw/arm: Wire up BMC boot flash for npcm750-evb and quanta-gsj

2020-09-10 Thread Havard Skinnemoen via
This allows these NPCM7xx-based boards to boot from a flash image, e.g. one built with OpenBMC. For example like this: IMAGE=${OPENBMC}/build/tmp/deploy/images/gsj/image-bmc qemu-system-arm -machine quanta-gsj -nographic \ -drive file=${IMAGE},if=mtd,bus=0,unit=0,format=raw,snapshot=on Re

[PATCH v9 10/14] hw/ssi: NPCM7xx Flash Interface Unit device model

2020-09-10 Thread Havard Skinnemoen via
This implements a device model for the NPCM7xx SPI flash controller. Direct reads and writes, and user-mode transactions have been tested in various modes. Protection features are not implemented yet. All the FIU instances are available in the SoC's address space, regardless of whether or not the

[PATCH v9 08/14] hw/nvram: NPCM7xx OTP device model

2020-09-10 Thread Havard Skinnemoen via
This supports reading and writing OTP fuses and keys. Only fuse reading has been tested. Protection is not implemented. Reviewed-by: Avi Fishman Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Tested-by: Alexander Bulekov Signed-off-by: Havard Skinnemoen --- include/hw/

Re: [PATCH 07/14] block/blklogwrites: drop error propagation

2020-09-10 Thread Markus Armbruster
Greg Kurz writes: > On Wed, 9 Sep 2020 21:59:23 +0300 > Vladimir Sementsov-Ogievskiy wrote: > >> It's simple to avoid error propagation in blk_log_writes_open(), we >> just need to refactor blk_log_writes_find_cur_log_sector() a bit. >> >> Signed-off-by: Vladimir Sementsov-Ogievskiy >> --- >>

[PATCH v9 05/14] hw/arm: Add two NPCM7xx-based machines

2020-09-10 Thread Havard Skinnemoen via
This adds two new machines, both supported by OpenBMC: - npcm750-evb: Nuvoton NPCM750 Evaluation Board. - quanta-gsj: A board with a NPCM730 chip. They rely on the NPCM7xx SoC device to do the heavy lifting. They are almost completely identical at the moment, apart from the SoC type, which cu

[PATCH v9 07/14] hw/arm: Load -bios image as a boot ROM for npcm7xx

2020-09-10 Thread Havard Skinnemoen via
If a -bios option is specified on the command line, load the image into the internal ROM memory region, which contains the first instructions run by the CPU after reset. If -bios is not specified, the vbootrom included with qemu is loaded by default. Reviewed-by: Tyrone Ting Reviewed-by: Cédric

[PATCH v9 12/14] hw/arm/npcm7xx: add board setup stub for CPU and UART clocks

2020-09-10 Thread Havard Skinnemoen via
When booting directly into a kernel, bypassing the boot loader, the CPU and UART clocks are not set up correctly. This makes the system appear very slow, and causes the initrd boot test to fail when optimization is off. The UART clock must run at 24 MHz. The default 25 MHz reference clock cannot a

[PATCH v9 13/14] docs/system: Add Nuvoton machine documentation

2020-09-10 Thread Havard Skinnemoen via
Reviewed-by: Cédric Le Goater Reviewed-by: Philippe Mathieu-Daudé Signed-off-by: Havard Skinnemoen --- docs/system/arm/nuvoton.rst | 92 + docs/system/target-arm.rst | 1 + 2 files changed, 93 insertions(+) create mode 100644 docs/system/arm/nuvoton.rst d

[PATCH v9 09/14] hw/mem: Stubbed out NPCM7xx Memory Controller model

2020-09-10 Thread Havard Skinnemoen via
This just implements the bare minimum to cause the boot block to skip memory initialization. Reviewed-by: Tyrone Ting Reviewed-by: Cédric Le Goater Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Tested-by: Alexander Bulekov Signed-off-by: Havard Skinnemoen --- include

[PATCH v9 04/14] hw/arm: Add NPCM730 and NPCM750 SoC models

2020-09-10 Thread Havard Skinnemoen via
The Nuvoton NPCM7xx SoC family are used to implement Baseboard Management Controllers in servers. While the family includes four SoCs, this patch implements limited support for two of them: NPCM730 (targeted for Data Center applications) and NPCM750 (targeted for Enterprise applications). This pat

[PATCH v9 03/14] hw/timer: Add NPCM7xx Timer device model

2020-09-10 Thread Havard Skinnemoen via
The NPCM730 and NPCM750 SoCs have three timer modules each holding five timers and some shared registers (e.g. interrupt status). Each timer runs at 25 MHz divided by a prescaler, and counts down from a configurable initial value to zero. When zero is reached, the interrupt flag for the timer is s

[PATCH v9 06/14] roms: Add virtual Boot ROM for NPCM7xx SoCs

2020-09-10 Thread Havard Skinnemoen via
This is a minimalistic boot ROM written specifically for use with QEMU. It supports loading the second-stage loader from SPI flash into RAM, SMP boot, and not much else. Reviewed-by: Philippe Mathieu-Daudé Tested-by: Philippe Mathieu-Daudé Signed-off-by: Havard Skinnemoen --- .gitmodules

[PATCH v9 00/14] Add Nuvoton NPCM730/NPCM750 SoCs and two BMC machines

2020-09-10 Thread Havard Skinnemoen via
This should be fully reviewed now, but the Timer patch may deserve another look, as I fixed a few bugs in it. Huge thanks to everyone who reviewed and/or tested this patchset, it has clearly improved a lot since I started. I also pushed this and the previous patchsets to my qemu fork on github. T

[PATCH v9 02/14] hw/misc: Add NPCM7xx Clock Controller device model

2020-09-10 Thread Havard Skinnemoen via
Enough functionality to boot the Linux kernel has been implemented. This includes: - Correct power-on reset values so the various clock rates can be accurately calculated. - Clock enables stick around when written. In addition, a best effort attempt to implement SECCNT and CNTR25M was mad

[PATCH v9 01/14] hw/misc: Add NPCM7xx System Global Control Registers device model

2020-09-10 Thread Havard Skinnemoen via
Implement a device model for the System Global Control Registers in the NPCM730 and NPCM750 BMC SoCs. This is primarily used to enable SMP boot (the boot ROM spins reading the SCRPAD register) and DDR memory initialization; other registers are best effort for now. The reset values of the MDLR and

Re: [PATCH] trace/simple: Enable tracing on startup only if the user specifies a trace option

2020-09-10 Thread Markus Armbruster
Markus Armbruster writes: > dubo...@gmail.com writes: > >> From: Josh DuBois >> >> Tracing can be enabled at the command line or via the >> monitor. Command-line trace options are recorded during >> trace_opt_parse(), but tracing is not enabled until the various >> front-ends later call trace_in

Re: [PATCH v5 5/8] s390/sclp: use cpu offset to locate cpu entries

2020-09-10 Thread Thomas Huth
On 10/09/2020 11.36, Collin Walling wrote: > The start of the CPU entry region in the Read SCP Info response data is > denoted by the offset_cpu field. As such, QEMU needs to begin creating > entries at this address. > > This is in preparation for when Read SCP Info inevitably introduces new > byt

[PATCH] spapr: Handle HPT allocation failure in nested guest

2020-09-10 Thread Fabiano Rosas
The nested KVM code does not yet support HPT guests. Calling the KVM_CAP_PPC_ALLOC_HTAB ioctl currently leads to KVM setting the guest as HPT and erroneously executing code in L1 that should only run in hypervisor mode, leading to an exception in the L1 vcpu thread when it enters the nested guest.

Re: elf2dmp: Fix memory leak on main() error paths

2020-09-10 Thread Thomas Huth
On 26/08/2020 12.15, AlexChen wrote: > From: AlexChen > > The 'kdgb' is allocating memory in get_kdbg(), but it is not freed > in both fill_header() and fill_context() failed branches, fix it. > > Signed-off-by: AlexChen > --- > contrib/elf2dmp/main.c | 4 ++-- > 1 file changed, 2 insertions(+

Re: [for-5.2 v4 09/10] host trust limitation: Alter virtio default properties for protected guests

2020-09-10 Thread David Gibson
On Mon, Sep 07, 2020 at 05:10:46PM +0200, Halil Pasic wrote: > On Fri, 24 Jul 2020 12:57:43 +1000 > David Gibson wrote: > > > The default behaviour for virtio devices is not to use the platforms normal > > DMA paths, but instead to use the fact that it's running in a hypervisor > > to directly ac

[Bug 1895219] [NEW] qemu git -vnc fails due to missing en-us keymap

2020-09-10 Thread Darren Blaber
Public bug reported: If trying to run qemu with -vnc :0, it will fail with: ./qemu-system-x86_64 -vnc :2 qemu-system-x86_64: -vnc :2: could not read keymap file: 'en-us' share/keymaps is missing en-us keymap and only has sl and sv, confirmed previous stable versions had en-us. Tried with multipl

[PATCH 1/1] accel/tcg/user-exec: support computing is_write for mips32

2020-09-10 Thread zou xu
>From 533ed02427bdaf0265f62fcb4f961854a41b7037 Mon Sep 17 00:00:00 2001 From: ZouXu Date: Wed, 9 Sep 2020 21:59:13 +0800 Subject: [PATCH 1/1] accel/tcg/user-exec: support computing is_write for mips32 Those MIPS32 instructions can cause store operation: SB/SH/SW/SD/SWL/SWR/SDL/SDR/CACHE SC/SCD/S

Re: device compatibility interface for live migration with assigned devices

2020-09-10 Thread Yan Zhao
On Thu, Sep 10, 2020 at 12:02:44PM -0600, Alex Williamson wrote: > On Thu, 10 Sep 2020 13:50:11 +0100 > Sean Mooney wrote: > > > On Thu, 2020-09-10 at 14:38 +0200, Cornelia Huck wrote: > > > On Wed, 9 Sep 2020 10:13:09 +0800 > > > Yan Zhao wrote: > > > > > > > > > still, I'd like to put it mo

Re: [for-5.2 v4 10/10] s390: Recognize host-trust-limitation option

2020-09-10 Thread David Gibson
On Thu, Sep 10, 2020 at 08:29:24PM +0200, Halil Pasic wrote: > On Thu, 10 Sep 2020 13:36:09 +0200 > Cornelia Huck wrote: > > > On Mon, 7 Sep 2020 17:22:53 +0200 > > Halil Pasic wrote: > > > > > On Fri, 24 Jul 2020 12:57:44 +1000 > > > David Gibson wrote: > > > > > > > At least some s390 cpu m

fixup! docs: Move object.h overview doc comment to qom.rst

2020-09-10 Thread Eduardo Habkost
On Thu, Sep 10, 2020 at 06:15:26PM -0400, Eduardo Habkost wrote: > Move the whole contents of the overview doc comment from object.h > to qom.rst. > > This makes the documentation source easier to read and edit, and > also solves the backslash escaping issue at the typecasting macro > examples. >

Re: [PATCH v8 00/14] Add Nuvoton NPCM730/NPCM750 SoCs and two BMC machines

2020-09-10 Thread Havard Skinnemoen
On Tue, Sep 8, 2020 at 6:32 PM Havard Skinnemoen wrote: > On Tue, Sep 8, 2020 at 12:52 PM Havard Skinnemoen > wrote: > > > > On Tue, Sep 8, 2020 at 9:58 AM Philippe Mathieu-Daudé > wrote: > > > > > > On 9/8/20 5:52 PM, Philippe Mathieu-Daudé wrote: > > > > On 9/8/20 5:02 PM, Alexander Bulekov w

Re: [PATCH v8 07/14] hw/arm: Load -bios image as a boot ROM for npcm7xx

2020-09-10 Thread Havard Skinnemoen
On Thu, Sep 3, 2020 at 11:59 AM Philippe Mathieu-Daudé wrote: > On 8/25/20 2:17 AM, Havard Skinnemoen via wrote: > > If a -bios option is specified on the command line, load the image into > > the internal ROM memory region, which contains the first instructions > > run by the CPU after reset. >

Re: [PATCH v4 1/7] usb/hcd-xhci: Make dma read/writes hooks pci free

2020-09-10 Thread David Gibson
On Tue, Sep 08, 2020 at 12:34:59PM +0200, Philippe Mathieu-Daudé wrote: > On 9/8/20 12:30 PM, Philippe Mathieu-Daudé wrote: > >>> On 8/28/20 9:19 PM, Sai Pavan Boddu wrote: > This patch starts making the hcd-xhci.c pci free, as part of this > restructuring dma read/writes are handled with

Re: [PATCH v4 1/7] usb/hcd-xhci: Make dma read/writes hooks pci free

2020-09-10 Thread David Gibson
On Tue, Sep 08, 2020 at 12:30:17PM +0200, Philippe Mathieu-Daudé wrote: > >> On 8/28/20 9:19 PM, Sai Pavan Boddu wrote: > >>> This patch starts making the hcd-xhci.c pci free, as part of this > >>> restructuring dma read/writes are handled without passing pci object. > >>> > >>> Signed-off-by: Sai

[REPORT] Nightly Performance Tests - Thursday, September 10, 2020

2020-09-10 Thread Ahmed Karaman
Host CPU : Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz Host Memory : 15.49 GB Start Time (UTC) : 2020-09-10 22:30:02 End Time (UTC) : 2020-09-10 23:02:44 Execution Time : 0:32:42.042974 Status : SUCCESS Note: Changes denoted by '-' are less than 0.01%. -

[PATCH 9/9] docs: Move object.h overview doc comment to qom.rst

2020-09-10 Thread Eduardo Habkost
Move the whole contents of the overview doc comment from object.h to qom.rst. This makes the documentation source easier to read and edit, and also solves the backslash escaping issue at the typecasting macro examples. Signed-off-by: Eduardo Habkost --- docs/devel/qom.rst | 373 ++

[PATCH 8/9] docs: Create docs/devel/qom.rst

2020-09-10 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- docs/devel/index.rst | 1 + docs/devel/qom.rst | 5 + 2 files changed, 6 insertions(+) create mode 100644 docs/devel/qom.rst diff --git a/docs/devel/index.rst b/docs/devel/index.rst index 04773ce076..c34b43ec28 100644 --- a/docs/devel/index.rst +++ b/doc

[PATCH 7/9] qom: Add code block markup to all code blocks

2020-09-10 Thread Eduardo Habkost
Convert all example/codelisting markup to Sphinx code-block. There are a few sections where backslashes at the end of lines break code formatting. A comment was added noting that this is an issue. Signed-off-by: Eduardo Habkost --- include/qom/object.h | 135 ++-

[PATCH 3/9] qom: Use ``code`` Sphinx syntax where appropriate

2020-09-10 Thread Eduardo Habkost
Replace gtkdoc markup with Sphinx ``code`` syntax. Signed-off-by: Eduardo Habkost --- include/qom/object.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/qom/object.h b/include/qom/object.h index f85718380d..4fb528d841 100644 --- a/include/qom/object.h +++ b/in

[PATCH 6/9] qom: Indent existing code examples

2020-09-10 Thread Eduardo Habkost
This indents existing code examples that are not indented yet, just to make future conversion to Sphinx markup easier to review. Signed-off-by: Eduardo Habkost --- include/qom/object.h | 376 +-- 1 file changed, 188 insertions(+), 188 deletions(-) diff --

[PATCH 4/9] qom: Add kernel-doc markup to introduction doc comment

2020-09-10 Thread Eduardo Habkost
Add DOC: section keyword to introduction doc comment, so it will be rendered by kernel-doc. Signed-off-by: Eduardo Habkost --- include/qom/object.h | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/include/qom/object.h b/include/qom/object.h index 4fb528d841..d2eecdf872 1006

[PATCH 1/9] qom: Document all function parameters in doc comments

2020-09-10 Thread Eduardo Habkost
kernel-doc requires all function parameters to be documented, so document them all. Signed-off-by: Eduardo Habkost --- include/qom/object.h | 47 ++-- 1 file changed, 45 insertions(+), 2 deletions(-) diff --git a/include/qom/object.h b/include/qom/object.

[PATCH 0/9] docs: Convert QOM documentation to kernel-doc + Sphinx

2020-09-10 Thread Eduardo Habkost
This converts the existing gtk-doc markup in object.h (that probably was never rendered into an actual document) to kernel-doc + Sphinx syntax, and reference them at docs/devel/qom.rst. Eduardo Habkost (9): qom: Document all function parameters in doc comments qom: Use kernel-doc private/publi

[PATCH 5/9] qom: Reformat section titles using Sphinx syntax

2020-09-10 Thread Eduardo Habkost
Signed-off-by: Eduardo Habkost --- include/qom/object.h | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/include/qom/object.h b/include/qom/object.h index d2eecdf872..da9ecb310f 100644 --- a/include/qom/object.h +++ b/include/qom/object.h @@ -120,7 +120,8 @@ typede

[PATCH 2/9] qom: Use kernel-doc private/public tags in structs

2020-09-10 Thread Eduardo Habkost
Use kernel-doc syntax for indicating private and public struct fields. Signed-off-by: Eduardo Habkost --- include/qom/object.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/include/qom/object.h b/include/qom/object.h index ad2b91ec6c..f85718380d 100644 --- a/include/q

Re: [PATCH 07/14] block/blklogwrites: drop error propagation

2020-09-10 Thread Ari Sundholm
Hi Vladimir! Thank you for working on this. My comments below. On 9/9/20 9:59 PM, Vladimir Sementsov-Ogievskiy wrote: It's simple to avoid error propagation in blk_log_writes_open(), we just need to refactor blk_log_writes_find_cur_log_sector() a bit. Signed-off-by: Vladimir Sementsov-Ogievski

[PATCH v5 5/7] hw/misc/mps2-fpgaio: Use the LED device

2020-09-10 Thread Philippe Mathieu-Daudé
Per the 'ARM MPS2 and MPS2+ FPGA Prototyping Boards Technical Reference Manual' (100112_0200_07_en): 2.1 Overview of the MPS2 and MPS2+ hardware The MPS2 and MPS2+ FPGA Prototyping Boards contain the following components and interfaces: * User switches and user LEDs:

[PATCH v5 7/7] hw/arm/tosa: Replace fprintf() calls by LED devices

2020-09-10 Thread Philippe Mathieu-Daudé
The recently added LED device reports LED status changes with the 'led_set_intensity' trace event. It is less invasive than the fprintf() calls. We need however to have a binary built with tracing support. Signed-off-by: Philippe Mathieu-Daudé --- hw/arm/tosa.c | 40 +++-

[PATCH v5 3/7] hw/misc/led: Emit a trace event when LED intensity has changed

2020-09-10 Thread Philippe Mathieu-Daudé
Track the LED intensity, and emit a trace event when it changes. Reviewed-by: Richard Henderson Signed-off-by: Philippe Mathieu-Daudé --- hw/misc/led.c| 4 hw/misc/trace-events | 1 + 2 files changed, 5 insertions(+) diff --git a/hw/misc/led.c b/hw/misc/led.c index 3ef4f5e0469..eb

[PATCH v5 4/7] hw/arm/aspeed: Add the 3 front LEDs drived by the PCA9552 #1

2020-09-10 Thread Philippe Mathieu-Daudé
The Witherspoon has 3 LEDs connected to a PCA9552. Add them. The names and reset values are taken from: https://github.com/open-power/witherspoon-xml/blob/master/witherspoon.xml Example booting obmc-phosphor-image: $ qemu-system-arm -M witherspoon-bmc -trace led_change_intensity 1592693373.99

[PATCH v5 6/7] hw/misc/mps2-scc: Use the LED device

2020-09-10 Thread Philippe Mathieu-Daudé
Per the 'ARM MPS2 and MPS2+ FPGA Prototyping Boards Technical Reference Manual' (100112_0200_07_en): 2.1 Overview of the MPS2 and MPS2+ hardware The MPS2 and MPS2+ FPGA Prototyping Boards contain the following components and interfaces: * User switches and user LEDs:

[PATCH v5 1/7] hw/misc/led: Add a LED device

2020-09-10 Thread Philippe Mathieu-Daudé
Add a LED device which can be connected to a GPIO output. They can also be dimmed with PWM devices. For now we do not implement the dimmed mode, but in preparation of a future implementation, we start using the LED intensity. LEDs are limited to a fixed set of colors. Signed-off-by: Philippe Math

[PATCH v5 2/7] hw/misc/led: Allow connecting from GPIO output

2020-09-10 Thread Philippe Mathieu-Daudé
Some devices expose GPIO lines. Add a GPIO qdev input to our LED device, so we can connect a GPIO output using qdev_connect_gpio_out(). When used with GPIOs, the intensity can only be either minium or maximum. This depends of the polarity of the GPIO (which can be inverted). Declare the GpioPolar

[PATCH v5 0/7] hw/misc: Add LED device

2020-09-10 Thread Philippe Mathieu-Daudé
Hello, These patches are part of the GSoC unselected 'QEMU visualizer' project. This series introduce a LED device that can be easily connected to a GPIO output. Since v4: - Fixed typos (Luc) - Removed TYPE_TOSA_MISC_GPIO qdev conversion patch (Peter) Since v3: - Rebased (TYPE_TOSA_MISC_GPIO) -

Re: [PATCH v7 03/25] block: Fixes nfs compiling error on msys2/mingw

2020-09-10 Thread Yonggang Luo
On Fri, Sep 11, 2020 at 4:16 AM Peter Lieven wrote: > > > Am 10.09.2020 um 12:30 schrieb Yonggang Luo : > > These compiling errors are fixed: > ../block/nfs.c:27:10: fatal error: poll.h: No such file or directory > 27 | #include > | ^~~~ > compilation terminated. > > ../block

Re: elf2dmp: Fix memory leak on main() error paths

2020-09-10 Thread Viktor Prutyanov
On Wed, 26 Aug 2020 18:15:53 +0800 AlexChen wrote: > From: AlexChen > > The 'kdgb' is allocating memory in get_kdbg(), but it is not freed > in both fill_header() and fill_context() failed branches, fix it. > > Signed-off-by: AlexChen > --- > contrib/elf2dmp/main.c | 4 ++-- > 1 file changed

Re: [PULL 06/10] configure: don't enable ppc64abi32-linux-user by default

2020-09-10 Thread Peter Maydell
On Thu, 10 Sep 2020 at 14:15, Alex Bennée wrote: > > The user can still enable this explicitly but they will get a warning > at the end of configure for their troubles. This also drops any builds > of ppc64abi32 from our CI tests. > > Signed-off-by: Alex Bennée > Reviewed-by: Philippe Mathieu-Dau

Re: [PATCH v7 03/25] block: Fixes nfs compiling error on msys2/mingw

2020-09-10 Thread Peter Lieven
> Am 10.09.2020 um 12:30 schrieb Yonggang Luo : > > These compiling errors are fixed: > ../block/nfs.c:27:10: fatal error: poll.h: No such file or directory > 27 | #include > | ^~~~ > compilation terminated. > > ../block/nfs.c:63:5: error: unknown type name 'blkcnt_t' > 6

Re: [PATCH 00/77] Patch Round-up for stable 5.0.1, freeze on 2020-09-10

2020-09-10 Thread Philippe Mathieu-Daudé
On 9/10/20 8:16 PM, Michael Roth wrote: > Quoting Philippe Mathieu-Daudé (2020-09-04 04:20:00) >> On 9/3/20 10:58 PM, Michael Roth wrote: >>> Hi everyone, >>> >>> The following new patches are queued for QEMU stable v5.0.1: >>> >>> https://github.com/mdroth/qemu/commits/stable-5.0-staging >>> >>>

[PATCH] hw/arm/aspeed: Add machine properties to define the flash models

2020-09-10 Thread Cédric Le Goater
Some machines don't have much differences a part from the flash model being used. Introduce new machine properties to change them from the command line. Cc: 郁雷 Signed-off-by: Cédric Le Goater --- hw/arm/aspeed.c | 45 +++-- 1 file changed, 43 insertions(+

Re: [PATCH v2 2/3] hw/arm/mps2: New board model mps2-an500

2020-09-10 Thread Peter Maydell
On Fri, 4 Sep 2020 at 14:38, Philippe Mathieu-Daudé wrote: > > On 9/3/20 10:20 PM, Peter Maydell wrote: > > Implement a model of the MPS2 with the AN500 firmware. This is > > similar to the AN385, with the following differences: > > * Cortex-M7 CPU > > * PSRAM is at 0x6000_ > > * Ethernet i

Re: [PATCH] qemu-img: avoid unaligned read requests during convert

2020-09-10 Thread Peter Lieven
> Am 10.09.2020 um 18:58 schrieb Max Reitz : > > On 01.09.20 14:51, Peter Lieven wrote: >> in case of large continous areas that share the same allocation status >> it happens that the value of s->sector_next_status is unaligned to the >> cluster size or even request alignment of the source. A

Re: [PATCH] decodetree: Improve identifier matching

2020-09-10 Thread Peter Maydell
On Thu, 3 Sep 2020 at 20:23, Richard Henderson wrote: > > Only argument set members have to be C identifiers, everything > else gets prefixed during conversion to C. Some places just > checked the leading character, and some places matched a leading > character plus a C identifier. > > Convert ev

Re: [RISU PATCH 0/2] arm.risu: two minor fixes

2020-09-10 Thread Peter Maydell
Ping for review? Mostly I'm interested in whether either Richard or Alex knows what's up with the discrepancy between the VLDR/VSTR patterns in current risu and the generated aarch32 binaries/traces that have been floating around... thanks -- PMM On Tue, 1 Sep 2020 at 17:20, Peter Maydell wrote:

[PATCH 17/18] chardev: Rename WIN_CHARDEV to CHARDEV_WIN

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- include/chardev/char-win.h | 2 +- chardev/char-pipe.c| 2 +- chardev/char-win.c | 14 +++--- 3 files chang

[PATCH 18/18] chardev: Rename WIN_STDIO_CHARDEV to CHARDEV_WIN_STDIO

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/char-win-stdio.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/chardev/char-win-stdio.c

[PATCH 16/18] chardev: Rename WCTABLET_CHARDEV to CHARDEV_WCTABLET

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/wctablet.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/chardev/wctablet.c b/chardev/wcta

[PATCH 12/18] chardev: Rename SPICE_CHARDEV to CHARDEV_SPICE

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- include/chardev/spice.h | 2 +- chardev/spice.c | 16 2 files changed, 9 insertions(+), 9 deletions(-) dif

[PATCH 13/18] chardev: Rename TESTDEV_CHARDEV to CHARDEV_TESTDEV

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/testdev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/chardev/testdev.c b/chardev/testdev.c index

[PATCH 11/18] chardev: Rename SOCKET_CHARDEV to CHARDEV_SOCKET

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/char-socket.c | 68 +-- 1 file changed, 34 insertions(+), 34 deletions(-) diff --git

[PATCH 10/18] chardev: Rename RINGBUF_CHARDEV to CHARDEV_RINGBUF

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/char-ringbuf.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/chardev/char-ringbuf.c b/char

[PATCH 07/18] chardev: Rename MUX_CHARDEV to CHARDEV_MUX

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/chardev-internal.h | 2 +- chardev/char-fe.c | 4 ++-- chardev/char-mux.c | 22 +++--- cha

[PATCH 15/18] chardev: Rename VC_CHARDEV to CHARDEV_VC

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- ui/console.c | 10 +- ui/gtk.c | 8 ui/spice-app.c | 2 +- 3 files changed, 10 insertions(+), 10 deletions

[PATCH 14/18] chardev: Rename UDP_CHARDEV to CHARDEV_UDP

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/char-udp.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/chardev/char-udp.c b/chardev/ch

[PATCH 06/18] chardev: Rename FD_CHARDEV to CHARDEV_FD

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- include/chardev/char-fd.h | 2 +- chardev/char-fd.c | 14 +++--- chardev/char-serial.c | 2 +- 3 files changed,

[PATCH 05/18] chardev: Rename BAUM_CHARDEV to CHARDEV_BRAILLE

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_BRAILLE constant name and the QOM type name string ("chardev-braille"). Signed-off-by: Eduardo Habkost --- chardev/baum.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/chardev/baum.c b/chardev/

[PATCH 09/18] chardev: Rename PTY_CHARDEV to CHARDEV_PTY

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/char-pty.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/chardev/char-pty.c b/

[PATCH 08/18] chardev: Rename PARALLEL_CHARDEV to CHARDEV_PARALLEL

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_* constant names and the QOM type name strings ("chardev-*"). Signed-off-by: Eduardo Habkost --- chardev/char-parallel.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/chardev/char-parallel.c b/ch

[PATCH 03/18] chardev: Use DECLARE_INSTANCE_CHECKER macro for PARALLEL_CHARDEV

2020-09-10 Thread Eduardo Habkost
Use DECLARE_INSTANCE_CHECKER instead of manually defining the instance type checking macro. Signed-off-by: Eduardo Habkost --- chardev/char-parallel.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/chardev/char-parallel.c b/chardev/char-parallel.c index e32aa2d226..3d670

[PATCH 01/18] chardev: Move PARALLEL_CHARDEV macro to common code

2020-09-10 Thread Eduardo Habkost
The macro is exactly the same for both Linux and BSD, so move its definition outside the host OS #ifdef blocks. Signed-off-by: Eduardo Habkost --- chardev/char-parallel.c | 9 +++-- 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/chardev/char-parallel.c b/chardev/char-parallel.

[PATCH 02/18] chardev: Move ParallelChardev typedef to common code

2020-09-10 Thread Eduardo Habkost
The struct itself may have a different definition on each OS, but we can define the typedef in common code, outside the host OS #ifdefs. Signed-off-by: Eduardo Habkost --- chardev/char-parallel.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/chardev/char-parallel.c

[PATCH 00/18] chardev: QOM cleanups

2020-09-10 Thread Eduardo Habkost
Some chardev QOM cleanup patches had to be dropped from my queue due to build erros introduced by code movements across ifdef boundaries at char-parallel.c. This series redo the changes from those patches, but the macro renames are now a little different: In this version I have decided to rename

[PATCH 04/18] chardev: Rename MOUSE_CHARDEV to CHARDEV_MSMOUSE

2020-09-10 Thread Eduardo Habkost
Rename instance and class type checkers to match the TYPE_CHARDEV_MSMOUSE constant name and the QOM type name string ("chardev-msmouse"). Signed-off-by: Eduardo Habkost --- chardev/msmouse.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/chardev/msmouse.c b/char

Re: [PATCH v2 1/2] Introduce (x86) CPU model deprecation API

2020-09-10 Thread Philippe Mathieu-Daudé
On 9/10/20 9:12 PM, Eduardo Habkost wrote: > On Thu, Sep 10, 2020 at 07:29:03AM +0200, Philippe Mathieu-Daudé wrote: >> On 9/9/20 8:15 PM, Eduardo Habkost wrote: >>> Hi, >>> >>> Thanks for the patch, and sorry for taking so long to review >>> this. I'm finally getting to the patches that were post

Re: [PATCH 00/10] edk2: adopt the edk2-stable202008 release

2020-09-10 Thread Philippe Mathieu-Daudé
On 9/8/20 9:29 AM, Laszlo Ersek wrote: > Ref:https://bugs.launchpad.net/qemu/+bug/1852196 > Repo: https://github.com/lersek/qemu.git > Branch: edk2stable202008_lp_1852196 > > This series consumes the following upstream edk2 releases: > > https://github.com/tianocore/edk2/releases/tag/edk2

Re: [PATCH 00/77] Patch Round-up for stable 5.0.1, freeze on 2020-09-10

2020-09-10 Thread Philippe Mathieu-Daudé
On 9/10/20 8:16 PM, Michael Roth wrote: > Quoting Philippe Mathieu-Daudé (2020-09-04 04:20:00) >> On 9/3/20 10:58 PM, Michael Roth wrote: >>> Hi everyone, >>> >>> The following new patches are queued for QEMU stable v5.0.1: >>> >>> https://github.com/mdroth/qemu/commits/stable-5.0-staging >>> >>>

Re: [PATCH 1/6] hw/ssi/aspeed_smc: Rename max_slaves as max_devices

2020-09-10 Thread Cédric Le Goater
On 9/10/20 9:01 AM, Philippe Mathieu-Daudé wrote: > In order to use inclusive terminology, rename max_slaves > as max_devices.> > Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Cédric Le Goater > --- > include/hw/ssi/aspeed_smc.h | 2 +- > hw/ssi/aspeed_smc.c | 40 +

Re: [PATCH 02/63] chardev: Rename TYPE_CHARDEV_* to TYPE_*_CHARDEV

2020-09-10 Thread Eduardo Habkost
On Wed, Sep 02, 2020 at 06:42:10PM -0400, Eduardo Habkost wrote: > This will make the TYPE_* constants consistent with the name of > most type checking macros we have today. > > Signed-off-by: Eduardo Habkost I will drop this and send a separate patch to rename the type checking macros instead,

Re: [PATCH v2 1/2] Introduce (x86) CPU model deprecation API

2020-09-10 Thread Eduardo Habkost
On Thu, Sep 10, 2020 at 07:29:03AM +0200, Philippe Mathieu-Daudé wrote: > On 9/9/20 8:15 PM, Eduardo Habkost wrote: > > Hi, > > > > Thanks for the patch, and sorry for taking so long to review > > this. I'm finally getting to the patches that were postponed to > > 5.2. > > Sorry I missed that pa

Re: [Bug 1895080] [NEW] pgb_reserved_va: Assertion `addr == test' failed

2020-09-10 Thread Laurent Vivier
Le 10/09/2020 à 07:21, Launchpad Bug Tracker a écrit : > This problem occurs on CentOS-7.5 (64-bit) with qemu-5.1.0, qemu head > (commit 9435a8b3dd35f1f926f1b9127e8a906217a5518a) for riscv32-linux- > user. I tried to build qemu-5.1 on CentOS-7.5.1 but as python 3.5 is not available, I gave up. >

Re: [PATCH v2 6/6] tools/virtiofsd: xattr name mapping examples

2020-09-10 Thread Dr. David Alan Gilbert
* Ján Tomko (jto...@redhat.com) wrote: > On a Thursday in 2020, Dr. David Alan Gilbert (git) wrote: > > From: "Dr. David Alan Gilbert" > > > > Add a few examples of xattrmaps to the documentation. > > > > Signed-off-by: Dr. David Alan Gilbert > > --- > > docs/tools/virtiofsd.rst | 49 +

Re: [for-5.2 v4 10/10] s390: Recognize host-trust-limitation option

2020-09-10 Thread Halil Pasic
On Thu, 10 Sep 2020 13:36:09 +0200 Cornelia Huck wrote: > On Mon, 7 Sep 2020 17:22:53 +0200 > Halil Pasic wrote: > > > On Fri, 24 Jul 2020 12:57:44 +1000 > > David Gibson wrote: > > > > > At least some s390 cpu models support "Protected Virtualization" (PV), > > > a mechanism to protect guest

Re: [PATCH v5 06/11] riscv: Initial commit of OpenTitan machine

2020-09-10 Thread Alistair Francis
On Wed, Sep 9, 2020 at 12:51 PM Palmer Dabbelt wrote: > > On Wed, 09 Sep 2020 12:00:07 PDT (-0700), Peter Maydell wrote: > > On Wed, 9 Sep 2020 at 19:00, Alistair Francis wrote: > >> > >> On Tue, Sep 8, 2020 at 7:52 AM Peter Maydell > >> wrote: > >> > ...shouldn't the riscv64-softmmu config hav

[PULL 26/33] pc87312: Rename TYPE_PC87312_SUPERIO to TYPE_PC87312

2020-09-10 Thread Eduardo Habkost
This will make the type name constant consistent with the name of the type checking macro. Signed-off-by: Eduardo Habkost Reviewed-by: Hervé Poussineau Message-Id: <20200902224311.1321159-21-ehabk...@redhat.com> Signed-off-by: Eduardo Habkost --- include/hw/isa/pc87312.h | 4 ++-- hw/isa/pc873

Re: Master cannot execute MTE instructions

2020-09-10 Thread Derrick McKee
Thanks for the help. The virt,mte=on did it! On Thu, Sep 10, 2020 at 1:49 PM Peter Maydell wrote: > On Thu, 10 Sep 2020 at 18:43, Andrew Jones wrote: > > > > On Thu, Sep 10, 2020 at 05:17:17PM +0100, Peter Maydell wrote: > > > What QEMU command line are you using to run this? > > > In particula

[PULL 29/33] omap_intc: Use typedef name for instance_size

2020-09-10 Thread Eduardo Habkost
This makes the code consistent with the rest of QOM code in QEMU, and will make automated conversion to type declaration macros simpler. Signed-off-by: Eduardo Habkost Message-Id: <20200824215936.2961951-3-ehabk...@redhat.com> Signed-off-by: Eduardo Habkost --- hw/intc/omap_intc.c | 2 +- 1 fil

[PULL 25/33] vfio: Rename PCI_VFIO to VFIO_PCI

2020-09-10 Thread Eduardo Habkost
Make the type checking macro name consistent with the TYPE_* constant. Signed-off-by: Eduardo Habkost Reviewed-by: Eric Auger Message-Id: <20200902224311.1321159-56-ehabk...@redhat.com> Signed-off-by: Eduardo Habkost --- hw/vfio/pci.h | 2 +- hw/vfio/pci.c | 22 +++--- 2 files

[PULL 28/33] xilinx_axidma: Use typedef name for instance_size

2020-09-10 Thread Eduardo Habkost
This makes the code consistent with the rest of QOM code in QEMU, and will make automated conversion to type declaration macros simpler. Signed-off-by: Eduardo Habkost Message-Id: <20200824215936.2961951-2-ehabk...@redhat.com> Signed-off-by: Eduardo Habkost --- hw/dma/xilinx_axidma.c | 4 ++--

[PULL 20/33] esp: Rename ESP_STATE to ESP

2020-09-10 Thread Eduardo Habkost
Make the type checking macro name consistent with the TYPE_* constant. Signed-off-by: Eduardo Habkost Reviewed-by: Hervé Poussineau Message-Id: <20200902224311.1321159-40-ehabk...@redhat.com> Signed-off-by: Eduardo Habkost --- include/hw/scsi/esp.h | 2 +- hw/dma/sparc32_dma.c | 2 +- hw/m68k

  1   2   3   4   5   >