diff options
author | Alan Wu <[email protected]> | 2023-03-02 16:21:05 -0500 |
---|---|---|
committer | GitHub <[email protected]> | 2023-03-02 16:21:05 -0500 |
commit | 34026afd0434fc482b2a6419ec6aeece07dcf52f (patch) | |
tree | 5114cfe00c280caa4648aa0c2af0f5d1aad3406e /yjit/src/invariants.rs | |
parent | a9f4e5cc6099f2b2f6202e5ac252edddc07492f8 (diff) |
YJIT: Delete stale `frozen_bytes` related code (#7423)
The code and comments in there have been disabled by comments for a long
time. The issues that the counter used to solve are now solved more
comprehensively by "runningness" [tracking][1] introduced by Code GC
and [delayed deallocation][2].
Having a single counter doesn't fit our current model where code pages
that could be touched or not are interleaved, anyway.
Just delete the code.
[1]: e7c71c6c9271b0c29f210769159090e17128e740
[2]: a0b0365e905e1ac51998ace7e6fc723406a2f157
Notes
Notes:
Merged-By: maximecb <[email protected]>
Diffstat (limited to 'yjit/src/invariants.rs')
-rw-r--r-- | yjit/src/invariants.rs | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/yjit/src/invariants.rs b/yjit/src/invariants.rs index 97a7b9b069..0a654b7555 100644 --- a/yjit/src/invariants.rs +++ b/yjit/src/invariants.rs @@ -546,17 +546,6 @@ pub extern "C" fn rb_yjit_tracing_invalidate_all() { cb.set_pos(old_pos); cb.set_dropped_bytes(old_dropped_bytes); - // Freeze invalidated part of the codepage. We only want to wait for - // running instances of the code to exit from now on, so we shouldn't - // change the code. There could be other ractors sleeping in - // branch_stub_hit(), for example. We could harden this by changing memory - // protection on the frozen range. - assert!( - CodegenGlobals::get_inline_frozen_bytes() <= old_pos, - "frozen bytes should increase monotonically" - ); - CodegenGlobals::set_inline_frozen_bytes(old_pos); - CodegenGlobals::get_outlined_cb() .unwrap() .mark_all_executable(); |