Skip to content

Conversation

@kwasd
Copy link
Contributor

@kwasd kwasd commented Nov 6, 2025

When there are multiple libigc versions in a Windows driver store, Windows CI breaks:

LIBIGC1_VERSION=32.0.101.6737
32.0.101.8247
LEVEL_ZERO_VERSION=1.24.2
AGAMA_VERSION=32.0.101.6737
GPU_DEVICE=Intel(R) Arc(TM) 140V GPU (16GB)
TORCH_VERSION=Not installed
COMPILER_VERSION=2025.2.0
Error: Unable to process file command 'env' successfully.
Error: Invalid format '32.0.101.8247'

To fix this, we will get igc64.dll version from the currenlty loaded graphics driver.

Test run: https://github.com/intel/intel-xpu-backend-for-triton/actions/runs/19313141656/job/55238243773#step:9:41

resolves #5440

@kwasd kwasd requested a review from Copilot November 6, 2025 21:34

This comment was marked as off-topic.

@intel intel deleted a comment from Copilot AI Nov 6, 2025
@intel intel deleted a comment from Copilot AI Nov 6, 2025
@kwasd kwasd marked this pull request as draft November 6, 2025 23:24
@kwasd kwasd marked this pull request as ready for review November 10, 2025 20:09
@kwasd kwasd requested a review from anmyachev November 10, 2025 20:35
@kwasd kwasd enabled auto-merge (squash) November 10, 2025 20:36
@kwasd kwasd requested a review from Egor-Krivov November 10, 2025 21:10
pwsh -Command '
$igc = @(Get-Process -Name dwm -Module | Where-Object ModuleName -eq 'igc64.dll')
if ($igc.Count -gt 1) {
Write-Host "MULTIPLE"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this a possible scenario? I understand when it comes to DriverStore (previous approach), which can contain multiple drivers, but we've switched to an approach where the driver currently in use is requested. Is it possible that multiple drivers are in use at the same time?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, after updating the driver, but before reboot, It lists 2 versions of igc64 loaded for dwm.exe. I suppose this it is done on purpose to allow fast fallback on driver crashes.

For example, see #5440 (comment).

However your comment highlighted another issue: when an interactive session is present on machine, this will return "MULTIPLE" despite there's only one library, since we have 2 dwms in this case: one for a login screen, and one for a session. I'll fix it with something like sort -u

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Now it handles multiple objects

Copy link
Contributor

Choose a reason for hiding this comment

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

What do you plan to do if script return MULTIPLE? Will this be a signal that the machine needs to be rebooted?

Copy link
Contributor

Choose a reason for hiding this comment

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

ping @kwasd

@kwasd kwasd disabled auto-merge November 11, 2025 11:22
@kwasd kwasd enabled auto-merge (squash) November 12, 2025 21:39
@kwasd kwasd requested a review from whitneywhtsang November 12, 2025 21:39
@kwasd kwasd disabled auto-merge November 12, 2025 21:54
@kwasd kwasd enabled auto-merge (squash) November 12, 2025 21:59
@kwasd kwasd requested a review from anmyachev November 12, 2025 22:20
@kwasd kwasd merged commit 5c4937f into main Nov 13, 2025
80 of 87 checks passed
@kwasd kwasd deleted the fix/windows-libigc-version branch November 13, 2025 11:42
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.

[CI] capture-hw-details.sh: igc64.dll version retrieval is unreliable

3 participants