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

Port open PRs from upstream for edge-core platforms: AS4630-54TE, AS5835-54X, AS7326-56X, and AS7726-32X #4

Open
wants to merge 78 commits into
base: master
Choose a base branch
from

Conversation

matofeder
Copy link
Member

@matofeder matofeder commented Oct 7, 2024

This PR ports open PRs from upstream for edge-core platforms: AS4630-54TE, AS5835-54X, AS7326-56X, and AS7726-32X.

Inspired by the edge-core PR [Sync PRs and Fix bugs] For platforms: AS4630-54TE, AS5835-54X, AS7326-56X, AS7726-32X, AS9716-32D, and AS9736-64D and investigation described within SovereignCloudStack/issues#706

List of ported PRs:

Relates to:

lukasstockner and others added 30 commits October 4, 2024 09:43
Add system-health.service into service dependency to prevent from
system-health.service start failed due to sonic_platform not installed.

Signed-off-by: Brandon Chuang <[email protected]>
as4630_54te:
  1. Fix wrong fan direction
  2. Fix wrong get_status_led()
  3. Fix wrong get_status() when psu power good failed

as5835_54x:
  1. Fix wrong get_status_led()
  2. Fix wrong get_status() when psu power good failed
  3. Fix wrong sysfs path of rear fan

Signed-off-by: Brandon Chuang <[email protected]>
Why I did it
    The customer report of the PCIe Bus Errors upon the SDK initialization.
The root cause is BRCM MAC chipset DNX/XGS series does not support the ASPM function.

How I did it
    Based on the internal info and discussion, update "pcie_aspm=off" into
ONIE_PLATFORM_EXTRA_CMDLINE_LINUX of installer.conf to resolve it.

Signed-off-by: Brandon Chuang <[email protected]>
chihhan123 and others added 12 commits October 9, 2024 12:02
2. sync disk data before shutdown DUT
This commit resolves a permission denied issue during the mv command in bookworm
by adding escalated sudo privileges. For more details, see the related PR
and discussion:
sonic-net/sonic-buildimage#18789

Signed-off-by: Matej Feder <[email protected]>
Why I did it:
    After DUT starts up, the syncd cannot work.
    And, the syslog will indicate:
      - ERR syncd#syncd: [none] SAI_API_SWITCH:platform_process_command:430 Platform command "init bcm" failed, rc = -1.
      - INFO syncd#supervisord: syncd Scache on unit 0 exhausted the stable_size of 76303156#015

How I dit it:
    Modify SDK configuration file of as4630_54te (followed as4630_54pe)

How to verify it:
    Check syncd is running normally after DUT starts up, and no related error syslog exists.
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.

7 participants