Skip to content

Commit c8567c7

Browse files
committed
rebase
Signed-off-by: Sergey V Maslov <[email protected]>
1 parent 20c49d8 commit c8567c7

File tree

2 files changed

+34
-25
lines changed

2 files changed

+34
-25
lines changed

sycl/include/sycl/ext/oneapi/backend/level_zero.hpp

+17-9
Original file line numberDiff line numberDiff line change
@@ -60,35 +60,39 @@ struct interop<backend::level_zero,
6060
using type = ze_image_handle_t;
6161
};
6262

63+
namespace ext {
64+
namespace oneapi {
6365
namespace level_zero {
6466
// Since Level-Zero is not doing any reference counting itself, we have to
6567
// be explicit about the ownership of the native handles used in the
6668
// interop functions below.
6769
//
6870
enum class ownership { transfer, keep };
6971
} // namespace level_zero
72+
} // namespace oneapi
73+
} // namespace ext
7074

7175
namespace detail {
7276

7377
template <> struct BackendInput<backend::level_zero, context> {
7478
using type = struct {
7579
interop<backend::level_zero, context>::type NativeHandle;
7680
std::vector<device> DeviceList;
77-
level_zero::ownership Ownership;
81+
ext::oneapi::level_zero::ownership Ownership;
7882
};
7983
};
8084

8185
template <> struct BackendInput<backend::level_zero, queue> {
8286
using type = struct {
8387
interop<backend::level_zero, queue>::type NativeHandle;
84-
level_zero::ownership Ownership;
88+
ext::oneapi::level_zero::ownership Ownership;
8589
};
8690
};
8791

8892
template <> struct BackendInput<backend::level_zero, event> {
8993
using type = struct {
9094
interop<backend::level_zero, event>::type NativeHandle;
91-
level_zero::ownership Ownership;
95+
ext::oneapi::level_zero::ownership Ownership;
9296
};
9397
};
9498

@@ -118,6 +122,8 @@ template <> struct InteropFeatureSupportMap<backend::level_zero> {
118122
};
119123
} // namespace detail
120124

125+
namespace ext {
126+
namespace oneapi {
121127
namespace level_zero {
122128
// Implementation of various "make" functions resides in libsycl.so and thus
123129
// their interface needs to be backend agnostic.
@@ -204,38 +210,40 @@ T make(const context &Context,
204210
Ownership == ownership::keep);
205211
}
206212
} // namespace level_zero
213+
} // namespace oneapi
214+
} // namespace ext
207215

208216
// Specialization of sycl::make_context for Level-Zero backend.
209217
template <>
210218
context make_context<backend::level_zero>(
211219
const backend_input_t<backend::level_zero, context> &BackendObject,
212220
const async_handler &Handler) {
213-
return level_zero::make_context(
221+
return ext::oneapi::level_zero::make_context(
214222
BackendObject.DeviceList,
215223
detail::pi::cast<pi_native_handle>(BackendObject.NativeHandle),
216-
BackendObject.Ownership == level_zero::ownership::keep);
224+
BackendObject.Ownership == ext::oneapi::level_zero::ownership::keep);
217225
}
218226

219227
// Specialization of sycl::make_queue for Level-Zero backend.
220228
template <>
221229
queue make_queue<backend::level_zero>(
222230
const backend_input_t<backend::level_zero, queue> &BackendObject,
223231
const context &TargetContext, const async_handler Handler) {
224-
return level_zero::make_queue(
232+
return ext::oneapi::level_zero::make_queue(
225233
TargetContext,
226234
detail::pi::cast<pi_native_handle>(BackendObject.NativeHandle),
227-
BackendObject.Ownership == level_zero::ownership::keep);
235+
BackendObject.Ownership == ext::oneapi::level_zero::ownership::keep);
228236
}
229237

230238
// Specialization of sycl::make_event for Level-Zero backend.
231239
template <>
232240
event make_event<backend::level_zero>(
233241
const backend_input_t<backend::level_zero, event> &BackendObject,
234242
const context &TargetContext) {
235-
return level_zero::make_event(
243+
return ext::oneapi::level_zero::make_event(
236244
TargetContext,
237245
detail::pi::cast<pi_native_handle>(BackendObject.NativeHandle),
238-
BackendObject.Ownership == level_zero::ownership::keep);
246+
BackendObject.Ownership == ext::oneapi::level_zero::ownership::keep);
239247
}
240248

241249
// TODO: remove this specialization when generic is changed to call

sycl/test/basic_tests/interop-level-zero-2020.cpp

