Repository navigation
Conversation
This was referenced Oct 18, 2026
Dripl1x
added a commit
to Dripl1x/AnyPS5
that referenced
this pull request
Oct 18, 2026
Complete the publication checklist and series cross-links for boykopovar#7474. No source or test changes.
|
Some comparisons could not be completed because a pull request conflicts with the base branch. Review these shared files manually before concluding that there is no overlap.
|
github-actions
Bot
added
libc
build
prx
agc
Complete the publication checklist and series cross-links for boykopovar#7474. No source or test changes.
Dripl1x
force-pushed
the
codex/ui-lab-immediate-exit-cache
branch
from
October 18, 2026 18:28
791d244 to
2f52825
Compare
This was referenced Oct 18, 2026
12 tasks done
Contributor
|
Intel and AMD Windows data: the 10 tests built from Windows 11 Pro 26300, WinLibs GCC 15.2.0 r7, Release, every execution test built,
The test files don't build on I also ran full CTest with this PR and the 14 other open AGC PRs I tested in the same batch merged together (#7647, #7641, #7630, #7627, #7603, #7600, #7591, #7579, #7535, #7533, #7462, #7050, #7048, #7650). Neither GPU showed a failure beyond AI-assisted: yes (Claude Code) |
This was referenced Oct 18, 2026
Merged
This branch has not been deployed
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
2 participants
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.
What
This follows the ten-item AnyPS5 graphics and memory-management performance series as item 11: a separate startup/cache correctness fix. It does not contribute to the FPS table.
Homebrew UI Lab can hit its submission-retirement timeout during a cold pipeline compile and exit immediately. Immediate guest exit terminates the host process without unwinding the Vulkan device, so its pipeline cache can be lost and the next launch repeats the failure. Register one host exit-drain callback, save the existing device's pipeline cache through SaveNow before process termination, and bound the drain to eight seconds. Reject null/conflicting registration, consume the callback once, report exceptions and preserve the final trace flush/termination path. The general immediate-exit behavior is tested independently of this title.
Tested
Rebased onto main
5057673804b28d217559010d7ac1f702824f240b; current head2f5282560358c1485c593bdbde7bc2691ca3dde0. The constructor conflict is resolved by preserving main'sMapDriverMemory()initialization and registering the cache-saving exit callback immediately after it. The owned seven-file change is retained. Runtime and title evidence below predates this rebase; no fresh build/test or FPS result is claimed for the rebased head.Windows 11 Pro build 26300 and native Ubuntu 26.04 on the ext4 mirror: 9/9 targeted exit tests passed on each OS. The new drain case covers delayed completion before final flush, a throwing callback and a callback that blocks past the eight-second bound. Existing requested/system/immediate/quick/nested exits, main return and finalizer-before-shutdown cases pass. The negative control without the callback drain fails. Conventions and whitespace checks pass.
Controlled UI Lab cache-state comparison on the pinned base: without the fix, two cold-start attempts fail; with it, the first run still exits with code 1, preserves completed cache work, and the second reaches rendering (28.63 seconds, 255 recorded presents). This fixes cache preservation across the initial failure; it does not remove the guest's cold-start timeout or make the first launch continue into the UI. Work that cannot finish its cache save inside eight seconds can still be lost. No FPS gain or complete startup-failure cure is claimed. GPU: NVIDIA RTX 3070 Laptop, NVIDIA 617.42, Windows 11 Pro build 26300.
Base
6447ed1ed7fd11c7eef60df336e8d66fac271188, local topic headcf0615a3c1a0f758628e4d75a7d1ae33ec8cadab. Runtime evidence predates this commit and is tied to its saved module hashes and the same owned seven-file source change. The Windows/Linux tests above were rerun before committing.Dependencies, series and overlap
No dependency on the ten performance patches; place this after them in the publication order. #6958 changes periodic cache snapshots and leaves SaveNow blocking; this fix calls SaveNow during a bounded immediate-exit drain. #6771 changes the cache file's versioned location, not immediate-exit persistence. Neither is claimed as this fix. The item-10 resident-import/artwork repair is a different bug and remains in item 10.
The overlap audit checked 787 open PRs and this branch merges cleanly into main
e5c2b3c325e485ba7fd417c47c46f48aa2c89115. #6160 inserts driver-memory initialization at the same constructor site; manual inspection shows a separate AGC FS-table implementation. Keep both initializers when resolving that textual conflict. #6248 supplies macOS executable-path handling in the shared exit test; #6044 relocates tests. These are integration overlaps, not exit-drain dependencies. Related issue #4399 describes the cold compile exceeding the guest retirement budget; this PR preserves cache work rather than removing that budget.The dedicated overlap report is retained in the common series evidence package. Measurements and focused tests refer to the pinned source revision above; no full-suite CI run is claimed. A Game bring-up discussion/title ID link has not been verified, so none is invented on the checklist.
Checklist
5057673804b2; constructor conflict resolved preserving both initializers; historical validation scope is listed aboveNotImplemented_nid_no_patch); silent stubs are listed under Silent stubs in their library's technical debt file.mdfiles or images added to the repository (attach them to this PR)The historical test snapshot above predates the current rebase; its results are not presented as tests of the rebased tree.
Series index
These are the ten performance topics from the same profiling effort, followed by a separate UI Lab cache-preservation fix. Membership in the series does not imply a dependency.
Reproduction evidence
Series traces, summary lines, module hashes, focused-test logs and UI contact sheets. Game measurements use the pinned base and source revisions described above; assets contain no game binaries.