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

Fix Network stack not being loaded #26

Conversation

gustavo16a
Copy link

@gustavo16a gustavo16a commented Oct 21, 2024

Description

Due to the changes in the commit 4c4ceb2, the Network Stack requires the use of a new Random Number Generator (RNG). The UefiPayloadPkg has not been patched on default EDK2, since network implementation is not default in it. Therefore, the solution was to apply something that was done in another platforms in EDK2 adding the RngDxe and the Hash2DxeCrypto drivers (commits 3e72240 and e10d832).

  • Breaking change?
    • Breaking change - Does this PR cause a break in build or boot behavior?
    • Examples: Does it add a new library class or move a module to a different repo.
  • Impacts security?
    • Security - Does this PR have a direct security impact?
    • Examples: Crypto algorithm change or buffer overflow fix.
  • Includes tests?
    • Tests - Does this PR include any explicit test code?
    • Examples: Unit tests or integration tests.

How This Was Tested

The Loading of the network stack was done in an proprietary device in development

Integration Instructions

N/A

Due to the changes in the commit  
4c4ceb2, the Network Stack requires the use of a new Random Number Generator (RNG).
The UefiPayloadPkg has not been patched on default EDK2, since network implementation is not default in it. Therefore, the solution was to apply something that was done in another platforms in EDK2 adding the RngDxe and the Hash2DxeCrypto drivers (commits 3e72240 and e10d832).
@MrChromebox MrChromebox merged commit 849bce0 into MrChromebox:uefipayload_2408 Oct 21, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants