Skip to content

openxr_data: improve error message when app is using wrong GPU - #297

Open
ImSapphire wants to merge 1 commit into
Supreeeme:mainfrom
ImSapphire:improve-multi-gpu-error
Open

openxr_data: improve error message when app is using wrong GPU#297
ImSapphire wants to merge 1 commit into
Supreeeme:mainfrom
ImSapphire:improve-multi-gpu-error

Conversation

@ImSapphire

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/openxr_data.rs Outdated
Comment thread src/openxr_data.rs
Comment on lines +441 to +446
let inst = unsafe {
ash::Instance::load(
ash::Entry::load().unwrap().static_fn(),
vk::Instance::from_raw(info.instance as _),
)
};

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Might be worth also just finding a way to pass through the vulkan instance to this function.

@ImSapphire ImSapphire Feb 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I can't really figure out a good way to do this because of how many abstractions the graphics backend and session restart code goes through. it seems like the VulkanData with the ash::Instance is dropped, so not sure that it's possible as-is?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

You should be able to pull it off the DynFrameController on the Compositor, with a simple match.

@ImSapphire
ImSapphire force-pushed the improve-multi-gpu-error branch from 9470ae0 to b068d8d Compare February 10, 2026 21:21
@ImSapphire
ImSapphire requested a review from Supreeeme February 16, 2026 19:54
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