[NO MERGE] Integrate CK varlen cross attention for small-seq (s_q=1, s_kv<=16) by VeeraRajasekhar · Pull Request #461 · ROCm/TransformerEngine

VeeraRajasekhar · 2026-02-24T20:34:49Z

Integrate the CK team's unfused variable-length attention HIP kernels from varlen_attn/ into Transformer Engine's ROCm fused-attn path as a specialized path for specialized cross-attention (Q length 1, KV length 2-16, large batch)..

Add fused_attn_smallseq.hpp and fused_attn_smallseq.cpp under fused_attn_rocm/: declarations and implementation adapted from varlen_attn/attn_fwd.cpp and attn_bwd.cpp (scores, mask+softmax, output; grad_V, grad_attn, softmax bwd, grad_Q/grad_K). Runtime dispatch over max_seqlen_kv in {2,4,6,8,12,16}, head_dim 128, BF16.
Add fused_attn_smallseq.cpp to the ROCm fused-attn build in transformer_engine/common/CMakeLists.txt.
In fused_attn_ck_fwd: when THD and no bias, branch to small-seq path when max_seqlen_q==1 and 2<=max_seqlen_kv<=16. On shape query (Aux_CTX_Tensors->size == 0) skip get_runtime_max_seqlen (cu_seqlens pointers are null); use host max_seqlen_kv and set output_S to attention-weights shape {max_tokens_q, h_q, 1, runtime_max_seqlen_kv} and dtype QKV_type. On real run (size >= 2) call get_runtime_max_seqlen then fused_attn_smallseq_fwd. Use sequence count b_varlen = max_tokens_q (not segment count b) for get_runtime_max_seqlen, output_S shape, workspace size, and small-seq fwd so varlen kernel indexing matches Q and cu_seqlens_kv (THD may pass segment-level cu_seqlens; varlen kernel expects sequence-level batch).
In fused_attn_ck_bwd: same THD/small-seq condition. On workspace query (workspace->data.dptr == nullptr) skip get_runtime_max_seqlen and use host max_seqlen_kv; on real run call get_runtime_max_seqlen then fused_attn_smallseq_bwd. Use b_varlen = max_tokens_q_bwd for get_runtime_max_seqlen, workspace size, and small-seq bwd.
Reuse softmax LSE auxiliary buffer for attention weights in the small-seq path (forward write, backward read);
JAX attention.py: in NVTE_CK block, when THD and q_max_seqlen==1 and kv_max_seqlen<=16 set softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, kv_max_seqlen) and softmax_dtype = q_dtype so Python aux buffer matches C++ attention-weights convention.
Add test_ck_unfused_smallseq_backend in tests/jax/test_fused_attn.py (parametrized s_kv in {2,4,6,8,12,16}, b=30720, s_q=1, THD_THD_THD, SeqDescFormat.Seqlens) and optional NVTE_LOG_CK_SMALLSEQ debug logging in C++.

Description

Please include a brief summary of the changes, relevant motivation and context.

Fixes # (issue)

Type of change

Documentation change (change only to the documentation, either a fix or a new content)
Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Infra/Build change
Code refactoring

Changes

Please list the changes introduced in this PR:

Change A
Change B

Checklist:

I have read and followed the contributing guidelines
The functionality is complete
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
I have added tests that prove my fix is effective or that my feature works
New and existing unit tests pass locally with my changes

Integrate the CK team's unfused variable-length attention HIP kernels from varlen_attn/ into Transformer Engine's ROCm fused-attn path as a specialized path for specialized cross-attention (Q length 1, KV length 2-16, large batch).. - Add fused_attn_smallseq.hpp and fused_attn_smallseq.cpp under fused_attn_rocm/: declarations and implementation adapted from varlen_attn/attn_fwd.cpp and attn_bwd.cpp (scores, mask+softmax, output; grad_V, grad_attn, softmax bwd, grad_Q/grad_K). Runtime dispatch over max_seqlen_kv in {2,4,6,8,12,16}, head_dim 128, BF16. - Add fused_attn_smallseq.cpp to the ROCm fused-attn build in transformer_engine/common/CMakeLists.txt. - In fused_attn_ck_fwd: when THD and no bias, branch to small-seq path when max_seqlen_q==1 and 2<=max_seqlen_kv<=16. On shape query (Aux_CTX_Tensors->size == 0) skip get_runtime_max_seqlen (cu_seqlens pointers are null); use host max_seqlen_kv and set output_S to attention-weights shape {max_tokens_q, h_q, 1, runtime_max_seqlen_kv} and dtype QKV_type. On real run (size >= 2) call get_runtime_max_seqlen then fused_attn_smallseq_fwd. Use sequence count b_varlen = max_tokens_q (not segment count b) for get_runtime_max_seqlen, output_S shape, workspace size, and small-seq fwd so varlen kernel indexing matches Q and cu_seqlens_kv (THD may pass segment-level cu_seqlens; varlen kernel expects sequence-level batch). - In fused_attn_ck_bwd: same THD/small-seq condition. On workspace query (workspace->data.dptr == nullptr) skip get_runtime_max_seqlen and use host max_seqlen_kv; on real run call get_runtime_max_seqlen then fused_attn_smallseq_bwd. Use b_varlen = max_tokens_q_bwd for get_runtime_max_seqlen, workspace size, and small-seq bwd. - Reuse softmax LSE auxiliary buffer for attention weights in the small-seq path (forward write, backward read); - JAX attention.py: in NVTE_CK block, when THD and q_max_seqlen==1 and kv_max_seqlen<=16 set softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, kv_max_seqlen) and softmax_dtype = q_dtype so Python aux buffer matches C++ attention-weights convention. - Add test_ck_unfused_smallseq_backend in tests/jax/test_fused_attn.py (parametrized s_kv in {2,4,6,8,12,16}, b=30720, s_q=1, THD_THD_THD, SeqDescFormat.Seqlens) and optional NVTE_LOG_CK_SMALLSEQ debug logging in C++.

