Re: [PATCH-for-5.1] hw/misc/milkymist-pfpu: Fix pFPU region size

Le 19/07/2020 à 19:23, Michael Walle a écrit : > Hi Philippe, > > Am 2020-07-18 11:37, schrieb Philippe Mathieu-Daudé: >> The last microcode word (address 0x6000.6ffc) is not reachable. >> Correct the programmable FPU I/O size (which is 4 KiB) to be >> able to use all the microcode area. >> >> Sig

Re: [PATCH 9/9] vhost-vdpa: allow pre-opend file descriptor

On 2020/8/31 下午7:16, Cindy Lu wrote: On Mon, 31 Aug 2020 at 16:30, Jason Wang wrote: This patch allows to initialize vhost-vdpa network backend with pre opened vhost-vdpa file descriptor. This is useful for running unprivileged qemu through libvirt. Cc: Eric Blake Cc: Markus Armbruster Sig

Re: [PATCH] hw/net/can: Add missing fallthrough statements

Le 14/07/2020 à 00:09, Pavel Pisa a écrit : > Hello Laurent and others, > > On Monday 06 of July 2020 18:35:50 Laurent Vivier wrote: >> Le 30/06/2020 à 09:55, Thomas Huth a écrit : >>> Add fallthrough annotations to be able to compile the code without >>> warnings when using -Wimplicit-fallthrough

Re: [PATCH 12/12] target/cris: Remove superfluous breaks

Le 13/07/2020 à 11:39, Thomas Huth a écrit : > On 13/07/2020 11.05, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove superfluous breaks, as there is a "return" before them. >> >> Signed-off-by: Liao Pingfang >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> target/cr

Re: [PATCH 11/12] target/sh4: Remove superfluous breaks

Le 13/07/2020 à 11:38, Thomas Huth a écrit : > On 13/07/2020 11.05, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove superfluous breaks, as there is a "return" before them. >> >> Signed-off-by: Liao Pingfang >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> target/sh

Re: [PATCH 08/12] block/vmdk: Remove superfluous breaks

Le 13/07/2020 à 11:36, Thomas Huth a écrit : > On 13/07/2020 11.05, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove superfluous breaks, as there is a "return" before them. >> >> Signed-off-by: Liao Pingfang >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> block/vmd

Re: [PATCH 09/12] hw: Remove superfluous breaks

Le 13/07/2020 à 11:37, Thomas Huth a écrit : > On 13/07/2020 11.05, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove superfluous breaks, as there is a "return" before them. >> >> Signed-off-by: Liao Pingfang >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> hw/block/

Re: [PATCH 07/12] vnc: Remove the superfluous break

Le 13/07/2020 à 11:35, Thomas Huth a écrit : > On 13/07/2020 11.04, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove the superfluous break, as there is a "return" before. >> >> Signed-off-by: Liao Pingfang a >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> ui/vnc-enc

Re: [PATCH 04/12] scsi: Remove superfluous breaks

Le 13/07/2020 à 11:33, Thomas Huth a écrit : > On 13/07/2020 11.04, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove superfluous breaks, as there is a "return" before them. >> >> Signed-off-by: Liao Pingfang >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> scsi/uti

Re: [PATCH 02/12] target/ppc: Remove superfluous breaks

Le 13/07/2020 à 11:30, Thomas Huth a écrit : > On 13/07/2020 11.03, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove superfluous breaks, as there is a "return" before them. >> >> Signed-off-by: Liao Pingfang >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> target/p

Re: [PATCH 01/12] target/arm/kvm: Remove superfluous break

Le 13/07/2020 à 11:30, Thomas Huth a écrit : > On 13/07/2020 11.03, Yi Wang wrote: >> From: Liao Pingfang >> >> Remove superfluous break. >> >> Signed-off-by: Liao Pingfang >> Signed-off-by: Yi Wang >> Reviewed-by: Philippe Mathieu-Daudé >> --- >> target/arm/kvm64.c | 1 - >> 1 file changed,

Re: [PULL 06/13] Introduce a new flag for i440fx to disable PCI hotplug on the root bus

On Fri, Aug 28, 2020 at 6:40 PM Julia Suvorova wrote: > > On Fri, Aug 28, 2020 at 11:53 AM Ani Sinha wrote: > > > > > > Ani > > On Aug 28, 2020, 15:19 +0530, Igor Mammedov , wrote: > > > > On Thu, 27 Aug 2020 23:29:34 +0530 > > > > Ani Sinha wrote: > > > > > > On Thu, Aug 27, 2020 at 11:11 PM Ig

Re: [PATCH] cirrus: handle wraparound in cirrus_invalidate_region

Gerd Hoffmann 于2020年9月1日周二 下午1:15写道： > > Hi, > > > > > > off_cur_end = ((off_cur + bytesperline - 1) & > > > > > s->cirrus_addr_mask) + 1; > [ ... ] > > > > > +memory_region_set_dirty(&s->vga.vram, 0, off_cur_end); > > > > > > > > And here be 'off_cur_end -1' > > > > > > --

[PATCH 04/13] cutils: introduce get_relocated_path

