From 93641f3721262d4b442946ae76fea3335ac10949 Mon Sep 17 00:00:00 2001 From: Sujin Kang Date: Thu, 17 Jun 2021 14:01:03 -0700 Subject: [PATCH] Unifying the platform api for get_pcie_aer_stats with PcieBase (#197) Unifying the platform api for get_pcie_aer_stats with PcieBase --- sonic_platform_base/sonic_pcie/pcie_common.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sonic_platform_base/sonic_pcie/pcie_common.py b/sonic_platform_base/sonic_pcie/pcie_common.py index d448a653367c..58590ea8cef9 100644 --- a/sonic_platform_base/sonic_pcie/pcie_common.py +++ b/sonic_platform_base/sonic_pcie/pcie_common.py @@ -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]") @@ -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")