Re: [PATCH v2 1/1] hw/riscv/boot: Check the error of fdt_pack()

2021-07-14 Thread Alistair Francis
On Thu, Jul 15, 2021 at 9:01 AM Alistair Francis wrote: > > Coverity reports that we don't check the error result of fdt_pack(), so > let's save the result and assert that it is 0. > > Fixes: Coverity CID 1458136 > Signed-off-by: Alistair Francis > Reviewed-by: Bin Meng Thanks! Applied to risc

Re: [PATCH v1 1/1] hw/riscv/boot: Check the error of fdt_pack()

2021-07-14 Thread Alistair Francis
On Wed, Jul 14, 2021 at 6:46 PM Peter Maydell wrote: > > On Wed, 14 Jul 2021 at 08:22, Alistair Francis > wrote: > > > > Coverity reports that we don't check the error result of fdt_pack(), so > > let's save the result and assert that it is 0. > > > > Fixes: Coverity CID 1458136 > > Signed-off-b

Re: [PULL 00/11] riscv-to-apply queue

2021-07-14 Thread Alistair Francis
On Wed, Jul 14, 2021 at 4:01 AM Peter Maydell wrote: > > On Mon, 12 Jul 2021 at 23:53, Alistair Francis > wrote: > > > > The following changes since commit 57e28d34c0cb04abf7683ac6a12c87ede447c320: > > > > Merge remote-tracking branch 'remotes/cohuck-gitlab/tags/s390x-20210708' > > into stagi

Re: [PATCH v1 2/9] s390x: toplogy: adding drawers and books to smp parsing

2021-07-14 Thread Markus Armbruster
Pierre Morel writes: > Drawers and Books are levels 4 and 3 of the S390 CPU > topology. > We allow the user to define these levels and we will > store the values inside the S390CcwMachineState. Double-checking: are these members specific to S390? > > Signed-off-by: Pierre Morel > --- [...] >

Re: [PATCH] Fix CPUID_Fn8000001E_EBX for AMD

2021-07-14 Thread 成家瑶
Hello, I am Jade Cheng working for ByteDance, sending this email is aimed at ping you guys to check the patch I submitted earlier, link is attached below: https://patchew.org/QEMU/20210630082551.12956-1-chengjia...@bytedance.com/ Please do me a favor to give it a review, and let me know if you ha

Re: [PATCH v2 02/21] docs: collect the disparate device emulation docs into one section

2021-07-14 Thread Markus Armbruster
Cc: QOM maintainers for additional eyes. Alex Bennée writes: > While we are at it add a brief preamble that explains some of the > common concepts in QEMU's device emulation which will hopefully lead > to less confusing about our dizzying command line options. > > Signed-off-by: Alex Bennée > M

Re: [RFC PATCH 5/6] i386/sev: add support to encrypt BIOS when SEV-SNP is enabled

2021-07-14 Thread Dov Murik
On 14/07/2021 21:52, Brijesh Singh wrote: > > > On 7/14/21 12:08 PM, Connor Kuehl wrote: >> On 7/9/21 3:55 PM, Brijesh Singh wrote: >>> The KVM_SEV_SNP_LAUNCH_UPDATE command is used for encrypting the bios >>> image used for booting the SEV-SNP guest. >>> >>> Signed-off-by: Brijesh Singh >>>

Re: About two-dimensional page translation (e.g., Intel EPT) and shadow page table in Linux QEMU/KVM

2021-07-14 Thread harry harry
Hi Sean, > No, each vCPU has its own MMU instance, where an "MMU instance" is (mostly) a > KVM > construct. Per-vCPU MMU instances are necessary because each vCPU has its own > relevant state, e.g. CR0, CR4, EFER, etc..., that affects the MMU instance in > some way. E.g. the MMU instance is use

Re: [PATCH v2 10/21] contrib/gitdm: add domain-map/group-map mappings for Samsung

2021-07-14 Thread Klaus Jensen
On Jul 14 19:20, Alex Bennée wrote: > Minwoo's work from their personal address are treated as personal > contributions. > > Signed-off-by: Alex Bennée > Acked-by: Klaus Jensen > Cc: Gollu Appalanaidu > Cc: Minwoo Im > Message-Id: <220210714093719.21429-11-alex.ben...@linaro.org> > --- > cont

Re: [PATCH v3 0/9] tests: Add test cases for TPM 1.2 ACPI tables

