[PULL 33/57] target/arm: Move regime_el to internals.h

2020-06-26 Thread Peter Maydell
From: Richard Henderson We will shortly need this in mte_helper.c as well. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-22-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/internals.h | 36

[PULL 04/57] hw/i2c/core: Add i2c_try_create_slave() and i2c_realize_and_unref()

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé Extract i2c_try_create_slave() and i2c_realize_and_unref() from i2c_create_slave(). We can now set properties on a I2CSlave before it is realized. This is in line with the recent qdev/QOM changes merged in commit 6675a653d2e. Reviewed-by: Corey Minyard

[PULL 21/57] target/arm: Implement the IRG instruction

2020-06-26 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-10-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/helper-a64.h| 2 ++ target/arm/internals.h | 5 +++ target/arm/mte_helper.c| 72

[PULL 28/57] target/arm: Implement the STGP instruction

2020-06-26 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-17-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate-a64.c | 29 ++--- 1 file changed, 26 insertions(+), 3

[PULL 30/57] target/arm: Simplify DC_ZVA

2020-06-26 Thread Peter Maydell
From: Richard Henderson Now that we know that the operation is on a single page, we need not loop over pages while probing. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-19-richard.hender...@linaro.org Signed-off-by: Peter Maydell ---

[PULL 05/57] hw/misc/pca9552: Rename 'nr_leds' as 'pin_count'

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé The PCA9552 device does not expose LEDs, but simple pins to connnect LEDs to. To be clearer with the device model, rename 'nr_leds' as 'pin_count'. Reviewed-by: Cédric Le Goater Signed-off-by: Philippe Mathieu-Daudé Tested-by: Cédric Le Goater Message-id:

[PULL 24/57] target/arm: Implement the GMI instruction

2020-06-26 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-13-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate-a64.c | 15 +++ 1 file changed, 15 insertions(+) diff --git

[PULL 27/57] target/arm: Implement LDG, STG, ST2G instructions

2020-06-26 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-16-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/helper-a64.h| 7 ++ target/arm/helper.h| 2 + target/arm/mte_helper.c| 194

[PULL 19/57] target/arm: Add MTE system registers

2020-06-26 Thread Peter Maydell
From: Richard Henderson This is TFSRE0_EL1, TFSR_EL1, TFSR_EL2, TFSR_EL3, RGSR_EL1, GCR_EL1, GMID_EL1, and PSTATE.TCO. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-8-richard.hender...@linaro.org Signed-off-by: Peter Maydell ---

[PULL 01/57] hw/arm/aspeed: Remove extraneous MemoryRegion object owner

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé I'm confused by this code, 'bmc' is created as: bmc = g_new0(AspeedBoardState, 1); Then we use it as QOM owner for different MemoryRegion objects. But looking at memory_region_init_ram (similarly for ROM): void memory_region_init_ram(MemoryRegion *mr,

[PULL 23/57] target/arm: Implement the ADDG, SUBG instructions

2020-06-26 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-12-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/helper-a64.h| 1 + target/arm/internals.h | 9 +++ target/arm/mte_helper.c| 10

[PULL 18/57] target/arm: Add DISAS_UPDATE_NOCHAIN

2020-06-26 Thread Peter Maydell
From: Richard Henderson Add an option that writes back the PC, like DISAS_UPDATE_EXIT, but does not exit back to the main loop. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-7-richard.hender...@linaro.org Signed-off-by: Peter Maydell ---

[PULL 14/57] target/arm: Improve masking of SCR RES0 bits

2020-06-26 Thread Peter Maydell
From: Richard Henderson Protect reads of aa64 id registers with ARM_CP_STATE_AA64. Use this as a simpler test than arm_el_is_aa64, since EL3 cannot change mode. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-3-richard.hender...@linaro.org

[PULL 00/57] target-arm queue

2020-06-26 Thread Peter Maydell
Mostly this is RTH's memtag series, but there are also some cleanups from Philippe. thanks -- PMM The following changes since commit 10f7ffabf9c507fc02382b89912003b1c43c3231: Merge remote-tracking branch 'remotes/mcayland/tags/qemu-macppc-20200626' into staging (2020-06-26 12:14:18 +0100

Re: [PATCH v9 00/46] target/arm: Implement ARMv8.5-MemTag, system mode

2020-06-26 Thread Peter Maydell
On Fri, 26 Jun 2020 at 04:31, Richard Henderson wrote: > > Version 9 incorporates some review: > > * Adjust some commentary. > * Added an assert for tbi in aarch64_tr_init_disas_context > * Split arm,armv8.5-memtag to a new, final, patch. We might > just leave this one out of qemu

[PULL 17/57] target/arm: Rename DISAS_UPDATE to DISAS_UPDATE_EXIT

2020-06-26 Thread Peter Maydell
From: Richard Henderson Emphasize that the is_jmp option exits to the main loop. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-6-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/translate.h | 14 --

[PULL 15/57] target/arm: Add support for MTE to SCTLR_ELx

2020-06-26 Thread Peter Maydell
From: Richard Henderson This does not attempt to rectify all of the res0 bits, but does clear the mte bits when not enabled. Since there is no high-part mapping of SCTLR, aa32 mode cannot write to these bits. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id:

[PULL 08/57] hw/misc/pca9552: Add a 'description' property for debugging purpose

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé Add a description field to distinguish between multiple devices. Reviewed-by: Cédric Le Goater Signed-off-by: Philippe Mathieu-Daudé Tested-by: Cédric Le Goater Message-id: 20200623072723.6324-6-f4...@amsat.org Signed-off-by: Peter Maydell ---

[PULL 16/57] target/arm: Add support for MTE to HCR_EL2 and SCR_EL3

2020-06-26 Thread Peter Maydell
From: Richard Henderson Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20200626033144.790098-5-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/helper.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git

[PULL 09/57] hw/misc/pca9552: Trace GPIO High/Low events

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé Add a trivial representation of the PCA9552 GPIOs. Example booting obmc-phosphor-image: $ qemu-system-arm -M witherspoon-bmc -trace pca955x_gpio_status 1592689902.327837:pca955x_gpio_status pca-unspecified GPIOs 0-15 [*...]

[PULL 06/57] hw/misc/pca9552: Rename generic code as pca955x

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé Various code from the PCA9552 device model is generic to the PCA955X family. We'll split the generic code in a base class in the next commit. To ease review, first do a dumb renaming. Reviewed-by: Cédric Le Goater Signed-off-by: Philippe Mathieu-Daudé Tested-by:

[PULL 11/57] hw/misc/pca9552: Trace GPIO change events

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé Emit a trace event when a GPIO change its state. Example booting obmc-phosphor-image: $ qemu-system-arm -M witherspoon-bmc -trace pca955x_gpio_change 1592690552.687372:pca955x_gpio_change pca1 GPIO id:0 status: 0 -> 1 1592690552.690169:pca955x_gpio_change

[PULL 07/57] hw/misc/pca9552: Add generic PCA955xClass, parent of TYPE_PCA9552

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé Extract the code common to the PCA955x family in PCA955xClass, keeping the PCA9552 specific parts into pca9552_class_init(). Remove the 'TODO' comment added in commit 5141d4158cf. Suggested-by: Cédric Le Goater Reviewed-by: Cédric Le Goater Signed-off-by: Philippe

[PULL 03/57] hw/arm/aspeed: QOM'ify AspeedMachineState

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé AspeedMachineState seems crippled. We use incorrectly 2 different structures to do the same thing. Merge them altogether: - Move AspeedMachine fields to AspeedMachineState - AspeedMachineState is now QOM - Remove unused AspeedMachine structure Signed-off-by:

[PULL 12/57] hw/misc/pca9552: Model qdev output GPIOs

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé The PCA9552 has 16 GPIOs which can be used as input, output or PWM mode. QEMU models the output GPIO with the qemu_irq type. Let the device expose the 16 GPIOs to allow us to later connect LEDs to these outputs. Reviewed-by: Cédric Le Goater Signed-off-by: Philippe

[PULL 02/57] hw/arm/aspeed: Rename AspeedBoardState as AspeedMachineState

2020-06-26 Thread Peter Maydell
From: Philippe Mathieu-Daudé To have a more consistent naming, rename AspeedBoardState as AspeedMachineState. Suggested-by: Cédric Le Goater Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Cédric Le Goater Message-id: 20200623072132.2868-3-f4...@amsat.org Signed-off-by: Peter Maydell ---

Re: [PATCH v2 00/16] Crazy shit around -global (pardon my french)

2020-06-26 Thread John Snow
On 6/25/20 12:45 AM, Markus Armbruster wrote: > John Snow writes: > >> On 6/22/20 5:42 AM, Markus Armbruster wrote: >>> There are three ways to configure backends: >>> >>> * -nic, -serial, -drive, ... (onboard devices) >>> >>> * Set the property with -device, or, if you feel masochistic, with

Re: [PATCH QEMU v25 09/17] vfio: Add load state functions to SaveVMHandlers

2020-06-26 Thread Dr. David Alan Gilbert
* Alex Williamson (alex.william...@redhat.com) wrote: > On Sun, 21 Jun 2020 01:51:18 +0530 > Kirti Wankhede wrote: > > > Sequence during _RESUMING device state: > > While data for this device is available, repeat below steps: > > a. read data_offset from where user application should write

Re: [PATCH QEMU v25 05/17] vfio: Add VM state change handler to know state of VM

2020-06-26 Thread Dr. David Alan Gilbert
* Kirti Wankhede (kwankh...@nvidia.com) wrote: > > > On 6/23/2020 4:20 AM, Alex Williamson wrote: > > On Sun, 21 Jun 2020 01:51:14 +0530 > > Kirti Wankhede wrote: > > > > > VM state change handler gets called on change in VM's state. This is used > > > to set > > > VFIO device state to

