Skip to content

Fix L0 dlopen #1358

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

Merged
merged 1 commit into from
Jun 9, 2025
Merged

Fix L0 dlopen #1358

merged 1 commit into from
Jun 9, 2025

Conversation

lukaszstolarczuk
Copy link
Contributor

No description provided.

@lukaszstolarczuk lukaszstolarczuk force-pushed the fix-l0-dlopen branch 2 times, most recently from 404cc1a to 5a1d869 Compare June 6, 2025 11:19
@lukaszstolarczuk lukaszstolarczuk changed the title [v0.11.x] Fix L0 dlopen - WIP [v0.11.x] Fix L0 dlopen Jun 6, 2025
@lukaszstolarczuk lukaszstolarczuk changed the base branch from v0.11.x to main June 6, 2025 14:14
@lukaszstolarczuk lukaszstolarczuk changed the title [v0.11.x] Fix L0 dlopen Fix L0 dlopen Jun 6, 2025
@lukaszstolarczuk lukaszstolarczuk marked this pull request as ready for review June 6, 2025 14:14
@lukaszstolarczuk lukaszstolarczuk requested a review from a team as a code owner June 6, 2025 14:15
unversioned lib '.so' is only available in the devel packages
@lukaszstolarczuk lukaszstolarczuk added this to the v0.11.x milestone Jun 6, 2025
@lukaszstolarczuk
Copy link
Contributor Author

tagged as v0.11.x milestone to potentially include this in a future patch release if any (from v0.11.x branch)

@ldorau ldorau merged commit 9879e8c into oneapi-src:main Jun 9, 2025
88 checks passed
@lukaszstolarczuk lukaszstolarczuk deleted the fix-l0-dlopen branch June 9, 2025 07:27
@vinser52
Copy link
Contributor

vinser52 commented Jun 9, 2025

Just curious, what is the reason?

@pbalcer
Copy link
Contributor

pbalcer commented Jun 9, 2025

Just curious, what is the reason?

Unversioned .so files are typically only installed alongside -dev packages. In this case, UMF required that level-zero-dev loader was installed. Normal runtime level-zero package wasn't enough (because it only installed the .so.1 and .so.1.minor files).

No one noticed because most everyone here has the dev package, but it broke on a CI system where we only had the minimum set of driver packages installed straight from apt.

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.

4 participants