On 9/15/26 16:22, Michael S. Tsirkin wrote:
On Tue, Sep 15, 2026 at 12:17:31PM +0100, David Vrabel wrote:
On 15/09/2026 11:01, Michael S. Tsirkin wrote:

On Tue, Sep 15, 2026 at 09:25:05AM +0200, Cédric Le Goater wrote:
Hello,

The xio3130-downstream and x3130-upstream devices lacks x-speed and
x-width device properties, making it impossible to configure PCIe link
speed/width when building PIX switch topologies for GPU/NIC passthrough.

Fix that and preserve migration compatibility with a 2.5GT/s x1 default.

Thanks,

C.

Given they are clearly intended for users, time to rename to width/speed
everywhere?

A more comprehensive series (hw/pci-bridge: Configurable apparent bandwidth
for the PCIe topology) is here:

https://patchew.org/QEMU/[email protected]/

This adds new upstream and switch port devices with configurable speed/width
and renames the options so they are non-experimental (but do not the open
question about the usability of the speed option with the need to set 2.5
GT/s as the string "2_5").

I did consider adding options to the XIO3130 device but since the real
hardware is Gen 1 only and does not support speeds/widths other 2.5GT/s/1x I
didn't consider it appropriate to claim these emulated ports are more
capable than the real hardware.

David

Right that makes more sense to me.  Cédric what do you think?

I didn't know about it.

Regarding HW accuracy, I agree XIO3130 being Gen 1, the new speed
and link are inaccurate and this could confuse some tools checking
vendor/device ID.

Regarding the code size, user facing changes, existing topology,
migration, etc, boosting XIO3130 capabilities is safer. What is
worrisome is the downstream impact. backports will be a problem.
That said I didn't check.

David's proposal is preferable, as the device doesn't pretend
to be real HW and should be pursued for upstream. Mine would be
more a fallback solution for downstream. Hopefully not,

Cheers,

C.


Reply via email to