Skip to content

fix: use cnrtMemcpyAsync_V2 for Cambricon async copies#30

Merged
voltjia merged 1 commit into
masterfrom
fix/cambricon-async-memcpy
Jul 24, 2026
Merged

fix: use cnrtMemcpyAsync_V2 for Cambricon async copies#30
voltjia merged 1 commit into
masterfrom
fix/cambricon-async-memcpy

Conversation

@spike-zhu

Copy link
Copy Markdown
Contributor

Summary

  • Update Cambricon MemcpyAsync in src/native/cambricon/runtime_.h to use cnrtMemcpyAsync_V2.
  • Remove the unused <type_traits> dependency after eliminating runtime signature dispatch.

Motivation

After InfiniCore switched its Cambricon runtime implementation to InfiniRT,
InfiniLM inference failed during queue synchronization with:

CN_ERROR_INVALID_ADDRESS
cnrtQueueSync: MLU queue sync failed

Type of Change

  • feat - new feature / new backend capability / new public API
  • fix - bug fix
  • perf - performance improvement without behavior change
  • refactor - code restructuring without behavior change
  • test - adding or fixing tests only
  • docs - documentation only
  • build / ci - build system or CI configuration
  • chore - tooling, formatting, or other non-code changes
  • Breaking change (requires a ! in the Conventional Commits prefix or a BREAKING CHANGE: footer)

Platforms Affected

  • CPU (WITH_CPU)
  • NVIDIA (WITH_NVIDIA)
  • Iluvatar (WITH_ILUVATAR)
  • Hygon (WITH_HYGON)
  • MetaX (WITH_METAX)
  • Moore (WITH_MOORE)
  • Cambricon (WITH_CAMBRICON)
  • Ascend (WITH_ASCEND)
  • Build system / CMake / generated headers
  • Public headers / installed consumer API
  • Documentation only

Smoke Build and Test Result

image

Test Results on Supported Platforms

Platform Affected Build / Smoke Result Full Result / Notes
CPU
NVIDIA
Iluvatar
Hygon
MetaX
Moore
Cambricon
Ascend
Full `ctest` output (optional)
paste here

Benchmark / Performance Impact

Notes for Reviewers

InfiniLM 测试截图:

修改前
img_v3_0213j_876e6b63-14ae-429c-b6d7-62f163181ddg

修改后
image

@spike-zhu
spike-zhu requested a review from voltjia July 15, 2026 07:59
@spike-zhu spike-zhu self-assigned this Jul 15, 2026
@voltjia voltjia changed the title fix: use cnrtMemcpyAsync_V2 for Cambricon async copies fix: use cnrtMemcpyAsync_V2 for Cambricon async copies Jul 24, 2026
@voltjia
voltjia merged commit 82a9e58 into master Jul 24, 2026
6 checks passed
@voltjia
voltjia deleted the fix/cambricon-async-memcpy branch July 24, 2026 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants