Skip to content

Conversation

@xzhang-brcm
Copy link

Summary:
Increase max NC-SI response payload to 1480 in kernel for big response packets
that are used in Broadcom OEM GetDebugData.

Test plan:

  • fby2 and fbgp2 image build (pass)
  • Update and boot the image on YosemiteV2 and V2.5 (pass)
  • BMC initializes NC-SI interface with NIC (pass)
  • Use ncsi-util send various commands such as 0x18, 0x19 and 0x1a (pass)
  • Use ncsi-util to perform PLDM firmware update (pass)
  • Use ncsi-uitl Broadcom OEM GetDebugData command to retrieve core dump and
    crash dump from NIC (pass)

root@dhcp-10-67-92-132:~# ncsi-util -m brcm -d coredump -o /tmp/nic.core
Debug data length: 14216032
Received: 14216032/14216032 (100%)
GetDebugData completed

root@dhcp-10-67-92-132:~# ncsi-util -m brcm -d crashdump -o /tmp/nic.crash
Debug data length: 47824
Received: 47824/47824 (100%)
GetDebugData completed

Summary:
Increase max NC-SI response payload to 1480 in kernel for big response packets
that are used in Broadcom OEM GetDebugData.

Test plan:
- fby2 and fbgp2 image build (pass)
- Update and boot the image on YosemiteV2 and V2.5 (pass)
- BMC initializes NC-SI interface with NIC (pass)
- Use ncsi-util send various commands such as 0x18, 0x19 and 0x1a (pass)
- Use ncsi-util to perform PLDM firmware update (pass)
- Use ncsi-uitl Broadcom OEM GetDebugData command to retrieve core dump and
  crash dump from NIC (pass)
@facebook-github-bot
Copy link

Hi @xzhang-brcm!

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file.

In order for us to review and merge your code, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

If you have received this in error or have any questions, please contact us at [email protected]. Thanks!

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Aug 23, 2020
@facebook-github-bot
Copy link

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@benwei13
Copy link
Contributor

Hi @xzhang-brcm , thanks a lot for this update! I will work with internal team to add this to next release.

Copy link

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

facebook-github-bot pushed a commit that referenced this pull request Sep 14, 2020
Summary:
Increase max NC-SI response payload to 1480 in kernel for big response packets
that are used in Broadcom OEM GetDebugData.

Signed-off-by: Xiaobing Zhang [email protected]
Closes #2

Test Plan:
- fby2 and fbgp2 image build (pass)
- Update and boot the image on YosemiteV2 and V2.5 (pass)
- BMC initializes NC-SI interface with NIC (pass)
- Use ncsi-util send various commands such as 0x18, 0x19 and 0x1a (pass)
- Use ncsi-util to perform PLDM firmware update (pass)
- Use ncsi-uitl Broadcom OEM GetDebugData command to retrieve core dump and
  crash dump from NIC (pass)

root@dhcp-10-67-92-132:~# ncsi-util -m brcm -d coredump -o /tmp/nic.core
Debug data length: 14216032
Received:   14216032/14216032 (100%)
GetDebugData completed

root@dhcp-10-67-92-132:~# ncsi-util -m brcm -d crashdump -o /tmp/nic.crash
Debug data length: 47824
Received:      47824/47824 (100%)
GetDebugData completed

Reviewed By: tao-ren

Differential Revision: D23288998

Pulled By: benwei13

fbshipit-source-id: 6c03c79011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants