[UR][L0] Fix POWER_LIMIT device info where it is not supported (intel#18536)

Some devices don't provide access to power information, which was
incorrectly handled. This makes the following changes:
* When the output pointer is nullptr, `UNSUPPORTED_ENUM` is still
  returned.
* Return `UNSUPPORTED_ENUM` rather than `UNSUPPORTED_FEATURE`.
* In the conformance test, use the proper property name when checking if
  it is optional (this is technically a NFC, since the one incorrectly
  used is optional anyway).

Author: RossBrunton

unified-runtime/source/adapters/level_zero/device.cpp

@@ -1315,19 +1315,27 @@ ur_result_t urDeviceGetInfo(
   }
   case UR_DEVICE_INFO_MIN_POWER_LIMIT:
   case UR_DEVICE_INFO_MAX_POWER_LIMIT: {
-    if (!ParamValue) {
-      // If ParamValue is nullptr, then we are only interested in the size of
-      // the value.
-      return ReturnValue(int32_t{0});
-    }
-
     [[maybe_unused]] auto [ZesDevice, Ignored, Result] =
         getZesDeviceData(Device);
     if (Result != UR_RESULT_SUCCESS)
       return Result;
 
     zes_pwr_handle_t ZesPwrHandle = nullptr;
-    ZE2UR_CALL(zesDeviceGetCardPowerDomain, (ZesDevice, &ZesPwrHandle));
+    auto DomainResult = zesDeviceGetCardPowerDomain(ZesDevice, &ZesPwrHandle);
+    if (DomainResult == ZE_RESULT_ERROR_UNSUPPORTED_FEATURE) {
+      return UR_RESULT_ERROR_UNSUPPORTED_ENUMERATION;
+    } else if (DomainResult != ZE_RESULT_SUCCESS) {
+      return ze2urResult(DomainResult);
+    }
+
+    if (!ParamValue) {
+      // If ParamValue is nullptr, then we are only interested in the size of
+      // the value.
+      // Do this after calling getCardPowerDomain so that UNSUPPORTED is
+      // returned correctly if required
+      return ReturnValue(int32_t{0});
+    }
+
     ZesStruct<zes_power_properties_t> PowerProperties;
     ZE2UR_CALL(zesPowerGetProperties, (ZesPwrHandle, &PowerProperties));
 

unified-runtime/test/conformance/device/urDeviceGetInfo.cpp

@@ -2608,7 +2608,7 @@ TEST_P(urDeviceGetInfoTest, SuccessFanSpeed) {
 
   ASSERT_SUCCESS_OR_OPTIONAL_QUERY(
       urDeviceGetInfo(device, property_name, 0, nullptr, &property_size),
-      UR_DEVICE_INFO_COMPOSITE_DEVICE);
+      property_name);
 
   ASSERT_EQ(property_size, sizeof(int32_t));
 
@@ -2629,7 +2629,7 @@ TEST_P(urDeviceGetInfoTest, SuccessMaxPowerLimit) {
 
   ASSERT_SUCCESS_OR_OPTIONAL_QUERY(
       urDeviceGetInfo(device, property_name, 0, nullptr, &property_size),
-      UR_DEVICE_INFO_COMPOSITE_DEVICE);
+      property_name);
 
   ASSERT_EQ(property_size, sizeof(int32_t));
 
@@ -2650,7 +2650,7 @@ TEST_P(urDeviceGetInfoTest, SuccessMinPowerLimit) {
 
   ASSERT_SUCCESS_OR_OPTIONAL_QUERY(
       urDeviceGetInfo(device, property_name, 0, nullptr, &property_size),
-      UR_DEVICE_INFO_COMPOSITE_DEVICE);
+      property_name);
 
   ASSERT_EQ(property_size, sizeof(int32_t));