Skip to content

Commit 5e1fdc5

Browse files
martygrantkbenzie
martygrant
and
kbenzie
committed
Update setErrorMessage to use int32_t instead of ur_result_t for the urAdapterGetLastError entry point (#18264)
Fixes intel/llvm#18260 Also updates `checkUrResult` in SYCL RT to remove warning level check as `urAdapterGetLastError` is now only used for errors. --------- Co-authored-by: Kenneth Benzie (Benie) <[email protected]>
1 parent ed3217e commit 5e1fdc5

File tree

13 files changed

+32
-49
lines changed

13 files changed

+32
-49
lines changed

‎source/adapters/cuda/adapter.cpp‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,9 +90,10 @@ UR_APIEXPORT ur_result_t UR_APICALL urAdapterRelease(ur_adapter_handle_t) {
9090
}
9191

9292
UR_APIEXPORT ur_result_t UR_APICALL urAdapterGetLastError(
93-
ur_adapter_handle_t, const char **ppMessage, int32_t * /*pError*/) {
93+
ur_adapter_handle_t, const char **ppMessage, int32_t *pError) {
9494
*ppMessage = ErrorMessage;
95-
return ErrorMessageCode;
95+
*pError = ErrorMessageCode;
96+
return UR_RESULT_SUCCESS;
9697
}
9798

9899
UR_APIEXPORT ur_result_t UR_APICALL urAdapterGetInfo(ur_adapter_handle_t,

‎source/adapters/cuda/common.cpp‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -135,12 +135,11 @@ std::string getCudaVersionString() {
135135
}
136136

137137
// Global variables for ZER_EXT_RESULT_ADAPTER_SPECIFIC_ERROR
138-
thread_local ur_result_t ErrorMessageCode = UR_RESULT_SUCCESS;
138+
thread_local int32_t ErrorMessageCode = 0;
139139
thread_local char ErrorMessage[MaxMessageSize]{};
140140

141141
// Utility function for setting a message and warning
142-
[[maybe_unused]] void setErrorMessage(const char *pMessage,
143-
ur_result_t ErrorCode) {
142+
[[maybe_unused]] void setErrorMessage(const char *pMessage, int32_t ErrorCode) {
144143
assert(strlen(pMessage) < MaxMessageSize);
145144
// Copy at most MaxMessageSize - 1 bytes to ensure the resultant string is
146145
// always null terminated.

‎source/adapters/cuda/common.hpp‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,12 +48,11 @@ void checkErrorUR(ur_result_t Result, const char *Function, int Line,
4848
std::string getCudaVersionString();
4949

5050
constexpr size_t MaxMessageSize = 256;
51-
extern thread_local ur_result_t ErrorMessageCode;
51+
extern thread_local int32_t ErrorMessageCode;
5252
extern thread_local char ErrorMessage[MaxMessageSize];
5353

5454
// Utility function for setting a message and warning
55-
[[maybe_unused]] void setErrorMessage(const char *pMessage,
56-
ur_result_t ErrorCode);
55+
[[maybe_unused]] void setErrorMessage(const char *pMessage, int32_t ErrorCode);
5756

5857
void setPluginSpecificMessage(CUresult cu_res);
5958

‎source/adapters/cuda/enqueue.cpp‎

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1682,10 +1682,9 @@ urEnqueueUSMAdvise(ur_queue_handle_t hQueue, const void *pMem, size_t size,
16821682
UR_CHECK_ERROR(cuPointerGetAttribute(
16831683
&IsManaged, CU_POINTER_ATTRIBUTE_IS_MANAGED, (CUdeviceptr)pMem));
16841684
if (!IsManaged) {
1685-
setErrorMessage(
1686-
"Memory advice ignored as memory advices only works with USM",
1687-
UR_RESULT_SUCCESS);
1688-
return UR_RESULT_ERROR_ADAPTER_SPECIFIC;
1685+
logger::warning(
1686+
"Memory advice ignored as memory advices only works with USM.");
1687+
return UR_RESULT_SUCCESS;
16891688
}
16901689

16911690
ur_result_t Result = UR_RESULT_SUCCESS;

‎source/adapters/hip/common.cpp‎

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -157,22 +157,14 @@ hipError_t getHipVersionString(std::string &Version) {
157157
}
158158

159159
// Global variables for UR_RESULT_ADAPTER_SPECIFIC_ERROR
160-
thread_local ur_result_t ErrorMessageCode = UR_RESULT_SUCCESS;
160+
thread_local int32_t ErrorMessageCode = 0;
161161
thread_local char ErrorMessage[MaxMessageSize]{};
162162

163163
// Utility function for setting a message and warning
164-
[[maybe_unused]] void setErrorMessage(const char *pMessage,
165-
ur_result_t ErrorCode) {
164+
[[maybe_unused]] void setErrorMessage(const char *pMessage, int32_t ErrorCode) {
166165
assert(strlen(pMessage) < MaxMessageSize);
167166
// Copy at most MaxMessageSize - 1 bytes to ensure the resultant string is
168167
// always null terminated.
169168
strncpy(ErrorMessage, pMessage, MaxMessageSize - 1);
170169
ErrorMessageCode = ErrorCode;
171170
}
172-
173-
// Returns plugin specific error and warning messages; common implementation
174-
// that can be shared between adapters
175-
ur_result_t urGetLastResult(ur_platform_handle_t, const char **ppMessage) {
176-
*ppMessage = &ErrorMessage[0];
177-
return ErrorMessageCode;
178-
}

‎source/adapters/hip/common.hpp‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,12 +106,11 @@ void checkErrorUR(ur_result_t Result, const char *Function, int Line,
106106
hipError_t getHipVersionString(std::string &Version);
107107

108108
constexpr size_t MaxMessageSize = 256;
109-
extern thread_local ur_result_t ErrorMessageCode;
109+
extern thread_local int32_t ErrorMessageCode;
110110
extern thread_local char ErrorMessage[MaxMessageSize];
111111

112112
// Utility function for setting a message and warning
113-
[[maybe_unused]] void setErrorMessage(const char *Message,
114-
ur_result_t ErrorCode);
113+
[[maybe_unused]] void setErrorMessage(const char *Message, int32_t ErrorCode);
115114

116115
// Helper method to return a (non-null) pointer's attributes, or std::nullopt in
117116
// the case that the pointer is unknown to the HIP subsystem.

‎source/adapters/level_zero/adapter.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -712,7 +712,7 @@ ur_result_t urAdapterGetLastError(
712712
*Message = ErrorMessage;
713713
*Error = ErrorAdapterNativeCode;
714714

715-
return ErrorMessageCode;
715+
return UR_RESULT_SUCCESS;
716716
}
717717

718718
ur_result_t urAdapterGetInfo(ur_adapter_handle_t, ur_adapter_info_t PropName,

‎source/adapters/level_zero/common.cpp‎

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -342,13 +342,12 @@ getZeStructureType<ze_intel_device_block_array_exp_properties_t>() {
342342
#endif // ZE_INTEL_DEVICE_BLOCK_ARRAY_EXP_NAME
343343

344344
// Global variables for ZER_EXT_RESULT_ADAPTER_SPECIFIC_ERROR
345-
thread_local ur_result_t ErrorMessageCode = UR_RESULT_SUCCESS;
345+
thread_local int32_t ErrorMessageCode = 0;
346346
thread_local char ErrorMessage[MaxMessageSize]{};
347347
thread_local int32_t ErrorAdapterNativeCode;
348348

349349
// Utility function for setting a message and warning
350-
[[maybe_unused]] void setErrorMessage(const char *pMessage,
351-
ur_result_t ErrorCode,
350+
[[maybe_unused]] void setErrorMessage(const char *pMessage, int32_t ErrorCode,
352351
int32_t AdapterErrorCode) {
353352
assert(strlen(pMessage) < MaxMessageSize);
354353
// Copy at most MaxMessageSize - 1 bytes to ensure the resultant string is
@@ -357,8 +356,3 @@ thread_local int32_t ErrorAdapterNativeCode;
357356
ErrorMessageCode = ErrorCode;
358357
ErrorAdapterNativeCode = AdapterErrorCode;
359358
}
360-
361-
ur_result_t zerPluginGetLastError(char **message) {
362-
*message = &ErrorMessage[0];
363-
return ErrorMessageCode;
364-
}

‎source/adapters/level_zero/common.hpp‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -373,11 +373,10 @@ constexpr char ZE_SUPPORTED_EXTENSIONS[] =
373373

374374
// Global variables for ZER_EXT_RESULT_ADAPTER_SPECIFIC_ERROR
375375
constexpr size_t MaxMessageSize = 256;
376-
extern thread_local ur_result_t ErrorMessageCode;
376+
extern thread_local int32_t ErrorMessageCode;
377377
extern thread_local char ErrorMessage[MaxMessageSize];
378378
extern thread_local int32_t ErrorAdapterNativeCode;
379379

380380
// Utility function for setting a message and warning
381-
[[maybe_unused]] void setErrorMessage(const char *pMessage,
382-
ur_result_t ErrorCode,
381+
[[maybe_unused]] void setErrorMessage(const char *pMessage, int32_t ErrorCode,
383382
int32_t AdapterErrorCode);

‎source/adapters/native_cpu/common.cpp‎

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11,21 +11,14 @@
1111
#include "common.hpp"
1212

1313
// Global variables for UR_RESULT_ADAPTER_SPECIFIC_ERROR
14-
// See urGetLastResult
15-
thread_local ur_result_t ErrorMessageCode = UR_RESULT_SUCCESS;
14+
thread_local int32_t ErrorMessageCode = 0;
1615
thread_local char ErrorMessage[MaxMessageSize]{};
1716

1817
// Utility function for setting a message and warning
19-
[[maybe_unused]] void setErrorMessage(const char *pMessage,
20-
ur_result_t ErrorCode) {
18+
[[maybe_unused]] void setErrorMessage(const char *pMessage, int32_t ErrorCode) {
2119
assert(strlen(pMessage) < MaxMessageSize);
2220
// Copy at most MaxMessageSize - 1 bytes to ensure the resultant string is
2321
// always null terminated.
2422
strncpy(ErrorMessage, pMessage, MaxMessageSize - 1);
2523
ErrorMessageCode = ErrorCode;
2624
}
27-
28-
ur_result_t urGetLastResult(ur_platform_handle_t, const char **ppMessage) {
29-
*ppMessage = &ErrorMessage[0];
30-
return ErrorMessageCode;
31-
}

0 commit comments

Comments
 (0)