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: common: Fix GPIO return wrong value #265

Closed

Conversation

DelphineChiu
Copy link

Summary:

  • GPIO returns wrong value and it causes host delays power on because host BIOS is waiting for BMC ready bit until time out.
  • The root cause is that we didn't clean temporary variable value.

Test plan:

  • Build code: Pass
  • Host power on: Pass

Log:

  1. Check host can power on successfully.
  • Before fix
    root@bmc-oob:# power-util slot1 on
    Powering fru 1 to ON state...
    root@bmc-oob:
    # power-util slot1 status
    Power status for fru 1 : ON

root@bmc-oob:~# sol-util slot1 --history

root@bmc-oob:~#

  • After fix
    root@bmc-oob:~# power-util slot1 on
    Powering fru 1 to ON state...

root@bmc-oob:~# power-util slot1 status
Power status for fru 1 : ON

root@bmc-oob:~# sol-util slot1 --history

Mon Apr 25 19:34:32 2022 0004237 [19][A1][A3][A3][A7][A9][A9][A2][A2][AA][AF][E0][E0][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][AF][AF][19][A1][A3][A3][A7][A9][A9][A2][A2][A7][A7][A7][A9][A9][A8][AA][AE][E0][E0][E0][E1][E4][E3][E5][AF][AF][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][B1][B1][B1][B6][7E][B4][7E][B8][C5][B2][C6][B3][B3][B3][B3][B3][B3][B3][B6][B6][B6][B0][B7][B6][B7][B6][B6][7E][7E][7E][B7][B7][B6][B6][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][D8][7E][B7][B7][70][7E][70][70][B7][7E][B7][B7][BE][BE][7E][7E][D2][7E][D2][D6][70][B9][7E][B7][B7][B7][B7][B7][B7][B7][B7][B8][B8][B8][B8][B8][B8][B8][B8][B8][D7][C9][DA][D9][DB][BA][B9][70][70][70][7E][7E][CB][BB][BB][BB][BB][BB][BB][BB][BB][BB][BB][7E][7E][D0][7E][D0][7E][D0][7E][D1][7E][D1][7E][70][7E][B7][CA][CA][DC][7E][CC][BC][BC][BC][BC][BC][CE][C6][7E][BF][AF][AF][AF][AF][19][A1][A3][A3][A7][A9][A9][A2][A2][A7][A7][A7][A9][A9][A8][AA][AE][E0][E0][E0][E1][E4][E3][E5][AF][AF][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][B1][B1][B1][B6][7E][B4][7E][B8][C5][B2][C6][B3][B3][B3][B3][B3][B3][B3][B6][B6][B6][B0][B7][B6][B7][B6][B6][7E][7E][7E][B7][B7][B6][B6][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][D8][7E][B7][B7][70][7E][70][70][B7][7E][B7][B7][BE][BE][7E][7E][D2][7E][D2][D6][70][B9][7E][B7][B7][B7][B7][B7][B7][B7][B7][B8][B8][B8][B8][B8][B8][B8][B8][B8][D7][C9][DA][D9][DB][BA][B9][70][70][70][7E][7E][CB][BB][BB][BB][BB][BB][BB][BB][BB][BB][BB][7E][7E][D0][7E][D0][7E][D0][7E][D1][7E][D1][7E][70][7E][B7][CA][CA][DC][7E][CC][BC][BC][BC][BC][BC][CE][C6][7E][BF][AF][AF][AF][E6][E7][E9][EB][EC][ED][EE][4F][61][68][70][91][92][94][94][94][94][94][94][94][94][95][96][92][92][92][92][92][99][91][92][92][92][92][92][92][97]
Version 2.22.1283. Copyright (C) 2022 AMI
BIOS Date: 04/18/2022 11:24:25 Ver: F0ES_1A16
...

Summary:
- GPIO returns wrong value and it causes host delays power on because host BIOS is waiting for BMC ready bit until time out.
- The root cause is that we didn't clean temporary variable value.

Test plan:
- Build code: Pass
- Host power on: Pass

