[PATCH v4] fuzz: add oss-fuzz build-script

2020-06-11 Thread Alexander Bulekov
It is neater to keep this in the QEMU repo, since any change that requires an update to the oss-fuzz build configuration, can make the necessary changes in the same series. Suggested-by: Philippe Mathieu-Daudé Signed-off-by: Alexander Bulekov Reviewed-by: Darren Kenny --- v4: adds fatal() for

[Bug 1882851] Re: QEMU video freezes with "Guest disabled display" (virtio driver)

2020-06-11 Thread Diego Viola
** Attachment added: "bisectlog.txt" https://bugs.launchpad.net/qemu/+bug/1882851/+attachment/5383160/+files/bisectlog.txt -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1882851 Title: QEMU

[Bug 1882851] Re: QEMU video freezes with "Guest disabled display" (virtio driver)

2020-06-11 Thread Diego Viola
OK, I was able to bisect, here is the result: [diego@arch-zoom linux]$ git bisect bad 3954ff10e06e4fbc548fc02ff1fcaaac3228fed5 is the first bad commit commit 3954ff10e06e4fbc548fc02ff1fcaaac3228fed5 Author: Gerd Hoffmann Date: Thu Dec 12 13:53:44 2019 +0100 drm/virtio: skip set_scanout if

Re: [PATCH 0/5] linux-user: Support extended clone(CLONE_VM)

2020-06-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200612014606.147691-1-...@google.com/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN ===

Re: [PATCH 9/9] migration/ram: calculate un/encoded_size only whenneeded.

2020-06-11 Thread maozy
On 6/12/20 3:05 AM, Dr. David Alan Gilbert wrote: * Mao Zhongyi (maozhon...@cmss.chinamobile.com) wrote: Signed-off-by: Mao Zhongyi --- migration/ram.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/migration/ram.c b/migration/ram.c index

RE: [PATCH] kvm: support to get/set dirty log initial-all-set capability

2020-06-11 Thread Zhoujian (jay)
> -Original Message- > From: Paolo Bonzini [mailto:pbonz...@redhat.com] > Sent: Wednesday, March 18, 2020 6:48 PM > To: Zhoujian (jay) ; qemu-devel@nongnu.org; > k...@vger.kernel.org > Cc: m...@redhat.com; coh...@redhat.com; pet...@redhat.com; wangxin (U) > ; Huangweidong (C) > ;

[PATCH 5/6] audio/jack: honour the enable state of the audio device

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 29 + 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index 249cbd3265..b2b53985ae 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -56,7 +56,7 @@

[PATCH 6/6] audio/jack: simplify the re-init code path

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index b2b53985ae..72ed7c4929 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -395,6 +395,10 @@ static int

[PATCH 4/6] audio/jack: do not remove ports when finishing

2020-06-11 Thread Geoffrey McRae
This fixes a hang when there is a communications issue with the JACK server. Simply closing the connection is enough to completely clean up and as such we do not need to remove the ports first. Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 3 --- 1 file changed, 3 deletions(-) diff

[PATCH 3/6] audio/jack: remove invalid set of input support bool

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index fb8efd7af7..58c7344497 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -607,9 +607,6 @@ static int

[PATCH 2/6] audio/jack: remove unused stopped state

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 4 1 file changed, 4 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index d0b6f748f2..fb8efd7af7 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -38,7 +38,6 @@ struct QJack; typedef enum QJackState {

[PATCH 0/6] audio/jack: fixes to overall jack behaviour

2020-06-11 Thread Geoffrey McRae
Sorry for the spam, resubmitted due to missing subject on this cover letter. Seems patchew.org can't find the associated patches without it. This patch set addresses several issues that cause inconsistent behaviour in the guest when the sound device is stopped and started or the JACK server stops

[PATCH 1/6] audio/jack: fix invalid minimum buffer size check

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index 722ddb1dfe..d0b6f748f2 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -434,17 +434,6 @@ static

Re: [PULL 000/115] Huge miscellaneous pull request for 2020-06-11

2020-06-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200611194449.31468-1-pbonz...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20200611194449.31468-1-pbonz...@redhat.com Subject: [PULL 000/115] Huge miscellaneous pull request

[PATCH 5/5] linux-user: Add PDEATHSIG test for clone process hierarchy.

2020-06-11 Thread Josh Kunz
Certain process-level linux features like subreapers, and PDEATHSIG, depend on the guest's process hierarchy being emulated correctly on the host. This change adds a test that makes sure PDEATHSIG works for a guest process created with `clone`. Signed-off-by: Josh Kunz ---

[PATCH 2/5] linux-user: Make fd_trans task-specific.

2020-06-11 Thread Josh Kunz
The file-descriptor translation subsystem used by QEMU uses some global variables to track file descriptors an their associated state. In the future (when clone is implemented) it may be possible to have two processes that share memory, but have a unique set of file descriptors. This change

[PATCH 4/5] linux-user: Support CLONE_VM and extended clone options

2020-06-11 Thread Josh Kunz
The `clone` system call can be used to create new processes that share attributes with their parents, such as virtual memory, file system location, file descriptor tables, etc. These can be useful to a variety of guest programs. Before this patch, QEMU had support for a limited set of these

[PATCH 0/5] linux-user: Support extended clone(CLONE_VM)

2020-06-11 Thread Josh Kunz
This patch series implements extended support for the `clone` system call. As best I can tell, any option combination including `CLONE_VM` should be supported with the addition of this patch series. The implementation is described in greater detail in the patches themselves. Testing: * All

[PATCH 3/5] linux-user: Make sigact_table part of the task state.

2020-06-11 Thread Josh Kunz
sigact_table stores the signal handlers for the given process. Once we support CLONE_VM, two tasks using the same virtual memory may need different signal handler tables (e.g., if CLONE_SIGHAND is not provided). Here we make sigact_table part of the TaskState, so it can be duplicated as needed

[PATCH 1/5] linux-user: Refactor do_fork to use new `qemu_clone`

2020-06-11 Thread Josh Kunz
This is pre-work for adding full support for the `CLONE_VM` `clone` flag. In a follow-up patch, we'll add support to `clone.c` for `clone_vm`-type clones beyond threads. CLONE_VM support is more complicated, so first we're splitting existing clone mechanisms (pthread_create, and fork) into a

[PATCH 3/6] audio/jack: remove invalid set of input support bool

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index fb8efd7af7..58c7344497 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -607,9 +607,6 @@ static int

[PATCH 4/6] audio/jack: do not remove ports when finishing

2020-06-11 Thread Geoffrey McRae
This fixes a hang when there is a communications issue with the JACK server. Simply closing the connection is enough to completely clean up and as such we do not need to remove the ports first. Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 3 --- 1 file changed, 3 deletions(-) diff

[PATCH 2/6] audio/jack: remove unused stopped state

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 4 1 file changed, 4 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index d0b6f748f2..fb8efd7af7 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -38,7 +38,6 @@ struct QJack; typedef enum QJackState {

[PATCH 5/6] audio/jack: honour the enable state of the audio device

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 29 + 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index 249cbd3265..b2b53985ae 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -56,7 +56,7 @@

[PATCH 1/6] audio/jack: fix invalid minimum buffer size check

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index 722ddb1dfe..d0b6f748f2 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -434,17 +434,6 @@ static

[PATCH 6/6] audio/jack: simplify the re-init code path

2020-06-11 Thread Geoffrey McRae
Signed-off-by: Geoffrey McRae --- audio/jackaudio.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/audio/jackaudio.c b/audio/jackaudio.c index b2b53985ae..72ed7c4929 100644 --- a/audio/jackaudio.c +++ b/audio/jackaudio.c @@ -395,6 +395,10 @@ static int

[PATCH 0/6] *** SUBJECT HERE ***

2020-06-11 Thread Geoffrey McRae
This patch set addresses several issues that cause inconsistent behaviour in the guest when the sound device is stopped and started or the JACK server stops responding on the host. Geoffrey McRae (6): audio/jack: fix invalid minimum buffer size check audio/jack: remove unused stopped state

[PATCH v7 6/9] qcow2_format.py: Dump bitmap table serialized entries

2020-06-11 Thread Andrey Shinkevich
Add bitmap table information to the QCOW2 metadata dump. It extends the output of the test case #291 Bitmap name bitmap-1 ... Bitmap table typeoffset size 0 serialized 4718592 65536 1 serialized

[PATCH v7 5/9] qcow2_format.py: pass cluster size to substructures

2020-06-11 Thread Andrey Shinkevich
The cluster size of an image is the QcowHeader class member and may be obtained by dependent extension structures such as Qcow2BitmapExt for further bitmap table details print. Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/qcow2_format.py | 19 ++- 1 file changed, 14

[PATCH v7 2/9] qcow2: Fix capitalization of header extension constant.

2020-06-11 Thread Andrey Shinkevich
Make the capitalization of the hexadecimal numbers consistent for the QCOW2 header extension constants in docs/interop/qcow2.txt. Suggested-by: Eric Blake Signed-off-by: Andrey Shinkevich --- block/qcow2.c | 2 +- docs/interop/qcow2.txt | 2 +- 2 files changed, 2 insertions(+), 2

[PATCH v7 9/9] qcow2_format.py: support dumping metadata in JSON format

2020-06-11 Thread Andrey Shinkevich
Implementation of dumping QCOW2 image metadata. The sample output: { "Header_extensions": [ { "name": "Feature table", "magic": 1745090647, "length": 192, "data_str": "" }, { "name": "Bitmaps",

[PATCH v7 0/9] iotests: Dump QCOW2 dirty bitmaps metadata

2020-06-11 Thread Andrey Shinkevich
Note: based on the Vladimir's series [v5 00/13] iotests: Dump QCOW2 dirty bitmaps metadata Add dirty bitmap information to QCOW2 metadata dump in the qcow2_format.py. v7: 01: Fix for magic hexadecimal output in 291 02: Bitmap table output format improvement. 03: Incremental change in

[PATCH v7 8/9] qcow2_format.py: collect fields to dump in JSON format

2020-06-11 Thread Andrey Shinkevich
As __dict__ is being extended with class members we do not want to print in JSON format dump, make a light copy of the initial __dict__ and extend the copy by adding lists we have to print in the JSON output. Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/qcow2_format.py | 5 - 1

[PATCH v7 4/9] qcow2_format.py: Dump bitmap directory information

2020-06-11 Thread Andrey Shinkevich
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 flag auto table size8 (bytes)

[PATCH v7 1/9] iotests: Fix for magic hexadecimal output in 291

2020-06-11 Thread Andrey Shinkevich
This issue was introduced in the earlier patch: "qcow2_format: refactor QcowHeaderExtension as a subclass of Qcow2Struct". Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/291.out | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/qemu-iotests/291.out

[PATCH v7 3/9] qcow2_format.py: make printable data an extension class member

2020-06-11 Thread Andrey Shinkevich
Let us differ binary data type from string one for the extension data variable and keep the string as the QcowHeaderExtension class member. Signed-off-by: Andrey Shinkevich Reviewed-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/qcow2_format.py | 14

[PATCH v7 7/9] qcow2.py: Introduce '-j' key to dump in JSON format

2020-06-11 Thread Andrey Shinkevich
Add the command key to the qcow2.py arguments list to dump QCOW2 metadata in JSON format. Here is the suggested way to do that. The implementation of the dump in JSON format is in the patch that follows. Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/qcow2.py| 20

Re: [PATCH] tests: disassemble-asm.sh: generate AML in readable format

2020-06-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200611165112.30979-1-...@redhat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20200611165112.30979-1-...@redhat.com Subject: [PATCH] tests: disassemble-asm.sh: generate AML in readable

[PATCH] target/i386: reimplement f2xm1 using floatx80 operations

2020-06-11 Thread Joseph Myers
The x87 f2xm1 emulation is currently based around conversion to double. This is inherently unsuitable for a good emulation of any floatx80 operation, even before considering that it is a particularly naive implementation using double (computing with pow and then subtracting 1 rather than

Re: [PATCH 00/10] target/arm: Convert 2-reg-scalar to decodetree

2020-06-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200611144529.8873-1-peter.mayd...@linaro.org/ Hi, This series failed the docker-mingw@fedora build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT

Re: [PATCH 00/10] target/arm: Convert 2-reg-scalar to decodetree

2020-06-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200611144529.8873-1-peter.mayd...@linaro.org/ Hi, This series failed the docker-quick@centos7 build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT

[Bug 1882851] Re: QEMU video freezes with "Guest disabled display" (virtio driver)

2020-06-11 Thread Diego Viola
I can reproduce it with current linux git master[1]. 1. git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1882851 Title: QEMU video

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

2020-06-11 Thread Havard Skinnemoen
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 \ -bios ~/qemu/bootrom/npcm7xx_bootrom.bin \ -drive

[PATCH v2 12/12] docs/system: Add Nuvoton machine documentation

2020-06-11 Thread Havard Skinnemoen
Change-Id: I88e437cb22a8441e0c0e672dfb57568ac81172d8 Signed-off-by: Havard Skinnemoen --- docs/system/arm/nuvoton.rst | 89 + docs/system/target-arm.rst | 1 + 2 files changed, 90 insertions(+) create mode 100644 docs/system/arm/nuvoton.rst diff --git

[PATCH v2 09/12] hw/mem: Stubbed out NPCM7xx Memory Controller model

2020-06-11 Thread Havard Skinnemoen
This just implements the bare minimum to cause the boot block to skip memory intialization. Change-Id: I26fd5f3b2af5d07a24911e7e58789f7b52f78d71 Reviewed-by: Tyrone Ting Signed-off-by: Havard Skinnemoen --- MAINTAINERS | 2 + hw/arm/npcm7xx.c| 11 +

[PATCH v2 08/12] hw/nvram: NPCM7xx OTP device model

2020-06-11 Thread Havard Skinnemoen
This supports reading and writing OTP fuses and keys. Only fuse reading has been tested. Protection is not implemented. Change-Id: Ie67d7e2ed5e1010061b7383c71a8ba0611794103 Reviewed-by: Avi Fishman Signed-off-by: Havard Skinnemoen --- MAINTAINERS| 2 + hw/arm/npcm7xx.c

[PATCH v2 10/12] hw/ssi: NPCM7xx Flash Interface Unit device model

2020-06-11 Thread Havard Skinnemoen
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

[PATCH v2 04/12] hw/timer: Add NPCM7xx Timer device model

2020-06-11 Thread Havard Skinnemoen
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

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

2020-06-11 Thread Havard Skinnemoen
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. A minimal Apache-2.0-licensed boot ROM can be found at https://github.com/google/vbootrom It is by no means feature complete,

[PATCH v2 05/12] hw/arm: Add NPCM730 and NPCM750 SoC models

2020-06-11 Thread Havard Skinnemoen
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

[PATCH v2 06/12] hw/arm: Add two NPCM7xx-based machines

2020-06-11 Thread Havard Skinnemoen
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

[PATCH v2 02/12] hw/misc: Add NPCM7xx System Global Control Registers device model

2020-06-11 Thread Havard Skinnemoen
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

[PATCH v2 01/12] npcm7xx: Add config symbol

2020-06-11 Thread Havard Skinnemoen
Add a config symbol for the NPCM7xx BMC SoC family that subsequent patches can use in Makefiles. Change-Id: I6e4d5c58716cb6fe4ea5d06f148beeafda55f9a5 Reviewed-by: Tyrone Ting Acked-by: Joel Stanley Signed-off-by: Havard Skinnemoen --- default-configs/arm-softmmu.mak | 1 + hw/arm/Kconfig

[PATCH v2 03/12] hw/misc: Add NPCM7xx Clock Controller device model

2020-06-11 Thread Havard Skinnemoen
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

[PATCH v2 00/12] Add Nuvoton NPCM730/NPCM750 SoCs and two BMC machines

2020-06-11 Thread Havard Skinnemoen
This patch series models enough of the Nuvoton NPCM730 and NPCM750 SoCs to boot an OpenBMC image built for quanta-gsj. This includes device models for: - Global Configuration Registers - Clock Control - Timers - Fuses - Memory Controller - Flash Controller These modules, along with

Re: [PATCH v6 1/8] qcow2: Fix capitalization of header extension constant.

2020-06-11 Thread Eric Blake
On 6/11/20 4:19 PM, Andrey Shinkevich wrote: Make the capitalization of the hexadecimal numbers consistent for the QCOW2 header extension constants in docs/interop/qcow2.txt. Suggested-by: Eric Blake Signed-off-by: Andrey Shinkevich --- block/qcow2.c | 2 +- docs/interop/qcow2.txt

Re: [PULL v2 0/3] NBD patches through 2020-06-09

2020-06-11 Thread Peter Maydell
On Wed, 10 Jun 2020 at 19:07, Eric Blake wrote: > > The following changes since commit 31d321c2b3574dcc74e9f6411af06bca6b5d10f4: > > Merge remote-tracking branch > 'remotes/philmd-gitlab/tags/sparc-next-20200609' into staging (2020-06-09 > 17:29:47 +0100) > > are available in the Git

[PATCH v6 0/8] iotests: Dump QCOW2 dirty bitmaps metadata

2020-06-11 Thread Andrey Shinkevich
Note: based on the Vladimir's series [v5 00/13] iotests: Dump QCOW2 dirty bitmaps metadata Add dirty bitmap information to QCOW2 metadata dump in the qcow2_format.py. v6: 01: Fixing capitalization of header extension constant. (Suggested by Eric) 02: The cluster size global

[PATCH v6 1/8] qcow2: Fix capitalization of header extension constant.

2020-06-11 Thread Andrey Shinkevich
Make the capitalization of the hexadecimal numbers consistent for the QCOW2 header extension constants in docs/interop/qcow2.txt. Suggested-by: Eric Blake Signed-off-by: Andrey Shinkevich --- block/qcow2.c | 2 +- docs/interop/qcow2.txt | 2 +- 2 files changed, 2 insertions(+), 2

[PATCH v6 3/8] qcow2_format.py: Dump bitmap directory info

2020-06-11 Thread Andrey Shinkevich
Read and dump entries from the bitmap directory of QCOW2 image with the script qcow2.py. Header extension: magic 0x23852875 (Bitmaps) ... Bitmap name bitmap-1 flag auto table size8 (bytes) bitmap_table_offset 0x9

[PATCH v6 5/8] qcow2_format.py: Dump bitmap table serialized entries

2020-06-11 Thread Andrey Shinkevich
Add bitmap table information to the QCOW2 metadata dump. Bitmap name bitmap-1 ... Bitmap tabletypeoffset size 0 serialized 0xa 65536 1 all-zeroes 0x0 65536 2 all-zeroes 0x0

[PATCH v6 8/8] qcow2_format.py: support dumping metadata in JSON format

2020-06-11 Thread Andrey Shinkevich
Implementation of dumping QCOW2 image metadata. The sample output: { "Header_extensions": [ { "name": "Feature table", "magic": 1745090647, "length": 192, "data_str": "" }, { "name": "Bitmaps",

[PATCH v6 7/8] qcow2_format.py: collect fields to dump in JSON format

2020-06-11 Thread Andrey Shinkevich
As __dict__ is being extended with class members we do not want to print in JSON format dump, make a light copy of the initial __dict__ and extend the copy by adding lists we have to print in the JSON output. Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/qcow2_format.py | 5 - 1

[PATCH v6 2/8] qcow2_format.py: make printable data an extension class member

2020-06-11 Thread Andrey Shinkevich
Let us differ binary data type from string one for the extension data variable and keep the string as the QcowHeaderExtension class member. Signed-off-by: Andrey Shinkevich Reviewed-by: Eric Blake Reviewed-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/qcow2_format.py | 14

[PATCH v6 6/8] qcow2.py: Introduce '-j' key to dump in JSON format

2020-06-11 Thread Andrey Shinkevich
Add the command key to the qcow2.py arguments list to dump QCOW2 metadata in JSON format. Here is the suggested way to do that. The implementation of the dump in JSON format is in the patch that follows. Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/qcow2.py| 20

[PATCH v6 4/8] qcow2_format.py: pass cluster size to substructures

2020-06-11 Thread Andrey Shinkevich
The cluster size of an image is the QcowHeader class member and may be obtained by dependent extension structures such as Qcow2BitmapExt for further bitmap table details print. Signed-off-by: Andrey Shinkevich --- tests/qemu-iotests/qcow2_format.py | 19 ++- 1 file changed, 14

Re: [PATCH v3 1/2] PCI: vmd: Filter resource type bits from shadow register

2020-06-11 Thread Derrick, Jonathan
On Fri, 2020-05-29 at 17:18 +0100, Lorenzo Pieralisi wrote: > On Fri, May 29, 2020 at 03:53:37PM +, Derrick, Jonathan wrote: > > On Fri, 2020-05-29 at 11:33 +0100, Lorenzo Pieralisi wrote: > > > On Wed, May 27, 2020 at 11:02:39PM -0400, Jon Derrick wrote: > > > > Versions of VMD with the Host

[PULL 106/115] target/i386: sev: Remove redundant policy field

2020-06-11 Thread Paolo Bonzini
From: David Gibson SEVState::policy is set from the final value of the policy field in the parameter structure for the KVM_SEV_LAUNCH_START ioctl(). But, AFAICT that ioctl() won't ever change it from the original supplied value which comes from SevGuestState::policy. So, remove this field and

[PULL 113/115] replay: fix replay shutdown for console mode

2020-06-11 Thread Paolo Bonzini
From: Pavel Dovgalyuk When QEMU is used without any graphical window, QEMU execution is terminated with the signal (e.g., Ctrl-C). Signal processing in QEMU does not include qemu_system_shutdown_request call. That is why shutdown event is not recorded by record/replay in this case. This patch

[PULL 109/115] checkpatch: reversed logic with acpi test checks

2020-06-11 Thread Paolo Bonzini
From: "Michael S. Tsirkin" Logic reversed: allowed list should just be ignored. Instead we only take that into account :( Fixes: e11b06a880ca ("checkpatch: ignore allowed diff list") Signed-off-by: Michael S. Tsirkin Message-Id: <20200602053614.54745-1-...@redhat.com> Reviewed-by: Philippe

[PATCH 1/1] util/getauxval: Porting to FreeBSD getauxval feature

2020-06-11 Thread David CARLIER
0001-util-getauxval-Porting-to-FreeBSD-the-getauxval-feat.patch Description: Binary data

[PULL 108/115] target/i386: sev: Unify SEVState and SevGuestState

2020-06-11 Thread Paolo Bonzini
From: David Gibson SEVState is contained with SevGuestState. We've now fixed redundancies and name conflicts, so there's no real point to the nested structure. Just move all the fields of SEVState into SevGuestState. This eliminates the SEVState structure, which as a bonus removes the

[PULL 103/115] target/i386: sev: Embed SEVState in SevGuestState

2020-06-11 Thread Paolo Bonzini
From: David Gibson Currently SevGuestState contains only configuration information. For runtime state another non-QOM struct SEVState is allocated separately. Simplify things by instead embedding the SEVState structure in SevGuestState. Signed-off-by: David Gibson Reviewed-by: Richard

[PULL 105/115] target/i386: sev: Remove redundant cbitpos and reduced_phys_bits fields

2020-06-11 Thread Paolo Bonzini
From: David Gibson The SEVState structure has cbitpos and reduced_phys_bits fields which are simply copied from the SevGuestState structure and never changed. Now that SEVState is embedded in SevGuestState we can just access the original copy directly. Signed-off-by: David Gibson Reviewed-by:

[PULL 102/115] target/i386: sev: Rename QSevGuestInfo

2020-06-11 Thread Paolo Bonzini
From: David Gibson At the moment this is a purely passive object which is just a container for information used elsewhere, hence the name. I'm going to change that though, so as a preliminary rename it to SevGuestState. That name risks confusion with both SEVState and SevState, but I'll be

[PULL 096/115] i386: hvf: Move lazy_flags into CPUX86State

2020-06-11 Thread Paolo Bonzini
From: Roman Bolshakov The lazy flags are still needed for instruction decoder. Signed-off-by: Roman Bolshakov Message-Id: <20200528193758.51454-12-r.bolsha...@yadro.com> Signed-off-by: Paolo Bonzini Reviewed-by: Philippe Mathieu-Daudé --- include/sysemu/hvf.h| 7 +

Re: [PULL v2 00/16] Trivial branch for 5.1 patches

2020-06-11 Thread Peter Maydell
On Wed, 10 Jun 2020 at 14:12, Laurent Vivier wrote: > > The following changes since commit 49ee11555262a256afec592dfed7c5902d5eefd2: > > Merge remote-tracking branch 'remotes/vivier2/tags/linux-user-for-5.1-pull-= > request' into staging (2020-06-08 11:04:57 +0100) > > are available in the Git

[PULL 094/115] i386: hvf: Drop copy of RFLAGS defines

2020-06-11 Thread Paolo Bonzini
From: Roman Bolshakov Use the ones provided in target/i386/cpu.h instead. Signed-off-by: Roman Bolshakov Message-Id: <20200528193758.51454-10-r.bolsha...@yadro.com> Signed-off-by: Paolo Bonzini --- target/i386/hvf/x86.c| 2 +- target/i386/hvf/x86.h| 20

[PULL 091/115] i386: hvf: Use IP from CPUX86State

2020-06-11 Thread Paolo Bonzini
From: Roman Bolshakov Drop and replace rip field from HVFX86EmulatorState in favor of eip from common CPUX86State. Signed-off-by: Roman Bolshakov Message-Id: <20200528193758.51454-7-r.bolsha...@yadro.com> Signed-off-by: Paolo Bonzini Reviewed-by: Philippe Mathieu-Daudé ---

[PULL 069/115] KVM: Kick resamplefd for split kernel irqchip

2020-06-11 Thread Paolo Bonzini
From: Peter Xu This is majorly only for X86 because that's the only one that supports split irqchip for now. When the irqchip is split, we face a dilemma that KVM irqfd will be enabled, however the slow irqchip is still running in the userspace. It means that the resamplefd in the kernel irqfds

Re: [PATCH v3 0/6] Add strace support for printing arguments of selected syscalls

2020-06-11 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200611155109.3648-1-filip.boz...@syrmia.com/ Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20200611155109.3648-1-filip.boz...@syrmia.com Subject: [PATCH v3 0/6] Add strace support for

[PULL 088/115] i386: hvf: Drop useless declarations in sysemu

2020-06-11 Thread Paolo Bonzini
From: Roman Bolshakov They're either declared elsewhere or have no use. While at it, rename _hvf_cpu_synchronize_post_init() to do_hvf_cpu_synchronize_post_init(). Signed-off-by: Roman Bolshakov Message-Id: <20200528193758.51454-3-r.bolsha...@yadro.com> Signed-off-by: Paolo Bonzini

[PULL 111/115] hw/usb: Move device-specific declarations to new 'hcd-musb.h' header

2020-06-11 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Move the declarations for the MUSB-HDRC USB2.0 OTG compliant core into a separate header. Reviewed-by: Peter Maydell Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20200601141536.15192-3-f4...@amsat.org> Signed-off-by: Paolo

[PULL 050/115] megasas: use unsigned type for positive numeric fields

2020-06-11 Thread Paolo Bonzini
From: Prasad J Pandit Use unsigned type for the MegasasState fields which hold positive numeric values. Signed-off-by: Prasad J Pandit Reviewed-by: Darren Kenny Message-Id: <20200513192540.1583887-4-ppan...@redhat.com> Signed-off-by: Paolo Bonzini --- hw/scsi/megasas.c | 38

[PULL 086/115] target/ppc: Restrict PPCVirtualHypervisorClass to system-mode

2020-06-11 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé The code related to PPC Virtual Hypervisor is pointless in user-mode. Acked-by: David Gibson Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20200526172427.17460-5-f4...@amsat.org> Signed-off-by: Paolo Bonzini --- target/ppc/cpu.h| 4 ++--

[PULL 099/115] xen: fix build without pci passthrough

2020-06-11 Thread Paolo Bonzini
From: Anthony PERARD Xen PCI passthrough support may not be available and thus the global variable "has_igd_gfx_passthru" might be compiled out. Common code should not access it in that case. Unfortunately, we can't use CONFIG_XEN_PCI_PASSTHROUGH directly in xen-common.c so this patch instead

[PULL 114/115] stubs: move Xen stubs to accel/

2020-06-11 Thread Paolo Bonzini
Keep them close to the other accelerator-dependent stubs, so as to remove stubs that are not needed by tools. Signed-off-by: Paolo Bonzini --- accel/stubs/Makefile.objs | 1 + stubs/xen-hvm.c => accel/stubs/xen-stub.c | 14 +++--- stubs/Makefile.objs

[PULL 115/115] target/i386: Remove obsolete TODO file

2020-06-11 Thread Paolo Bonzini
From: Thomas Huth The last real change to this file is from 2012, so it is very likely that this file is completely out-of-date and ignored today. Let's simply remove it to avoid confusion if someone finds it by accident. Signed-off-by: Thomas Huth Message-Id:

[PULL 085/115] sysemu/hvf: Only declare hvf_allowed when HVF is available

2020-06-11 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé When HVF is not available, the hvf_allowed variable does not exist. Reviewed-by: Edgar E. Iglesias Reviewed-by: Cornelia Huck Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Roman Bolshakov Message-Id: <20200526172427.17460-4-f4...@amsat.org> Signed-off-by:

[PULL 110/115] exec/memory: Remove unused MemoryRegionMmio type

2020-06-11 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Since commit 62a0db942dec ('memory: Remove old_mmio accessors') this structure is unused. Remove it. Suggested-by: Peter Maydell Reviewed-by: Peter Maydell Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id:

[PULL 112/115] exec/cpu-common: Move MUSB specific typedefs to 'hw/usb/hcd-musb.h'

2020-06-11 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé The CPUReadMemoryFunc/CPUWriteMemoryFunc typedefs are legacy remnant from before the conversion to MemoryRegions. Since they are now only used in tusb6010.c and hcd-musb.c, move them to "hw/usb/musb.h" and rename them appropriately. Suggested-by: Peter Maydell

[PULL 080/115] exec: set map length to zero when returning NULL

2020-06-11 Thread Paolo Bonzini
From: Prasad J Pandit When mapping physical memory into host's virtual address space, 'address_space_map' may return NULL if BounceBuffer is in_use. Set and return '*plen = 0' to avoid later NULL pointer dereference. Reported-by: Alexander Bulekov Fixes:

[PULL 081/115] target/i386: fix IEEE x87 floating-point exception raising

2020-06-11 Thread Paolo Bonzini
From: Joseph Myers Most x87 instruction implementations fail to raise the expected IEEE floating-point exceptions because they do nothing to convert the exception state from the softfloat machinery into the exception flags in the x87 status word. There is special-case handling of division to

[PULL 107/115] target/i386: sev: Remove redundant handle field

2020-06-11 Thread Paolo Bonzini
From: David Gibson The user can explicitly specify a handle via the "handle" property wired to SevGuestState::handle. That gets passed to the KVM_SEV_LAUNCH_START ioctl() which may update it, the final value being copied back to both SevGuestState::handle and SEVState::handle. AFAICT, nothing

[PULL 092/115] i386: hvf: Drop fetch_rip from HVFX86EmulatorState

2020-06-11 Thread Paolo Bonzini
From: Roman Bolshakov The field is used to print address of instructions that have no parser in decode_invalid(). RIP from VMCS is saved into fetch_rip before decoding starts but it's also saved into env->eip in load_regs(). Therefore env->eip can be used instead of fetch_rip. While at it,

[PULL 058/115] disas: Let disas::read_memory() handler return EIO on error

2020-06-11 Thread Paolo Bonzini
From: Philippe Mathieu-Daudé Both cpu_memory_rw_debug() and address_space_read() return an error on failed transaction. Check the returned value, and return EIO in case of error. Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Paolo Bonzini --- disas.c | 13 - 1 file

[PULL 082/115] target/i386: correct fix for pcmpxstrx substring search

2020-06-11 Thread Paolo Bonzini
From: Joseph Myers This corrects a bug introduced in my previous fix for SSE4.2 pcmpestri / pcmpestrm / pcmpistri / pcmpistrm substring search, commit ae35eea7e4a9f21dd147406dfbcd0c4c6aaf2a60. That commit fixed a bug that showed up in four GCC tests with one libc implementation. The tests in

[PULL 073/115] replay: implement fair mutex

2020-06-11 Thread Paolo Bonzini
From: Pavel Dovgalyuk In record/replay icount mode main loop thread and vCPU thread do not perform simultaneously. They take replay mutex to synchronize the actions. Sometimes vCPU thread waits for locking the mutex for very long time, because main loop releases the mutex and takes it back

[PULL 089/115] i386: hvf: Drop unused variable

2020-06-11 Thread Paolo Bonzini
From: Roman Bolshakov Signed-off-by: Roman Bolshakov Message-Id: <20200528193758.51454-5-r.bolsha...@yadro.com> Signed-off-by: Paolo Bonzini Reviewed-by: Philippe Mathieu-Daudé --- target/i386/hvf/x86.h | 1 - 1 file changed, 1 deletion(-) diff --git a/target/i386/hvf/x86.h

[PULL 100/115] target/i386: sev: Remove unused QSevGuestInfoClass

2020-06-11 Thread Paolo Bonzini
From: David Gibson This structure is nothing but an empty wrapper around the parent class, which by QOM conventions means we don't need it at all. Signed-off-by: David Gibson Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id:

  1   2   3   4   5   >