Skip to content

[HUMANMADE] texture_cache: keep the Switch 1 GOB order when requested - #865

Open
LukeWarnut wants to merge 11 commits into
Eglantine-Community:nx2-experimentalfrom
LukeWarnut:nx2
Open

LukeWarnut wants to merge 11 commits into
Eglantine-Community:nx2-experimentalfrom
LukeWarnut:nx2

Conversation

@LukeWarnut

@LukeWarnut LukeWarnut commented Nov 14, 2026 •

Copy link
Copy Markdown

Summary

The Switch 2 versions of Super Mario Odyssey and ARMS have their textures stored pre-tiled in the Switch 1 GOB layout. Their texture headers set word 2 bit 24 (load_store_hint in TICEntry), while native titles such as OoT leave it clear. The rest of the header matches. This bit is used as an indication to use the Switch 1 layout.

Type of change

  • Bug fix
  • Game compatibility / game-support
  • Performance
  • New feature
  • Refactor / cleanup
  • CI / build
  • Documentation

Testing

  • Tested on Windows
  • Tested on Linux
  • Tested on macOS
  • Tested on NVIDIA
  • Tested on AMD
  • Tested on Intel

Games tested:

ARMS (Switch 2) (5.5.1)
Super Mario Odyssey (Switch 2) (1.4.1)

Checklist

  • I built it locally and it compiles without new warnings
  • I did not add any copyrighted game data, keys or firmware
  • Docs updated if needed
  • I added AI Disclosure if one was used

Screenshots / logs

Before After
0400000000010000_2026-11-13_22-33-58-192

Summary by CodeRabbit

  • Bug Fixes
    • Improved handling of block-linear textures that use Maxwell GOB layout, including texture uploads and image conversions.
    • Cube-array textures now initialize with their configured dimensions and layer count without being blocked by the load/store hint.

PastaLaPate and others added 11 commits November 11, 2026 11:40
…kw-pipeline-cache-dir

video_core: Fix shader cache folder creation for MKW
…d-vk-hardening

vulkan: skip pipelines the driver refuses, harden texture cache and query copies
…evert-720-upstream/amd-vk-hardening

Revert "vulkan: skip pipelines the driver refuses, harden texture cache and query copies"
…evert-812-video_core/mkw-pipeline-cache-dir

Revert "video_core: Fix shader cache folder creation for MKW"
…abel it with its branch

Scheduled runs use the default branch's workflow files, so the daily build of
nx2-experimental was built with nx2's build-linux.yml. daily.yml now dispatches
ci.yml on the daily branch instead, so it builds with that branch's own files.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@github-actions
github-actions Bot changed the base branch from nx2 to nx2-experimental November 14, 2026 03:46
@github-actions

Copy link
Copy Markdown

Retargeted to nx2-experimental for pre-build testing. It will be promoted to nx2 after validation.

@coderabbitai

coderabbitai Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a392a76b-3cdf-45fb-a45e-4757b3c812af

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@aidankinzett

Copy link
Copy Markdown
Collaborator

hi could you rebase onto nx2-experimental

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.

3 participants