Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

XCB window is mapped and flushed upon window creation #10

Closed
wants to merge 1 commit into from

Conversation

RedRingRico
Copy link

I had an issue when running samples 2 and beyond on Gentoo GNU/Linux with the GNOME 3 desktop. This seems to be specific to XCB. Mapping the window then flushing the connection appears to have resolved the issue.

@Ekzuzy
Copy link
Collaborator

Ekzuzy commented Jun 4, 2016

But the same code is called in a RenderingLoop(...) function. Did You try moving a the beginning of this function?

@RedRingRico
Copy link
Author

Sorry, I should have given more information. When I run sample 2, unmodified, I get the following in my terminal:
X Error of failed request: BadDrawable (invalid Pixmap or Window parameter)
Major opcode of failed request: 14 (X_GetGeometry)
Resource id in failed request: 0xc00000
Serial number of failed request: 35
Current serial number in output stream: 35

This happens during the call to vkGetPhysicalDeviceSurfaceCapabilitiesKHR in Tutuorial02::CreateSwapChain( ).

@s-ol
Copy link
Contributor

s-ol commented Nov 6, 2016

only the flush is required and the same problem happens with the xlib platform too. I didn't check for existing PRs before I submitted #15, that PR fixes both.

See this stackoverflow question: http://stackoverflow.com/questions/36486036/vulkan-on-x11-vkgetphysicaldevicesurfacecapabilitieskhr-error

@Ekzuzy
Copy link
Collaborator

Ekzuzy commented Nov 7, 2016

Sorry for keeping You waiting for such a long time. X11, XCB and Linux in general are not areas I know too good (if at all). And, unfortunately, other matters dragged me away from this issue. I tried to understand and investigate it and ask around what can cause this problem, but without any reasonable outcome. But the solution s-ol now presented looks much better and I will merge it soon, I just need to test it locally - though the problem itself didn't appear on the computers I have access to. RedRingRico, does the solution from #15 help You?

@RedRingRico
Copy link
Author

It works just fine. I propose you reject my pull request and accept s-ol's as it does the same job with less code.

@RedRingRico RedRingRico closed this Dec 2, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants