Ignore subclass when finding GPU PCI devices

When enumerating PCI devices to find GPUs, only filter on the base class. This allows devices with class PCI_CLASS_DISPLAY_OTHER to be found. Bug: b/178748337 Change-Id: Ia0cd29dddc49a16428da16c1ef1b94040c1221f7 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2817973Reviewed-by: Corentin Wallez <cwallez@chromium.org> Reviewed-by: Jonah Ryan-Davis <jonahr@google.com> Commit-Queue: Jonah Ryan-Davis <jonahr@google.com>

Ignore subclass when finding GPU PCI devices
0615bbde · David Stevens · Commit Bot · 77600bcb · 0615bbde
Commit 0615bbde authored Apr 12, 2021 by David Stevens Committed by Commit Bot Apr 12, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 7 deletions

SystemInfo_libpci.cpp src/gpu_info_util/SystemInfo_libpci.cpp +2 -7

No files found.
--- a/src/gpu_info_util/SystemInfo_libpci.cpp
+++ b/src/gpu_info_util/SystemInfo_libpci.cpp
@@ -105,14 +105,9 @@ bool GetPCIDevicesWithLibPCI(std::vector<GPUDeviceInfo> *devices)
        pci.FillInfo(device, PCI_FILL_IDENT | PCI_FILL_CLASS);
        // Skip non-GPU devices
-        switch (device->device_class)
+        if (device->device_class >> 8 != PCI_BASE_CLASS_DISPLAY)
        {
-            case PCI_CLASS_DISPLAY_VGA:
+            continue;
-            case PCI_CLASS_DISPLAY_XGA:
-            case PCI_CLASS_DISPLAY_3D:
-                break;
-            default:
-                continue;
        }
        // Skip unknown devices