Skip to content

(tools) add Broadcom (bnxt) support to env_check.sh and env_setup.sh#386

Merged
QizhouZhang97 merged 10 commits into
mainfrom
dev/bnxt_tool
Jun 18, 2026
Merged

(tools) add Broadcom (bnxt) support to env_check.sh and env_setup.sh#386
QizhouZhang97 merged 10 commits into
mainfrom
dev/bnxt_tool

Conversation

@QizhouZhang97

Copy link
Copy Markdown
Contributor

(tools) add bnxt env check
image

@QizhouZhang97 QizhouZhang97 requested a review from jhchouuu June 11, 2026 10:39
@QizhouZhang97 QizhouZhang97 changed the title (tools) add bnxt env check (tools) add Broadcom (bnxt) support to env_check.sh and env_setup.sh Jun 18, 2026
@QizhouZhang97

Copy link
Copy Markdown
Contributor Author

Summary

Add Broadcom NetXtreme-E (bnxt / Thor2) support to mori check and mori setup, previously ionic-only.

env_check.sh

  • Auto-detect bnxt vs ionic via PCI vendor ID; run vendor-specific checks for firmware, driver, userspace lib, QoS/PFC, and DCQCN
  • Replace hardcoded GID index (-x 1, link-local, not routable through ToR) with automatic best-GID picker mirroring MORI's ScoreGidCandidate — prefers RoCEv2, IPv4-mapped over link-local
  • Rewrite intra/inter-node mesh as parallel mesh_execute runner with configurable concurrency, per-pair timeouts, and retry logic
  • Add SSH connection multiplexing (ControlMaster) for inter-node tests
  • Add intra_reachable() to skip cross-subnet pairs that can't hairpin through the ToR

env_setup.sh

  • Add bnxt_setup_pfc (via dcb), bnxt_setup_dcqcn (via configfs), bnxt_mori_env_setup parallel to existing ionic functions
  • CNP service profile detection for 2-TC vs 3-TC layout
  • Extract shared QoS constants (ROCE_PRIO, ROCE_DSCP, ROCE_BW) used by both vendors

deploy-mori skill

  • Add MPI deps to base install; remove unsupported ConnectX references

Test plan

  • mori check / mori setup on bnxt node
  • mori check <peer_ip> inter-node on bnxt
  • Regression check on ionic node

@QizhouZhang97 QizhouZhang97 merged commit 1db01d8 into main Jun 18, 2026
22 of 23 checks passed
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.

2 participants