Bug 94917

Summary:	radeonsi supports GL_ARB_shader_storage_buffer_object with 0 GL_MAX_COMBINED_SHADER_STORAGE_BLOCKS
Product:	Mesa	Reporter:	James Legg <jlegg>
Component:	Drivers/Gallium/radeonsi	Assignee:	Default DRI bug account <dri-devel>
Status:	VERIFIED FIXED	QA Contact:	Default DRI bug account <dri-devel>
Severity:	normal
Priority:	medium
Version:	git
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	Program which test for the issue

Description James Legg 2016-04-13 10:38:04 UTC

Created attachment 122892 [details]
Program which test for the issue

When mesa git fa46848e51a619aba5a748316fe8fe4c2e17d243 is built against LLVM 3.8.0, with a Radeon R9 270, support for the GL_ARB_shader_storage_buffer_object extension is claimed. However, GL_MAX_COMBINED_SHADER_STORAGE_BLOCKS is 0. The minimum required by the ARB_shader_storage_buffer_object specification is 8.

https://cgit.freedesktop.org/mesa/mesa/commit/?id=bfd11c599600960f966d5d217b8dc20442108ef1 makes it look as if support was only intended for LLVM 3.9 or later.

Comment 1 Nicolai Hähnle 2016-04-13 14:07:22 UTC

Hi James, thanks for the report. Probably some confusion in how the statetracker maps Gallium cap bits to extension support, I'm going to take a look.

Comment 2 Nicolai Hähnle 2016-04-13 14:17:36 UTC

https://patchwork.freedesktop.org/patch/80793/ should fix this.

Comment 3 Nicolai Hähnle 2016-04-13 15:13:37 UTC

Should be fixed in Mesa master, commit 9e9a2bb44aa7b317919793f2320bd995ff373bbb

Comment 4 James Legg 2016-04-14 08:38:31 UTC

Confirmed. Thanks for the quick fix.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.