From 01aa67a7f2228da50131785829ef4362fea4855d Mon Sep 17 00:00:00 2001 From: worthant Date: Wed, 22 Jul 2026 05:49:25 +0300 Subject: [PATCH] smoke: fix CUDA device assert and tg128 column The CUDA device line in the bench log is 'Device 0: NVIDIA ...' with no 'cuda' on the same line, so the combined regex never matched and a fully working CUDA run (60k t/s pp512 on an RTX 5090) was reported as FAIL. Also the tg128 extraction read the test-name column instead of the t/s column. Both asserts validated against the real bench.log from that run. --- .github/scripts/gpu-smoke/remote-smoke.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/scripts/gpu-smoke/remote-smoke.sh b/.github/scripts/gpu-smoke/remote-smoke.sh index 5050a70ac92a..99c5d7c7e827 100755 --- a/.github/scripts/gpu-smoke/remote-smoke.sh +++ b/.github/scripts/gpu-smoke/remote-smoke.sh @@ -85,11 +85,12 @@ case "$BACKEND" in ;; linux-x64-cuda-13.3) grep -q "load_backend: loaded CUDA backend" bench.log || fail "CUDA backend not loaded" - grep -Eiq "cuda.*(NVIDIA|GeForce|RTX|H[0-9]+)" bench.log || fail "CUDA device is not an NVIDIA GPU" + # bench log format: " Device 0: NVIDIA GeForce RTX 5090, compute capability 12.0" + grep -Eq "Device [0-9]+: NVIDIA" bench.log || fail "CUDA device is not an NVIDIA GPU" ;; *) fail "unknown backend $BACKEND" ;; esac -TG=$(awk -F'|' '/tg128/ {gsub(/ /,"",$7); print $7}' bench.log | head -1) +TG=$(awk -F'|' '/tg128/ {gsub(/^ +| +$/,"",$8); split($8,a," "); print a[1]; exit}' bench.log) echo "tg128: ${TG:-?} t/s" echo "SMOKE OK: $BACKEND @ $TAG ($VERSION_LINE)"