Skip to content

Commit

Permalink
Unifying the platform api for get_pcie_aer_stats with PcieBase (sonic…
Browse files Browse the repository at this point in the history
…-net#197)

Unifying the platform api for get_pcie_aer_stats with PcieBase
  • Loading branch information
sujinmkang authored Jun 17, 2021
1 parent 7670b49 commit 93641f3
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions sonic_platform_base/sonic_pcie/pcie_common.py
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ def load_config_file(self):
config_file = "{}/pcie{}.yaml".format(self.config_path, conf_rev)
try:
with open(config_file) as conf_file:
self.confInfo = yaml.load(conf_file)
self.confInfo = yaml.safe_load(conf_file)
except IOError as e:
print("Error: {}".format(str(e)))
print("Not found config file, please add a config file manually, or generate it by running [pcieutil pcie_generate]")
Expand Down Expand Up @@ -101,9 +101,9 @@ def get_pcie_check(self):
return self.confInfo

# return AER stats of PCIe device
def get_pcie_aer_stats(self, domain=0, bus=0, device=0, func=0):
def get_pcie_aer_stats(self, domain=0, bus=0, dev=0, func=0):
aer_stats = {'correctable': {}, 'fatal': {}, 'non_fatal': {}}
dev_path = os.path.join('/sys/bus/pci/devices', '%04x:%02x:%02x.%d' % (domain, bus, device, func))
dev_path = os.path.join('/sys/bus/pci/devices', '%04x:%02x:%02x.%d' % (domain, bus, dev, func))

# construct AER sysfs filepath
correctable_path = os.path.join(dev_path, "aer_dev_correctable")
Expand Down

0 comments on commit 93641f3

Please sign in to comment.