Re: [PATCH v4 1/3] target/nios2: Eliminate nios2_semi_is_lseek

2022-06-09 Thread Peter Maydell
On Wed, 8 Jun 2022 at 03:40, Richard Henderson wrote: > > Reorg nios2_semi_return_* to gdb_syscall_complete_cb. > Use the 32-bit version normally, and the 64-bit version > for HOSTED_LSEEK. > > Signed-off-by: Richard Henderson Reviewed-by: Peter Maydell thanks -- PMM

Re: [PATCH 28/50] pckbd: move mapping of I8042_MMIO registers to MIPS magnum machine

2022-06-09 Thread Peter Maydell
On Sun, 22 May 2022 at 19:19, Mark Cave-Ayland wrote: > > Now that the register memory region is exposed as a SysBus memory region, move > the mapping of the I8042_MMIO registers from i8042_mm_init() to the MIPS > magnum > machine (which is its only user). > > Signed-off-by: Mark Cave-Ayland >

Re: [PATCH 21/50] pckbd: introduce new I8042_MMIO QOM type

2022-06-09 Thread Peter Maydell
On Sun, 22 May 2022 at 19:19, Mark Cave-Ayland wrote: > > Currently i8042_mm_init() creates a new KBDState directly which is used by > the MIPS > magnum machine. Introduce a new I8042_MMIO QOM type that will soon be used to > allow the MIPS magnum machine to be wired up using standard qdev

Re: [PATCH 27/50] pckbd: alter i8042_mm_init() to return a I8042_MMIO device

2022-06-09 Thread Peter Maydell
On Sun, 22 May 2022 at 19:19, Mark Cave-Ayland wrote: > > This exposes the I8042_MMIO device to the caller to allow the register memory > region to be mapped outside of i8042_mm_init(). > > Signed-off-by: Mark Cave-Ayland Reviewed-by: Peter Maydell I'm not sure where best to put this review

Re: [PATCH v3 2/3] ui: Deliver refresh rate via QemuUIInfo

2022-06-09 Thread Gerd Hoffmann
> --- a/include/ui/console.h > +++ b/include/ui/console.h > @@ -139,6 +139,7 @@ typedef struct QemuUIInfo { > int yoff; > uint32_t width; > uint32_t height; > +uint32_t refresh_rate; > } QemuUIInfo; > > /* cursor data format is 32bit RGBA */ > @@ -426,7 +427,6 @@

Re: [PATCH v4 3/3] target/nios2: Use semihosting/syscalls.h

2022-06-09 Thread Peter Maydell
On Wed, 8 Jun 2022 at 03:41, Richard Henderson wrote: > > This separates guest file descriptors from host file descriptors, > and utilizes shared infrastructure for integration with gdbstub. > > Signed-off-by: Richard Henderson The commit message presents this as just a no-behaviour-change

Re: [PATCH v15 6/9] default-configs: Add loongarch linux-user support

2022-06-09 Thread WANG Xuerui
On 2022/6/9 10:42, Song Gao wrote: This patch adds loongarch64 linux-user default configs file. Signed-off-by: Song Gao Signed-off-by: Xiaojuan Yang Reviewed-by: Richard Henderson --- configs/targets/loongarch64-linux-user.mak | 3 +++ 1 file changed, 3 insertions(+) create mode 100644

Re: [PATCH 24/50] pckbd: add size qdev property to I8042_MMIO device

2022-06-09 Thread Peter Maydell
On Sun, 22 May 2022 at 19:19, Mark Cave-Ayland wrote: > > This will soon be used to set the size of the register memory region using a > qdev property. > > Signed-off-by: Mark Cave-Ayland > --- > hw/input/pckbd.c | 2 ++ > include/hw/input/i8042.h | 1 + Reviewed-by: Peter Maydell

Re: [PULL 15/18] qapi: introduce x-query-ramblock QMP command

2022-06-09 Thread David Hildenbrand
On 09.06.22 12:19, Daniel P. Berrangé wrote: > On Thu, Jun 09, 2022 at 12:07:31PM +0200, Claudio Fontana wrote: >> Hello all, >> >> it would be really good to be able to rely on this command or something >> similar, >> to be able to know the approximate size of a migration before starting it. >>

Re: [PATCH] hw/usb/hcd-ehci.c: print diagnostics when cpage out of range