Log:
1. Check host can power on successfully.
- Before fix
root@bmc-oob:~# power-util slot1 on
Powering fru 1 to ON state...
root@bmc-oob:~# power-util slot1 status
Power status for fru 1 : ON

root@bmc-oob:~# sol-util slot1 --history

root@bmc-oob:~#

- After fix
root@bmc-oob:~# power-util slot1 on
Powering fru 1 to ON state...

root@bmc-oob:~# power-util slot1 status
Power status for fru 1 : ON

root@bmc-oob:~# sol-util slot1 --history

Mon Apr 25 19:34:32 2022 0004237 [19][A1][A3][A3][A7][A9][A9][A2][A2][AA][AF][E0][E0][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][AF][AF][19][A1][A3][A3][A7][A9][A9][A2][A2][A7][A7][A7][A9][A9][A8][AA][AE][E0][E0][E0][E1][E4][E3][E5][AF][AF][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][B1][B1][B1][B6][7E][B4][7E][B8][C5][B2][C6][B3][B3][B3][B3][B3][B3][B3][B6][B6][B6][B0][B7][B6][B7][B6][B6][7E][7E][7E][B7][B7][B6][B6][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][D8][7E][B7][B7][70][7E][70][70][B7][7E][B7][B7][BE][BE][7E][7E][D2][7E][D2][D6][70][B9][7E][B7][B7][B7][B7][B7][B7][B7][B7][B8][B8][B8][B8][B8][B8][B8][B8][B8][D7][C9][DA][D9][DB][BA][B9][70][70][70][7E][7E][CB][BB][BB][BB][BB][BB][BB][BB][BB][BB][BB][7E][7E][D0][7E][D0][7E][D0][7E][D1][7E][D1][7E][70][7E][B7][CA][CA][DC][7E][CC][BC][BC][BC][BC][BC][CE][C6][7E][BF][AF][AF][AF][AF][19][A1][A3][A3][A7][A9][A9][A2][A2][A7][A7][A7][A9][A9][A8][AA][AE][E0][E0][E0][E1][E4][E3][E5][AF][AF][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][B1][B1][B1][B6][7E][B4][7E][B8][C5][B2][C6][B3][B3][B3][B3][B3][B3][B3][B6][B6][B6][B0][B7][B6][B7][B6][B6][7E][7E][7E][B7][B7][B6][B6][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][D8][7E][B7][B7][70][7E][70][70][B7][7E][B7][B7][BE][BE][7E][7E][D2][7E][D2][D6][70][B9][7E][B7][B7][B7][B7][B7][B7][B7][B7][B8][B8][B8][B8][B8][B8][B8][B8][B8][D7][C9][DA][D9][DB][BA][B9][70][70][70][7E][7E][CB][BB][BB][BB][BB][BB][BB][BB][BB][BB][BB][7E][7E][D0][7E][D0][7E][D0][7E][D1][7E][D1][7E][70][7E][B7][CA][CA][DC][7E][CC][BC][BC][BC][BC][BC][CE][C6][7E][BF][AF][AF][AF][E6][E7][E9][EB][EC][ED][EE][4F][61][68][70][91][92][94][94][94][94][94][94][94][94][95][96][92][92][92][92][92][99][91][92][92][92][92][92][92][97]
Version 2.22.1283. Copyright (C) 2022 AMI
BIOS Date: 04/18/2022 11:24:25 Ver: F0ES_1A16
...
@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 Apr 26, 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.

facebook-github-bot pushed a commit that referenced this pull request Apr 26, 2022
Summary:
- GPIO returns wrong value and it causes host delays power on because host BIOS is waiting for BMC ready bit until time out.
- The root cause is that we didn't clean temporary variable value.

Pull Request resolved: #265

Test Plan:
- Build code: Pass
- Host power on: Pass

Log:
1. Check host can power on successfully.
- Before fix
root@bmc-oob:~# power-util slot1 on
Powering fru 1 to ON state...
root@bmc-oob:~# power-util slot1 status
Power status for fru 1 : ON

root@bmc-oob:~# sol-util slot1 --history

root@bmc-oob:~#

- After fix
root@bmc-oob:~# power-util slot1 on
Powering fru 1 to ON state...

root@bmc-oob:~# power-util slot1 status
Power status for fru 1 : ON

root@bmc-oob:~# sol-util slot1 --history

Mon Apr 25 19:34:32 2022 0004237 [19][A1][A3][A3][A7][A9][A9][A2][A2][AA][AF][E0][E0][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][AF][AF][19][A1][A3][A3][A7][A9][A9][A2][A2][A7][A7][A7][A9][A9][A8][AA][AE][E0][E0][E0][E1][E4][E3][E5][AF][AF][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][B1][B1][B1][B6][7E][B4][7E][B8][C5][B2][C6][B3][B3][B3][B3][B3][B3][B3][B6][B6][B6][B0][B7][B6][B7][B6][B6][7E][7E][7E][B7][B7][B6][B6][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][D8][7E][B7][B7][70][7E][70][70][B7][7E][B7][B7][BE][BE][7E][7E][D2 (https://github.com/facebook/OpenBIC/commit/8fa5e407830d85f03d93019b3d6bf79a39c69532)][7E][D2 (https://github.com/facebook/OpenBIC/commit/8fa5e407830d85f03d93019b3d6bf79a39c69532)][D6][70][B9][7E][B7][B7][B7][B7][B7][B7][B7][B7][B8][B8][B8][B8][B8][B8][B8][B8][B8][D7][C9][DA][D9][DB][BA][B9][70][70][70][7E][7E][CB][BB][BB][BB][BB][BB][BB][BB][BB][BB][BB][7E][7E][D0][7E][D0][7E][D0][7E][D1][7E][D1][7E][70][7E][B7][CA][CA][DC][7E][CC][BC][BC][BC][BC][BC][CE][C6][7E][BF][AF][AF][AF][AF][19][A1][A3][A3][A7][A9][A9][A2][A2][A7][A7][A7][A9][A9][A8][AA][AE][E0][E0][E0][E1][E4][E3][E5][AF][AF][B0][BF][B5][7E][CF][7E][CD][B0][7E][C1][70][B1][B1][7E][C2][7E][70][7E][7E][B1][B1][B1][B6][7E][B4][7E][B8][C5][B2][C6][B3][B3][B3][B3][B3][B3][B3][B6][B6][B6][B0][B7][B6][B7][B6][B6][7E][7E][7E][B7][B7][B6][B6][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][B7][D8][7E][B7][B7][70][7E][70][70][B7][7E][B7][B7][BE][BE][7E][7E][D2 (https://github.com/facebook/OpenBIC/commit/8fa5e407830d85f03d93019b3d6bf79a39c69532)][7E][D2 (https://github.com/facebook/OpenBIC/commit/8fa5e407830d85f03d93019b3d6bf79a39c69532)][D6][70][B9][7E][B7][B7][B7][B7][B7][B7][B7][B7][B8][B8][B8][B8][B8][B8][B8][B8][B8][D7][C9][DA][D9][DB][BA][B9][70][70][70][7E][7E][CB][BB][BB][BB][BB][BB][BB][BB][BB][BB][BB][7E][7E][D0][7E][D0][7E][D0][7E][D1][7E][D1][7E][70][7E][B7][CA][CA][DC][7E][CC][BC][BC][BC][BC][BC][CE][C6][7E][BF][AF][AF][AF][E6][E7][E9][EB][EC][ED][EE][4F][61][68][70][91][92][94][94][94][94][94][94][94][94][95][96][92][92][92][92][92][99][91][92][92][92][92][92][92][97]
Version 2.22.1283. Copyright (C) 2022 AMI
BIOS Date: 04/18/2022 11:24:25 Ver: F0ES_1A16
...

Reviewed By: garnermic

Differential Revision: D35936400

Pulled By: GoldenBug

fbshipit-source-id: 1bd680a5bddd6805cb17e8048cf3d2023a1b685a
@SaraLin-wiwynn SaraLin-wiwynn deleted the Sara/fby3.5-Fix_gpio_value branch May 5, 2022 05:35
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