hakmem/docs/status/ENV_CLEANUP_TASK.md

ENV Cleanup Task - Phase 2 Complete

Last Updated: 2025-11-28 Branch: master Scope: Gate debug ENV variables behind !HAKMEM_BUILD_RELEASE

---

🎯 Task Summary

Successfully gated debug-only environment variables behind #if !HAKMEM_BUILD_RELEASE to eliminate getenv() overhead in production builds.

✅ Performance Results

Metric	Baseline	After Phase 1	After Phase 2	Status
Larson 1T (1 10 1 1000 100 10000 42)	30.2M ops/s	30.4M ops/s	30.4M ops/s	✅ Maintained
Build Status	Clean	Clean	Clean	✅ No warnings
Commits	-	6 atomic	9 atomic	✅ Incremental
ENV Variables Gated	-	3 vars	9 vars	✅ Phase 2 Done

Architecture: E1-CORRECT (Phase after 930c5283b Larson fix) Verification Method: Build + benchmark after each commit

---

📋 Work Completed

Phase 1: Core Debug Variables ✅ DONE

Step 1: core/tiny_debug.h

Commit: 3833d4e3e - ENV Cleanup Step 1 Performance: 30.0M ops/s Changes:

• Wrapped entire file with #if !HAKMEM_BUILD_RELEASE
• Added no-op stubs for release builds
• ENV Variables Gated:
  • HAKMEM_TINY_ALLOC_DEBUG (1 site)

Step 2a: core/hakmem_tiny_slow.inc

Commit: d6c2ea6f3 - ENV Cleanup Step 2a Performance: 30.5M ops/s (+0.5M) Changes: Gated debug dump on slow path failure (line 78) ENV Variables: Same as Step 1 (call site only)

Step 2b: core/tiny_superslab_free.inc.h

Commit: 0567e2957 - ENV Cleanup Step 2b Performance: 30.3M ops/s Changes: Gated debug dump in watch path (line 51) ENV Variables: Same as Step 1 (call site only)

Step 2c: core/hakmem_tiny_alloc.inc

Commit: 794bf996f - ENV Cleanup Step 2c Performance: 30.15M ops/s Changes: Gated debug dump on allocation failure (line 330) ENV Variables: Same as Step 1 (call site only)

Step 3: core/tiny_fastcache.h

Commit: 42747a108 - ENV Cleanup Step 3 Performance: 30.34M ops/s Changes: Gated profiling feature ENV Variables Gated:

• HAKMEM_TINY_PROFILE (1 site)

Step 4: core/tiny_region_id.h

Commit: 316ea4dfd - ENV Cleanup Step 4 Performance: 30.31M ops/s Changes: Gated watch address debug feature ENV Variables Gated:

• HAKMEM_WATCH_ADDR (1 site)

Phase 2: Low-Risk Debug Variables ✅ DONE

Step 5: core/ptr_trace.h

Commit: 35e8e4c34 - ENV Cleanup Step 5 Performance: 29.2M ops/s (-4% acceptable variance) Changes: Gated pointer trace debug infrastructure ENV Variables Gated:

• HAKMEM_PTR_TRACE_DUMP (1 site)
• HAKMEM_PTR_TRACE_VERBOSE (1 site)

Step 6: core/hakmem_debug.c

Commit: d0d2814f1 - ENV Cleanup Step 6 Performance: 30.3M ops/s Changes: Gated timing instrumentation ENV Variables Gated:

• HAKMEM_TIMING (1 site)

Step 7: core/box/free_local_box.c

Commit: cfa5e4e91 - ENV Cleanup Step 7 Performance: 30.4M ops/s (baseline match) Changes: Gated freelist diagnostic blocks ENV Variables Gated:

• HAKMEM_TINY_SLL_DIAG (2 additional sites)
• HAKMEM_TINY_FREELIST_MASK (1 site)
• HAKMEM_SS_FREE_DEBUG (1 site) Critical Fix: Wrapped entire diagnostic blocks to avoid scoping issues with static variables

---

📊 Statistics

Phase 1 + Phase 2 Combined

• Files Modified: 9 files
• Commits: 9 atomic commits
• ENV Variables Gated: 9 unique variables
  • HAKMEM_TINY_ALLOC_DEBUG (4 call sites)
  • HAKMEM_TINY_PROFILE (1 site)
  • HAKMEM_WATCH_ADDR (1 site)
  • HAKMEM_PTR_TRACE_DUMP (1 site)
  • HAKMEM_PTR_TRACE_VERBOSE (1 site)
  • HAKMEM_TIMING (1 site)
  • HAKMEM_TINY_SLL_DIAG (2 additional sites)
  • HAKMEM_TINY_FREELIST_MASK (1 site)
  • HAKMEM_SS_FREE_DEBUG (1 site)
• Performance Impact: +0.2M ops/s (+0.7% improvement from baseline 30.2M)
• Build Impact: 0 regressions, 0 new warnings

Verification Method

Each commit followed this workflow:

1. Edit single file with debug ENV gating
2. make clean && make -j8 larson_hakmem
3. ./larson_hakmem 1 10 1 1000 100 10000 42 2>/dev/null
4. Verify 25-35M ops/s range (baseline ±20%)
5. Atomic commit with performance data

---

🔍 Lessons Learned

What Worked ✅

1. Incremental Approach: One file per commit prevented bulk regressions
2. Build + Benchmark: Immediate verification after each change
3. No-op Stubs: Release builds compile cleanly without #ifdef cascades
4. Small Commits: Easy to identify and revert if issues occur

What Failed ❌ (Previous Attempt - Before Phase 1)

1. Bulk Changes: 69 variables in 2 commits caused 40x regression (30M → 0.8M ops/s)
2. Linker Errors: Gating function definitions without gating call sites
3. Background Benchmarks: Running 6+ benchmarks caused OOM (6.9GB)

What Failed ❌ (Phase 2 - Fixed)

1. Scoping Issues in free_local_box.c:
   • Problem: Gated only getenv calls, left static variables in #else branch
   • Symptom: Crash (exit 134) during benchmark
   • Fix: Wrap entire diagnostic blocks in #if !HAKMEM_BUILD_RELEASE
   • Lesson: When debug code has state (static vars, atomics), gate the entire block

Key Takeaway

"1からやりなおし" (Start over from scratch) - When performance regresses unexpectedly, reset to last known good state and retry incrementally. "Scope Entire Blocks" - Don't gate just getenv; gate all dependent code including static variables.

---

📁 Files Modified

Phase 1: Core Debug Infrastructure

• core/tiny_debug.h - Debug dump infrastructure (TINY_ALLOC_DEBUG)
• core/hakmem_tiny_slow.inc - Slow path debug dump call
• core/tiny_superslab_free.inc.h - Free path debug dump call
• core/hakmem_tiny_alloc.inc - Alloc failure debug dump call
• core/tiny_fastcache.h - FastCache profiling (TINY_PROFILE)
• core/tiny_region_id.h - Watch address debugging (WATCH_ADDR)

Phase 2: Low-Risk Debug Variables

• core/ptr_trace.h - Pointer trace debugging (PTR_TRACE_DUMP/VERBOSE)
• core/hakmem_debug.c - Timing instrumentation (TIMING)
• core/box/free_local_box.c - Freelist diagnostics (SLL_DIAG, FREELIST_MASK, SS_FREE_DEBUG)

---

🎯 Next Steps

Phase 3: SuperSlab Registry Debug Variables (Pending)

Target file: core/hakmem_super_registry.c/.h

ENV Variables Identified (8 vars):

• HAKMEM_SUPER_LOOKUP_DEBUG (header inline function)
• HAKMEM_SUPER_REG_DEBUG (register/unregister debug)
• HAKMEM_SS_LRU_DEBUG (3 call sites: evict_one, lru_pop, lru_push)
• HAKMEM_SUPERSLAB_MAX_CACHED (LRU configuration)
• HAKMEM_SUPERSLAB_MAX_MEMORY_MB (LRU configuration)
• HAKMEM_SUPERSLAB_TTL_SEC (LRU configuration)
• HAKMEM_SS_PREWARM_DEBUG (prewarm logging)
• HAKMEM_PREWARM_SUPERSLABS (prewarm count)

Estimated Impact: 8 variables, higher complexity (LRU + prewarm logic) Risk Level: Medium (touches hot path SuperSlab registry)

Phase 4: Medium-Risk Variables (Pending)

• core/front/tiny_heap_v2.h - HeapV2 feature flags
• core/page_arena.h - Page arena configuration
• Various _STATS and _DEBUG variables

Estimated Variables: 40-50 variables Risk Level: Medium (may affect hot paths)

Phase 5: Experimental Features (Pending - Investigation Needed)

• Ultra features: HAKMEM_TINY_ULTRA, ULTRA_VALIDATE, ULTRA_SLIM
• HeapV2: HAKMEM_TINY_FRONT_V2, HEAP_V2_CLASS_MASK
• BG system: HAKMEM_BATCH_BG, L25_BG_DRAIN

Status: Need investigation before deprecation Risk Level: High (may be production features)

---

✅ Completion Criteria

Phase 1 ✅ COMPLETE

• 6 core debug files gated
• All builds succeed with no new warnings
• Performance maintained at 30M ± 2% ops/s
• 6 atomic commits with verification data
• Documentation complete Status: ✅ COMPLETE (2025-11-28)

Phase 2 ✅ COMPLETE

• 3 low-risk debug files gated
• All builds succeed with no new warnings
• Performance maintained at 30M ± 2% ops/s
• 3 atomic commits with verification data
• Scoping issues fixed (free_local_box.c)
• Documentation updated Status: ✅ COMPLETE (2025-11-28)

---

📝 Related Documents

• docs/CONFIGURATION.md - ENV variable reference
• docs/status/CURRENT_TASK.md - Main task tracking
• PERFORMANCE_HISTORY_62M_TO_80M.md - Performance history

---

🔒 Safety Notes

DO NOT TOUCH (Production ENVs):

• core/hakmem_config.c - Production configuration
• Any _ENABLE variables that affect features
• Capacity/threshold tuning variables

Always Verify:

• Performance: 25-35M ops/s Larson range
• Build: Zero new warnings
• Functionality: Full benchmark suite (when available)