This checkin addresses the compatibility issue of passing arguments of type VA_LIST between components. The type VA_LIST is mapped onto the compiler specific implementation of varargs. As a result, modules build with different compilers may not use the same VA_LIST structure. The solution to this issue is to define a new type called BASE_LIST that is a compiler independent method of passing varargs between modules.

1) Update all implementations of ReportStatusCodeExtractDebugInfo() to use an argument of type BASE_LIST instead of VA_LIST.
2) Update status code listeners to use the BASE_LIST based APIs in the PrintLib instead of the VA_LIST based APIs, since ReportStatusCodeExtractDebugInfo() was updated to return a parameter of type BASE_LIST.



git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@8407 6f19259b-4bc3-4df7-8a09-765794883524
This commit is contained in:
mdkinney
2009-05-30 23:54:11 +00:00
parent ca9938b842
commit b5a8e95c5e
2 changed files with 6 additions and 6 deletions

View File

@ -695,7 +695,7 @@ Returns:
CHAR8 *Filename; CHAR8 *Filename;
CHAR8 *Description; CHAR8 *Description;
CHAR8 *Format; CHAR8 *Format;
VA_LIST Marker; BASE_LIST Marker;
UINT32 ErrorLevel; UINT32 ErrorLevel;
UINTN CharCount; UINTN CharCount;
@ -720,7 +720,7 @@ Returns:
// //
// Process PEI_DEBUG () macro to Serial // Process PEI_DEBUG () macro to Serial
// //
AsciiVSPrint (Buffer, EFI_STATUS_CODE_DATA_MAX_SIZE, Format, Marker); AsciiBSPrint (Buffer, EFI_STATUS_CODE_DATA_MAX_SIZE, Format, Marker);
} else if ((CodeType & EFI_STATUS_CODE_TYPE_MASK) == EFI_ERROR_CODE) { } else if ((CodeType & EFI_STATUS_CODE_TYPE_MASK) == EFI_ERROR_CODE) {
// //

View File

@ -229,7 +229,7 @@ EFIAPI
ReportStatusCodeExtractDebugInfo ( ReportStatusCodeExtractDebugInfo (
IN CONST EFI_STATUS_CODE_DATA *Data, IN CONST EFI_STATUS_CODE_DATA *Data,
OUT UINT32 *ErrorLevel, OUT UINT32 *ErrorLevel,
OUT VA_LIST *Marker, OUT BASE_LIST *Marker,
OUT CHAR8 **Format OUT CHAR8 **Format
) )
{ {
@ -258,7 +258,7 @@ ReportStatusCodeExtractDebugInfo (
// The first 12 * UINTN bytes of the string are really an // The first 12 * UINTN bytes of the string are really an
// argument stack to support varargs on the Format string. // argument stack to support varargs on the Format string.
// //
*Marker = (VA_LIST) (DebugInfo + 1); *Marker = (BASE_LIST) (DebugInfo + 1);
*Format = (CHAR8 *)(((UINT64 *)*Marker) + 12); *Format = (CHAR8 *)(((UINT64 *)*Marker) + 12);
return TRUE; return TRUE;