From 4ad9d87d7ac1cf831b9b95af4dcac5a88fac9a6f Mon Sep 17 00:00:00 2001 From: jljusten Date: Thu, 12 Feb 2009 17:15:08 +0000 Subject: * MdeModulePkg/Core/Pei/Dependency/Dependency.h Decrease MAX_GRAMMAR_SIZE from 256 to 64. In Dependency.c => PeimDispatchReadiness, a local variable 'EvalStack' makes an array of EVAL_STACK_ENTRY structures to store a stack for processing the dependency expression. Due to alignment considerations, sizeof (EVAL_STACK_ENTRY) is 16 bytes on 64 bit architectures. This means the size of the EvalStack local variable in PeimDispatchReadiness grows to 4kb. In the PEI phase, the stack area may be very small, and therefore 4kb is much to large. Reducing MAX_GRAMMAR_SIZE to 64 will lower the size of EvalStack to 1kb. While reducing this to 1kb solves an issue seen with GCC on X64, it is still a large usage of stack for the PEI phase, and future improvements may still need to be made. git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@7514 6f19259b-4bc3-4df7-8a09-765794883524 --- MdeModulePkg/Core/Pei/Dependency/Dependency.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'MdeModulePkg') diff --git a/MdeModulePkg/Core/Pei/Dependency/Dependency.h b/MdeModulePkg/Core/Pei/Dependency/Dependency.h index c15fd0d816..ff99783ac2 100644 --- a/MdeModulePkg/Core/Pei/Dependency/Dependency.h +++ b/MdeModulePkg/Core/Pei/Dependency/Dependency.h @@ -17,7 +17,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. #define _PEI_DEPENDENCY_H_ -#define MAX_GRAMMAR_SIZE 256 +#define MAX_GRAMMAR_SIZE 64 // // type definitions -- cgit v1.2.3