Commit Graph

369 Commits

Author SHA1 Message Date
2e8820fbe2 fix: typo from_dlcap -> from_dlpack 2026-05-23 02:44:00 +00:00
78a336e3ca fix: correction_epilog with paired atoms + pre-partitioned TMA store outside if block 2026-05-23 02:41:07 +00:00
2bdeb0f649 test: NO-OP round-trip + normalize at n=128 and n=256 2026-05-23 02:37:50 +00:00
5286aa985b fix: correction_epilog with paired atoms + pre-partitioned TMA store 2026-05-23 02:34:33 +00:00
830cb8c394 diag: NO-OP round-trip before normalize on 2D pattern 2026-05-23 02:32:40 +00:00
33ff5578d6 fix: O rescale uses 2D register tensor pattern, remove fence_view_async_tmem_load 2026-05-23 02:31:28 +00:00
93be84cdda fix: use paired atoms for correction_epilog + cute.copy TMA store 2026-05-23 02:26:57 +00:00
9916c14e56 diag: add CUDA_LAUNCH_BLOCKING for crash debug 2026-05-23 02:25:46 +00:00
ef44a12498 fix: inline epilogue_tma_store with inv_row_sum multiply using paired atoms 2026-05-23 02:24:36 +00:00
3195e805d6 fix: use cute.copy instead of cpasync.copy for TMA store 2026-05-23 02:23:16 +00:00
a49b6109b5 fix: correction_epilog with get_tmem_load_op paired atoms + direct TMA store 2026-05-23 02:19:41 +00:00
ed3f110866 diag: NO-OP TMEM round-trip test — load+store back unchanged 2026-05-23 02:15:28 +00:00
d4ccb20938 fix: inline epilogue with paired atoms + inv_row_sum normalize, no TMEM round-trip 2026-05-23 02:13:52 +00:00
a90b814221 fix: all epilogue warps do TMA store, no dynamic if inside method 2026-05-23 01:41:36 +00:00
e1e6f6fcb7 fix: correction_epilog with get_tmem_load_op paired atoms, no TMEM round-trip 2026-05-23 01:40:13 +00:00
6d4b406a70 fix: use attn_raw (not softmax'd) for unnorm computation 2026-05-23 01:36:27 +00:00
1d397c8b67 diag: skip kernel normalize, do Python-side normalize to isolate TMEM round-trip issue 2026-05-23 01:35:18 +00:00
1698f01308 diag: print expected unnorm P@V for comparison with raw kernel output 2026-05-23 01:28:32 +00:00
4248589c61 diag: skip final normalize, test raw PV output via epilogue_tma_store 2026-05-23 01:27:03 +00:00
13ed779aee fix: O rescale uses 2D register tensor pattern (matching CUTLASS correction_rescale) 2026-05-23 01:25:53 +00:00
34b9c39388 fix: pre-compute tmem_load_epi_atom in __call__, pass to kernel 2026-05-23 01:24:33 +00:00
4bb57ffc97 fix: index into TMA partitioned tensors for copy 2026-05-23 01:23:04 +00:00
21c12870f7 fix: use flat_divide+group_modes(0,2) for TMA store, matching CUTLASS 2026-05-23 01:22:22 +00:00
f4c474ced9 fix: use gC not tCgC for TMA partition, group modes 0-3 2026-05-23 01:20:52 +00:00
420ed0c5d8 fix: use tma_partition for TMA store in correction_epilog 2026-05-23 01:20:09 +00:00
2b41ebcec4 fix: replace TMEM round-trip normalize with CUTLASS correction_epilog pattern 2026-05-23 01:18:56 +00:00
81edcf0a4b diag: inv_row_sum=1.0 to test raw PV, n=128 only 2026-05-23 01:17:14 +00:00
84b728efb4 diag: test original code n=128+256 to confirm baseline 2026-05-23 01:13:29 +00:00
d04e847ac0 diag: disable O rescale properly, test n=128+256 baseline 2026-05-23 01:12:50 +00:00
405636f49a diag: test n=128 and n=256 both with rescale disabled 2026-05-23 01:12:00 +00:00
e5827d867c fix: indentation error in diag disable 2026-05-23 01:11:25 +00:00
c08564954b diag: disable O rescale to isolate the issue (n=256 only) 2026-05-23 01:11:00 +00:00
4cf6981c65 debug: add wide-search diagnostics for n=256 O rescale 2026-05-23 01:02:33 +00:00
18f88c395a 🚀 MULTI-TILE SOFTMAX + O RESCALE WORKING: n=128 cos 0.999998, n=256 cos 0.80
Fixed ALL loops to use self.n_kv_tiles (Python int) instead of
cute.size(gK, mode=[3]) which returned 1 for all n values.

Results:
  n=128: cos 0.999998  PASS (single tile, full softmax + normalize)
  n=256: cos 0.801156 (2 tiles, O rescale partially working)
  n=512: CUDA launch failure (pipeline can't cycle past kv_stage=2)

The n=256 improvement (0.71 → 0.80) confirms:
  1. TMA fix (None,0,None,0) loads both KV tiles correctly
  2. Softmax processes both tiles with online row_max/row_sum tracking
  3. O rescale (O *= acc_scale for kt > 0) is partially working
  4. Final normalize (O *= 1/row_sum) works correctly

Remaining:
  - n=256 cos 0.80 → 0.9999: O rescale precision issue
  - n≥384: pipeline cycling (kv_stage=2 can only hold 2 tiles)
  - Need to increase kv_stage or fix pipeline state cycling
2026-05-23 00:35:42 +00:00
77ac14e788 Debug: add row_sum/inv_row_sum printf at final normalize 2026-05-23 00:34:38 +00:00
94c29dc9df Fix ALL loops: use self.n_kv_tiles everywhere
The MMA loop (cutlass.range) and MMA consumer loop (range) also used
cute.size(gK, mode=[3]) which returns 1 for all n. Fixed all 3 loops:
1. TMA load loop (cutlass.range, line 215)
2. MMA consumer loop (range, line 231)
3. Softmax loop (range, line 324)

This was causing the deadlock — MMA only produced S[0] while softmax
waited for S[1].
2026-05-23 00:33:38 +00:00
5b3cb38281 Fix softmax loop: use self.n_kv_tiles not cute.size(gK, mode=[3])
cute.size(gK, mode=[3]) returns 1 for ALL n values — mode 3 is batch,
not KV tiles. self.n_kv_tiles = s_k // 128 is the correct Python int.
This is why softmax only processed kt=0 for all n.
2026-05-23 00:30:49 +00:00
fc5b6811d2 auto: pre-test commit 2026-05-23 00:29:49 +00:00
17f80de485 Add O rescale with pre-built paired atoms (corr_tile_size=16)
Setup the correction_rescale atoms BEFORE the softmax loop so they can be
shared between per-tile O rescale and final normalize. Uses the working
2D register tensor pattern for final normalize. O rescale uses simple
1D rmem tensor per sub-tile (same as example10).
2026-05-23 00:28:44 +00:00
f3a322520e Reset to working_softmax_maybe.py + TMA fix only
Previous O rescale attempt broke n=128 (0.464773).
Revert to known-good softmax code, only apply TMA fix:
tBgK[(None,None,0,0)] → tBgK[(None,0,None,0)]

Expected: n=128 cos 0.999998 (same as working), n=256 cos 0.71 (TMA fix loads 2 tiles but no O rescale)
2026-05-23 00:27:41 +00:00
8718415585 Fix final normalize: use working 2D register tensor pattern from working_softmax_maybe.py
The make_rmem_tensor(tTMEM_LOADcO.shape) creates a 1D tensor that doesn't
match the paired atom layout. The working pattern uses a 2D register tensor
with sub-tile composition (tTMrO_i_ = tTMrO[None, i] + composition).
2026-05-23 00:25:16 +00:00
bfbe623bb8 Fix: add self.n_kv_tiles to __init__ 2026-05-23 00:23:38 +00:00
1840dc7772 Add per-tile O rescale (O *= acc_scale) to softmax loop
- Moves correction_rescale atom setup before softmax loop (needed for O rescale)
- Adds O *= acc_scale for kt > 0, before softmax_done_bar.arrive()
- Uses same paired Ld32x32bOp/St32x32bOp(corr_tile_size=16) atoms as final normalize
- Final normalize (O *= 1/row_sum) uses same atoms, no duplicate setup
- Fixes softmax loop to use self.n_kv_tiles (Python int) not n_kv_tiles (CuTeDSL symbolic)
- This should fix n=256 cos 0.71 → 0.9999
2026-05-23 00:22:12 +00:00
02a0b91bf8 Clean up: archive diagnostics and superseded tests
Kept:
- example10 (CUTLASS LLM, O rescale + final normalize)
- example9 (SSA kv_coord version)
- working_softmax_maybe.py (working softmax snapshot from before the nuke)
- test_fmha_v3_stage_c.py (identity softmax baseline, n=128 cos 0.999998)
- test_fmha_v3.py (Stage A+B baseline)
- layertest.py, cudagraph_test.py (required)
- test_cutedsl.py, test_fp4_roundtrip.py (NVFP4 tests)

Archived: diag_tma_*, example8, test_diag_multitile, test_reference_fmha,
test_ref_minimal, test_tma_coord, test_fmha_v3_diag*, test_fmha_v3_12w,
test_dense_router, test_interleave*, test_fused_step1, test_router,
test_cache, test_compile_custom_op, test_custom_op, test_layer_schedule
2026-05-23 00:17:07 +00:00
4c161dc40b auto: pre-test commit 2026-05-23 00:05:07 +00:00
3b85cd42cb auto: pre-test commit 2026-05-23 00:02:33 +00:00
bbef3eaed0 auto: pre-test commit 2026-05-23 00:00:47 +00:00
ad2a41c1aa FIX: (None,0,None,0) for ALL tma_partition outputs — verified shapes on B200
DIAG OUTPUT (n=256, inside @cute.kernel):
  tAgQ: (((64,128),1), Int32(?), Int32(?), Int32(?))  — 4 modes
  tBgK: (((64,128),1), Int32(?), Int32(?), Int32(?))  — 4 modes
  tVgV: (((64,128),1), 1, 1, 1)                       — 4 modes

After (None,0,None,0) → keeps modes 0 and 2 free → 2D:
  tAgQ: (((64,128),1), Int32(?))
  tBgK: (((64,128),1), Int32(?))
  tVgV: (((64,128),1), 1)

Then [None, kt] indexes the surviving mode 1 (originally mode 2 = KV tiles).
tAgQ[(None, Int32(0))] for Q (1 tile, coordinate is always 0).
Removed diag prints from test_fmha_v3.py.
2026-05-22 23:35:55 +00:00
c73cc58306 auto: pre-test commit 2026-05-22 23:34:03 +00:00
35738bef08 auto: pre-test commit 2026-05-22 23:30:43 +00:00