Skip to content

Commit 2bb9744

Browse files
author
Ewan Crawford
committed
Rename urCommandBufferEnqueueExp to urEnqueueCommandBufferExp
The API to enqueue a closed command-buffer to a queue is defined in the YAML as a part of the command-buffer class, but it should be part of the enqueue class like other enqueue API extensions. This PR updates the YAML and regenerates UR code, making the associated changes to adapters and CTS. Closes #2600
1 parent 8e6ea72 commit 2bb9744

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

51 files changed

+376
-376
lines changed

‎include/ur_api.h‎

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -417,8 +417,6 @@ typedef enum ur_function_t {
417417
UR_FUNCTION_COMMAND_BUFFER_APPEND_USM_PREFETCH_EXP = 240,
418418
/// Enumerator for ::urCommandBufferAppendUSMAdviseExp
419419
UR_FUNCTION_COMMAND_BUFFER_APPEND_USM_ADVISE_EXP = 241,
420-
/// Enumerator for ::urCommandBufferEnqueueExp
421-
UR_FUNCTION_COMMAND_BUFFER_ENQUEUE_EXP = 242,
422420
/// Enumerator for ::urCommandBufferUpdateSignalEventExp
423421
UR_FUNCTION_COMMAND_BUFFER_UPDATE_SIGNAL_EVENT_EXP = 243,
424422
/// Enumerator for ::urCommandBufferUpdateWaitEventsExp
@@ -429,6 +427,8 @@ typedef enum ur_function_t {
429427
UR_FUNCTION_ENQUEUE_EVENTS_WAIT_WITH_BARRIER_EXT = 246,
430428
/// Enumerator for ::urPhysicalMemGetInfo
431429
UR_FUNCTION_PHYSICAL_MEM_GET_INFO = 249,
430+
/// Enumerator for ::urEnqueueCommandBufferExp
431+
UR_FUNCTION_ENQUEUE_COMMAND_BUFFER_EXP = 250,
432432
/// @cond
433433
UR_FUNCTION_FORCE_UINT32 = 0x7fffffff
434434
/// @endcond
@@ -7037,8 +7037,8 @@ typedef enum ur_command_t {
70377037
UR_COMMAND_READ_HOST_PIPE = 25,
70387038
/// Event created by ::urEnqueueWriteHostPipe
70397039
UR_COMMAND_WRITE_HOST_PIPE = 26,
7040-
/// Event created by ::urCommandBufferEnqueueExp
7041-
UR_COMMAND_COMMAND_BUFFER_ENQUEUE_EXP = 0x1000,
7040+
/// Event created by ::urEnqueueCommandBufferExp
7041+
UR_COMMAND_ENQUEUE_COMMAND_BUFFER_EXP = 0x1000,
70427042
/// Event created by ::urBindlessImagesWaitExternalSemaphoreExp
70437043
UR_COMMAND_EXTERNAL_SEMAPHORE_WAIT_EXP = 0x2000,
70447044
/// Event created by ::urBindlessImagesSignalExternalSemaphoreExp
@@ -10992,8 +10992,8 @@ UR_APIEXPORT ur_result_t UR_APICALL urCommandBufferAppendUSMAdviseExp(
1099210992
/// - ::UR_RESULT_ERROR_DEVICE_LOST
1099310993
/// - ::UR_RESULT_ERROR_ADAPTER_SPECIFIC
1099410994
/// - ::UR_RESULT_ERROR_INVALID_NULL_HANDLE
10995-
/// + `NULL == hCommandBuffer`
1099610995
/// + `NULL == hQueue`
10996+
/// + `NULL == hCommandBuffer`
1099710997
/// - ::UR_RESULT_ERROR_INVALID_COMMAND_BUFFER_EXP
1099810998
/// - ::UR_RESULT_ERROR_INVALID_QUEUE
1099910999
/// - ::UR_RESULT_ERROR_INVALID_EVENT
@@ -11003,11 +11003,11 @@ UR_APIEXPORT ur_result_t UR_APICALL urCommandBufferAppendUSMAdviseExp(
1100311003
/// + If event objects in phEventWaitList are not valid events.
1100411004
/// - ::UR_RESULT_ERROR_OUT_OF_HOST_MEMORY
1100511005
/// - ::UR_RESULT_ERROR_OUT_OF_RESOURCES
11006-
UR_APIEXPORT ur_result_t UR_APICALL urCommandBufferEnqueueExp(
11007-
/// [in] Handle of the command-buffer object.
11008-
ur_exp_command_buffer_handle_t hCommandBuffer,
11006+
UR_APIEXPORT ur_result_t UR_APICALL urEnqueueCommandBufferExp(
1100911007
/// [in] The queue to submit this command-buffer for execution.
1101011008
ur_queue_handle_t hQueue,
11009+
/// [in] Handle of the command-buffer object.
11010+
ur_exp_command_buffer_handle_t hCommandBuffer,
1101111011
/// [in] Size of the event wait list.
1101211012
uint32_t numEventsInWaitList,
1101311013
/// [in][optional][range(0, numEventsInWaitList)] pointer to a list of
@@ -13486,6 +13486,18 @@ typedef struct ur_enqueue_events_wait_with_barrier_ext_params_t {
1348613486
ur_event_handle_t **pphEvent;
1348713487
} ur_enqueue_events_wait_with_barrier_ext_params_t;
1348813488

13489+
///////////////////////////////////////////////////////////////////////////////
13490+
/// @brief Function parameters for urEnqueueCommandBufferExp
13491+
/// @details Each entry is a pointer to the parameter passed to the function;
13492+
/// allowing the callback the ability to modify the parameter's value
13493+
typedef struct ur_enqueue_command_buffer_exp_params_t {
13494+
ur_queue_handle_t *phQueue;
13495+
ur_exp_command_buffer_handle_t *phCommandBuffer;
13496+
uint32_t *pnumEventsInWaitList;
13497+
const ur_event_handle_t **pphEventWaitList;
13498+
ur_event_handle_t **pphEvent;
13499+
} ur_enqueue_command_buffer_exp_params_t;
13500+
1348913501
///////////////////////////////////////////////////////////////////////////////
1349013502
/// @brief Function parameters for urEnqueueCooperativeKernelLaunchExp
1349113503
/// @details Each entry is a pointer to the parameter passed to the function;
@@ -14165,18 +14177,6 @@ typedef struct ur_command_buffer_append_usm_advise_exp_params_t {
1416514177
ur_exp_command_buffer_command_handle_t **pphCommand;
1416614178
} ur_command_buffer_append_usm_advise_exp_params_t;
1416714179

14168-
///////////////////////////////////////////////////////////////////////////////
14169-
/// @brief Function parameters for urCommandBufferEnqueueExp
14170-
/// @details Each entry is a pointer to the parameter passed to the function;
14171-
/// allowing the callback the ability to modify the parameter's value
14172-
typedef struct ur_command_buffer_enqueue_exp_params_t {
14173-
ur_exp_command_buffer_handle_t *phCommandBuffer;
14174-
ur_queue_handle_t *phQueue;
14175-
uint32_t *pnumEventsInWaitList;
14176-
const ur_event_handle_t **pphEventWaitList;
14177-
ur_event_handle_t **pphEvent;
14178-
} ur_command_buffer_enqueue_exp_params_t;
14179-
1418014180
///////////////////////////////////////////////////////////////////////////////
1418114181
/// @brief Function parameters for urCommandBufferUpdateKernelLaunchExp
1418214182
/// @details Each entry is a pointer to the parameter passed to the function;

‎include/ur_api_funcs.def‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -132,6 +132,7 @@ _UR_API(urEnqueueReadHostPipe)
132132
_UR_API(urEnqueueWriteHostPipe)
133133
_UR_API(urEnqueueEventsWaitWithBarrierExt)
134134
_UR_API(urEnqueueKernelLaunchCustomExp)
135+
_UR_API(urEnqueueCommandBufferExp)
135136
_UR_API(urEnqueueCooperativeKernelLaunchExp)
136137
_UR_API(urEnqueueTimestampRecordingExp)
137138
_UR_API(urEnqueueNativeCommandExp)
@@ -181,7 +182,6 @@ _UR_API(urCommandBufferAppendMemBufferReadRectExp)
181182
_UR_API(urCommandBufferAppendMemBufferFillExp)
182183
_UR_API(urCommandBufferAppendUSMPrefetchExp)
183184
_UR_API(urCommandBufferAppendUSMAdviseExp)
184-
_UR_API(urCommandBufferEnqueueExp)
185185
_UR_API(urCommandBufferUpdateKernelLaunchExp)
186186
_UR_API(urCommandBufferUpdateSignalEventExp)
187187
_UR_API(urCommandBufferUpdateWaitEventsExp)

‎include/ur_ddi.h‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1117,6 +1117,12 @@ typedef ur_result_t(UR_APICALL *ur_pfnEnqueueKernelLaunchCustomExp_t)(
11171117
const size_t *, const size_t *, uint32_t, const ur_exp_launch_property_t *,
11181118
uint32_t, const ur_event_handle_t *, ur_event_handle_t *);
11191119

1120+
///////////////////////////////////////////////////////////////////////////////
1121+
/// @brief Function-pointer for urEnqueueCommandBufferExp
1122+
typedef ur_result_t(UR_APICALL *ur_pfnEnqueueCommandBufferExp_t)(
1123+
ur_queue_handle_t, ur_exp_command_buffer_handle_t, uint32_t,
1124+
const ur_event_handle_t *, ur_event_handle_t *);
1125+
11201126
///////////////////////////////////////////////////////////////////////////////
11211127
/// @brief Function-pointer for urEnqueueCooperativeKernelLaunchExp
11221128
typedef ur_result_t(UR_APICALL *ur_pfnEnqueueCooperativeKernelLaunchExp_t)(
@@ -1142,6 +1148,7 @@ typedef ur_result_t(UR_APICALL *ur_pfnEnqueueNativeCommandExp_t)(
11421148
/// @brief Table of EnqueueExp functions pointers
11431149
typedef struct ur_enqueue_exp_dditable_t {
11441150
ur_pfnEnqueueKernelLaunchCustomExp_t pfnKernelLaunchCustomExp;
1151+
ur_pfnEnqueueCommandBufferExp_t pfnCommandBufferExp;
11451152
ur_pfnEnqueueCooperativeKernelLaunchExp_t pfnCooperativeKernelLaunchExp;
11461153
ur_pfnEnqueueTimestampRecordingExp_t pfnTimestampRecordingExp;
11471154
ur_pfnEnqueueNativeCommandExp_t pfnNativeCommandExp;
@@ -1590,12 +1597,6 @@ typedef ur_result_t(UR_APICALL *ur_pfnCommandBufferAppendUSMAdviseExp_t)(
15901597
const ur_event_handle_t *, ur_exp_command_buffer_sync_point_t *,
15911598
ur_event_handle_t *, ur_exp_command_buffer_command_handle_t *);
15921599

1593-
///////////////////////////////////////////////////////////////////////////////
1594-
/// @brief Function-pointer for urCommandBufferEnqueueExp
1595-
typedef ur_result_t(UR_APICALL *ur_pfnCommandBufferEnqueueExp_t)(
1596-
ur_exp_command_buffer_handle_t, ur_queue_handle_t, uint32_t,
1597-
const ur_event_handle_t *, ur_event_handle_t *);
1598-
15991600
///////////////////////////////////////////////////////////////////////////////
16001601
/// @brief Function-pointer for urCommandBufferUpdateKernelLaunchExp
16011602
typedef ur_result_t(UR_APICALL *ur_pfnCommandBufferUpdateKernelLaunchExp_t)(
@@ -1639,7 +1640,6 @@ typedef struct ur_command_buffer_exp_dditable_t {
16391640
ur_pfnCommandBufferAppendMemBufferFillExp_t pfnAppendMemBufferFillExp;
16401641
ur_pfnCommandBufferAppendUSMPrefetchExp_t pfnAppendUSMPrefetchExp;
16411642
ur_pfnCommandBufferAppendUSMAdviseExp_t pfnAppendUSMAdviseExp;
1642-
ur_pfnCommandBufferEnqueueExp_t pfnEnqueueExp;
16431643
ur_pfnCommandBufferUpdateKernelLaunchExp_t pfnUpdateKernelLaunchExp;
16441644
ur_pfnCommandBufferUpdateSignalEventExp_t pfnUpdateSignalEventExp;
16451645
ur_pfnCommandBufferUpdateWaitEventsExp_t pfnUpdateWaitEventsExp;

‎include/ur_print.h‎

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2626,6 +2626,16 @@ urPrintEnqueueEventsWaitWithBarrierExtParams(
26262626
const struct ur_enqueue_events_wait_with_barrier_ext_params_t *params,
26272627
char *buffer, const size_t buff_size, size_t *out_size);
26282628

2629+
///////////////////////////////////////////////////////////////////////////////
2630+
/// @brief Print ur_enqueue_command_buffer_exp_params_t struct
2631+
/// @returns
2632+
/// - ::UR_RESULT_SUCCESS
2633+
/// - ::UR_RESULT_ERROR_INVALID_SIZE
2634+
/// - `buff_size < out_size`
2635+
UR_APIEXPORT ur_result_t UR_APICALL urPrintEnqueueCommandBufferExpParams(
2636+
const struct ur_enqueue_command_buffer_exp_params_t *params, char *buffer,
2637+
const size_t buff_size, size_t *out_size);
2638+
26292639
///////////////////////////////////////////////////////////////////////////////
26302640
/// @brief Print ur_enqueue_cooperative_kernel_launch_exp_params_t struct
26312641
/// @returns
@@ -3161,16 +3171,6 @@ urPrintCommandBufferAppendUsmAdviseExpParams(
31613171
const struct ur_command_buffer_append_usm_advise_exp_params_t *params,
31623172
char *buffer, const size_t buff_size, size_t *out_size);
31633173

3164-
///////////////////////////////////////////////////////////////////////////////
3165-
/// @brief Print ur_command_buffer_enqueue_exp_params_t struct
3166-
/// @returns
3167-
/// - ::UR_RESULT_SUCCESS
3168-
/// - ::UR_RESULT_ERROR_INVALID_SIZE
3169-
/// - `buff_size < out_size`
3170-
UR_APIEXPORT ur_result_t UR_APICALL urPrintCommandBufferEnqueueExpParams(
3171-
const struct ur_command_buffer_enqueue_exp_params_t *params, char *buffer,
3172-
const size_t buff_size, size_t *out_size);
3173-
31743174
///////////////////////////////////////////////////////////////////////////////
31753175
/// @brief Print ur_command_buffer_update_kernel_launch_exp_params_t struct
31763176
/// @returns

‎include/ur_print.hpp‎

Lines changed: 55 additions & 55 deletions
Original file line numberDiff line numberDiff line change
@@ -1159,9 +1159,6 @@ inline std::ostream &operator<<(std::ostream &os, enum ur_function_t value) {
11591159
case UR_FUNCTION_COMMAND_BUFFER_APPEND_USM_ADVISE_EXP:
11601160
os << "UR_FUNCTION_COMMAND_BUFFER_APPEND_USM_ADVISE_EXP";
11611161
break;
1162-
case UR_FUNCTION_COMMAND_BUFFER_ENQUEUE_EXP:
1163-
os << "UR_FUNCTION_COMMAND_BUFFER_ENQUEUE_EXP";
1164-
break;
11651162
case UR_FUNCTION_COMMAND_BUFFER_UPDATE_SIGNAL_EVENT_EXP:
11661163
os << "UR_FUNCTION_COMMAND_BUFFER_UPDATE_SIGNAL_EVENT_EXP";
11671164
break;
@@ -1177,6 +1174,9 @@ inline std::ostream &operator<<(std::ostream &os, enum ur_function_t value) {
11771174
case UR_FUNCTION_PHYSICAL_MEM_GET_INFO:
11781175
os << "UR_FUNCTION_PHYSICAL_MEM_GET_INFO";
11791176
break;
1177+
case UR_FUNCTION_ENQUEUE_COMMAND_BUFFER_EXP:
1178+
os << "UR_FUNCTION_ENQUEUE_COMMAND_BUFFER_EXP";
1179+
break;
11801180
default:
11811181
os << "unknown enumerator";
11821182
break;
@@ -9882,8 +9882,8 @@ inline std::ostream &operator<<(std::ostream &os, enum ur_command_t value) {
98829882
case UR_COMMAND_WRITE_HOST_PIPE:
98839883
os << "UR_COMMAND_WRITE_HOST_PIPE";
98849884
break;
9885-
case UR_COMMAND_COMMAND_BUFFER_ENQUEUE_EXP:
9886-
os << "UR_COMMAND_COMMAND_BUFFER_ENQUEUE_EXP";
9885+
case UR_COMMAND_ENQUEUE_COMMAND_BUFFER_EXP:
9886+
os << "UR_COMMAND_ENQUEUE_COMMAND_BUFFER_EXP";
98879887
break;
98889888
case UR_COMMAND_EXTERNAL_SEMAPHORE_WAIT_EXP:
98899889
os << "UR_COMMAND_EXTERNAL_SEMAPHORE_WAIT_EXP";
@@ -16174,6 +16174,53 @@ operator<<(std::ostream &os, [[maybe_unused]] const struct
1617416174
return os;
1617516175
}
1617616176

16177+
///////////////////////////////////////////////////////////////////////////////
16178+
/// @brief Print operator for the ur_enqueue_command_buffer_exp_params_t type
16179+
/// @returns
16180+
/// std::ostream &
16181+
inline std::ostream &
16182+
operator<<(std::ostream &os,
16183+
[[maybe_unused]] const struct ur_enqueue_command_buffer_exp_params_t
16184+
*params) {
16185+
16186+
os << ".hQueue = ";
16187+
16188+
ur::details::printPtr(os, *(params->phQueue));
16189+
16190+
os << ", ";
16191+
os << ".hCommandBuffer = ";
16192+
16193+
ur::details::printPtr(os, *(params->phCommandBuffer));
16194+
16195+
os << ", ";
16196+
os << ".numEventsInWaitList = ";
16197+
16198+
os << *(params->pnumEventsInWaitList);
16199+
16200+
os << ", ";
16201+
os << ".phEventWaitList = ";
16202+
ur::details::printPtr(
16203+
os, reinterpret_cast<const void *>(*(params->pphEventWaitList)));
16204+
if (*(params->pphEventWaitList) != NULL) {
16205+
os << " {";
16206+
for (size_t i = 0; i < *params->pnumEventsInWaitList; ++i) {
16207+
if (i != 0) {
16208+
os << ", ";
16209+
}
16210+
16211+
ur::details::printPtr(os, (*(params->pphEventWaitList))[i]);
16212+
}
16213+
os << "}";
16214+
}
16215+
16216+
os << ", ";
16217+
os << ".phEvent = ";
16218+
16219+
ur::details::printPtr(os, *(params->pphEvent));
16220+
16221+
return os;
16222+
}
16223+
1617716224
///////////////////////////////////////////////////////////////////////////////
1617816225
/// @brief Print operator for the
1617916226
/// ur_enqueue_cooperative_kernel_launch_exp_params_t type
@@ -18604,53 +18651,6 @@ operator<<(std::ostream &os, [[maybe_unused]] const struct
1860418651
return os;
1860518652
}
1860618653

18607-
///////////////////////////////////////////////////////////////////////////////
18608-
/// @brief Print operator for the ur_command_buffer_enqueue_exp_params_t type
18609-
/// @returns
18610-
/// std::ostream &
18611-
inline std::ostream &
18612-
operator<<(std::ostream &os,
18613-
[[maybe_unused]] const struct ur_command_buffer_enqueue_exp_params_t
18614-
*params) {
18615-
18616-
os << ".hCommandBuffer = ";
18617-
18618-
ur::details::printPtr(os, *(params->phCommandBuffer));
18619-
18620-
os << ", ";
18621-
os << ".hQueue = ";
18622-
18623-
ur::details::printPtr(os, *(params->phQueue));
18624-
18625-
os << ", ";
18626-
os << ".numEventsInWaitList = ";
18627-
18628-
os << *(params->pnumEventsInWaitList);
18629-
18630-
os << ", ";
18631-
os << ".phEventWaitList = ";
18632-
ur::details::printPtr(
18633-
os, reinterpret_cast<const void *>(*(params->pphEventWaitList)));
18634-
if (*(params->pphEventWaitList) != NULL) {
18635-
os << " {";
18636-
for (size_t i = 0; i < *params->pnumEventsInWaitList; ++i) {
18637-
if (i != 0) {
18638-
os << ", ";
18639-
}
18640-
18641-
ur::details::printPtr(os, (*(params->pphEventWaitList))[i]);
18642-
}
18643-
os << "}";
18644-
}
18645-
18646-
os << ", ";
18647-
os << ".phEvent = ";
18648-
18649-
ur::details::printPtr(os, *(params->pphEvent));
18650-
18651-
return os;
18652-
}
18653-
1865418654
///////////////////////////////////////////////////////////////////////////////
1865518655
/// @brief Print operator for the
1865618656
/// ur_command_buffer_update_kernel_launch_exp_params_t type
@@ -19859,6 +19859,9 @@ inline ur_result_t UR_APICALL printFunctionParams(std::ostream &os,
1985919859
os << (const struct ur_enqueue_events_wait_with_barrier_ext_params_t *)
1986019860
params;
1986119861
} break;
19862+
case UR_FUNCTION_ENQUEUE_COMMAND_BUFFER_EXP: {
19863+
os << (const struct ur_enqueue_command_buffer_exp_params_t *)params;
19864+
} break;
1986219865
case UR_FUNCTION_ENQUEUE_COOPERATIVE_KERNEL_LAUNCH_EXP: {
1986319866
os << (const struct ur_enqueue_cooperative_kernel_launch_exp_params_t *)
1986419867
params;
@@ -20033,9 +20036,6 @@ inline ur_result_t UR_APICALL printFunctionParams(std::ostream &os,
2003320036
os << (const struct ur_command_buffer_append_usm_advise_exp_params_t *)
2003420037
params;
2003520038
} break;
20036-
case UR_FUNCTION_COMMAND_BUFFER_ENQUEUE_EXP: {
20037-
os << (const struct ur_command_buffer_enqueue_exp_params_t *)params;
20038-
} break;
2003920039
case UR_FUNCTION_COMMAND_BUFFER_UPDATE_KERNEL_LAUNCH_EXP: {
2004020040
os << (const struct ur_command_buffer_update_kernel_launch_exp_params_t *)
2004120041
params;

0 commit comments

Comments
 (0)