[https://nvbugs/6162860][fix] Set free_gpu_memory_fraction=0.6 only when torch_compile=True for test_bfloat16_#14109
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThis PR adjusts KV cache memory allocation in the Changestest_bfloat16_4gpus memory configuration and waiver cleanup
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
af5e3cc to
ec2fcfc
Compare
ec2fcfc to
50bbfad
Compare
|
/bot run --disable-fail-fast |
|
PR_Github #49386 [ run ] triggered by Bot. Commit: |
|
PR_Github #49386 [ run ] completed with state |
50bbfad to
479540c
Compare
…torch_compile The variant ep4-mtp_nextn=2-attention_dp=True-cuda_graph=True-overlap_scheduler=True-torch_compile=True fails on RTX Pro 6000 Blackwell Desktop with cublas handle creation failure during warmup. With torch_compile=True, piecewise CUDA graph workspaces and lazy cublasLt handle allocation cannot fit alongside a 0.75 KV cache reservation. Lower free_gpu_memory_fraction to 0.6 only when torch_compile is enabled and remove the matching waiver. Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com>
Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com>
479540c to
7391428
Compare
|
/bot run --disable-fail-fast |
|
PR_Github #50496 [ run ] triggered by Bot. Commit: |
|
PR_Github #50496 [ run ] completed with state |
…hen torch_compile=True for test_bfloat16_ (NVIDIA#14109) Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com>
…fp8 with torch_compile Piecewise CUDA-graph capture under torch_compile=True needs additional GPU workspace beyond what the default kv_cache fraction (0.9) reserves on the 44 GiB L40S. As a result, _capture_piecewise_cuda_graphs OOMs in the executor_extra stage during executor creation. Mirror the established fix from NVIDIA#14109 (nvbugs/6162860, TestDeepSeekV3Lite): when torch_compile=True, set free_gpu_memory_fraction=0.6 so the executor has room for graph workspace. The non-compile path keeps the default and already passes. Also remove the now-obsolete L40S waiver for this test. Signed-off-by: tensorrt-cicd <90828364+tensorrt-cicd@users.noreply.github.com>
Summary
Test plan
Links
Summary by CodeRabbit