Skip to content

Request dedicated GPU from GPU drivers#850

Open
TorutheRedFox wants to merge 1 commit intosmartcmd:mainfrom
TorutheRedFox:use-dedicated-gpu
Open

Request dedicated GPU from GPU drivers#850
TorutheRedFox wants to merge 1 commit intosmartcmd:mainfrom
TorutheRedFox:use-dedicated-gpu

Conversation

@TorutheRedFox
Copy link

Description

Makes the game use the dedicated GPU rather than the integrated GPU if it's available.

Changes

Previous Behavior

Previously, the integrated GPU would be used unless an override was set in Windows' or the GPU driver's settings for the executable.

Root Cause

Missing exports for AMD PowerXpress and Nvidia Optimus requesting the use of the dedicated GPU for the process if available.

New Behavior

The dedicated GPU (if available) is used for the game instead of the integrated GPU.

Fix Implementation

Exports were added to Windows64_Minecraft.cpp to flag the executable as preferring the dedicated GPU.

AI Use Disclosure

No AI has been used in this PR.

@codeHusky
Copy link
Collaborator

Could you provide a demonstration video of this being effective?

@TorutheRedFox
Copy link
Author

Could you provide a demonstration video of this being effective?

not quite sure how to show it in a video aside from file handles to shader caches in processexplorer pointing to the nvidia folder rather than the intel one

@codeHusky
Copy link
Collaborator

You could show a video of the app running on the correct GPU index in task manager

@TorutheRedFox
Copy link
Author

about that, my task manager doesn't show that

@C0deProto
Copy link

C0deProto commented Mar 7, 2026

about that, my task manager doesn't show that

i can pull it into a test environment and see if it does
Edit: seem to be still relying on the integrated with the same gpu usage (or at least in my test between the current build and this PR) this would probably work as a fix for laptops but not desktops

@Fivebuss
Copy link

Fivebuss commented Mar 7, 2026

Maybe add a F3 menu piece that shows what GPU is in use?
That would be very nice

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