Commit 6ced0bba authored by Alex Williamson's avatar Alex Williamson
Browse files

vfio/pci: Add a separate option for IGD OpRegion support 



The IGD OpRegion is enabled automatically when running in legacy mode,
but it can sometimes be useful in universal passthrough mode as well.
Without an OpRegion, output spigots don't work, and even though Intel
doesn't officially support physical outputs in UPT mode, it's a
useful feature.  Note that if an OpRegion is enabled but a monitor is
not connected, some graphics features will be disabled in the guest
versus a headless system without an OpRegion, where they would work.

Signed-off-by: default avatarAlex Williamson <alex.williamson@redhat.com>
Reviewed-by: default avatarGerd Hoffmann <kraxel@redhat.com>
Tested-by: default avatarGerd Hoffmann <kraxel@redhat.com>
parent c4c45e94

hw/vfio/pci-quirks.c

+2 −2
Original line number Diff line number Diff line
@@ -1055,7 +1055,7 @@ typedef struct VFIOIGDQuirk { 
 * the table and to write the base address of that memory to the ASLS register

 * of the IGD device.

 */

static int vfio_pci_igd_opregion_init(VFIOPCIDevice *vdev,

int vfio_pci_igd_opregion_init(VFIOPCIDevice *vdev,

                               struct vfio_region_info *info)

{

    int ret;

hw/vfio/pci.c

+31 −0
Original line number Diff line number Diff line
@@ -2562,6 +2562,35 @@ static int vfio_initfn(PCIDevice *pdev) 
        vfio_bar_quirk_setup(vdev, i);

    }



    if (!vdev->igd_opregion &&

        vdev->features & VFIO_FEATURE_ENABLE_IGD_OPREGION) {

        struct vfio_region_info *opregion;



        if (vdev->pdev.qdev.hotplugged) {

            error_report("Cannot support IGD OpRegion feature on hotplugged "

                         "device %s", vdev->vbasedev.name);

            ret = -EINVAL;

            goto out_teardown;

        }



        ret = vfio_get_dev_region_info(&vdev->vbasedev,

                        VFIO_REGION_TYPE_PCI_VENDOR_TYPE | PCI_VENDOR_ID_INTEL,

                        VFIO_REGION_SUBTYPE_INTEL_IGD_OPREGION, &opregion);

        if (ret) {

            error_report("Device %s does not support requested IGD OpRegion "

                         "feature", vdev->vbasedev.name);

            goto out_teardown;

        }



        ret = vfio_pci_igd_opregion_init(vdev, opregion);

        g_free(opregion);

        if (ret) {

            error_report("Device %s IGD OpRegion initialization failed",

                         vdev->vbasedev.name);

            goto out_teardown;

        }

    }



    /* QEMU emulates all of MSI & MSIX */

    if (pdev->cap_present & QEMU_PCI_CAP_MSIX) {

        memset(vdev->emulated_config_bits + pdev->msix_cap, 0xff,
@@ -2686,6 +2715,8 @@ static Property vfio_pci_dev_properties[] = { 
                    VFIO_FEATURE_ENABLE_VGA_BIT, false),

    DEFINE_PROP_BIT("x-req", VFIOPCIDevice, features,

                    VFIO_FEATURE_ENABLE_REQ_BIT, true),

    DEFINE_PROP_BIT("x-igd-opregion", VFIOPCIDevice, features,

                    VFIO_FEATURE_ENABLE_IGD_OPREGION_BIT, false),

    DEFINE_PROP_BOOL("x-no-mmap", VFIOPCIDevice, vbasedev.no_mmap, false),

    DEFINE_PROP_BOOL("x-no-kvm-intx", VFIOPCIDevice, no_kvm_intx, false),

    DEFINE_PROP_BOOL("x-no-kvm-msi", VFIOPCIDevice, no_kvm_msi, false),

hw/vfio/pci.h

+6 −0
Original line number Diff line number Diff line
@@ -129,6 +129,9 @@ typedef struct VFIOPCIDevice { 
#define VFIO_FEATURE_ENABLE_VGA (1 << VFIO_FEATURE_ENABLE_VGA_BIT)

#define VFIO_FEATURE_ENABLE_REQ_BIT 1

#define VFIO_FEATURE_ENABLE_REQ (1 << VFIO_FEATURE_ENABLE_REQ_BIT)

#define VFIO_FEATURE_ENABLE_IGD_OPREGION_BIT 2

#define VFIO_FEATURE_ENABLE_IGD_OPREGION \

                                (1 << VFIO_FEATURE_ENABLE_IGD_OPREGION_BIT)

    int32_t bootindex;

    uint32_t igd_gms;

    uint8_t pm_cap;
@@ -161,4 +164,7 @@ void vfio_setup_resetfn_quirk(VFIOPCIDevice *vdev); 


int vfio_populate_vga(VFIOPCIDevice *vdev);



int vfio_pci_igd_opregion_init(VFIOPCIDevice *vdev,

                               struct vfio_region_info *info);



#endif /* HW_VFIO_VFIO_PCI_H */