-
Notifications
You must be signed in to change notification settings - Fork 12
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
Added gpu device option #17
base: master
Are you sure you want to change the base?
Conversation
I do want something like this. I too experimented with udev replacements (busybox's |
This code does work for me at least. And I'm using busybox mdev (though planning to switch to toybox), with libudev-zero. I kinda have a really weird setup as I've been creating a distro with no GNU runtime dependencies and this is the only compositor I've found that doesn't need any of the glib stuff. What's stopping the use of Vulkan's GPU discovery? I'm gessing that libdrm requires more information than Vulkan can provide. |
Right now for simplicity DRM and Vulkan open the GPU independently. This is a problem if the system has more than one GPU, because they might open different GPUs. There are two possibilities:
Either way, I need to do a bit more research. I think 1 is the way to go though. I'll turn this comment into an issue to keep track of this. |
@Ella-0 feel free to implement |
I'll look into doing that. |
This is a little hack and I'm not sure whether or not you want it merged. I need it as my operating system uses libudev-zero (a daemon-less libudev replacement) which doesn't implement all the features of udev so I have to manually specify the card to get it to work though it does work which I'm mildly surprised about. The thing that concerns me most in the patch is the duplicate branching in
drm_setup
to handle the freeing ofdevpath
though I can't think of a better way of handling it.