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

[2405] Set EFI_MEMORY_SP as System Memory #1061

Closed
wants to merge 1 commit into from

Commits on Jul 25, 2024

  1. Set EFI_MEMORY_SP as System Memory (microsoft#886)

    ## Description
    
    When supplying DxeCore with a resource descriptor HOB, a platform can
    choose which memory type to specify. For EFI_MEMORY_SP resource
    descriptor HOBs, instead of blindly setting GcdReserved as the memory
    type, respect what the resource descriptor HOB specified. Closes microsoft#884.
    
    - [x] Impacts functionality?
    - **Functionality** - Does the change ultimately impact how firmware
    functions?
    - Examples: Add a new library, publish a new PPI, update an algorithm,
    ...
    - [ ] Impacts security?
    - **Security** - Does the change have a direct security impact on an
    application,
        flow, or firmware?
      - Examples: Crypto algorithm change, buffer overflow fix, parameter
        validation improvement, ...
    - [ ] Breaking change?
    - **Breaking change** - Will anyone consuming this change experience a
    break
        in build or boot behavior?
    - Examples: Add a new library class, move a module to a different repo,
    call
        a function in a new library class in a pre-existing module, ...
    - [ ] Includes tests?
      - **Tests** - Does the change include any explicit test code?
      - Examples: Unit tests, integration tests, robot tests, ...
    - [ ] Includes documentation?
    - **Documentation** - Does the change contain explicit documentation
    additions
        outside direct code modifications (and comments)?
    - Examples: Update readme file, add feature readme file, link to
    documentation
        on an a separate Web page, ...
    
    ## How This Was Tested
    
    Tested on virtual platforms with CXL memory attached.
    
    ## Integration Instructions
    
    N/A.
    os-d committed Jul 25, 2024
    Configuration menu
    Copy the full SHA
    134f0fa View commit details
    Browse the repository at this point in the history