Re: [PATCH] hw/timer/a9gtimer: Clear pending interrupt, after the clear of Event flag

2020-06-26 Thread Peter Maydell
On Tue, 16 Jun 2020 at 08:04, Václav Vanc wrote: > > On 6/15/20 1:04 PM, Peter Maydell wrote: > We must distinguish between two cases: > 1, Auto-increment is disabled. > I just run some test on SABRE Lite (i.MX6) board. > I had auto-increment disabled, I verified, that GIC is configured for >

Re: [PATCH QEMU v25 13/17] vfio: create mapped iova list when vIOMMU is enabled

2020-06-26 Thread Peter Xu
On Thu, Jun 25, 2020 at 11:40:39AM -0600, Alex Williamson wrote: > On Thu, 25 Jun 2020 20:04:08 +0530 > Kirti Wankhede wrote: > > > On 6/25/2020 12:25 AM, Alex Williamson wrote: > > > On Sun, 21 Jun 2020 01:51:22 +0530 > > > Kirti Wankhede wrote: > > > > > >> Create mapped iova list when

Re: [RFC PATCH 1/3] hw/i2c/smbus_eeprom: Set QOM parent

2020-06-26 Thread Philippe Mathieu-Daudé
On 6/26/20 4:26 PM, BALATON Zoltan wrote: > On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: >> On 6/26/20 4:03 PM, BALATON Zoltan wrote: >>> On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: + Eduardo / Mark / Edgard / Alistair / Fred for QOM design. On 6/26/20 12:54 PM, BALATON

Re: [PATCH 1/3] haiku build fix second batch

2020-06-26 Thread David CARLIER
No idea if I ll have energy to redo all over again I lost previous patches. Fine no problems... On Fri, 26 Jun 2020 at 15:33, Peter Maydell wrote: > > On Fri, 26 Jun 2020 at 15:10, David CARLIER wrote: > > > > I sent again minux the ones already approved. > > Sorry, I intended that you should

Re: [PATCH 06/46] error: Avoid error_propagate() when error is not used here

2020-06-26 Thread Vladimir Sementsov-Ogievskiy
24.06.2020 19:43, Markus Armbruster wrote: When all we do with an Error we receive into a local variable is propagating to somewhere else, we can just as well receive it there right away. Coccinelle script: @@ identifier fun, err, errp; expression list args; @@ -

Re: [PATCH 1/3] haiku build fix second batch

2020-06-26 Thread Peter Maydell
On Fri, 26 Jun 2020 at 15:10, David CARLIER wrote: > > I sent again minux the ones already approved. Sorry, I intended that you should send the whole series, including the ones which I've reviewed. (Where I gave a reviewed-by tag, include that tag in the commit message when you resend the

Re: [PATCH] migration/block-dirty-bitmap: fix add_bitmaps_to_list

2020-06-26 Thread Eric Blake
On 6/26/20 8:06 AM, Vladimir Sementsov-Ogievskiy wrote: We shouldn't fail, if found unnamed bitmap in a unnamed node or node We shouldn't fail when finding an unnamed with auto-generated node name, as bitmap migration ignores such bitmaps at all. such bitmaps in the first place. Fixes:

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

2020-06-26 Thread Andrey Shinkevich
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 commit ID: c8bb23cbdbe32f5) The case does

Re: [PATCH QEMU v25 07/17] vfio: Register SaveVMHandlers for VFIO device

2020-06-26 Thread Dr. David Alan Gilbert
* Kirti Wankhede (kwankh...@nvidia.com) wrote: > Define flags to be used as delimeter in migration file stream. > Added .save_setup and .save_cleanup functions. Mapped & unmapped migration > region from these functions at source during saving or pre-copy phase. > Set VFIO device state depending on

Re: [PATCH 5/5] haiku build fix

2020-06-26 Thread Eric Blake
On 6/26/20 7:54 AM, David CARLIER wrote: From 6fa7a4108236f513201192654e07c7044a3d7e58 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:51:37 + Subject: [PATCH 3/3] qemu_init_exec_dir Haiku implementation meta-comment: the original subject line says 5/5, while this

Re: [RFC PATCH 2/3] hw/i2c/smbus_eeprom: Add description based on child name

2020-06-26 Thread Philippe Mathieu-Daudé
On 6/26/20 1:00 PM, BALATON Zoltan wrote: > On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: >> Suggested-by: Markus Armbruster >> Signed-off-by: Philippe Mathieu-Daudé >> --- >> hw/i2c/smbus_eeprom.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/hw/i2c/smbus_eeprom.c

Re: [RFC PATCH 1/3] hw/i2c/smbus_eeprom: Set QOM parent

2020-06-26 Thread BALATON Zoltan
On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: On 6/26/20 4:03 PM, BALATON Zoltan wrote: On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: + Eduardo / Mark / Edgard / Alistair / Fred for QOM design. On 6/26/20 12:54 PM, BALATON Zoltan wrote: On Fri, 26 Jun 2020, BALATON Zoltan wrote:

Re: [PATCH 1/5] haiku build fix

2020-06-26 Thread Eric Blake
On 6/26/20 5:07 AM, David CARLIER wrote: From 4d0933384d2bfcd0fc8c4c06eed2d07f3f1b7f8b Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 10:35:40 + Subject: [PATCH 1/5] Haiku build fix enabling BSD symbols. Signed-off-by: David Carlier --- configure | 4 ++-- 1 file

Re: [PATCH QEMU v25 07/17] vfio: Register SaveVMHandlers for VFIO device

2020-06-26 Thread Dr. David Alan Gilbert
* Alex Williamson (alex.william...@redhat.com) wrote: > On Wed, 24 Jun 2020 00:51:06 +0530 > Kirti Wankhede wrote: > > > On 6/23/2020 4:20 AM, Alex Williamson wrote: > > > On Sun, 21 Jun 2020 01:51:16 +0530 > > > Kirti Wankhede wrote: > > > > > >> Define flags to be used as delimeter in

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

2020-06-26 Thread Ahmed Karaman
On Thu, Jun 25, 2020 at 11:45 AM Aleksandar Markovic wrote: > > сре, 24. јун 2020. у 17:32 Ahmed Karaman > је написао/ла: > > > > Syntax: > > topN_perf.py [-h] [-n] -- \ > > [] \ > > [] > > > > [-h] - Print the script arguments help message. > > [-n] -

Re: [RFC PATCH 1/3] hw/i2c/smbus_eeprom: Set QOM parent

2020-06-26 Thread Philippe Mathieu-Daudé
On 6/26/20 4:03 PM, BALATON Zoltan wrote: > On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: >> + Eduardo / Mark / Edgard / Alistair / Fred for QOM design. >> >> On 6/26/20 12:54 PM, BALATON Zoltan wrote: >>> On Fri, 26 Jun 2020, BALATON Zoltan wrote: On Fri, 26 Jun 2020, Philippe

Re: [PATCH QEMU v25 15/17] vfio: Add ioctl to get dirty pages bitmap during dma unmap.

2020-06-26 Thread Dr. David Alan Gilbert
* Alex Williamson (alex.william...@redhat.com) wrote: > On Thu, 25 Jun 2020 20:31:12 +0530 > Kirti Wankhede wrote: > > > On 6/25/2020 12:26 AM, Alex Williamson wrote: > > > On Sun, 21 Jun 2020 01:51:24 +0530 > > > Kirti Wankhede wrote: > > > > > >> With vIOMMU, IO virtual address range can

Re: [PATCH 1/3] haiku build fix second batch

2020-06-26 Thread David CARLIER
I sent again minux the ones already approved. On Fri, 26 Jun 2020 at 14:39, Peter Maydell wrote: > > On Fri, 26 Jun 2020 at 14:02, David CARLIER wrote: > > > > From 95ef79ddff73eebd1f1bec6673c2c68209fab107 Mon Sep 17 00:00:00 2001 > > From: David Carlier > > Date: Fri, 26 Jun 2020 13:56:14

Re: [PATCH] timer: Handle decrements of PIT counter

2020-06-26 Thread Kevin O'Connor
On Fri, Jun 26, 2020 at 04:09:57PM +0300, Roman Bolshakov wrote: > On Tue, Jun 23, 2020 at 11:00:24PM -0400, Kevin O'Connor wrote: > > Good catch. Could we fix it using the patch below instead though? > > > > -Kevin > > > > > > --- a/src/hw/timer.c > > +++ b/src/hw/timer.c > > @@ -180,7 +180,7

[PATCH 3/3] syscall skipped for haiku used only in qemu_signalfd anyway

2020-06-26 Thread David CARLIER
>From a548479cab82200d9df33a70f24aeebb00eb70ad Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:58:34 + Subject: [PATCH 3/3] syscall skipped for haiku used only in qemu_signalfd anyway Signed-off-by: David Carlier --- util/compatfd.c | 2 ++ 1 file changed, 2

[PATCH 2/3] drm build ignored for Haiku.

2020-06-26 Thread David CARLIER
>From 338a25285d945c891eb4fa34a18cb45b8b05dbb1 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:57:08 + Subject: [PATCH 2/3] drm build ignored for Haiku. Signed-off-by: David Carlier --- util/Makefile.objs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [PATCH 2/3] define SIGIO for haiku

2020-06-26 Thread Philippe Mathieu-Daudé
Maybe "Define SIGIO on Haiku OS" as patch subject... On 6/26/20 4:03 PM, David CARLIER wrote: > From d36aba6190152c626f668fef3704ee6b61bfc323 Mon Sep 17 00:00:00 2001 > From: David Carlier > Date: Fri, 26 Jun 2020 13:45:04 + > Subject: [PATCH 2/3] define SIGIO for haiku > > Signed-off-by:

[PATCH 1/3] Include endian.h for Haiku to solve bswap* macros build

2020-06-26 Thread David CARLIER
>From 95ef79ddff73eebd1f1bec6673c2c68209fab107 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:56:14 + Subject: [PATCH 1/3] Include endian.h for Haiku to solve bswap* macros build failure. Signed-off-by: David Carlier --- include/qemu/bswap.h | 2 ++ 1 file changed,

[PATCH 0/3] second batch of haiku build fix

2020-06-26 Thread David CARLIER
>From a548479cab82200d9df33a70f24aeebb00eb70ad Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 15:04:30 + Subject: [PATCH 0/3] second batch of haiku build fix David Carlier (3): Include endian.h for Haiku to solve bswap* macros build failure. drm build ignored for

Re: [PATCH] MAINTAINERS: Add an entry for OpenSBI firmware

2020-06-26 Thread Philippe Mathieu-Daudé
On 6/26/20 3:13 PM, Bin Meng wrote: > List me as the maintainer for OpenSBI firmware related files. > > Signed-off-by: Bin Meng > --- > > MAINTAINERS | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 1b40446..b0e2dd2 100644 > --- a/MAINTAINERS >

Re: [REPORT] [GSoC - TCG Continuous Benchmarking] [#1] Measuring Basic Performance Metrics of QEMU

2020-06-26 Thread Ahmed Karaman
On Thu, Jun 25, 2020 at 3:38 PM Aleksandar Markovic wrote: > > In introductory section on Callgrind, maybe you could add the general > form of command line that performs profiling using Callgrind, > something like this: > > valgrind --tool=callgrind [callgrind options] prog [prog options] > > Not

[PATCH 3/3] qemu_init_exec_dir Haiku implementation

2020-06-26 Thread David CARLIER
>From 6fa7a4108236f513201192654e07c7044a3d7e58 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:51:37 + Subject: [PATCH 3/3] qemu_init_exec_dir Haiku implementation Signed-off-by: David Carlier --- util/oslib-posix.c | 19 +++ 1 file changed, 19

Re: [RFC PATCH 1/3] hw/i2c/smbus_eeprom: Set QOM parent

2020-06-26 Thread BALATON Zoltan
On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: + Eduardo / Mark / Edgard / Alistair / Fred for QOM design. On 6/26/20 12:54 PM, BALATON Zoltan wrote: On Fri, 26 Jun 2020, BALATON Zoltan wrote: On Fri, 26 Jun 2020, Philippe Mathieu-Daudé wrote: Suggested-by: Markus Armbruster

[PATCH 2/3] define SIGIO for haiku

2020-06-26 Thread David CARLIER
>From d36aba6190152c626f668fef3704ee6b61bfc323 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:45:04 + Subject: [PATCH 2/3] define SIGIO for haiku Signed-off-by: David Carlier --- include/qemu/osdep.h | 4 1 file changed, 4 insertions(+) diff --git

[PATCH 1/3] : haiku check sys signal.h presence

2020-06-26 Thread David CARLIER
>From a4dfa918e6eea7a5ccc4375d83d1f0162e3bb122 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:43:59 + Subject: [PATCH 1/3] check sys/signal.h presence Signed-off-by: David Carlier --- configure | 8 hw/xen/xen-legacy-backend.c | 1 -

Re: [PATCH 01/17] block/nvme: Avoid further processing if trace event not enabled

2020-06-26 Thread Philippe Mathieu-Daudé
On 6/26/20 12:36 PM, Stefan Hajnoczi wrote: > On Thu, Jun 25, 2020 at 08:48:22PM +0200, Philippe Mathieu-Daudé wrote: >> Avoid further processing if TRACE_NVME_SUBMIT_COMMAND_RAW is >> not enabled. > > Why? > > This saves 8 trace events, each with 8 arguments. I guess the intent is > to improve

[PATCH v2 0/3] following up haiku build fix

2020-06-26 Thread David CARLIER
>From 6fa7a4108236f513201192654e07c7044a3d7e58 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 14:48:28 + Subject: [PATCH 0/3] following up haiku build fix David Carlier (3): check sys/signal.h presence define SIGIO for haiku qemu_init_exec_dir Haiku implementation

QEMU | Pipeline #160473765 has failed for master | 10f7ffab

2020-06-26 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: 10f7ffab ( https://gitlab.com/qemu-project/qemu/-/commit/10f7ffabf9c507fc02382b89912003b1c43c3231 ) Commit Message: Merge

Re: [PATCH RESEND 5/9] hw/arm/smmuv3: Store the starting level in SMMUTransTableInfo

2020-06-26 Thread Auger Eric
Hi Peter, On 6/25/20 5:15 PM, Peter Maydell wrote: > On Thu, 11 Jun 2020 at 17:15, Eric Auger wrote: >> >> Compute the starting level on CD decoding and store it >> into SMMUTransTableInfo. We will need this information >> on IOTLB lookup so let's avoid to recompute it each time. >> >>

[PULL v2 09/31] exec: fetch the alignment of Linux devdax pmem character device nodes

2020-06-26 Thread Paolo Bonzini
From: Jingqi Liu If the backend file is devdax pmem character device, the alignment specified by the option 'align=NUM' in the '-object memory-backend-file' needs to match the alignment requirement of the devdax pmem character device. This patch uses the interfaces of libdaxctl to fetch the

[PULL v2 25/31] osdep: Make MIN/MAX evaluate arguments only once

2020-06-26 Thread Paolo Bonzini
From: Eric Blake I'm not aware of any immediate bugs in qemu where a second runtime evaluation of the arguments to MIN() or MAX() causes a problem, but proactively preventing such abuse is easier than falling prey to an unintended case down the road. At any rate, here's the conversation that

[PULL v2 00/31] Misc patches for 2020-06-24

2020-06-26 Thread Paolo Bonzini
The following changes since commit 5acc270a355120ce967ca1f1eeca0abbdb9303c8: Merge remote-tracking branch 'remotes/xtensa/tags/20200625-xtensa' into staging (2020-06-25 21:20:45 +0100) are available in the Git repository at: git://github.com/bonzini/qemu.git tags/for-upstream for you to

Re: [PATCH RESEND 1/9] hw/arm/smmu-common: Factorize some code in smmu_ptw_64()

2020-06-26 Thread Auger Eric
Hi Peter, On 6/25/20 4:49 PM, Peter Maydell wrote: > On Thu, 11 Jun 2020 at 17:15, Eric Auger wrote: >> >> Page and block PTE decoding can share some code. Let's >> first handle table PTE and factorize some code shared by >> page and block PTEs. >> >> Signed-off-by: Eric Auger >> --- >>

Re: [PATCH RESEND 6/9] hw/arm/smmu-common: Manage IOTLB block entries

2020-06-26 Thread Auger Eric
Hi Peter, On 6/25/20 5:30 PM, Peter Maydell wrote: > On Thu, 11 Jun 2020 at 17:16, Eric Auger wrote: >> >> At the moment each entry in the IOTLB corresponds to a page sized >> mapping (4K, 16K or 64K), even if the page belongs to a mapped >> block. In case of block mapping this unefficiently

Re: [PATCH RESEND 3/9] hw/arm/smmu: Simplify the IOTLB key format

2020-06-26 Thread Auger Eric
Hi Peter, On 6/25/20 5:03 PM, Peter Maydell wrote: > On Thu, 11 Jun 2020 at 17:15, Eric Auger wrote: >> >> Instead of using a Jenkins hash function to generate >> the key let's just use a 64 bit unsigned integer that >> contains the asid and the 40 upper bits of the iova. >> A maximum of 52-bit

[PATCH 04/13] iotests: update snapshot test for new output format

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch updates iotests that verify qemu monitor output. New output format for snapshot listing include ICOUNT column. Signed-off-by: Pavel Dovgalyuk --- 0 files changed diff --git a/tests/qemu-iotests/267.out b/tests/qemu-iotests/267.out index 8dddb4baa4..c4f3d8dd33

[PATCH 11/13] gdbstub: add reverse continue support in replay mode

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds support of the reverse continue operation for gdbstub. Reverse continue finds the last breakpoint that would happen in normal execution from the beginning to the current moment. Implementation of the reverse continue replays the execution twice: to find the

[PATCH 03/13] migration: introduce icount field for snapshots

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk Saving icount as a parameters of the snapshot allows navigation between them in the execution replay scenario. This information can be used for finding a specific snapshot for proceeding the recorded execution to the specific moment of the time. E.g., 'reverse step' action

[PATCH 02/13] qcow2: introduce icount field for snapshots

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch introduces the icount field for saving within the snapshot. It is required for navigation between the snapshots in record/replay mode. Signed-off-by: Pavel Dovgalyuk Acked-by: Kevin Wolf --- 0 files changed diff --git a/block/qcow2-snapshot.c

[PATCH 05/13] qapi: introduce replay.json for record/replay-related stuff

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds replay.json file. It will be used for adding record/replay-related data structures and commands. Signed-off-by: Pavel Dovgalyuk Reviewed-by: Markus Armbruster --- 0 files changed diff --git a/MAINTAINERS b/MAINTAINERS index 5dd86c7f94..ca5b1cf7f1 100644

[PATCH 10/13] gdbstub: add reverse step support in replay mode

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk GDB remote protocol supports two reverse debugging commands: reverse step and reverse continue. This patch adds support of the first one to the gdbstub. Reverse step is intended to step one instruction in the backwards direction. This is not possible in regular execution.

[PATCH 01/13] replay: provide an accessor for rr filename

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds an accessor function for the name of the record/replay log file. Adding an accessor instead of making variable global, prevents accidental modification of this variable by other modules. Signed-off-by: Pavel Dovgalyuk --- 0 files changed diff --git

[PATCH 07/13] replay: introduce breakpoint at the specified step

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch introduces replay_break, replay_delete_break qmp and hmp commands. These commands allow stopping at the specified instruction. It may be useful for debugging when there are some known events that should be investigated. replay_break command has one argument -

[PATCH 00/13] Reverse debugging

2020-06-26 Thread Pavel Dovgalyuk
GDB remote protocol supports reverse debugging of the targets. It includes 'reverse step' and 'reverse continue' operations. The first one finds the previous step of the execution, and the second one is intended to stop at the last breakpoint that would happen when the program is executed

[PATCH 09/13] replay: flush rr queue before loading the vmstate

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk Non-empty record/replay queue prevents saving and loading the VM state, because it includes pending bottom halves and block coroutines. But when the new VM state is loaded, we don't have to preserve the consistency of the current state anymore. Therefore this patch just

[PATCH 06/13] replay: introduce info hmp/qmp command

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch introduces 'info replay' monitor command and corresponding qmp request. These commands request the current record/replay mode, replay log file name, and the instruction count (number of recorded/replayed instructions). The instruction count can be used with the

[PATCH 13/13] tests/acceptance: add reverse debugging test

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This is a test for GDB reverse debugging commands: reverse step and reverse continue. Every test in this suite consists of two phases: record and replay. Recording saves the execution of some instructions and makes an initial VM snapshot to allow reverse execution. Replay

[PATCH 08/13] replay: implement replay-seek command

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch adds hmp/qmp commands replay_seek/replay-seek that proceed the execution to the specified instruction count. The command automatically loads nearest snapshot and replays the execution to find the desired instruction count. Signed-off-by: Pavel Dovgalyuk

[PATCH 12/13] replay: describe reverse debugging in docs/replay.txt

2020-06-26 Thread Pavel Dovgalyuk
From: Pavel Dovgalyuk This patch updates the documentation and describes usage of the reverse debugging in QEMU+GDB. Signed-off-by: Pavel Dovgalyuk --- 0 files changed diff --git a/docs/replay.txt b/docs/replay.txt index 70c27edb36..18d6169f3b 100644 --- a/docs/replay.txt +++

Re: [PATCH 1/3] haiku build fix second batch

2020-06-26 Thread Peter Maydell
On Fri, 26 Jun 2020 at 14:02, David CARLIER wrote: > > From 95ef79ddff73eebd1f1bec6673c2c68209fab107 Mon Sep 17 00:00:00 2001 > From: David Carlier > Date: Fri, 26 Jun 2020 13:56:14 + > Subject: [PATCH 1/3] Include endian.h for Haiku to solve bswap* macros build > failure. > >

Re: [PATCH 5/5] haiku build fix

2020-06-26 Thread Thomas Huth
On 26/06/2020 12.07, David CARLIER wrote: From 68d4d4312eccd212b4d2484e09425816ebd2346a Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 11:01:54 + Subject: [PATCH 5/5] Last chunk of build fix Signed-off-by: David Carlier --- include/qemu/bswap.h | 2 ++

Re: RFC: use VFIO over a UNIX domain socket to implement device offloading

2020-06-26 Thread Stefan Hajnoczi
On Thu, Jun 25, 2020 at 08:54:25PM -0700, John G Johnson wrote: > > > > On Jun 23, 2020, at 5:27 AM, Stefan Hajnoczi wrote: > > > > On Thu, Jun 18, 2020 at 02:38:04PM -0700, John G Johnson wrote: > >>> On Jun 15, 2020, at 3:49 AM, Stefan Hajnoczi wrote: > >>> An issue with file descriptor

Re: [PATCH v9 46/46] target/arm: Add arm,armv8.5-memtag to dtb

2020-06-26 Thread Peter Maydell
On Fri, 26 Jun 2020 at 04:32, Richard Henderson wrote: > > The mte-v4 linux arm kernel development branch requires these tags. > It is still an open question as to whether they will be required > for the final commit. > > Signed-off-by: Richard Henderson > --- > v9: Split from patch creating the

Re: [PATCH v9 44/46] target/arm: Add allocation tag storage for system mode

2020-06-26 Thread Peter Maydell
On Fri, 26 Jun 2020 at 04:32, Richard Henderson wrote: > > Look up the physical address for the given virtual address, > convert that to a tag physical address, and finally return > the host address that backs it. > > Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell thanks -- PMM

Re: [PATCH v3 0/2] linux-user: support of semtimedop syscall

2020-06-26 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20200626124612.58593-1-mky...@tachyum.com/ Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH v3 0/2] linux-user: support of semtimedop syscall Type: series Message-id:

[PATCH] MAINTAINERS: Add an entry for OpenSBI firmware

2020-06-26 Thread Bin Meng
List me as the maintainer for OpenSBI firmware related files. Signed-off-by: Bin Meng --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 1b40446..b0e2dd2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2609,6 +2609,13 @@ F:

Re: [PATCH v9 43/46] target/arm: Create tagged ram when MTE is enabled

2020-06-26 Thread Peter Maydell
On Fri, 26 Jun 2020 at 04:32, Richard Henderson wrote: > > Signed-off-by: Richard Henderson > --- > v5: Assign cs->num_ases to the final value first. > Downgrade to ID_AA64PFR1.MTE=1 if tag memory is not available. > v6: Add secure tag memory for EL3. > v8: Add arm,armv8.5-memtag. > v9:

Re: [PATCH v5 09/11] macio: Add dummy screamer register area

2020-06-26 Thread Mark Cave-Ayland
On 16/06/2020 14:47, BALATON Zoltan wrote: > The only thing this returns is an idle status so the firmware > continues, otherwise just ignores and logs access for debugging. This > is a stop gap until proper implementation of this device lands. > > Signed-off-by: BALATON Zoltan > --- >

Re: [PATCH] timer: Handle decrements of PIT counter

2020-06-26 Thread Roman Bolshakov
On Tue, Jun 23, 2020 at 11:00:24PM -0400, Kevin O'Connor wrote: > On Sat, Jun 13, 2020 at 02:19:12PM +0300, Roman Bolshakov wrote: > > There's a fallback to PIT if TSC is not present but it doesn't work > > properly. It prevents boot from floppy on isapc and 486 cpu [1][2]. > > > > SeaBIOS

Re: [PATCH v5 08/11] mac_oldworld: Add machine ID register

2020-06-26 Thread Mark Cave-Ayland
On 16/06/2020 14:47, BALATON Zoltan wrote: > The G3 beige machine has a machine ID register that is accessed by the > firmware to deternine the board config. Add basic emulation of it. > > Signed-off-by: BALATON Zoltan > --- > v4: Move MermoryRegion to MachineState, use constants > >

[PATCH] migration/block-dirty-bitmap: fix add_bitmaps_to_list

2020-06-26 Thread Vladimir Sementsov-Ogievskiy
We shouldn't fail, if found unnamed bitmap in a unnamed node or node with auto-generated node name, as bitmap migration ignores such bitmaps at all. Fixes: 82640edb88faa Fixes: 4ff5cc121b089 Signed-off-by: Vladimir Sementsov-Ogievskiy --- migration/block-dirty-bitmap.c | 6 +- 1 file

[PATCH 3/3] haiku build fix

2020-06-26 Thread David CARLIER
>From a548479cab82200d9df33a70f24aeebb00eb70ad Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:58:34 + Subject: [PATCH 3/3] syscall skipped for haiku used only in qemu_signalfd anyway Signed-off-by: David Carlier --- util/compatfd.c | 2 ++ 1 file changed, 2

Re: [PATCH] gitlab-ci: Fix the change rules after moving the YML files

2020-06-26 Thread Bin Meng
Hi Philippe, On Thu, Jun 25, 2020 at 11:28 PM Philippe Mathieu-Daudé wrote: > > On 6/25/20 5:16 PM, Thomas Huth wrote: > > The edk2.yml and opensbi.yml files have recently been moved/renamed, > > but the change has not been reflected in the rules in the YML files > > yet. > > > > Fixes:

Re: [PULL 00/12] Block patches

2020-06-26 Thread Stefan Hajnoczi
On Fri, Jun 26, 2020 at 11:49 AM Peter Maydell wrote: > > On Fri, 26 Jun 2020 at 11:25, Stefan Hajnoczi wrote: > > > > On Thu, Jun 25, 2020 at 02:31:14PM +0100, Peter Maydell wrote: > > > On Wed, 24 Jun 2020 at 11:02, Stefan Hajnoczi wrote: > > > > > > > > The following changes since commit >

Re: [PATCH v5 07/11] mac_oldworld: Map macio to expected address at reset

2020-06-26 Thread Mark Cave-Ayland
On 16/06/2020 14:47, BALATON Zoltan wrote: > Add a reset function that maps macio to the address expected by the > firmware of the board at startup. > > Signed-off-by: BALATON Zoltan > --- > hw/ppc/mac.h | 12 > hw/ppc/mac_oldworld.c | 15 ++- > 2 files

[PATCH 2/3] haiku build fix second batch

2020-06-26 Thread David CARLIER
>From 338a25285d945c891eb4fa34a18cb45b8b05dbb1 Mon Sep 17 00:00:00 2001 From: David Carlier Date: Fri, 26 Jun 2020 13:57:08 + Subject: [PATCH 2/3] drm build ignored for Haiku. Signed-off-by: David Carlier --- util/Makefile.objs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

<    1   2   3   4   5   6   >