Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(569)

Unified Diff: gpu/config/gpu_info_collector_linux.cc

Issue 2725873007: gpu/config: Use angle::GetSystemInfo on Linux (Closed)
Patch Set: "Fix" gn check Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « gpu/config/gpu_info_collector_linux.h ('k') | gpu/config/gpu_info_collector_ozone.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: gpu/config/gpu_info_collector_linux.cc
diff --git a/gpu/config/gpu_info_collector_linux.cc b/gpu/config/gpu_info_collector_linux.cc
index 1247db3167bc19301bfcc8857c89a91a4053dc04..fd440f30d16612a0a94053f7254f000c49356dc9 100644
--- a/gpu/config/gpu_info_collector_linux.cc
+++ b/gpu/config/gpu_info_collector_linux.cc
@@ -2,8 +2,6 @@
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "gpu/config/gpu_info_collector_linux.h"
-
#include <stddef.h>
#include <stdint.h>
@@ -20,6 +18,7 @@
#include "base/trace_event/trace_event.h"
#include "gpu/config/gpu_info_collector.h"
#include "gpu/config/gpu_switches.h"
+#include "third_party/angle/src/gpu_info_util/SystemInfo.h"
#include "third_party/re2/src/re2/re2.h"
#include "ui/gl/gl_bindings.h"
#include "ui/gl/gl_context.h"
@@ -27,160 +26,8 @@
#include "ui/gl/gl_surface.h"
#include "ui/gl/gl_switches.h"
-#if defined(USE_LIBPCI)
-#include "library_loaders/libpci.h" // nogncheck
-#endif
-
namespace gpu {
-namespace {
-
-#if defined(USE_LIBPCI)
-// This checks if a system supports PCI bus.
-// We check the existence of /sys/bus/pci or /sys/bug/pci_express.
-bool IsPciSupported() {
- const base::FilePath pci_path("/sys/bus/pci/");
- const base::FilePath pcie_path("/sys/bus/pci_express/");
- return (base::PathExists(pci_path) ||
- base::PathExists(pcie_path));
-}
-#endif // defined(USE_LIBPCI)
-
-// Scan /sys/module/amdgpu/version.
-// Return empty string on failing.
-std::string CollectDriverVersionAMDBrahma() {
- const base::FilePath ati_file_path("/sys/module/amdgpu/version");
- if (!base::PathExists(ati_file_path))
- return std::string();
- std::string contents;
- if (!base::ReadFileToString(ati_file_path, &contents))
- return std::string();
- size_t begin = contents.find_first_of("0123456789");
- if (begin != std::string::npos) {
- size_t end = contents.find_first_not_of("0123456789.", begin);
- if (end == std::string::npos)
- return contents.substr(begin);
- else
- return contents.substr(begin, end - begin);
- }
- return std::string();
-}
-
-// Scan /etc/ati/amdpcsdb.default for "ReleaseVersion".
-// Return empty string on failing.
-std::string CollectDriverVersionAMDCatalyst() {
- const base::FilePath ati_file_path("/etc/ati/amdpcsdb.default");
- if (!base::PathExists(ati_file_path))
- return std::string();
- std::string contents;
- if (!base::ReadFileToString(ati_file_path, &contents))
- return std::string();
- base::StringTokenizer t(contents, "\r\n");
- while (t.GetNext()) {
- std::string line = t.token();
- if (base::StartsWith(line, "ReleaseVersion=",
- base::CompareCase::SENSITIVE)) {
- size_t begin = line.find_first_of("0123456789");
- if (begin != std::string::npos) {
- size_t end = line.find_first_not_of("0123456789.", begin);
- if (end == std::string::npos)
- return line.substr(begin);
- else
- return line.substr(begin, end - begin);
- }
- }
- }
- return std::string();
-}
-
-const uint32_t kVendorIDIntel = 0x8086;
-const uint32_t kVendorIDNVidia = 0x10de;
-const uint32_t kVendorIDAMD = 0x1002;
-
-CollectInfoResult CollectPCIVideoCardInfo(GPUInfo* gpu_info) {
- DCHECK(gpu_info);
-
-#if !defined(USE_LIBPCI)
- return kCollectInfoNonFatalFailure;
-#else
-
- if (IsPciSupported() == false) {
- VLOG(1) << "PCI bus scanning is not supported";
- return kCollectInfoNonFatalFailure;
- }
-
- // TODO(zmo): be more flexible about library name.
- LibPciLoader libpci_loader;
- if (!libpci_loader.Load("libpci.so.3") &&
- !libpci_loader.Load("libpci.so")) {
- VLOG(1) << "Failed to locate libpci";
- return kCollectInfoNonFatalFailure;
- }
-
- pci_access* access = (libpci_loader.pci_alloc)();
- DCHECK(access != NULL);
- (libpci_loader.pci_init)(access);
- (libpci_loader.pci_scan_bus)(access);
- bool primary_gpu_identified = false;
- for (pci_dev* device = access->devices;
- device != NULL; device = device->next) {
- // Fill the IDs and class fields.
- (libpci_loader.pci_fill_info)(device, 33);
- bool is_gpu = false;
- switch (device->device_class) {
- case PCI_CLASS_DISPLAY_VGA:
- case PCI_CLASS_DISPLAY_XGA:
- case PCI_CLASS_DISPLAY_3D:
- is_gpu = true;
- break;
- case PCI_CLASS_DISPLAY_OTHER:
- default:
- break;
- }
- if (!is_gpu)
- continue;
- if (device->vendor_id == 0 || device->device_id == 0)
- continue;
-
- GPUInfo::GPUDevice gpu;
- gpu.vendor_id = device->vendor_id;
- gpu.device_id = device->device_id;
-
- if (!primary_gpu_identified) {
- primary_gpu_identified = true;
- gpu_info->gpu = gpu;
- } else {
- // TODO(zmo): if there are multiple GPUs, we assume the non Intel
- // one is primary. Revisit this logic because we actually don't know
- // which GPU we are using at this point.
- if (gpu_info->gpu.vendor_id == kVendorIDIntel &&
- gpu.vendor_id != kVendorIDIntel) {
- gpu_info->secondary_gpus.push_back(gpu_info->gpu);
- gpu_info->gpu = gpu;
- } else {
- gpu_info->secondary_gpus.push_back(gpu);
- }
- }
- }
-
- // Detect Optimus or AMD Switchable GPU.
- if (gpu_info->secondary_gpus.size() == 1 &&
- gpu_info->secondary_gpus[0].vendor_id == kVendorIDIntel) {
- if (gpu_info->gpu.vendor_id == kVendorIDNVidia)
- gpu_info->optimus = true;
- if (gpu_info->gpu.vendor_id == kVendorIDAMD)
- gpu_info->amd_switchable = true;
- }
-
- (libpci_loader.pci_cleanup)(access);
- if (!primary_gpu_identified)
- return kCollectInfoNonFatalFailure;
- return kCollectInfoSuccess;
-#endif
-}
-
-} // namespace anonymous
-
CollectInfoResult CollectContextGraphicsInfo(GPUInfo* gpu_info) {
DCHECK(gpu_info);
@@ -194,51 +41,15 @@ CollectInfoResult CollectContextGraphicsInfo(GPUInfo* gpu_info) {
CollectInfoResult CollectBasicGraphicsInfo(GPUInfo* gpu_info) {
DCHECK(gpu_info);
- CollectInfoResult result = CollectPCIVideoCardInfo(gpu_info);
-
- std::string driver_version;
- switch (gpu_info->gpu.vendor_id) {
- case kVendorIDAMD:
- driver_version = CollectDriverVersionAMDBrahma();
- if (!driver_version.empty()) {
- gpu_info->driver_vendor = "ATI / AMD (Brahma)";
- gpu_info->driver_version = driver_version;
- } else {
- driver_version = CollectDriverVersionAMDCatalyst();
- if (!driver_version.empty()) {
- gpu_info->driver_vendor = "ATI / AMD (Catalyst)";
- gpu_info->driver_version = driver_version;
- }
- }
- break;
- case kVendorIDNVidia:
- driver_version = CollectDriverVersionNVidia();
- if (!driver_version.empty()) {
- gpu_info->driver_vendor = "NVIDIA";
- gpu_info->driver_version = driver_version;
- }
- break;
- case kVendorIDIntel:
- // In dual-GPU cases, sometimes PCI scan only gives us the
- // integrated GPU (i.e., the Intel one).
- if (gpu_info->secondary_gpus.size() == 0) {
- driver_version = CollectDriverVersionNVidia();
- if (!driver_version.empty()) {
- gpu_info->driver_vendor = "NVIDIA";
- gpu_info->driver_version = driver_version;
- gpu_info->optimus = true;
- // Put Intel to the secondary GPU list.
- gpu_info->secondary_gpus.push_back(gpu_info->gpu);
- // Put NVIDIA as the primary GPU.
- gpu_info->gpu.vendor_id = kVendorIDNVidia;
- gpu_info->gpu.device_id = 0; // Unknown Device.
- }
- }
- break;
+ angle::SystemInfo system_info;
+ if (angle::GetSystemInfo(&system_info)) {
+ gpu_info->basic_info_state = kCollectInfoSuccess;
+ FillGPUInfoFromSystemInfo(gpu_info, &system_info);
+ } else {
+ gpu_info->basic_info_state = kCollectInfoNonFatalFailure;
}
- gpu_info->basic_info_state = result;
- return result;
+ return gpu_info->basic_info_state;
}
CollectInfoResult CollectDriverInfoGL(GPUInfo* gpu_info) {
« no previous file with comments | « gpu/config/gpu_info_collector_linux.h ('k') | gpu/config/gpu_info_collector_ozone.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698