Skip to content

Commit 8396018

Browse files
committed
Clarify impl deferal of clUpdateMutableCommandsKHR
Address point 1) of KhronosGroup#1309 by clarifying when an implementation may defer propogating the changes from `clUpdateMutableCommandsKHR`. I belive the only semantic implication of this is the removing the restriction > This is only possible with a command-buffer created with the CL_COMMAND_BUFFER_SIMULTANEOUS_USE_KHR flag, as without this the enqueued command-buffer must complete before any modification occurs. So that implementations without simultaneous use support can still defer.
1 parent 6662fc3 commit 8396018

File tree

1 file changed

+4
-9
lines changed

1 file changed

+4
-9
lines changed

api/opencl_runtime_layer.asciidoc

+4-9
Original file line numberDiff line numberDiff line change
@@ -16199,15 +16199,10 @@ kernel commands using that object as an argument have had their arguments
1619916199
replaced with a different object.
1620016200
====
1620116201

16202-
To facilitate performant usage for pipelined work flows, where applications
16203-
repeatedly call command-buffer update then enqueue, implementations may
16204-
defer some of the work to allow {clUpdateMutableCommandsKHR} to return
16205-
immediately.
16206-
Deferring any recompilation until {clEnqueueCommandBufferKHR} avoids
16207-
blocking in host code and keeps device occupancy high.
16208-
This is only possible with a command-buffer created with the
16209-
{CL_COMMAND_BUFFER_SIMULTANEOUS_USE_KHR} flag, as without this the enqueued
16210-
command-buffer must complete before any modification occurs.
16202+
Implementations may defer some of the work to allow {clUpdateMutableCommandsKHR}
16203+
to return immediately, provided the user observable behavior in subsequent
16204+
command-buffer usage is specification conformant. Deferring work helps keep
16205+
device occupancy high by avoiding blocking in host code.
1621116206

1621216207
[open,refpage='clUpdateMutableCommandsKHR',desc='Modify configuration of mutable-command handles to update behavior for future enqueues',type='protos']
1621316208
--

0 commit comments

Comments
 (0)