Re: [PATCH v4 3/3] memory_hotplug.c: send DEVICE_UNPLUG_ERROR in acpi_memory_hotplug_write()

2021-07-11 Thread Daniel Henrique Barboza
On 7/10/21 3:57 AM, Markus Armbruster wrote: Igor Mammedov writes: On Fri, 09 Jul 2021 13:25:43 +0200 Markus Armbruster wrote: Igor Mammedov writes: On Thu, 08 Jul 2021 15:08:57 +0200 Markus Armbruster wrote: Daniel Henrique Barboza writes: MEM_UNPLUG_ERROR is deprecated si

Re: [PATCH v2] docs: Add '-device intel-iommu' entry

2021-07-11 Thread Eric Auger
Hi Peter, On 7/7/21 5:41 PM, Peter Xu wrote: > The parameters of intel-iommu device are non-trivial to understand. Add an > entry for it so that people can reference to it when using. > > There're actually a few more options there, but I hide them explicitly because > they shouldn't be used by n

[Bug 1873769] Re: SB16 audio playback freezes emulation in Windows 95 guest

2021-07-11 Thread Vladislav K. Valtchev
** Changed in: qemu Status: Expired => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1873769 Title: SB16 audio playback freezes emulation in Windows 95 guest Status in QEMU: Con

[Bug 1779955] Re: qemu linux-user requires read permissions on memory passed to syscalls that should only need write access

2021-07-11 Thread Peter Maydell
Still a bug. ** Changed in: qemu Status: Expired => Confirmed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1779955 Title: qemu linux-user requires read permissions on memory passed to sys

[Bug 1873769] Re: SB16 audio playback freezes emulation in Windows 95 guest

2021-07-11 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/469 ** Tags added: audio ** Changed in: qemu Status: Confir

[Bug 1779955] Re: qemu linux-user requires read permissions on memory passed to syscalls that should only need write access

2021-07-11 Thread Thomas Huth
This is an automated cleanup. This bug report has been moved to QEMU's new bug tracker on gitlab.com and thus gets marked as 'expired' now. Please continue with the discussion here: https://gitlab.com/qemu-project/qemu/-/issues/470 ** Changed in: qemu Status: Confirmed => Expired ** Bug

Re: [PATCH v4 1/5] configure,meson: add option to enable LTO

2021-07-11 Thread Thomas Huth
On 05/12/2020 00.06, Daniele Buono wrote: This patch allows to compile QEMU with link-time optimization (LTO). Compilation with LTO is handled directly by meson. This patch only adds the option in configure and forwards the request to meson Tested with all major versions of clang from 6 to 12 S

Re: [PATCH v3 0/8] dp8393x: fixes and improvements

2021-07-11 Thread Philippe Mathieu-Daudé
On 7/11/21 4:08 AM, Finn Thain wrote: > On Sat, 10 Jul 2021, Philippe Mathieu-Daudé wrote: > >> >> The last 2 patches are included for Mark, but I don't plan to merge >> >> them without Finn's Ack, and apparently they require more work. >> > > > I tested the patch series both with and without th

[PATCH v4 0/5] dp8393x: fixes and improvements

2021-07-11 Thread Philippe Mathieu-Daudé
Hi Mark, This should be the last respin. Since v3: - dropped worrying patches - squashed migration patch - added tags Patch #3 (dp8393x: Store CAM registers as 16-bit) still misses your S-o-b tag. Based-on mips-next. Mark Cave-Ayland (1): dp8393x: don't force 32-bit register access Philippe

[PATCH v4 3/5] dp8393x: Store CAM registers as 16-bit

2021-07-11 Thread Philippe Mathieu-Daudé
Per the DP83932C datasheet from July 1995: 4.0 SONIC Registers 4.1 THE CAM UNIT The Content Addressable Memory (CAM) consists of sixteen 48-bit entries for complete address filtering of network packets. Each entry corresponds to a 48-bit destination address that is user progra

[PATCH v4 4/5] dp8393x: Rewrite dp8393x_get() / dp8393x_put()

2021-07-11 Thread Philippe Mathieu-Daudé
Instead of accessing N registers via a single address_space API call using a temporary buffer (stored in the device state) and updating each register, move the address_space call in the register put/get. The load/store and word size checks are moved to put/get too. This simplifies a bit, making the

[PATCH v4 1/5] dp8393x: Replace address_space_rw(is_write=1) by address_space_write()

2021-07-11 Thread Philippe Mathieu-Daudé
Replace address_space_rw(is_write=1) by address_space_write() and remove pointless cast. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland Tested-by: Finn Thain Message-Id: <20210710174954.2577195-2-f4...@amsat.org> --- hw/net/dp8393x.c | 8 1 file changed, 4 inserti

[PATCH v4 5/5] dp8393x: don't force 32-bit register access

2021-07-11 Thread Philippe Mathieu-Daudé
From: Mark Cave-Ayland Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set .impl.min_access_size and .impl.max_access_size to 4 to try and fix the Linux jazzsonic driver which uses 32-bit accesses. The problem with forcing the register access to 32-bit in this way is that since the dp

[PATCH v4 2/5] dp8393x: Replace 0x40 magic value by SONIC_REG16_COUNT definition

2021-07-11 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland Tested-by: Finn Thain Message-Id: <20210710174954.2577195-3-f4...@amsat.org> --- hw/net/dp8393x.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/net/dp8393x.c b/hw/net/dp8393x.c index 9118364aa33.

Re: [PULL 0/4] NBD patches for soft freeze, 2021-07-09

2021-07-11 Thread Peter Maydell
On Fri, 9 Jul 2021 at 15:13, Eric Blake wrote: > > The following changes since commit 9db3065c62a983286d06c207f4981408cf42184d: > > Merge remote-tracking branch > 'remotes/vivier2/tags/linux-user-for-6.1-pull-request' into staging > (2021-07-08 16:30:18 +0100) > > are available in the Git repo

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Coiby Xu
Hi Peter, On Sat, Jul 10, 2021 at 02:30:36PM +0100, Peter Maydell wrote: I've noticed recently that intermittently 'make check' will hang on my aarch32 test system (really an aarch64 box with an aarch32 chroot). I have a newbie question. How do you do an aarch32 chroot on an aarch64 box? At le

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Peter Maydell
On Sun, 11 Jul 2021 at 13:10, Coiby Xu wrote: > > Hi Peter, > > On Sat, Jul 10, 2021 at 02:30:36PM +0100, Peter Maydell wrote: > >I've noticed recently that intermittently 'make check' will hang on > >my aarch32 test system (really an aarch64 box with an aarch32 chroot). > > I have a newbie questi

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Richard Henderson
On 7/11/21 5:16 AM, Peter Maydell wrote: On Sun, 11 Jul 2021 at 13:10, Coiby Xu wrote: Hi Peter, On Sat, Jul 10, 2021 at 02:30:36PM +0100, Peter Maydell wrote: I've noticed recently that intermittently 'make check' will hang on my aarch32 test system (really an aarch64 box with an aarch32 ch

Re: [PULL 00/17] target-arm queue

2021-07-11 Thread Peter Maydell
On Fri, 9 Jul 2021 at 17:10, Peter Maydell wrote: > > Arm changes for before softfreeze: mostly my PL061/GPIO patches, > but also a new M-profile board and various other things. > > thanks > -- PMM > > The following changes since commit 05de778b5b8ab0b402996769117b88c7ea5c7c61: > > Merge remote-

[PULL SUBSYSTEM qemu-pseries] pseries: Update SLOF firmware image

2021-07-11 Thread Alexey Kardashevskiy
The following changes since commit 82123b756a1a2f1965350e5794aaa7b5c6a15282: target/ppc: Support for H_RPT_INVALIDATE hcall (2021-07-09 11:01:06 +1000) are available in the Git repository at: g...@github.com:aik/qemu.git tags/qemu-slof-20210711 for you to fetch changes up to

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Peter Maydell
On Sun, 11 Jul 2021 at 14:23, Richard Henderson wrote: > I believe that I had to perform the install under tcg because I couldn't find > the right > magic to boot off the debian cdrom with kvm. Weird, it ought not in theory to care... -- PMM

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Richard Henderson
On 7/11/21 7:21 AM, Peter Maydell wrote: On Sun, 11 Jul 2021 at 14:23, Richard Henderson wrote: I believe that I had to perform the install under tcg because I couldn't find the right magic to boot off the debian cdrom with kvm. Weird, it ought not in theory to care... Looking back at the

Re: clang build error on i686

2021-07-11 Thread Richard Henderson
On 7/3/21 10:45 AM, Peter Maydell wrote: On Sat, 3 Jul 2021 at 15:37, Cole Robinson wrote: Hi, I'm hitting build errors with clang on i686 userspace on x86_64 kernel. Affects both qemu 6.0.0 and qemu.git, tested with fedora clang-12.0.1~rc3-1.fc35.i686. Full build log from the 6.0.0 build: ht

Re: [PATCH 0/4] support dirtyrate measurement with dirty bitmap

2021-07-11 Thread Hyman Huang
在 2021/7/10 2:20, Peter Xu 写道: Yong, On Sun, Jun 27, 2021 at 01:38:13PM +0800, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) the dirtyrate measurement implemented by page-sampling originally, it is not accurate in some scenarios, so we have introduced dirty-ring based dirtyrate mea

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Peter Maydell
On Sat, 10 Jul 2021 at 14:30, Peter Maydell wrote: > > I've noticed recently that intermittently 'make check' will hang on > my aarch32 test system (really an aarch64 box with an aarch32 chroot). > > I think from grep that this must be the vhost-user-blk test. I've also now seen this on qemu-syst

[PULL 4/9] tests: Add suffix 'tpm2' or 'tpm12' to ACPI table files

2021-07-11 Thread Stefan Berger
Cc: Michael S. Tsirkin Cc: Igor Mammedov Signed-off-by: Stefan Berger Reviewed-by: Igor Mammedov Message-id: 20210708183814.925960-5-stef...@linux.vnet.ibm.com --- tests/qtest/bios-tables-test.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/qtest/bios-tables-test.

[PULL 7/9] tests: acpi: prepare for new TPM 1.2 related tables

2021-07-11 Thread Stefan Berger
Cc: Michael S. Tsirkin Cc: Igor Mammedov Signed-off-by: Stefan Berger Acked-by: Igor Mammedov Message-id: 20210708183814.925960-8-stef...@linux.vnet.ibm.com --- tests/data/acpi/q35/DSDT.tis.tpm12 | 0 tests/data/acpi/q35/TCPA.tis.tpm12 | 0 tests/qtest/bios-tables-test-allowe

[PULL 9/9] tests: acpi: tpm1.2: Add expected TPM 1.2 ACPI blobs

2021-07-11 Thread Stefan Berger
The TCPA.tis.tpm12 file contains the following: [000h 4]Signature : "TCPA"[Trusted Computing Platform Alliance table] [004h 0004 4] Table Length : 0032 [008h 0008 1] Revision : 02 [009h 0009 1] Checks

[PULL 3/9] tests: acpi: Prepare for renaming of TPM2 related ACPI files

2021-07-11 Thread Stefan Berger
Cc: Michael S. Tsirkin Cc: Igor Mammedov Signed-off-by: Stefan Berger Reviewed-by: Igor Mammedov Message-id: 20210708183814.925960-4-stef...@linux.vnet.ibm.com --- tests/data/acpi/q35/DSDT.tis.tpm2 | 0 tests/data/acpi/q35/TPM2.tis.tpm2 | 0 tests/qtest/bios-tables-test-all

[PULL 8/9] tests: acpi: Add test cases for TPM 1.2 with TCPA table

2021-07-11 Thread Stefan Berger
Cc: Michael S. Tsirkin Cc: Igor Mammedov Signed-off-by: Stefan Berger Reviewed-by: Igor Mammedov Message-id: 20210708183814.925960-9-stef...@linux.vnet.ibm.com --- tests/qtest/bios-tables-test.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/tests/qtest/bios-tab

[PULL 2/9] tests: Add tpm_version field to TPMTestState and fill it

2021-07-11 Thread Stefan Berger
Signed-off-by: Stefan Berger Reviewed-by: Igor Mammedov Message-id: 20210708183814.925960-3-stef...@linux.vnet.ibm.com --- tests/qtest/bios-tables-test.c| 5 +++-- tests/qtest/tpm-crb-test.c| 1 + tests/qtest/tpm-emu.c | 13 ++--- tests/qtest/tpm-emu.h

[PULL 6/9] tests: tpm: Create TPM 1.2 response in TPM emulator

2021-07-11 Thread Stefan Berger
Signed-off-by: Stefan Berger Acked-by: Igor Mammedov Message-id: 20210708183814.925960-7-stef...@linux.vnet.ibm.com --- tests/qtest/tpm-emu.c | 5 + tests/qtest/tpm-emu.h | 3 +++ 2 files changed, 8 insertions(+) diff --git a/tests/qtest/tpm-emu.c b/tests/qtest/tpm-emu.c index 8baf49eafd..3

Re: [PATCH] qga-win: Add support of Windows Server 2022 in get-osinfo command

2021-07-11 Thread Konstantin Kostiuk
ping On Sun, Jul 4, 2021 at 8:51 AM Konstantin Kostiuk wrote: > ping > > On Mon, Jun 21, 2021 at 3:50 PM Kostiantyn Kostiuk > wrote: > >> Signed-off-by: Kostiantyn Kostiuk >> --- >> qga/commands-win32.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/qga/commands

[PULL 0/9] Merge tpm 2021/07/11 v1

2021-07-11 Thread Stefan Berger
This series extends TPM-related ACPI test cases to also verify the TPM 1.2 ACPI table 'TCPA'. Stefan The following changes since commit 9db3065c62a983286d06c207f4981408cf42184d: Merge remote-tracking branch 'remotes/vivier2/tags/linux-user-for-6.1-pull-request' into staging (2021-07-08 16

[PULL 5/9] tests: acpi: tpm2: Add the renamed ACPI files and drop old ones

2021-07-11 Thread Stefan Berger
Cc: Michael S. Tsirkin Cc: Igor Mammedov Signed-off-by: Stefan Berger Acked-by: Igor Mammedov Message-id: 20210708183814.925960-6-stef...@linux.vnet.ibm.com --- tests/data/acpi/q35/DSDT.tis| Bin 8465 -> 0 bytes tests/data/acpi/q35/DSDT.tis.tpm2 | Bin 0 -> 8465 bytes

[PULL 1/9] tests: Rename TestState to TPMTestState

2021-07-11 Thread Stefan Berger
Signed-off-by: Stefan Berger Reviewed-by: Igor Mammedov Message-id: 20210708183814.925960-2-stef...@linux.vnet.ibm.com --- tests/qtest/bios-tables-test.c| 2 +- tests/qtest/tpm-crb-test.c| 4 ++-- tests/qtest/tpm-emu.c | 6 +++--- tests/qtest/tpm-emu.h | 6 +++

Re: [PATCH 1/3] qga-win: Increase VSS freeze timeout to 60 secs instead of 10

2021-07-11 Thread Konstantin Kostiuk
ping On Sun, Jul 4, 2021 at 8:52 AM Konstantin Kostiuk wrote: > ping > > On Mon, Apr 5, 2021 at 4:14 PM Basil Salman wrote: > >> Currently Requester freeze times out after 10 seconds, while >> the default timeout for Writer Freeze is 60 seconds. according to >> VSS Documentation [1]. >> [1]: >>

Re: [GIT PULL v2] I2C/IPMI bug fixes for QEMU 6.1

2021-07-11 Thread Peter Maydell
On Fri, 9 Jul 2021 at 17:30, Corey Minyard wrote: > > The following changes since commit 38848ce565849e5b867a5e08022b3c755039c11a: > > Merge remote-tracking branch > 'remotes/pmaydell/tags/pull-target-arm-20210616' into staging (2021-06-16 > 17:02:30 +0100) > > are available in the Git reposit

Re: clang build error on i686

2021-07-11 Thread Peter Maydell
On Sun, 11 Jul 2021 at 16:17, Richard Henderson wrote: > > On 7/3/21 10:45 AM, Peter Maydell wrote: > > On Sat, 3 Jul 2021 at 15:37, Cole Robinson wrote: > >> > >> Hi, I'm hitting build errors with clang on i686 userspace on x86_64 > >> kernel. Affects both qemu 6.0.0 and qemu.git, tested with fe

Re: [PULL 00/17] s390x update for softfreeze

2021-07-11 Thread Peter Maydell
On Fri, 9 Jul 2021 at 16:23, Cornelia Huck wrote: > > On Fri, Jul 09 2021, Peter Maydell wrote: > > > On Thu, 8 Jul 2021 at 16:19, Cornelia Huck wrote: > >> > >> The following changes since commit > >> 9aef0954195cc592e86846dbbe7f3c2c5603690a: > >> > >> Merge remote-tracking branch 'remotes/b

Re: [PATCH v4 2/5] dp8393x: Replace 0x40 magic value by SONIC_REG16_COUNT definition

2021-07-11 Thread Mark Cave-Ayland
On 11/07/2021 11:36, Philippe Mathieu-Daudé wrote: Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland Tested-by: Finn Thain Message-Id: <20210710174954.2577195-3-f4...@amsat.org> --- hw/net/dp8393x.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

Re: [PATCH v4 0/5] dp8393x: fixes and improvements

2021-07-11 Thread Mark Cave-Ayland
On 11/07/2021 11:36, Philippe Mathieu-Daudé wrote: Hi Mark, This should be the last respin. Since v3: - dropped worrying patches - squashed migration patch - added tags Patch #3 (dp8393x: Store CAM registers as 16-bit) still misses your S-o-b tag. Based-on mips-next. Mark Cave-Ayland (1):

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

2021-07-11 Thread harry harry
Hi all, I hope you are very well! May I know whether it is possible to enable two-dimensional page translation (e.g., Intel EPT) mechanisms and shadow page table mechanisms in Linux QEMU/KVM at the same time on a physical server? For example, if the physical server has 80 cores, is it possible to

Re: [RFC PATCH v2 00/22] target/mips: Reintroduce the R5900 CPU (without testing)

2021-07-11 Thread Philippe Mathieu-Daudé
On 3/9/21 3:56 PM, Philippe Mathieu-Daudé wrote: > First part (TCG, testing postponed) of RFC v1: > https://www.mail-archive.com/qemu-devel@nongnu.org/msg782449.html > > 3 patches can still be improved for performance, but the improvement > is not yet addressed in this series. Other patches are go

Re: [PATCH v3 0/5] target/mips: Reintroduce the R5900 CPU

2021-07-11 Thread Philippe Mathieu-Daudé
On 3/12/21 5:24 PM, Philippe Mathieu-Daudé wrote: > I'm running out of time to address Richard's comments on the new > opcodes, so let's KISS and only fix RDHWR (after adding LQ/SQ). > > Missing review: 3 & 5 > - target/mips/tx79: Move RDHWR usermode kludge to trans_SQ() > - tests/tcg/mips: Test u

[PULL 0/6] hw/nvme patches

2021-07-11 Thread Klaus Jensen
From: Klaus Jensen Hi Pater, The following changes since commit 9516034d05a8c71ef157a59f525e4c4f7ed79827: Merge remote-tracking branch 'remotes/cminyard/tags/for-qemu-6.1-2' into staging (2021-07-11 14:32:49 +0100) are available in the Git repository at: git://git.infradead.org/qemu-nvme

[PULL 2/6] hw/nvme: mark nvme-subsys non-hotpluggable

2021-07-11 Thread Klaus Jensen
From: Klaus Jensen We currently lack the infrastructure to handle subsystem hotplugging, so disable it. Reviewed-by: Hannes Reinecke Signed-off-by: Klaus Jensen --- hw/nvme/subsys.c | 1 + 1 file changed, 1 insertion(+) diff --git a/hw/nvme/subsys.c b/hw/nvme/subsys.c index 192223d17ca1..dc7

[PULL 1/6] hw/nvme: remove NvmeCtrl parameter from ns setup/check functions

2021-07-11 Thread Klaus Jensen
From: Klaus Jensen The nvme_ns_setup and nvme_ns_check_constraints should not depend on the controller state. Refactor and remove it. Reviewed-by: Hannes Reinecke Signed-off-by: Klaus Jensen --- hw/nvme/nvme.h | 2 +- hw/nvme/ctrl.c | 2 +- hw/nvme/ns.c | 37 ++

Re: [PATCH v4 0/5] dp8393x: fixes and improvements

2021-07-11 Thread Philippe Mathieu-Daudé
On 7/11/21 12:36 PM, Philippe Mathieu-Daudé wrote: > Hi Mark, > > This should be the last respin. > > Since v3: > - dropped worrying patches > - squashed migration patch > - added tags > > Patch #3 (dp8393x: Store CAM registers as 16-bit) still > misses your S-o-b tag. > > Based-on mips-next. >

[PULL 4/6] hw/nvme: error handling for too many mappings

2021-07-11 Thread Klaus Jensen
From: Padmakar Kalghatgi If the number of PRP/SGL mappings exceed 1024, reads and writes will fail because of an internal QEMU limitation of max 1024 vectors. Signed-off-by: Padmakar Kalghatgi Reviewed-by: Klaus Jensen [k.jensen: changed the error message to be more generic] Signed-off-by: Kla

[PULL 6/6] hw/nvme: fix controller hot unplugging

2021-07-11 Thread Klaus Jensen
From: Klaus Jensen Prior to this patch the nvme-ns devices are always children of the NvmeBus owned by the NvmeCtrl. This causes the namespaces to be unrealized when the parent device is removed. However, when subsystems are involved, this is not what we want since the namespaces may be attached

[PULL 3/6] hw/nvme: unregister controller with subsystem at exit

2021-07-11 Thread Klaus Jensen
From: Klaus Jensen Make sure the controller is unregistered from the subsystem when device is removed. Reviewed-by: Hannes Reinecke Signed-off-by: Klaus Jensen --- hw/nvme/nvme.h | 1 + hw/nvme/ctrl.c | 4 hw/nvme/subsys.c | 5 + 3 files changed, 10 insertions(+) diff --git a/hw

Re: [PATCH v3 4/4] replication: Remove workaround

2021-07-11 Thread Lukas Straub
On Fri, 9 Jul 2021 10:49:23 +0300 Vladimir Sementsov-Ogievskiy wrote: > 07.07.2021 21:15, Lukas Straub wrote: > > Remove the workaround introduced in commit > > 6ecbc6c52672db5c13805735ca02784879ce8285 > > "replication: Avoid blk_make_empty() on read-only child". > > > > It is not needed anymore

[PULL 5/6] tests/qtest/nvme-test: add persistent memory region test

2021-07-11 Thread Klaus Jensen
From: Gollu Appalanaidu This will test the PMR functionality. Signed-off-by: Gollu Appalanaidu Reviewed-by: Klaus Jensen [k.jensen: replaced memory-backend-file with memory-backend-ram] Signed-off-by: Klaus Jensen --- tests/qtest/nvme-test.c | 61 - 1

[PATCH v4 0/4] replication: Bugfix and properly attach children

2021-07-11 Thread Lukas Straub
Hello Everyone, A while ago Kevin noticed that the replication driver doesn't properly attach the children it wants to use. Instead, it directly copies the BdrvChilds from it's backing file, which is wrong. Ths Patchset fixes the problem, fixes a potential crash in replication_co_writev due to miss

[PATCH v4 3/4] replication: Properly attach children

2021-07-11 Thread Lukas Straub
The replication driver needs access to the children block-nodes of it's child so it can issue bdrv_make_empty() and bdrv_co_pwritev() to manage the replication. However, it does this by directly copying the BdrvChilds, which is wrong. Fix this by properly attaching the block-nodes with bdrv_attach

[PATCH v4 1/4] replication: Remove s->active_disk

2021-07-11 Thread Lukas Straub
s->active_disk is bs->file. Remove it and use local variables instead. Signed-off-by: Lukas Straub Reviewed-by: Vladimir Sementsov-Ogievskiy --- block/replication.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/block/replication.c b/blo

[PATCH v4 4/4] replication: Remove workaround

2021-07-11 Thread Lukas Straub
Remove the workaround introduced in commit 6ecbc6c52672db5c13805735ca02784879ce8285 "replication: Avoid blk_make_empty() on read-only child". It is not needed anymore since s->hidden_disk is guaranteed to be writable when secondary_do_checkpoint() runs. Because replication_start(), _do_checkpoint(

[PATCH v4 2/4] replication: Reduce usage of s->hidden_disk and s->secondary_disk

2021-07-11 Thread Lukas Straub
In preparation for the next patch, initialize s->hidden_disk and s->secondary_disk later and replace access to them with local variables in the places where they aren't initialized yet. Signed-off-by: Lukas Straub Reviewed-by: Vladimir Sementsov-Ogievskiy --- block/replication.c | 45 ++

[PULL 01/19] hw/pci-host: Rename Raven ASIC PCI bridge as raven.c

2021-07-11 Thread Philippe Mathieu-Daudé
The ASIC PCI bridge chipset from Motorola is named 'Raven'. This chipset is used in the PowerPC Reference Platform (PReP), but not restricted to it. Rename it accordingly. Signed-off-by: Philippe Mathieu-Daudé Acked-by: David Gibson Message-Id: <20210417103028.601124-5-f4...@amsat.org> --- hw/p

[PULL 00/19] MIPS patches for 2021-07-11

2021-07-11 Thread Philippe Mathieu-Daudé
The following changes since commit 9516034d05a8c71ef157a59f525e4c4f7ed79827: Merge remote-tracking branch 'remotes/cminyard/tags/for-qemu-6.1-2' into staging (2021-07-11 14:32:49 +0100) are available in the Git repository at: https://github.com/philmd/qemu.git tags/mips-next-202

[PULL 02/19] hw/pci-host/raven: Add PCI_IO_BASE_ADDR definition

2021-07-11 Thread Philippe Mathieu-Daudé
Rather than using the magic 0x8000 number for the PCI I/O BAR physical address on the main system bus, use a definition. Signed-off-by: Philippe Mathieu-Daudé Acked-by: David Gibson Message-Id: <20210417103028.601124-6-f4...@amsat.org> --- hw/pci-host/raven.c | 11 +++ 1 file change

[PULL 09/19] target/mips/tx79: Introduce PPACW opcode (Parallel Pack to Word)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the PPACW opcode (Parallel Pack to Word). Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210214175912.732946-22-f4...@amsat.org> Signed-off-by: Philippe Mathieu-Daudé --- target/mips/tcg/tx79.decode | 1 + target/mips/tcg/tx79_translate.c |

[PULL 10/19] target/mips/tx79: Introduce PROT3W opcode (Parallel Rotate 3 Words)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the PROT3W opcode (Parallel Rotate 3 Words). Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210214175912.732946-25-f4...@amsat.org> Signed-off-by: Philippe Mathieu-Daudé --- target/mips/tcg/tx79.decode | 1 + target/mips/tcg/tx79_translate.c

[PULL 06/19] target/mips/tx79: Introduce PEXTL[BHW] opcodes (Parallel Extend Lower)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the 'Parallel Extend Lower' opcodes: - PEXTLB (Parallel Extend Upper from Byte) - PEXTLH (Parallel Extend Upper from Halfword) - PEXTLW (Parallel Extend Upper from Word) Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210309145653.743937-13-f4...

[PULL 13/19] target/mips: Rewrite UHI errno_mips() using switch statement

2021-07-11 Thread Philippe Mathieu-Daudé
Linking on Haiku OS fails: /boot/system/develop/tools/bin/../lib/gcc/x86_64-unknown-haiku/8.3.0/../../../../x86_64-unknown-haiku/bin/ld: error: libqemu-mips-softmmu.fa.p/target_mips_tcg_sysemu_mips-semi.c.o(.rodata) is too large (0x405a bytes) /boot/system/develop/tools/bin/../lib/gc

[PULL 03/19] target/mips/tx79: Introduce PAND/POR/PXOR/PNOR opcodes (parallel logic)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the parallel logic opcodes: - PAND (Parallel AND) - POR (Parallel OR) - PXOR (Parallel XOR) - PNOR (Parallel NOR) Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210214175912.732946-16-f4...@amsat.org> Signed-off-by: Philippe Mathieu-Daudé ---

[PULL 07/19] target/mips/tx79: Introduce PCEQ* opcodes (Parallel Compare for Equal)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the 'Parallel Compare for Equal' opcodes: - PCEQB (Parallel Compare for Equal Byte) - PCEQH (Parallel Compare for Equal Halfword) - PCEQW (Parallel Compare for Equal Word) Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210309145653.743937-14-f4.

[PULL 19/19] dp8393x: don't force 32-bit register access

2021-07-11 Thread Philippe Mathieu-Daudé
From: Mark Cave-Ayland Commit 3fe9a838ec "dp8393x: Always use 32-bit accesses" set .impl.min_access_size and .impl.max_access_size to 4 to try and fix the Linux jazzsonic driver which uses 32-bit accesses. The problem with forcing the register access to 32-bit in this way is that since the dp

[PULL 17/19] dp8393x: Store CAM registers as 16-bit

2021-07-11 Thread Philippe Mathieu-Daudé
Per the DP83932C datasheet from July 1995: 4.0 SONIC Registers 4.1 THE CAM UNIT The Content Addressable Memory (CAM) consists of sixteen 48-bit entries for complete address filtering of network packets. Each entry corresponds to a 48-bit destination address that is user progra

[PULL 04/19] target/mips/tx79: Introduce PSUB* opcodes (Parallel Subtract)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the 'Parallel Subtract' opcodes: - PSUBB (Parallel Subtract Byte) - PSUBH (Parallel Subtract Halfword) - PSUBW (Parallel Subtract Word) Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <820210309145653.743937-11-f4...@amsat.org> --- target/mips/tcg/

[PULL 08/19] target/mips/tx79: Introduce PCGT* (Parallel Compare for Greater Than)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the 'Parallel Compare for Greater Than' opcodes: - PCGTB (Parallel Compare for Greater Than Byte) - PCGTH (Parallel Compare for Greater Than Halfword) - PCGTW (Parallel Compare for Greater Than Word) Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <

[PULL 05/19] target/mips/tx79: Introduce PEXTUW (Parallel Extend Upper from Word)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the PEXTUW opcode (Parallel Extend Upper from Word). Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210309145653.743937-12-f4...@amsat.org> --- target/mips/tcg/tx79.decode | 4 target/mips/tcg/tx79_translate.c | 30 ++

[PULL 0/4] SD/MMC patches for 2021-07-11

2021-07-11 Thread Philippe Mathieu-Daudé
The following changes since commit 9516034d05a8c71ef157a59f525e4c4f7ed79827: Merge remote-tracking branch 'remotes/cminyard/tags/for-qemu-6.1-2' into staging (2021-07-11 14:32:49 +0100) are available in the Git repository at: https://github.com/philmd/qemu.git tags/sdmmc-2021071

[PULL 12/19] target/mips/tx79: Introduce SQ opcode (Store Quadword)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the SQ opcode (Store Quadword). Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210214175912.732946-27-f4...@amsat.org> Signed-off-by: Philippe Mathieu-Daudé --- target/mips/tcg/tx79.decode | 1 + target/mips/tcg/tx79_translate.c | 27 +++

[PULL 1/4] hw/sd/sdcard: When card is in wrong state, log which state it is

2021-07-11 Thread Philippe Mathieu-Daudé
We report the card is in an inconsistent state, but don't precise in which state it is. Add this information, as it is useful when debugging problems. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Bin Meng Message-Id: <20210624142209.1193073-2-f4...@amsat.org> Reviewed-by: Alexander Bulekov

[PULL 11/19] target/mips/tx79: Introduce LQ opcode (Load Quadword)

2021-07-11 Thread Philippe Mathieu-Daudé
Introduce the LQ opcode (Load Quadword) and remove unreachable code. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Richard Henderson Message-Id: <20210214175912.732946-26-f4...@amsat.org> Signed-off-by: Philippe Mathieu-Daudé --- target/mips/tcg/tx79.decode | 8 target/mips

[PULL 15/19] dp8393x: Replace address_space_rw(is_write=1) by address_space_write()

2021-07-11 Thread Philippe Mathieu-Daudé
Replace address_space_rw(is_write=1) by address_space_write() and remove pointless cast. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland Tested-by: Finn Thain Message-Id: <20210710174954.2577195-2-f4...@amsat.org> Tested-by: Mark Cave-Ayland --- hw/net/dp8393x.c | 8 --

[PULL 2/4] hw/sd/sdcard: Extract address_in_range() helper, log invalid accesses

2021-07-11 Thread Philippe Mathieu-Daudé
Multiple commands have to check the address requested is valid. Extract this code pattern as a new address_in_range() helper, and log invalid accesses as guest errors. Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Bin Meng Message-Id: <20210624142209.1193073-3-f4...@amsat.org> --- hw/sd/sd

[PULL 14/19] dp8393x: fix CAM descriptor entry index

2021-07-11 Thread Philippe Mathieu-Daudé
From: Mark Cave-Ayland Currently when a LOAD CAM command is executed the entries are loaded into the CAM from memory in order which is incorrect. According to the datasheet the first entry in the CAM descriptor is the entry index which means that each descriptor may update any single entry in the

[PULL 16/19] dp8393x: Replace 0x40 magic value by SONIC_REG_COUNT definition

2021-07-11 Thread Philippe Mathieu-Daudé
Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Mark Cave-Ayland Tested-by: Finn Thain Message-Id: <20210710174954.2577195-3-f4...@amsat.org> Tested-by: Mark Cave-Ayland --- hw/net/dp8393x.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/hw/net/dp8393x.c b/hw/net

Re: [PULL 00/19] MIPS patches for 2021-07-11

2021-07-11 Thread Philippe Mathieu-Daudé
ble in the Git repository at: > > https://github.com/philmd/qemu.git tags/mips-next-20210711 Resending with correct tag.

[PULL 18/19] dp8393x: Rewrite dp8393x_get() / dp8393x_put()

2021-07-11 Thread Philippe Mathieu-Daudé
Instead of accessing N registers via a single address_space API call using a temporary buffer (stored in the device state) and updating each register, move the address_space call in the register put/get. The load/store and word size checks are moved to put/get too. This simplifies a bit, making the

Re: [PULL 0/9] Trivial branch for 6.1 patches

2021-07-11 Thread Peter Maydell
On Fri, 9 Jul 2021 at 21:32, Laurent Vivier wrote: > > The following changes since commit 05de778b5b8ab0b402996769117b88c7ea5c7c61: > > Merge remote-tracking branch 'remotes/mst/tags/for_upstream' into staging > (2021-07-09 14:30:01 +0100) > > are available in the Git repository at: > > git:/

Re: [PATCH 3/9] tests/acceptance: Tag NetBSD tests as 'os:netbsd'

2021-07-11 Thread Philippe Mathieu-Daudé
On 7/5/21 10:55 AM, Philippe Mathieu-Daudé wrote: > Hi Niek, > > On 7/4/21 2:35 PM, Niek Linnenbank wrote: >> for test_arm_orangepi_uboot_netbsd9: >> >> Reviewed-by: Niek Linnenbank > > > > Thanks for the review. Does your R-b tag applies for this single > patch o

[PULL 4/4] hw/sd: sdhci: Enable 64-bit system bus capability in the default SD/MMC host controller

2021-07-11 Thread Philippe Mathieu-Daudé
From: Joanne Koong The default SD/MMC host controller uses SD spec v2.00. 64-bit system bus capability was added in v2. In this change, we arrive at 0x157834b4 by computing (0x057834b4 | (1ul << 28)) where 28 represents the BUS64BIT SDHC_CAPAB field. Signed-off-by: Joanne Koong Reviewed-by: B

[PULL 3/4] hw/sd/sdcard: Check for valid address range in SEND_WRITE_PROT (CMD30)

2021-07-11 Thread Philippe Mathieu-Daudé
OSS-Fuzz found sending illegal addresses when querying the write protection bits triggers an assertion: qemu-fuzz-i386: hw/sd/sd.c:824: uint32_t sd_wpbits(SDState *, uint64_t): Assertion `wpnum < sd->wpgrps_size' failed. ==11578== ERROR: libFuzzer: deadly signal #8 0x7628e091 in __asser

[PULL RESEND 00/19] MIPS patches for 2021-07-11

2021-07-11 Thread Philippe Mathieu-Daudé
The following changes since commit 9516034d05a8c71ef157a59f525e4c4f7ed79827: Merge remote-tracking branch 'remotes/cminyard/tags/for-qemu-6.1-2' into staging (2021-07-11 14:32:49 +0100) are available in the Git repository at: https://github.com/philmd/qemu.git tags/mips-2021071

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Coiby Xu
On Sun, Jul 11, 2021 at 06:23:41AM -0700, Richard Henderson wrote: On 7/11/21 5:16 AM, Peter Maydell wrote: On Sun, 11 Jul 2021 at 13:10, Coiby Xu wrote: Hi Peter, On Sat, Jul 10, 2021 at 02:30:36PM +0100, Peter Maydell wrote: I've noticed recently that intermittently 'make check' will hang

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Coiby Xu
On Sun, Jul 11, 2021 at 04:53:51PM +0100, Peter Maydell wrote: On Sat, 10 Jul 2021 at 14:30, Peter Maydell wrote: I've noticed recently that intermittently 'make check' will hang on my aarch32 test system (really an aarch64 box with an aarch32 chroot). I think from grep that this must be the

Re: intermittent hang in qos-test for qemu-system-i386 on 32-bit arm host

2021-07-11 Thread Coiby Xu
On Mon, Jul 12, 2021 at 06:20:33AM +0800, Coiby Xu wrote: On Sun, Jul 11, 2021 at 04:53:51PM +0100, Peter Maydell wrote: On Sat, 10 Jul 2021 at 14:30, Peter Maydell wrote: I've noticed recently that intermittently 'make check' will hang on my aarch32 test system (really an aarch64 box with an

RE: [PATCH v3 26/40] tests/tcg: make test-mmap a little less aggressive

2021-07-11 Thread Taylor Simpson
> -Original Message- > From: Alex Bennée > Sent: Friday, July 9, 2021 8:30 AM > To: qemu-devel@nongnu.org > Cc: f...@euphon.net; berra...@redhat.com; f4...@amsat.org; > aurel...@aurel32.net; pbonz...@redhat.com; stefa...@redhat.com; > cr...@redhat.com; c...@braap.org; aa...@os.amperecomp

Re: [PATCH v4 1/3] qapi/qdev.json: add DEVICE_UNPLUG_ERROR QAPI event

2021-07-11 Thread David Gibson
On Thu, Jul 08, 2021 at 03:01:20PM +0200, Markus Armbruster wrote: > Daniel Henrique Barboza writes: > > > At this moment we only provide one event to report a hotunplug error, > > MEM_UNPLUG_ERROR. As of Linux kernel 5.12 and QEMU 6.0.0, the pseries > > machine is now able to report unplug error

Re: [PATCH] ppc/pegasos2: Allow setprop in VOF

2021-07-11 Thread David Gibson
On Fri, Jul 09, 2021 at 03:19:13PM +0200, BALATON Zoltan wrote: > Linux needs setprop to fix up the device tree, otherwise it's not > finding devices and cannot boot. Since recent VOF change now we need > to add a callback to allow this which is what this patch does. > > Signed-off-by: BALATON Zol

[PATCH] vfio/pci: Add pba_offset PCI quirk for BAIDU KUNLUN AI processor

2021-07-11 Thread 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 --- hw/vfio/pci.c| 8 include/hw/pci/

[Bug 1914021] Re: qemu: uncaught target signal 4 (Illegal instruction) but gdb remote-debug exited normally

2021-07-11 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/1914021 Title: qemu: uncau

[Bug 1914667] Re: High cpu usage when guest is idle on qemu-system-i386

2021-07-11 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/1914667 Title: High cpu us

[Bug 1914294] Re: Windows XP displays black screen when smp option is used

2021-07-11 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/1914294 Title: Windows XP

[Bug 1913505] Re: Windows XP slow on Apple M1

2021-07-11 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/1913505 Title: Windows XP

[Bug 1912857] Re: virtio-serial blocks hostfwd ssh on windows 10 host

2021-07-11 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/1912857 Title: virtio-seri

  1   2   >