Updating graphics drivers for Intel Gen7 GPUs does not solve the startup crash

[rocketguedes]

Bug Description

I cant start minecraft cause show this error telling me i have old driver for Intel, but I have a dedicated NVIDIA GTX 1050 ti, idk why sodium is using the integrated GPU, or caring about that.

My CPU i5 3570 have a HD Graphics 2500, but im not using it. Has no HDMI connect to it. I just use the QSV on OBS.

Reproduction Steps

Just start from the Official Minecraft Launcher (Fabric Loader + Fabric API + Sodium + Iris)

Log File

latest.log

Crash Report

no crash report, game not start

[pajicadvance]

We're already aware of this problem - a fix is currently in the works.

[jellysquid3]

The bug report's assumption as written is not correct. The problem is that Minecraft 1.20.5 changed the version of Oshi being used, and it now returns different version strings which Sodium doesn't know how to deal with. Not helping matters is that Intel uses both 10.18.10.5161 and 15.33.53.5161 in different places (why?!) to mean the same thing.

[rocketguedes]

The bug report's assumption as written is not correct. The problem is that Minecraft 1.20.5 changed the version of Oshi being used, and it now returns different version strings which Sodium doesn't know how to deal with. Not helping matters is that Intel uses both 10.18.10.5161 and 15.33.53.5161 in different places (why?!) to mean the same thing.

You're right! Using ATlauncher I created a 1.20.4 instance using the latest available mods versions and it has been started as expected with no crashes. The crash only happens in version 1.20.5.

[Felix14-v2]

Create a PR if this is so simple...

[IMS212]

The problem is that at the point this check runs, we cannot know what GPU Minecraft is running on. It's not smart to assume it's running on the better GPU.

[IMS212]

There is a config. You can pass in -Dsodium.checks.win32.intelGen7=false.

[jellysquid3]

You can be frustrated about not being able to use the mod due to a bug. But going on a tirade about how you supposedly know better, while having zero context as to why the code is written that way or the problems involved, is unacceptable.

The problem is not that we do not check for multiple GPUs. The problem is that OSHI (code that we do not write, and for which Minecraft updated) has changed the way they provide information about GPUs, and it no longer works reliably or returns the correct information. We have to replace a lot of code to fix these problems correctly.