Add the function that will compute a relocated version of the directories in CONFIG_QEMU_*DIR and CONFIG_QEMU_*PATH. Signed-off-by: Paolo Bonzini --- include/qemu/cutils.h | 12 + meson.build | 4 +-- util/cutils.c | 61 +++ 3 fi

[PATCH 10/13] qemu-bridge-helper: relocate path to default ACL

Signed-off-by: Paolo Bonzini --- qemu-bridge-helper.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/qemu-bridge-helper.c b/qemu-bridge-helper.c index 88b26747fc..a26e1663f0 100644 --- a/qemu-bridge-helper.c +++ b/qemu-bridge-helper.c @@ -40,6 +40,7 @@ #endif #in

[PATCH 11/13] qga: relocate path to default configuration and hook

Signed-off-by: Paolo Bonzini --- qga/main.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/qga/main.c b/qga/main.c index 3febf3b0fd..740f5f7303 100644 --- a/qga/main.c +++ b/qga/main.c @@ -29,6 +29,7 @@ #include "qapi/error.h" #include "channel.h" #include "qemu/bs

[PATCH 13/13] configure: use a platform-neutral prefix

Now that the installation is relocatable, there is no need to compile a Windows-format prefix into Win32 binaries. Instead, the prefix will only be used to compute installation-relative paths, and it can be any string. Drop the "Program Files" path completely: it is only usable on English version

[PATCH 00/13] Make QEMU installation relocatable

Right now, the installation of QEMU is not relocatable; there is a local hack in os_find_datadir() so that Windows binaries look for ROMs in the executable directory, but that has several limitations: - it does not extend to configuration files, icons, etc. - it does not allow changing the data d

[PATCH 06/13] module: relocate path to modules

Signed-off-by: Paolo Bonzini --- util/module.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/util/module.c b/util/module.c index aed04b578a..c117d770fd 100644 --- a/util/module.c +++ b/util/module.c @@ -19,6 +19,7 @@ #endif #include "qemu/queue.h" #include "qemu/module.

[PATCH 08/13] vl: relocate paths to data directories

As an additional advantage, the logic is now unified between POSIX and Win32 systems. Signed-off-by: Paolo Bonzini --- include/qemu-common.h | 1 - include/sysemu/sysemu.h | 2 +- os-posix.c | 20 os-win32.c | 11 --- softmmu/vl.c

[PATCH 12/13] ui: relocate paths to icons and translations

Signed-off-by: Paolo Bonzini --- ui/gtk.c | 10 -- ui/sdl2.c | 9 +++-- 2 files changed, 15 insertions(+), 4 deletions(-) diff --git a/ui/gtk.c b/ui/gtk.c index b0cc08ad6d..597f829ad9 100644 --- a/ui/gtk.c +++ b/ui/gtk.c @@ -51,6 +51,7 @@ #include #include "trace.h" +#include

[PATCH 02/13] oslib: do not call g_strdup from qemu_get_exec_dir

Just return the directory without requiring the caller to free it. This also removes a bogus check for NULL in os_find_datadir and module_load_one; g_strdup of a static variable cannot return NULL. Signed-off-by: Paolo Bonzini --- include/qemu/osdep.h| 8 ++-- os-posix.c |

[PATCH 03/13] oslib-posix: default exec_dir to bindir

If the exec_dir cannot be retrieved, just assume it's the installation directory that was specified at configure time. This makes it simpler to reason about what the callers will do if they get back an empty path. Signed-off-by: Paolo Bonzini --- meson.build| 2 +- util/oslib-posix.c |

[PATCH 09/13] vl: relocate path to configuration file

Signed-off-by: Paolo Bonzini --- softmmu/vl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/softmmu/vl.c b/softmmu/vl.c index 81e325fa44..b79de9cd66 100644 --- a/softmmu/vl.c +++ b/softmmu/vl.c @@ -2676,8 +2676,9 @@ static int global_init_func(void *opaque, QemuOpts *opts,

[PATCH 01/13] fuzz: use qemu_get_exec_dir

Make things consistent with how softmmu/vl.c uses os_find_datadir. Initializing the path to the executables will also be needed for get_relocatable_path to work. Signed-off-by: Paolo Bonzini --- tests/qtest/fuzz/fuzz.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/q

[PATCH 05/13] oslib-posix: relocate path to /var

Signed-off-by: Paolo Bonzini --- util/oslib-posix.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/util/oslib-posix.c b/util/oslib-posix.c index a68fccb52d..8e23b8fac0 100644 --- a/util/oslib-posix.c +++ b/util/oslib-posix.c @@ -339,8 +339,10 @@ int qemu_pipe(int pipefd

Re: [PATCH] gitlab-ci.yml: Run check-qtest and check-unit at the end of the fuzzer job

On 01/09/2020 08.12, Philippe Mathieu-Daudé wrote: > On Tue, Sep 1, 2020 at 8:05 AM Thomas Huth wrote: >> >> On 31/08/2020 23.31, Philippe Mathieu-Daudé wrote: >>> Hi Thomas, >>> >>> Le lun. 31 août 2020 17:33, Thomas Huth >> > a écrit : >>> >>> The fuzzer job finishes

[PATCH 07/13] net: relocate paths to helpers and scripts

Signed-off-by: Paolo Bonzini --- include/net/net.h | 4 ++-- net/tap.c | 28 +--- 2 files changed, 23 insertions(+), 9 deletions(-) diff --git a/include/net/net.h b/include/net/net.h index e7ef42d62b..897b2d7595 100644 --- a/include/net/net.h +++ b/include/net/ne

Re: [PATCH] gitlab-ci.yml: Run check-qtest and check-unit at the end of the fuzzer job

On Tue, Sep 1, 2020 at 8:05 AM Thomas Huth wrote: > > On 31/08/2020 23.31, Philippe Mathieu-Daudé wrote: > > Hi Thomas, > > > > Le lun. 31 août 2020 17:33, Thomas Huth > > a écrit : > > > > The fuzzer job finishes quite early, so we can run the unit tests and > >

Re: [PATCH] gitlab-ci.yml: Run check-qtest and check-unit at the end of the fuzzer job

On 31/08/2020 18.40, Alexander Bulekov wrote: > On 200831 1732, Thomas Huth wrote: >> The fuzzer job finishes quite early, so we can run the unit tests and >> qtests with -fsanitize=address here without extending the total test time. >> > > Sounds good to me, though its too bad that this is limite

Re: [PATCH v2 1/3] adb: Correct class size on TYPE_ADB_DEVICE

On 9/1/20 1:51 AM, David Gibson wrote: > The TypeInfo incorrectly just lets the class size be inherited. It won't > actually break things, since the class is abstract, but we should get it > right. > > Signed-off-by: David Gibson Reviewed-by: Cédric Le Goater > --- > hw/input/adb.c | 1 + >

Re: [PATCH] gitlab-ci.yml: Run check-qtest and check-unit at the end of the fuzzer job

On 31/08/2020 23.31, Philippe Mathieu-Daudé wrote: > Hi Thomas, > > Le lun. 31 août 2020 17:33, Thomas Huth > a écrit : > > The fuzzer job finishes quite early, so we can run the unit tests and > qtests with -fsanitize=address here without extending the total tes

[PATCH] riscv: sifive_test: Allow 16-bit writes to memory region

When shutting down the machine running a mainline Linux kernel, the following error happens: $ build/riscv64-softmmu/qemu-system-riscv64 -bios default -M virt \ -display none -initrd rootfs.cpio -kernel Image -m 512m \ -nodefaults -serial mon:stdio ... Requesting system poweroff [4.999

Re: [PATCH] docs/system: Fix grammar in documentation

On 31/08/20 22:01, Philippe Mathieu-Daudé wrote: > > > Le jeu. 27 août 2020 19:31, Stefan Weil > a écrit : > > Signed-off-by: Stefan Weil mailto:s...@weilnetz.de>> > > > Reviewed-by: Philippe Mathieu-Daudé > > > --- >  docs/system

[PATCH] usb: call usb_packet_cleanup on usb_packet_map failure

Signed-off-by: Gerd Hoffmann --- hw/usb/hcd-ehci.c | 2 ++ hw/usb/hcd-xhci.c | 1 + 2 files changed, 3 insertions(+) diff --git a/hw/usb/hcd-ehci.c b/hw/usb/hcd-ehci.c index 2b995443fbfd..67847a9cf5f1 100644 --- a/hw/usb/hcd-ehci.c +++ b/hw/usb/hcd-ehci.c @@ -1373,6 +1373,7 @@ static int ehci_ex

Re: [PATCH] xhci: check return value from usb_packet_map

On Tue, Sep 01, 2020 at 10:19:26AM +0530, P J P wrote: > +-- On Sun, 30 Aug 2020, Alexander Bulekov wrote --+ > | I think there is already a fix queued for this one: > | https://www.mail-archive.com/qemu-devel@nongnu.org/msg734424.html > > Yes, it looks similar. > > | > @@ -1615,7 +1615,14 @@ s

Re: [PATCH] cirrus: handle wraparound in cirrus_invalidate_region

Hi, > > > > off_cur_end = ((off_cur + bytesperline - 1) & > > > > s->cirrus_addr_mask) + 1; [ ... ] > > > > +memory_region_set_dirty(&s->vga.vram, 0, off_cur_end); > > > > > > And here be 'off_cur_end -1' > > > > --verbose please. I think this one is correct. > > Here the

Re: [PATCH] meson: use pkg-config method to find dependencies

On Mon, Aug 31, 2020 at 02:15:23PM +0200, Paolo Bonzini wrote: > On 31/08/20 13:06, Gerd Hoffmann wrote: > > > >> Keep "auto" for SDL so that it tries using > >> sdl-config too. > >> - sdl = dependency('sdl2', > >> - required: get_option('sdl'), > >> + sdl = dependency('sdl2',

Re: [PATCH] usb: fix setup_len init (CVE-2020-14364)

Hello Li, +-- On Tue, 25 Aug 2020, Li Qiang wrote --+ | Just see the page. | -->https://access.redhat.com/security/cve/CVE-2020-14364 | | The 'Attack Vector' of the CVSS score here is 'local'. | | I think this should be 'network' as the guest user can touch this in cloud | environment? What's

Re: [PATCH] cirrus: handle wraparound in cirrus_invalidate_region

Gerd Hoffmann 于2020年8月31日周一 下午7:23写道： > > > > for (y = 0; y < lines; y++) { > > > -off_cur = off_begin; > > > +off_cur = off_begin & s->cirrus_addr_mask; > > > off_cur_end = ((off_cur + bytesperline - 1) & > > > s->cirrus_addr_mask) + 1; > > > -assert(off_cur

Re: [PATCH] xhci: check return value from usb_packet_map

+-- On Sun, 30 Aug 2020, Alexander Bulekov wrote --+ | I think there is already a fix queued for this one: | https://www.mail-archive.com/qemu-devel@nongnu.org/msg734424.html Yes, it looks similar. | > @@ -1615,7 +1615,14 @@ static int xhci_setup_packet(XHCITransfer *xfer) | > xhci_xfer_cr

RE: [RFC PATCH v3 30/34] Hexagon (target/hexagon) TCG for instructions with multiple definitions

> -Original Message- > From: Richard Henderson > Sent: Monday, August 31, 2020 8:41 PM > To: Taylor Simpson ; qemu-devel@nongnu.org > Cc: phi...@redhat.com; laur...@vivier.eu; riku.voi...@iki.fi; > aleksandar.m.m...@gmail.com; a...@rev.ng > Subject: Re: [RFC PATCH v3 30/34] Hexagon (targ

Contributing to x86 device emulationi

Hello everyone, I am a software engineer working as an SDE in the industry and I have been wanting to contribute to QEMU. I mailed earlier introducing myself to the mailing list and I was told to talk to specific people who work with the subsystem I want to contribute to. I am interested in contr

Re: [PATCH v3 1/7] vhost: recheck dev state in the vhost_migration_log routine

On Mon, Aug 31, 2020 at 5:28 AM Dima Stepanov wrote: > > vhost-user devices can get a disconnect in the middle of the VHOST-USER > handshake on the migration start. If disconnect event happened right > before sending next VHOST-USER command, then the vhost_dev_set_log() > call in the vhost_migrati

Re: [PATCH v2 2/7] vhost: check queue state in the vhost_dev_set_log routine

On Mon, Aug 31, 2020 at 4:37 AM Dima Stepanov wrote: > > On Thu, Aug 27, 2020 at 09:46:03PM -0400, Raphael Norwitz wrote: > > On Mon, Aug 24, 2020 at 4:41 AM Dima Stepanov > > wrote: > > > > > > If the vhost-user-blk daemon provides only one virtqueue, but device was > > > added with several que

Re: [PATCH v2 0/2] util/hexdump: Cleanup qemu_hexdump()

On 2020/8/23 上午2:09, Philippe Mathieu-Daudé wrote: - Pass const void* buffer - Reorder arguments Supersedes: <20200822150457.1322519-1-f4...@amsat.org> Philippe Mathieu-Daudé (2): util/hexdump: Convert to take a void pointer argument util/hexdump: Reorder qemu_hexdump() arguments inc

Re: [RFC v3 1/1] memory: Skip bad range assertion if notifier supports arbitrary masks

On 2020/8/21 下午10:12, Peter Xu wrote: On Thu, Aug 20, 2020 at 10:28:00AM +0800, Jason Wang wrote: On 2020/8/19 下午11:50, Peter Xu wrote: On Wed, Aug 19, 2020 at 03:15:26PM +0800, Jason Wang wrote: Yes, actually, I feel confused after reading the codes. Is notifier->start IOVA or GPA? In vfio

Re: [RFC PATCH v3 30/34] Hexagon (target/hexagon) TCG for instructions with multiple definitions

On 8/31/20 4:10 PM, Taylor Simpson wrote: > > >> -Original Message- >> From: Richard Henderson >> Sent: Monday, August 31, 2020 1:20 PM >> To: Taylor Simpson ; qemu-devel@nongnu.org >> Cc: phi...@redhat.com; laur...@vivier.eu; riku.voi...@iki.fi; >> aleksandar.m.m...@gmail.com; a...@rev.

Re: [PATCH 0/1] tcg: Adjust simd_desc size encoding

On Tue, Sep 1, 2020 at 6:29 AM Richard Henderson < richard.hender...@linaro.org> wrote: > Frank, this is intended to address the vector size limitation > that you encountered with the risc-v rvv patch set, as per > > https://lists.gnu.org/archive/html/qemu-devel/2020-08/msg07924.html > > although

Re: [PATCH 0/7] target/arm: Add vSPE support to KVM guest

On Mon, 31 Aug 2020 at 15:56, Auger Eric wrote: > > Hi Haibo, > > On 8/7/20 10:10 AM, Haibo Xu wrote: > > This series add support for SPE(Statistical Profiling Extension)[1] > > in KVM guest. It's based on Andrew Murray's kernel KVM patches V2[2], > > and has been tested to ensure that guest can u

[PATCH v3 15/16] hw/riscv: clint: Avoid using hard-coded timebase frequency

From: Bin Meng At present the CLINT timestamp is using a hard-coded timebase frequency value SIFIVE_CLINT_TIMEBASE_FREQ. This might not be true for all boards. Add a new 'timebase-freq' property to the CLINT device, and update various functions to accept this as a parameter. Signed-off-by: Bin

[PATCH v3 12/16] hw/arm: xlnx: Set all boards' GEM 'phy-addr' property value to 23

From: Bin Meng When cadence_gem model was created for Xilinx boards, the PHY address was hard-coded to 23 in the GEM model. Now that we have introduced a property we can use that to tell GEM model what our PHY address is. Change all boards' GEM 'phy-addr' property value to 23, and set the PHY add

[PATCH v3 10/16] hw/riscv: microchip_pfsoc: Connect a DMA controller

From: Bin Meng On the Icicle Kit board, the HSS firmware utilizes the on-chip DMA controller to move the 2nd stage bootloader in the system memory. Let's connect a DMA controller to Microchip PolarFire SoC. Signed-off-by: Bin Meng --- (no changes since v2) Changes in v2: - connect 8 IRQs to

[PATCH v3 09/16] hw/dma: Add SiFive platform DMA controller emulation

From: Bin Meng Microchip PolarFire SoC integrates a DMA engine that supports: * Independent concurrent DMA transfers using 4 DMA channels * Generation of interrupts on various conditions during execution which is actually an IP reused from the SiFive FU540 chip. This creates a model to support b

[PATCH v3 08/16] hw/riscv: microchip_pfsoc: Connect a Cadence SDHCI controller and an SD card

From: Bin Meng Microchip PolarFire SoC integrates one Cadence SDHCI controller. On the Icicle Kit board, one eMMC chip and an external SD card connect to this controller depending on different configuration. As QEMU does not support eMMC yet, we just emulate the SD card configuration. To test th

[PATCH v3 16/16] hw/riscv: sifive_u: Connect a DMA controller

From: Bin Meng SiFive FU540 SoC integrates a platform DMA controller with 4 DMA channels. This connects the exsiting SiFive PDMA model to the SoC, and adds its device tree data as well. Signed-off-by: Bin Meng --- (no changes since v1) include/hw/riscv/sifive_u.h | 11 +++ hw/riscv/s

[PATCH v3 07/16] hw/sd: Add Cadence SDHCI emulation

From: Bin Meng Cadence SD/SDIO/eMMC Host Controller (SD4HC) is an SDHCI compatible controller. The SDHCI compatible registers start from offset 0x200, which are called Slot Register Set (SRS) in its datasheet. This creates a Cadence SDHCI model built on top of the existing generic SDHCI model. C

[PATCH v3 06/16] hw/riscv: microchip_pfsoc: Connect 5 MMUARTs

From: Bin Meng Microchip PolarFire SoC has 5 MMUARTs, and the Icicle Kit board wires 4 of them out. Let's connect all 5 MMUARTs. Signed-off-by: Bin Meng Reviewed-by: Alistair Francis --- (no changes since v1) include/hw/riscv/microchip_pfsoc.h | 20 hw/riscv/microchip_p

[PATCH v3 14/16] hw/riscv: microchip_pfsoc: Hook GPIO controllers

From: Bin Meng Microchip PolarFire SoC integrates 3 GPIOs controllers. It seems enough to create unimplemented devices to cover their register spaces at this point. With this commit, QEMU can boot to U-Boot (2nd stage bootloader) all the way to the Linux shell login prompt, with a modified HSS (

[PATCH v3 03/16] target/riscv: cpu: Set reset vector based on the configured property value

From: Bin Meng Now that we have the newly introduced 'resetvec' property in the RISC-V CPU and HART, instead of hard-coding the reset vector addr in the CPU's instance_init(), move that to riscv_cpu_realize() based on the configured property value from the RISC-V machines. Signed-off-by: Bin Men

[PATCH v3 05/16] hw/char: Add Microchip PolarFire SoC MMUART emulation

From: Bin Meng Microchip PolarFire SoC MMUART is ns16550 compatible, with some additional registers. Create a simple MMUART model built on top of the existing ns16550 model. Signed-off-by: Bin Meng --- (no changes since v2) Changes in v2: - change to update hw/char/meson.build - add impl.min

[PATCH v3 13/16] hw/riscv: microchip_pfsoc: Connect 2 Cadence GEMs

From: Bin Meng Microchip PolarFire SoC integrates 2 Candence GEMs to provide IEEE 802.3 standard-compliant 10/100/1000 Mbps ethernet interface. On the Icicle Kit board, GEM0 connects to a PHY at address 8 while GEM1 connects to a PHY at address 9. The 2nd stage bootloader (U-Boot) is using GEM1

[PATCH v3 02/16] hw/riscv: hart: Add a new 'resetvec' property

From: Bin Meng RISC-V machines do not instantiate RISC-V CPUs directly, instead they do that via the hart array. Add a new property for the reset vector address to allow the value to be passed to the CPU, before CPU is realized. Signed-off-by: Bin Meng Reviewed-by: Alistair Francis --- (no ch

[PATCH v3 04/16] hw/riscv: Initial support for Microchip PolarFire SoC Icicle Kit board

From: Bin Meng This is an initial support for Microchip PolarFire SoC Icicle Kit. The Icicle Kit board integrates a PolarFire SoC, with one SiFive's E51 plus four U54 cores and many on-chip peripherals and an FPGA. For more details about Microchip PolarFire Soc, please see: https://www.microsemi

[PATCH v3 11/16] hw/net: cadence_gem: Add a new 'phy-addr' property

From: Bin Meng At present the PHY address of the PHY connected to GEM is hard-coded to either 23 (BOARD_PHY_ADDRESS) or 0. This might not be the case for all boards. Add a new 'phy-addr' property so that board can specify the PHY address for each GEM instance. Signed-off-by: Bin Meng --- (no

[PATCH v3 01/16] target/riscv: cpu: Add a new 'resetvec' property

From: Bin Meng Currently the reset vector address is hard-coded in a RISC-V CPU's instance_init() routine. In a real world we can have 2 exact same CPUs except for the reset vector address, which is pretty common in the RISC-V core IP licensing business. Normally reset vector address is a config

[PATCH v3 00/16] hw/riscv: Add Microchip PolarFire SoC Icicle Kit board support

From: Bin Meng This adds support for Microchip PolarFire SoC Icicle Kit board. The Icicle Kit board integrates a PolarFire SoC, with one SiFive's E51 plus four U54 cores and many on-chip peripherals and an FPGA. For more details about Microchip PolarFire SoC, please see: https://www.microsemi.co

Re: [PATCH v7 0/8] Generalize start-powered-off property from ARM

David Gibson writes: > On Wed, Aug 26, 2020 at 02:55:27AM -0300, Thiago Jung Bauermann wrote: >> This version fixes `make check` failures in ppc/e500.c, mips/cps.c and >> sparc/sun4m.c. This was done by moving the qdev_realize_and_unref() call as >> close as possible to the object_new() call, i

Re: [PULL 00/76] target/microblaze improvements

Patchew URL: https://patchew.org/QEMU/20200831160601.833692-1-richard.hender...@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: [PULL 00/76] target/microblaze improvements

Patchew URL: https://patchew.org/QEMU/20200831160601.833692-1-richard.hender...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20200831160601.833692-1-richard.hender...@linaro.org Subject: [PULL 00/76] targ

[PATCH v2 1/3] adb: Correct class size on TYPE_ADB_DEVICE

The TypeInfo incorrectly just lets the class size be inherited. It won't actually break things, since the class is abstract, but we should get it right. Signed-off-by: David Gibson --- hw/input/adb.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/input/adb.c b/hw/input/adb.c index 013fc

[PATCH v2 2/3] ppc/pnv: Fix TypeInfo of PnvLpcController abstract class

From: Cédric Le Goater It was missing the instance_size field. Cc: Eduardo Habkost Signed-off-by: Cédric Le Goater Message-Id: <20200822083920.2668930-1-...@kaod.org> Signed-off-by: David Gibson --- hw/ppc/pnv_lpc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/hw/ppc

Re: [PATCH 3/3] spapr: Correct type for SPAPR_DRC_PCI

On Wed, Aug 26, 2020 at 12:21:37PM -0400, Eduardo Habkost wrote: > On Tue, Aug 25, 2020 at 09:16:27PM +1000, David Gibson wrote: > > TYPE_SPAPR_DRC_PCI inherits from TYPE_SPAPR_DRC_PHYSICAL, so its checker > > macro should use the corresponding instance type. We got away with it > > because we nev

[PATCH v2 3/3] spapr: Remove unnecessary DRC type-checker macros

spapr_drc.h includes typechecker macro boilerplate for the many different DRC subclasses. However, most of these types don't actually have different data in their class and/or instance, making these unneeded, unused, and in fact a bad idea. Remove them. Signed-off-by: David Gibson --- include/

[PATCH v2 0/3] ppc: Fixes for class and instance size problems

As requested, some fixes for the ppc things found by Eduardo's scripts. Changes since v1: * Rewrote 3/3 to make a more sensible fix Cédric Le Goater (1): ppc/pnv: Fix TypeInfo of PnvLpcController abstract class David Gibson (2): adb: Correct class size on TYPE_ADB_DEVICE spapr: Remove unn

RE: [RFC PATCH v3 00/34] Hexagon patch series

> -Original Message- > From: Richard Henderson > Sent: Monday, August 31, 2020 2:44 PM > To: Taylor Simpson ; qemu-devel@nongnu.org > Cc: phi...@redhat.com; laur...@vivier.eu; riku.voi...@iki.fi; > aleksandar.m.m...@gmail.com; a...@rev.ng > Subject: Re: [RFC PATCH v3 00/34] Hexagon patch

Re: [PATCH v2 2/7] target/arm: add ARMCPUClass->do_interrupt_locked

Le mar. 1 sept. 2020 00:02, Richard Henderson a écrit : > On 8/31/20 2:18 PM, Richard Henderson wrote: > > On 8/19/20 11:28 AM, Robert Foley wrote: > >> Adding ->do_interrupt_locked to ARMCPUClass is preparation for > >> pushing the BQL down into the per-arch implementation of ->do_interrupt. > >

RE: [PATCH] WHPX: vmware cpuid leaf for tsc and apic frequency

Can I please get some eyes on this? > -Original Message- > From: Sunil Muthuswamy > Sent: Thursday, July 30, 2020 3:11 PM > To: Paolo Bonzini ; Richard Henderson > ; Eduardo Habkost > Cc: qemu-devel@nongnu.org; Stefan Weil > Subject: [PATCH] WHPX: vmware cpuid leaf for tsc and apic freq

Re: [PATCH 1/1] tcg: Adjust simd_desc size encoding

Le mar. 1 sept. 2020 00:31, Richard Henderson a écrit : > With larger vector sizes, it turns out oprsz == maxsz, and we only > need to represent mismatch for oprsz <= 32. We do, however, need > to represent larger oprsz and do so without reducing SIMD_DATA_BITS. > > Reduce the size of the oprsz

RE: [RFC PATCH v3 30/34] Hexagon (target/hexagon) TCG for instructions with multiple definitions

> -Original Message- > From: Richard Henderson > Sent: Monday, August 31, 2020 1:20 PM > To: Taylor Simpson ; qemu-devel@nongnu.org > Cc: phi...@redhat.com; laur...@vivier.eu; riku.voi...@iki.fi; > aleksandar.m.m...@gmail.com; a...@rev.ng > Subject: Re: [RFC PATCH v3 30/34] Hexagon (targ

QEMU | Pipeline #184069766 has failed for master | 2f4c51c0

Your pipeline has failed. Project: QEMU ( https://gitlab.com/qemu-project/qemu ) Branch: master ( https://gitlab.com/qemu-project/qemu/-/commits/master ) Commit: 2f4c51c0 ( https://gitlab.com/qemu-project/qemu/-/commit/2f4c51c0f384d7888a04b4815861e6d5fd244d75 ) Commit Message: Merge remote-tr

[PATCH 0/1] tcg: Adjust simd_desc size encoding

Frank, this is intended to address the vector size limitation that you encountered with the risc-v rvv patch set, as per https://lists.gnu.org/archive/html/qemu-devel/2020-08/msg07924.html although not exactly like that email says. This will allow vectors up to 2k bytes in length. Please test, i

[PATCH 1/1] tcg: Adjust simd_desc size encoding

With larger vector sizes, it turns out oprsz == maxsz, and we only need to represent mismatch for oprsz <= 32. We do, however, need to represent larger oprsz and do so without reducing SIMD_DATA_BITS. Reduce the size of the oprsz field and increase the maxsz field. Steal the oprsz value of 24 to

Re: [PATCH v2 62/76] target/microblaze: Tidy mb_cpu_dump_state

Le mar. 1 sept. 2020 00:16, Richard Henderson a écrit : > On 8/31/20 2:08 PM, Philippe Mathieu-Daudé wrote: > > Hi Richard, > > > > Le ven. 28 août 2020 16:46, Richard Henderson < > richard.hender...@linaro.org > > > a écrit : > > > > Using lookup_symbol i

Re: [PATCH v2 62/76] target/microblaze: Tidy mb_cpu_dump_state

On 8/31/20 2:08 PM, Philippe Mathieu-Daudé wrote: > Hi Richard,  > > Le ven. 28 août 2020 16:46, Richard Henderson > a écrit : > > Using lookup_symbol is quite slow; remove that.  Decode the > > > Can we add some runtime option to enable its use instead

Re: [PATCH v2 7/7] target: Push BQL on ->cpu_exec_interrupt down into per-arch implementation

On 8/19/20 11:28 AM, Robert Foley wrote: > diff --git a/accel/tcg/cpu-exec.c b/accel/tcg/cpu-exec.c > index 499a8bdc5e..c4e54baa6f 100644 > --- a/accel/tcg/cpu-exec.c > +++ b/accel/tcg/cpu-exec.c > @@ -599,7 +599,6 @@ static inline bool cpu_handle_interrupt(CPUState *cpu, > True when it

[REPORT] Nightly Performance Tests - Monday, August 31, 2020

Host CPU : Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz Host Memory : 15.49 GB Start Time (UTC) : 2020-08-31 21:30:01 End Time (UTC) : 2020-08-31 22:03:18 Execution Time : 0:33:17.321826 Status : SUCCESS Note: Changes denoted by '-' are less than 0.01%. -

Re: [PATCH] libvhost-user: Add GCC_FMT_ATTR and fix format string

On Thu, Aug 27, 2020 at 5:09 PM Stefan Weil wrote: > > Signed-off-by: Stefan Weil Reviewed-by: Raphael Norwitz > --- > contrib/libvhost-user/libvhost-user.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/contrib/libvhost-user/libvhost-user.c > b/contrib/libvhost

Re: [PATCH v2 2/7] target/arm: add ARMCPUClass->do_interrupt_locked

On 8/31/20 2:18 PM, Richard Henderson wrote: > On 8/19/20 11:28 AM, Robert Foley wrote: >> Adding ->do_interrupt_locked to ARMCPUClass is preparation for >> pushing the BQL down into the per-arch implementation of ->do_interrupt. >> >> This is needed since ARM's *_cpu_exec_interrupt calls to *_do_i

Re: [PATCH v2 6/7] target: rename all *_cpu_exec_interrupt functions to *_cpu_exec_interrupt_locked

On 8/19/20 11:28 AM, Robert Foley wrote: > The rename of all *_cpu_exec_interrupt functions to > *_cpu_exec_interrupt_locked is preparation for pushing the BQL lock > around these functions down into the per-arch implementation of > *_cpu_exec_interrupt. In a later patch, which pushes down the loc

Re: [PATCH v2 5/7] accel/tcg: Change BQL critical section in cpu_handle_interrupt

On 8/19/20 11:28 AM, Robert Foley wrote: > @@ -625,7 +624,6 @@ static inline bool cpu_handle_interrupt(CPUState *cpu, > } > > /* If we exit via cpu_loop_exit/longjmp it is reset in cpu_exec */ > -qemu_mutex_unlock_iothread(); The comment no longer has an "it" to referen

Re: [PATCH] gitlab-ci.yml: Run check-qtest and check-unit at the end of the fuzzer job

Hi Thomas, Le lun. 31 août 2020 17:33, Thomas Huth a écrit : > The fuzzer job finishes quite early, so we can run the unit tests and > qtests with -fsanitize=address here without extending the total test time. > > Signed-off-by: Thomas Huth > --- > .gitlab-ci.yml | 5 +++-- > 1 file changed, 3

Re: [PATCH v2 4/7] target: Push BQL on ->do_interrupt down into per-arch implementation

On 8/19/20 11:28 AM, Robert Foley wrote: > avr is another exception. avr, arm and cris all had a similar > case where their *_cpu_exec_interrupt was calling to > the CPUClass ->do_interrupt. This causes an issue when we push > the lock down since ->do_interrupt will try to acquire the BQL, but >

Re: [PATCH v2 32/76] target/microblaze: Remove empty D macros

Le ven. 28 août 2020 16:32, Richard Henderson a écrit : > This is never used in op_helper.c and translate.c. There are > two trivial uses in helper.c which can be improved by always > logging MMU_EXCP to CPU_LOG_INT. > > Signed-off-by: Richard Henderson > Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v2 23/76] target/microblaze: Rename env_* tcg variables to cpu_*

Le ven. 28 août 2020 16:36, Richard Henderson a écrit : > This is cpu_imm, cpu_btaken, cpu_iflags, cpu_res_addr and cpu_res_val. > It is standard for these file-scope globals to begin with cpu_*. > > Signed-off-by: Richard Henderson > Reviewed-by: Philippe Mathieu-Daudé --- > target/microbla

Re: [PATCH v2 50/76] target/microblaze: Replace MSR_EE_FLAG with MSR_EE

Le ven. 28 août 2020 16:42, Richard Henderson a écrit : > There's no reason to define MSR_EE_FLAG; we can just use the > original MSR_EE define. Document the other flags copied into > tb_flags with iflag to reserve those bits. > > Signed-off-by: Richard Henderson > Reviewed-by: Philippe Mathie

Re: [PATCH 00/20] RX target update

Hello Yoshinori, Le jeu. 27 août 2020 14:43, Yoshinori Sato a écrit : > Hello. > This series Renesas RX updates. > > It consists of the following contents. > * Update firmware loader. > * Rewrite peripheal modules (Timer and SCI). > - Unified SH4 module. > - Using clock API > * New peripheal

Re: [PATCH v2 09/76] target/microblaze: Split out FSR from env->sregs

Le ven. 28 août 2020 16:25, Richard Henderson a écrit : > Continue eliminating the sregs array in favor of individual members. > Does not correct the width of FSR, yet. > > Signed-off-by: Richard Henderson > Reviewed-by: Philippe Mathieu-Daudé --- > target/microblaze/cpu.h | 1 + >

Re: [PATCH v2 62/76] target/microblaze: Tidy mb_cpu_dump_state

Hi Richard, Le ven. 28 août 2020 16:46, Richard Henderson a écrit : > Using lookup_symbol is quite slow; remove that. Decode the > Can we add some runtime option to enable its use instead? various bits of iflags; only show imm, btaken, btarget when > they are relevant to iflags. Improve form

Re: [PATCH v2 3/7] target/cris: add CRISCPUClass->do_interrupt_locked

On 8/19/20 11:28 AM, Robert Foley wrote: > Adding ->do_interrupt_locked to CRISCPUClass is preparation for > pushing the BQL down into the per-arch implementation of ->do_interrupt. > > This is needed since Cris's *_cpu_exec_interrupt calls to *_do_interrupt. > With the push down of the BQL into *

  1   2   3   4   5   >