tests/jax/test_fused_attn.py

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp

transformer_engine/jax/cpp_extensions/attention.py

wangye805 · 2026-02-25T05:52:14Z

Let's make this PR work for jax extension first. Later we can support pytorch.

One key difference btw jax and pytorch fused-attn dispatch is that pytorch can calculate, request, and allocate softmax_aux, workspace during runtime with actual cu_seqlen_q/kv data. However, in jax extension, softmax_aux and workspace calculation is done in

TransformerEngine/transformer_engine/jax/cpp_extensions/attention.py

Lines 364 to 375 in b685686

    
               if backend == NVTE_Fused_Attn_Backend.NVTE_AOTriton: 
        
                   softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, config.max_segments_per_seq) 
        
                   softmax_dtype = dtypes.canonicalize_dtype(jnp.float32) 
        
               elif backend == NVTE_Fused_Attn_Backend.NVTE_CK: 
        
                   if config.qkv_layout.is_thd(): 
        
                       softmax_shape = (*batch_shape, q_max_seqlen, attn_heads, 1) 
        
                   else: 
        
                       softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, 1) 
        
                   softmax_dtype = dtypes.canonicalize_dtype(jnp.float32) 
        
               else: 
        
                   raise ValueError(f"Unsupported {backend=}") 
        
           softmax_aux_aval = q_aval.update(shape=softmax_shape, dtype=softmax_dtype)

and

TransformerEngine/transformer_engine/jax/cpp_extensions/attention.py

Line 394 in b685686

wkspace_info = transformer_engine_jax.get_fused_attn_fwd_workspace_sizes(

, without knowing actual runtime cu_seqlen_q/kv. Aux tensors are prepared in

TransformerEngine/transformer_engine/jax/csrc/extensions/attention.cpp

Line 36 in b685686

    
           void PrepareFusedAttnForwardAuxTensors(NVTETensorPack *tensor_pack, const size_t input_batch,

, also without the knowledge of runtime cu_seqlen_q/kv

General guideline:
1). Pre-allocate large enough softmax_aux and workspace ahead of time. Do not modify the aux preparation function or the c++ level aux workspace calculation/preparation, since we know our softmax aux and workspace size will be large enough for both flow, and the special flow only need a valid start pointer address.
2). During actual kernel dispatch, we do a seqlen_q/kv check, if it satisfy the special cross-attn condition, we launch it here
3). Use an env to guard this new flow and disable it when CP is used

transformer_engine/common/ck_fused_attn/src/ck_fused_attn_utils.cpp

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

transformer_engine/common/ck_fused_attn/src/ck_fused_attn_utils.cpp

wangye805 · 2026-02-26T02:05:03Z

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp

+  NVTE_CHECK(workspace != nullptr, "small-seq bwd requires workspace.");
+
+  float sqr_dk_scale = attn_scale;
+  hipStream_t hip_stream = reinterpret_cast<hipStream_t>(stream);


Probably no need for this cast. cudaStream_t will be hipified correctly to hipStream_t

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp

…port to small-seq kernels

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp

transformer_engine/jax/cpp_extensions/attention.py

transformer_engine/jax/csrc/extensions/attention.cpp

tests/jax/test_fused_attn.py

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

