Skip to content

Conversation

kbenzie
Copy link
Contributor

@kbenzie kbenzie commented Jul 22, 2025

Automated changes by create-pull-request GitHub action

npmiller and others added 5 commits July 22, 2025 00:43
`UR_CHECK_ERROR` throws an exception so we can't use it from a
destructor.

Since devices only get released at application shutdown, there's not
much we can do if `nvmlShutdown` reports an error, and the OS will
promptly cleanup anything left as it tears down the process anyway.

So just skip checking the error code from `nvmlShutdown`, this is
already what we do for other cleanup/shutdown in this destructor.

Fixes: intel/llvm#19500
The following entry points are implemented here:
* `DEVICE_INFO_GLOBAL_VARIABLE_SUPPORT` (is now true)
* `urProgramGetGlobalVariablePointer`
* `urEnqueueDeviceGlobalVariableRead`
* `urEnqueueDeviceGlobalVariableWrite`

In addition, the enqueue interface has been moved out into a helper
function and the program handler now handles the `@global_id_mapping`
property.
The issue causing the crash upstream has been fixed (it was an unhandled
llvm::Error), so we can now delete events properly.
@kbenzie kbenzie requested a review from a team as a code owner July 22, 2025 00:43
Copy link
Contributor

Unified Runtime -> intel/llvm Repo Move Notice

Information

The source code of Unified Runtime has been moved to intel/llvm under the unified-runtime top-level directory,
all future development will now be carried out there. This was done in intel/llvm#17043.

The code will be mirrored to oneapi-src/unified-runtime and the specification will continue to be hosted at oneapi-src.github.io/unified-runtime.

The contribution guide will be updated with new instructions for contributing to Unified Runtime.

PR Migration

All open PRs including this one will be marked with the auto-close label and shall be automatically closed after 30 days.

Should you wish to continue with your PR you will need to migrate it to intel/llvm.
We have provided a script to help automate this process.

If your PR should remain open and not be closed automatically, you can remove the auto-close label.


This is an automated comment.

@kbenzie kbenzie merged commit 1b0d70e into main Jul 22, 2025
@kbenzie kbenzie deleted the mirror-commits- branch July 22, 2025 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants