[Feat] Wire TransProviderType::AIV to the static-library backend#1028
Merged
Infinite666 merged 3 commits intoJun 17, 2026
Merged
Conversation
hczl
requested review from
Infinite666,
Wwwzff,
mag1c-h and
nrj868
as code owners
yuanzhg078
reviewed
Jun 16, 2026
hczl
force-pushed
the
feat/aiv-transport-wiring
branch
5 times, most recently
from
7861953 to
0e9ba55
Compare
pyxyzc
reviewed
Jun 16, 2026
pyxyzc
reviewed
Jun 16, 2026
hczl
force-pushed
the
feat/aiv-transport-wiring
branch
4 times, most recently
from
ea489c1 to
bc9d200
Compare
yuanzhg078
reviewed
Jun 16, 2026
hczl
force-pushed
the
feat/aiv-transport-wiring
branch
2 times, most recently
from
90ceeba to
2777dfb
Compare
Vendor the public interface shipped by libumc_asu_transport_provider.a (npu2cpu_rdma) under trans/include/aiv_transport/: aiv_transport.h declares the UC::ASU::AIVTransport contract plus the CreateAIVTransProvider() factory. Status/StatusCode reuse the existing UCM asu_transport/types.h to avoid shipping a duplicate vendor copy.
src/aiv_trans_provider.h provides a header-only AIVTransProvider : public TransProvider that forwards every call to an impl_ created by CreateAIVTransProvider() (libumc_asu_transport_provider.a). After renaming the vendor contract to AIVTransport there is no longer a name clash with UCM's own TransProvider, so the adapter includes aiv_transport/aiv_transport.h directly and stays header-only, matching aicpu_trans_provider.h. SendIoBatch / RegisterMemoryDesc / UnregisterMemoryDesc are rebuilt field-by-field when forwarding since the vendor and UCM structs are now distinct types.
asu_transport_impl.cpp constructs AIVTransProvider for the AIV backend; the asu CMake adds an ASU_AIV_PROVIDER_ROOT cache entry plus find_path / find_library checks (FATAL_ERROR on miss, mirroring the existing ASCEND_ROOT / ASCENDCL discovery style) and links libumc_asu_transport_provider.a. Co-authored-by: Cursor <cursoragent@cursor.com>
hczl
force-pushed
the
feat/aiv-transport-wiring
branch
from
2777dfb to
58c50a3
Compare
Infinite666
approved these changes
Jun 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
内容
将
asu_transport_impl.cpp中case TransProviderType::AIV的 UNSUPPORTED 占位替换为构造AIVTransProvider；CMake 增加ASU_AIV_PROVIDER_ROOT配置项