Repository navigation
Take the import census ahead of the engine, and stop re-walking a finished ladder (issue 105) - #114
Merged
Merged
Conversation
…ished ladder (issue 105) The build-283f3e9 report from the AU7200 has no engine imports line, and the trail says why: the launch ended within five seconds of the engine failing, on the first rung of a second walk the post-failure thread started behind the early walk's verdict, with the census queued behind it. The second launch of the build never reached the UI, so the only page the set serves is the no-provider page, which had no build line and nothing of the census. - NativeProbe: a walk whose verdict is already on the books says so and asks nothing; EngineFailedHere() and Standing() expose the ledger's state. - EngineImports: RunInBackground + WaitForCensus, claimed under the lock before the thread starts (the harness found the race). - BrowserApp (elm): on an install whose ledger records the engine's failure, the census runs after the walk and ahead of the engine, waited for up to 60 s, the way the walk is. - DiagServer: the no-provider page carries build, engine imports and the stub block. - Program (elm): a breadcrumb before Elementary.Initialize(), which is where the second launch's three lines stop in front of. - NuiMseWatch (issue 100): loadedmetadata carries the duration and the loadstart/playing lines say "live page" on a /live address, so the next freeze trail can say whether the element on screen held a live stream. Harnesses: probeladder (resume, peek, early, notearly updated), engineimports (background scenario), msewatch (duration line) all pass; all five packages and OverscanProbe build. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
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.
What the build-283f3e9 report said
The AU7200's page from
build-283f3e9(issue 105) has noengine imports:line. The previous-run block shows why: the early walk reached its verdict in 248 ms, the engine failed as before, and the post-failure thread started the walk again; its last line isprobe: open res/, with no header read behind it, no deadline miss five seconds later and no heartbeat tick ten seconds later. The launch was over within five seconds ofENGINE FAILURE, the Q80's shape, and the census was queued behind that second walk. The second launch (thethis runblock) shows three lines and no UI, exactly as thebuild-3996334page did, so on this set the only page served is the no-provider page, which carried neitherbuild :nor the census.Changes
NativeProbe: a walk whose verdict is already on the books (this launch or an earlier one) saysverdict already on the books — …; not walking againand asks nothing.Standing()andEngineFailedHere()expose the ledger's state.EngineImports:RunInBackgroundandWaitForCensus; the census is claimed under the lock before the thread starts.BrowserApp(elm): on an install whose ledger records the engine's failure, the census runs after the walk and ahead of the engine, waited for up to 60 s, on the walk's own reasoning.DiagServer: the no-provider page carriesbuild :,engine imports:and the stub block.Program(elm):Elementary: initializingbreadcrumb before the call the second launch stops in front of.NuiMseWatch(issue 100, FUSIONOF'sbuild-644c119trails):loadedmetadatacarriesdur …, andloadstart/playingsaylive pagewhile the address is/@user/live, read-only, so the next freeze trail can say whether the on-screen element held a live stream.docs/INTERNALS.md: new subsection The census has to go ahead of the engine too, state of App doesn't load on Samsung AU7200 (Tizen OS 6.0) #105 updated, the Videos stops playing after few seconds on tiktok site #100 bullet updated with the reading of both trails and what the next report decides.Checks
tools/probeladder/run.sh: all ten scenarios pass (resume,peek,early,notearlyupdated to the no-re-walk behaviour).tools/engineimports/run.sh: passes, with the newbackgroundscenario (which found the claim race in the first version).tools/msewatch/run.sh: passes, with the duration line held../build.sh alland theOverscanProbebuild: clean.🤖 Generated with Claude Code