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

fby3.5 : Fix ASD connection performance issue #277

Closed
wants to merge 1 commit into from

Conversation

DelphineChiu
Copy link

Summary:

  • Decrease ASD connection cscript time from 14min to 4min
  • Reduce ipmb tx/rx sleep polling time from 10ms to 1ms

Test Plan:

  • Build code: Pass
  • Do experiments as below:
    • Before :
      Test item Test time
      ASD connect cscript 14m:16.25 sec
      go() 5 sec
      halt() 19 sec
      pcie.topology() 6m:34 sec
      CPU IERR 16m:02.56 sec
      DIMM correctable error 47m:09.32 sec(DDR5)
      PCIE correctable error 30.40 sec

    • After reduce polling time to 1ms:
      Test item Test time
      ASD connect cscript 3m:52s
      go() 3 sec
      halt() 8 sec
      pcie.topology() 1m:55 sec
      CPU IERR 9m:42 sec
      DIMM correctable error 6m:20 sec
      PCIE correctable error 29.7 sec

Summary:
- Decrease ASD connection cscript time from 14min to 4min
- Reduce ipmb tx/rx sleep polling time from 10ms to 1ms

Test Plan:
- Build code: Pass
- Do experiments as below:
  - Before :
  Test item                              Test time
  ASD connect cscript	                 14m:16.25 sec
  go()	                                 5 sec
  halt()	                         19 sec
  pcie.topology()	                 6m:34 sec
  CPU IERR                               16m:02.56 sec
  DIMM correctable error                 47m:09.32 sec(DDR5)
  PCIE correctable error	         30.40 sec

  - After reduce polling time to 1ms:
  Test item                              Test time
  ASD connect cscript	                 3m:52s
  go()	                                 3 sec
  halt()	                         8 sec
  pcie.topology()	                 1m:55 sec
  CPU IERR	                         9m:42 sec
  DIMM correctable error	         6m:20 sec
  PCIE correctable error	         29.7 sec
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 11, 2022
@facebook-github-bot
Copy link
Contributor

@GoldenBug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

KaidanWu-wiwynn added a commit to Wiwynn/OpenBIC that referenced this pull request Jun 21, 2022
Remove PMIC support from official v2022.19.1 version
Revert "fby3.5 : Fix ASD connection performance issue (facebook#277)"
This reverts commit 559e44d.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants