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

GL 4.6: GL_PARAMETER_BUFFER and memory barriers #17

Closed
NicolBolas opened this issue Sep 27, 2017 · 7 comments
Closed

GL 4.6: GL_PARAMETER_BUFFER and memory barriers #17

NicolBolas opened this issue Sep 27, 2017 · 7 comments

Comments

@NicolBolas
Copy link

ARB_indirect_parameters (and the equivalent GL 4.6 specification) does not define a glMemoryBarrier bit that allows writes from image load/store and SSBOs to be visible to parameter buffer reads. And none of the existing barriers are stated to provide this functionality.

So at present, it seems impossible to use image load/store or SSBOs to write indirect parameters. Only atomic counters can write to them from shaders, assuming that is the intent of the atomic counter feature.

@ghost ghost assigned pdaniell-nv Oct 5, 2017
@nhaehnle
Copy link

nhaehnle commented Oct 6, 2017

This only affects the drawcount. COMMAND_BUFFER_BARRIER_BIT should be extended to mention MultiDraw*IndirectCount.

While we're at it, the type of the maxdrawcount of MultiDrawArraysIndirectCount should be sizei (like MultiDrawElementsIndirectCount).

@NicolBolas
Copy link
Author

I assume you mean GL_COMMAND_BARRIER_BIT.

@nhaehnle
Copy link

nhaehnle commented Oct 6, 2017

Yeah, obviously :)

@pdaniell-nv
Copy link

Discussed in the WG meeting today. We agree with Nicolai's suggestion. @nhaehnle can you make a proposed spec change here for Jon to incorporate? Thanks.

nhaehnle added a commit to nhaehnle/OpenGL-Registry that referenced this issue Oct 23, 2017
Otherwise, there is no way to reliably write to a PARAMETER_BUFFER_ARB
from a shader.

Issue: KhronosGroup/OpenGL-API#17
@nhaehnle
Copy link

nhaehnle commented Oct 23, 2017

Also sent a pull request for the spec internally. Not sure if that's the best procedure, but it's only a handful of words that change...

https://gitlab.khronos.org/opengl/API/merge_requests/4

@oddhack
Copy link
Contributor

oddhack commented Jan 23, 2018

This will be fixed in the next public spec update.

@oddhack oddhack closed this as completed Jan 23, 2018
@oddhack
Copy link
Contributor

oddhack commented Jan 23, 2018

(Coming soon).

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

No branches or pull requests

4 participants