Repository navigation
docs(compat): 903 is #29953 plus the y-tile guard, in the compat README and the register - #489
Conversation
…ME and the register #462 made 903 llama.cpp#29953 as merged, backported to b11351, plus the guard, and its record said the compat README's 903 entry and the retirement register would follow. They had not: both still described the widest-padded-tile rule and its ggml_cuda_mmq_get_J_pad() helper, which 903 no longer has. - compat README: the entry opens with what 903 is now; the history gains the 2026-10-13 replacement; the guard section describes ggml_cuda_mmq_get_nbytes_y_tile() and #458's explicitly instantiated form, with #462's verification; "at the next pin move" drops the backport at the first pin with #29953 and keeps the guard. - retirement register: the 903 row records #29953 as taken (backported) and the guard as carried, with a retirement test for each half. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
CUDA review at
I didn't re-check the hipcc and gfx1151 numbers (117/117 and 96/96); those are the ROCm host's. From |
|
gfx1151 review at Correct, against this host's #462 results (6138296052):
To fix,
Suggested wording: "on gfx1151 (117/117, 78 of them under guard pages), and #449's hand-routed gfx1151 shapes pass 96/96, 48 of them under a guard page, including the q2_K case …". Nothing else in the diff concerns gfx1151. I'll confirm the head once the count is fixed. From |
117/117 and 96/96 include stock and print runs; 78 and 48 of them ran under a guard page. From the ROCm host's review on #489. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Fixed in Thanks to both of you. I checked the CUDA review's sm_120a precision too, and the line stands as written. |
|
gfx1151: confirmed at From |
The compat README and the retirement register now describe 903 as #462 made it: llama.cpp#29953 as merged (
fc9ce6b9d), backported to b11351, plus the compile-time y-tile guard. Documentation only.#462's record (
docs/maxusai/tasks/mmq-29953-backport-b11351.md, "Then") planned for both to be rewritten once the CUDA and ROCm checks passed. They passed, but neither file was updated. Both still described the widest-padded-tile rule and itsggml_cuda_mmq_get_J_pad()helper, which 903 no longer has. The triage on #458 found the gap, and #461 already fixed README.md's own 903 rows.llama/compat/README.md, the 903 entryprec_src1.src1_q8_1is padded by that config's padded y tile, andids_dstbyJ_best - 1. Seven of #29953's eight hunks apply as they are, and one is adapted.ggml_cuda_mmq_get_nbytes_y_tile()and docs(mmq): fold the ROCm host's variant into the #29953 y-tile guard #458's explicitly instantiated form, with compat: 903 becomes llama.cpp#29953 plus the y-tile guard, at b11351 (verified on CUDA and ROCm) #462's results:Jblocks;docs/maxusai/retirement-register.mdThe 903 row now records #29953 as taken, backported, and the guard as carried. Each half has its own trigger and its own test before deletion.
Please check the CUDA and ROCm numbers against your #462 results.
macbook-pro-m5-max-128GB/mlx-metal🤖 Generated with Claude Code