Skip to content
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

Importing external memory via named NT handle reference along with pointer reference #943

Closed
nikhiljnv opened this issue Jun 23, 2023 · 0 comments · Fixed by #1177 · May be fixed by KhronosGroup/OpenCL-CTS#1895
Closed

Comments

@nikhiljnv
Copy link
Contributor

cl_khr_external_memory
allows importing external memory specified via NT handle through CL_EXTERNAL_MEMORY_HANDLE_OPAQUE_WIN32_KHR
The spec says "CL_EXTERNAL_MEMORY_HANDLE_OPAQUE_WIN32_KHR specifies an NT handle ..."
This is mostly assuming reference to external memory is specified via void* NT handle pointer.
However, NT handle references can also be specified via named string.

This issue tracks discussion wrt supporting named NT handle references.

nikhiljnv added a commit that referenced this issue Nov 21, 2023
Currently, cl_khr_external_semaphore and cl_khr_external_memory
define properties to specify external win32 NT handle via pointer.
Win32 NT handles can also be specified via named strings.
Add properties to specify external Win32 NT handles via named strings too.

Fixes #943
nikhiljnv added a commit that referenced this issue May 21, 2024
Currently, cl_khr_external_semaphore and cl_khr_external_memory define properties
to specify external win32 NT handle via pointer. Win32 NT handles can also be specified
via named strings.

Add properties to specify external Win32 NT handles via named strings too.

Fixes #943
nikhiljnv added a commit that referenced this issue Jun 11, 2024
Address review comments.

Fixes #943
nikhiljnv added a commit that referenced this issue Jun 11, 2024
nikhiljnv added a commit that referenced this issue Jun 18, 2024
Address review comments from Ben to remove NULL
from named win32 handle description.

Fixes #943
nikhiljnv added a commit that referenced this issue Jun 18, 2024
Address review comments from Ben to remove NULL
from named win32 handle description.

Fixes #943
nikhiljnv added a commit that referenced this issue Jun 18, 2024
Latest updates to drop NULL from named handle type description.

Fixes #943
nikhiljnv added a commit that referenced this issue Jun 18, 2024
* Updates to win32 named handle type changes.

Address review comments.

Fixes #943

* Drop NULL from named win32 handle description.

Address review comments from Ben to remove NULL
from named win32 handle description.

Fixes #943
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant