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

[ll][WIP][Postponed] Vulkan: Add MoltenVK support #1570

Closed
wants to merge 5 commits into from
Closed

[ll][WIP][Postponed] Vulkan: Add MoltenVK support #1570

wants to merge 5 commits into from

Conversation

davll
Copy link
Contributor

@davll davll commented Oct 14, 2017

In order to run Vulkan on macOS/iOS, we should do the below:

  1. update Ash to support MVK extensions
  2. create surface from UIView / NSView

TODO

  • Compile
  • Test

Current Status

Failed to run example/hal/quad... :(

[mvk-info] MoltenVK version 0.18.2. Vulkan version 1.0.51.
[mvk-info] GPU device NVIDIA GeForce GT 750M supports the following Metal Feature Sets:
	OSX GPU Family 1 v2
	OSX GPU Family 1 v1
[mvk-info] GPU device Intel Iris Pro Graphics supports the following Metal Feature Sets:
	OSX GPU Family 1 v2
	OSX GPU Family 1 v1
AdapterInfo { name: "NVIDIA GeForce GT 750M", vendor: 0, device: 0, software_rendering: false }
AdapterInfo { name: "Intel Iris Pro Graphics", vendor: 0, device: 0, software_rendering: false }
[mvk-info] You require a license for the MoltenVK Vulkan Core feature set. Reverting to evaluation mode.
Segmentation fault: 11

It seems that somewhere in the Rust runtime has failed to do core::ptr::drop_in_place<alloc::boxed::Box<Any>>.

UPDATE:

The segmentation fault disappears when I commented out swapchain creation code.
functor.create_swapchain_khr
I'll investigate there later.

How to test

Download and install libMoltenVk.dylib to $HOME/.local/lib

cd examples/hal/quad
DYLD_LIBRARY_PATH=$HOME/.local/lib RUST_BACKTRACE=1 cargo run --features vulkan

Run debugger

DYLD_LIBRARY_PATH=$HOME/.local/lib RUST_BACKTRACE=1 /Applications/Xcode.app/Contents/Developer/usr/bin/lldb target/debug/quad

@davll
Copy link
Contributor Author

davll commented Oct 14, 2017

I don't think it is worth to keep developing and fixing the PR, and therefore decided to stop working on it. Hope someone can pick it up and continue on it.

@davll davll changed the title [ll][WIP] Vulkan: Add MoltenVK support [ll][WIP][Postponed] Vulkan: Add MoltenVK support Oct 14, 2017
@Skepfyr Skepfyr mentioned this pull request Apr 16, 2018
63 tasks
@kvark
Copy link
Member

kvark commented May 17, 2018

@davll is this feature desirable? Given how close our Metal backend now to be conformant, I wonder what value does MoltenVK integration bring to our Vulkan backend.

@davll
Copy link
Contributor Author

davll commented May 17, 2018

OK, I think it should be closed due to no activity.

@davll davll closed this May 17, 2018
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