Skip to content

fix(kv-offload): support LLVM 22 RDMA bindings - #785

Draft
ClSlaid wants to merge 1 commit into
pegainfer-project:mainfrom
ClSlaid:agent/fix-libclang22-bindgen
Draft

fix(kv-offload): support LLVM 22 RDMA bindings#785
ClSlaid wants to merge 1 commit into
pegainfer-project:mainfrom
ClSlaid:agent/fix-libclang22-bindgen

Conversation

@ClSlaid

@ClSlaid ClSlaid commented Jul 29, 2026

Copy link
Copy Markdown

Summary

  • Require rdma-mummy-sys 0.2.5 directly in openinfer-kv-offload.
    The pinned pegaflow-core revision permits 0.2.4, so this requirement
    raises the resolver's minimum version.
  • Update Cargo.lock to rdma-mummy-sys 0.2.5. Its bindgen update removes
    obsolete transitive dependencies from the lockfile.
  • Add a compile regression test that accesses ibv_context.ops and
    ibv_mr.lkey.

Motivation

With LLVM/libclang 22, bindgen 0.66 generates vendored rdma-core structs such
as ibv_context as opaque _address shells. rdma-mummy-sys 0.2.4 then
fails to compile when its wrapper accesses fields such as ops, context,
and lkey.

rdma-mummy-sys 0.2.5 upgrades bindgen to 0.72.1 to support LLVM 22
(upstream commit).

Impact

OpenInfer now builds when bindgen loads libclang 22, without overriding
LIBCLANG_PATH to select an older installation. RDMA runtime behavior is
unchanged.

Validation

  • cargo fmt --check
  • env -u LIBCLANG_PATH CUDA_HOME=/opt/cuda cargo check --release -p openinfer-server
  • env -u LIBCLANG_PATH CUDA_HOME=/opt/cuda cargo test --release -p openinfer-kv-offload --test rdma_bindings

The release check and regression test pass with libclang 22.

@ClSlaid
ClSlaid force-pushed the agent/fix-libclang22-bindgen branch from 508c9ea to d6ebfa6 Compare July 29, 2026 16:22
@ClSlaid ClSlaid changed the title fix(build): support LLVM 22 RDMA bindings Update rdma-mummy-sys for LLVM 22 Jul 29, 2026
rdma-mummy-sys 0.2.4 uses bindgen 0.66, which emits opaque
ibverbs structs with LLVM 22. Require 0.2.5, whose bindgen 0.72
update restores the generated fields.

Add a compile regression test for representative ibv_context and
ibv_mr fields.
@ClSlaid
ClSlaid force-pushed the agent/fix-libclang22-bindgen branch from d6ebfa6 to eec486f Compare July 30, 2026 03:42
@ClSlaid ClSlaid changed the title Update rdma-mummy-sys for LLVM 22 fix(kv-offload): support LLVM 22 RDMA bindings Jul 30, 2026
@github-actions

Copy link
Copy Markdown

This pull request has been inactive for 14 days. It will be closed after another 30 days unless there is new activity.

@github-actions github-actions Bot added the stale Automatically marked after inactivity label Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stale Automatically marked after inactivity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant