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

Race conditions in Vulkan? #191

Closed
kephale opened this issue Jun 28, 2018 · 4 comments
Closed

Race conditions in Vulkan? #191

kephale opened this issue Jun 28, 2018 · 4 comments
Assignees
Labels
needs-verification Issue needs verification

Comments

@kephale
Copy link
Member

kephale commented Jun 28, 2018

Vulkan fails to start with these as the last messages. By fails to start I mean that when opening SciView the render output never shows up. However, this only works when log level is info. When log level is debug the startup time is a bit slower, hence the suspicion of a race condition.
[ Note that this doesn't always happen, but it is repeatable ]

[SciJava-cfff6af-Thread-1] INFO VulkanRenderer - Recreating Swapchain at frame 0
[SciJava-cfff6af-Thread-1] INFO FXSwapchain - Created 2 swapchain images
[SciJava-2a073cbf-Thread-1] INFO VulkanRenderer - Recreating Swapchain at frame 0
[SciJava-2a073cbf-Thread-1] ERROR VulkanRenderer - Call to allocate image staging memory of size 1920000 failed: A device memory allocation has failed.

A related observation, sometimes when attempting to open volumes in Vulkan, they never show up (but other elements of the scene do show up).

@skalarproduktraum
Copy link
Member

Slightly strange... what version of Nvidia's driver are you using? And could you try running this with validation layers enabled (with Vulkan SDK installed, run scenery/sciview with -Dscenery.VulkanRenderer.EnableValidations=true)?

@skalarproduktraum skalarproduktraum added the needs-verification Issue needs verification label Jun 29, 2018
@skalarproduktraum skalarproduktraum self-assigned this Jun 29, 2018
@skalarproduktraum
Copy link
Member

Any more info on this, @kephale? Otherwise I'll close this because I cannot reproduce it.

@kephale
Copy link
Member Author

kephale commented Aug 20, 2018

I haven't seen this in a while, but I've added an unsightly amount of Thread.sleep calls to my scripts. IMHO there is still a notable design issue that needs to be addressed with the async implementation relative to user expectations of synchronous control. The most common glaring example of this is the need to use Thread.sleep when doing screenshots and such.

@skalarproduktraum
Copy link
Member

As there's no more information on this. I'll close it for the moment. Feel free to reopen in case this still occurs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-verification Issue needs verification
Projects
None yet
Development

No branches or pull requests

2 participants