Additional information is now printed when dpctl.dump is called.

Author: diptorupd

backends/source/dppl_sycl_device_interface.cpp

@@ -35,7 +35,7 @@ using namespace cl::sycl;
 namespace
 {
 // Create wrappers for C Binding types (see CBindingWrapping.h).
- DEFINE_SIMPLE_CONVERSION_FUNCTIONS(device, DPPLSyclDeviceRef)
+DEFINE_SIMPLE_CONVERSION_FUNCTIONS(device, DPPLSyclDeviceRef)
 
  /*!
  * @brief Helper function to print the metadata for a sycl::device.
@@ -54,6 +54,23 @@ void dump_device_info (const device & Device)
        << Device.get_info<info::device::vendor>() << '\n';
     ss << std::setw(4) << " " << std::left << std::setw(16) << "Profile"
        << Device.get_info<info::device::profile>() << '\n';
+    ss << std::setw(4) << " " << std::left << std::setw(16) << "Device type";
+
+    try {
+        if (Device.has(aspect::accelerator))
+            ss << "accelerator" << '\n';
+        else if (Device.has(aspect::cpu))
+            ss << "cpu" << '\n';
+        else if (Device.has(aspect::custom))
+            ss << "custom" << '\n';
+        else if (Device.has(aspect::gpu))
+            ss << "gpu" << '\n';
+        else if (Device.has(aspect::host))
+            ss << "host" << '\n';
+    } catch (runtime_error re) {
+        // \todo handle errors
+        ss << "unknown\n";
+    }
 
     std::cout << ss.str();
 }

backends/source/dppl_sycl_platform_interface.cpp

@@ -38,8 +38,15 @@ using namespace cl::sycl;
  * found on the system:
  *      - info::platform::name
  *      - info::platform::version
+ *      - info::platform::vendor
  *      - info::platform::profile
+ *      - backend (opencl, cuda, level-zero, host)
+ *      - number of devices on the platform
  *
+ * Additionally, for each device we print out:
+ *      - info::device::name
+ *      - info::device::driver_version
+ *      - type of the device based on the aspects cpu, gpu, accelerator.
  */
 void DPPLPlatform_DumpInfo ()
 {
@@ -51,14 +58,54 @@ void DPPLPlatform_DumpInfo ()
       std::cout << "---Platform " << i << '\n';
       std::stringstream ss;
 
+      auto vendor = p.get_info<info::platform::vendor>();
+      if (vendor.empty())
+         vendor = "unknown";
+
       ss << std::setw(4) << " " << std::left << std::setw(12) << "Name"
          << p.get_info<info::platform::name>() << '\n';
       ss << std::setw(4) << " " << std::left << std::setw(12) << "Version"
          << p.get_info<info::platform::version>() << '\n';
       ss << std::setw(4) << " " << std::left << std::setw(12) << "Vendor"
-         << p.get_info<info::platform::vendor>() << '\n';
+         << vendor << '\n';
       ss << std::setw(4) << " " << std::left << std::setw(12) << "Profile"
          << p.get_info<info::platform::profile>() << '\n';
+      ss << std::setw(4) << " " << std::left << std::setw(12) << "Backend";
+      p.is_host() ? (ss << "unknown") : (ss << p.get_backend());
+      ss << '\n';
+
+      // Get number of devices on the platform
+      auto devices = p.get_devices();
+
+      ss << std::setw(4) << " " << std::left << std::setw(12) << "Devices"
+         << devices.size() << '\n';
+      // Print some of the device information
+      for (auto dn = 0ul; dn < devices.size(); ++dn) {
+         ss << std::setw(4) << "---Device " << dn << '\n';
+         ss << std::setw(8) << " " << std::left << std::setw(20)
+            << "Name" << devices[dn].get_info<info::device::name>() << '\n';
+         ss << std::setw(8) << " " << std::left << std::setw(20)
+            << "Driver version"
+            << devices[dn].get_info<info::device::driver_version>() << '\n';
+         ss << std::setw(8) << " " << std::left << std::setw(20)
+            << "Device type";
+
+         try {
+            if (devices[dn].has(aspect::accelerator))
+               ss << "accelerator" << '\n';
+            else if (devices[dn].has(aspect::cpu))
+               ss << "cpu" << '\n';
+            else if (devices[dn].has(aspect::custom))
+               ss << "custom" << '\n';
+            else if (devices[dn].has(aspect::gpu))
+               ss << "gpu" << '\n';
+            else if (devices[dn].has(aspect::host))
+               ss << "host" << '\n';
+         } catch (runtime_error re) {
+            // \todo handle errors
+            ss << "unknown\n";
+         }
+      }
 
       std::cout << ss.str();
       ++i;