Skip to content

Commit 4c7d2f8

Browse files
authored
[XPTI] Rename ur to ur.call in XPTI (#14971)
Closes: #14922
1 parent b91fc8c commit 4c7d2f8

File tree

7 files changed

+14
-14
lines changed

7 files changed

+14
-14
lines changed

sycl/cmake/modules/FetchUnifiedRuntime.cmake

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -117,13 +117,13 @@ if(SYCL_UR_USE_FETCH_CONTENT)
117117
endfunction()
118118

119119
set(UNIFIED_RUNTIME_REPO "https://github.com/oneapi-src/unified-runtime.git")
120-
# commit cabf128094eff9ff7b79bdff559640a8a111f0c3
121-
# Merge: a96fcbc5 15bca3b6
120+
# commit d0a50523006fa6f283da6a36811081add3bb22fc
121+
# Merge: 804851e4 04deb8b3
122122
# Author: Omar Ahmed <[email protected]>
123-
# Date: Mon Aug 19 16:20:45 2024 +0100
124-
# Merge pull request #1984 from rafbiels/rafbiels/cuda-stream-race-cond
125-
# Fix race condition in CUDA stream creation
126-
set(UNIFIED_RUNTIME_TAG cabf128094eff9ff7b79bdff559640a8a111f0c3)
123+
# Date: Tue Aug 20 16:28:30 2024 +0100
124+
# Merge pull request #1940 from RossBrunton/ross/urcall
125+
# [XPTI] Use `ur.call` rather than `ur` in XPTI
126+
set(UNIFIED_RUNTIME_TAG d0a50523006fa6f283da6a36811081add3bb22fc)
127127

128128
set(UMF_BUILD_EXAMPLES OFF CACHE INTERNAL "EXAMPLES")
129129
# Due to the use of dependentloadflag and no installer for UMF and hwloc we need

sycl/test-e2e/XPTI/Inputs/test_collector.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ XPTI_CALLBACK_API void xptiTraceInit(unsigned int MajorVersion,
2121
std::string_view NameView{StreamName};
2222
using type = xpti::trace_point_type_t;
2323

24-
if (NameView == "ur") {
24+
if (NameView == "ur.call") {
2525
uint8_t StreamID = xptiRegisterStream(StreamName);
2626
for (type t : std::initializer_list<type>{type::function_with_args_begin})
2727
xptiRegisterCallback(StreamID, static_cast<uint16_t>(t), syclUrCallback);

sycl/test-e2e/XPTI/basic_event_collection_linux.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
#include "basic_event_collection.inc"
77
//
8-
// CHECK: xptiTraceInit: Stream Name = ur
8+
// CHECK: xptiTraceInit: Stream Name = ur.call
99
// CHECK: xptiTraceInit: Stream Name = sycl.experimental.mem_alloc
1010
// CHECK: xptiTraceInit: Stream Name = sycl
1111
// CHECK-NEXT: Graph create

sycl/tools/sycl-prof/collector.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ XPTI_CALLBACK_API void xptiTraceInit(unsigned int /*major_version*/,
7777
}
7878

7979
std::string_view NameView{StreamName};
80-
if (NameView == "ur") {
80+
if (NameView == "ur.call") {
8181
uint8_t StreamID = xptiRegisterStream(StreamName);
8282
xptiRegisterCallback(StreamID, xpti::trace_function_with_args_begin,
8383
urBeginEndCallback);

sycl/tools/sycl-sanitize/collector.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ XPTI_CALLBACK_API void xptiTraceInit(unsigned int /*major_version*/,
3232
unsigned int /*minor_version*/,
3333
const char * /*version_str*/,
3434
const char *StreamName) {
35-
if (std::string_view(StreamName) == "ur") {
35+
if (std::string_view(StreamName) == "ur.call") {
3636
uint8_t StreamID = xptiRegisterStream(StreamName);
3737
xptiRegisterCallback(StreamID, xpti::trace_function_with_args_begin,
3838
tpCallback);
@@ -45,7 +45,7 @@ XPTI_CALLBACK_API void xptiTraceInit(unsigned int /*major_version*/,
4545
}
4646

4747
XPTI_CALLBACK_API void xptiTraceFinish(const char *StreamName) {
48-
if (std::string_view(StreamName) == "ur") {
48+
if (std::string_view(StreamName) == "ur.call") {
4949
bool hadLeak = false;
5050
auto &GS = USMAnalyzer::getInstance();
5151
if (GS.ActivePointers.size() > 0) {

sycl/tools/sycl-trace/collector.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ XPTI_CALLBACK_API void xptiTraceInit(unsigned int /*major_version*/,
177177
unsigned int /*minor_version*/,
178178
const char * /*version_str*/,
179179
const char *StreamName) {
180-
if (std::string_view(StreamName) == "ur" &&
180+
if (std::string_view(StreamName) == "ur.call" &&
181181
std::getenv("SYCL_TRACE_UR_ENABLE")) {
182182
urPrintersInit();
183183
uint16_t StreamID = xptiRegisterStream(StreamName);
@@ -233,7 +233,7 @@ XPTI_CALLBACK_API void xptiTraceInit(unsigned int /*major_version*/,
233233
}
234234

235235
XPTI_CALLBACK_API void xptiTraceFinish(const char *StreamName) {
236-
if (std::string_view(StreamName) == "ur" &&
236+
if (std::string_view(StreamName) == "ur.call" &&
237237
std::getenv("SYCL_TRACE_UR_ENABLE"))
238238
urPrintersFinish();
239239
#ifdef SYCL_HAS_LEVEL_ZERO

sycl/tools/sycl-trace/main.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ int main(int argc, char **argv, char *env[]) {
2323
cl::values(
2424
// TODO graph dot
2525
// clEnumValN(PI, "plugin", "Trace Plugin Interface calls"),
26-
clEnumValN(UR, "ur", "Trace Unified Runtime calls"),
26+
clEnumValN(UR, "ur.call", "Trace Unified Runtime calls"),
2727
clEnumValN(ZE, "level_zero", "Trace Level Zero calls"),
2828
clEnumValN(CU, "cuda", "Trace CUDA Driver API calls"),
2929
clEnumValN(SYCL, "sycl", "Trace SYCL API calls"),

0 commit comments

Comments
 (0)