+17-16
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,8 @@ int main() {
9494
// expected-warning@+2 {{'get_native' is deprecated: Use SYCL 2020 sycl::get_native free function}}
9595
// expected-warning@+1 {{'get_native<sycl::backend::ext_oneapi_level_zero>' is deprecated: Use SYCL 2020 sycl::get_native free function}}
9696
ZeProgram = Program.get_native<backend::ext_oneapi_level_zero>();
97-
// expected-warning@+2 {{'get_native' is deprecated: Use SYCL 2020 sycl::get_native free function}}
98-
// expected-warning@+1 {{'get_native<sycl::backend::ext_oneapi_level_zero>' is deprecated: Use SYCL 2020 sycl::get_native free function}}
97+
// expected-warning@+3 {{'get_native' is deprecated: Use SYCL 2020 sycl::get_native free function}}
98+
// expected-warning@+2 {{'get_native<sycl::backend::ext_oneapi_level_zero>' is deprecated: Use SYCL 2020 sycl::get_native free function}}
9999
/*ZeKernelBundle*/ (
100100
void)KernelBundle.get_native<backend::ext_oneapi_level_zero>();
101101

@@ -127,36 +127,37 @@ int main() {
127127

128128
backend_input_t<backend::ext_oneapi_level_zero, context> InteropContextInput{
129129
ZeContext, std::vector<device>(1, InteropDevice),
130-
level_zero::ownership::keep};
130+
ext::oneapi::level_zero::ownership::keep};
131131
context InteropContext =
132132
make_context<backend::ext_oneapi_level_zero>(InteropContextInput);
133133

134134
queue InteropQueue = make_queue<backend::ext_oneapi_level_zero>(
135-
{ZeQueue, level_zero::ownership::keep}, Context);
135+
{ZeQueue, ext::oneapi::level_zero::ownership::keep}, Context);
136136
event InteropEvent = make_event<backend::ext_oneapi_level_zero>(
137-
{ZeEvent, level_zero::ownership::keep}, Context);
137+
{ZeEvent, ext::oneapi::level_zero::ownership::keep}, Context);
138138
kernel_bundle<bundle_state::executable> InteropKernelBundle =
139139
make_kernel_bundle<backend::ext_oneapi_level_zero,
140140
bundle_state::executable>(ZeKernelBundle.front(),
141141
Context);
142142

143143
// Check deprecated
144144
// expected-warning@+1 {{'make<sycl::platform, nullptr>' is deprecated: Use SYCL 2020 sycl::make_platform free function}}
145-
auto P = level_zero::make<platform>(ZeDriver);
145+
auto P = ext::oneapi::level_zero::make<platform>(ZeDriver);
146146
// expected-warning@+1 {{'make<sycl::device, nullptr>' is deprecated: Use SYCL 2020 sycl::make_device free function}}
147-
auto D = level_zero::make<device>(P, ZeDevice);
147+
auto D = ext::oneapi::level_zero::make<device>(P, ZeDevice);
148148
// expected-warning@+1 {{'make<sycl::context, nullptr>' is deprecated: Use SYCL 2020 sycl::make_context free function}}
149-
auto C = level_zero::make<context>(std::vector<device>(1, D), ZeContext,
150-
level_zero::ownership::keep);
151-
// expected-warning@+2 {{'make<sycl::queue, nullptr>' is deprecated: Use SYCL 2020 sycl::make_queue free function}}
152-
auto Q =
153-
level_zero::make<queue>(Context, ZeQueue, level_zero::ownership::keep);
154-
// expected-warning@+2 {{'make<sycl::event, nullptr>' is deprecated: Use SYCL 2020 sycl::make_event free function}}
155-
auto E =
156-
level_zero::make<event>(Context, ZeEvent, level_zero::ownership::keep);
149+
auto C = ext::oneapi::level_zero::make<context>(
150+
std::vector<device>(1, D), ZeContext,
151+
ext::oneapi::level_zero::ownership::keep);
152+
// expected-warning@+1 {{'make<sycl::queue, nullptr>' is deprecated: Use SYCL 2020 sycl::make_queue free function}}
153+
auto Q = ext::oneapi::level_zero::make<queue>(
154+
Context, ZeQueue, ext::oneapi::level_zero::ownership::keep);
155+
// expected-warning@+1 {{'make<sycl::event, nullptr>' is deprecated: Use SYCL 2020 sycl::make_event free function}}
156+
auto E = ext::oneapi::level_zero::make<event>(
157+
Context, ZeEvent, ext::oneapi::level_zero::ownership::keep);
157158
// expected-warning@+2 {{'program' is deprecated: program class is deprecated, use kernel_bundle instead}}
158159
// expected-warning@+1 {{'make<sycl::program, nullptr>' is deprecated: Use SYCL 2020 sycl::make_kernel_bundle free function}}
159-
auto PR = level_zero::make<program>(Context, ZeProgram);
160+
auto PR = ext::oneapi::level_zero::make<program>(Context, ZeProgram);
160161

161162
return 0;
162163
}

0 commit comments

Comments
 (0)