Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[sonic_pcie] Add get_pcie_aer_stats and its common implementation #144

Merged
merged 2 commits into from
Jan 14, 2021

Conversation

ArunSaravananBalachandran
Copy link
Contributor

  • Introduce a new platform API get_pcie_aer_stats in class PcieBase for retrieving AER stats of a PCIe device.
  • Add common implementation of get_pcie_aer_stats in class PcieUtil

Ref: sonic-net/SONiC#702

@sujinmkang sujinmkang merged commit 7fc76b9 into sonic-net:master Jan 14, 2021
sujinmkang pushed a commit to sonic-net/sonic-platform-daemons that referenced this pull request Jan 26, 2021
In pcied, added support to collect AER stats belonging to different severities for AER supported PCIe devices and update it in STATE_DB.

The key used to represent a PCIE device for storing its AER stats in STATE_DB is of the format PCIE_DEVICE|<Bus>:<Dev>.<Fn>.
For every device, AER stats will be stored as key, value pairs where key is of the format <severity>|<AER Error type> and the device ID will be stored with key id.

HLD: sonic-net/SONiC#678, sonic-net/SONiC#720
Depends on: sonic-net/sonic-platform-common#144
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-platform-common that referenced this pull request Oct 25, 2024
…onic-net#144)

[pcied] Remove unnecessary message and move the configuration path from plugin directory to platform directory

This PR fixes sonic-net/sonic-buildimage#5819 and fixes sonic-net/sonic-buildimage#6437.

Remove unnecessary repeated message
Move the configuration path from plugin directory to platform directory
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