Skip to content

Conversation

@sarnex
Copy link
Contributor

@sarnex sarnex commented Mar 10, 2025

I talked to the KMD team and they recommended doing it this way for the xe module.

Maybe this will improve stability.

I hardcoded the PCI ID but we only have one Linux BMG runner so it should be fine for now.

Confirmed this works by checking the dmesg log of the runner.

Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
Signed-off-by: Sarnie, Nick <nick.sarnie@intel.com>
@sarnex sarnex temporarily deployed to WindowsCILock March 13, 2025 14:39 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 13, 2025 14:39 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 13, 2025 15:10 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 13, 2025 15:10 — with GitHub Actions Inactive
@sarnex sarnex temporarily deployed to WindowsCILock March 13, 2025 15:11 — with GitHub Actions Inactive
@sarnex sarnex marked this pull request as ready for review March 13, 2025 15:51
@sarnex sarnex requested review from a team as code owners March 13, 2025 15:51
// XFAIL: spirv-backend && run-mode
// XFAIL-TRACKER: CMPLRLLVM-64705

// XFAIL: linux && arch-intel_gpu_bmg_g21 && !igc-dev && run-mode
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird that these tests are failing due to GPU reset.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah its consistent too, i have absolutely no idea whats going on

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me look into this.

@sarnex sarnex merged commit 10e1d00 into intel:sycl Mar 13, 2025
32 of 33 checks passed
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