Skip to content

Bug 2006718 - Fix some typos in StartupExcessiveResourceUseTest#15

Closed
moztcampbell wants to merge 1 commit into
mozilla-firefox:autolandfrom
moztcampbell:bug2005839
Closed

Bug 2006718 - Fix some typos in StartupExcessiveResourceUseTest#15
moztcampbell wants to merge 1 commit into
mozilla-firefox:autolandfrom
moztcampbell:bug2005839

Conversation

@moztcampbell
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

View this pull request in Lando to land it once approved.

@moztcampbell
Copy link
Copy Markdown
Contributor Author

Try (triggered manually on Lando page) is here: https://treeherder.mozilla.org/jobs?revision=ee64f506f6b81ca838811e762a3c3a1c60369f7b&repo=try

@moztcampbell moztcampbell requested a review from boek December 23, 2025 16:33
Copy link
Copy Markdown
Contributor

@boek boek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done!

@moztcampbell moztcampbell requested a review from a team December 23, 2025 16:37
Copy link
Copy Markdown
Contributor

@boek boek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve!

@lando-worker
Copy link
Copy Markdown

lando-worker Bot commented Dec 23, 2025

Pull request closed by commit 3d98859

lando-worker Bot pushed a commit that referenced this pull request Dec 23, 2025
@lando-worker lando-worker Bot closed this Dec 23, 2025
@moztcampbell moztcampbell deleted the bug2005839 branch December 23, 2025 17:30
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Jan 1, 2026
Pull request: mozilla-firefox/firefox#15

UltraBlame original commit: 350fb35b1d475e9fd2f9d63ad0ab5f0a839c57e4
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Jan 1, 2026
Pull request: mozilla-firefox/firefox#15

UltraBlame original commit: 350fb35b1d475e9fd2f9d63ad0ab5f0a839c57e4
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Jan 1, 2026
Pull request: mozilla-firefox/firefox#15

UltraBlame original commit: 350fb35b1d475e9fd2f9d63ad0ab5f0a839c57e4
akliuxingyuan pushed a commit to akliuxingyuan/firefox-android that referenced this pull request Feb 25, 2026
runlevel5 added a commit to runlevel5/firefox that referenced this pull request May 11, 2026
…"lvx-only dest write"

Twelve more probes building on the trunc_sat helper. Key results:

  mozilla-firefox#10  load-via-scratch loadUnalignedSimd128: 20/20 REPRO (no help)
  mozilla-firefox#15  compact single-write-to-dest SIMD: 10/10 REPRO at 30s
  mozilla-firefox#16  scalar body with correct lane mapping: 6/10 = 60% REPRO
  mozilla-firefox#17  sync;isync between two dest writes: 9/10 REPRO (no help)
  mozilla-firefox#18  single-instruction xvcvspsxws dest,src: 10/10 REPRO
  mozilla-firefox#19  3-insn body with 50 nops between dest writes: 8/10 REPRO
  mozilla-firefox#20  xvcvspsxws scratch + stxvd2x + lvx dest: 10/20 = 50% REPRO  *
  mozilla-firefox#21  NaN-correct probe mozilla-firefox#20 (3 SIMD ops on scratch): 7/10 REPRO

Probe mozilla-firefox#20 is the best partial fix. The KEY pattern: dest is written
exactly once, via Altivec lvx from memory, with the trunc_sat result
computed in scratch first. Adding more SIMD ops on scratch (probe mozilla-firefox#21)
hurts even though the dest-write count is unchanged.

NaN handling currently dropped (NaN -> INT32_MIN instead of NaN -> 0).
Visually breaks lc.wasm rendering but does not affect the OOB rate
measurement. Needs a wasm-correct re-derivation before landing.

Residual 50% fires at DIFFERENT bytecode locations (29750, 105808)
with the SAME captured vr5 = vperm-control byte pattern. Same root-cause
corruption manifesting through other Ion SIMD paths. Helper-by-helper
patching reduces but cannot eliminate -- the bug appears upstream of
multiple lowered SIMD ops.

Build verified clean, source committed for the ppc64 branch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants