Revision: 19151
          http://sourceforge.net/p/edk2/code/19151
Author:   vanjeff
Date:     2015-12-08 05:19:34 +0000 (Tue, 08 Dec 2015)
Log Message:
-----------
UefiCpuPkg/MtrrLib: Add PCD PcdCpuNumberOfReservedVariableMtrrs

Current MtrrLib reserves 2 variable MTRRs for some legacy OS boot (CSM boots)
may require some MTRRs to be reserved for OS use. But UEFI OS boot will not use
MTRRs.

Per Scott's suggestion in
link: http://article.gmane.org/gmane.comp.bios.edk2.devel/4099
Add one PCD PcdCpuNumberOfReservedVariableMtrrs to specify the number of
variable MTRRs reserved for OS use. Setting its default value to 2 is for
back-compatibility.

Cc: Scott Duplichan <[email protected]>
Cc: Feng Tian <[email protected]>
Cc: Michael Kinney <[email protected]>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jeff Fan <[email protected]>
Suggested-by: Scott Duplichan <[email protected]>
Reviewed-by: Feng Tian <[email protected]>

Modified Paths:
--------------
    trunk/edk2/UefiCpuPkg/Include/Library/MtrrLib.h
    trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
    trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.inf
    trunk/edk2/UefiCpuPkg/UefiCpuPkg.dec

Modified: trunk/edk2/UefiCpuPkg/Include/Library/MtrrLib.h
===================================================================
--- trunk/edk2/UefiCpuPkg/Include/Library/MtrrLib.h     2015-12-08 03:06:41 UTC 
(rev 19150)
+++ trunk/edk2/UefiCpuPkg/Include/Library/MtrrLib.h     2015-12-08 05:19:34 UTC 
(rev 19151)
@@ -1,7 +1,7 @@
 /** @file
   MTRR setting library
 
-  Copyright (c) 2008 - 2011, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2008 - 2015, Intel Corporation. All rights reserved.<BR>
   This program and the accompanying materials
   are licensed and made available under the terms and conditions of the BSD 
License
   which accompanies this distribution.  The full text of the license may be 
found at
@@ -26,6 +26,7 @@
 #define  MTRR_NUMBER_OF_VARIABLE_MTRR  32
 //
 // Firmware need reserve 2 MTRR for OS
+// Note: It is replaced by PCD PcdCpuNumberOfReservedVariableMtrrs
 //
 #define  RESERVED_FIRMWARE_VARIABLE_MTRR_NUMBER  2
 

Modified: trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.c
===================================================================
--- trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.c     2015-12-08 03:06:41 UTC 
(rev 19150)
+++ trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.c     2015-12-08 05:19:34 UTC 
(rev 19151)
@@ -140,13 +140,15 @@
   )
 {
   UINT32  VariableMtrrCount;
+  UINT32  ReservedMtrrNumber;
 
   VariableMtrrCount = GetVariableMtrrCount ();
-  if (VariableMtrrCount < RESERVED_FIRMWARE_VARIABLE_MTRR_NUMBER) {
+  ReservedMtrrNumber = PcdGet32 (PcdCpuNumberOfReservedVariableMtrrs);
+  if (VariableMtrrCount < ReservedMtrrNumber) {
     return 0;
   }
 
-  return VariableMtrrCount - RESERVED_FIRMWARE_VARIABLE_MTRR_NUMBER;
+  return VariableMtrrCount - ReservedMtrrNumber;
 }
 
 /**

Modified: trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.inf
===================================================================
--- trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.inf   2015-12-08 03:06:41 UTC 
(rev 19150)
+++ trunk/edk2/UefiCpuPkg/Library/MtrrLib/MtrrLib.inf   2015-12-08 05:19:34 UTC 
(rev 19151)
@@ -1,7 +1,7 @@
 ## @file
 #  MTRR library provides APIs for MTRR operation.
 #
-#  Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
+#  Copyright (c) 2006 - 2015, Intel Corporation. All rights reserved.<BR>
 #  This program and the accompanying materials
 #  are licensed and made available under the terms and conditions of the BSD 
License
 #  which accompanies this distribution.  The full text of the license may be 
found at
@@ -41,3 +41,6 @@
   CpuLib
   DebugLib
 
+[Pcd]
+  gUefiCpuPkgTokenSpaceGuid.PcdCpuNumberOfReservedVariableMtrrs   ## 
SOMETIMES_CONSUMES
+

Modified: trunk/edk2/UefiCpuPkg/UefiCpuPkg.dec
===================================================================
--- trunk/edk2/UefiCpuPkg/UefiCpuPkg.dec        2015-12-08 03:06:41 UTC (rev 
19150)
+++ trunk/edk2/UefiCpuPkg/UefiCpuPkg.dec        2015-12-08 05:19:34 UTC (rev 
19151)
@@ -167,6 +167,11 @@
   # @Prompt SMM CPU Synchronization Method.
   gUefiCpuPkgTokenSpaceGuid.PcdCpuSmmSyncMode|0x00|UINT8|0x60000014
 
+  ## Specifies the number of variable MTRRs reserved for OS use. The default 
number of
+  #  MTRRs reserved for OS use is 2.
+  # @Prompt Number of reserved variable MTRRs.
+  
gUefiCpuPkgTokenSpaceGuid.PcdCpuNumberOfReservedVariableMtrrs|0x2|UINT32|0x00000015
+
 [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
   ## Specifies timeout value in microseconds for the BSP to detect all APs for 
the first time.
   # @Prompt Timeout for the BSP to detect all APs for the first time.


------------------------------------------------------------------------------
Go from Idea to Many App Stores Faster with Intel(R) XDK
Give your users amazing mobile app experiences with Intel(R) XDK.
Use one codebase in this all-in-one HTML5 development environment.
Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to