Skip to content

[Mellanox] Use the explicit mst device of SPC during fw-upgrade#98

Closed
vivekrnv wants to merge 2 commits intomasterfrom
fix_mlnx_fw_upgrade
Closed

[Mellanox] Use the explicit mst device of SPC during fw-upgrade#98
vivekrnv wants to merge 2 commits intomasterfrom
fix_mlnx_fw_upgrade

Conversation

@vivekrnv
Copy link
Owner

@vivekrnv vivekrnv commented May 3, 2024

Why I did it

mlxfwmanager on smartwitch will show more than 1 device. Thus update the mlnx-fw-upgrade script to operate explicitly on Spectrum PCI device.

Work item tracking
  • Microsoft ADO (number only):

How I did it

  1. Used the mlxfwmanager --query-format XML command to get the output in XML format and used xmlstarlet cmd to parse the output
    Example cmd output:
<Devices>
    <Device pciName="/dev/mst/mt41692_pciconf0" type="BlueField3" psid="MT_0000001139" partNumber="BF3-COM-DPU_QP_Ax">
      <Versions>
        <FW current="32.41.1000" available="N/A"/>
      </Versions>
      <MACs Base_Mac="N/A" />
      <GUIDs Base_Guid="N/A" />
      <Status>No matching image found</Status>
      <Description>NVIDIA BlueField-3 DPU based COM express; IPN QP</Description>
    </Device>
    <Device pciName="/dev/mst/mt53104_pci_cr0" type="Spectrum3" psid="MT_0000001189" partNumber="MSN4280-WS2RXO_Ax">
      <Versions>
        <FW current="30_2012_4038-006" available="N/A"/>
      </Versions>
      <MACs Base_Mac="N/A" />
      <Status>No matching image found</Status>
      <Description>NVIDIA Spectrum-3 based 400GbE 2U Ethernet Smart-Switch with ONIE; 28 ports QSFP-DD; 4 DPUs; 2 power supplies (AC); x86 CPU - 16 Cores; Secure-boot Capable; standard depth; C2P airflow; Tool-less Rail Kit</Description>
    </Device>
</Devices>
  1. Exclusively operate on the PCI device inferred from the output to Wait, fetch psid and Current FW version
  2. Use awk for a reliable extraction of Available FW output

How to verify it

Verify the script if it upgrades FW properly during switch init and during reboot.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@vivekrnv vivekrnv force-pushed the fix_mlnx_fw_upgrade branch from 09d4c8b to 0b95fd4 Compare May 8, 2024 22:54
@vivekrnv vivekrnv force-pushed the fix_mlnx_fw_upgrade branch from 0b95fd4 to a774493 Compare May 8, 2024 23:07
@vivekrnv vivekrnv requested a review from oleksandrivantsiv May 8, 2024 23:08
@vivekrnv vivekrnv marked this pull request as ready for review May 8, 2024 23:08
@vivekrnv vivekrnv closed this May 9, 2024
vivekrnv pushed a commit that referenced this pull request Aug 19, 2025
…er-dependency (sonic-net#23716)

#### Why I did it
sonic-swss-common
```
dc05565 2025-08-14 | Revert "[strict yang] Add YANG based cfg_schema.h to schema.h (sonic-net#1008)" (sonic-net#1067) [jingwenxie]
e782b12 2025-08-12 | Add c-api/Rust wrappers for ConfigDBConnector and EventPublisher (sonic-net#1062) [Qi Luo]
1484a85 2025-08-08 | Ignore nonexistent include in database_global.json (sonic-net#1041) [yue-fred-gao]
839ddf0 2025-08-08 | [strict yang] Add YANG based cfg_schema.h to schema.h (sonic-net#1008) [jingwenxie]
```

sonic-dash-ha
```
6163990 2025-08-12 | Update sonic_db_config_initialize_global with extra argument (#98) [yue-fred-gao]
01a88ef 2025-08-07 | Suppress update from consumer bridge if there is no change (#89) [yue-fred-gao]
```
vivekrnv pushed a commit that referenced this pull request Sep 24, 2025
…e latest HEAD automatically (sonic-net#23931)

#### Why I did it
src/wpasupplicant/sonic-wpa-supplicant
```
* 31c1c8db1 - (HEAD -> master, origin/master, origin/HEAD) Changes in HOSTAPD to Support PAC (#90) (67 minutes ago) [Vijaya Kumar Abbaraju]
* ed735fe67 - [CodeQL]: Failed CI on CodeQL (#98) (2 hours ago) [Ze Gan]
* d22c2f48e - [action] Fix codeQL github action issue. (#96) (14 hours ago) [Liu Shilong]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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.

3 participants