2021-07-14 Thread Markus Armbruster
Philippe Mathieu-Daudé writes: > On 7/14/21 4:43 PM, Markus Armbruster wrote: >> Philippe Mathieu-Daudé writes: >> >>> +Markus [...] >>> IMO the "right" solution is to check via QMP if TMP is supported >>> or not. This is now doable since commit caff255a546 ("tpm: Return >>> QMP error when TP

Re: [PATCH 17/17] target/riscv: Remove gen_get_gpr

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:48 PM Richard Henderson wrote: > > This function is now unused. > The corresponding gen_set_gpr function is still in use. > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/translate.c | 17 + > 1 file cha

Re: [PATCH 16/17] target/riscv: Use gpr_{src,dst} for RVV

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:43 PM Richard Henderson wrote: > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/insn_trans/trans_rvv.c.inc | 79 +++-- > 1 file changed, 20 insertions(+), 59 deletions(-) > > diff --git a/target/risc

Re: [PATCH 15/17] target/riscv: Use gen_arith for mulh and mulhu

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:40 PM Richard Henderson wrote: > > Split out gen_mulh and gen_mulhu and use the common helper. > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/translate.c| 16 > target/riscv/insn_trans

Re: [PATCH 13/17] target/riscv: Use gpr_{src,dst} for RVD

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:42 PM Richard Henderson wrote: > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/insn_trans/trans_rvd.c.inc | 116 +--- > 1 file changed, 44 insertions(+), 72 deletions(-) > > diff --git a/target/risc

Re: [PATCH 12/17] target/riscv: Use gpr_{src,dst} for RVF

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:49 PM Richard Henderson wrote: > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/insn_trans/trans_rvf.c.inc | 131 +--- > 1 file changed, 49 insertions(+), 82 deletions(-) > > diff --git a/target/risc

Re: [PATCH 11/17] target/riscv: Use gpr_{src,dst} for RVB

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:37 PM Richard Henderson wrote: > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/insn_trans/trans_rvb.c.inc | 11 --- > 1 file changed, 4 insertions(+), 7 deletions(-) > > diff --git a/target/riscv/insn_trans/tra

Re: [PATCH 10/17] target/riscv: Use gpr_{src,dst} for RVA

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:39 PM Richard Henderson wrote: > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/insn_trans/trans_rva.c.inc | 42 + > 1 file changed, 14 insertions(+), 28 deletions(-) > > diff --git a/target/risc

Re: [PATCH 08/17] target/riscv: Use gpr_{src, dst} for word shift operations

2021-07-14 Thread Alistair Francis
On Fri, Jul 9, 2021 at 2:35 PM Richard Henderson wrote: > > For trans_sllw, we can just use gen_shiftw. The others use > various tricks to reduce the tcg operation count. > > Signed-off-by: Richard Henderson Reviewed-by: Alistair Francis Alistair > --- > target/riscv/insn_trans/trans_rvi.c.

[Bug 1924914] Re: Running sway in a QEMU VM results in a GPU hang of the guest (virtio-gpu driver)

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1924914 Title: Running swa

[Bug 1924987] Re: Storage | Two decimal digits precision

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1924987 Title: Storage | T

[Bug 1925094] Re: DISCARD support for Crypto Block Devices

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1925094 Title: DISCARD sup

[Bug 1920934] Re: Heap-use-after-free in io_writex / cputlb.c results in Linux kernel crashes

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1920934 Title: Heap-use-af

[Bug 1925109] Re: usbredirparser: bulk transfer length exceeds limits

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1925109 Title: usbredirpar

[Bug 1926202] Re: qemu-user can't run some ppc binaries

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1926202 Title: qemu-user c

[Bug 1925966] Re: Win10 guest freezes randomly

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1925966 Title: Win10 guest

[Bug 1926596] Re: qemu-monitor-event command gets stuck randomly

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1926596 Title: qemu-monito

[Bug 1926231] Re: SCSI passthrough of SATA cdrom -> errors & performance issues

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1926231 Title: SCSI passth

[Bug 1922773] Re: RISCV32 illegal instruction exception

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1922773 Title: RISCV32 ill

[Bug 1926782] Re: configure script --extra-cflags not passed to config-meson.cross

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1926782 Title: configure s

[Bug 1926952] Re: SPICE support broken with 6.0

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1926952 Title: SPICE suppo

[Bug 1926174] Re: Laggy and/or displaced mouse input on CloudReady (Chrome OS) VM

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1926174 Title: Laggy and/o

[Bug 1926497] Re: dp83932 stops working after a short while

2021-07-14 Thread Launchpad Bug Tracker
[Expired for QEMU because there has been no activity for 60 days.] ** Changed in: qemu Status: Incomplete => Expired -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1926497 Title: dp83932 sto

Re: [PATCH V2 00/18] vhost-vDPA multiqueue

2021-07-14 Thread Jason Wang
在 2021/7/12 下午1:44, Jason Wang 写道: 在 2021/7/6 下午4:26, Jason Wang 写道: Hi All: This patch implements the multiqueue support for vhost-vDPA. The most important requirement the control virtqueue support. The virtio-net and vhost-net core are tweak to support control virtqueue as if what data que

Re: [PULL V2 3/6] hmp-commands: Add new HMP command for filter passthrough

2021-07-14 Thread Jason Wang
在 2021/7/15 上午11:07, Zhang, Chen 写道: Oh, good idea. I will quick update it. By the way, please review or tag other patches in this series. Maybe we can catch up 6.1 merge window. Thanks Chen Note that we probably miss the soft-freeze since it's actually a new feature. It could only be d

RE: [PULL V2 3/6] hmp-commands: Add new HMP command for filter passthrough

2021-07-14 Thread Zhang, Chen
> -Original Message- > From: Dr. David Alan Gilbert > Sent: Thursday, July 15, 2021 2:48 AM > To: Zhang, Chen > Cc: Jason Wang ; qemu-dev de...@nongnu.org>; Eric Blake ; Markus Armbruster > ; Daniel P. Berrangé ; Gerd > Hoffmann ; Li Zhijian ; Lukas > Straub > Subject: Re: [PULL V2 3

tests/acceptance/multiprocess.py test failure

2021-07-14 Thread Cleber Rosa
Hi everyone, The tests/acceptance/multiprocess.py:Multiprocess.test_multiprocess_x86_64 is currently failing (as of a9649a719a44894b81f38dc1c5c1888ee684acef). Unfortunately CI was unable to catch this issue earlier, because tests that require KVM are not yet running (but this should change soon).

Re: [PATCH for-6.2 04/34] target/arm: Fix signed VADDV

2021-07-14 Thread Richard Henderson
On 7/13/21 6:36 AM, Peter Maydell wrote: A cut-and-paste error meant we handled signed VADDV like unsigned VADDV; fix the type used. Signed-off-by: Peter Maydell --- target/arm/mve_helper.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) Reviewed-by: Richard Henderson r~

Re: [PATCH for-6.2 02/34] target/arm: Print MVE VPR in CPU dumps

2021-07-14 Thread Richard Henderson
On 7/13/21 6:36 AM, Peter Maydell wrote: Include the MVE VPR register value in the CPU dumps produced by arm_cpu_dump_state() if we are printing FPU information. This makes it easier to interpret debug logs when predication is active. Signed-off-by: Peter Maydell --- target/arm/cpu.c | 3 +++

Re: [PATCH v2 1/2] Hexagon (target/hexagon) remove put_user_*/get_user_*

2021-07-14 Thread Richard Henderson
On 7/14/21 3:55 PM, Taylor Simpson wrote: +target_ulong pc = env->gpr[HEX_REG_PC]; +uint8_t width = env->mem_log_stores[slot_num].width; +target_ulong va = env->mem_log_stores[slot_num].va; + +switch (width) { case 1: -put_user_u8(env->mem_log_stores[slot_num].data32

[PATCH v2 1/1] hw/riscv/boot: Check the error of fdt_pack()

2021-07-14 Thread Alistair Francis
Coverity reports that we don't check the error result of fdt_pack(), so let's save the result and assert that it is 0. Fixes: Coverity CID 1458136 Signed-off-by: Alistair Francis Reviewed-by: Bin Meng --- hw/riscv/boot.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/

[PATCH v2 2/2] linux-test (tests/tcg/multiarch/linux-test.c) add check

2021-07-14 Thread Taylor Simpson
Add a check that the SIGSEGV handler is called Reviewed-by: Richard Henderson Signed-off-by: Taylor Simpson --- tests/tcg/multiarch/linux-test.c | 8 1 file changed, 8 insertions(+) diff --git a/tests/tcg/multiarch/linux-test.c b/tests/tcg/multiarch/linux-test.c index c8c6aed..e2d88f8

[PATCH v2 0/2] SIGSEGV fixes

2021-07-14 Thread Taylor Simpson
The Hexagon target was silently failing the SIGSEGV test because the signal handler was not called. Patch 1/2 fixes the Hexagon target Patch 2/2 adds a check that the signal handler is called Changes in v2 Address feedback from Richard Henderson - Replace put_user_* with cpu_st*_data_r

[PATCH v2 1/2] Hexagon (target/hexagon) remove put_user_*/get_user_*

2021-07-14 Thread Taylor Simpson
Replace put_user_* with cpu_st*_data_ra Replace get_user_* with cpu_ld*_data_ra Since these functions need the PC, we mark load/store instructions with the A_IMPLICIT_READS_PC attribute in hex_common.py Suggested-by: Richard Henderson Signed-off-by: Taylor Simpson --- target/hexagon/op_helper.

[PATCH 1/1] acpi: Consolidate the handling of OEM ID and OEM Table ID fields

2021-07-14 Thread Marian Postevca
Introduces structure AcpiBuildOem to hold the value of OEM fields and uses dedicated helper functions to initialize/set the values. Unnecessary dynamically allocated OEM fields are re-factored to static allocation. Signed-off-by: Marian Postevca --- hw/acpi/hmat.h | 2 +- hw/i

[PATCH v6 0/1] Rework ACPI OEM fields handling to simplify code (was: acpi: Remove duplicated code handling OEM ID and OEM table ID fields)

2021-07-14 Thread Marian Postevca
This patch consolidates ACPI OEM fields handling by: - Moving common code in PC and MICROVM to X86. - Changes unnecessary dynamic memory allocation to static allocation - Uses dedicated structure to keep values of fields instead of two separate strings - Adds helper functions to initialize the st

Re: [PULL 36/40] vl: switch -M parsing to keyval

2021-07-14 Thread Alex Williamson
On Wed, 14 Jul 2021 07:16:43 +0100 Mark Cave-Ayland wrote: > On 14/07/2021 00:10, Alex Williamson wrote: > > > On Tue, 13 Jul 2021 13:43:47 -0600 > > Alex Williamson wrote: > > > >> On Tue, 6 Jul 2021 12:01:37 +0200 > >> Paolo Bonzini wrote: > >> > >>> Switch from QemuOpts to keyval. Th

[PATCH 1/1] acpi: Consolidate the handling of OEM ID and OEM Table ID fields

2021-07-14 Thread Marian Postevca
Introduces structure AcpiBuildOem to hold the value of OEM fields and uses dedicated helper functions to initialize/set the values. Unnecessary dynamically allocated OEM fields are re-factored to static allocation. Signed-off-by: Marian Postevca --- hw/acpi/hmat.h | 2 +- hw/i

Re: [PATCH 6/6] tests/acceptance/virtio-gpu.py: provide kernel and initrd hashes

2021-07-14 Thread Willian Rampazzo
On Wed, Jul 14, 2021 at 2:41 PM Cleber Rosa wrote: > > By providing kernel and initrd hashes, the test guarantees the > integrity of the images used and avoids the warnings set by > fetch_asset() when hashes are lacking. > > Signed-off-by: Cleber Rosa > --- > tests/acceptance/virtio-gpu.py | 10

Re: [PATCH 5/6] tests/acceptance/virtio-gpu.py: use virtio-vga-gl

2021-07-14 Thread Willian Rampazzo
On Wed, Jul 14, 2021 at 2:41 PM Cleber Rosa wrote: > > Since 49afbca3b, the use of an optional virgl renderer is not > available anymore, and since b36eb8860f, the way to choose a GL based > rendered is to use the "virtio-vga-gl" device. > > Signed-off-by: Cleber Rosa > --- > tests/acceptance/vi

Re: [PATCH 4/6] tests/acceptance/virtio-gpu.py: combine kernel command line

2021-07-14 Thread Willian Rampazzo
On Wed, Jul 14, 2021 at 2:41 PM Cleber Rosa wrote: > > Both tests use the same kernel command line arguments, so there's no > need to have a common and then an additional set of arguments. > > Signed-off-by: Cleber Rosa > --- > tests/acceptance/virtio-gpu.py | 12 +++- > 1 file changed,

Re: [PATCH 3/6] tests/acceptance/virtio-gpu.py: combine CPU tags

2021-07-14 Thread Willian Rampazzo
On Wed, Jul 14, 2021 at 2:41 PM Cleber Rosa wrote: > > Like previously done with the arch tags, all tests use the same CPU > value so it's possible to combine them at the class level. > > Signed-off-by: Cleber Rosa > --- > tests/acceptance/virtio-gpu.py | 3 +-- > 1 file changed, 1 insertion(+),

Re: [PATCH 2/6] tests/acceptance/virtio-gpu.py: combine x86_64 arch tags

2021-07-14 Thread Willian Rampazzo
On Wed, Jul 14, 2021 at 2:41 PM Cleber Rosa wrote: > > The test class in question is x86_64 specific, so it's possible to set > the tags at the class level. > > Signed-off-by: Cleber Rosa > --- > tests/acceptance/virtio-gpu.py | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > Reviewed

Re: [PATCH 1/6] tests/acceptance/virtio-gpu.py: use require_accelerator()

2021-07-14 Thread Willian Rampazzo
On Wed, Jul 14, 2021 at 2:41 PM Cleber Rosa wrote: > > Since efe30d501 there's a shorthand for requiring specific > accelerators, and canceling the test if it's not available. > > Signed-off-by: Cleber Rosa > --- > tests/acceptance/virtio-gpu.py | 10 ++ > 1 file changed, 2 insertions(+)

[PULL v2 3/3] vfio/pci: Add pba_offset PCI quirk for BAIDU KUNLUN AI processor

2021-07-14 Thread Alex Williamson
From: Cai Huoqing Fix pba_offset initialization value for BAIDU KUNLUN Virtual Function device. The KUNLUN hardware returns an incorrect value for the VF PBA offset, and add a quirk to instead return a hardcoded value of 0xb400. Signed-off-by: Cai Huoqing Link: https://lore.kernel.org/r/2021071

[PULL v2 2/3] vfio/pci: Change to use vfio_pci_is()

2021-07-14 Thread Alex Williamson
From: Cai Huoqing Make use of vfio_pci_is() helper function. Signed-off-by: Cai Huoqing Link: https://lore.kernel.org/r/20210713014831.742-1-caihuoq...@baidu.com [aw: commit log wording] Signed-off-by: Alex Williamson --- hw/vfio/pci.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions

[PULL v2 1/3] vfio: Fix CID 1458134 in vfio_register_ram_discard_listener()

2021-07-14 Thread Alex Williamson
From: David Hildenbrand CID 1458134: Integer handling issues (BAD_SHIFT) In expression "1 << ctz64(container->pgsizes)", left shifting by more than 31 bits has undefined behavior. The shift amount, "ctz64(container->pgsizes)", is 64. Commit 5e3b981c330c ("vfio: Support for RamDi

[PULL v2 0/3] VFIO update 2021-07-14 (for v6.1)

2021-07-14 Thread Alex Williamson
The following changes since commit a9649a719a44894b81f38dc1c5c1888ee684acef: Merge remote-tracking branch 'remotes/cleber-gitlab/tags/python-next-pull-request' into staging (2021-07-14 18:09:09 +0100) are available in the Git repository at: git://github.com/awilliam/qemu-vfio.git tags/vfio

Re: Host folder sharing via USB issue

2021-07-14 Thread Programmingkid
> On Jul 14, 2021, at 6:35 AM, Vladimir Sementsov-Ogievskiy > wrote: > > 14.07.2021 00:04, Programmingkid wrote: >> Hi I have noticed that host folder sharing via USB has recently stopped >> working. After doing some git bisecting I found this as the patch that seems >> to be the issue: >>

[PULL 0/5] OVMF patches for 2021-07-14

2021-07-14 Thread Philippe Mathieu-Daudé
gs/fw-edk2-20210714 for you to fetch changes up to 2669350db2c3df33f4e68c518e9f31f91502a83d: MAINTAINERS: remove Laszlo Ersek's entries (2021-07-14 22:28:59 +0200) Patches related to EDK2/OVMF - MAINTAINERS: remove Laszlo

[PULL 1/5] gitlab-ci: Extract EDK2 job rules to reusable section

2021-07-14 Thread Philippe Mathieu-Daudé
All jobs depending on 'docker-edk2' job must use at most all the rules that triggers it. The simplest way to ensure that is to always use the same rules. Extract all the rules to a reusable section, and include this section (with the 'extends' keyword) in both 'docker-edk2' and 'build-edk2' jobs.

Re: [PULL 0/3] VFIO update 2021-07-13 (for v6.1)

2021-07-14 Thread Alex Williamson
On Wed, 14 Jul 2021 20:45:40 +0100 Peter Maydell wrote: > On Tue, 13 Jul 2021 at 22:42, Alex Williamson > wrote: > > > > The following changes since commit 961fef0ace0b2a997544deec2f01981b9c9c96e3: > > > > Merge remote-tracking branch > > 'remotes/alistair/tags/pull-riscv-to-apply-20210712' i

Re: [PATCH-for-6.1] gitlab-ci: Extract EDK2 job rules to reusable section

2021-07-14 Thread Philippe Mathieu-Daudé
On 7/14/21 12:10 PM, Philippe Mathieu-Daudé wrote: > All jobs depending on 'docker-edk2' job must use at most all > the rules that triggers it. The simplest way to ensure that > is to always use the same rules. Extract all the rules to a > reusable section, and include this section (with the 'exten

Re: [PULL 00/23] pc,pci,virtio: lots of new features

2021-07-14 Thread Peter Maydell
On Tue, 13 Jul 2021 at 23:10, Michael S. Tsirkin wrote: > > The following changes since commit 708f50199b59476ec4b45ebcdf171550086d6292: > > Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2021-07-09-v2' > into staging (2021-07-13 14:32:20 +0100) > > are available in the Git repositor

Re: [PATCH-for-6.1] gitlab-ci: Extract EDK2 job rules to reusable section

2021-07-14 Thread Philippe Mathieu-Daudé
On 7/14/21 8:14 PM, Cleber Rosa wrote: > > Philippe Mathieu-Daudé writes: > >> All jobs depending on 'docker-edk2' job must use at most all >> the rules that triggers it. The simplest way to ensure that >> is to always use the same rules. Extract all the rules to a >> reusable section, and includ

Re: [PATCH v3 0/9] tests: Add test cases for TPM 1.2 ACPI tables

2021-07-14 Thread Philippe Mathieu-Daudé
On 7/14/21 4:43 PM, Markus Armbruster wrote: > Philippe Mathieu-Daudé writes: > >> +Markus >> >> On 7/12/21 5:47 PM, Stefan Berger wrote: >>> >>> On 7/12/21 11:29 AM, Philippe Mathieu-Daudé wrote: Hi Stefan, On 7/12/21 5:09 PM, Stefan Berger wrote: > This series of patches adds

[PATCH] hw/intc/arm_gicv3: Fix GICv3 redistributor security checking

2021-07-14 Thread Tianrui Wei
For redistributor to send sgi, we must test NSACR bits in secure mode. However, current implementation inverts the security check, wrongly skipping this it when the CPU is in secure state, and only carrying out the check when the CPU is not secure or security is not implemented. This patch corrects

Re: [PULL 0/3] VFIO update 2021-07-13 (for v6.1)

2021-07-14 Thread Peter Maydell
On Tue, 13 Jul 2021 at 22:42, Alex Williamson wrote: > > The following changes since commit 961fef0ace0b2a997544deec2f01981b9c9c96e3: > > Merge remote-tracking branch > 'remotes/alistair/tags/pull-riscv-to-apply-20210712' into staging (2021-07-13 > 17:51:50 +0100) > > are available in the Git

Re: [PULL 00/23] Python and Acceptance Tests

2021-07-14 Thread Peter Maydell
On Tue, 13 Jul 2021 at 22:19, Cleber Rosa wrote: > > The following changes since commit 708f50199b59476ec4b45ebcdf171550086d6292: > > Merge remote-tracking branch 'remotes/ericb/tags/pull-nbd-2021-07-09-v2' > into staging (2021-07-13 14:32:20 +0100) > > are available in the Git repository at: >

Re: [PATCH v2 0/6] python: AQMP-TUI Prototype

2021-07-14 Thread Niteesh G. S.
Hello all, UPDATE: The pipelines have run and all tests passed #336491916 Usually, the pipelines start running as soon as I push my code. But this time they took longer to start and there was no sign of starting. This is my first experience with pipelines so I assumed I messed up something from m

Re: [RFC PATCH 6/6] i386/sev: populate secrets and cpuid page and finalize the SNP launch

2021-07-14 Thread Brijesh Singh
On 7/14/21 12:29 PM, Dr. David Alan Gilbert wrote:>> +struct snp_pre_validated_range { +uint32_t start; +uint32_t end; +}; Just a thought, but maybe use a 'Range' from include/qemu/range.h ? I will look into it. thanks

Re: [RFC PATCH 5/6] i386/sev: add support to encrypt BIOS when SEV-SNP is enabled

2021-07-14 Thread Brijesh Singh
On 7/14/21 12:08 PM, Connor Kuehl wrote: On 7/9/21 3:55 PM, Brijesh Singh wrote: The KVM_SEV_SNP_LAUNCH_UPDATE command is used for encrypting the bios image used for booting the SEV-SNP guest. Signed-off-by: Brijesh Singh --- target/i386/sev.c| 33 -

Re: [PULL V2 3/6] hmp-commands: Add new HMP command for filter passthrough

2021-07-14 Thread Dr. David Alan Gilbert
* Zhang Chen (chen.zh...@intel.com) wrote: > Add hmp_passthrough_filter_add and hmp_passthrough_filter_del make user > can maintain object network passthrough list in human monitor > > Signed-off-by: Zhang Chen > --- > hmp-commands.hx | 26 + > include/monitor/hmp.h | 2 ++

Re: [PATCH v2 19/21] contrib/gitdm: add an explicit academic entry for BU

2021-07-14 Thread Alexander Bulekov
On 210714 1920, Alex Bennée wrote: > For some reason Alexander's contributions were not getting grouped > from the plain "edu" mapping. > > Signed-off-by: Alex Bennée > Cc: Alexander Bulekov Reviewed-by: Alexander Bulekov Thanks! > Message-Id: <20210714093638.21077-20-alex.ben...@linaro.org>

Re: [PATCH 1/2] qapi/run-state: Add a new shutdown cause 'migration-completed'

2021-07-14 Thread Dr. David Alan Gilbert
* Kunkun Jiang (jiangkun...@huawei.com) wrote: > On 2021/7/6 18:27, Dr. David Alan Gilbert wrote: > > * Kunkun Jiang (jiangkun...@huawei.com) wrote: > > > Hi Daniel, > > > > > > On 2021/7/5 20:48, Daniel P. Berrangé wrote: > > > > On Mon, Jul 05, 2021 at 08:36:52PM +0800, Kunkun Jiang wrote: > > >

[PATCH v2 20/21] contrib/gitdm: add a new interns group-map for GSoC/Outreachy work

2021-07-14 Thread Alex Bennée
It makes sense to put our various interns in a group so we can see the overall impact of GSoC and Outreachy on the project. Signed-off-by: Alex Bennée Cc: Ahmed Karaman Cc: Mahmoud Mandour Cc: César Belley Message-Id: <20210714093638.21077-21-alex.ben...@linaro.org> --- contrib/gitdm/group-ma

[PATCH v2 16/21] contrib/gitdm: add domain-map for Syrmia

2021-07-14 Thread Alex Bennée
We have multiple engineers contributing from this domain. Signed-off-by: Alex Bennée Cc: Filip Bozuta Cc: Aleksandar Rikalo Message-Id: <20210714093638.21077-17-alex.ben...@linaro.org> --- contrib/gitdm/domain-map | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/gitdm/domain-map b/c

[PATCH v2 15/21] contrib/gitdm: add domain-map for ZTE

2021-07-14 Thread Alex Bennée
Signed-off-by: Alex Bennée Cc: Yi Wang Cc: Yun Liu Message-Id: <20210714093638.21077-16-alex.ben...@linaro.org> --- contrib/gitdm/domain-map | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/gitdm/domain-map b/contrib/gitdm/domain-map index e42861cd11..bada5384dd 100644 --- a/contrib/

[PATCH v2 11/21] contrib/gitdm: add domain-map for Eldorado

2021-07-14 Thread Alex Bennée
Signed-off-by: Alex Bennée Cc: Bruno Larsen (billionai) Cc: Luis Pires Message-Id: <20210714093638.21077-12-alex.ben...@linaro.org> --- contrib/gitdm/domain-map | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/gitdm/domain-map b/contrib/gitdm/domain-map index beeb24341e..41875c9e75 1

[PATCH v2 17/21] contrib/gitdm: add domain-map for NVIDIA

2021-07-14 Thread Alex Bennée
Signed-off-by: Alex Bennée Cc: Kirti Wankhede Cc: Yishai Hadas Message-Id: <20210714093638.21077-18-alex.ben...@linaro.org> --- contrib/gitdm/domain-map | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/gitdm/domain-map b/contrib/gitdm/domain-map index 0b0cd9feee..329ff09029 100644 --

[PATCH v2 19/21] contrib/gitdm: add an explicit academic entry for BU

2021-07-14 Thread Alex Bennée
For some reason Alexander's contributions were not getting grouped from the plain "edu" mapping. Signed-off-by: Alex Bennée Cc: Alexander Bulekov Message-Id: <20210714093638.21077-20-alex.ben...@linaro.org> --- contrib/gitdm/group-map-academics | 3 +++ 1 file changed, 3 insertions(+) diff --g

[PATCH v2 04/21] configure: remove needless if leg

2021-07-14 Thread Alex Bennée
It was pointed out in review of the previous patch that the if leg isn't needed as the for loop will not enter on an empty $device_archs. Fixes: d1d5e9eefd ("configure: allow the selection of alternate config in the build") Signed-off-by: Alex Bennée Message-Id: <20210714093638.21077-5-alex.ben.

[PATCH v2 12/21] contrib/gitdm: add domain-map/group-map for Wind River

2021-07-14 Thread Alex Bennée
As per discussion at: http://patchwork.ozlabs.org/project/qemu-devel/patch/20201004180443.2035359-19-f4...@amsat.org/ I've added Bin's personal email as an individual contributor. Signed-off-by: Alex Bennée Acked-by: Bin Meng Cc: Ruimei Yan Cc: Xuzhou Cheng Message-Id: <20210714093719.2142

[PATCH v2 13/21] contrib/gitdm: un-ironically add a mapping for LWN

2021-07-14 Thread Alex Bennée
I think this mainly comes from kernel-doc stuff imported into the QEMU tree. Signed-off-by: Alex Bennée Cc: Jonathan Corbet Message-Id: <20210714093638.21077-14-alex.ben...@linaro.org> --- contrib/gitdm/domain-map | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/gitdm/domain-map b/co

[PATCH v2 18/21] contrib/gitdm: add group-map for Netflix

2021-07-14 Thread Alex Bennée
Warner confirmed he works for Netflix on IRC. Message-Id: <20210714093638.21077-19-alex.ben...@linaro.org> Signed-off-by: Alex Bennée Reviewed-by: Warner Losh --- contrib/gitdm/group-map-netflix | 5 + gitdm.config| 1 + 2 files changed, 6 insertions(+) create mode 1006

[PATCH v2 02/21] docs: collect the disparate device emulation docs into one section

2021-07-14 Thread Alex Bennée
While we are at it add a brief preamble that explains some of the common concepts in QEMU's device emulation which will hopefully lead to less confusing about our dizzying command line options. Signed-off-by: Alex Bennée Message-Id: <20210714093638.21077-3-alex.ben...@linaro.org> --- docs/system

[PATCH v2 21/21] contrib/gitdm: add more individual contributor entries.

2021-07-14 Thread Alex Bennée
Also ensure Li's canonical gmail address is used. Message-Id: <20210714100628.10460-1-alex.ben...@linaro.org> Signed-off-by: Alex Bennée Acked-by: Li Qiang Acked-by: Chetan Pant Acked-by: Akihiko Odaki --- v2 - dropped un-acked individuals --- contrib/gitdm/aliases | 3 +++ c

[PATCH v2 03/21] docs: add a section on the generalities of vhost-user

2021-07-14 Thread Alex Bennée
While we do mention some of this stuff in the various daemons and manuals the subtleties of the socket and memory sharing are sometimes missed. This document attempts to give some background on vhost-user daemons in general terms. Signed-off-by: Alex Bennée Cc: Stefan Hajnoczi Message-Id: <20210

[PATCH v2 10/21] contrib/gitdm: add domain-map/group-map mappings for Samsung

2021-07-14 Thread Alex Bennée
Minwoo's work from their personal address are treated as personal contributions. Signed-off-by: Alex Bennée Acked-by: Klaus Jensen Cc: Gollu Appalanaidu Cc: Minwoo Im Message-Id: <220210714093719.21429-11-alex.ben...@linaro.org> --- contrib/gitdm/domain-map| 1 + contrib/gitdm/gro

[PATCH v2 01/21] gitignore: Update with some filetypes

2021-07-14 Thread Alex Bennée
From: Viresh Kumar Update .gitignore to ignore .swp and .patch files. Signed-off-by: Viresh Kumar Signed-off-by: Alex Bennée Reviewed-by: Alex Bennée Message-Id: <79262dbe1f7888eb02e1911501eebafa6f2f6400.1616583806.git.viresh.ku...@linaro.org> Message-Id: <20210714093638.21077-2-alex.ben...@

[PATCH v2 09/21] gitdm.config: sort the corporate GroupMap entries

2021-07-14 Thread Alex Bennée
Lets try and keep them that way. Signed-off-by: Alex Bennée Message-Id: <20210714093638.21077-10-alex.ben...@linaro.org> --- gitdm.config | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gitdm.config b/gitdm.config index 7378238c20..a3542d2fc7 100644 --- a/gitdm.config +

[PATCH v2 14/21] contrib/gitdm: add domain-map for Crudebyte

2021-07-14 Thread Alex Bennée
Let me know if you want to be moved to individual contributor or you want to keep with the work domain mapping. Signed-off-by: Alex Bennée Reviewed-by: Christian Schoenebeck Message-Id: <20210714093638.21077-15-alex.ben...@linaro.org> --- contrib/gitdm/domain-map | 1 + 1 file changed, 1 insert

[PATCH v2 07/21] contrib/gitdm: add domain-map for MontaVista

2021-07-14 Thread Alex Bennée
Signed-off-by: Alex Bennée Acked-by: Corey Minyard Cc: Corey Minyard Message-Id: <20210714093638.21077-8-alex.ben...@linaro.org> --- contrib/gitdm/domain-map | 1 + 1 file changed, 1 insertion(+) diff --git a/contrib/gitdm/domain-map b/contrib/gitdm/domain-map index 0074da618f..efbbb15643 1006

[PATCH v2 06/21] .mailmap: fix up some broken commit authors

2021-07-14 Thread Alex Bennée
Fixes: 49a6f3bffb ("target/arm: Correct the encoding of MDCCSR_EL0 and DBGDSCRint") Fixes: 5a07192a04 ("target/i386: Fix handling of k_gs_base register in 32-bit mode in gdbstub") Signed-off-by: Alex Bennée Cc: Nick Hudson Cc: Marek Dolata Message-Id: <20210714093638.21077-7-alex.ben...@linaro

[PATCH v2 08/21] contrib/gitdm: add a group mapping for robot scanners

2021-07-14 Thread Alex Bennée
This mostly affects Reported-by: tags Signed-off-by: Alex Bennée Message-Id: <20210714093638.21077-9-alex.ben...@linaro.org> --- contrib/gitdm/group-map-robots | 7 +++ gitdm.config | 3 +++ 2 files changed, 10 insertions(+) create mode 100644 contrib/gitdm/group-map-robot

[PATCH v2 05/21] contrib/gitdm: add some new aliases to fix up commits

2021-07-14 Thread Alex Bennée
Signed-off-by: Alex Bennée Cc: Yuval Shaia Message-Id: <20210714093638.21077-6-alex.ben...@linaro.org> --- contrib/gitdm/aliases | 3 +++ 1 file changed, 3 insertions(+) diff --git a/contrib/gitdm/aliases b/contrib/gitdm/aliases index c1e744312f..c6ed215e68 100644 --- a/contrib/gitdm/aliases ++

[PATCH v2 00/21] documentation and metadata updates

2021-07-14 Thread Alex Bennée
Hi, Posting v2 as the v1 posting got double posted due to git problems so it's easier if we reset here. As with previous release cycles I'll switch to maintaining a single tree of patches for the current release at any given time. The usual patch admission rules apply so you won't find any featur

Re: [PATCH-for-6.1] gitlab-ci: Extract EDK2 job rules to reusable section

2021-07-14 Thread Cleber Rosa
Philippe Mathieu-Daudé writes: > All jobs depending on 'docker-edk2' job must use at most all > the rules that triggers it. The simplest way to ensure that > is to always use the same rules. Extract all the rules to a > reusable section, and include this section (with the 'extends' > keyword) in

[PATCH 2/2] target/arm: Add sve-default-vector-length cpu property

2021-07-14 Thread Richard Henderson
Mirror the behavour of /proc/sys/abi/sve_default_vector_length under the real linux kernel. We have no way of passing along a real default across exec like the kernel can, but this is a decent way of adjusting the startup vector length of a process. Resolves: https://gitlab.com/qemu-project/qemu/

[PATCH 0/2] target/arm: Add sve-default-vector-length cpu property

2021-07-14 Thread Richard Henderson
This is intended to resolve #482. I'm not sure where to document this. The convenient place would be in docs/system/arm/cpu-features.rst, where all of the rest of the sve documentation lives. But that's explicitly system-only, and this new properly is explicitly user-only. Suggestions? r~ R

[PATCH 1/2] target/arm: Export aarch64_sve_zcr_get_valid_len

2021-07-14 Thread Richard Henderson
Rename from sve_zcr_get_valid_len and make accessible from outside of helper.c. Signed-off-by: Richard Henderson --- target/arm/cpu.h| 2 ++ target/arm/helper.c | 8 +--- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/target/arm/cpu.h b/target/arm/cpu.h index be9a4dceae..

  1   2   3   4   >