Skip to content

fby3.5 : Fix ASD connection performance issue#277

Closed
DelphineChiu wants to merge 1 commit into
facebook:mainfrom
Wiwynn:Ingrid/asd-issue
Closed

fby3.5 : Fix ASD connection performance issue#277
DelphineChiu wants to merge 1 commit into
facebook:mainfrom
Wiwynn:Ingrid/asd-issue

Conversation

@DelphineChiu

Copy link
Copy Markdown

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
Copy Markdown
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.

3 participants