-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[EXP][CMDBUF] L0 Immediate command-list support #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ZE2UR_CALL( | ||
zeCommandQueueExecuteCommandLists, | ||
(ZeCommandQueue, 1, &ZeCommandList, CommandListPtr->second.ZeFence)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think if we're now submitting the command list ourselves rather than through the queue we could probably not store it it in this iterator type to clean things up a little bit, since that was only necessary for that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The iterator still help us keep track of fences to release them in the command-buffer destructor. But it is true that we can also use a simpler container for that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed the way we track fences by using a vector instead of the CommandList map.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Adds support for L0 immediate command-list. The command-list containing the graph operations is still batch cmd-list but graphs can now be submitted even though immediate queue is requested by users. Prefix and Suffix additional command list types follow the Queue type.
2004e99
to
3933753
Compare
Upstream UR PR : oneapi-src#1218 |
Adds support for L0 immediate command-list.
The command-list containing the graph operations is still batch command-list but graphs can now be submitted even though immediate queue is requested by users.
Prefix and Suffix additional command list types follow Queue type.