Re: [PATCH v6 4/4] qapi: rephrase return docs to avoid type name

2025-07-13 Thread Markus Armbruster
Markus Armbruster writes: > John Snow writes: > >> Well, I tried. Maybe not very hard. Sorry! > > Recommend to explain *why* we want to avoid the type name. > > "Returns: " is rendered like "Return: – > ". Mentioning the type in the description again is > commonly redundant. Rephrase su

Re: [PATCH v7 4/6] target/arm: Add FEAT_TCR2

2025-07-13 Thread Pierrick Bouvier
On 7/13/25 2:59 PM, Richard Henderson wrote: On 7/11/25 08:08, Gustavo Romero wrote: Add FEAT_TCR2, which introduces the TCR2_EL1 and TCR2_EL2 registers. These registers are extensions of the TCR_ELx registers and provide top-level control of the EL10 and EL20 translation regimes. Since the bit

Re: [PATCH v7 3/6] target/arm: Add FEAT_SCTLR2

2025-07-13 Thread Pierrick Bouvier
On 7/13/25 9:27 AM, Richard Henderson wrote: On 7/11/25 23:02, Richard Henderson wrote: On 7/11/25 08:08, Gustavo Romero wrote: Add FEAT_SCTLR2, which introduces the SCTLR2_EL1, SCTLR2_EL2, and SCTLR2_EL3 registers. These registers are extensions of the SCTLR_ELx ones. Because the bits in thes

[PULL 10/13] net: Allow network backends to advertise max TX queue size

2025-07-13 Thread Jason Wang
From: Laurent Vivier This commit refactors how the maximum transmit queue size for virtio-net devices is determined, making the mechanism more generic and extensible. Previously, virtio_net_max_tx_queue_size() contained hardcoded checks for specific network backend types (vhost-user and vhost-vd

[PULL 07/13] net: Consolidate vhost feature bits into vhost_net structure

2025-07-13 Thread Jason Wang
From: Laurent Vivier Previously, the vhost_net_get_feature_bits() function in hw/net/vhost_net.c used a large switch statement to determine the appropriate feature bits based on the NetClientDriver type. This created unnecessary coupling between the generic vhost layer and specific network backe

[PULL 08/13] net: Add get_acked_features callback to VhostNetOptions

2025-07-13 Thread Jason Wang
From: Laurent Vivier This patch continues the effort to decouple the generic vhost layer from specific network backend implementations. Previously, the vhost_net initialization code contained a hardcoded check for the vhost-user client type to retrieve its acked features by calling vhost_user_ge

[PULL 03/13] net: Refactor stream logic for reuse in '-net passt'

2025-07-13 Thread Jason Wang
From: Laurent Vivier To prepare for the implementation of '-net passt', this patch moves the generic stream handling functions from net/stream.c into new net/stream_data.c and net/stream_data.h files. This refactoring introduces a NetStreamData struct that encapsulates the generic fields and log

[PULL 12/13] net: Add passt network backend

2025-07-13 Thread Jason Wang
From: Laurent Vivier This commit introduces support for passt as a new network backend. passt is an unprivileged, user-mode networking solution that provides connectivity for virtual machines by launching an external helper process. The implementation reuses the generic stream data handling logi

[PULL 11/13] net: Add is_vhost_user flag to vhost_net struct

2025-07-13 Thread Jason Wang
From: Laurent Vivier Introduce a boolean is_vhost_user field to the vhost_net structure. This flag is initialized during vhost_net_init based on whether the backend is vhost-user. This refactoring simplifies checks for vhost-user specific behavior, replacing direct comparisons of 'net->nc->info-

[PULL 00/13] Net patches

2025-07-13 Thread Jason Wang
The following changes since commit 9a4e273ddec3927920c5958d2226c6b38b543336: Merge tag 'pull-tcg-20250711' of https://gitlab.com/rth7680/qemu into staging (2025-07-13 01:46:04 -0400) are available in the Git repository at: https://github.com/jasowang/qemu.git tags/net-pull-request for you

[PULL 09/13] net: Add save_acked_features callback to vhost_net

2025-07-13 Thread Jason Wang
From: Laurent Vivier This commit introduces a save_acked_features function pointer to vhost_net and converts the vhost_net function into a generic dispatcher. The vhost-user backend provides the callback, making its function static. With this change, no other module has a direct dependency on th

[PULL 06/13] net: Add get_vhost_net callback to NetClientInfo

2025-07-13 Thread Jason Wang
From: Laurent Vivier The get_vhost_net() function previously contained a large switch statement to find the VHostNetState pointer based on the net client's type. This created a tight coupling, requiring the generic vhost layer to be aware of every specific backend that supported vhost, such as ta

[PULL 13/13] net/passt: Implement vhost-user backend support

2025-07-13 Thread Jason Wang
From: Laurent Vivier This commit adds support for the vhost-user interface to the passt network backend, enabling high-performance, accelerated networking for guests using passt. The passt backend can now operate in a vhost-user mode, where it communicates with the guest's virtio-net device over

[PULL 01/13] net: fix buffer overflow in af_xdp_umem_create()

2025-07-13 Thread Jason Wang
From: Anastasia Belova s->pool has n_descs elements so maximum i should be n_descs - 1. Fix the upper bound. Found by Linux Verification Center (linuxtesting.org) with SVACE. Fixes: cb039ef3d9 ("net: add initial support for AF_XDP network backend") Cc: qemu-sta...@nongnu.org Reviewed-by: Ilya M

[PULL 04/13] net: Define net_client_set_link()

2025-07-13 Thread Jason Wang
From: Laurent Vivier The code to set the link status is currently located in qmp_set_link(). This function identifies the device by name, searches for the corresponding NetClientState, and then updates the link status. In some parts of the code, such as vhost-user.c, the NetClientState are alrea

[PULL 02/13] virtio-net: Add queues for RSS during migration

2025-07-13 Thread Jason Wang
From: Akihiko Odaki virtio_net_pre_load_queues() inspects vdev->guest_features to tell if VIRTIO_NET_F_RSS or VIRTIO_NET_F_MQ is enabled to infer the required number of queues. This works for VIRTIO_NET_F_MQ but it doesn't for VIRTIO_NET_F_RSS because only the lowest 32 bits of vdev->guest_featur

[PULL 05/13] vhost_net: Rename vhost_set_vring_enable() for clarity

2025-07-13 Thread Jason Wang
From: Laurent Vivier This is a cosmetic change with no functional impact. The function vhost_set_vring_enable() is specific to vhost_net and is used outside of vhost_net.c (specifically, in hw/net/virtio-net.c). To prevent confusion with other similarly named vhost functions, such as the one fou

Re: [PATCH] vfio, migration: save device parent pci config

2025-07-13 Thread Huaitong Han
Hi, mst, clg Any comments? Thanks. Huaitong Han Michael Tokarev 于2025年7月13日周日 17:12写道: > > Ping? Has this been forgotten? > > Thanks, > > /mjt > > On 05.06.2025 11:33, oen...@gmail.com wrote: > > From: Huguanshen Chen > > > > On arm64 virtualization platform, vfio-user devices lose their int

Re: [PATCH] target/loongarch: Fix valid virtual address checking

2025-07-13 Thread gaosong
在 2025/7/14 上午9:54, Bibo Mao 写道: On LoongArch64 system, the high 32 bit of 64 bit virtual address should be 0x0[0-7]yyy or 0x8yyy. The bit from 47 to 63 should be all 0 or all 1. Function get_physical_address() only checks bit 48 to 63, there will be problem with the following test case.

Re: [PATCH v2 03/18] i386/cpu: Add default cache model for Intel CPUs with level < 4

2025-07-13 Thread Mi, Dapeng
On 7/11/2025 6:21 PM, Zhao Liu wrote: > Old Intel CPUs with CPUID level < 4, use CPUID 0x2 leaf (if available) > to encode cache information. > > Introduce a cache model "legacy_intel_cpuid2_cache_info" for the CPUs > with CPUID level < 4, based on legacy_l1d_cache, legacy_l1i_cache, > legacy_l2_

Re: [PATCH] net/tap: drop too small packets

2025-07-13 Thread Jason Wang
On Wed, Jul 9, 2025 at 10:43 PM Vladimir Sementsov-Ogievskiy wrote: > > On 07.07.25 06:49, Jason Wang wrote: > > On Thu, Jul 3, 2025 at 10:59 PM Daniil Tatianin > > wrote: > >> > >> On 7/3/25 1:55 PM, Vladimir Sementsov-Ogievskiy wrote: > >> > >>> Theoretically tap_read_packet() may return size l

[PATCH] target/loongarch: Fix valid virtual address checking

2025-07-13 Thread Bibo Mao
On LoongArch64 system, the high 32 bit of 64 bit virtual address should be 0x0[0-7]yyy or 0x8yyy. The bit from 47 to 63 should be all 0 or all 1. Function get_physical_address() only checks bit 48 to 63, there will be problem with the following test case. On physical machine, there is bus

Re: [PATCH v10 (RESEND) 00/20] Change ghes to use HEST-based offsets and add support for error inject

2025-07-13 Thread Gavin Shan
Hi Mauro, On 6/13/25 1:17 AM, Mauro Carvalho Chehab wrote: Hi Michael, This is v10 of the patch series, rebased to apply after release 10.0. The only difference against v9 is a minor confict resolution. I sent already the patch with conflicts, but, as you didn't pick, I'm assuming you're optin

Re: [PATCH v7 4/6] target/arm: Add FEAT_TCR2

2025-07-13 Thread Richard Henderson
On 7/11/25 08:08, Gustavo Romero wrote: Add FEAT_TCR2, which introduces the TCR2_EL1 and TCR2_EL2 registers. These registers are extensions of the TCR_ELx registers and provide top-level control of the EL10 and EL20 translation regimes. Since the bits in these registers depend on other CPU featu

Re: [PATCH RFC] virtio-net: Fix VLAN filter table initialization timing

2025-07-13 Thread Konstantin Shkolnyy
On 13-Jul-25 06:52, Akihiko Odaki wrote: Konstantin, please see if this patch fixes your workload. Yes, it does. It delays vlans[] initialization until virtio_net_set_status(0xf) which later also calls vhost_net_start() which is where the NIC is actually programmed.

Re: [PATCH] intel_iommu: Allow both Status Write and Interrupt Flag in QI wait

2025-07-13 Thread Michael S. Tsirkin
On Fri, Jul 11, 2025 at 10:47:47AM +0100, David Woodhouse wrote: > From: David Woodhouse > > FreeBSD does both, and this appears to be perfectly valid. The VT-d > spec even talks about the ordering (the status write should be done > first, unsurprisingly). > > We certainly shouldn't assert() and

Re: [PATCH v14 5/7] hw/acpi: add cache hierarchy to pptt table

2025-07-13 Thread Michael S. Tsirkin
On Mon, Jul 07, 2025 at 01:19:06PM +0100, Alireza Sanaee wrote: > Add cache topology to PPTT table. With this patch, both ACPI PPTT table > and device tree will represent the same cache topology given users > input. > > Co-developed-by: Jonathan Cameron > Signed-off-by: Alireza Sanaee > --- > h

Re: [PATCH v9 2/2] hw/i386: Add the ramfb romfile compatibility

2025-07-13 Thread Michael S. Tsirkin
On Thu, Jul 03, 2025 at 11:03:15PM -0400, Shaoqin Huang wrote: > ramfb is a sysbus device so it can only used for machine types where it > is explicitly enabled: > > # git grep machine_class_allow_dynamic_sysbus_dev.*TYPE_RAMFB_DEVICE > hw/arm/virt.c:machine_class_allow_dynamic_sysbus_dev(

Re: [PATCH] vhost-user: enable asym capabilities for user backend

2025-07-13 Thread Michael S. Tsirkin
On Thu, Jun 26, 2025 at 12:24:51PM +, Rajesh Mudimadugula wrote: > This patch enables and advertises asym capabilties > for backend user application. > > Signed-off-by: Rajesh Mudimadugula You want to CC all maintainers really. The commit log also should include motivation for each change, n

Re: [PATCH v7 3/6] target/arm: Add FEAT_SCTLR2

2025-07-13 Thread Richard Henderson
On 7/11/25 23:02, Richard Henderson wrote: On 7/11/25 08:08, Gustavo Romero wrote: Add FEAT_SCTLR2, which introduces the SCTLR2_EL1, SCTLR2_EL2, and SCTLR2_EL3 registers. These registers are extensions of the SCTLR_ELx ones. Because the bits in these registers depend on other CPU features, and

[PATCH] hvf: arm: Remove $pc from trace_hvf_data_abort()

2025-07-13 Thread Zenghui Yu
We don't synchronize vcpu registers from the hardware accelerator (e.g., by cpu_synchronize_state()) in the Dabort handler, so env->pc points to the instruction which has nothing to do with the Dabort at all. And it doesn't seem to make much sense to log PC in every Dabort handler, let's just remo

Re: [PATCH RFC] virtio-net: Fix VLAN filter table initialization timing

2025-07-13 Thread Michael S. Tsirkin
hidden, false); > @@ -3990,7 +3992,6 @@ static void virtio_net_reset(VirtIODevice *vdev) > memset(n->mac_table.macs, 0, MAC_TABLE_ENTRIES * ETH_ALEN); > memcpy(&n->mac[0], &n->nic->conf->macaddr, sizeof(n->mac)); > qemu_format_nic_info_str(qemu_get_queue(n->nic), n->mac); > -memset(n->vlans, 0, MAX_VLAN >> 3); > > /* Flush any async TX */ > for (i = 0; i < n->max_queue_pairs; i++) { > > --- > base-commit: f0737158b483e7ec2b2512145aeab888b85cc1f7 > change-id: 20250713-vlan-8c107a65ad91 > > Best regards, > -- > Akihiko Odaki

[PATCH RFC] virtio-net: Fix VLAN filter table initialization timing

2025-07-13 Thread Akihiko Odaki
fo_str(qemu_get_queue(n->nic), n->mac); -memset(n->vlans, 0, MAX_VLAN >> 3); /* Flush any async TX */ for (i = 0; i < n->max_queue_pairs; i++) { --- base-commit: f0737158b483e7ec2b2512145aeab888b85cc1f7 change-id: 20250713-vlan-8c107a65ad91 Best regards, -- Akihiko Odaki

Re: [PATCH 3/3] hw/ppc: Add stub for pnv_chip_find_core()

2025-07-13 Thread Michael Tokarev
Hi! Has this change been forgotten? Thanks, /mjt On 26.05.2025 14:51, Philippe Mathieu-Daudé wrote: On 26/5/25 13:48, Cédric Le Goater wrote: On 5/26/25 13:23, Philippe Mathieu-Daudé wrote: Since commit 9808ce6d5cb, building QEMU configured with '--without-default-devices' fails:    Undefi

[PATCH] pcie_sriov: Fix configuration and state synchronization

2025-07-13 Thread Akihiko Odaki
Fix issues in PCIe SR-IOV configuration register handling that caused inconsistent internal state due to improper write mask handling and incorrect migration behavior. Two main problems were identified: 1. VF Enable bit write mask handling: pcie_sriov_config_write() incorrectly assumed that it

Re: [PATCH] vfio, migration: save device parent pci config

2025-07-13 Thread Michael Tokarev
Ping? Has this been forgotten? Thanks, /mjt On 05.06.2025 11:33, oen...@gmail.com wrote: From: Huguanshen Chen On arm64 virtualization platform, vfio-user devices lose their interrupts after migration to the destination. This issue occurs because qemu fails to deliver the msi device id to t

[PULL 3/6] accel/kvm: Adjust the note about the minimum required kernel version

2025-07-13 Thread Michael Tokarev
From: Thomas Huth Since commit 126e7f78036 ("kvm: require KVM_CAP_IOEVENTFD and KVM_CAP_IOEVENTFD_ANY_LENGTH") we require at least kernel 4.5 to be able to use KVM. Adjust the upgrade_note accordingly. While we're at it, remove the text about kvm-kmod and the SourceForge URL since this is not act

[PULL 4/6] docs/system/target-i386: Remove the sentence about RHEL 7 being supported

2025-07-13 Thread Michael Tokarev
From: Thomas Huth According to our "Supported build platforms" policy, RHEL 7 is not supported anymore, so let's remove the related sentence from the x86 documentation. Signed-off-by: Thomas Huth Reviewed-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé Reviewed-by: Zhao Liu Reviewed-by:

[PULL 6/6] docs/devel/tracing: Update trace.h creation rune to include SPDX

2025-07-13 Thread Michael Tokarev
From: Peter Maydell checkpatch now checks that new files have an SPDX line. If you use the shell rune in tracing.rst to create a trace.h wrapper header, this triggers checkpatch to complain. Although these files are tiny, it's worth having the SPDX line to avoid having to add extra exception cas

[PULL 5/6] hw/uefi: Create and use trace.h wrapper header

2025-07-13 Thread Michael Tokarev
From: Peter Maydell The documentation of the trace subsystem (docs/devel/tracing.rst) says that each subdirectory which uses trace events should create a wrapper trace.h file which includes the trace/trace-foo.h generated header, and that .c files then #include "trace.h". We didn't follow this p

[PULL 1/6] hw/usb/dev-hid: Support side and extra mouse buttons for usb-tablet

2025-07-13 Thread Michael Tokarev
From: Thomas Lambertz The necessary plumbing for side- and extra mouse buttons to reach usb-tablet is already done. But the descriptor advertises three buttons max. Increase this to 5. Buttons are now identical to usb-mouse. Signed-off-by: Thomas Lambertz Signed-off-by: Michael Tokarev --- hw

[PULL 0/6] Trivial patches for 2025-07-13

2025-07-13 Thread Michael Tokarev
The following changes since commit 9a4e273ddec3927920c5958d2226c6b38b543336: Merge tag 'pull-tcg-20250711' of https://gitlab.com/rth7680/qemu into staging (2025-07-13 01:46:04 -0400) are available in the Git repository at: https://gitlab.com/mjt0k/qemu.git tags/pull-trivial-patches for you

[PULL 2/6] docs: remove repeated word

2025-07-13 Thread Michael Tokarev
From: Andrew Kreimer The word 'find' appears twice, remove the extra one. Signed-off-by: Andrew Kreimer Reviewed-by: Markus Armbruster Reviewed-by: Michael Tokarev Signed-off-by: Michael Tokarev --- docs/system/introduction.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --g

Re: [PATCH] net: fix buffer overflow in af_xdp_umem_create()

2025-07-13 Thread Michael Tokarev
Ping? Thanks, /mjt On 02.06.2025 13:09, Ilya Maximets wrote: On 6/2/25 10:57 AM, Anastasia Belova wrote: s->pool has n_descs elements so maximum i should be n_descs - 1. Fix the upper bound. Found by Linux Verification Center (linuxtesting.org) with SVACE. Fixes: cb039ef3d9 ("net: add initi

Re: [PULL 000/119] target-arm queue

2025-07-13 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/10.1 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 0/9] tcg/linux-user/fpu patch queue

2025-07-13 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/10.1 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 00/15] functional tests and s390x patches

2025-07-13 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/10.1 for any user-visible changes. signature.asc Description: PGP signature

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

2025-07-13 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/10.1 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL v2 00/36] Accelerators patches for 2025-07-04

2025-07-13 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/10.1 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL v2 0/4] loongarch-to-apply queue

2025-07-13 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/10.1 for any user-visible changes. signature.asc Description: PGP signature

Re: [PULL 00/26] Migration patches for 2025-07-11

2025-07-13 Thread Stefan Hajnoczi
Applied, thanks. Please update the changelog at https://wiki.qemu.org/ChangeLog/10.1 for any user-visible changes. signature.asc Description: PGP signature