Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: 622a2c7abf98d36525f8122ae724d381148121df
      
https://github.com/tianocore/edk2/commit/622a2c7abf98d36525f8122ae724d381148121df
  Author: Oliver Smith-Denny <[email protected]>
  Date:   2026-06-13 (Sat, 13 Jun 2026)

  Changed paths:
    M EmbeddedPkg/Library/PrePiLib/PrePi.h

  Log Message:
  -----------
  EmbeddedPkg: PrePi.h: Drop Duplicate HOB Defintions

PrePi.h has some duplicate HOB definitions from HobLib.h.
Instead of redefining them, use HobLib.h.

Signed-off-by: Oliver Smith-Denny <[email protected]>


  Commit: e63bba000a464fe35fb215e99bc4dcf01ebeabd6
      
https://github.com/tianocore/edk2/commit/e63bba000a464fe35fb215e99bc4dcf01ebeabd6
  Author: Oliver Smith-Denny <[email protected]>
  Date:   2026-06-13 (Sat, 13 Jun 2026)

  Changed paths:
    M EmbeddedPkg/Library/PrePiLib/FwVol.c

  Log Message:
  -----------
  EmbeddedPkg: PrePiLib: Skip Extracting if FV3 HOB Found

PrePiLib only searches for the existence of an FV2 HOB to
determine if an FV has already been extracted. However,
FV2 HOBs have been superceded by FV3 HOBs for describing extracted
FVs.

This commit updates PrePiLib to check for the existence of either
an FV2 or FV3 HOB for this FV and skips extracting if either is
found.

Signed-off-by: Oliver Smith-Denny <[email protected]>


  Commit: 6eca4a4a95942e9d7eb1809524e4116fb3ac1cd3
      
https://github.com/tianocore/edk2/commit/6eca4a4a95942e9d7eb1809524e4116fb3ac1cd3
  Author: Oliver Smith-Denny <[email protected]>
  Date:   2026-06-13 (Sat, 13 Jun 2026)

  Changed paths:
    M EmbeddedPkg/Library/PrePiLib/FwVol.c

  Log Message:
  -----------
  EmbeddedPkg: PrePiLib: Zero Volume Info Buffer Before Writing

FfsGetVolumeInfo() in PrePiLib does not zero the VolumeInfo
buffer before writing to it. However, it has some fields that
are expected to be zero if certain conditions are not met.

For example, the FvName is expected to be all 0's if the
extended header is not present in the FV. Currently,
the FvName field will point to garbage if the caller does
not zero the buffer first.

Flip the expectation and have FfsGetVolumeInfo() zero the
buffer before it writes to it.

Signed-off-by: Oliver Smith-Denny <[email protected]>


  Commit: 7ad30b28b2e86ccc9cab9917d272f486140323bb
      
https://github.com/tianocore/edk2/commit/7ad30b28b2e86ccc9cab9917d272f486140323bb
  Author: Dmitry Antipov <[email protected]>
  Date:   2026-06-13 (Sat, 13 Jun 2026)

  Changed paths:
    M EmbeddedPkg/Include/Library/PrePiLib.h
    M EmbeddedPkg/Library/PrePiLib/FwVol.c
    M EmbeddedPkg/Library/PrePiLib/PrePiLib.c
    M OvmfPkg/Library/PeilessStartupLib/DxeLoad.c
    M OvmfPkg/RiscVVirt/Library/PlatformSecLib/PlatformSecLib.c

  Log Message:
  -----------
  EmbeddedPkg, OvmfPkg: PrePiLib: Use Correct FvName in FV2 HOB Production

PrePiLib is incorrectly producing FV2 HOBs. The FvName field of
the FV2 HOB is intended to be the parent FV's name. This way the
HOB consumer phase can identify when an FV has already been
extracted: if the file name of the extracted FV and the parent
FV name match in the FV2 HOB.

With the incorrect FvName, DXE will re-extract the already
extracted FV, which can be a large performance hit (some platforms
we have seen ~1 second).

This requires an update to FfsProcessFvFile to pass the parent
FV handle through so the FV2 HOB can be correctly produced.
OvmfPkg consumers are also updated.

Continuous-integration-options: PatchCheck.ignore-multi-package

Signed-off-by: Oliver Smith-Denny <[email protected]>


  Commit: 6683b764976c3e87a1161e21554540f8dffd86b4
      
https://github.com/tianocore/edk2/commit/6683b764976c3e87a1161e21554540f8dffd86b4
  Author: Oliver Smith-Denny <[email protected]>
  Date:   2026-06-13 (Sat, 13 Jun 2026)

  Changed paths:
    M OvmfPkg/Library/PeilessStartupLib/DxeLoad.c

  Log Message:
  -----------
  OvmfPkg: PeilessStartupLib: Fix FV2 HOB Production

PeilessStartupLib is incorrectly producing FV2 HOBs.
The FvName field of the FV2 HOB is intended to be the
parent FV's name. This way the HOB consumer phase can
identify when an FV has already been extracted: if the
file name of the extracted FV and the parent FV name
match in the FV2 HOB.

With the incorrect FvName, DXE will re-extract the already
extracted FV, which can be a large performance hit (some platforms
we have seen ~1 second).

Signed-off-by: Oliver Smith-Denny <[email protected]>


  Commit: c6c02f8cae2642b9a8987ef14b1606afb0688b1e
      
https://github.com/tianocore/edk2/commit/c6c02f8cae2642b9a8987ef14b1606afb0688b1e
  Author: Oliver Smith-Denny <[email protected]>
  Date:   2026-06-13 (Sat, 13 Jun 2026)

  Changed paths:
    M EmbeddedPkg/Include/Library/PrePiLib.h
    M EmbeddedPkg/Library/PrePiLib/FwVol.c
    M EmbeddedPkg/Library/PrePiLib/PrePiLib.c
    M OvmfPkg/Library/PeilessStartupLib/DxeLoad.c

  Log Message:
  -----------
  EmbeddedPkg, OvmfPkg: Produce FV3 HOBs for Extracted FVs

EmbeddedPkg's PrePiLib and OvmfPkg's PeilessStartupLib (which
uses PrePiLib) only produce FV2 HOBs for extracted FVs. However,
FV3 HOBs have superceded FV2 HOBs for describing extracted FVs.

This commit updates both of these libs to also produce FV3 HOBs,
following PEI's pattern of producing an FV, FV2, and FV3 HOB for
each extracted FV.

Eventually, only FV3 HOBs will be produced, so this paves the way
for that.

Continuous-integration-options: PatchCheck.ignore-multi-package

Signed-off-by: Oliver Smith-Denny <[email protected]>


Compare: https://github.com/tianocore/edk2/compare/4bfc280b5797...c6c02f8cae26

To unsubscribe from these emails, change your notification settings at 
https://github.com/tianocore/edk2/settings/notifications


_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to