patina-v16.0.1

What's Changed

• Add test coverage for memory protection error handling @garybeihl (#1051)
  
  Change Details

    ## Description

  Task #1030 requests improved unit test coverage for memory protection error handling. This commit adds comprehensive test coverage for all three error paths introduced by #176, improving overall coverage metrics.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

  How This Was Tested

  • All 387 tests pass successfully with no regressions.
  • Coverage verified with cargo llvm-cov

  Integration Instructions

  N/A if nothing is required.
  

  
  

  ---

  
  

• Section Extractor: Improve code coverage @vineelko (#1100)
  
  Change Details

    ## Description

  Added few basic unit tests to improve code coverage for Section Extractor

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

  How This Was Tested

  cargo make coverage

  Integration Instructions

  NA
  

  
  

  ---

  
  

• patina\_dxe\_core: Make GicBases a field struct @Javagedes (#1102)
  
  Change Details

    ## Description

  Updates GicBases and HwInterruptProtocolInstaller to be field structs instead of tuple structs

  Closes #1091

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

  How This Was Tested

  CI

  Integration Instructions

  N/A

    </blockquote>
    <hr>
  </details>
  

• patina\_dxe\_core: Delete std example @Javagedes (#1101)
  
  Change Details

    ## Description

  This commit deletes the std example for reasoning described in #1090.

  closes #1090

  ref patina-devops removal of build command: OpenDevicePartnership/patina-devops#50

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

  How This Was Tested

  N/A

  Integration Instructions

  N/A

    </blockquote>
    <hr>
  </details>
  

🐛 Bug Fixes

• patina\_dxe\_core: Do not wrap `set_hob_list` in debug\_assert @Javagedes (#1106)
  
  Change Details

    ## Description

  The compiler, on release builds, is removing the set_hob_list call due to being wrapped in a debug_assert. This results in in the hob list not being set, and us failing to get the hob list later. Due to lack to tests down the main setup code path, this was missed.

  This commit wraps the code in an assert instead and changes the interface.

  REF: OpenDevicePartnership/patina-qemu#86

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

  How This Was Tested

  Boot to shell Q35 RELEASE

  Integration Instructions

  N/A
  

  
  

  ---

  
  

• [Partial Revert]: Do not include the console logger for tests @vineelko (#1099)
  
  Change Details

    ## Description

  • The console logger was added to tests to improve code coverage for the log::* APIs in #1069.
  • However, the patina_dxe_core crate currently has a few tests (uefi_memory_map) that use their own logger for custom behavior.
  • Since both loggers end up in the same test binary (patina_dxe_core-0fa2c65cbd247021.exe), we cannot have two loggers in the same process (the uefi_memory_map logger initialization can panic if the console logger is already enabled).
  • Until we finalize a proper mechanism that supports both scenarios(code coverage + uefi_memory_map debug logs), this
    PR reverts the previously added console logger.

  ===

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

  How This Was Tested

  cargo make all

  Integration Instructions

  NA
  

  
  

  ---

  
  

Full Changelog: patina-v16.0.0...v16.0.1