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.

Sign up for GitHub

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

Jump to bottom

[sonic_pcie] Add get_pcie_aer_stats and its common implementation #144

Merged
merged 2 commits into from
Jan 14, 2021
Merged

[sonic_pcie] Add get_pcie_aer_stats and its common implementation #144

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

@ArunSaravananBalachandran ArunSaravananBalachandran mentioned this pull request Nov 16, 2020
Merged
@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
@ArunSaravananBalachandran
[pcied] Add PCIe AER stats collection (#100)
1fcaa57 
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sujinmkang sujinmkang sujinmkang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ArunSaravananBalachandran @sujinmkang