Skip to content

[Mellanox]: Fix PCIEd config for SN4600c#6892

Merged
lguohan merged 1 commit intosonic-net:masterfrom
nazariig:master-sn4600c-pcied
Feb 26, 2021
Merged

[Mellanox]: Fix PCIEd config for SN4600c#6892
lguohan merged 1 commit intosonic-net:masterfrom
nazariig:master-sn4600c-pcied

Conversation

@nazariig
Copy link
Collaborator

@nazariig nazariig commented Feb 25, 2021

Signed-off-by: Nazarii Hnydyn nazariig@nvidia.com

PCIEd pollutes log with the next errors:

Feb 25 14:20:06.051698 sonic WARNING pmon#pcied[66]: PCIe Device: Ethernet controller: Mellanox Technologies Device cf70 Not Found
Feb 25 14:20:06.051698 sonic WARNING pmon#pcied[66]: PCIe Device: Ethernet controller: Intel Corporation I210 Gigabit Network Connection (rev 03) Not Found
Feb 25 14:20:06.052063 sonic ERR pmon#pcied[66]: PCIe device status check : FAILED

Why I did it

  • To fix PCIEd log errors

How I did it

  • Fixed pcie.yaml

How to verify it

  1. supervisorctl restart pcied

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

  • N/A

A picture of a cute animal (not mandatory but encouraged)

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@lguohan lguohan merged commit 39b1c12 into sonic-net:master Feb 26, 2021
yxieca pushed a commit that referenced this pull request Mar 4, 2021
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
lolyu pushed a commit to lolyu/sonic-buildimage that referenced this pull request Sep 13, 2021
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants