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

[nvidia-bluefield] Add the possibility to compile BFB image #29

Closed
wants to merge 1 commit into from

Conversation

oleksandrivantsiv
Copy link
Owner

@oleksandrivantsiv oleksandrivantsiv commented Jul 4, 2024

Why I did it

Add the possibility to compile BFB (BlueField boot stream) image for the nvidia-bluefield platform.
The BFB image fully overwrites the DPU's SSD and can be used for DPU recovery.

Work item tracking
  • Microsoft ADO (number only):

How I did it

Add new bfb image type to build_image.sh script. Add scripts under the "nvidia-bluefield" platform to create the binary file.

How to verify it

Configure nvidia-bluefield platform. Run make target/sonic-nvidia-bluefield.bfb command to compile the image.

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

@vivekrnv
Copy link

vivekrnv commented Jul 4, 2024

I think the comment should be make target/sonic-nvidia-bluefield.bfb

GRUB_AA64=grubnetaa64.efi
GRUB_CFG="" # Common Grub Config
BF2_BOOT_ARGS="console=ttyAMA1 console=hvc0 console=ttyAMA0 earlycon=pl011,0x01000000 earlycon=pl011,0x01800000"
BF2_GRUB_CFG="$BF2_BOOT_ARGS isolcpus=1-7 nohz_full=1-7 rcu_nocbs=1-7"
Copy link

Choose a reason for hiding this comment

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

Do we need BF2 grub cfg and boot args?

Copy link
Owner Author

Choose a reason for hiding this comment

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

we don't need it anymore. But I don't want to change this file right now, because this will require additional testing and we don't have time for this. I suggest to submit it as is with BF2 support and cleanup it later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants