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

Main l0 adapter release lib 0 8 x #1420

Closed

Conversation

nrspruit
Copy link
Contributor

@nrspruit nrspruit commented Mar 7, 2024

No description provided.

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
- In Windows, SYCL and UMF both define DLLMain such that a DLLMain
  for only the adapter's is not possible. To fix this, the
  L0 adapter inits the global adapter at variable init and
  registers an atexit teardown after refcnt == 0.

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
- To handle the customer usecases, don't return uinit when the adapter
  is used after the adapter library has already closed.
- Given the global adapter is gone, but get adapter is called, then a
  temp adapter struct is allocated and an ondemand atexit cleanup of
this memory is registered.

Signed-off-by: Neil R. Spruit <neil.r.spruit@intel.com>
@kbenzie
Copy link
Contributor

kbenzie commented Mar 8, 2024

I didn't notice this PR so did the cherry-pick resolution myself, looks like I did the exact same conflict resolution in #1422 but with the addition of reverting the v0.8.12 change the was decided not to be included.

@nrspruit
Copy link
Contributor Author

nrspruit commented Mar 8, 2024

Closing since this is done, thanks!

@nrspruit nrspruit closed this Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants