Skip to content

Add GL_MESA_texture_const_bandwidth#643

Merged
oddhack merged 1 commit intoKhronosGroup:mainfrom
robclark:const-bw-tiling
Mar 21, 2025
Merged

Add GL_MESA_texture_const_bandwidth#643
oddhack merged 1 commit intoKhronosGroup:mainfrom
robclark:const-bw-tiling

Conversation

@robclark
Copy link
Copy Markdown
Contributor

@robclark robclark commented Feb 3, 2025

GL_MESA_texture_const_bandwidth adds a way for an app to request the use of a format which does have data dependent bandwidth characteristics, in order to avoid a potential side-channel.

GL_MESA_texture_const_bandwidth adds a way for an app to request the use
of a format which does have data dependent bandwidth characteristics, in
order to avoid a potential side-channel.
@robclark
Copy link
Copy Markdown
Contributor Author

robclark commented Feb 3, 2025

fwiw, original MESA implementation is at https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25464

@Triang3l
Copy link
Copy Markdown

Triang3l commented Feb 5, 2025

The enumeration name finally chosen in Mesa, GL_CONST_BW_TILING_MESA, seems somewhat unusual considering that OpenGL doesn't seem to use such shortenings frequently. _CONSTANT is referenced 27 times in glext.h, while _CONST is used only for CONST_EYE in the ancient GL_NV_texture_shader. The original GL_CONSTANT_BANDWIDTH_TILING_MESA suggestion from that GitLab merge request seems much more consistent with the rest of OpenGL and more intuitive.

@zmike
Copy link
Copy Markdown
Contributor

zmike commented Mar 12, 2025

WG 12 March 2025: Approved to merge. @robclark should make any final changes then ping me again.

@robclark
Copy link
Copy Markdown
Contributor Author

So what is the verdict on naming. I can rename the enums, but renaming the extension would be an ABI break, since this extension has been in mesa for a while.

@zmike
Copy link
Copy Markdown
Contributor

zmike commented Mar 13, 2025

The verdict is do whatever you want.

@robclark
Copy link
Copy Markdown
Contributor Author

The verdict is do whatever you want.

I'd vote to land it as-is, since it has been in mesa for a while now

@zmike zmike added this to the Approved to Merge milestone Mar 17, 2025
@zmike
Copy link
Copy Markdown
Contributor

zmike commented Mar 17, 2025

@oddhack please merge

@oddhack oddhack merged commit ffa81e5 into KhronosGroup:main Mar 21, 2025
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

Successfully merging this pull request may close these issues.

4 participants