[GL-C3] `LLImageGL` → `LLTextureData` + `LLPixelFormat`

sky_line commented

2026-09-27 05:33:52 +00:00

Owner

Scope: Keep the CPU pixels (mVKCpuData), discard state machine, serial/generation, setSubImageVK, replaceVKCpuData, alpha analysis and pick mask. Drop mTexName, bindless and GL upload. The LLPixelFormat enum (components, BGR order, type) replaces GL enums in LLImageGL/setExplicitFormat/the media plugin messages, with the translation at the plugin boundary. Material store fields become VK texture indices (deletes the getBindlessHandle calls).

Prereq: B4, B5

Acceptance: Textures, media on a prim (CEF/VLC/GStreamer), UI images, picking transparent HUDs, and the "cpu tex" memory line unchanged

Risk: high (L)

Shared acceptance baseline: links; logs in under VK and renders prims/mesh/rigged/terrain/water/sky/particles; click-select object/avatar/attachment; region crossing + teleport; no GLSL Compilation Error|SIGSEGV in the log.

Source: research/gl_removal_map.md

**Scope:** Keep the CPU pixels (`mVKCpuData`), discard state machine, serial/generation, `setSubImageVK`, `replaceVKCpuData`, alpha analysis and pick mask. Drop `mTexName`, bindless and GL upload. The `LLPixelFormat` enum (components, BGR order, type) replaces GL enums in `LLImageGL`/`setExplicitFormat`/the media plugin messages, with the translation at the plugin boundary. Material store fields become VK texture indices (deletes the `getBindlessHandle` calls). **Prereq:** B4, B5 **Acceptance:** Textures, media on a prim (CEF/VLC/GStreamer), UI images, picking transparent HUDs, and the "cpu tex" memory line unchanged **Risk:** high (L) **Shared acceptance baseline:** links; logs in under VK and renders prims/mesh/rigged/terrain/water/sky/particles; click-select object/avatar/attachment; region crossing + teleport; no `GLSL Compilation Error|SIGSEGV` in the log. Source: research/gl_removal_map.md

sky_line added this to the GL removal C: replace still-needed GL types milestone

2026-09-27 05:33:52 +00:00

sky_line added the

gl-removal

label

2026-09-27 05:33:52 +00:00

sky_line commented

2026-10-04 02:45:47 +00:00

Author

Owner

Status check (verification sweep): not done. LLImageGL is reduced to CPU pixels + VK bookkeeping (96fa180da0, v1.0.0-beta.1), but it's still named LLImageGL (229 lines across 41 files). LLTextureData and LLPixelFormat don't exist, and media plugins still use GL format enums (9ebabce7c7 defines GL_BGRA_EXT for them).

Status check (verification sweep): not done. LLImageGL is reduced to CPU pixels + VK bookkeeping (96fa180da0, v1.0.0-beta.1), but it's still named LLImageGL (229 lines across 41 files). LLTextureData and LLPixelFormat don't exist, and media plugins still use GL format enums (9ebabce7c7 defines GL_BGRA_EXT for them).

sky_line added this to the Slipstream project

2026-10-06 15:04:15 +00:00

sky_line added the

status

up-next

label

2026-10-10 05:06:52 +00:00

sky_line commented

2026-10-10 06:24:39 +00:00

Author

Owner

Verified against forgejo/main: PARTIAL, leaving open. Done: LLPixelFormat/LLPixelType enums exist (llcommon/llpixelformat.h) and replace GL enums in LLImageGL/setExplicitFormat; mTexName is gone. Remaining: LLImageGL is still the class (not LLTextureData) and still carries ~11 GL-REMOVAL-STUB blocks (llrender/llimagegl.h:64,80,108,111,130,163,195,218,294,372,410: createGLTexture/destroyGLTexture/getHasGLTexture/setFilteringOption/LLImageGLThread/sDefaultGLTexture etc.); bindless leftovers: LLCleanTexture::getBindlessHandle stub returning 0 (llcleantexture.h:41) with 4 call sites (llcleanmaterialstore.cpp:242-247), bindlessGeneration (llcleanmaterialstore.cpp:218); material store fields are still *_handle, not VK texture indices; LLImageGL name/GL-flavored API (createGLTexture, setGLTextureCreated) still used by lldynamictexture.cpp and others.

sky_line referenced this issue from a commit

2026-10-10 09:06:16 +00:00

GL-C3 (#72): delete the LLImageGL / LLGLTexture GL stubs; fold callers to the stub values

sky_line referenced this issue from a commit

2026-10-10 09:06:16 +00:00

GL-C3 (#72): material store texture slots are renderer texture indices, not GL bindless handles

sky_line referenced this issue from a commit

2026-10-10 09:06:16 +00:00

GL-C3 (#72): VK material mirror fields named *_tex (texture index + format code)

sky_line referenced this issue from a commit

2026-10-10 09:06:16 +00:00

GL-C3 (#72): LLImageGL -> LLTextureData (lltexturedata.{h,cpp}); own method names

sky_line commented

2026-10-10 09:06:28 +00:00

Author

Owner

Pushed to main at 07c8a3ac73. LLImageGL -> LLTextureData (renamed throughout), GL stubs deleted, material store fields renamed. To test: texture streaming, avatar bakes, previews, media, minimap and world map.

Pushed to main at 07c8a3ac73. `LLImageGL` -> `LLTextureData` (renamed throughout), GL stubs deleted, material store fields renamed. To test: texture streaming, avatar bakes, previews, media, minimap and world map.

sky_line added

and removed

labels

2026-10-10 09:06:28 +00:00

sky_line commented

2026-10-11 13:24:47 +00:00

Author

Owner