Skip to content

[None][fix] Update DeepGEMM revision#14150

Merged
Barry-Delaney merged 2 commits into
NVIDIA:feat/deepseek_v4from
lfr-0531:user/fanrongl/update-deepgemm-67fc648
May 15, 2026
Merged

[None][fix] Update DeepGEMM revision#14150
Barry-Delaney merged 2 commits into
NVIDIA:feat/deepseek_v4from
lfr-0531:user/fanrongl/update-deepgemm-67fc648

Conversation

@lfr-0531
Copy link
Copy Markdown
Collaborator

Description

  • Update the DeepGEMM fetch content revision to 67fc64863d43521080bf2005e6528d0fceee9510.

Testing

  • Not run; pre-commit was skipped per request for this single JSON pin update.

Signed-off-by: Fanrong Li <lfr-0531@users.noreply.github.com>
@lfr-0531 lfr-0531 requested a review from a team as a code owner May 14, 2026 16:48
@lfr-0531 lfr-0531 requested review from Barry-Delaney and removed request for a team May 14, 2026 16:49
@lfr-0531
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

1 similar comment
@lfr-0531
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48468 [ run ] triggered by Bot. Commit: 6f16693 Link to invocation

Follow-up to the DeepGEMM bump in 3rdparty/fetch_content.json
(c491439ed -> 67fc64863). The attribution scanner keys
scripts/attribution/data/ entries on <dep>/<git_tag> read from
fetch_content.json, so the prior pin's entries are orphaned after
the bump. Refresh both YAMLs so the scan still maps bundled
DeepGEMM sources at the new pin.

- dependency_metadata.yml: re-key the deepgemm entry and update
  the source URL to point at 67fc64863.
- files_to_dependency.yml: re-key the deepgemm entry and refresh
  the 50-entry checksum list. Checksums are 16-byte BLAKE2b
  digests of file content (see scripts/attribution/sbom/src/
  trtllm_sbom/storage.py::compute_cas_address) over the TRT-LLM
  build inputs, i.e. csrc/**/*.{hpp,cpp} plus
  deep_gemm/include/deep_gemm/common/types.hpp. Methodology was
  validated by exactly reproducing the old pin's 50-entry list
  under the same rule. 5 of the 50 files changed content
  between SHAs (csrc/apis/{attention,mega}.hpp,
  csrc/jit_kernels/heuristics/mega_moe.hpp, and
  csrc/jit_kernels/impls/{sm100_fp8_fp4_mega_moe,
  smxx_fp8_fp4_paged_mqa_logits}.hpp).

cpp/tensorrt_llm/deep_gemm/CMakeLists.txt does not need changes
at the new pin: csrc layout is unchanged (1 cpp + 1 cu + 48 hpp)
and the only "_C" import patterns under deep_gemm/ are
"from . import _C" (in __init__.py) and "from .. import _C" (in
mega/__init__.py), both already covered by the existing
string(REPLACE ...) rules.

Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
@lfr-0531
Copy link
Copy Markdown
Collaborator Author

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48502 [ run ] triggered by Bot. Commit: 9e5847d Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48502 [ run ] completed with state SUCCESS. Commit: 9e5847d
/LLM/main/L0_MergeRequest_PR pipeline #38298 completed with status: 'FAILURE'

CI Report

⚠️ Action Required:

  • Please check the failed tests and fix your PR
  • If you cannot view the failures, ask the CI triggerer to share details
  • Once fixed, request an NVIDIA team member to trigger CI again

CI Agent Failure Analysis

Link to invocation

@Barry-Delaney
Copy link
Copy Markdown
Collaborator

/bot run --disable-fail-fast

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48562 [ run ] triggered by Bot. Commit: 9e5847d Link to invocation

@tensorrt-cicd
Copy link
Copy Markdown
Collaborator

PR_Github #48562 [ run ] completed with state SUCCESS. Commit: 9e5847d
/LLM/main/L0_MergeRequest_PR pipeline #38351 completed with status: 'SUCCESS'

CI Report

Link to invocation

@Barry-Delaney Barry-Delaney merged commit e6339b5 into NVIDIA:feat/deepseek_v4 May 15, 2026
6 checks passed
lfr-0531 added a commit to lfr-0531/TensorRT-LLM that referenced this pull request May 29, 2026
Signed-off-by: Fanrong Li <lfr-0531@users.noreply.github.com>
Signed-off-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
Co-authored-by: Fanrong Li <lfr-0531@users.noreply.github.com>
Co-authored-by: Barry Kang <43644113+Barry-Delaney@users.noreply.github.com>
(cherry picked from commit e6339b5)
Signed-off-by: Fanrong Li <lfr-0531@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants