Skip to content

Commit

Permalink
Resolving issue for populating sensor Aux names.
Browse files Browse the repository at this point in the history 
Issue: Sensor auxName not getting picked up which is provided in the Json configuration.
Testing: tested on Umbriel #1
Validation results: Port_0 and Port_1 are the sensorAuxName provided in json configuration file.

root@umbriel:~# busctl tree xyz.openbmc_project.PLDM
`-/xyz
  `-/xyz/openbmc_project
    |-/xyz/openbmc_project/inventory
    | |-/xyz/openbmc_project/inventory/platformmetrics
    | `-/xyz/openbmc_project/inventory/system
    |   `-/xyz/openbmc_project/inventory/system/chassis
    |     |-/xyz/openbmc_project/inventory/system/chassis/HGX_ERoT_BMC_0
    |     |-/xyz/openbmc_project/inventory/system/chassis/HGX_ERoT_FPGA_0
    |     |-/xyz/openbmc_project/inventory/system/chassis/HGX_ERoT_NVSwitch_0
    |     |-/xyz/openbmc_project/inventory/system/chassis/HGX_ERoT_NVSwitch_1
    |     `-/xyz/openbmc_project/inventory/system/chassis/HGX_ERoT_PCIeSwitch_0
    |-/xyz/openbmc_project/pldm
    | `-/xyz/openbmc_project/pldm/sensor_polling
    |-/xyz/openbmc_project/sensors
    | `-/xyz/openbmc_project/sensors/none
    |   |-/xyz/openbmc_project/sensors/none/PLDM_Sensor_36865_6
    |   |-/xyz/openbmc_project/sensors/none/PLDM_Sensor_54792_6
    |   `-/xyz/openbmc_project/sensors/none/PLDM_Sensor_54793_6
    |-/xyz/openbmc_project/software
    | |-/xyz/openbmc_project/software/HGX_FW_ERoT_BMC_0
    | |-/xyz/openbmc_project/software/HGX_FW_ERoT_FPGA_0
    | |-/xyz/openbmc_project/software/HGX_FW_ERoT_NVSwitch_0
    | |-/xyz/openbmc_project/software/HGX_FW_ERoT_NVSwitch_1
    | `-/xyz/openbmc_project/software/HGX_FW_ERoT_PCIeSwitch_0
    `-/xyz/openbmc_project/state
      |-/xyz/openbmc_project/state/PLDM_Sensor_3_6
      | `-/xyz/openbmc_project/state/PLDM_Sensor_3_6/Id_0
      |-/xyz/openbmc_project/state/PLDM_Sensor_4_6
      | `-/xyz/openbmc_project/state/PLDM_Sensor_4_6/Port_0
      `-/xyz/openbmc_project/state/PLDM_Sensor_5_6
        `-/xyz/openbmc_project/state/PLDM_Sensor_5_6/Port_1

Fixes JIRA https://jirasw.nvidia.com/browse/DGXOPENBMC-10147
Signed-off-by: Utkarsh Yadav <[email protected]>
  • Loading branch information
@uyadavnvd
uyadavnvd committed Mar 18, 2024
1 parent 25fc95c commit c85ff5a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion platform-mc/terminus.cpp
View file
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@ Terminus::Terminus(tid_t tid, uint64_t supportedTypes, UUID& uuid,

{
maxBufferSize = 256;
scanInventories();
}

void Terminus::interfaceAdded(sdbusplus::message::message& m)
Expand Down Expand Up @@ -338,7 +339,6 @@ bool Terminus::parsePDRs()
nvidia::nvidiaInitTerminus(*this);
#endif

scanInventories();
updateAssociations();

interfaceAddedMatch = std::make_unique<sdbusplus::bus::match_t>(
Expand Down

0 comments on commit c85ff5a

Please sign in to comment.