- tests/jax: CK small-seq tests use fixture to set/restore NVTE_FUSED_ATTN_CK_SMALLSEQ=1; parametrize dtype (BF16/FP16) and add sequence-packing cases (2048-2-4, 2-4096-8192); when env set, num_segments_per_seq = max_seqlen_q for THD else 2. - JAX attention.py: THD softmax shape/dtype uses small-seq path only when env=1, else original layout - JAX attention.cpp: Added env guard - fused_attn_smallseq: Use TRANSFORMER_ENGINE_TYPE_SWITCH_16BIT for fwd/bwd; add FP16 (__half) support; fix __half*float with T(scale).

VeeraRajasekhar · 2026-02-28T03:30:56Z

https://github.com/ROCm/TransformerEngine/actions/runs/22507932253/job/65210673064

CI run, no issues

wangye805 · 2026-03-02T01:49:46Z

ci/jax.sh

@@ -58,6 +58,7 @@ run_test_config() {
    run_default_fa 1 test_custom_call_compute.py
    run_default_fa 1 test_functions.py
    run 1 test_fused_attn.py


Do we need to skip the small seq tests for regular ck/aiter flow?

Yes, skipped with both v2 and v3

wangye805 · 2026-03-02T01:53:01Z

tests/jax/test_fused_attn.py

                )
-            self.seqlens_kv, self.offsets_kv = get_seqlens_and_offsets(self.segment_ids_kv)
+            else:
+                if is_hip_extension() and os.environ.get("NVTE_FUSED_ATTN_CK_SMALLSEQ", "0") == "1":


Emm, maybe we can make a dedicated generate_random_segment_ids_small_seq to separate it from the original generate_random_segment_ids function

Updated with a new function call which takes care of generating inputs for this new flow

wangye805 · 2026-03-02T01:57:17Z

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

+    }
+
+    if (runtime_max_seqlen_q == 1 && runtime_max_seqlen_kv >= 2 && runtime_max_seqlen_kv <= 16) {
+      fused_attn_rocm::fused_attn_smallseq_fwd(


If NVTE_LOG_CK_CONFIG=1, also log whether we are running the ck-small seq flow or regular ck/aiter flow

wangye805 · 2026-03-02T01:58:08Z

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

+    }
+
+    if (runtime_max_seqlen_q == 1 && runtime_max_seqlen_kv >= 2 && runtime_max_seqlen_kv <= 16) {
+


Same here for logging which bwd flow to run

wangye805 · 2026-03-02T02:02:00Z

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp

+    const T* V_ptr         = static_cast<const T*>(devPtrV);
+    T* O_ptr               = static_cast<T*>(devPtrO);
+    T* attn_workspace      = static_cast<T*>(attn_weights_buffer);
+    const int* cu_kv       = static_cast<const int*>(devPtrCuSeqlensKV);


Will it run into issues if we don't pass in cu_seqlen_q/cu_seqlen_q_padded?

For example, if there are several empty segments for q/kv but for all non-empty ones, s_q always equal to 1?

wangye805 · 2026-03-02T02:04:03Z

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

+
+    size_t runtime_max_seqlen_q = static_cast<size_t>(ck_fused_attn::get_runtime_max_seqlen(
+        static_cast<uint64_t>(b), devPtrCuSeqlensQ, nullptr, max_seqlen_workspace, stream));
+    size_t runtime_max_seqlen_kv = static_cast<size_t>(ck_fused_attn::get_runtime_max_seqlen(


Here we don't pass cu_seqlen_padded into runtime max seqlen check. What if max_seqlen without padding satisfy the ck kernel condition but with padding they do not? Can ck kernel handle those corner cases?

wangye805 · 2026-03-02T02:12:28Z

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.h

+#define TRANSFORMER_ENGINE_FUSED_ATTN_ROCM_FUSED_ATTN_SMALLSEQ_H_
+
+#include "../common.h"
+#include "transformer_engine/fused_attn.h"


Check for whether we really need those header files? In this .h or .cpp?

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp

tests/jax/test_fused_attn.py

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp

Micky774 · 2026-03-03T11:43:40Z

transformer_engine/jax/cpp_extensions/attention.py

+                        )
+                        ck_smallseq_softmax_aux_size = (
+                            batch_size * attn_heads * q_max_seqlen
+                            * min(kv_max_seqlen, 16) * 2


Looking at the implementation, we only support kv_max_seqlen<=16 right? So should this be checked via an assertion instead of enforced via min?

We cannot keep that cause, we care about run_time_max_seq_len, here we don't know the run_time_max_seqlen, for example

TransformerEngine/tests/jax/test_fused_attn.py

Lines 1280 to 1281 in 4537cce

pytest.param(2048, 2, 4, 16, 16, 128, 128, id="seqpack-2048-2-4-16-16-128-128"),

pytest.param(2, 4096, 8192, 16, 16, 128, 128, id="seqpack-2-4096-8192-16-16-128-128"),

this test cases s_kv is not 16 but the the num of segments and inputs are chosen in such a way that

TransformerEngine/transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

Lines 954 to 955 in 4537cce

size_t runtime_max_seqlen_kv = static_cast<size_t>(ck_fused_attn::get_runtime_max_seqlen(

b, devPtrCuSeqlensKV, nullptr, max_seqlen_workspace, stream));

this returns <=16

So this test would break for any case where the runtime_max_seqlen_kv is actually >16?

Micky774 · 2026-03-03T11:46:07Z

transformer_engine/jax/cpp_extensions/attention.py

+                        if ck_standard_softmax_aux_size >= ck_smallseq_softmax_aux_size:
+                            softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, 1)
+                            softmax_dtype = dtypes.canonicalize_dtype(q_dtype)
+                        else:
+                            softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, min(kv_max_seqlen, 16))
+                            softmax_dtype = dtypes.canonicalize_dtype(q_dtype)


Suggested change

if ck_standard_softmax_aux_size >= ck_smallseq_softmax_aux_size:

softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, 1)

softmax_dtype = dtypes.canonicalize_dtype(q_dtype)

else:

softmax_shape = (*batch_shape, attn_heads, q_max_seqlen, min(kv_max_seqlen, 16))

softmax_dtype = dtypes.canonicalize_dtype(q_dtype)

softmax_shape = (*batch_shape, attn_heads, q_max_seqlen)

softmax_dtype = dtypes.canonicalize_dtype(q_dtype)

if ck_standard_softmax_aux_size >= ck_smallseq_softmax_aux_size:

softmax_shape += (1,)

else:

softmax_shape += (min(kv_max_seqlen, 16),)

transformer_engine/jax/cpp_extensions/attention.py

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp

Micky774 · 2026-03-03T21:31:22Z

transformer_engine/jax/cpp_extensions/attention.py

+                        )
+                        ck_smallseq_softmax_aux_size = (
+                            batch_size * attn_heads * q_max_seqlen
+                            * min(kv_max_seqlen, 16) * 2


So this test would break for any case where the runtime_max_seqlen_kv is actually >16?

VeeraRajasekhar requested review from ipanfilo, wangye805 and wenchenvincent as code owners February 24, 2026 20:34

VeeraRajasekhar removed request for ipanfilo and wenchenvincent February 24, 2026 20:35

VeeraRajasekhar self-assigned this Feb 24, 2026

wangye805 requested changes Feb 25, 2026

View reviewed changes

Addressed comments

b3ef62c

wangye805 requested changes Feb 26, 2026

View reviewed changes

VeeraRajasekhar added 2 commits February 27, 2026 04:28

Addressed reviews

db685c4

Guard CK small-seq behind NVTE_FUSED_ATTN_CK_SMALLSEQ=1; add FP16 sup…

b6a5ee8

…port to small-seq kernels

wangye805 requested changes Feb 27, 2026

View reviewed changes

transformer_engine/common/fused_attn_rocm/fused_attn_ck.cpp Show resolved Hide resolved

VeeraRajasekhar added 3 commits February 27, 2026 18:16

Disabled xla_gpu_graph_level

c737072

Updated XLA_FLAGS in ci/jax.sh

4537cce

wangye805 requested changes Mar 2, 2026

View reviewed changes

transformer_engine/common/fused_attn_rocm/fused_attn_smallseq.cpp Outdated Show resolved Hide resolved

Micky774 requested changes Mar 3, 2026

View reviewed changes

Adressed comments

c6e0eae

VeeraRajasekhar force-pushed the veergopu/fused-varlen-ck-smallseq-integration branch from b5c5fb7 to c6e0eae Compare March 3, 2026 21:17

Micky774 requested changes Mar 3, 2026

View reviewed changes

Refactored input generation for smallseq flow

366945e

		}

		if (runtime_max_seqlen_q == 1 && runtime_max_seqlen_kv >= 2 && runtime_max_seqlen_kv <= 16) {

	pytest.param(2048, 2, 4, 16, 16, 128, 128, id="seqpack-2048-2-4-16-16-128-128"),
	pytest.param(2, 4096, 8192, 16, 16, 128, 128, id="seqpack-2-4096-8192-16-16-128-128"),

	size_t runtime_max_seqlen_kv = static_cast<size_t>(ck_fused_attn::get_runtime_max_seqlen(
	b, devPtrCuSeqlensKV, nullptr, max_seqlen_workspace, stream));

Conversation

VeeraRajasekhar commented Feb 24, 2026

Description

Type of change

Changes

Checklist:

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

wangye805 commented Feb 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

VeeraRajasekhar commented Feb 28, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

VeeraRajasekhar Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

wangye805 commented Feb 25, 2026 •

edited

Loading

VeeraRajasekhar Mar 3, 2026 •

edited

Loading