Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Mac compatibility for Vulkan 1.3.216+ #3258

Closed
wants to merge 1 commit into from

Conversation

mirasrael
Copy link

Description

According to documentation https://vulkan.lunarg.com/doc/sdk/1.3.275.0/mac/getting_started.html since 1.3.216 VK_ERROR_INCOMPATIBLE_DRIVER should be provided to avoid VK_ERROR_INCOMPATIBLE_DRIVER error.

According to documentation https://vulkan.lunarg.com/doc/sdk/1.3.275.0/mac/getting_started.html since 1.3.216 VK_ERROR_INCOMPATIBLE_DRIVER should be provided to avoid VK_ERROR_INCOMPATIBLE_DRIVER error.
@mirasrael mirasrael force-pushed the fix-macos-compatibilty branch from d760b4f to 3ba1b30 Compare February 26, 2024 07:52
@Zorro666 Zorro666 self-requested a review February 26, 2024 09:23
Copy link
Collaborator

@Zorro666 Zorro666 left a comment

Choose a reason for hiding this comment

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

I think this change is not wanted to upstream and best to keep local to your fork.
The rationale being:

  1. Mac is not officially supported or maintained by RenderDoc.
  2. There is more that needs to be done to get Vulkan to work on Mac with latest MoltenVk. This change is the first problem and some of the future problems are strictly speaking MoltenVk driver bugs that have to be worked around in RenderDoc.
  3. Personally I have had MoltenVk specific changes like this (and more) on my fork for a long time and the code has not conflicted and it has not been a trouble to keep the changes locally.
  4. VK_KHR_portability_subset is described as a provisional extension and must be used with caution.

@Zorro666 Zorro666 closed this Feb 26, 2024
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