Repository navigation
Conversation
Owner
Author
Reporter validation — no measurable benefit (2026-11-08)Tested Windows artifact from run 42427749053. Reporter observed no significant performance improvement and all saved photos corrupt. Importantly, this experiment did not include photo DMA readback fix #17, so photo corruption cannot be attributed to the texture upload change alone. With no observed benefit, do not merge; close the experiment, keep branch for reference. No release. |
This was referenced Nov 8, 2026
This was referenced Nov 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Isolated performance experiment
Port notsapinho/Eglantine commit 6e73445ccb, adapting the
UnswizzleLinearImagedeclaration to our baseline'sutil.hlayout. Read pitch-linear textures directly from guest memory into staging (avoid intermediate duplicate copy).No photo DMA readback fix is in this branch. This is a stand-alone performance A/B test against
nx2, not a candidate for direct merge without user testing. Build via read-onlyfork-windows-checkworkflow uploads a temporary Windows artifact and creates no release.Check Pokopia FPS, shader stalls, photos, scene appearance, and loading. User confirmed notsapinho build 24 reaches steady 60 FPS, but that is a combined result of multiple commits; this patch's effect is not yet verified in Frameropia.
Related #15. Do not merge or release yet.