On 6/10/25 19:14, Steven Sistare wrote:
On 6/10/2025 1:11 PM, Cédric Le Goater wrote:
On 6/10/25 19:05, Steven Sistare wrote:
On 6/10/2025 12:31 PM, Michael S. Tsirkin wrote:
On Wed, Jun 04, 2025 at 03:48:40PM +0200, Cédric Le Goater wrote:
I don't see any advantage to making this a class attribute. I looked for
examples
of using such attributes for vfio to configure pci, and found very little. It
sounds like overkill since vfio already sets and gets PCIDevice members directly
in many places.
I defined skip_reset_on_cpr based on this existing example:
vfio_instance_init()
pci_dev->cap_present |= QEMU_PCI_CAP_EXPRESS
pci_dev->cap_present can be modified at realize time. skip_reset_on_cpr
is a constant, for which a class attribute are more appropriate.
This is minor.
Michael,
Are you ok with the 'skip_reset_on_cpr' bool ?
Generally yes, but maybe cap_present bit is even cleaner?
vfio already pokes at it, and we have history of encoding
quirks there, see QEMU_PCIE_LNKSTA_DLLLA_BITNR for example.
Sure, I can send a new version based on a cap_present bit
QEMU_PCI_SKIP_RESET_ON_CPR.
Please send an update of patch "pci: skip reset during cpr" in the v5 series.
Hopefully it will apply cleanly.
Please clarify: do you want me to send a delta that applies on top of the
old patch, or send a new version of the old patch?
a new version of the patch.
Thanks,
C.
