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.

Sign up for GitHub

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

Jump to bottom

[NPU] Use run-time ze_loader for npu plugin #27659

Merged
merged 7 commits into from
Dec 4, 2024
Merged

[NPU] Use run-time ze_loader for npu plugin #27659

merged 7 commits into from
Dec 4, 2024

Conversation

XinWangIntel
Copy link
Contributor

@XinWangIntel XinWangIntel commented Nov 21, 2024 •
edited
Loading

Details:

  • Not build or export ze_loader
  • Create level-zero-headers to export ze headers that needed by extra modules

Tickets:

  • 157614

@github-actions github-actions bot added category: build OpenVINO cmake script / infra category: NPU OpenVINO NPU plugin labels Nov 21, 2024
@XinWangIntel XinWangIntel force-pushed the 157614-runtime-ze-loader branch from 3dfc73d to 188c204 Compare November 26, 2024 07:28
@github-actions github-actions bot added category: CI OpenVINO public CI github_actions Pull requests that update GitHub Actions code labels Nov 29, 2024
@XinWangIntel
Copy link
Contributor Author

build_jenkins

@XinWangIntel XinWangIntel force-pushed the 157614-runtime-ze-loader branch 3 times, most recently from ca12556 to d75790e Compare December 2, 2024 06:34
@XinWangIntel
Copy link
Contributor Author

build_jenkins

@XinWangIntel XinWangIntel marked this pull request as ready for review December 3, 2024 02:56
@XinWangIntel XinWangIntel requested review from a team as code owners December 3, 2024 02:56
@XinWangIntel XinWangIntel force-pushed the 157614-runtime-ze-loader branch from d75790e to f0672c4 Compare December 3, 2024 02:56
@ilya-lavrenov ilya-lavrenov added this to the 2025.0 milestone Dec 3, 2024
@XinWangIntel XinWangIntel force-pushed the 157614-runtime-ze-loader branch from f0672c4 to 39e4368 Compare December 3, 2024 08:28
@XinWangIntel
Remove ze deps
3551011 
Signed-off-by: Xin Wang <[email protected]>
@XinWangIntel
Remove install
cc77997 
Signed-off-by: Xin Wang <[email protected]>
@XinWangIntel
wrap ze to import and interface lib
336cc6e 
Signed-off-by: Xin Wang <[email protected]>
@XinWangIntel
Add level-zero-headers target
4db81b8 
Signed-off-by: Xin Wang <[email protected]>
@XinWangIntel
Remove ze_loader from workflow
1aa10b9 
Signed-off-by: Xin Wang <[email protected]>

Signed-off-by: Xin Wang <[email protected]>
@XinWangIntel
Restore env to support tbb
c2386c6 
Signed-off-by: Xin Wang <[email protected]>

Signed-off-by: Xin Wang <[email protected]>
@XinWangIntel XinWangIntel force-pushed the 157614-runtime-ze-loader branch from 39e4368 to d648a23 Compare December 3, 2024 09:04
@PatrikStepan PatrikStepan added this pull request to the merge queue Dec 4, 2024
Merged via the queue into openvinotoolkit:master with commit 99d7cd4 Dec 4, 2024
158 checks passed
@ilya-lavrenov ilya-lavrenov mentioned this pull request Jan 1, 2025
Merged
ilya-lavrenov added a commit that referenced this pull request Jan 4, 2025
@ilya-lavrenov
Fixed build with system level-zero (#28241)
65f6ce8 
### Details:
 - When system level-zero is used, we cannot rely on submodule.
- It's a regression after
#27659
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pereanub pereanub pereanub approved these changes

@StefaniaHergane StefaniaHergane StefaniaHergane approved these changes

Assignees

@pereanub pereanub

Labels
category: build OpenVINO cmake script / infra category: CI OpenVINO public CI category: NPU OpenVINO NPU plugin github_actions Pull requests that update GitHub Actions code
Projects
None yet
Milestone
2025.0
Development

Successfully merging this pull request may close these issues.
5 participants
@XinWangIntel @pereanub @StefaniaHergane @ilya-lavrenov @PatrikStepan