Repository navigation
ruby: backport final JIT review fixes to deployed LuaJIT omnibus - #77
Merged
dalehamel merged 0 commit intoAug 13, 2026
Merged
Conversation
dalehamel
force-pushed
the
dale/luajit-omnibus
branch
from
August 13, 2026 22:41
7c16777 to
19aafba
Compare
dalehamel
force-pushed
the
dale/luajit-omnibus-ruby-jit-review-fixes
branch
from
August 13, 2026 22:41
a94bd38 to
19aafba
Compare
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.
Purpose
Create a regression-testing copy of the currently deployed LuaJIT omnibus and backport only the final two Ruby JIT review fixes. PR #50 itself remains unchanged.
Verified deployed base
The production chain resolves to the same exact profiler revision:
container-os-tools-shopifyproduction manifest uses Prophiler imagee22944afb0c0e03641728a5e56188dd49ab6e2e3.profiles-processorcommit pinsebpf-profiler/opentelemetry-ebpf-profilerandlib/otel-errorsEXPECTED_COMMITto7c16777d686d62553714456f9ca8dfbfcb6a5fa8.7c16777dis the current head of LuaJIT unwinding — omnibus snapshot (tarantool + openresty, amd64/arm64) #50 (dale/luajit-omnibus).cos-toolbox-shopifyDaemonSets sampled across core-us, core-eu, and apps-us use immutable image digestsha256:606d8720…; Artifact Registry maps that digest to tage22944af…. All desired pods use that digest (core-us 824/824 ready; the other sampled fleets were fully updated with one node each in ordinary readiness churn).This PR is based directly on
7c16777d. It does not merge or rebase upstreammain; that synchronization is intentionally deferred.Backports
Exactly two source commits are added:
ruby: bound unlabeled JIT fallback to executable mappingsPR_SET_VMA-named reservations, including non-executable VMAs.ruby: report partial JIT mapping publicationNo generated ABI types, error catalog, eBPF source, or eBPF blobs changed.
Validation
process,processmanager,interpreter/ruby, and hermeticinterpreter/luajittests.fibintegration test passed with resolved Lua frames (2passing traces,0failures).git diff --checkpassed.