Skip to content

Add patch to extend the ipmb buffer#209

Closed
Yi-Shum wants to merge 1 commit into
facebook:mainfrom
Yi-Shum:Add_ipmb_driver_patch
Closed

Add patch to extend the ipmb buffer#209
Yi-Shum wants to merge 1 commit into
facebook:mainfrom
Yi-Shum:Add_ipmb_driver_patch

Conversation

@Yi-Shum

@Yi-Shum Yi-Shum commented Mar 16, 2022

Copy link
Copy Markdown
Collaborator

Summary:

  • Extend ipmb buffer from 250 bytes to 252 bytes.
  • Fix the issue that HOST cannot get post code using the command "ipmitool raw 0x30 0x49".
    It is caused by ipmb driver losing the data excess 250 bytes.

Test Plan:

  • Build Code: Pass
  • Command Test: Pass

LOG:

[root@localhost ~]# ipmitool raw 0x30 0x49
 92 92 92 92 92 92 92 92 99 92 98 97 92 92 92 92
 92 92 92 92 92 91 99 92 92 92 92 92 92 92 92 ef
 96 95 94 94 94 94 94 94 94 94 94 94 94 92 91 70
 68 61 4f 47 42 41 40 7f 7f 15 0d 0c 0b 06 04 00
 50 22 02 23 03 ee ed ec eb e9 e7 e6 af bf 7e c6
 ce bc bc bc bc bc cc 7e dc ca ca b7 7e 70 7e d1
 7e d1 7e d0 7e d0 7e d0 7e 7e bb bb bb bb bb bb
 bb bb bb bb cb 7e 70 70 70 b9 ba db d9 da c9 d7
 b8 b8 b8 b8 b8 b8 b8 b8 b8 b7 b7 b7 b9 70 d6 d2
 7e d2 7e 7e be be 7e b7 70 70 7e 70 7e b7 b7 b6
 b6 b7 b7 7e 7e 7e b6 b6 b7 b0 b6 b6 b6 b3 b3 c6
 b2 c5 b8 7e b4 7e b6 b1 b1 b1 7e 7e 70 7e c2 7e
 b1 b1 70 c1 7e b0 cd 73 7e cf 7e b5 bf b0 af af
 e5 e3 e4 e1 e0 e0 e0 ae aa a8 a9 a9 a7 a7 a7 a2
 a2 a9 a9 a7 a3 a3 a1 00 19 55 52 15 4d 4a 49 49
 48 7f 02 00
Summary:
- Extend ipmb buffer from 250 bytes to 252 bytes.
- Fix the issue that HOST cannot get post code using the command "ipmitool raw 0x30 0x49".
It is caused by ipmb driver losing the data excess 250 bytes.

Test Plan:
- Build Code: Pass
- Command Test: Pass

LOG:
```
[root@localhost ~]# ipmitool raw 0x30 0x49
 92 92 92 92 92 92 92 92 99 92 98 97 92 92 92 92
 92 92 92 92 92 91 99 92 92 92 92 92 92 92 92 ef
 96 95 94 94 94 94 94 94 94 94 94 94 94 92 91 70
 68 61 4f 47 42 41 40 7f 7f 15 0d 0c 0b 06 04 00
 50 22 02 23 03 ee ed ec eb e9 e7 e6 af bf 7e c6
 ce bc bc bc bc bc cc 7e dc ca ca b7 7e 70 7e d1
 7e d1 7e d0 7e d0 7e d0 7e 7e bb bb bb bb bb bb
 bb bb bb bb cb 7e 70 70 70 b9 ba db d9 da c9 d7
 b8 b8 b8 b8 b8 b8 b8 b8 b8 b7 b7 b7 b9 70 d6 d2
 7e d2 7e 7e be be 7e b7 70 70 7e 70 7e b7 b7 b6
 b6 b7 b7 7e 7e 7e b6 b6 b7 b0 b6 b6 b6 b3 b3 c6
 b2 c5 b8 7e b4 7e b6 b1 b1 b1 7e 7e 70 7e c2 7e
 b1 b1 70 c1 7e b0 cd 73 7e cf 7e b5 bf b0 af af
 e5 e3 e4 e1 e0 e0 e0 ae aa a8 a9 a9 a7 a7 a7 a2
 a2 a9 a9 a7 a3 a3 a1 00 19 55 52 15 4d 4a 49 49
 48 7f 02 00
```
@Yi-Shum Yi-Shum requested a review from GoldenBug March 16, 2022 09:14
@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 Mar 16, 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.

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