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

Is this intended to work on Windows #16

Closed
dneto0 opened this issue Feb 5, 2020 · 10 comments
Closed

Is this intended to work on Windows #16

dneto0 opened this issue Feb 5, 2020 · 10 comments

Comments

@dneto0
Copy link
Contributor

dneto0 commented Feb 5, 2020

I'm not asking for it to be supported. I just don't know if I should even try.

E.g. it's based on pthreads.h which is not native to Windows. There are emulation libraries, but I don't want to work to hard for that at the moment

@djdeath
Copy link
Member

djdeath commented Feb 5, 2020

It should, I think we even got it to compile on windows at some point.

@djdeath
Copy link
Member

djdeath commented Feb 5, 2020

To clarify the answer, this layer was part of the mesa repository which has mutex implementations for win32/linux so that made it possible.

Because I had to cut the dependencies this started relying on pthread.

The main reason this isn't written in c++ is that the layer is using embedded lists for efficiency. Not sure what would be the replacement with c++...

@gfxstrand
Copy link

If we can rely on C11, we can just use the built-in mutex stuff from C11.

@djdeath
Copy link
Member

djdeath commented May 12, 2020

Thanks, I did not know about that. Will update.

@gfxstrand
Copy link

Unfortunately, it doesn't look like visual studio supports actual C11. It just supports C11 features in C++. 😦

@gfxstrand
Copy link

Maybe clang on Windows would work with C11?

@sfricke-samsung
Copy link
Contributor

As a point of reference, the Sync2 layer is building on Windows

@mbechard
Copy link

mbechard commented Jul 7, 2021

I'm looking to use the timeline_semaphore layer on Windows to enable Vulkan support on GPUs as old as the Nvidia 880M. It seems like that layer doesn't compile on windows right now. Is this layer just not supported on Windows?

@djdeath
Copy link
Member

djdeath commented Jul 7, 2021

Not right now unfortunately.

@jeremyg-lunarg
Copy link
Collaborator

Closing. Timeline semaphore doesn't support windows. Sync2 and any future extensions supported should work on windows.

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

No branches or pull requests

6 participants