runtime-rs/ch: Fix inverted vcpu/tid mapping in get_ch_vcpu_tids by chiranjeevi-max · Pull Request #12526 · kata-containers/kata-containers (original) (raw)

@chiranjeevi-max

The VcpuThreadIds struct expects a mapping from vcpu_id to thread_id, but get_ch_vcpu_tids() was inserting (tid, vcpu_id) instead of (vcpu_id, tid).

This caused move_vcpus_to_sandbox_cgroup() to interpret vcpu IDs (0, 1, 2...) as process IDs when sandbox_cgroup_only=false, leading to failed attempts to read /proc/0/status.

Fixes: #12479 Signed-off-by: Chiranjeevi Uddanti 244287281+chiranjeevi-max@users.noreply.github.com

RuoqingHe

approved these changes Feb 15, 2026

sprt

sprt

sprt approved these changes Feb 18, 2026

@chiranjeevi-max

Add unit test for get_ch_vcpu_tids() and integration test that creates a pod with sandbox_cgroup_only=false to verify it starts successfully.

Signed-off-by: Chiranjeevi Uddanti 244287281+chiranjeevi-max@users.noreply.github.com Co-authored-by: Antigravity antigravityagent@google.com

Redent0r

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

[ Show hidden characters]({{ revealButtonHref }})