Skip to content

R25.10 hotfix fix cpu topology (#939)#940

Open
wmousa wants to merge 1 commit intomainfrom
R25.10-Hotfix-fix-cpu-topology-main
Open

R25.10 hotfix fix cpu topology (#939)#940
wmousa wants to merge 1 commit intomainfrom
R25.10-Hotfix-fix-cpu-topology-main

Conversation

@wmousa
Copy link
Copy Markdown
Contributor

@wmousa wmousa commented Mar 30, 2026

  • Wrokaround the SMT issue for chess by settin the cpu to 2 for spdk-proxy-container

  • fix: enforce HT-aware SPDK core alignment and dynamic CPU limits

    • build_unisolated_stride: add nodes_per_socket param; pre-adjust
      num_unisolated so remaining SPDK cores are divisible by 4 (nodes_per_socket>=2, HT enabled) or 2 (HT enabled), preserving pool-stride and HT pair selection;
      return adjusted num_unisolated alongside the core list - generate_core_allocation: unpack new return value and pass nodes_per_socket into build_unisolated_stride
    • storage_deploy_spdk.yaml.j2: set cpu for spdk-proxy-container and copy-script
      to 2 if HT enabled and CPU topology enabled, 1 if CPU topology enabled without
      HT, omit cpu limit/request entirely when CPU topology is disabled
    • kubernetes.py: pass HT_ENABLED (via is_hyperthreading_enabled_via_siblings)
      to the pod template

* Wrokaround the SMT issue for chess by settin the cpu to 2 for spdk-proxy-container

* fix: enforce HT-aware SPDK core alignment and dynamic CPU limits

  - build_unisolated_stride: add nodes_per_socket param; pre-adjust
    num_unisolated so remaining SPDK cores are divisible by 4 (nodes_per_socket>=2,                                                                                                                                                               HT enabled) or 2 (HT enabled), preserving pool-stride and HT pair selection;
    return adjusted num_unisolated alongside the core list                                                                                                                                                                                      - generate_core_allocation: unpack new return value and pass nodes_per_socket                                                                                                                                                                   into build_unisolated_stride
  - storage_deploy_spdk.yaml.j2: set cpu for spdk-proxy-container and copy-script
    to 2 if HT enabled and CPU topology enabled, 1 if CPU topology enabled without
    HT, omit cpu limit/request entirely when CPU topology is disabled
  - kubernetes.py: pass HT_ENABLED (via is_hyperthreading_enabled_via_siblings)
    to the pod template
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.

1 participant