2022-06-09 Thread Gerd Hoffmann
Hi, > if (cpage > 4) { > fprintf(stderr, "cpage out of range (%u)\n", cpage); > +bytes = get_field(p->qtd.token, QTD_TOKEN_TBYTES); > +offset = p->qtd.bufptr[0] & ~QTD_BUFPTR_MASK; > +cpage = get_field(p->qtd.token, QTD_TOKEN_CPAGE); >

Re: [PATCH 07/20] migration: rename qemu_update_position to qemu_file_credit_transfer

2022-06-09 Thread Dr. David Alan Gilbert
* Daniel P. Berrangé (berra...@redhat.com) wrote: > The qemu_update_position method name gives the misleading impression > that it is changing the current file offset. Most of the files are > just streams, however, so there's no concept of a file offset in the > general case. > > What this method

Re: [PATCH v3 0/3] Misc AC97 clean ups

2022-06-09 Thread Gerd Hoffmann
On Wed, May 18, 2022 at 12:37:18PM +0200, Paolo Bonzini wrote: > On 5/17/22 21:47, BALATON Zoltan wrote: > > On Mon, 9 May 2022, BALATON Zoltan wrote: > > > On Mon, 2 May 2022, BALATON Zoltan wrote: > > > > On Sat, 23 Apr 2022, BALATON Zoltan wrote: > > > > > During trying to implement via-ac97 I

Re: [PATCH 06/20] migration: rename qemu_ftell to qemu_file_total_transferred

2022-06-09 Thread Dr. David Alan Gilbert
* Daniel P. Berrangé (berra...@redhat.com) wrote: > The name 'ftell' gives the misleading impression that the QEMUFile > objects are seekable. This is not the case, as in general we just > have an opaque stream. The users of this method are only interested > in the total bytes processed. This

Re: [PULL 15/18] qapi: introduce x-query-ramblock QMP command

2022-06-09 Thread Daniel P . Berrangé
On Thu, Jun 09, 2022 at 12:07:31PM +0200, Claudio Fontana wrote: > Hello all, > > it would be really good to be able to rely on this command or something > similar, > to be able to know the approximate size of a migration before starting it. > > in QEMU ram_bytes_total() returns what I would

Re: [PATCH] usbredir: avoid queuing hello packet on snapshot restore

2022-06-09 Thread Gerd Hoffmann
On Fri, May 06, 2022 at 09:18:50PM -0700, Joelle van Dyne wrote: > When launching QEMU with "-loadvm", usbredir_create_parser() should avoid > setting up the hello packet (just as with "-incoming". On the latest version > of libusbredir, usbredirparser_unserialize() will return error if the parser

Re: [PULL 15/18] qapi: introduce x-query-ramblock QMP command

2022-06-09 Thread Claudio Fontana
Hello all, it would be really good to be able to rely on this command or something similar, to be able to know the approximate size of a migration before starting it. in QEMU ram_bytes_total() returns what I would like to have, but there is currently no QMP way to get it without starting a

Re: [PATCH] ui: move 'pc-bios/keymaps' to 'ui/keymaps'

2022-06-09 Thread Gerd Hoffmann
On Tue, May 24, 2022 at 10:48:14AM +0100, Daniel P. Berrangé wrote: > Ping, any thoughts on this ? > > On Tue, Apr 26, 2022 at 05:01:50PM +0100, Daniel P. Berrangé wrote: > > The 'keymaps' directory contents is nothing to do with the firmware > > blobs. The 'pc-bios/keymaps' directory appears to

Re: [PATCH v15 8/9] target/loongarch: Adjust functions and structure to support user-mode

2022-06-09 Thread WANG Xuerui
On 2022/6/9 10:42, Song Gao wrote: Some functions and member of the structure are different with softmmu-mode So we need adjust them to support user-mode. Signed-off-by: Song Gao Signed-off-by: Xiaojuan Yang --- target/loongarch/cpu.c| 22 ++--

Re: [PATCH] hw/usb/hcd-ehci: fix writeback order

2022-06-09 Thread Gerd Hoffmann
On Sun, May 08, 2022 at 05:32:22PM +0200, Arnout Engelen wrote: > The 'active' bit passes control over a qTD between the guest and the > controller: set to 1 by guest to enable execution by the controller, > and the controller sets it to '0' to hand back control to the guest. > >

Re: [PATCH 1/2] ui/gtk: detach VCS for additional guest displays

2022-06-09 Thread Daniel P . Berrangé
On Tue, May 31, 2022 at 01:23:26PM -0700, Dongwon Kim wrote: > Detaching any addtional guest displays in case there are multiple > displays assigned to the guest OS (e.g. max_outputs=n) so that > all of them are visible upon lauching. > > Cc: Daniel P. Berrangé > Cc: Markus Armbruster > Cc:

Re: [PATCH 05/20] migration: rename 'pos' field in QEMUFile to 'bytes_processed'

2022-06-09 Thread Daniel P . Berrangé
On Thu, Jun 09, 2022 at 10:51:27AM +0100, Dr. David Alan Gilbert wrote: > * Daniel P. Berrangé (berra...@redhat.com) wrote: > > This makes the field name align with the newly introduced method > > names in the previous commit. > > I think that's the method in the following commits? Opps, yeah, I

Re: [PATCH 05/20] migration: rename 'pos' field in QEMUFile to 'bytes_processed'

2022-06-09 Thread Dr. David Alan Gilbert
* Daniel P. Berrangé (berra...@redhat.com) wrote: > On Thu, Jun 09, 2022 at 10:51:27AM +0100, Dr. David Alan Gilbert wrote: > > * Daniel P. Berrangé (berra...@redhat.com) wrote: > > > This makes the field name align with the newly introduced method > > > names in the previous commit. > > > > I

Re: [PATCH] hw/usb/hcd-ehci.c: print diagnostics when cpage out of range

2022-06-09 Thread Arnout Engelen
On Thu, Jun 9, 2022, at 12:09, Gerd Hoffmann wrote: > Hi, > > > if (cpage > 4) { > > fprintf(stderr, "cpage out of range (%u)\n", cpage); > > +bytes = get_field(p->qtd.token, QTD_TOKEN_TBYTES); > > +offset = p->qtd.bufptr[0] & ~QTD_BUFPTR_MASK; > >

Re: [PATCH 2/2] ui/gtk: a new array param monitor to specify the target displays

2022-06-09 Thread Gerd Hoffmann
On Tue, May 31, 2022 at 01:23:27PM -0700, Dongwon Kim wrote: > New integer array parameter, 'monitor' is for specifying the target > displays where individual QEMU windows are placed upon launching. > > The array contains a series of numbers representing the monitor where > QEMU windows are

Re: [PATCH 05/20] migration: rename 'pos' field in QEMUFile to 'bytes_processed'

2022-06-09 Thread Dr. David Alan Gilbert
* Daniel P. Berrangé (berra...@redhat.com) wrote: > This makes the field name align with the newly introduced method > names in the previous commit. I think that's the method in the following commits? tbh I'm not sure about this; 'pos' is still passed to writev_buffer and get_buffer to say where

Re: [PATCH v15 4/9] linux-user: Add LoongArch syscall support

2022-06-09 Thread WANG Xuerui
On 2022/6/9 10:42, Song Gao wrote: We should disable '__BITS_PER_LONG' at [1] before run gensyscalls.sh [1] arch/loongarch/include/uapi/asm/bitsperlong.h I'm not sure why this is necessary, is this for building on 32-bit where __BITS_PER_LONG are (incorrectly) reflecting the host bitness?

Re: [PATCH v5 4/6] docs: Add CanoKey documentation

2022-06-09 Thread Gerd Hoffmann
On Thu, May 19, 2022 at 08:39:38PM +0800, Hongren (Zenithal) Zheng wrote: > Signed-off-by: Hongren (Zenithal) Zheng > --- > docs/system/device-emulation.rst | 1 + > docs/system/devices/canokey.rst | 168 +++ > 2 files changed, 169 insertions(+) > create mode

Re: [PATCH v6 11/18] job.h: rename job API functions called with job_mutex held

2022-06-09 Thread Stefan Hajnoczi
On Mon, Mar 14, 2022 at 09:37:00AM -0400, Emanuele Giuseppe Esposito wrote: > /** > - * Release a reference that was previously acquired with job_ref() or > + * Release a reference that was previously acquired with job_ref_locked() or > * job_create(). If it's the last reference to the object,

Re: [PATCH v15 1/9] linux-user: Add LoongArch generic header files

2022-06-09 Thread WANG Xuerui
On 2022/6/9 10:42, Song Gao wrote: This includes: - sockbits.h - target_errno_defs.h - target_fcntl.h - termbits.h - target_resource.h - target_structs.h Signed-off-by: Song Gao Signed-off-by: Xiaojuan Yang Reviewed-by: Richard Henderson Reviewed-by: Philippe Mathieu-Daudé ---

Re: [PATCH v6 10/18] jobs: rename static functions called with job_mutex held

2022-06-09 Thread Stefan Hajnoczi
On Mon, Mar 14, 2022 at 09:36:59AM -0400, Emanuele Giuseppe Esposito wrote: > @@ -530,20 +540,24 @@ void job_enter(Job *job) > job_enter_cond(job, NULL); > } > > -/* Yield, and schedule a timer to reenter the coroutine after @ns > nanoseconds. > +/* > + * Yield, and schedule a timer to

Re: [PATCH] ui/console: allow display device to be labeled with given id

2022-06-09 Thread Gerd Hoffmann
On Thu, May 26, 2022 at 07:08:14AM +, Wen, Jianxian wrote: > The update makes it easier to find and specify devices. > They can only be found by device type name without the id field, > for example, devices of the same type have the same label. > The update also adds a head field, > which is

Re: [PATCH 2/2] ui/gtk: a new array param monitor to specify the target displays

2022-06-09 Thread Daniel P . Berrangé
On Tue, May 31, 2022 at 01:23:27PM -0700, Dongwon Kim wrote: > New integer array parameter, 'monitor' is for specifying the target > displays where individual QEMU windows are placed upon launching. > > The array contains a series of numbers representing the monitor where > QEMU windows are

[PULL 55/55] target/arm: Add ID_AA64SMFR0_EL1

2022-06-09 Thread Peter Maydell
From: Richard Henderson This register is allocated from the existing block of id registers, so it is already RES0 for cpus that do not implement SME. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-21-richard.hender...@linaro.org Signed-off-by:

Re: [PATCH 0/2] Fixes for ui/gtk-gl-area

2022-06-09 Thread Gerd Hoffmann
On Sun, Jun 05, 2022 at 10:50:28AM +0200, Volker Rümelin wrote: > The first patch fixes a GL context leak. > > The second patch fixes a black guest screen on Wayland with OpenGL > accelerated QEMU graphics devices. This bug doesn't seem to be related to > issues #910, #865, #671 or #298. Both

Re: [PATCH v6 12/18] block_job: rename block_job functions called with job_mutex held

2022-06-09 Thread Stefan Hajnoczi
On Mon, Mar 14, 2022 at 09:37:01AM -0400, Emanuele Giuseppe Esposito wrote: > @@ -135,32 +137,37 @@ void block_job_remove_all_bdrv(BlockJob *job); > bool block_job_has_bdrv(BlockJob *job, BlockDriverState *bs); > > /** > - * block_job_set_speed: > + * block_job_set_speed_locked: > * @job:

[PULL 42/55] target/arm: Use el_is_in_host for sve_exception_el

2022-06-09 Thread Peter Maydell
From: Richard Henderson The ARM pseudocode function CheckNormalSVEEnabled uses this predicate now, and I think it's a bit clearer. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-8-richard.hender...@linaro.org Signed-off-by: Peter Maydell ---

Re: [PATCH 04/20] migration: rename rate limiting fields in QEMUFile

2022-06-09 Thread Dr. David Alan Gilbert
* Daniel P. Berrangé (berra...@redhat.com) wrote: > This renames the following QEMUFile fields > > * bytes_xfer -> rate_limit_used > * xfer_limit -> rate_limit_max > > The intent is to make it clear that 'bytes_xfer' is specifically related > to rate limiting of data and applies to data

Re: [PATCH v6 13/18] job.h: define unlocked functions

2022-06-09 Thread Stefan Hajnoczi
On Mon, Mar 14, 2022 at 09:37:02AM -0400, Emanuele Giuseppe Esposito wrote: > /** > * @job: The job to enter. > + * Called with job_mutex *not* held. > * > * Continue the specified job by entering the coroutine. > + * Called with job_mutex lock *not* held. A similar comment was added just

[PULL 41/55] target/arm: Use el_is_in_host for sve_zcr_len_for_el

2022-06-09 Thread Peter Maydell
From: Richard Henderson The ARM pseudocode function NVL uses this predicate now, and I think it's a bit clearer. Simplify the pseudocode condition by noting that IsInHost is always false for EL1. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id:

Re: [PATCH v6 03/18] job.c: API functions not used outside should be static

2022-06-09 Thread Stefan Hajnoczi
On Mon, Mar 14, 2022 at 09:36:52AM -0400, Emanuele Giuseppe Esposito wrote: > job_event_* functions can all be static, as they are not used > outside job.c. > > Same applies for job_txn_add_job(). > > Signed-off-by: Emanuele Giuseppe Esposito > --- > include/qemu/job.h | 18 --

[PULL 37/55] linux-user/aarch64: Introduce sve_vq

2022-06-09 Thread Peter Maydell
From: Richard Henderson Add an interface function to extract the digested vector length rather than the raw zcr_el[1] value. This fixes an incorrect return from do_prctl_set_vl where we didn't take into account the set of vector lengths supported by the cpu. Reviewed-by: Peter Maydell

[PATCH 4/4] virtio-mmio: cleanup reset

2022-06-09 Thread Paolo Bonzini
Make virtio_mmio_soft_reset reset the virtio device, which is performed by both the "soft" and the "hard" reset; and then call virtio_mmio_soft_reset from virtio_mmio_reset to emphasize that the latter is a superset of the former. Signed-off-by: Paolo Bonzini --- hw/virtio/virtio-mmio.c | 17

[PULL 34/55] target/arm: Move stage_1_mmu_idx, arm_stage1_mmu_idx to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-28-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/helper.c | 32 target/arm/ptw.c| 28

[PULL 51/55] target/arm: Use expand_pred_b in mve_helper.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Use the function instead of the array directly. Because the function performs its own masking, via the uint8_t parameter, we need to do nothing extra within the users: the bits above the first 2 (_uh) or 4 (_uw) will be discarded by assignment to the local bmask

Re: [PATCH v6 05/18] job.h: add _locked duplicates for job API functions called with and without job_mutex

2022-06-09 Thread Stefan Hajnoczi
On Mon, Mar 14, 2022 at 09:36:54AM -0400, Emanuele Giuseppe Esposito wrote: > In preparation to the job_lock/unlock usage, create _locked > duplicates of some functions, since they will be sometimes called with > job_mutex held (mostly within job.c), > and sometimes without (mostly from JobDrivers

[PATCH 3/4] virtio: stop ioeventfd on reset

2022-06-09 Thread Paolo Bonzini
All calls to virtio_bus_reset are preceded by virtio_bus_stop_ioeventfd, move the call in virtio_bus_reset: that makes sense and clarifies that the vdc->reset function is called with ioeventfd already stopped. Signed-off-by: Paolo Bonzini --- hw/s390x/virtio-ccw.c | 1 -

[PULL 24/55] target/arm: Move {arm_s1_, }regime_using_lpae_format to tlb_helper.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson These functions are used for both page table walking and for deciding what format in which to deliver exception results. Since ptw.c is only present for system mode, put the functions into tlb_helper.c. Signed-off-by: Richard Henderson Message-id:

[PULL 54/55] target/arm: Add isar_feature_aa64_sme

2022-06-09 Thread Peter Maydell
From: Richard Henderson This will be used for implementing FEAT_SME. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-20-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/cpu.h | 5 + 1 file changed, 5 insertions(+)

[PULL 30/55] target/arm: Move regime_is_user to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-24-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 1 - target/arm/helper.c | 24 target/arm/ptw.c| 22

Re: [PATCH v2 00/35] pc/q35: refactor ISA and SMBUS AML generation

2022-06-09 Thread Igor Mammedov
On Wed, 8 Jun 2022 09:53:05 -0400 Igor Mammedov wrote: > Changelog: > since v1: > * add tis 2.0 clarification to commit message (Ani Sinha) > * rebase on top of pci tree > * pick up acks tests fail due to new cxl testcase, so I need to fixup whitelisting/blob updating patches

[PATCH 2/4] virtio-mmio: stop ioeventfd on legacy reset

2022-06-09 Thread Paolo Bonzini
If the queue PFN is set to zero on a virtio-mmio device, the device is reset. In that case however the virtio_bus_stop_ioeventfd function was not called; add it so that the behavior is similar to when status is set to 0. Signed-off-by: Paolo Bonzini --- hw/virtio/virtio-mmio.c | 1 + 1 file

Re: [PATCH] ui/cocoa: Fix poweroff request code

2022-06-09 Thread Gerd Hoffmann
On Sun, May 29, 2022 at 12:45:06PM +0200, Philippe Mathieu-Daudé wrote: > On 29/5/22 10:25, Akihiko Odaki wrote: > > Signed-off-by: Akihiko Odaki > > --- > > ui/cocoa.m | 6 +- > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > diff --git a/ui/cocoa.m b/ui/cocoa.m > > index

[PATCH 0/4] virtio: various cleanups to reset code

2022-06-09 Thread Paolo Bonzini
Patches 1, 3 and 4 are cleanups with no functional changes (intended, at least). Patch 2 is a small fix to legacy virtio-mmio reset, whose behavior differed slightly compared to zeroing the status of the device. Paolo Paolo Bonzini (4): s390x: simplify virtio_ccw_reset_virtio virtio-mmio:

[PULL 40/55] target/arm: Add el_is_in_host

2022-06-09 Thread Peter Maydell
From: Richard Henderson This (newish) ARM pseudocode function is easier to work with than open-coded tests for HCR_E2H etc. Use of the function will be staged into the code base in parts. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id:

[PULL 32/55] target/arm: Move regime_translation_disabled to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-26-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 17 target/arm/helper.c | 47

[PULL 53/55] target/arm: Export bfdotadd from vec_helper.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson We will need this over in sme_helper.c. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-19-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/vec_internal.h | 13 +

[PULL 31/55] target/arm: Move regime_ttbr to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-25-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 1 - target/arm/helper.c | 16 target/arm/ptw.c| 16

[PULL 49/55] target/arm: Export sve contiguous ldst support functions

2022-06-09 Thread Peter Maydell
From: Richard Henderson Export all of the support functions for performing bulk fault analysis on a set of elements at contiguous addresses controlled by a predicate. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-15-richard.hender...@linaro.org

[PATCH 1/4] s390x: simplify virtio_ccw_reset_virtio

2022-06-09 Thread Paolo Bonzini
Call virtio_bus_reset instead of virtio_reset, so that the function need not receive the VirtIODevice. Signed-off-by: Paolo Bonzini --- hw/s390x/virtio-ccw.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c index

[PULL 23/55] target/arm: Move arm_{ldl,ldq}_ptw to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Move the ptw load functions, plus 3 common subroutines: S1_ptw_translate, ptw_attrs_are_device, and regime_translation_big_endian. This also allows get_phys_addr_lpae to become static again. Signed-off-by: Richard Henderson Message-id:

[PULL 52/55] target/arm: Move expand_pred_h to vec_internal.h

2022-06-09 Thread Peter Maydell
From: Richard Henderson Move the data to vec_helper.c and the inline to vec_internal.h. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-18-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/vec_internal.h | 7 +++

[PULL 47/55] target/arm: Rename sve_zcr_len_for_el to sve_vqm1_for_el

2022-06-09 Thread Peter Maydell
From: Richard Henderson This will be used for both Normal and Streaming SVE, and the value does not necessarily come from ZCR_ELx. While we're at it, emphasize the units in which the value is returned. Patch produced by git grep -l sve_zcr_len_for_el | \ xargs -n1 sed -i

[PULL 20/55] target/arm: Move get_level1_table_address to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-14-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 4 ++-- target/arm/helper.c | 26 +- target/arm/ptw.c| 23

[PULL 50/55] target/arm: Move expand_pred_b to vec_internal.h

2022-06-09 Thread Peter Maydell
From: Richard Henderson Put the inline function near the array declaration. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-16-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/vec_internal.h | 8 +++-

[PULL 46/55] target/arm: Use uint32_t instead of bitmap for sve vq's

2022-06-09 Thread Peter Maydell
From: Richard Henderson The bitmap need only hold 15 bits; bitmap is over-complicated. We can simplify operations quite a bit with plain logical ops. The introduction of SVE_VQ_POW2_MAP eliminates the need for looping in order to search for powers of two. Simply perform the logical ops and use

[PULL 43/55] target/arm: Hoist arm_is_el2_enabled check in sve_exception_el

2022-06-09 Thread Peter Maydell
From: Richard Henderson This check is buried within arm_hcr_el2_eff(), but since we have to have the explicit check for CPTR_EL2.TZ, we might as well just check it once at the beginning of the block. Once this is done, we can test HCR_EL2.{E2H,TGE} directly, rather than going through

[PULL 14/55] target/arm: Move get_phys_addr_pmsav7 to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-8-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 10 +-- target/arm/helper.c | 194 +---

[PULL 48/55] target/arm: Split out load/store primitives to sve_ldst_internal.h

2022-06-09 Thread Peter Maydell
From: Richard Henderson Begin creation of sve_ldst_internal.h by moving the primitives that access host and tlb memory. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-14-richard.hender...@linaro.org Signed-off-by: Peter Maydell ---

[PULL 38/55] target/arm: Remove route_to_el2 check from sve_exception_el

2022-06-09 Thread Peter Maydell
From: Richard Henderson We handle this routing in raise_exception. Promoting the value early means that we can't directly compare FPEXC_EL and SVEEXC_EL. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-4-richard.hender...@linaro.org

[PULL 35/55] target/arm: Pass CPUARMState to arm_ld[lq]_ptw

2022-06-09 Thread Peter Maydell
From: Richard Henderson The use of ARM_CPU to recover env from cs calls object_class_dynamic_cast, which shows up on the profile. This is pointless, because all callers already have env, and the reverse operation, env_cpu, is only pointer arithmetic. Signed-off-by: Richard Henderson

[PULL 17/55] target/arm: Move pmsav7_use_background_region to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-11-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 2 -- target/arm/helper.c | 19 --- target/arm/ptw.c| 21

[PULL 45/55] target/arm: Merge aarch64_sve_zcr_get_valid_len into caller

2022-06-09 Thread Peter Maydell
From: Richard Henderson This function is used only once, and will need modification for Streaming SVE mode. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-11-richard.hender...@linaro.org Signed-off-by: Peter Maydell --- target/arm/internals.h |

[PULL 27/55] target/arm: Move check_s2_mmu_setup to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-21-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 2 -- target/arm/helper.c | 70 -

[PULL 33/55] target/arm: Move arm_cpu_get_phys_page_attrs_debug to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-27-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/helper.c | 26 -- target/arm/ptw.c| 24 2

[PULL 16/55] target/arm: Move pmsav8_mpu_lookup to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson This is the final user of get_phys_addr_pmsav7_default within helper.c, so make it static within ptw.c. Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-10-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell ---

[PULL 44/55] target/arm: Do not use aarch64_sve_zcr_get_valid_len in reset

2022-06-09 Thread Peter Maydell
From: Richard Henderson We don't need to constrain the value set in zcr_el[1], because it will be done by sve_zcr_len_for_el. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-10-richard.hender...@linaro.org Signed-off-by: Peter Maydell ---

[PULL 25/55] target/arm: Move arm_pamax, pamax_map into ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-19-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 2 -- target/arm/helper.c | 25 - target/arm/ptw.c| 25

[PULL 29/55] target/arm: Move ap_to_tw_prot etc to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-23-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 10 -- target/arm/helper.c | 77 --

[PULL 09/55] target/arm: Move get_phys_addr to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Begin moving all of the page table walking functions out of helper.c, starting with get_phys_addr(). Create a temporary header file, "ptw.h", in which to share declarations between the two C files while we are moving functions. Move a few declarations to "internals.h",

[PULL 26/55] target/arm: Move get_S1prot, get_S2prot to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-20-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 3 -- target/arm/helper.c | 128

[PULL 22/55] target/arm: Move get_phys_addr_lpae to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-16-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 10 ++ target/arm/helper.c | 416 +---

[PULL 21/55] target/arm: Move combine_cacheattrs and subroutines to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson There are a handful of helpers for combine_cacheattrs that we can move at the same time as the main entry point. Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-15-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell

[PULL 39/55] target/arm: Remove fp checks from sve_exception_el

2022-06-09 Thread Peter Maydell
From: Richard Henderson Instead of checking these bits in fp_exception_el and also in sve_exception_el, document that we must compare the results. The only place where we have not already checked that FP EL is zero is in rebuild_hflags_a64. Reviewed-by: Peter Maydell Signed-off-by: Richard

[PULL 19/55] target/arm: Move m_is_{ppb,system}_region to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-13-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 3 --- target/arm/helper.c | 15 --- target/arm/ptw.c| 16

[PULL 07/55] xlnx-zynqmp: fix the irq mapping for the display port and its dma

2022-06-09 Thread Peter Maydell
From: Frederic Konrad When the display port has been initially implemented the device driver wasn't using interrupts. Now that the display port driver waits for vblank interrupt it has been noticed that the irq mapping is wrong. So use the value from the linux device tree and the ultrascale+

[PULL 08/55] target/arm: Move stage_1_mmu_idx decl to internals.h

2022-06-09 Thread Peter Maydell
From: Richard Henderson Move the decl from ptw.h to internals.h. Provide an inline version for user-only, just as we do for arm_stage1_mmu_idx. Move an endif down to make the definition in helper.c be system only. Signed-off-by: Richard Henderson Message-id:

[PULL 36/55] target/arm: Rename TBFLAG_A64 ZCR_LEN to VL

2022-06-09 Thread Peter Maydell
From: Richard Henderson With SME, the vector length does not only come from ZCR_ELx. Comment that this is either NVL or SVL, like the pseudocode. Reviewed-by: Peter Maydell Signed-off-by: Richard Henderson Message-id: 20220607203306.657998-2-richard.hender...@linaro.org Signed-off-by: Peter

[PULL 13/55] target/arm: Move get_phys_addr_pmsav7_default to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-7-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 3 +++ target/arm/helper.c | 41 -

[PULL 15/55] target/arm: Move get_phys_addr_pmsav8 to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-9-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 5 --- target/arm/helper.c | 75 ---

[PULL 28/55] target/arm: Move aa32_va_parameters to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-22-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 3 --- target/arm/helper.c | 64 -

[PULL 10/55] target/arm: Move get_phys_addr_v5 to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-4-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 15 +++-- target/arm/helper.c | 137 +++-

[PULL 12/55] target/arm: Move get_phys_addr_pmsav5 to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-6-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 4 --- target/arm/helper.c | 85 -

[PULL 06/55] xlnx_dp: Fix the interrupt disable logic

2022-06-09 Thread Peter Maydell
From: Sai Pavan Boddu Fix interrupt disable logic. Mask value 1 indicates that interrupts are disabled. Signed-off-by: Sai Pavan Boddu Reviewed-by: Edgar E. Iglesias Signed-off-by: Frederic Konrad Acked-by: Alistair Francis Message-id: 20220601172353.3220232-4-fkon...@xilinx.com

[PULL 03/55] Fix 'writeable' typos

2022-06-09 Thread Peter Maydell
We have about 30 instances of the typo/variant spelling 'writeable', and over 500 of the more common 'writable'. Standardize on the latter. Change produced with: sed -i -e 's/\([Ww][Rr][Ii][Tt]\)[Ee]\([Aa][Bb][Ll][Ee]\)/\1\2/g' $(git grep -il writeable) and then hand-undoing the instance in

[PULL 04/55] xlnx_dp: fix the wrong register size

2022-06-09 Thread Peter Maydell
From: Frederic Konrad The core and the vblend registers size are wrong, they should respectively be 0x3B0 and 0x1E0 according to: https://www.xilinx.com/htmldocs/registers/ug1087/ug1087-zynq-ultrascale-registers.html. Let's fix that and use macros when creating the mmio region. Fixes:

[PULL 11/55] target/arm: Move get_phys_addr_v6 to ptw.c

2022-06-09 Thread Peter Maydell
From: Richard Henderson Signed-off-by: Richard Henderson Message-id: 20220604040607.269301-5-richard.hender...@linaro.org Reviewed-by: Peter Maydell Signed-off-by: Peter Maydell --- target/arm/ptw.h| 11 +-- target/arm/helper.c | 161 +---

[PULL 01/55] target/arm: Declare support for FEAT_RASv1p1

2022-06-09 Thread Peter Maydell
The architectural feature RASv1p1 introduces the following new features: * new registers ERXPFGCDN_EL1, ERXPFGCTL_EL1 and ERXPFGF_EL1 * new bits in the fine-grained trap registers that control traps for these new registers * new trap bits HCR_EL2.FIEN and SCR_EL3.FIEN that control traps

[PULL 05/55] xlnx_dp: Introduce a vblank signal

2022-06-09 Thread Peter Maydell
From: Sai Pavan Boddu Add a periodic timer which raises vblank at a frequency of 30Hz. Note that this is a migration compatibility break for the xlnx-zcu102 board type. Signed-off-by: Sai Pavan Boddu Signed-off-by: Edgar E. Iglesias Signed-off-by: Frederic Konrad Acked-by: Alistair Francis

[PULL 02/55] target/arm: Implement FEAT_DoubleFault

2022-06-09 Thread Peter Maydell
The FEAT_DoubleFault extension adds the following: * All external aborts on instruction fetches and translation table walks for instruction fetches must be synchronous. For QEMU this is already true. * SCR_EL3 has a new bit NMEA which disables the masking of SError interrupts by

[PULL 00/55] target-arm queue

2022-06-09 Thread Peter Maydell
tags/pull-target-arm-20220609 for you to fetch changes up to 414c54d515dba16bfaef643a8acec200c05f229a: target/arm: Add ID_AA64SMFR0_EL1 (2022-06-08 19:38:59 +0100) target-arm queue: * target/arm: Declare support for FEAT_RASv1p1

<    1   2   3   4   5   >