Re: [PATCH V2] hw/riscv: virt: Remove size restriction for pflash

2022-11-08 Thread Philippe Mathieu-Daudé
On 7/11/22 18:34, Daniel P. Berrangé wrote: On Mon, Nov 07, 2022 at 06:32:01PM +0100, Andrew Jones wrote: On Mon, Nov 07, 2022 at 04:19:10PM +, Daniel P. Berrangé wrote: On Mon, Nov 07, 2022 at 03:50:44PM +, Alex Bennée wrote: Sunil V L writes: On Mon, Nov 07, 2022 at 01:06:38PM

[PATCH] checkpatch: typo fix

2022-11-08 Thread Michael S. Tsirkin
remove inline #inline - it's an obvious typo. Should just be remove inline. Signed-off-by: Michael S. Tsirkin --- scripts/checkpatch.pl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index bc7d4780ec..6ecabfb2b5 100755 ---

Re: [PATCH v2 0/3] block: Start/end drain on correct AioContext

2022-11-08 Thread Kevin Wolf
Am 07.11.2022 um 16:13 hat Hanna Reitz geschrieben: > Hi, > > v1 cover letter: > https://lists.nongnu.org/archive/html/qemu-block/2022-09/msg00389.html > > bdrv_replace_child_noperm() drains the child via > bdrv_parent_drained_{begin,end}_single(). When it removes a child, the >

[PATCH for-7.2] block/blkio: Set BlockDriver::has_variable_length to false

2022-11-08 Thread Alberto Faria
Setting it to true can cause the device size to be queried from libblkio in otherwise fast paths, degrading performance. Set it to false and require users to refresh the device size explicitly instead. Fixes: 4c8f4fda0504 ("block/blkio: Tolerate device size changes") Suggested-by: Kevin Wolf

Re: [PATCH v2 1/9] block: call bdrv_co_drain_begin in a coroutine

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/8/22 18:13, Emanuele Giuseppe Esposito wrote: Am 08/11/2022 um 15:33 schrieb Vladimir Sementsov-Ogievskiy: On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: It seems that bdrv_open_driver() forgot to create a coroutine where to call bs->drv->bdrv_co_drain_begin(), a callback marked

[PATCH v5 06/11] Hexagon (target/hexagon) Remove next_PC from runtime state

2022-11-08 Thread Taylor Simpson
The imported files don't properly mark all CONDEXEC instructions, so we add some logic to hex_common.py to add the attribute. Acked-by: Richard Henderson Signed-off-by: Taylor Simpson --- target/hexagon/cpu.h| 1 - target/hexagon/gen_tcg.h| 6 ++

Re: [PULL 0/2] Net patches

2022-11-08 Thread Stefan Hajnoczi
On Mon, 7 Nov 2022 at 23:20, Jason Wang wrote: > > The following changes since commit 524fc737431d240f9d9f10aaf381003092868bac: > > util/log: Ignore per-thread flag if global file already there (2022-11-07 > 16:00:02 -0500) > > are available in the git repository at: > >

[PATCH v5 00/11] Hexagon (target/hexagon) performance and bug fixes

2022-11-08 Thread Taylor Simpson
1) Performance improvement Add pkt and insn to DisasContext Many functions need information from all 3 structures, so merge them together. 2) Bug fix Fix predicated assignment to .tmp and .cur 3) Performance improvement Add overrides for S2_asr_r_r_sat/S2_asl_r_r_sat These functions will not be

[PATCH v6 10/10] vdpa: Always start CVQ in SVQ mode

2022-11-08 Thread Eugenio Pérez
Isolate control virtqueue in its own group, allowing to intercept control commands but letting dataplane run totally passthrough to the guest. Signed-off-by: Eugenio Pérez --- v6: * Disable control SVQ if the device does not support it because of features. v5: * Fixing the not adding cvq

[PATCH v6 08/10] vdpa: Store x-svq parameter in VhostVDPAState

2022-11-08 Thread Eugenio Pérez
CVQ can be shadowed two ways: - Device has x-svq=on parameter (current way) - The device can isolate CVQ in its own vq group QEMU needs to check for the second condition dynamically, because CVQ index is not known at initialization time. Since this is dynamic, the CVQ isolation could vary with

[PATCH v6 07/10] vdpa: Add asid parameter to vhost_vdpa_dma_map/unmap

2022-11-08 Thread Eugenio Pérez
So the caller can choose which ASID is destined. No need to update the batch functions as they will always be called from memory listener updates at the moment. Memory listener updates will always update ASID 0, as it's the passthrough ASID. All vhost devices's ASID are 0 at this moment.

[PATCH v6 09/10] vdpa: Add listener_shadow_vq to vhost_vdpa

2022-11-08 Thread Eugenio Pérez
The memory listener that thells the device how to convert GPA to qemu's va is registered against CVQ vhost_vdpa. This series try to map the memory listener translations to ASID 0, while it maps the CVQ ones to ASID 1. Let's tell the listener if it needs to register them on iova tree or not.

[PATCH v6 01/10] vdpa: Use v->shadow_vqs_enabled in vhost_vdpa_svqs_start & stop

2022-11-08 Thread Eugenio Pérez
This function used to trust in v->shadow_vqs != NULL to know if it must start svq or not. This is not going to be valid anymore, as qemu is going to allocate svq unconditionally (but it will only start them conditionally). Signed-off-by: Eugenio Pérez --- hw/virtio/vhost-vdpa.c | 4 ++-- 1

[PATCH v6 04/10] vdpa: add vhost_vdpa_net_valid_svq_features

2022-11-08 Thread Eugenio Pérez
It will be reused at vdpa device start so let's extract in its own function Signed-off-by: Eugenio Pérez --- net/vhost-vdpa.c | 26 +- 1 file changed, 17 insertions(+), 9 deletions(-) diff --git a/net/vhost-vdpa.c b/net/vhost-vdpa.c index e370ecb8eb..d3b1de481b 100644

Re: [PULL 00/55] MIPS patches for 2022-10-30

2022-11-08 Thread Paolo Bonzini
Il mar 8 nov 2022, 16:09 Thomas Huth ha scritto: > >> If it is the last thing, we should put in the "Build Dependencies" > >> part of the release notes that a C++ compiler is no longer required > >> and mention that the configure options to specify it will go away in > >> a future release. > > >

Re: [PATCH v3 06/17] vfio/migration: Fix NULL pointer dereference bug

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/3/22 19:16, Avihai Horon wrote: As part of its error flow, vfio_vmstate_change() accesses MigrationState->to_dst_file without any checks. This can cause a NULL pointer dereference if the error flow is taken and MigrationState->to_dst_file is not set. For example, this can happen if VM is

[PATCH] qapi/block-core: Fix BlockdevOptionsNvmeIoUring @path description

2022-11-08 Thread Alberto Faria
The nvme-io_uring BlockDriver's path option must point at the character device of an NVMe namespace, not at an image file. Fixes: fd66dbd424f5 ("blkio: add libblkio block driver") Suggested-by: Stefano Garzarella Signed-off-by: Alberto Faria --- qapi/block-core.json | 2 +- 1 file changed, 1

[PATCH v5 04/11] Hexagon (target/hexagon) Only use branch_taken when packet has multi cof

2022-11-08 Thread Taylor Simpson
When a packet has more than one change-of-flow instruction, only the first one to branch is considered. We use the branch_taken variable to keep track of this. However, when there is a single cof instruction, we don't need the same amount of bookkeeping. We add the pkt_has_multi_cof member to

[PATCH v6 00/10] ASID support in vhost-vdpa net

2022-11-08 Thread Eugenio Pérez
Control VQ is the way net devices use to send changes to the device state, like the number of active queues or its mac address. QEMU needs to intercept this queue so it can track these changes and is able to migrate the device. It can do it from 1576dbb5bbc4 ("vdpa: Add x-svq to

[PATCH v6 03/10] vhost: Allocate SVQ device file descriptors at device start

2022-11-08 Thread Eugenio Pérez
The next patches will start control SVQ if possible. However, we don't know if that will be possible at qemu boot anymore. Delay device file descriptors until we know it at device start. Signed-off-by: Eugenio Pérez --- hw/virtio/vhost-shadow-virtqueue.c | 31 ++

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

2022-11-08 Thread Daniel Henrique Barboza
Phil, On 11/1/22 19:49, Philippe Mathieu-Daudé wrote: 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

Re: [PATCH v3 4/4] scripts: add script to compare compatible properties

2022-11-08 Thread Maksim Davydov
On 11/8/22 18:37, Vladimir Sementsov-Ogievskiy wrote: On 11/3/22 13:27, Maksim Davydov wrote: This script run QEMU to obtain compat_props of machines and default values of different types and produce appropriate table. This table can be used to compare machine types to choose the most

Re: [PATCH v3 05/17] vfio/migration: Fix wrong enum usage

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/3/22 19:16, Avihai Horon wrote: vfio_migration_init() initializes VFIOMigration->device_state using enum of VFIO migration protocol v2. Current implemented protocol is v1 so v1 enum should be used. Fix it. Fixes: 429c72800654 ("vfio/migration: Fix incorrect initialization value for

Re: [PATCH v3 09/30] nbd/server: Clean up abuse of BlockExportOptionsNbd member @arg

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/4/22 19:06, Markus Armbruster wrote: block-export-add argument @name defaults to the value of argument @node-name. nbd_export_create() implements this by copying @node_name to @name. It leaves @has_node_name false, violating the "has_node_name == !!node_name" invariant. Unclean. Falls

Re: [PATCH v2 5/9] block: distinguish between bdrv_create running in coroutine and not

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: Call two different functions depending on whether bdrv_create is in coroutine or not, following the same pattern as generated_co_wrapper functions. This allows to also call the coroutine function directly, without using CreateCo or relying in

Call for FOSDEM presentations on QEMU, KVM, and rust-vmm

2022-11-08 Thread Stefan Hajnoczi
Hi, The yearly FOSDEM open source conference is now accepting talk proposals. FOSDEM '23 will be held in Brussels, Belgium on 4 & 5 February. FOSDEM is a huge free conference about all things open source and an opportunity for anyone to present QEMU or KVM topics. Both in-person and pre-recorded

Re: [RFC PATCH 3/3] kvm: Atomic memslot updates

2022-11-08 Thread Paolo Bonzini
On 11/4/22 16:14, Emanuele Giuseppe Esposito wrote: +g_assert(qemu_mutex_iothread_locked()); Please add a comment here: /* Block further invocations of the ioctls outside the BQL. */ +CPU_FOREACH(cpu) { +qemu_lockcnt_lock(>in_ioctl_lock); +} +

[PATCH v5 05/11] Hexagon (target/hexagon) Remove PC from the runtime state

2022-11-08 Thread Taylor Simpson
Add pc field to Packet structure For helpers that need PC, pass an extra argument Remove slot arg from conditional jump helpers On a trap0, copy pkt->pc into hex_gpr[HEX_REG_PC] Reviewed-by: Richard Henderson Signed-off-by: Taylor Simpson --- target/hexagon/gen_tcg.h| 7 +++

[PATCH v5 01/11] Hexagon (target/hexagon) Add pkt and insn to DisasContext

2022-11-08 Thread Taylor Simpson
This enables us to reduce the number of parameters to many functions In particular, the generated functions previously took all 3 as arguments Not only does this simplify the code, it improves the translation time Reviewed-by: Richard Henderson Signed-off-by: Taylor Simpson ---

[RFC PATCH] tests/plugins: add a new vcpu state tracking plugin

2022-11-08 Thread Alex Bennée
Although we call qemu_plugin_register_vcpu_idle_cb() in the bb test we don't really exercise the rest of the state change callbacks. Add a new test that tests the whole API. [AJB: I wrote this in an attempt to flush out a reproducer for #1195 although so far now joy.] Signed-off-by: Alex Bennée

[PATCH v6 06/10] vdpa: Allocate SVQ unconditionally

2022-11-08 Thread Eugenio Pérez
SVQ may run or not in a device depending on runtime conditions (for example, if the device can move CVQ to its own group or not). Allocate the SVQ array unconditionally at startup, since its hard to move this allocation elsewhere. Signed-off-by: Eugenio Pérez --- hw/virtio/vhost-vdpa.c | 4

[PATCH v6 05/10] vdpa: move SVQ vring features check to net/

2022-11-08 Thread Eugenio Pérez
The next patches will start control SVQ if possible. However, we don't know if that will be possible at qemu boot anymore. Since the moved checks will be already evaluated at net/ to know if it is ok to shadow CVQ, move them. Signed-off-by: Eugenio Pérez --- hw/virtio/vhost-vdpa.c | 33

[PATCH v6 02/10] vhost: set SVQ device call handler at SVQ start

2022-11-08 Thread Eugenio Pérez
By the end of this series CVQ is shadowed as long as the features support it. Since we don't know at the beginning of qemu running if this is supported, move the event notifier handler setting to the start of the SVQ, instead of the start of qemu run. Signed-off-by: Eugenio Pérez ---

Re: [PATCH v3 03/17] migration: Block migration comment or code is wrong

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/3/22 19:16, Avihai Horon wrote: From: Juan Quintela And it appears that what is wrong is the code. During bulk stage we need to make sure that some block is dirty, but no games with max_size at all. :) That made me interested in, why we need this one block, so I decided to search

Re: [PULL v4 45/83] tests: acpi: whitelist DSDT before generating PCI-ISA bridge AML automatically

2022-11-08 Thread Igor Mammedov
On Tue, 8 Nov 2022 08:49:01 -0500 "Michael S. Tsirkin" wrote: > On Tue, Nov 08, 2022 at 02:36:41PM +0100, Igor Mammedov wrote: > > On Mon, 7 Nov 2022 17:51:11 -0500 > > "Michael S. Tsirkin" wrote: > > > > > From: Igor Mammedov > > > > > > Signed-off-by: Igor Mammedov > > > Message-Id:

Re: [PULL v4 45/83] tests: acpi: whitelist DSDT before generating PCI-ISA bridge AML automatically

2022-11-08 Thread Michael S. Tsirkin
On Tue, Nov 08, 2022 at 02:36:41PM +0100, Igor Mammedov wrote: > On Mon, 7 Nov 2022 17:51:11 -0500 > "Michael S. Tsirkin" wrote: > > > From: Igor Mammedov > > > > Signed-off-by: Igor Mammedov > > Message-Id: <20221017102146.2254096-3-imamm...@redhat.com> > > Reviewed-by: Michael S. Tsirkin >

Re: [PATCH v2 1/9] block: call bdrv_co_drain_begin in a coroutine

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: It seems that bdrv_open_driver() forgot to create a coroutine where to call bs->drv->bdrv_co_drain_begin(), a callback marked as coroutine_fn. Because there is no active I/O at this point, the coroutine should end right after entering, so the

Re: [PULL v4 00/83] pci,pc,virtio: features, tests, fixes, cleanups

2022-11-08 Thread Stefan Hajnoczi
On Tue, Nov 08, 2022 at 01:23:16AM -0500, Michael S. Tsirkin wrote: > On Mon, Nov 07, 2022 at 05:47:16PM -0500, Michael S. Tsirkin wrote: > > Changes from v3: > > Applied and squashed fix by Ani for modular build breakage > > Reordered Julia's patches to avoid bisect breakage > >

Re: [PATCH V2] hw/riscv: virt: Remove size restriction for pflash

2022-11-08 Thread Daniel P . Berrangé
On Tue, Nov 08, 2022 at 03:12:42PM +0100, Philippe Mathieu-Daudé wrote: > On 7/11/22 18:34, Daniel P. Berrangé wrote: > > On Mon, Nov 07, 2022 at 06:32:01PM +0100, Andrew Jones wrote: > > > On Mon, Nov 07, 2022 at 04:19:10PM +, Daniel P. Berrangé wrote: > > > > On Mon, Nov 07, 2022 at

Re: [PATCH v2 1/9] block: call bdrv_co_drain_begin in a coroutine

2022-11-08 Thread Emanuele Giuseppe Esposito
Am 08/11/2022 um 15:33 schrieb Vladimir Sementsov-Ogievskiy: > On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: >> It seems that bdrv_open_driver() forgot to create a coroutine >> where to call bs->drv->bdrv_co_drain_begin(), a callback >> marked as coroutine_fn. >> >> Because there is no

Re: [PATCH] hw/arm/boot: set initrd parameters to 64bit in fdt

2022-11-08 Thread Schspa Shi
Peter Maydell writes: > On Tue, 8 Nov 2022 at 13:54, Peter Maydell wrote: >> >> On Tue, 8 Nov 2022 at 12:52, Schspa Shi wrote: >> > Alex Bennée writes: >> > > There is a whole comment in boot.c talking about keeping initrd within >> > > lowmem: >> > > >> > > /* >> > > * We want to

Re: GTK clipboard implementation causing regression, falling through the cracks?

2022-11-08 Thread Claudio Fontana
On 11/8/22 12:42, Gerd Hoffmann wrote: > On Mon, Oct 24, 2022 at 12:49:19PM +0200, Claudio Fontana wrote: >> Hi all, >> >> the GTK clipboard implementation seems to be causing some stability issues >> (guest CPUs stuck), >> >> Gerd can you take a look? > > Sorry, no. Just back online from sick

Re: [PATCH v2 2/9] block-copy: add missing coroutine_fn annotations

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/8/22 19:19, Vladimir Sementsov-Ogievskiy wrote: This is a lot better than our "coroutine_fn" sign, which actually do no check (and can't do). Don't you plan to swap a "coroutine_fn" noop marker with more meaningful IN_COROUTINE(); (or something like this, which just do

Re: [PATCH v1 5/9] hw/virtio: introduce virtio_device_should_start

2022-11-08 Thread Alex Bennée
"Michael S. Tsirkin" writes: > On Tue, Nov 08, 2022 at 11:21:26AM +, Alex Bennée wrote: >> >> "Michael S. Tsirkin" writes: >> >> > On Tue, Nov 08, 2022 at 10:23:15AM +, Alex Bennée wrote: >> >> >> >> "Michael S. Tsirkin" writes: >> >> >> >> > On Tue, Nov 08, 2022 at 09:23:04AM

[PATCH-for-7.2] Revert "hw/block/pflash_cfi: Error out if dev length isn't power of 2"

2022-11-08 Thread Philippe Mathieu-Daudé
Commit 334c388f25 ("pflash_cfi: Error out if device length isn't a power of two") aimed to finish the effort started by commit 06f1521795 ("pflash: Require backend size to match device, improve errors"), but unfortunately we are not quite there since various machines are still ready to accept

Re: [PATCH v2 2/9] block-copy: add missing coroutine_fn annotations

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: These functions end up calling bdrv_common_block_status_above(), a generated_co_wrapper function. generated_co_wrapper is not a coroutine_fn. Сonversely it's a function that do a class coroutine wrapping - start a coroutine and do POLL to

Re: [PATCH V2] hw/riscv: virt: Remove size restriction for pflash

2022-11-08 Thread Andrew Jones
On Tue, Nov 08, 2022 at 03:12:42PM +0100, Philippe Mathieu-Daudé wrote: > On 7/11/22 18:34, Daniel P. Berrangé wrote: > > On Mon, Nov 07, 2022 at 06:32:01PM +0100, Andrew Jones wrote: > > > On Mon, Nov 07, 2022 at 04:19:10PM +, Daniel P. Berrangé wrote: > > > > On Mon, Nov 07, 2022 at

Re: [PATCH v2 1/9] block: call bdrv_co_drain_begin in a coroutine

2022-11-08 Thread Kevin Wolf
Am 08.11.2022 um 15:33 hat Vladimir Sementsov-Ogievskiy geschrieben: > On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: > > It seems that bdrv_open_driver() forgot to create a coroutine > > where to call bs->drv->bdrv_co_drain_begin(), a callback > > marked as coroutine_fn. > > > > Because

Re: [PATCH v3 4/4] scripts: add script to compare compatible properties

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/3/22 13:27, Maksim Davydov wrote: This script run QEMU to obtain compat_props of machines and default values of different types and produce appropriate table. This table can be used to compare machine types to choose the most suitable machine. Also this table in json or csv format should

Re: [PATCH V2] hw/riscv: virt: Remove size restriction for pflash

2022-11-08 Thread Markus Armbruster
Peter Maydell writes: > On Mon, 7 Nov 2022 at 14:08, Sunil V L wrote: >> >> On Mon, Nov 07, 2022 at 01:06:38PM +, Peter Maydell wrote: >> > On Mon, 7 Nov 2022 at 13:03, Sunil V L wrote: >> > > >> > > The pflash implementation currently assumes fixed size of the >> > > backend storage. Due

QOM: should you be able to cast from an interface class to the concrete class?

2022-11-08 Thread Peter Maydell
Hi; in the QOM model, are you supposed to be able to cast from an interface class to the concrete class that is implementing it? To give a specific example, if I have a ResettableClass *rc should I be able to do DeviceClass *dc = DEVICE_CLASS(rc); (assuming that the rc I have is actually from a

Re: [PATCH v2 2/9] block-copy: add missing coroutine_fn annotations

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
[add Stefan] On 11/8/22 18:09, Emanuele Giuseppe Esposito wrote: Am 08/11/2022 um 15:48 schrieb Vladimir Sementsov-Ogievskiy: On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: These functions end up calling bdrv_common_block_status_above(), a generated_co_wrapper function.

Re: [PULL 00/55] MIPS patches for 2022-10-30

2022-11-08 Thread Konstantin Kostiuk
On Tue, Nov 8, 2022 at 5:23 PM Philippe Mathieu-Daudé wrote: > On 8/11/22 16:09, Thomas Huth wrote: > > On 08/11/2022 15.23, Philippe Mathieu-Daudé wrote: > >> On 8/11/22 14:59, Peter Maydell wrote: > > >>> Was this the last use of C++ in the tree, or am I forgetting > >>> some other part that

[PULL 1/3] memory: Fix wrong end address dump

2022-11-08 Thread Philippe Mathieu-Daudé
From: Zhenzhong Duan The end address of memory region section isn't correctly calculated which leads to overflowed mtree dump: Dispatch Physical sections .. #70 @2000..00011fff io [ROOT] #71 @5000..5fff (noname) #72

[PULL 2/3] hw/sd/sdhci: Do not set Buf Wr Ena before writing block (CVE-2022-3872)

2022-11-08 Thread Philippe Mathieu-Daudé
When sdhci_write_block_to_card() is called to transfer data from the FIFO to the SD bus, the data is already present in the buffer and we have to consume it directly. See the description of the 'Buffer Write Enable' bit from the 'Present State' register (prnsts::SDHC_SPACE_AVAILABLE) in Table

[PULL 0/3] Memory/SDHCI/ParallelFlash patches for v7.2.0-rc0

2022-11-08 Thread Philippe Mathieu-Daudé
The following changes since commit ade760a2f63804b7ab1839fbc3e5ddbf30538718: Merge tag 'pull-request-2022-11-08' of https://gitlab.com/thuth/qemu into staging (2022-11-08 11:34:06 -0500) are available in the Git repository at: https://github.com/philmd/qemu.git tags/memflash-20221108

Re: [PULL v4 45/83] tests: acpi: whitelist DSDT before generating PCI-ISA bridge AML automatically

2022-11-08 Thread Ani Sinha
On Tue, Nov 8, 2022 at 7:09 PM Ani Sinha wrote: > > On Tue, Nov 8, 2022 at 7:06 PM Igor Mammedov wrote: > > > > On Mon, 7 Nov 2022 17:51:11 -0500 > > "Michael S. Tsirkin" wrote: > > > > > From: Igor Mammedov > > > > > > Signed-off-by: Igor Mammedov > > > Message-Id:

Re: [PULL 00/55] MIPS patches for 2022-10-30

2022-11-08 Thread Thomas Huth
On 08/11/2022 15.23, Philippe Mathieu-Daudé wrote: On 8/11/22 14:59, Peter Maydell wrote: On Sun, 30 Oct 2022 at 22:29, Philippe Mathieu-Daudé wrote: The following changes since commit 344744e148e6e865f5a57e745b02a87e5ea534ad:    Merge tag 'dump-pull-request' of

Re: [PATCH v2 2/9] block-copy: add missing coroutine_fn annotations

2022-11-08 Thread Emanuele Giuseppe Esposito
Am 08/11/2022 um 15:48 schrieb Vladimir Sementsov-Ogievskiy: > On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: >> These functions end up calling bdrv_common_block_status_above(), a >> generated_co_wrapper function. > > generated_co_wrapper is not a coroutine_fn. Сonversely it's a function

[PATCH v5 10/11] Hexagon (target/hexagon) Use direct block chaining for direct jump/branch

2022-11-08 Thread Taylor Simpson
Direct block chaining is documented here https://qemu.readthedocs.io/en/latest/devel/tcg.html#direct-block-chaining Recall that Hexagon allows packets with multiple jumps where only the first one with a true predicate will actually jump. We can use tcg_gen_goto_tb/tcg_gen_exit_tb when the

[PATCH v5 09/11] Hexagon (target/hexagon) Add overrides for various forms of jump

2022-11-08 Thread Taylor Simpson
Reviewed-by: Richard Henderson Signed-off-by: Taylor Simpson --- target/hexagon/gen_tcg.h | 203 +++ target/hexagon/genptr.c | 43 + 2 files changed, 246 insertions(+) diff --git a/target/hexagon/gen_tcg.h b/target/hexagon/gen_tcg.h index

[PATCH v5 08/11] Hexagon (target/hexagon) Add overrides for compound compare and jump

2022-11-08 Thread Taylor Simpson
Acked-by: Richard Henderson Signed-off-by: Taylor Simpson --- target/hexagon/gen_tcg.h | 177 +++ target/hexagon/genptr.c | 90 2 files changed, 267 insertions(+) diff --git a/target/hexagon/gen_tcg.h b/target/hexagon/gen_tcg.h index

Re: [PATCH v3 02/17] migration: No save_live_pending() method uses the QEMUFile parameter

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/3/22 19:16, Avihai Horon wrote: From: Juan Quintela So remove it everywhere. Signed-off-by: Juan Quintela Reviewed-by: Vladimir Sementsov-Ogievskiy -- Best regards, Vladimir

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

2022-11-08 Thread Stefan Hajnoczi
On Tue, 8 Nov 2022 at 13:10, Philippe Mathieu-Daudé wrote: > > On 8/11/22 18:26, Daniel Henrique Barboza wrote: > > This commit caused a regression [1] that prevents machines that uses > > Open Virtual Machine Firmware (OVMF) to boot. > > > > This is a long standing behavior with how pflash

Re: [PATCH-for-7.2] Revert "hw/block/pflash_cfi: Error out if dev length isn't power of 2"

2022-11-08 Thread Stefan Hajnoczi
Applied to staging. Thanks! Stefan

Re: [PULL 00/14] MIPS patches for 2022-11-08

2022-11-08 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 0/5] s390x fix and white space cleanup

2022-11-08 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] hw/arm/boot: set initrd parameters to 64bit in fdt

2022-11-08 Thread Peter Maydell
On Tue, 8 Nov 2022 at 13:54, Peter Maydell wrote: > > On Tue, 8 Nov 2022 at 12:52, Schspa Shi wrote: > > Alex Bennée writes: > > > There is a whole comment in boot.c talking about keeping initrd within > > > lowmem: > > > > > > /* > > > * We want to put the initrd far enough into RAM

Re: [QEMU][PATCH v2 4/5] tests/qtest: Introduce tests for Xilinx VERSAL CANFD controller

2022-11-08 Thread Francisco Iglesias
On [2022 Oct 21] Fri 22:47:45, Vikram Garhwal wrote: > The QTests perform three tests on the Xilinx VERSAL CANFD controller: > Tests the CANFD controllers in loopback. > Tests the CANFD controllers in normal mode with CAN frame. > Tests the CANFD controllers in normal mode with CANFD

Re: [PATCH v2 0/3] block: Start/end drain on correct AioContext

2022-11-08 Thread Kevin Wolf
Am 08.11.2022 um 15:13 hat Kevin Wolf geschrieben: > Am 07.11.2022 um 16:13 hat Hanna Reitz geschrieben: > > Hi, > > > > v1 cover letter: > > https://lists.nongnu.org/archive/html/qemu-block/2022-09/msg00389.html > > > > bdrv_replace_child_noperm() drains the child via > >

Re: [PULL 00/55] MIPS patches for 2022-10-30

2022-11-08 Thread Philippe Mathieu-Daudé
On 8/11/22 14:59, Peter Maydell wrote: On Sun, 30 Oct 2022 at 22:29, Philippe Mathieu-Daudé wrote: The following changes since commit 344744e148e6e865f5a57e745b02a87e5ea534ad: Merge tag 'dump-pull-request' of https://gitlab.com/marcandre.lureau/qemu into staging (2022-10-26 10:53:49

Re: [PULL v4 45/83] tests: acpi: whitelist DSDT before generating PCI-ISA bridge AML automatically

2022-11-08 Thread Ani Sinha
On Tue, Nov 8, 2022 at 7:06 PM Igor Mammedov wrote: > > On Mon, 7 Nov 2022 17:51:11 -0500 > "Michael S. Tsirkin" wrote: > > > From: Igor Mammedov > > > > Signed-off-by: Igor Mammedov > > Message-Id: <20221017102146.2254096-3-imamm...@redhat.com> > > Reviewed-by: Michael S. Tsirkin > >

Re: [PATCH v2 3/9] nbd/server.c: add missing coroutine_fn annotations

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: These functions end up calling bdrv_*() implemented as generated_co_wrapper functions. Same here. Sorry that I joined only on v3. In past we had a lot of "coroutine wrappers", each IO function in block/io.c and many in block.c had two

Re: [PATCH v2 8/9] block: bdrv_create is never called in non-coroutine context

2022-11-08 Thread Kevin Wolf
Am 04.11.2022 um 10:56 hat Emanuele Giuseppe Esposito geschrieben: > Delete the if case and make sure it won't be called again > in coroutines. > > Signed-off-by: Emanuele Giuseppe Esposito > Reviewed-by: Paolo Bonzini In the subject line, it should be "never called in coroutine context"

Re: [PATCH v1 5/9] hw/virtio: introduce virtio_device_should_start

2022-11-08 Thread Michael S. Tsirkin
On Tue, Nov 08, 2022 at 11:21:26AM +, Alex Bennée wrote: > > "Michael S. Tsirkin" writes: > > > On Tue, Nov 08, 2022 at 10:23:15AM +, Alex Bennée wrote: > >> > >> "Michael S. Tsirkin" writes: > >> > >> > On Tue, Nov 08, 2022 at 09:23:04AM +, Alex Bennée wrote: > >> >> The

Re: [PATCH v2 1/9] block: call bdrv_co_drain_begin in a coroutine

2022-11-08 Thread Emanuele Giuseppe Esposito
Am 08/11/2022 um 16:52 schrieb Vladimir Sementsov-Ogievskiy: > On 11/8/22 18:13, Emanuele Giuseppe Esposito wrote: >> >> >> Am 08/11/2022 um 15:33 schrieb Vladimir Sementsov-Ogievskiy: >>> On 11/4/22 12:56, Emanuele Giuseppe Esposito wrote: It seems that bdrv_open_driver() forgot to create

[PATCH v5 02/11] Hexagon (target/hexagon) Fix predicated assignment to .tmp and .cur

2022-11-08 Thread Taylor Simpson
Here are example instructions with a predicated .tmp/.cur assignment if (p1) v12.tmp = vmem(r7 + #0) if (p0) v12.cur = vmem(r9 + #0) The .tmp/.cur indicates that references to v12 in the same packet take the result of the load. However, when the predicate is false, the value at the start

[PATCH v5 03/11] Hexagon (target/hexagon) Add overrides for S2_asr_r_r_sat/S2_asl_r_r_sat

2022-11-08 Thread Taylor Simpson
These instructions will not be generated by idef-parser, so we override them manually. Test cases added to tests/tcg/hexagon/usr.c Co-authored-by: Matheus Tavares Bernardino Signed-off-by: Matheus Tavares Bernardino Signed-off-by: Taylor Simpson --- target/hexagon/gen_tcg.h | 10 +++-

Re: QOM: should you be able to cast from an interface class to the concrete class?

2022-11-08 Thread Daniel P . Berrangé
On Tue, Nov 08, 2022 at 04:01:56PM +, Peter Maydell wrote: > Hi; in the QOM model, are you supposed to be able to cast from > an interface class to the concrete class that is implementing it? > > To give a specific example, if I have a ResettableClass *rc > should I be able to do DeviceClass

Re: [PATCH v3 01/17] migration: Remove res_compatible parameter

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/3/22 19:16, Avihai Horon wrote: From: Juan Quintela It was only used for RAM, and in that case, it means that this amount of data was sent for memory. Not clear for me, what means "this amount of data was sent for memory"... That amount of data was not yet sent, actually. Just

[PULL 3/3] Revert "hw/block/pflash_cfi: Error out if dev length isn't power of 2"

2022-11-08 Thread Philippe Mathieu-Daudé
From: Daniel Henrique Barboza Commit 334c388f25 ("pflash_cfi: Error out if device length isn't a power of two") aimed to finish the effort started by commit 06f1521795 ("pflash: Require backend size to match device, improve errors"), but unfortunately we are not quite there since various

Re: [PATCH-for-7.2 1/2] hw/sd/sdhci: Do not set Buf Wr Ena before writing block (CVE-2022-3872)

2022-11-08 Thread Alexander Bulekov
On 221108 1225, Alexander Bulekov wrote: > On 221107 2312, Philippe Mathieu-Daudé wrote: > > When sdhci_write_block_to_card() is called to transfer data from > > the FIFO to the SD bus, the data is already present in the buffer > > and we have to consume it directly. > > > > See the description

Re: [PULL 00/55] MIPS patches for 2022-10-30

2022-11-08 Thread Philippe Mathieu-Daudé
On 8/11/22 16:09, Thomas Huth wrote: On 08/11/2022 15.23, Philippe Mathieu-Daudé wrote: On 8/11/22 14:59, Peter Maydell wrote: Was this the last use of C++ in the tree, or am I forgetting some other part that still needs the C++ compiler? If it is the last thing, we should put in the "Build

[PATCH v5 11/11] Hexagon (target/hexagon) Use direct block chaining for tight loops

2022-11-08 Thread Taylor Simpson
Direct block chaining is documented here https://qemu.readthedocs.io/en/latest/devel/tcg.html#direct-block-chaining Hexagon inner loops end with the endloop0 instruction To go back to the beginning of the loop, this instructions writes to PC from register SA0 (start address 0). To use direct

[PATCH] block: m25p80: fix dummy byte count read from spansion cfg register

2022-11-08 Thread Ramon Aerne
Spansion nor-flash stores the dummy read count in bits in a config register. This is currently read and used as the byte count which is wrong. This patch fixes this bit to byte conversion without warning about unsupported configurations (such as bits % 8 != 0) Signed-off-by: Ramon Aerne ---

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

2022-11-08 Thread Philippe Mathieu-Daudé
On 8/11/22 18:26, Daniel Henrique Barboza wrote: This commit caused a regression [1] that prevents machines that uses Open Virtual Machine Firmware (OVMF) to boot. This is a long standing behavior with how pflash handles images. More information about why this happens can be found in [2] and

Re: [PATCH-for-7.2 0/2] hw/sd/sdhci: Do not set Buf Wr Ena before writing block (CVE-2022-3872)

2022-11-08 Thread Stefan Hajnoczi
Applied to the staging tree. Thanks! Stefan

Re: [PULL 0/2] Net patches

2022-11-08 Thread Stefan Hajnoczi
On Mon, 7 Nov 2022 at 23:20, Jason Wang wrote: > Si-Wei Liu (1): > vhost-vdpa: fix assert !virtio_net_get_subqueue(nc)->async_tx.elem in > virtio_net_reset I have applied just this patch to the staging tree. Thanks, Stefan

Re: [PATCH v3 03/17] migration: Block migration comment or code is wrong

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/8/22 21:36, Vladimir Sementsov-Ogievskiy wrote: On 11/3/22 19:16, Avihai Horon wrote: From: Juan Quintela And it appears that what is wrong is the code. During bulk stage we need to make sure that some block is dirty, but no games with max_size at all. :) That made me interested in,

Re: [PATCH v3 04/17] migration: Simplify migration_iteration_run()

2022-11-08 Thread Vladimir Sementsov-Ogievskiy
On 11/3/22 19:16, Avihai Horon wrote: From: Juan Quintela Signed-off-by: Juan Quintela Signed-off-by: Avihai Horon --- migration/migration.c | 25 + 1 file changed, 13 insertions(+), 12 deletions(-) diff --git a/migration/migration.c b/migration/migration.c index

Re: [PATCH v3 4/4] hw/nvme: add polling support

2022-11-08 Thread Klaus Jensen
On Nov 8 12:39, John Levon wrote: > On Fri, Nov 04, 2022 at 07:32:12AM +0100, Klaus Jensen wrote: > > > On Nov 3 21:19, Jinhao Fan wrote: > > > On 11/3/2022 8:10 PM, Klaus Jensen wrote: > > > > I agree that the spec is a little unclear on this point. In any case, in > > > > Linux, when the

RE: [PATCH v4 10/11] Hexagon (target/hexagon) Use direct block chaining for direct jump/branch

2022-11-08 Thread Taylor Simpson
> -Original Message- > From: Richard Henderson > Sent: Tuesday, November 8, 2022 1:24 AM > To: Taylor Simpson ; qemu-devel@nongnu.org > Cc: phi...@linaro.org; a...@rev.ng; a...@rev.ng; Brian Cain > ; Matheus Bernardino (QUIC) > > Subject: Re: [PATCH v4 10/11] Hexagon (target/hexagon)

RE: [PATCH v4 03/11] Hexagon (target/hexagon) Add overrides for S2_asr_r_r_sat/S2_asl_r_r_sat

2022-11-08 Thread Taylor Simpson
> -Original Message- > From: Richard Henderson > Sent: Tuesday, November 8, 2022 1:20 AM > To: Taylor Simpson ; qemu-devel@nongnu.org > Cc: phi...@linaro.org; a...@rev.ng; a...@rev.ng; Brian Cain > ; Matheus Bernardino (QUIC) > > Subject: Re: [PATCH v4 03/11] Hexagon (target/hexagon)

[PATCH] gtk: disable GTK Clipboard with a new option 'gtk_clipboard'

2022-11-08 Thread Claudio Fontana
The GTK Clipboard implementation may cause guest hangs. Therefore implement a new configure switch --enable-gtk-clipboard, disabled by default, as a meson option. Regenerate the meson build options to include it. The initialization of the clipboard is gtk.c, as well as the compilation of

[PATCH v5 07/11] Hexagon (target/hexagon) Add overrides for direct call instructions

2022-11-08 Thread Taylor Simpson
Add overrides for J2_call J2_callt J2_callf Reviewed-by: Richard Henderson Signed-off-by: Taylor Simpson --- target/hexagon/gen_tcg.h | 8 ++ target/hexagon/genptr.c | 55 2 files changed, 63 insertions(+) diff --git

[PATCH] Revert "hw/block/pflash_cfi0{1, 2}: Error out if device length isn't a power of two"

2022-11-08 Thread Daniel Henrique Barboza
This commit caused a regression [1] that prevents machines that uses Open Virtual Machine Firmware (OVMF) to boot. This is a long standing behavior with how pflash handles images. More information about why this happens can be found in [2] and commit 06f1521795 ("pflash: Require backend size to

Re: [PATCH-for-7.2 1/2] hw/sd/sdhci: Do not set Buf Wr Ena before writing block (CVE-2022-3872)

2022-11-08 Thread Alexander Bulekov
On 221107 2312, Philippe Mathieu-Daudé wrote: > When sdhci_write_block_to_card() is called to transfer data from > the FIFO to the SD bus, the data is already present in the buffer > and we have to consume it directly. > > See the description of the 'Buffer Write Enable' bit from the > 'Present

Re: [PULL v3 49/81] acpi: pc: vga: use AcpiDevAmlIf interface to build VGA device descriptors

2022-11-08 Thread B
Am 7. November 2022 22:28:31 UTC schrieb "Michael S. Tsirkin" : >On Mon, Nov 07, 2022 at 10:07:52PM +, Bernhard Beschow wrote: >> Am 7. November 2022 13:00:36 UTC schrieb "Michael S. Tsirkin" >> : >> >On Mon, Nov 07, 2022 at 06:16:25PM +0530, Ani Sinha wrote: >> >> On Mon, Nov 7, 2022 at

Re: [PATCH trivial for 7.2] hw/ssi/sifive_spi.c: spelling: reigster

2022-11-08 Thread Palmer Dabbelt
On Sat, 05 Nov 2022 04:53:29 PDT (-0700), m...@tls.msk.ru wrote: Fixes: 0694dabe9763847f3010b54ab3ec7d367d2f0ff0 Not sure if I missed something in QEMU land, but those are usually listed more like Fixes: 0694dabe97 ("hw/ssi: Add SiFive SPI controller support") Checkpatch isn't failing,

[PATCH v1 15/24] vfio-user: forward msix BAR accesses to server

2022-11-08 Thread John Johnson
Server holds device current device pending state Use irq masking commands in socket case Signed-off-by: John G Johnson Signed-off-by: Elena Ufimtseva Signed-off-by: Jagannathan Raman --- hw/vfio/ccw.c | 1 + hw/vfio/common.c | 26 + hw/vfio/pci.c

  1   2   3   >