Branch: refs/heads/personal/joeyvagedes/report-status-code
  Home:   https://github.com/tianocore/edk2
  Commit: cd69c853df0ba8bb72409d22ec6541f6c25992c5
      
https://github.com/tianocore/edk2/commit/cd69c853df0ba8bb72409d22ec6541f6c25992c5
  Author: Joey Vagedes <[email protected]>
  Date:   2025-12-22 (Mon, 22 Dec 2025)

  Changed paths:
    M MdeModulePkg/Library/DxeReportStatusCodeLib/DxeReportStatusCodeLib.inf
    M MdeModulePkg/Library/DxeReportStatusCodeLib/ReportStatusCodeLib.c

  Log Message:
  -----------
  MdeModulePkg: DxeReportStatusCodeLib: Do not query protocol

This commit stops the library from querying the protocol database for
the status code protocol at the time of library function use. This logic
can result in a TPL inversion if the protocol has not yet been cached
and the caller is calling the library function when the TPL is higher
then TPL_NOTIFY.

Instead, the protocol is now located during the constructor of the
library. If the protocol is not found, then an on-protocol install event
is registered which will cache the protocol. A destructor is also added
to close the event in the scenario that the driver is unloaded before
the protocol is installed.

Signed-off-by: Joey Vagedes <[email protected]>


  Commit: f34e3daec54a981dd606729e9d01ea082d1b4f53
      
https://github.com/tianocore/edk2/commit/f34e3daec54a981dd606729e9d01ea082d1b4f53
  Author: Joey Vagedes <[email protected]>
  Date:   2025-12-22 (Mon, 22 Dec 2025)

  Changed paths:
    M MdeModulePkg/Library/RuntimeDxeReportStatusCodeLib/ReportStatusCodeLib.c

  Log Message:
  -----------
  MdeModulePkg: RuntimeDxeReportStatusCodeLib: Do not query protocol

This commit stops the library from querying the protocol database for
the status code protocol at the time of library function use. This logic
can result in a TPL inversion if the protocol has not yet been cached
and the caller is calling the library function when the TPL is higher
then TPL_NOTIFY.

Instead, the protocol is now located during the constructor of the
library. If the protocol is not found, then an on-protocol install event
is registered which will cache the protocol. A destructor is also added
to close the event in the scenario that the driver is unloaded before
the protocol is installed.

Signed-off-by: Joey Vagedes <[email protected]>


  Commit: 13ccf72805c6ced94b9e2175065883bd8923fee5
      
https://github.com/tianocore/edk2/commit/13ccf72805c6ced94b9e2175065883bd8923fee5
  Author: Joey Vagedes <[email protected]>
  Date:   2025-12-29 (Mon, 29 Dec 2025)

  Changed paths:
    A 
MdeModulePkg/Library/DxeReportStatusCodeLib/GoogleTest/DxeReportStatusCodeLibGoogleTest.cpp
    A 
MdeModulePkg/Library/DxeReportStatusCodeLib/GoogleTest/DxeReportStatusCodeLibGoogleTest.inf
    M MdeModulePkg/Test/MdeModulePkgHostTest.dsc

  Log Message:
  -----------
  MdeModulePkg: Add unit tests for DxeReportStatusCodeLib

This commit adds a host based unit test for DxeReportStatusCodeLib
that ensures that the platform cannot invert the TPL on an initial
call to ReportStatusCode functionality.

Signed-off-by: Joey Vagedes <[email protected]>


Compare: 
https://github.com/tianocore/edk2/compare/cd69c853df0b%5E...13ccf72805c6

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