On 06/29/16 16:47, Bruce Cran wrote: > On 6/27/2016 1:25 AM, Cinnamon Shia wrote: >> Let data of DXE memory status code can be used by other modules. >> 1. Save the address of DXE memory status code table to >> DxeConfigurationTable. >> 2. Save the address of SMM memory status code table to >> SmmConfigurationTable. >> 3. Move RUNTIME_MEMORY_STATUSCODE_HEADER to its public header file. > > I'm getting an error building OVMF today, which appears related: > > In file included from > /home/bcran/workspace/edk2/IntelFrameworkModulePkg/Universal/StatusCode/RuntimeDxe/SerialStatusCodeWorker.c:15:0: > > /home/bcran/workspace/edk2/IntelFrameworkModulePkg/Universal/StatusCode/RuntimeDxe/StatusCodeRuntimeDxe.h:63:3: > error: conflicting types for ‘RUNTIME_MEMORY_STATUSCODE_HEADER’ > } RUNTIME_MEMORY_STATUSCODE_HEADER; > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > In file included from > /home/bcran/workspace/edk2/IntelFrameworkModulePkg/Universal/StatusCode/RuntimeDxe/StatusCodeRuntimeDxe.h:22:0, > > from > /home/bcran/workspace/edk2/IntelFrameworkModulePkg/Universal/StatusCode/RuntimeDxe/SerialStatusCodeWorker.c:15: > > /home/bcran/workspace/edk2/MdeModulePkg/Include/Guid/MemoryStatusCodeRecord.h:76:3: > note: previous declaration of ‘RUNTIME_MEMORY_STATUSCODE_HEADER’ was here > } RUNTIME_MEMORY_STATUSCODE_HEADER; > ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ >
Yes, the file IntelFrameworkModulePkg/Universal/StatusCode/RuntimeDxe/StatusCodeRuntimeDxe.h includes MdeModulePkg/Include/Guid/MemoryStatusCodeRecord.h but also defines the RUNTIME_MEMORY_STATUSCODE_HEADER type, which has now become redundant. As far as I can see, it should be fixable by removing the RUNTIME_MEMORY_STATUSCODE_HEADER typedef from "IntelFrameworkModulePkg/Universal/StatusCode/RuntimeDxe/StatusCodeRuntimeDxe.h". Thanks Laszlo _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
