Skip to content

tests/unit: Add TX load balancing unit test (shmem_txlb)#57

Draft
bcmIntc wants to merge 1 commit into
openshmem-org:mainfrom
bcmIntc:bcm_txlb_test
Draft

tests/unit: Add TX load balancing unit test (shmem_txlb)#57
bcmIntc wants to merge 1 commit into
openshmem-org:mainfrom
bcmIntc:bcm_txlb_test

Conversation

@bcmIntc
Copy link
Copy Markdown
Collaborator

@bcmIntc bcmIntc commented Apr 6, 2026

Tests the OFI TX NIC selection feature added in b68a86a..HEAD:
--enable-ofi-tx-load-balancing-round-robin
--enable-ofi-tx-load-balancing-random

Three verification layers:

  1. All-to-all put data integrity across 3 message sizes x 16 iters (exercises SHMEM_GET_TRANSMIT_NIC_IDX on every operation)
  2. Round-robin counter advancement: snapshots shmem_internal_nic_rr_idx before/after a known number of puts and asserts exact delta (USE_OFI_TX_LOAD_BALANCING_ROUND_ROBIN builds only)
  3. shmemx_pcntr_get_issued_write consistency: total issued writes >= expected operation count (ENABLE_SHMEMX_TESTS builds)

Provides test coverage for PR: Sandia-OpenSHMEM/SOS#1224

Tests the OFI TX NIC selection feature added in b68a86a..HEAD:
  --enable-ofi-tx-load-balancing-round-robin
  --enable-ofi-tx-load-balancing-random

Three verification layers:
1. All-to-all put data integrity across 3 message sizes x 16 iters
   (exercises SHMEM_GET_TRANSMIT_NIC_IDX on every operation)
2. Round-robin counter advancement: snapshots shmem_internal_nic_rr_idx
   before/after a known number of puts and asserts exact delta
   (USE_OFI_TX_LOAD_BALANCING_ROUND_ROBIN builds only)
3. shmemx_pcntr_get_issued_write consistency: total issued writes >=
   expected operation count (ENABLE_SHMEMX_TESTS builds)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bcmIntc bcmIntc self-assigned this Apr 6, 2026
@markbrown314 markbrown314 self-requested a review April 13, 2026 15:41
@bcmIntc bcmIntc changed the title Add TX load balancing unit test (shmem_txlb) tests/unit: Add TX load balancing unit test (shmem_txlb) Apr 18, 2026
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