From fb1e57f1bf9b4eaaebd24b83d73ece749e25ae32 Mon Sep 17 00:00:00 2001 From: Felipe Cardoso Date: Thu, 26 Mar 2026 19:23:03 +0100 Subject: [PATCH] feat: make llama-rocm-7.2 a required toolbox in benchmark setup MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ROCm 7.2 is now created alongside vulkan-radv during setup, giving Vulkan vs ROCm comparison in baseline and post-optimization benchmarks. Smoke test: ROCm 7.2 on Qwen3.5-0.8B → 8090 t/s pp512, 161 t/s tg128 (vs Vulkan: 8900 t/s pp512, 177 t/s tg128) Co-Authored-By: Claude Opus 4.6 (1M context) --- scripts/benchmark/setup.sh | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/benchmark/setup.sh b/scripts/benchmark/setup.sh index ba10d0e..818635f 100644 --- a/scripts/benchmark/setup.sh +++ b/scripts/benchmark/setup.sh @@ -15,8 +15,8 @@ log_header "Benchmark Setup" # ── 1. Check toolbox containers ────────────────────────── log_info "Checking toolbox containers..." -REQUIRED_TOOLBOXES=("llama-vulkan-radv") -OPTIONAL_TOOLBOXES=("llama-rocm-6.4.4" "llama-rocm-7.2" "llama-vulkan-amdvlk") +REQUIRED_TOOLBOXES=("llama-vulkan-radv" "llama-rocm-7.2") +OPTIONAL_TOOLBOXES=("llama-rocm-6.4.4" "llama-vulkan-amdvlk") existing=$(detect_toolbox_names 2>/dev/null || true) missing=()