Skip to content

Conversation

@winstonzhang-intel
Copy link
Contributor

Based on the existence of NPU driver for L0, GPU flag is now passed to zeInit

@winstonzhang-intel winstonzhang-intel marked this pull request as ready for review September 20, 2023 20:14
@winstonzhang-intel winstonzhang-intel requested a review from a team as a code owner September 20, 2023 20:14
@winstonzhang-intel winstonzhang-intel marked this pull request as draft September 20, 2023 20:14
@winstonzhang-intel winstonzhang-intel marked this pull request as ready for review September 20, 2023 20:16
@jandres742
Copy link
Contributor

thanks @winstonzhang-intel . It looks good to me.

@smaslov-intel : what do you think?

@smaslov-intel
Copy link
Contributor

thanks @winstonzhang-intel . It looks good to me.

@smaslov-intel : what do you think?

Look OK for now, but how is the L0 NPU going to be enabled?

Based on the existence of NPU driver for L0, GPU flag is now passed to
zeInit

Signed-off-by: Zhang, Winston <winston.zhang@intel.com>
@winstonzhang-intel winstonzhang-intel temporarily deployed to WindowsCILock September 21, 2023 00:23 — with GitHub Actions Inactive
@winstonzhang-intel winstonzhang-intel temporarily deployed to WindowsCILock September 21, 2023 00:48 — with GitHub Actions Inactive
@jandres742
Copy link
Contributor

thanks @winstonzhang-intel . It looks good to me.
@smaslov-intel : what do you think?

Look OK for now, but how is the L0 NPU going to be enabled?

For now, official support in L0 adapter is for GPU devices. Any other L0 driver that we want to enable through the adapter would need first to have some separate testing from a branch, where we can confirm that it can work with the adapter and coexist with other drivers, and once we have confirmed that, we can move add the flag here officially.

@jandres742
Copy link
Contributor

@intel/llvm-gatekeepers : please merge when possible.

@aelovikov-intel aelovikov-intel merged commit 37b904f into intel:sycl Sep 22, 2023
veselypeta pushed a commit to veselypeta/llvm that referenced this pull request Sep 25, 2023
Based on the existence of NPU driver for L0, GPU flag is now passed to
zeInit

Signed-off-by: Zhang, Winston <winston.zhang@intel.com>
@winstonzhang-intel winstonzhang-intel deleted the zeinit_flag branch September 25, 2023 21:18
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