Re: [PATCH v3 3/3] target/arm: set ID_AA64ISAR0.TLB to 2 for max AARCH64 CPU type

2021-03-10 Thread Richard Henderson
On 3/9/21 6:29 PM, Rebecca Cran wrote: @@ -651,6 +651,7 @@ static void aarch64_max_initfn(Object *obj) t = FIELD_DP64(t, ID_AA64ISAR0, DP, 1); t = FIELD_DP64(t, ID_AA64ISAR0, FHM, 1); t = FIELD_DP64(t, ID_AA64ISAR0, TS, 2); /* v8.5-CondM */ +t = FIELD_DP64(t,

[PATCH v1 6/7] tests/tcg: Use Hexagon Docker image

2021-03-10 Thread Alex Bennée
From: Alessandro Di Federico [PMD: Split from 'Add Hexagon Docker image' patch] Signed-off-by: Alessandro Di Federico Signed-off-by: Philippe Mathieu-Daudé Signed-off-by: Alex Bennée Message-Id: <20210228222314.304787-5-f4...@amsat.org> Message-Id: <20210305092328.31792-6-alex.ben...@linaro.o

[PATCH v1 7/7] gitlab: add build-user-hexagon test

2021-03-10 Thread Alex Bennée
We special case this as the container with the cross compiler for the tests takes so long to build it is manually uploaded into the registry. Signed-off-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Wainer dos Santos Moschetta Message-Id: <20210305092328.31792-7-alex.ben...@l

Re: [RFC] hw/display: add virtio-ramfb device

2021-03-10 Thread Laszlo Ersek
On 03/10/21 17:42, Joelle van Dyne wrote: > On Wed, Mar 10, 2021 at 4:45 AM Laszlo Ersek wrote: >> >> On 03/10/21 10:51, Gerd Hoffmann wrote: >>> On Tue, Mar 09, 2021 at 01:35:13PM -0800, Joelle van Dyne wrote: Like virtio-vga, but using ramfb instead of legacy vga. Useful for booting fr

Re: [PATCH v1 0/7] testing/next (hexagon 2, electric boogaloo)

2021-03-10 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20210310192331.29284-1-alex.ben...@linaro.org/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20210310192331.29284-1-alex.ben...@linaro.org Subject: [PATCH v1 0/7] testing/next (h

Re: [PATCH 1/2] hw/mips/jazz: Use generic I/O bus via get_system_io()

2021-03-10 Thread Peter Xu
On Wed, Mar 10, 2021 at 08:12:54PM +0100, Philippe Mathieu-Daudé wrote: > No need to create a local ISA I/O MemoryRegion, use get_system_io(). > > This partly reverts commit 5c63bcf7501527b844f61624957bdba254d75bfc. I think it's not a clean revert of that, see below. > > Signed-off-by: Philippe

[PATCH v4 0/5] Allwinner H3 fixes for EMAC and acceptance tests

2021-03-10 Thread Niek Linnenbank
The following are maintenance patches for the Allwinner H3. The first patch resolves artifact download errors by updating the URLs to the armbian.com server. The second patch is a fix for the EMAC that is used by the Allwinner H3 / Orange Pi PC machine. All these patches have been reviewed and

[PATCH v4 2/5] tests/acceptance/boot_linux_console: remove Armbian 19.11.3 bionic test for orangepi-pc machine

2021-03-10 Thread Niek Linnenbank
The image for Armbian 19.11.3 bionic has been removed from the armbian server. Without the image as input the test arm_orangepi_bionic_19_11 cannot run. This commit removes the test completely and merges the code of the generic function do_test_arm_orangepi_uboot_armbian back with the 20.08 test.

[PATCH v4 3/5] tests/acceptance/boot_linux_console: change URL for test_arm_orangepi_bionic_20_08

2021-03-10 Thread Niek Linnenbank
Update the download URL of the Armbian 20.08 Bionic image for test_arm_orangepi_bionic_20_08 of the orangepi-pc machine. The archive.armbian.com URL contains more images and should keep stable for a longer period of time than dl.armbian.com. Signed-off-by: Niek Linnenbank Reviewed-by: Philippe M

[PATCH v4 1/5] hw/net/allwinner-sun8i-emac: traverse transmit queue using TX_CUR_DESC register value

2021-03-10 Thread Niek Linnenbank
Currently the emulated EMAC for sun8i always traverses the transmit queue from the head when transferring packets. It searches for a list of consecutive descriptors whichs are flagged as ready for processing and transmits their payloads accordingly. The controller stops processing once it finds a

[PATCH v4 5/5] tests/acceptance: drop ARMBIAN_ARTIFACTS_CACHED condition for orangepi-pc, cubieboard tests

2021-03-10 Thread Niek Linnenbank
Previously the ARMBIAN_ARTIFACTS_CACHED pre-condition was added to allow running tests that have already existing armbian.com artifacts stored in the local avocado cache, but do not have working URLs to download a fresh copy. At this time of writing the URLs for artifacts on the armbian.com serve

[PATCH v4 4/5] tests/acceptance: update sunxi kernel from armbian to 5.10.16

2021-03-10 Thread Niek Linnenbank
The linux kernel 4.20.7 binary for sunxi has been removed from apt.armbian.com: $ ARMBIAN_ARTIFACTS_CACHED=yes AVOCADO_ALLOW_LARGE_STORAGE=yes avocado --show=app,console run -t machine:orangepi-pc tests/acceptance/boot_linux_console.py Fetching asset from tests/acceptance/boot_linux_console

Re: [PULL 00/14] Gitlab, testing and misc patches

2021-03-10 Thread Peter Maydell
On Tue, 9 Mar 2021 at 10:55, Thomas Huth wrote: > > Hi Peter, > > the following changes since commit 229a834518b950d56fd1bc94923276504d0ee9d4: > > Merge remote-tracking branch 'remotes/philmd-gitlab/tags/renesas-20210306' > into staging (2021-03-08 15:45:48 +) > > are available in the Git

Re: [PATCH 1/2] hw/mips/jazz: Use generic I/O bus via get_system_io()

2021-03-10 Thread Philippe Mathieu-Daudé
On 3/10/21 8:49 PM, Peter Xu wrote: > On Wed, Mar 10, 2021 at 08:12:54PM +0100, Philippe Mathieu-Daudé wrote: >> No need to create a local ISA I/O MemoryRegion, use get_system_io(). >> >> This partly reverts commit 5c63bcf7501527b844f61624957bdba254d75bfc. > > I think it's not a clean revert of th

Re: [PATCH v1 4/7] tests/docker: add "fetch" sub-command

2021-03-10 Thread Philippe Mathieu-Daudé
On 3/10/21 8:23 PM, Alex Bennée wrote: > This simply wraps up fetching a build from the registry and tagging it > as the local build. > > Signed-off-by: Alex Bennée > --- > tests/docker/docker.py | 17 + > 1 file changed, 17 insertions(+) Reviewed-by: Philippe Mathieu-Daudé

Re: [PATCH v4 00/78] target/arm: Implement SVE2

2021-03-10 Thread Peter Maydell
On Tue, 9 Mar 2021 at 16:20, Richard Henderson wrote: > > After a 6-month hiatus, sve2 is back. This time, with RISU > testing vs FVP 11.13.36. > > Based-on: 20210309155305.11301-1-richard.hender...@linaro.org > ("target/arm: sve1 fixes") Are you hoping to squeeze this into 6.0, or can I delay r

Re: [PATCH] vfio: Support host translation granule size

2021-03-10 Thread Alex Williamson
On Wed, 10 Mar 2021 15:19:33 +0800 Kunkun Jiang wrote: > Hi Alex, > > On 2021/3/10 7:17, Alex Williamson wrote: > > On Thu, 4 Mar 2021 21:34:46 +0800 > > Kunkun Jiang wrote: > > > >> The cpu_physical_memory_set_dirty_lebitmap() can quickly deal with > >> the dirty pages of memory by bitmap-tr

Re: [PATCH 1/2] hw/mips/jazz: Use generic I/O bus via get_system_io()

2021-03-10 Thread Peter Xu
On Wed, Mar 10, 2021 at 09:11:40PM +0100, Philippe Mathieu-Daudé wrote: > >> /* ISA bus: IO space at 0x9000, mem space at 0x9100 */ > >> -memory_region_init(isa_io, NULL, "isa-io", 0x0001); > >> memory_region_init(isa_mem, NULL, "isa-mem", 0x0100); > >> -memory_reg

[PATCH v5 03/10] KVM: Create the KVMSlot dirty bitmap on flag changes

2021-03-10 Thread Peter Xu
Previously we have two places that will create the per KVMSlot dirty bitmap: 1. When a newly created KVMSlot has dirty logging enabled, 2. When the first log_sync() happens for a memory slot. The 2nd case is lazy-init, while the 1st case is not (which is a fix of what the 2nd case missed). T

[PATCH v5 02/10] KVM: Use a big lock to replace per-kml slots_lock

2021-03-10 Thread Peter Xu
Per-kml slots_lock will bring some trouble if we want to take all slots_lock of all the KMLs, especially when we're in a context that we could have taken some of the KML slots_lock, then we even need to figure out what we've taken and what we need to take. Make this simple by merging all KML slots

[PATCH v5 01/10] memory: Introduce log_sync_global() to memory listener

2021-03-10 Thread Peter Xu
Some of the memory listener may want to do log synchronization without being able to specify a range of memory to sync but always globally. Such a memory listener should provide this new method instead of the log_sync() method. Obviously we can also achieve similar thing when we put the global syn

[PATCH v5 04/10] KVM: Provide helper to get kvm dirty log

2021-03-10 Thread Peter Xu
Provide a helper kvm_slot_get_dirty_log() to make the function kvm_physical_sync_dirty_bitmap() clearer. We can even cache the as_id into KVMSlot when it is created, so that we don't even need to pass it down every time. Since at it, remove return value of kvm_physical_sync_dirty_bitmap() because

[PATCH v5 08/10] KVM: Add dirty-gfn-count property

2021-03-10 Thread Peter Xu
Add a parameter for dirty gfn count for dirty rings. If zero, dirty ring is disabled. Otherwise dirty ring will be enabled with the per-vcpu gfn count as specified. If dirty ring cannot be enabled due to unsupported kernel or illegal parameter, it'll fallback to dirty logging. By default, dirty

[PATCH v5 06/10] KVM: Simplify dirty log sync in kvm_set_phys_mem

2021-03-10 Thread Peter Xu
kvm_physical_sync_dirty_bitmap() on the whole section is inaccurate, because the section can be a superset of the memslot that we're working on. The result is that if the section covers multiple kvm memslots, we could be doing the synchronization for multiple times for each kvmslot in the section.

[PATCH v5 05/10] KVM: Provide helper to sync dirty bitmap from slot to ramblock

2021-03-10 Thread Peter Xu
kvm_physical_sync_dirty_bitmap() calculates the ramblock offset in an awkward way from the MemoryRegionSection that passed in from the caller. The truth is for each KVMSlot the ramblock offset never change for the lifecycle. Cache the ramblock offset for each KVMSlot into the structure when the K

[PATCH v5 00/10] KVM: Dirty ring support (QEMU part)

2021-03-10 Thread Peter Xu
This is v5 of the qemu dirty ring interface support. v5: - rebase - dropped patch "update-linux-headers: Include const.h" after rebase - dropped patch "KVM: Fixup kvm_log_clear_one_slot() ioctl return check" since similar patch got merged recently (38e0b7904eca7cd32f8953c3) = v4 cover l

[PATCH v5 10/10] KVM: Dirty ring support

2021-03-10 Thread Peter Xu
KVM dirty ring is a new interface to pass over dirty bits from kernel to the userspace. Instead of using a bitmap for each memory region, the dirty ring contains an array of dirtied GPAs to fetch (in the form of offset in slots). For each vcpu there will be one dirty ring that binds to it. kvm_di

[PATCH v5 07/10] KVM: Cache kvm slot dirty bitmap size

2021-03-10 Thread Peter Xu
Cache it too because we'll reference it more frequently in the future. Reviewed-by: Dr. David Alan Gilbert Signed-off-by: Peter Xu --- accel/kvm/kvm-all.c | 1 + include/sysemu/kvm_int.h | 1 + 2 files changed, 2 insertions(+) diff --git a/accel/kvm/kvm-all.c b/accel/kvm/kvm-all.c index 2

[PATCH v5 09/10] KVM: Disable manual dirty log when dirty ring enabled

2021-03-10 Thread Peter Xu
KVM_CAP_MANUAL_DIRTY_LOG_PROTECT2 is for KVM_CLEAR_DIRTY_LOG, which is only useful for KVM_GET_DIRTY_LOG. Skip enabling it for kvm dirty ring. More importantly, KVM_DIRTY_LOG_INITIALLY_SET will not wr-protect all the pages initially, which is against how kvm dirty ring is used - there's no way fo

Re: [PATCH v4] target/s390x: Implement the MVPG condition-code-option bit

2021-03-10 Thread David Hildenbrand
> Am 09.03.2021 um 22:05 schrieb Thomas Huth : > > On 04/03/2021 09.10, David Hildenbrand wrote: >>> On 03.03.21 22:36, Richard Henderson wrote: >>> On 3/3/21 1:22 PM, David Hildenbrand wrote: > Am 03.03.2021 um 22:19 schrieb Richard Henderson > : > > On 3/3/21 1:11 PM,

[PULL 00/22] Trivial branch for 6.0 patches

2021-03-10 Thread Laurent Vivier
branch-for-6.0-pull-request for you to fetch changes up to 538f049704e9b7a07eeaf326af772fdd30d89576: sysemu: Let VMChangeStateHandler take boolean 'running' argument (2021-03-0= 9 23:13:57 +0100) Pull request triv

[PULL 02/22] scsi: Silence gcc warning

2021-03-10 Thread Laurent Vivier
From: Eric Blake On Fedora 33, gcc 10.2.1 notes that scsi_cdb_length(buf) can set len==-1, which in turn overflows g_malloc(): [5/5] Linking target qemu-system-x86_64 In function ‘scsi_disk_new_request_dump’, inlined from ‘scsi_new_request’ at ../hw/scsi/scsi-disk.c:2608:9: ../hw/scsi/scsi-d

[PULL 06/22] vhost_user_gpu: Drop dead check for g_malloc() failure

2021-03-10 Thread Laurent Vivier
From: Markus Armbruster Signed-off-by: Markus Armbruster Reviewed-by: Marc-André Lureau Reviewed-by: Michael S. Tsirkin Message-Id: <20210126124240.2081959-3-arm...@redhat.com> Signed-off-by: Laurent Vivier --- hw/display/vhost-user-gpu.c | 1 - 1 file changed, 1 deletion(-) diff --git a/hw

[PULL 05/22] backends/dbus-vmstate: Fix short read error handling

2021-03-10 Thread Laurent Vivier
From: Markus Armbruster When dbus_vmstate_post_load() fails, it complains to stderr. Except on short read, where it checks with g_return_val_if_fail(). This fails silently if G_DISABLE_CHECKS is undefined (it should be), or else pads the short read with uninitialized bytes. Replace g_return_va

[PULL 01/22] Various spelling fixes

2021-03-10 Thread Laurent Vivier
From: Michael Tokarev An assorted set of spelling fixes in various places. Signed-off-by: Michael Tokarev Reviewed-by: Stefan Weil Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Peter Maydell Message-Id: <20210309111510.79495-1-...@msgid.tls.msk.ru> Signed-off-by: Laurent Vivier --- disa

[PULL 03/22] hw/elf_ops: Fix a typo

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé g_mapped_file_new_from_fd()'s parameter is named 'writable'. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Stefano Garzarella Reviewed-by: David Edmondson Message-Id: <20210225181344.3623720-1-phi...@redhat.com> Signed-off-by: Laurent Vivier --- include/hw/

[PULL 08/22] net: Use id_generate() in the network subsystem, too

2021-03-10 Thread Laurent Vivier
From: Thomas Huth We already got a global function called id_generate() to create unique IDs within QEMU. Let's use it in the network subsytem, too, instead of inventing our own ID scheme here. Signed-off-by: Thomas Huth Reviewed-by: Marc-André Lureau Message-Id: <20210215090225.1046239-1-th..

[PULL 10/22] exec/memory: Use struct Object typedef

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé We forward-declare Object typedef in "qemu/typedefs.h" since commit ca27b5eb7cd ("qom/object: Move Object typedef to 'qemu/typedefs.h'"). Use it everywhere to make the code simpler. Signed-off-by: Philippe Mathieu-Daudé Acked-by: David Gibson Reviewed-by: Laurent V

[PULL 07/22] MAINTAINERS: Fix the location of tools manuals

2021-03-10 Thread Laurent Vivier
From: Wainer dos Santos Moschetta The qemu-img.rst, qemu-nbd.rst, virtfs-proxy-helper.rst, qemu-trace-stap.rst, and virtiofsd.rst manuals were moved to docs/tools, so this update MAINTAINERS accordingly. Fixes: a08b4a9fe6c ("docs: Move tools documentation to tools manual") Signed-off-by: Wainer

[PULL 14/22] seccomp: Replace the word 'blacklist'

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Follow the inclusive terminology from the "Conscious Language in your Open Source Projects" guidelines [*] and replace the word "blacklist" appropriately. [*] https://github.com/conscious-lang/conscious-lang-docs/blob/main/faq.md Reviewed-by: Daniel P. Berrangé Ack

[PULL 09/22] fuzz-test: remove unneccessary debugging flags

2021-03-10 Thread Laurent Vivier
From: Alexander Bulekov These flags cause the output to look strange for 'make check', and they aren't needed to reproduce bugs, if they reappear. Suggested-by: Peter Maydell Signed-off-by: Alexander Bulekov Reviewed-by: Thomas Huth Reviewed-by: Philippe Mathieu-Daudé Message-Id: <2021021618

[PULL 11/22] virtio-gpu: Adjust code space style

2021-03-10 Thread Laurent Vivier
From: lijiejun Fix code style. Operator needs align with eight spaces, and delete line space. Signed-off-by: lijiejun Reviewed-by: Philippe Mathieu-Daudé Message-Id: <1615292050-108748-1-git-send-email-a_lijie...@163.com> Signed-off-by: Laurent Vivier --- hw/display/virtio-gpu-3d.c | 3 +--

[PULL 15/22] qemu-options: Replace the word 'blacklist'

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Follow the inclusive terminology from the "Conscious Language in your Open Source Projects" guidelines [*] and replace the word "blacklist" appropriately. [*] https://github.com/conscious-lang/conscious-lang-docs/blob/main/faq.md Reviewed-by: Daniel P. Berrangé Sig

[PULL 12/22] ui: Replace the word 'whitelist'

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Follow the inclusive terminology from the "Conscious Language in your Open Source Projects" guidelines [*] and replace the words "whitelist" appropriately. [*] https://github.com/conscious-lang/conscious-lang-docs/blob/main/faq.md Reviewed-by: Gerd Hoffmann Reviewe

[PULL 04/22] target/hexagon/gen_tcg_funcs: Fix a typo

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Taylor Simpson Message-Id: <20210225181507.3624509-1-f4...@amsat.org> Signed-off-by: Laurent Vivier --- target/hexagon/gen_tcg_funcs.py | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --

[PULL 18/22] hw/lm32/Kconfig: Introduce CONFIG_LM32_EVR for lm32-evr/uclinux boards

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé We want to be able to use the 'LM32' config for architecture specific features. Introduce CONFIG_LM32_EVR to select the lm32-evr / lm32-uclinux boards. Reviewed-by: Alex Bennée Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20210221225626.2589247-2-f4...@amsat.

[PULL 17/22] qemu-common.h: Update copyright string to 2021

2021-03-10 Thread Laurent Vivier
From: Peter Maydell Update the common copyright string that we use in -version reports, About dialogs, etc, to 2021. Signed-off-by: Peter Maydell Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Laurent Vivier Message-Id: <20210309162258.28633-1-peter.mayd...@linaro.org> Signed-off-by: Lauren

[PULL 19/22] hw/lm32/Kconfig: Rename CONFIG_LM32 -> CONFIG_LM32_DEVICES

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé We want to be able to use the 'LM32' config for architecture specific features. As CONFIG_LM32 is only used to select peripherals, rename it CONFIG_LM32_DEVICES. Reviewed-by: Alex Bennée Signed-off-by: Philippe Mathieu-Daudé Message-Id: <20210221225626.2589247-3-f4

[PULL 16/22] tests/fp/fp-test: Replace the word 'blacklist'

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Follow the inclusive terminology from the "Conscious Language in your Open Source Projects" guidelines [*] and replace the word "blacklist" appropriately. [*] https://github.com/conscious-lang/conscious-lang-docs/blob/main/faq.md Acked-by: Alex Bennée Reviewed-by:

[PULL 13/22] scripts/tracetool: Replace the word 'whitelist'

2021-03-10 Thread Laurent Vivier
From: Philippe Mathieu-Daudé Follow the inclusive terminology from the "Conscious Language in your Open Source Projects" guidelines [*] and replace the words "whitelist" appropriately. [*] https://github.com/conscious-lang/conscious-lang-docs/blob/main/faq.md Reviewed-by: Daniel P. Berrangé Re

Re: [PULL 00/22] Trivial branch for 6.0 patches

2021-03-10 Thread Laurent Vivier
---- > Pull request trivial patches 20210310 > > > > Alexander Bulekov (1): > fuzz-test: remove unneccessary debugging flags > > Eric Blake (1): > scsi: Si

Re: [PATCH v3 1/3] target/arm: Add support for FEAT_TLBIRANGE

2021-03-10 Thread Rebecca Cran
On 3/10/21 12:24 PM, Richard Henderson wrote: On 3/9/21 6:29 PM, Rebecca Cran wrote: +void tlb_flush_page_range_by_mmuidx(CPUState *cpu, target_ulong addr, +    unsigned int num_pages, uint16_t idxmap) I am not keen on this interface.  I think you should take e

[PATCH] hw/riscv: Fix OT IBEX reset vector

2021-03-10 Thread Alexander Wagner
The IBEX documentation [1] specifies the reset vector to be "the most significant 3 bytes of the boot address and the reset value (0x80) as the least significant byte". [1] https://github.com/lowRISC/ibex/blob/master/doc/03_reference/exception_interrupts.rst Signed-off-by: Alexander Wagner ---

Re: [PATCH 2/3] memory: Provide 'base address' argument to mtree_print_mr()

2021-03-10 Thread Mark Cave-Ayland
On 09/03/2021 21:54, Philippe Mathieu-Daudé wrote: +Peter/Mark/Edgar for SoC modelling On 3/9/21 10:39 AM, Philippe Mathieu-Daudé wrote: Hi Peter, On 3/9/21 12:40 AM, Peter Xu wrote:> On Sat, Mar 06, 2021 at 12:54:13AM +0100, Philippe Mathieu-Daudé wrote: @@ -3188,14 +3188,15 @@ void mtree_i

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread Mark Cave-Ayland
On 10/03/2021 13:24, Laurent Vivier wrote: Le 10/03/2021 à 14:10, Laurent Vivier a écrit : Le 10/03/2021 à 13:56, Laurent Vivier a écrit : Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: The 60Hz timer is initialised using timer_new_ns() meaning t

Re: [PULL 00/20] ppc-for-6.0 queue 20210310

2021-03-10 Thread David Gibson
remotes/mcayland/tags/qemu-sparc-20210307' > > into staging (2021-03-09 13:50:35 +) > > > > are available in the Git repository at: > > > > https://gitlab.com/dgibson/qemu.git tags/ppc-for-6.0-20210310 > > > > for you to fetch changes up to

Re: [PATCH 6/7] mac_via: fix 60Hz VIA1 timer interval

2021-03-10 Thread BALATON Zoltan
On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: On 10/03/2021 13:24, Laurent Vivier wrote: Le 10/03/2021 à 14:10, Laurent Vivier a écrit : Le 10/03/2021 à 13:56, Laurent Vivier a écrit : Le 10/03/2021 à 13:32, BALATON Zoltan a écrit : On Wed, 10 Mar 2021, Mark Cave-Ayland wrote: The 60Hz timer i

[PATCH 03/26] meson: Split out fpu/meson.build

2021-03-10 Thread Richard Henderson
Signed-off-by: Richard Henderson --- meson.build | 4 +--- fpu/meson.build | 1 + 2 files changed, 2 insertions(+), 3 deletions(-) create mode 100644 fpu/meson.build diff --git a/meson.build b/meson.build index f884a62682..c8a5ca65e5 100644 --- a/meson.build +++ b/meson.build @@ -1934,9 +19

[PATCH 04/26] tcg: Re-order tcg_region_init vs tcg_prologue_init

2021-03-10 Thread Richard Henderson
Instead of delaying tcg_region_init until after tcg_prologue_init is complete, do tcg_region_init first and let tcg_prologue_init shrink the first region by the size of the generated prologue. Signed-off-by: Richard Henderson --- accel/tcg/tcg-all.c | 11 - accel/tcg/translate-all.

[PATCH 00/26] tcg: Workaround macOS 11.2 mprotect bug

2021-03-10 Thread Richard Henderson
Supercedes: 20210210105527.74943-1-r.bolsha...@yadro.com ("util/osdep: Avoid mprotect() RWX->NONE on Big Sur 11.2") It took a few more patches than imagined to unify the two places in which we manipulate the tcg code_gen buffer, but the result is surely cleaner. There's a lot more that could be d

[PATCH 05/26] tcg: Remove error return from tcg_region_initial_alloc__locked

2021-03-10 Thread Richard Henderson
All callers immediately assert on error, so move the assert into the function itself. Signed-off-by: Richard Henderson --- tcg/tcg.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions(-) diff --git a/tcg/tcg.c b/tcg/tcg.c index 0a2e5710de..2b631fccdf 100644 --- a/tcg/tcg.c

[PATCH 09/26] accel/tcg: Inline cpu_gen_init

2021-03-10 Thread Richard Henderson
It consists of one function call and has only one caller. Signed-off-by: Richard Henderson --- accel/tcg/translate-all.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c index b9057567f4..6d3184e7da 100644 --- a/accel

[PATCH 01/26] meson: Split out tcg/meson.build

2021-03-10 Thread Richard Henderson
Signed-off-by: Richard Henderson --- meson.build | 9 ++--- tcg/meson.build | 13 + 2 files changed, 15 insertions(+), 7 deletions(-) create mode 100644 tcg/meson.build diff --git a/meson.build b/meson.build index adeec153d9..8bc472ddeb 100644 --- a/meson.build +++ b/meson.

[PATCH 02/26] meson: Move disas/tci.c to disas/meson.build

2021-03-10 Thread Richard Henderson
There's no reason to do this in the main meson.build. Signed-off-by: Richard Henderson --- meson.build | 1 - disas/meson.build | 2 ++ 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/meson.build b/meson.build index 8bc472ddeb..f884a62682 100644 --- a/meson.build +++ b/meson.

[PATCH 08/26] tcg: Split out region.c

2021-03-10 Thread Richard Henderson
Signed-off-by: Richard Henderson --- tcg/internal.h | 37 tcg/region.c| 570 tcg/tcg.c | 545 + tcg/meson.build | 1 + 4 files changed, 611 insertions(+), 542 deletions(-) create mode

[PATCH 10/26] accel/tcg: Move alloc_code_gen_buffer to tcg/region.c

2021-03-10 Thread Richard Henderson
Buffer management is integral to tcg. Do not leave the allocation to code outside of tcg/. This is code movement, with further cleanups to follow. Signed-off-by: Richard Henderson --- include/tcg/tcg.h | 2 +- accel/tcg/translate-all.c | 414 + tcg

[PATCH 13/26] accel/tcg: Merge tcg_exec_init into tcg_init_machine

2021-03-10 Thread Richard Henderson
There is only one caller, and shortly we will need access to the MachineState, which tcg_init_machine already has. Signed-off-by: Richard Henderson --- accel/tcg/internal.h | 2 ++ include/sysemu/tcg.h | 2 -- accel/tcg/tcg-all.c | 14 +- accel/tcg/translate-all.c |

[PATCH 07/26] tcg: Split out tcg_region_prologue_set

2021-03-10 Thread Richard Henderson
This has only one user, but will make more sense after some code motion. Always leave the tcg_init_ctx initialized to the first region, in preparation for tcg_prologue_init(). This also requires that we don't re-allocate the region for the first cpu, lest we hit the assertion for total number of

[PATCH 14/26] accel/tcg: Pass down max_cpus to tcg_init

2021-03-10 Thread Richard Henderson
Start removing the include of hw/boards.h from tcg/. Pass down the max_cpus value from tcg_init_machine, where we have the MachineState already. Signed-off-by: Richard Henderson --- include/tcg/tcg.h | 2 +- tcg/internal.h | 2 +- accel/tcg/tcg-all.c | 10 +- tcg/region.c

[PATCH 06/26] tcg: Split out tcg_region_initial_alloc

2021-03-10 Thread Richard Henderson
This has only one user, and currently needs an ifdef, but will make more sense after some code motion. Signed-off-by: Richard Henderson --- tcg/tcg.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/tcg/tcg.c b/tcg/tcg.c index 2b631fccdf..3316a22bde 100644 --- a

[PATCH 18/26] tcg: Tidy tcg_n_regions

2021-03-10 Thread Richard Henderson
Compute the value using straight division and bounds, rather than a loop. Pass in tb_size rather than reading from tcg_init_ctx.code_gen_buffer_size, Signed-off-by: Richard Henderson --- tcg/region.c | 29 - 1 file changed, 12 insertions(+), 17 deletions(-) diff --g

[PATCH 20/26] tcg: Allocate code_gen_buffer into struct tcg_region_state

2021-03-10 Thread Richard Henderson
Do not mess around with setting values within tcg_init_ctx. Put the values into 'region' directly, which is where they will live for the lifetime of the program. Signed-off-by: Richard Henderson --- tcg/region.c | 61 ++-- 1 file changed, 26 insert

[PATCH 11/26] accel/tcg: Rename tcg_init to tcg_init_machine

2021-03-10 Thread Richard Henderson
We shortly want to use tcg_init for something else. Since the hook is called init_machine, match that. Signed-off-by: Richard Henderson --- accel/tcg/tcg-all.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/accel/tcg/tcg-all.c b/accel/tcg/tcg-all.c index f132033999..30d8

[PATCH 17/26] tcg: Replace region.end with region.total_size

2021-03-10 Thread Richard Henderson
A size is easier to work with than an end point, particularly during initial buffer allocation. Signed-off-by: Richard Henderson --- tcg/region.c | 29 + 1 file changed, 17 insertions(+), 12 deletions(-) diff --git a/tcg/region.c b/tcg/region.c index 8fba0724e5..7fb2

[PATCH 22/26] tcg: Sink qemu_madvise call to common code

2021-03-10 Thread Richard Henderson
Move the call out of the N versions of alloc_code_gen_buffer and into tcg_region_init. Signed-off-by: Richard Henderson --- tcg/region.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tcg/region.c b/tcg/region.c index f25ec1ecad..7640aac243 100644 --- a/tcg/re

[PATCH 12/26] tcg: Create tcg_init

2021-03-10 Thread Richard Henderson
Perform both tcg_context_init and tcg_region_init. Do not leave this split to the caller. Signed-off-by: Richard Henderson --- include/tcg/tcg.h | 3 +-- tcg/internal.h| 1 + accel/tcg/translate-all.c | 3 +-- tcg/tcg.c | 9 - 4 files changed, 11 inser

[PATCH 16/26] tcg: Move MAX_CODE_GEN_BUFFER_SIZE to tcg-target.h

2021-03-10 Thread Richard Henderson
Remove the ifdef ladder and move each define into the appropriate header file. Signed-off-by: Richard Henderson --- tcg/aarch64/tcg-target.h | 1 + tcg/arm/tcg-target.h | 1 + tcg/i386/tcg-target.h| 2 ++ tcg/mips/tcg-target.h| 6 ++ tcg/ppc/tcg-target.h | 2 ++ tcg/risc

[PATCH 23/26] tcg: Do not set guard pages in the rx buffer

2021-03-10 Thread Richard Henderson
We only need guard pages in the rw buffer to avoid buffer overruns. Let the rx buffer keep large pages all the way through. Signed-off-by: Richard Henderson --- tcg/region.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/tcg/region.c b/tcg/region.c index 7640aac243..9

[PATCH 19/26] tcg: Tidy split_cross_256mb

2021-03-10 Thread Richard Henderson
Return output buffer and size via output pointer arguments, rather than returning size via tcg_ctx->code_gen_buffer_size. Signed-off-by: Richard Henderson --- tcg/region.c | 15 +++ 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/tcg/region.c b/tcg/region.c index 57017c

[PATCH 15/26] tcg: Introduce tcg_max_ctxs

2021-03-10 Thread Richard Henderson
Finish the divorce of tcg/ from hw/, and do not take the max cpu value from MachineState; just rememver what we were passed in tcg_init. Signed-off-by: Richard Henderson --- tcg/internal.h | 3 ++- tcg/region.c | 6 +++--- tcg/tcg.c | 23 ++- 3 files changed, 15 inse

[PATCH 26/26] tcg: When allocating for !splitwx, begin with PROT_NONE

2021-03-10 Thread Richard Henderson
There's a change in mprotect() behaviour [1] in the latest macOS on M1 and it's not yet clear if it's going to be fixed by Apple. In this case, instead of changing permissions of N guard pages, we change permissions of N rwx regions. The same number of syscalls are required either way. [1] https

[PATCH 21/26] tcg: Return the map protection from alloc_code_gen_buffer

2021-03-10 Thread Richard Henderson
Change the interface from a boolean error indication to a negative error vs a non-negative protection. For the moment this is only interface change, not making use of the new data. Signed-off-by: Richard Henderson --- tcg/region.c | 63 +++- 1 fil

[PATCH 25/26] tcg: Merge buffer protection and guard page protection

2021-03-10 Thread Richard Henderson
Do not handle protections on a case-by-case basis in the various alloc_code_gen_buffer instances; do it within a single loop in tcg_region_init. Signed-off-by: Richard Henderson --- tcg/region.c | 40 +--- 1 file changed, 29 insertions(+), 11 deletions(-) dif

[RFC] tests/migration: introduce multifd into guestperf toolkit

2021-03-10 Thread huangy81
From: Hyman Current guestperf tool does not support multifd migration, introducing it is good idea so that we can compare the performence of all type of migration. Signed-off-by: Hyman --- tests/migration/guestperf/comparison.py | 14 ++ tests/migration/guestperf/engine.py | 16

[PATCH 24/26] util/osdep: Add qemu_mprotect_rw

2021-03-10 Thread Richard Henderson
For --enable-tcg-interpreter on Windows, we will need this. Signed-off-by: Richard Henderson --- include/qemu/osdep.h | 1 + util/osdep.c | 9 + 2 files changed, 10 insertions(+) diff --git a/include/qemu/osdep.h b/include/qemu/osdep.h index ba15be9c56..5cc2e57bdf 100644 --- a/i

Re: [PATCH 16/26] tcg: Move MAX_CODE_GEN_BUFFER_SIZE to tcg-target.h

2021-03-10 Thread BALATON Zoltan
On Wed, 10 Mar 2021, Richard Henderson wrote: Remove the ifdef ladder and move each define into the appropriate header file. Signed-off-by: Richard Henderson --- tcg/aarch64/tcg-target.h | 1 + tcg/arm/tcg-target.h | 1 + tcg/i386/tcg-target.h| 2 ++ tcg/mips/tcg-target.h| 6 ++

Re: [PATCH 1/2] hw/mips/jazz: Use generic I/O bus via get_system_io()

2021-03-10 Thread Jiaxun Yang
On Thu, Mar 11, 2021, at 3:12 AM, Philippe Mathieu-Daudé wrote: > No need to create a local ISA I/O MemoryRegion, use get_system_io(). > > This partly reverts commit 5c63bcf7501527b844f61624957bdba254d75bfc. > > Signed-off-by: Philippe Mathieu-Daudé No luck to boot arcboot with current jazz

Re: [PATCH 00/26] tcg: Workaround macOS 11.2 mprotect bug

2021-03-10 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20210311002156.253711-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: 20210311002156.253711-1-richard.hender...@linaro.org Subject: [PATCH 00/26] tcg

Re: [PATCH] vfio: Support host translation granule size

2021-03-10 Thread Keqian Zhu
Hi Alex, On 2021/3/11 4:24, Alex Williamson wrote: > On Wed, 10 Mar 2021 15:19:33 +0800 > Kunkun Jiang wrote: > >> Hi Alex, >> >> On 2021/3/10 7:17, Alex Williamson wrote: >>> On Thu, 4 Mar 2021 21:34:46 +0800 >>> Kunkun Jiang wrote: >>> The cpu_physical_memory_set_dirty_lebitmap() can q

Re: [PULL 00/20] ppc-for-6.0 queue 20210310

2021-03-10 Thread Bin Meng
> > > b2ae1009d7cca2701e17eae55ae2d44fd22c942a: > > > > > > Merge remote-tracking branch > > > 'remotes/mcayland/tags/qemu-sparc-20210307' into staging (2021-03-09 > > > 13:50:35 +) > > > > > > are available in the Git reposi

Re: [RFC PATCH 18/21] contrib/gitdm: Add Wind River to the domain map

2021-03-10 Thread Bin Meng
Hi Philippe, On Thu, Oct 8, 2020 at 7:13 PM Bin Meng wrote: > > Hi Philippe, > > On Mon, Oct 5, 2020 at 2:05 AM Philippe Mathieu-Daudé wrote: > > > > There is a number of contributors from this domain, > > add its own entry to the gitdm domain map. > > > > Cc: Bill Paul > > Cc: Bin Meng > > Cc

Re: [PULL 00/20] ppc-for-6.0 queue 20210310

2021-03-10 Thread David Gibson
emu-sparc-20210307' > > into staging (2021-03-09 13:50:35 +) > > > > are available in the Git repository at: > > > >https://gitlab.com/dgibson/qemu.git tags/ppc-for-6.0-20210310 > > > > for you to fetch changes up to eb7f80fd26d73e7e1af105431da

Re: [PATCH v4 00/78] target/arm: Implement SVE2

2021-03-10 Thread Richard Henderson
On 3/10/21 2:17 PM, Peter Maydell wrote: On Tue, 9 Mar 2021 at 16:20, Richard Henderson wrote: After a 6-month hiatus, sve2 is back. This time, with RISU testing vs FVP 11.13.36. Based-on: 20210309155305.11301-1-richard.hender...@linaro.org ("target/arm: sve1 fixes") Are you hoping to sque

Question about edge-triggered interrupt

2021-03-10 Thread LIU Zhiwei
Hi folks, Currently, I am writing an interrupt controller (CLIC) for RISC-V. I can't find a good way to process edge-triggered interrupt. According to edge-triggered definition, if I select an edge-triggered  interrupt to serve , it will clean its pending status. However after serving the in

Re: [RFC PATCH v3 02/10] net: Pad short frames to minimum size before send from SLiRP/TAP

2021-03-10 Thread Jason Wang
On 2021/3/9 6:13 下午, Peter Maydell wrote: On Tue, 9 Mar 2021 at 09:01, Bin Meng wrote: Hi Jason, On Tue, Mar 9, 2021 at 5:00 PM Bin Meng wrote: Hi Jason, On Tue, Mar 9, 2021 at 4:57 PM Jason Wang wrote: On 2021/3/9 4:35 下午, Bin Meng wrote: Hi Jason, On Tue, Mar 9, 2021 at 4:23 PM Jas

Re: [PATCH v2 07/12] hw/block/pflash_cfi02: Factor out pflash_reset_state_machine()

2021-03-10 Thread Bin Meng
On Thu, Mar 11, 2021 at 1:36 AM Philippe Mathieu-Daudé wrote: > > There is multiple places resetting the internal state machine. > Factor the code out in a new pflash_reset_state_machine() method. > > Signed-off-by: Philippe Mathieu-Daudé > --- > hw/block/pflash_cfi02.c | 20 +++-

Re: [PATCH v2 08/12] hw/block/pflash_cfi02: Add DeviceReset method

2021-03-10 Thread Bin Meng
On Thu, Mar 11, 2021 at 1:22 AM Philippe Mathieu-Daudé wrote: > > Signed-off-by: Philippe Mathieu-Daudé > --- > hw/block/pflash_cfi02.c | 8 > 1 file changed, 8 insertions(+) > Reviewed-by: Bin Meng

Re: [PATCH v2 11/12] hw/block/pflash_cfi01: Correct the type of PFlashCFI01.ro

2021-03-10 Thread Bin Meng
On Thu, Mar 11, 2021 at 1:28 AM Philippe Mathieu-Daudé wrote: > > From: David Edmondson > > PFlashCFI01.ro is a bool, declare it as such. > > Signed-off-by: David Edmondson > Reviewed-by: Philippe Mathieu-Daudé > Message-Id: <20210216142721.1985543-3-david.edmond...@oracle.com> > Signed-off-by:

Re: [PATCH v2 12/12] hw/block/pflash_cfi: Replace DPRINTF with trace events

2021-03-10 Thread Bin Meng
On Thu, Mar 11, 2021 at 1:43 AM Philippe Mathieu-Daudé wrote: > > From: David Edmondson > > Rather than having a device specific debug implementation in > pflash_cfi01.c and pflash_cfi02.c, use the standard tracing facility. > > Signed-off-by: David Edmondson > Reviewed-by: Philippe Mathieu-Daud

Re: [PATCH v2 10/12] hw/block/pflash_cfi01: Extract pflash_mode_read_array()

2021-03-10 Thread Bin Meng
On Thu, Mar 11, 2021 at 1:37 AM Philippe Mathieu-Daudé wrote: > > The same pattern is used when setting the flash in READ_ARRAY mode: > - Set the state machine command to READ_ARRAY > - Reset the write_cycle counter > - Reset the memory region in ROMD > > Refactor the current code by extracting th

  1   2   3   4   5   >