Skip to content

common: fix BIC reset timing after update BIC#424

Closed
YoungLi123 wants to merge 1 commit into
facebook:mainfrom
YoungLi123:Young/Fix_BIC_reset_timing_after_update
Closed

common: fix BIC reset timing after update BIC#424
YoungLi123 wants to merge 1 commit into
facebook:mainfrom
YoungLi123:Young/Fix_BIC_reset_timing_after_update

Conversation

@YoungLi123

Copy link
Copy Markdown
Collaborator

Summary:

  • Sometimes BMC can't receive the last package response after update BIC image,
    because BIC didn't send out the response yet before BIC reset

Test Plan:

  • Build code: Pass

Before extending the time to reset
Failed Rate:
SLOT1: 1/441 loops
SLOT2: 2/433 loops
SLOT3: 1/444 loops
SLOT4: 5/449 loop

After extending the time to reset
Total 4 slots 7720 loops pass

Summary:
- Sometimes BMC can't receive the last package response after update BIC image,
  because BIC didn't send out the response yet before BIC reset

Test Plan:
- Build code: Pass

Before extending the time to reset
Failed Rate:
SLOT1: 1/441 loops
SLOT2: 2/433 loops
SLOT3: 1/444 loops
SLOT4: 5/449 loop

After extending the time to reset
Total 4 slots 7720 loops pass
@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 Jul 19, 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.

@YoungLi123 YoungLi123 deleted the Young/Fix_BIC_reset_timing_after_update branch October 24, 2022 03:01
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.

2 participants