]> git.proxmox.com Git - qemu-server.git/commit
cfg2cmd: netdev: fix value for tx_queue_size
authorFiona Ebner <f.ebner@proxmox.com>
Thu, 27 Jul 2023 09:10:01 +0000 (11:10 +0200)
committerFabian Grünbichler <f.gruenbichler@proxmox.com>
Thu, 27 Jul 2023 11:14:12 +0000 (13:14 +0200)
commit089aed811dba66e317dc945423a083e89a85aeb8
tree3d1c3d494d707d56f44e956442dd5eafc3b7d897
parent0d4e8cbde08dd272bcf91318fe67845a58712bec
cfg2cmd: netdev: fix value for tx_queue_size

Quoting from QEMU commit 4271f40383 ("virtio-net: correctly report
maximum tx_queue_size value"):

> Maximum value for tx_queue_size depends on the backend type.
> 1024 for vDPA/vhost-user, 256 for all the others.

> So the parameter is silently ignored and ethtool reports a different
> value than the one provided by the user.

Indeed, for a non-vDPA/vhost-user netdev, the guest will see TX: 256
instead of the specified 1024 here. With the mentioned QEMU commit (in
master and will be part of 8.1), using 1024 will be a hard error:

> Invalid tx_queue_size (= 1024), must be a power of 2 between 256 and 256

Since neither vhost-user, nor vhost-vdpa netdev types are exposed by
Proxmox VE, just changing the limit to the correct 256 should be fine.
No obvious issue during live-migration found.

Fixes: 620d6b32 ("virtio-net: increase defaults rx|tx-queue-size to 1024")
Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
14 files changed:
PVE/QemuServer.pm
test/cfg2cmd/efidisk-on-rbd.conf.cmd
test/cfg2cmd/ide.conf.cmd
test/cfg2cmd/netdev-7.1-multiqueues.conf.cmd
test/cfg2cmd/netdev-7.1.conf.cmd
test/cfg2cmd/q35-ide.conf.cmd
test/cfg2cmd/q35-linux-hostpci-mapping.conf.cmd
test/cfg2cmd/q35-linux-hostpci-multifunction.conf.cmd
test/cfg2cmd/q35-linux-hostpci-x-pci-overrides.conf.cmd
test/cfg2cmd/q35-linux-hostpci.conf.cmd
test/cfg2cmd/q35-simple.conf.cmd
test/cfg2cmd/qemu-xhci-7.1.conf.cmd
test/cfg2cmd/qemu-xhci-q35-7.1.conf.cmd
test/cfg2cmd/seabios_serial.conf.cmd