Re: [PATCH v12 1/1] vfio/nvgpu: Add vfio pci variant module for grace hopper

From: Ankit Agrawal
Date: Mon Oct 23 2023 - 08:48:30 EST


> After looking at Yishai's virtio-vfio-pci driver where BAR0 is emulated
> as an IO Port BAR, it occurs to me that there's no config space
> emulation of BAR2 (or BAR3) here.  Doesn't this mean that QEMU registers
> the BAR as 32-bit, non-prefetchable?  ie. VFIOBAR.type & .mem64 are
> wrong?

Maybe I didn't understand the question, but the PCI config space read/write
would still be handled by vfio_pci_core_read/write() which returns the
appropriate flags. I have checked that the device BARs are 64b and
prefetchable in the VM.

> We also need to decide how strictly variant drivers need to emulate
> vfio_pci_config_rw with respect to BAR sizing, where the core code
> provides emulation of sizing and Yishai's virtio driver only emulates
> the IO port indicator bit.

Sorry, it isn't clear to me how would resizable BAR is applicable in this
variant driver as the BAR represents the device memory. Should we be
exposing such feature as unsupported for this variant driver?