* [PULL 1/2] virtio-net: Fix num_buffers for version 1
2025-04-03 21:17 [PULL 0/2] virtio,pc: fixes Michael S. Tsirkin
@ 2025-04-03 21:17 ` Michael S. Tsirkin
2025-04-03 21:17 ` [PULL 2/2] hw/i386/amd_iommu: Assign pci-id 0x1419 for the AMD IOMMU device Michael S. Tsirkin
2025-04-04 17:07 ` [PULL 0/2] virtio,pc: fixes Stefan Hajnoczi
2 siblings, 0 replies; 4+ messages in thread
From: Michael S. Tsirkin @ 2025-04-03 21:17 UTC (permalink / raw)
To: qemu-devel; +Cc: Peter Maydell, Akihiko Odaki, Lei Yang, Jason Wang
From: Akihiko Odaki <akihiko.odaki@daynix.com>
The specification says the device MUST set num_buffers to 1 if
VIRTIO_NET_F_MRG_RXBUF has not been negotiated.
Fixes: df91055db5c9 ("virtio-net: enable virtio 1.0")
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
Message-Id: <20250108-buffers-v1-1-a0c85ff31aeb@daynix.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Tested-by: Lei Yang <leiyang@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
hw/net/virtio-net.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index de87cfadff..340c6b6422 100644
--- a/hw/net/virtio-net.c
+++ b/hw/net/virtio-net.c
@@ -1999,6 +1999,8 @@ static ssize_t virtio_net_receive_rcu(NetClientState *nc, const uint8_t *buf,
offsetof(typeof(hdr),
virtio_net.hdr.num_buffers),
sizeof(hdr.virtio_net.hdr.num_buffers));
+ } else {
+ hdr.virtio_net.hdr.num_buffers = cpu_to_le16(1);
}
guest_offset = n->has_vnet_hdr ?
--
MST
^ permalink raw reply related [flat|nested] 4+ messages in thread* [PULL 2/2] hw/i386/amd_iommu: Assign pci-id 0x1419 for the AMD IOMMU device
2025-04-03 21:17 [PULL 0/2] virtio,pc: fixes Michael S. Tsirkin
2025-04-03 21:17 ` [PULL 1/2] virtio-net: Fix num_buffers for version 1 Michael S. Tsirkin
@ 2025-04-03 21:17 ` Michael S. Tsirkin
2025-04-04 17:07 ` [PULL 0/2] virtio,pc: fixes Stefan Hajnoczi
2 siblings, 0 replies; 4+ messages in thread
From: Michael S. Tsirkin @ 2025-04-03 21:17 UTC (permalink / raw)
To: qemu-devel
Cc: Peter Maydell, Suravee Suthikulpanit, Daniel P . Berrangé,
Yan Vugenfirer, Marcel Apfelbaum, Paolo Bonzini,
Richard Henderson, Eduardo Habkost
From: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
Currently, the QEMU-emulated AMD IOMMU device use PCI vendor id 0x1022
(AMD) with device id zero (undefined). Eventhough this does not cause any
functional issue for AMD IOMMU driver since it normally uses information
in the ACPI IVRS table to probe and initialize the device per
recommendation in the AMD IOMMU specification, the device id zero causes
the Windows Device Manager utility to show the device as an unknown device.
Since Windows only recognizes AMD IOMMU device with device id 0x1419 as
listed in the machine.inf file, modify the QEMU AMD IOMMU model to use
the id 0x1419 to avoid the issue. This advertise the IOMMU as the AMD
IOMMU device for Family 15h (Models 10h-1fh).
Signed-off-by: Suravee Suthikulpanit <suravee.suthikulpanit@amd.com>
Message-Id: <20250325021140.5676-1-suravee.suthikulpanit@amd.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Yan Vugenfirer <yvugenfi@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---
hw/i386/amd_iommu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/hw/i386/amd_iommu.c b/hw/i386/amd_iommu.c
index 5b21cf134a..5f9b952799 100644
--- a/hw/i386/amd_iommu.c
+++ b/hw/i386/amd_iommu.c
@@ -1706,6 +1706,7 @@ static void amdvi_pci_class_init(ObjectClass *klass, void *data)
PCIDeviceClass *k = PCI_DEVICE_CLASS(klass);
k->vendor_id = PCI_VENDOR_ID_AMD;
+ k->device_id = 0x1419;
k->class_id = 0x0806;
k->realize = amdvi_pci_realize;
--
MST
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PULL 0/2] virtio,pc: fixes
2025-04-03 21:17 [PULL 0/2] virtio,pc: fixes Michael S. Tsirkin
2025-04-03 21:17 ` [PULL 1/2] virtio-net: Fix num_buffers for version 1 Michael S. Tsirkin
2025-04-03 21:17 ` [PULL 2/2] hw/i386/amd_iommu: Assign pci-id 0x1419 for the AMD IOMMU device Michael S. Tsirkin
@ 2025-04-04 17:07 ` Stefan Hajnoczi
2 siblings, 0 replies; 4+ messages in thread
From: Stefan Hajnoczi @ 2025-04-04 17:07 UTC (permalink / raw)
To: Michael S. Tsirkin; +Cc: qemu-devel, Peter Maydell
[-- Attachment #1: Type: text/plain, Size: 116 bytes --]
Applied, thanks.
Please update the changelog at https://wiki.qemu.org/ChangeLog/10.0 for any user-visible changes.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread