openbmc-linux icon indicating copy to clipboard operation
openbmc-linux copied to clipboard

Kernel 4.1: ftgmac: Increase maximum NC-SI response payload size

Open xzhang-brcm opened this issue 4 years ago • 3 comments

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

xzhang-brcm avatar Aug 23 '20 19:08 xzhang-brcm

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 avatar Aug 23 '20 19:08 facebook-github-bot

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

facebook-github-bot avatar Aug 23 '20 19:08 facebook-github-bot

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

benwei13 avatar Aug 23 '20 23:08 benwei13