Re: [PATCH 0/5] disas/nanomips: Format string fixes

2022-11-01 Thread Richard Henderson
On 11/1/22 22:44, Philippe Mathieu-Daudé wrote: Fix invalid string formats reported by Stefan: https://lore.kernel.org/qemu-devel/78553699-00c1-ad69-1d58-02f75a1f4...@weilnetz.de/ Philippe Mathieu-Daudé (5): disas/nanomips: Fix invalid PRId64 format calling img_format() disas/nanomips:

Re: [PATCH 1/2] target/loongarch: Add exception subcode

2022-11-01 Thread Richard Henderson
On 11/1/22 18:32, Song Gao wrote: We need subcodes to distinguish the same excode cs->exception_indexs, such as EXCCODE_ADEF/EXCCODE_ADEM. Signed-off-by: Song Gao --- target/loongarch/cpu.c | 7 +++-- target/loongarch/cpu.h | 58 ++ 2 files changed,

Re: [PATCH 2/2] target/loongarch: Fix raise_mmu_exception() set wrong exception_index

2022-11-01 Thread Richard Henderson
On 11/1/22 18:32, Song Gao wrote: When the address is invalid address, We should set exception_index according to MMUAccessType, and EXCCODE_ADEF need't update badinstr. Otherwise, The system enters an infinite loop. e.g: run test.c on system mode test.c: #include void (*func)(int *);

[PATCH] target/arm: Two fixes for secure ptw

2022-11-01 Thread Richard Henderson
Reversed the sense of non-secure in get_phys_addr_lpae, and failed to initialize attrs.secure for ARMMMUIdx_Phys_S. Fixes: 48da29e4 ("target/arm: Add ptw_idx to S1Translate") Resolves: https://gitlab.com/qemu-project/qemu/-/issues/1293 Signed-off-by: Richard Henderson --- target/arm/ptw.c | 15

Re: [PATCH RFC 1/4] net: Introduce qmp cmd "query-netdev"

2022-11-01 Thread Jason Wang
On Tue, Nov 1, 2022 at 12:19 AM wrote: > > From: Hyman Huang(黄勇) > > For netdev device that can offload virtio-net dataplane to slave, > such as vhost-net, vhost-user and vhost-vdpa, exporting it's > capability information and acked features would be more friendly for > developers. These

Re: [PATCH v3] target/arm: honor HCR_E2H and HCR_TGE in ats_write64()

2022-11-01 Thread Richard Henderson
On 11/1/22 17:42, Ake Koomsin wrote: We need to check HCR_E2H and HCR_TGE to select the right MMU index for the correct translation regime. To check for EL2&0 translation regime: - For S1E0*, S1E1* and S12E* ops, check both HCR_E2H and HCR_TGE - For S1E2* ops, check only HCR_E2H Signed-off-by:

RE: [PATCH 09/16] hw/9pfs: Disable unsupported flags and features for Windows

2022-11-01 Thread Shi, Guohuai
> -Original Message- > From: Christian Schoenebeck > Sent: Wednesday, November 2, 2022 02:59 > To: qemu-devel@nongnu.org > Cc: Greg Kurz ; Meng, Bin ; Shi, > Guohuai > Subject: Re: [PATCH 09/16] hw/9pfs: Disable unsupported flags and features > for Windows > > [Please note: This

Re: [PATCH 2/2] file-posix: add statx(STATX_DIOALIGN) support

2022-11-01 Thread Eric Biggers
On Tue, Nov 01, 2022 at 03:00:31PM -0400, Stefan Hajnoczi wrote: > /* Let's try to use the logical blocksize for the alignment. */ > -if (probe_logical_blocksize(fd, >bl.request_alignment) < 0) { > -bs->bl.request_alignment = 0; > +if (!bs->bl.request_alignment) { > +

RE: [PATCH 07/16] hw/9pfs: Implement Windows specific utilities functions for 9pfs

2022-11-01 Thread Shi, Guohuai
> -Original Message- > From: Christian Schoenebeck > Sent: Wednesday, November 2, 2022 02:22 > To: qemu-devel@nongnu.org > Cc: Greg Kurz ; Meng, Bin ; Shi, > Guohuai > Subject: Re: [PATCH 07/16] hw/9pfs: Implement Windows specific utilities > functions for 9pfs > > [Please note: This

Re: [PATCH 1/2] file-posix: fix Linux alignment probing when EIO is returned

2022-11-01 Thread Eric Biggers
On Tue, Nov 01, 2022 at 07:27:16PM -0700, Eric Biggers wrote: > On Tue, Nov 01, 2022 at 03:00:30PM -0400, Stefan Hajnoczi wrote: > > Linux dm-crypt returns errno EIO from unaligned O_DIRECT pread(2) calls. > > Citation needed. For direct I/O to block devices, the kernel's block layer > checks

Re: [PATCH 1/2] file-posix: fix Linux alignment probing when EIO is returned

2022-11-01 Thread Eric Biggers
On Tue, Nov 01, 2022 at 03:00:30PM -0400, Stefan Hajnoczi wrote: > Linux dm-crypt returns errno EIO from unaligned O_DIRECT pread(2) calls. Citation needed. For direct I/O to block devices, the kernel's block layer checks the alignment before the I/O is actually submitted to the underlying block

Re: Re: [PATCH v2 1/4] virtio-crypto: Support asynchronous mode

2022-11-01 Thread Lei He
On 2022/11/2 03:51, Michael S. Tsirkin wrote: On Tue, Nov 01, 2022 at 06:37:26AM -0400, Michael S. Tsirkin wrote: On Sat, Oct 08, 2022 at 04:50:27PM +0800, Lei He wrote: virtio-crypto: Modify the current interface of virtio-crypto device to support asynchronous mode. Signed-off-by: lei he

Re: [PATCH 0/2] i386/cpuid: Minor fixes for CPUID leaf 1f setup

2022-11-01 Thread Xiaoyao Li
On 7/12/2022 10:12 AM, Xiaoyao Li wrote: The issue that fixed by Patch 1 looks fatal though it doesn't appear on KVM because KVM always searches with assending order and hit with the correct cpuid leaf 0. Patch 2 removes the wrong constraint on CPUID leaf 1f Kindly ping. Xiaoyao Li (2):

Re: [PATCH 4/5] target/riscv: No need to re-start QEMU timer when timecmp == UINT64_MAX

2022-11-01 Thread Alistair Francis
On Mon, Oct 31, 2022 at 1:49 PM Anup Patel wrote: > > On Mon, Oct 31, 2022 at 6:25 AM Alistair Francis wrote: > > > > On Fri, Oct 28, 2022 at 2:53 AM Anup Patel wrote: > > > > > > The time CSR will wrap-around immediately after reaching UINT64_MAX > > > so we don't need to re-start QEMU timer

Re: [PULL 59/62] hw/block/pflash_cfi0{1, 2}: Error out if device length isn't a power of two

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 23:23, Stefan Hajnoczi wrote: There is a report that this commit breaks an existing OVMF setup: https://gitlab.com/qemu-project/qemu/-/issues/1290#note_1156507334 I'm not familiar with pflash. Please find a way to avoid a regression in QEMU 7.2 here. Long-standing problem with

Re: [PATCH v5 1/6] hw/sd/sdhci: MMIO region is implemented in 32-bit accesses

2022-11-01 Thread Bernhard Beschow
Am 31. Oktober 2022 11:53:57 UTC schrieb "Philippe Mathieu-Daudé" : >Signed-off-by: Philippe Mathieu-Daudé Indeed there is `offset & ~0x3` in both sdhci_{read,write}, so: Reviewed-by: Bernhard Beschow >--- > hw/sd/sdhci.c | 4 > 1 file changed, 4 insertions(+) > >diff --git

[PATCH v6 2/3] hw/sd/sdhci: Support big endian SD host controller interfaces

2022-11-01 Thread Philippe Mathieu-Daudé
Some SDHCI IP can be synthetized in various endianness: https://github.com/u-boot/u-boot/blob/v2021.04/doc/README.fsl-esdhc - CONFIG_SYS_FSL_ESDHC_BE ESDHC IP is in big-endian mode. Accessing ESDHC registers can be determined by ESDHC IP's endian mode or processor's endian mode. Our

[PATCH v6 0/3] ppc/e500: Add support for eSDHC

2022-11-01 Thread Philippe Mathieu-Daudé
This is a respin of Bernhard's v4 with Freescale eSDHC implemented as an 'UNIMP' region. See v4 cover here: https://lore.kernel.org/qemu-devel/20221018210146.193159-1-shen...@gmail.com/ Since v5: - Rebased (ppc-next merged) - Properly handle big-endian Since v4: - Do not rename ESDHC_*

[PATCH v6 3/3] hw/ppc/e500: Add Freescale eSDHC to e500plat

2022-11-01 Thread Philippe Mathieu-Daudé
Adds missing functionality to e500plat machine which increases the chance of given "real" firmware images to access SD cards. Signed-off-by: Bernhard Beschow Message-Id: <20221018210146.193159-8-shen...@gmail.com> [PMD: Simplify using create_unimplemented_device("esdhc")] Signed-off-by: Philippe

[PATCH v6 1/3] hw/sd/sdhci: MMIO region is implemented in 32-bit accesses

2022-11-01 Thread Philippe Mathieu-Daudé
Tested-by: Bernhard Beschow Signed-off-by: Philippe Mathieu-Daudé --- hw/sd/sdhci.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c index 306070c872..22c758ad91 100644 --- a/hw/sd/sdhci.c +++ b/hw/sd/sdhci.c @@ -1332,6 +1332,10 @@ sdhci_write(void *opaque,

Re: [PULL 59/62] hw/block/pflash_cfi0{1, 2}: Error out if device length isn't a power of two

2022-11-01 Thread Stefan Hajnoczi
There is a report that this commit breaks an existing OVMF setup: https://gitlab.com/qemu-project/qemu/-/issues/1290#note_1156507334 I'm not familiar with pflash. Please find a way to avoid a regression in QEMU 7.2 here. Thank you! Stefan

[PATCH] tests/unit/test-io-channel-command: Silence GCC error "maybe-uninitialized"

2022-11-01 Thread Bernhard Beschow
GCC issues a false positive warning, resulting in build failure with -Werror: In file included from /usr/lib/glib-2.0/include/glibconfig.h:9, from /usr/include/glib-2.0/glib/gtypes.h:34, from /usr/include/glib-2.0/glib/galloca.h:34, from

Re: [PATCH qemu.git 11/11] hw/timer/imx_epit: rework CR write handling

2022-11-01 Thread Axel Heider
Original Message From: Philippe Mathieu-Daudé [mailto:phi...@linaro.org] - simplify code, improve comments - fix https://gitlab.com/qemu-project/qemu/-/issues/1263 This doesn't match GitLab issues closing pattern:

Re: [PATCH qemu.git 04/11] hw/timer/imx_epit: remove explicit fields cnt and freq

2022-11-01 Thread Axel Heider
Original Message From: Philippe Mathieu-Daudé [mailto:phi...@linaro.org]> diff --git a/hw/timer/imx_epit.c b/hw/timer/imx_epit.c> index a79f58c963..37b04a1b53 100644 --- a/hw/timer/imx_epit.c +++ b/hw/timer/imx_epit.c @@ -77,23 +77,25 @@ static void

Re: [PULL 0/3] Block patches

2022-11-01 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/7.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL v3 for 7.2 00/31] testing and plugin updates

2022-11-01 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/7.2 for any user-visible changes. signature.asc Description: PGP signature

Re: [PATCH] linux-user: implement execveat

2022-11-01 Thread Laurent Vivier
Le 31/10/2022 à 09:40, Drew DeVault a écrit : References: https://gitlab.com/qemu-project/qemu/-/issues/1007 Signed-off-by: Drew DeVault --- linux-user/syscall.c | 50 ++-- 1 file changed, 44 insertions(+), 6 deletions(-) diff --git

Re: [PATCH v2 1/4] virtio-crypto: Support asynchronous mode

2022-11-01 Thread Michael S. Tsirkin
On Tue, Nov 01, 2022 at 06:37:26AM -0400, Michael S. Tsirkin wrote: > On Sat, Oct 08, 2022 at 04:50:27PM +0800, Lei He wrote: > > virtio-crypto: Modify the current interface of virtio-crypto > > device to support asynchronous mode. > > > > Signed-off-by: lei he > > --- > >

Announcing QEMU 7.2 soft freeze

2022-11-01 Thread Stefan Hajnoczi
QEMU has entered soft feature freeze for the upcoming 7.2 release. https://wiki.qemu.org/Planning/SoftFeatureFreeze v7.2.0-rc0 is scheduled for tagging on November 8th. If you encounter a bug that needs to be fixed before the release, please create a GitLab Issue with the "7.2" milestone:

Re: [PATCH v5 0/6] ppc/e500: Add support for two types of flash, cleanup

2022-11-01 Thread B
Am 1. November 2022 17:43:46 UTC schrieb "Philippe Mathieu-Daudé" : >On 1/11/22 17:58, Philippe Mathieu-Daudé wrote: >> On 1/11/22 17:01, Bernhard Beschow wrote: >>> Am 1. November 2022 10:41:51 UTC schrieb Bernhard Beschow >>> : On Mon, Oct 31, 2022 at 12:54 PM Philippe Mathieu-Daudé

Re: [PATCH v9 1/8] mm: Introduce memfd_restricted system call to create restricted user memory

2022-11-01 Thread Michael Roth
On Tue, Nov 01, 2022 at 10:19:44AM -0500, Michael Roth wrote: > On Tue, Nov 01, 2022 at 07:37:29PM +0800, Chao Peng wrote: > > On Mon, Oct 31, 2022 at 12:47:38PM -0500, Michael Roth wrote: > > > On Tue, Oct 25, 2022 at 11:13:37PM +0800, Chao Peng wrote: > > > > > > > > 3) Potentially useful

[PATCH 1/2] file-posix: fix Linux alignment probing when EIO is returned

2022-11-01 Thread Stefan Hajnoczi
Linux dm-crypt returns errno EIO from unaligned O_DIRECT pread(2) calls. Alignment probing fails on dm-crypt devices because the code expects EINVAL. Treating any errno as an "unaligned" indicator would be easy, but breaks commit 22d182e82b4b ("block/raw-posix: fix launching with failed disks").

[PATCH 2/2] file-posix: add statx(STATX_DIOALIGN) support

2022-11-01 Thread Stefan Hajnoczi
Linux commit 825cf206ed51 ("statx: add direct I/O alignment information") added an interface to fetch O_DIRECT alignment values for block devices and file systems. Prefer STATX_DIOALIGN to older interfaces and probing, but keep them as fallbacks in case STATX_DIOALIGN cannot provide the

[PATCH 0/2] file-posix: alignment probing improvements

2022-11-01 Thread Stefan Hajnoczi
These patches fix alignment probing with dm-crypt and add support for the new Linux statx(STATX_DIOALIGN) interface. Stefan Hajnoczi (2): file-posix: fix Linux alignment probing when EIO is returned file-posix: add statx(STATX_DIOALIGN) support block/file-posix.c | 96

Re: [v2] hw: misc: edu: fix 2 off-by-one errors

2022-11-01 Thread Christopher Friedt
Hi Jiri, Peter, Are you able to review the more recent version of this change? Look for the subject "[PATCH v4 x/3] hw: misc: edu: ..." I believe I addressed all concerns. Cheers, C On Mon, Oct 17, 2022 at 12:36 PM Christopher Friedt wrote: > > On Mon, Oct 17, 2022 at 2:23 AM Jiri Slaby

Re: [PATCH 09/16] hw/9pfs: Disable unsupported flags and features for Windows

2022-11-01 Thread Christian Schoenebeck
On Tuesday, November 1, 2022 4:34:54 PM CET Shi, Guohuai wrote: > > > -Original Message- > > From: Christian Schoenebeck > > Sent: Tuesday, November 1, 2022 23:04 > > To: qemu-devel@nongnu.org > > Cc: Shi, Guohuai ; Greg Kurz ; > > Meng, Bin > > Subject: Re: [PATCH 09/16] hw/9pfs:

Re: [PATCH 07/16] hw/9pfs: Implement Windows specific utilities functions for 9pfs

2022-11-01 Thread Christian Schoenebeck
On Tuesday, November 1, 2022 4:20:53 PM CET Shi, Guohuai wrote: > [...] > > > > Windows POSIX API and MinGW library do not provide the NO_FOLLOW > > > > flag, and do not allow opening a directory by POSIX open(). This > > > > causes all > > > > xxx_at() functions cannot work directly. However, we

[PATCH 2/2] tests: acpi: Fixup for tables in Arm HMAT series

2022-11-01 Thread Hesham Almatary via
Updates to issues with earlier patches in the pull request: tests: virt: Update expected *.acpihmatvirt tables Signed-off-by: Hesham Almatary --- tests/data/acpi/virt/APIC.acpihmatvirt | Bin 396 -> 412 bytes 1 file changed, 0 insertions(+), 0 deletions(-) diff --git

[PATCH 1/2] tests: q35: acpi: Fixup for tables in noinitiator test.

2022-11-01 Thread Hesham Almatary via
Updates to issues with earlier patches in the pull request: tests: acpi: q35: update expected blobs *.hmat-noinitiators expected HMAT Signed-off-by: Hesham Almatary --- tests/data/acpi/q35/DSDT.acpihmat-noinitiator | Bin 8553 -> 8691 bytes 1 file changed, 0 insertions(+), 0 deletions(-) diff

Re: [PATCH v5 0/6] ppc/e500: Add support for two types of flash, cleanup

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 18:43, Philippe Mathieu-Daudé wrote: On 1/11/22 17:58, Philippe Mathieu-Daudé wrote: On 1/11/22 17:01, Bernhard Beschow wrote: Am 1. November 2022 10:41:51 UTC schrieb Bernhard Beschow : On Mon, Oct 31, 2022 at 12:54 PM Philippe Mathieu-Daudé wrote: This is a respin of

Re: [PATCH v5 0/6] ppc/e500: Add support for two types of flash, cleanup

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 17:58, Philippe Mathieu-Daudé wrote: On 1/11/22 17:01, Bernhard Beschow wrote: Am 1. November 2022 10:41:51 UTC schrieb Bernhard Beschow : On Mon, Oct 31, 2022 at 12:54 PM Philippe Mathieu-Daudé wrote: This is a respin of Bernhard's v4 with Freescale eSDHC implemented as an

Re: [PATCH] migration: check magic value for deciding the mapping of channels

2022-11-01 Thread Daniel P . Berrangé
On Tue, Nov 01, 2022 at 09:10:14PM +0530, manish.mishra wrote: > > On 01/11/22 8:21 pm, Daniel P. Berrangé wrote: > > On Tue, Nov 01, 2022 at 02:30:29PM +, manish.mishra wrote: > > > Current logic assumes that channel connections on the destination side are > > > always established in the

[PATCH v9 08/17] msi: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of msi_init(), which calls pci_add_capability() in turn, is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki --- hw/pci/msi.c | 9 + 1 file changed, 1 insertion(+), 8

[PATCH v8 01/17] hw/vfio/pci: Ensure MSI and MSI-X do not overlap

2022-11-01 Thread Akihiko Odaki
pci_add_capability() checks whether capabilities overlap, and notifies its caller so that it can properly handle the case. However, in the most cases, the capabilities actually never overlap, and the interface incurred extra error handling code, which is often incorrect or suboptimal. For such

[PATCH v8 15/17] hw/vfio/pci: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
The code generating errors in pci_add_capability has a comment which says: > Verify that capabilities don't overlap. Note: device assignment > depends on this check to verify that the device is not broken. > Should never trigger for emulated devices, but it's helpful for > debugging these.

[PATCH v8 17/17] pci: Remove legacy errp from pci_add_capability

2022-11-01 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki --- hw/pci/pci.c | 20 +--- include/hw/pci/pci.h | 12 ++-- 2 files changed, 7 insertions(+), 25 deletions(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index cce57f572c..41de7643af 100644 --- a/hw/pci/pci.c +++ b/hw/pci/pci.c @@

Re: [PULL v2 00/13] tcg-next patch queue

2022-11-01 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/7.2 for any user-visible changes. signature.asc Description: PGP signature

[PATCH v9 10/17] pcie: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of a PCIe function which calls pci_add_capability() in turn is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki Acked-by: Jonathan Cameron (for CXL parts) --- docs/pcie_sriov.txt

Re: [PATCH] migration: check magic value for deciding the mapping of channels

2022-11-01 Thread manish.mishra
On 01/11/22 9:15 pm, Daniel P. Berrangé wrote: On Tue, Nov 01, 2022 at 09:10:14PM +0530, manish.mishra wrote: On 01/11/22 8:21 pm, Daniel P. Berrangé wrote: On Tue, Nov 01, 2022 at 02:30:29PM +, manish.mishra wrote: Current logic assumes that channel connections on the destination side

[PATCH v9 00/17] pci: Abort if pci_add_capability fails

2022-11-01 Thread Akihiko Odaki
pci_add_capability appears most PCI devices. Its error handling required lots of code, and led to inconsistent behaviors such as: - passing error_abort - passing error_fatal - asserting the returned value - propagating the error to the caller - skipping the rest of the function - just ignoring

[PATCH v9 17/17] pci: Remove legacy errp from pci_add_capability

2022-11-01 Thread Akihiko Odaki
Signed-off-by: Akihiko Odaki --- include/hw/pci/pci.h | 12 ++-- hw/pci/pci.c | 18 -- 2 files changed, 6 insertions(+), 24 deletions(-) diff --git a/include/hw/pci/pci.h b/include/hw/pci/pci.h index 555ac03010..da414dc728 100644 --- a/include/hw/pci/pci.h +++

Re: [PATCH] target/i386/cpu: disable PERFCORE for AMD when cpu.pmu is off

2022-11-01 Thread Liang Yan
Hey Vitaly, On 10/31/22 6:07 AM, Vitaly Kuznetsov wrote: Liang Yan writes: With cpu.pmu=off, perfctr_core could still be seen in an AMD guest cpuid. By further digging, I found cpu.perfctr_core did the trick. However, considering the 'enable_pmu' in KVM could work on both Intel and AMD, we

Re: [PATCH] hw/i386/pc.c: CXL Fixed Memory Window should not reserve e820 in bios

2022-11-01 Thread Jonathan Cameron via
On Wed, 26 Oct 2022 16:59:13 -0400 Gregory Price wrote: > Early-boot e820 records will be inserted by the bios/efi/early boot > software and be reported to the kernel via insert_resource. Later, when > CXL drivers iterate through the regions again, they will insert another > resource and make

Re: [PULL 21/86] bios-tables-test: add test for number of cores > 255

2022-11-01 Thread Igor Mammedov
On Tue, 1 Nov 2022 13:52:21 + Jonathan Cameron wrote: > On Mon, 31 Oct 2022 08:51:44 -0400 > "Michael S. Tsirkin" wrote: > > > From: Julia Suvorova > > > > The new test is run with a large number of cpus and checks if the > > core_count field in smbios_cpu_test (structure type 4) is

[PATCH v9 07/17] hw/nvme: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: [PATCH] migration: check magic value for deciding the mapping of channels

2022-11-01 Thread Daniel P . Berrangé
On Tue, Nov 01, 2022 at 02:30:29PM +, manish.mishra wrote: > Current logic assumes that channel connections on the destination side are > always established in the same order as the source and the first one will > always be the default channel followed by the multifid or post-copy > preemption

Re: [PATCH v7 01/17] hw/vfio/pci: Ensure MSI and MSI-X do not overlap

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 01:58, Akihiko Odaki wrote: pci_add_capability() checks whether capabilities overlap, and notifies its caller so that it can properly handle the case. However, in the most cases, the capabilities actually never overlap, and the interface incurred extra error handling code, which is

Re: [PATCH] virtio: add VIRTQUEUE_ERROR QAPI event

2022-11-01 Thread Vladimir Sementsov-Ogievskiy
ping On 9/19/22 22:48, Vladimir Sementsov-Ogievskiy wrote: For now we only log the vhost device error, when virtqueue is actually stopped. Let's add a QAPI event, which makes possible: - collect statistics of such errors - make immediate actions: take coredums or do some other debugging

[PATCH v9 09/17] hw/pci/pci_bridge: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of pci_bridge_ssvid_init(), which calls pci_add_capability() in turn, is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki --- include/hw/pci/pci_bridge.h| 5 ++---

[PATCH v9 02/17] pci: Allow to omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
pci_add_capability appears most PCI devices. Its error handling required lots of code, and led to inconsistent behaviors such as: - passing error_abort - passing error_fatal - asserting the returned value - propagating the error to the caller - skipping the rest of the function - just ignoring

Re: [PATCH v4 2/3] util/main-loop: Avoid adding the same HANDLE twice

2022-11-01 Thread Philippe Mathieu-Daudé
On 19/10/22 12:20, Bin Meng wrote: From: Bin Meng Fix the logic in qemu_add_wait_object() to avoid adding the same HANDLE twice, as the behavior is undefined when passing an array that contains same HANDLEs to WaitForMultipleObjects() API. Signed-off-by: Bin Meng --- (no changes since v3)

[PATCH v8 04/17] ahci: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

[PATCH v9 11/17] pci/shpc: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of shpc_init(), which calls pci_add_capability() in turn, is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki --- include/hw/pci/shpc.h | 3 +--

Re: [PATCH] migration: check magic value for deciding the mapping of channels

2022-11-01 Thread manish.mishra
On 01/11/22 8:21 pm, Daniel P. Berrangé wrote: On Tue, Nov 01, 2022 at 02:30:29PM +, manish.mishra wrote: Current logic assumes that channel connections on the destination side are always established in the same order as the source and the first one will always be the default channel

[PATCH v9 15/17] hw/vfio/pci: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
The code generating errors in pci_add_capability has a comment which says: > Verify that capabilities don't overlap. Note: device assignment > depends on this check to verify that the device is not broken. > Should never trigger for emulated devices, but it's helpful for > debugging these.

Re: [PATCH 15/16] tests/qtest: virtio-9p-test: Adapt the case for win32

2022-11-01 Thread Christian Schoenebeck
On Monday, October 24, 2022 6:57:58 AM CET Bin Meng wrote: > From: Guohuai Shi > > Windows does not provide the getuid() API. Let's create a local > one and return a fixed value 0 as the uid for testing. > > Signed-off-by: Guohuai Shi > Signed-off-by: Xuzhou Cheng > Signed-off-by: Bin Meng >

Re: [PULL 00/14] qemu-macppc queue 20221031

2022-11-01 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/7.2 for any user-visible changes. signature.asc Description: PGP signature

[PATCH v8 12/17] msix: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of msix_init(), which calls pci_add_capability() in turn, is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki --- hw/pci/msix.c | 8 ++-- 1 file changed, 2 insertions(+), 6

UI layer threading and locking strategy; memory_region_snapshot_and_clear_dirty() races

2022-11-01 Thread Peter Maydell
Hi; I'm trying to find out what the UI layer's threading and locking strategy is, at least as far as it applies to display device models. Specifically: * is the device's GraphicHwOps::gfx_update method always called from one specific thread, or might it be called from any thread? * is that

[PATCH v8 00/17] pci: Abort if pci_add_capability fails

2022-11-01 Thread Akihiko Odaki
pci_add_capability appears most PCI devices. Its error handling required lots of code, and led to inconsistent behaviors such as: - passing error_abort - passing error_fatal - asserting the returned value - propagating the error to the caller - skipping the rest of the function - just ignoring

[PATCH v8 16/17] virtio-pci: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: HMAT patches failure (was Re: [PULL 00/86] pci,pc,virtio: features, tests, fixes, cleanups)

2022-11-01 Thread Jonathan Cameron via
On Tue, 1 Nov 2022 06:32:05 -0400 "Michael S. Tsirkin" wrote: > On Mon, Oct 31, 2022 at 04:06:03PM -0400, Stefan Hajnoczi wrote: > > Here is another CI failure: > > > > qemu-system-i386: current -smp configuration requires kernel irqchip > > and X2APIC API support. > > Broken pipe > >

RE: [PATCH 07/16] hw/9pfs: Implement Windows specific utilities functions for 9pfs

2022-11-01 Thread Shi, Guohuai
> -Original Message- > From: Shi, Guohuai > Sent: Tuesday, November 1, 2022 23:13 > To: Christian Schoenebeck ; qemu-devel@nongnu.org > Cc: Greg Kurz ; Meng, Bin > Subject: RE: [PATCH 07/16] hw/9pfs: Implement Windows specific utilities > functions for 9pfs > > > > > -Original

[PATCH v8 08/17] msi: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of msi_init(), which calls pci_add_capability() in turn, is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki --- hw/pci/msi.c | 9 + 1 file changed, 1 insertion(+), 8

Re: [PULL 08/30] target/arm: Add ptw_idx to S1Translate

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 11:10, Philippe Mathieu-Daudé wrote: On 1/11/22 00:14, Philippe Mathieu-Daudé wrote: On 25/10/22 18:39, Peter Maydell wrote: From: Richard Henderson Hoist the computation of the mmu_idx for the ptw up to get_phys_addr_with_struct and get_phys_addr_twostage. This removes the

Re: [PATCH 07/16] hw/9pfs: Implement Windows specific utilities functions for 9pfs

2022-11-01 Thread Christian Schoenebeck
On Monday, October 24, 2022 6:57:50 AM CET Bin Meng wrote: > From: Guohuai Shi > > Windows POSIX API and MinGW library do not provide the NO_FOLLOW > flag, and do not allow opening a directory by POSIX open(). This > causes all xxx_at() functions cannot work directly. However, we > can provide

[PATCH v9 14/17] hw/pci-bridge/pcie_pci_bridge: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate heare because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

[PATCH v8 03/17] hw/i386/amd_iommu: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

[PATCH v9 01/17] hw/vfio/pci: Ensure MSI and MSI-X do not overlap

2022-11-01 Thread Akihiko Odaki
pci_add_capability() checks whether capabilities overlap, and notifies its caller so that it can properly handle the case. However, in the most cases, the capabilities actually never overlap, and the interface incurred extra error handling code, which is often incorrect or suboptimal. For such

Re: [PATCH 09/16] hw/9pfs: Disable unsupported flags and features for Windows

2022-11-01 Thread Christian Schoenebeck
On Monday, October 24, 2022 6:57:52 AM CET Bin Meng wrote: > From: Guohuai Shi > > Some flags and features are not supported on Windows, like mknod, > readlink, file mode, etc. Update the codes for Windows. > > Signed-off-by: Guohuai Shi > Signed-off-by: Bin Meng > --- > > hw/9pfs/9p-util.h

[PATCH v9 03/17] hw/i386/amd_iommu: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: [PATCH v5 0/6] ppc/e500: Add support for two types of flash, cleanup

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 17:01, Bernhard Beschow wrote: Am 1. November 2022 10:41:51 UTC schrieb Bernhard Beschow : On Mon, Oct 31, 2022 at 12:54 PM Philippe Mathieu-Daudé wrote: This is a respin of Bernhard's v4 with Freescale eSDHC implemented as an 'UNIMP' region. See v4 cover here:

[PATCH v9 05/17] e1000e: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: [PATCH v4 1/3] util/main-loop: Fix maximum number of wait objects for win32

2022-11-01 Thread Bin Meng
On Tue, Nov 1, 2022 at 8:03 PM Daniel P. Berrangé wrote: > > On Tue, Nov 01, 2022 at 09:14:55AM +0800, Bin Meng wrote: > > Hi Daniel, > > > > On Wed, Oct 26, 2022 at 12:41 AM Bin Meng wrote: > > > > > > On Wed, Oct 19, 2022 at 6:20 PM Bin Meng wrote: > > > > > > > > From: Bin Meng > > > > > >

Re: [PATCH 0/4] hw: make TCO watchdog actually work by default for Q35

2022-11-01 Thread Igor Mammedov
On Mon, 31 Oct 2022 11:48:58 -0400 "Michael S. Tsirkin" wrote: > On Mon, Oct 31, 2022 at 01:50:24PM +, Daniel P. Berrangé wrote: > > On Mon, Oct 31, 2022 at 01:19:30PM +, Daniel P. Berrangé wrote: > > > The TCO watchdog is unconditionally integrated into the Q35 machine > > > type by

[PATCH v9 04/17] ahci: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: [PATCH v8 17/17] pci: Remove legacy errp from pci_add_capability

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 14:57, Akihiko Odaki wrote: Signed-off-by: Akihiko Odaki --- hw/pci/pci.c | 20 +--- include/hw/pci/pci.h | 12 ++-- 2 files changed, 7 insertions(+), 25 deletions(-) diff --git a/hw/pci/pci.c b/hw/pci/pci.c index cce57f572c..41de7643af 100644 ---

[PATCH v8 05/17] e1000e: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: [PATCH v5 0/6] ppc/e500: Add support for two types of flash, cleanup

2022-11-01 Thread Bernhard Beschow
Am 1. November 2022 10:41:51 UTC schrieb Bernhard Beschow : >On Mon, Oct 31, 2022 at 12:54 PM Philippe Mathieu-Daudé >wrote: > >> This is a respin of Bernhard's v4 with Freescale eSDHC implemented >> as an 'UNIMP' region. See v4 cover here: >> >>

Re: [PATCH v8 01/17] hw/vfio/pci: Ensure MSI and MSI-X do not overlap

2022-11-01 Thread Philippe Mathieu-Daudé
On 1/11/22 14:57, Akihiko Odaki wrote: pci_add_capability() checks whether capabilities overlap, and notifies its caller so that it can properly handle the case. However, in the most cases, the capabilities actually never overlap, and the interface incurred extra error handling code, which is

[PATCH v8 13/17] pci/slotid: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of slotid_cap_init(), which calls pci_add_capability() in turn, is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki --- hw/pci/slotid_cap.c | 8 ++-- 1 file changed, 2

Re: [PATCH] migration: check magic value for deciding the mapping of channels

2022-11-01 Thread manish.mishra
Sorry for late patch on this. I mentioned i will send it last week itself, but later reliased it was festival week in India, so was mostly holidays. Thanks Manish Mishra On 01/11/22 8:00 pm, manish.mishra wrote: Current logic assumes that channel connections on the destination side are

[PATCH v8 09/17] hw/pci/pci_bridge: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. A caller of pci_bridge_ssvid_init(), which calls pci_add_capability() in turn, is expected to ensure that will not happen. Signed-off-by: Akihiko Odaki --- hw/pci-bridge/i82801b11.c | 14 ++

Re: [PATCH 11/16] hw/9pfs: Add Linux error number definition

2022-11-01 Thread Christian Schoenebeck
On Monday, October 24, 2022 6:57:54 AM CET Bin Meng wrote: > From: Guohuai Shi > > When using 9p2000.L protocol, the errno should use the Linux errno. > Currently magic numbers with comments are used. Replace these with > macros for future expansion. > > Signed-off-by: Guohuai Shi >

[PATCH v8 06/17] eepro100: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: [PULL 21/86] bios-tables-test: add test for number of cores > 255

2022-11-01 Thread Ani Sinha
On Tue, Nov 1, 2022 at 19:22 Jonathan Cameron wrote: > On Mon, 31 Oct 2022 08:51:44 -0400 > "Michael S. Tsirkin" wrote: > > > From: Julia Suvorova > > > > The new test is run with a large number of cpus and checks if the > > core_count field in smbios_cpu_test (structure type 4) is correct. >

[PULL v3 for 7.2 00/31] testing and plugin updates

2022-11-01 Thread Alex Bennée
The following changes since commit 5107fd3effb1cfec3b96d9e819f1605048640e31: net/vhost-vdpa.c: Fix clang compilation failure (2022-10-31 13:01:31 -0400) are available in the Git repository at: https://github.com/stsquad/qemu.git tags/pull-testing-for-7.2-011122-3 for you to fetch changes

[PATCH v9 16/17] virtio-pci: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

[PATCH v9 06/17] eepro100: Omit errp for pci_add_capability

2022-11-01 Thread Akihiko Odaki
Omitting errp for pci_add_capability() causes it to abort if capabilities overlap. This behavior is appropriate here because all of the capabilities set in this device are defined in the program and their overlap should not happen unless there is a programming error. Signed-off-by: Akihiko Odaki

Re: [PATCH v4 1/3] util/main-loop: Fix maximum number of wait objects for win32

2022-11-01 Thread Philippe Mathieu-Daudé
On 19/10/22 12:20, Bin Meng wrote: From: Bin Meng The maximum number of wait objects for win32 should be MAXIMUM_WAIT_OBJECTS, not MAXIMUM_WAIT_OBJECTS + 1. Signed-off-by: Bin Meng --- Changes in v4: - make the out of bounds access protection explicit Changes in v3: - move the check of

  1   2   >