Skip to content

gc-ratchet: re-pin baseline on the dedicated bench host (ratchets #7443's retention win) - #7446

Merged
proggeramlug merged 1 commit into
mainfrom
gc/7443-baseline-pin
Aug 5, 2026
Merged

gc-ratchet: re-pin baseline on the dedicated bench host (ratchets #7443's retention win)#7446
proggeramlug merged 1 commit into
mainfrom
gc/7443-baseline-pin

Conversation

@proggeramlug

Copy link
Copy Markdown
Contributor

One-file baseline update, measured on the new dedicated pinned quiet host (perry-macos.local, M1/8 GB, Spotlight disabled; cpu_active 4-7% during capture) at main 5e236e6e2.

  • Locks 12_large_live_set heap_used at 59.9 MB (gc: old-generation hole free list — swept holes become reusable capacity (#7437) #7443's win) — under the previous 105.6 MB pin, regressing back to unreclaimable scattered-survivor retention would still pass the increase-direction band. Pinning the improvement is what makes the ratchet a ratchet.
  • All counter families bit-identical to the previous pin (0% spread over 7 repeats); RSS/wall medians re-baseline on the dedicated host (observed wall spread ~1-2%, far inside the pinned_host bands).
  • No code changes.

…tention win

Captured on the new pinned quiet host (Mac mini M1, 8 GB, Spotlight
disabled, cpu_active 4-7% at capture) at main 5e236e6. Locks
12_large_live_set's heap_used at 59.9 MB — under the previous 105.6 MB
pin, a regression back to unreclaimable scattered-survivor retention
would have PASSED the increase-direction band. Counter families are
unchanged from the previous pin (0% spread across 7 repeats); memory
and wall medians re-baseline on the dedicated host, whose observed wall
spread (~1-2%) is far inside the pinned_host bands.
@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

💤 Files selected but had no reviewable changes (1)
  • benchmarks/gc_ratchet/baseline/gc-ratchet-v1.json
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2f58cbdd-c8b8-43c1-9eb2-527d528ec1dc

📥 Commits

Reviewing files that changed from the base of the PR and between fa11b0e and 8f1f6f3.

📒 Files selected for processing (1)
  • benchmarks/gc_ratchet/baseline/gc-ratchet-v1.json

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@proggeramlug
proggeramlug merged commit 5b0930c into main Aug 5, 2026
19 of 45 checks passed
@proggeramlug
proggeramlug deleted the gc/7443-baseline-pin branch August 5, 2026 13:26
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.

1 participant