]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Core/Dxe/Dispatcher/Dependency.c
MdeModulePkg DxeCore: Fix issue to print GUID value %g without pointer
[mirror_edk2.git] / MdeModulePkg / Core / Dxe / Dispatcher / Dependency.c
index a77dc31073e52e5f4e8cd23a842d04027c2a4cd1..1d8a57b87984f655cbbfa0c6620efce1885b2579 100644 (file)
@@ -5,8 +5,8 @@
   if a driver can be scheduled for execution.  The criteria for\r
   schedulability is that the dependency expression is satisfied.\r
 \r
-Copyright (c) 2006 - 2008, Intel Corporation. <BR>\r
-All rights reserved. This program and the accompanying materials\r
+Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>\r
+This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
 which accompanies this distribution.  The full text of the license may be found at\r
 http://opensource.org/licenses/bsd-license.php\r
@@ -16,7 +16,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 **/\r
 \r
-#include <DxeMain.h>\r
+#include "DxeMain.h"\r
 \r
 //\r
 // Global stack used to evaluate dependency expressions\r
@@ -33,7 +33,7 @@ BOOLEAN *mDepexEvaluationStackPointer = NULL;
 /**\r
   Grow size of the Depex stack\r
 \r
-  @retval EFI_SUCCESS           Stack successfully growed. \r
+  @retval EFI_SUCCESS           Stack successfully growed.\r
   @retval EFI_OUT_OF_RESOURCES  There is not enough system memory to grow the stack.\r
 \r
 **/\r
@@ -50,7 +50,7 @@ GrowDepexStack (
     Size = Size + (mDepexEvaluationStackEnd - mDepexEvaluationStack);\r
   }\r
 \r
-  NewStack = CoreAllocateBootServicesPool (Size * sizeof (BOOLEAN));\r
+  NewStack = AllocatePool (Size * sizeof (BOOLEAN));\r
   if (NewStack == NULL) {\r
     return EFI_OUT_OF_RESOURCES;\r
   }\r
@@ -60,15 +60,15 @@ GrowDepexStack (
     // Copy to Old Stack to the New Stack\r
     //\r
     CopyMem (\r
-      NewStack, \r
-      mDepexEvaluationStack, \r
+      NewStack,\r
+      mDepexEvaluationStack,\r
       (mDepexEvaluationStackEnd - mDepexEvaluationStack) * sizeof (BOOLEAN)\r
       );\r
 \r
     //\r
     // Free The Old Stack\r
     //\r
-    CoreFreePool (mDepexEvaluationStack);\r
+    FreePool (mDepexEvaluationStack);\r
   }\r
 \r
   //\r
@@ -86,9 +86,9 @@ GrowDepexStack (
 /**\r
   Push an element onto the Boolean Stack.\r
 \r
-  @param  Value                 BOOLEAN to push. \r
+  @param  Value                 BOOLEAN to push.\r
 \r
-  @retval EFI_SUCCESS           The value was pushed onto the stack. \r
+  @retval EFI_SUCCESS           The value was pushed onto the stack.\r
   @retval EFI_OUT_OF_RESOURCES  There is not enough system memory to grow the stack.\r
 \r
 **/\r
@@ -126,13 +126,13 @@ PushBool (
 /**\r
   Pop an element from the Boolean stack.\r
 \r
-  @param  Value                 BOOLEAN to pop. \r
+  @param  Value                 BOOLEAN to pop.\r
 \r
-  @retval EFI_SUCCESS           The value was popped onto the stack. \r
+  @retval EFI_SUCCESS           The value was popped onto the stack.\r
   @retval EFI_ACCESS_DENIED     The pop operation underflowed the stack.\r
 \r
 **/\r
-EFI_STATUS \r
+EFI_STATUS\r
 PopBool (\r
   OUT BOOLEAN  *Value\r
   )\r
@@ -149,7 +149,7 @@ PopBool (
   //\r
   mDepexEvaluationStackPointer--;\r
   *Value = *mDepexEvaluationStackPointer;\r
-  return EFI_SUCCESS;  \r
+  return EFI_SUCCESS;\r
 }\r
 \r
 \r
@@ -168,23 +168,23 @@ PopBool (
 **/\r
 EFI_STATUS\r
 CorePreProcessDepex (\r
-  IN  EFI_CORE_DRIVER_ENTRY   *DriverEntry  \r
+  IN  EFI_CORE_DRIVER_ENTRY   *DriverEntry\r
   )\r
 {\r
   UINT8  *Iterator;\r
-    \r
+\r
   Iterator = DriverEntry->Depex;\r
   if (*Iterator == EFI_DEP_SOR) {\r
     DriverEntry->Unrequested = TRUE;\r
   } else {\r
     DriverEntry->Dependent = TRUE;\r
   }\r
-    \r
+\r
   if (*Iterator == EFI_DEP_BEFORE) {\r
     DriverEntry->Before = TRUE;\r
   } else if (*Iterator == EFI_DEP_AFTER) {\r
     DriverEntry->After = TRUE;\r
-  } \r
+  }\r
 \r
   if (DriverEntry->Before || DriverEntry->After) {\r
     CopyMem (&DriverEntry->BeforeAfterGuid, Iterator + 1, sizeof (EFI_GUID));\r
@@ -201,16 +201,16 @@ CorePreProcessDepex (
   routine in this case. The SOR is just ignored and is a nop in the grammer.\r
   POSTFIX means all the math is done on top of the stack.\r
 \r
-  @param  DriverEntry           DriverEntry element to update. \r
+  @param  DriverEntry           DriverEntry element to update.\r
 \r
-  @retval TRUE                  If driver is ready to run. \r
-  @retval FALSE                 If driver is not ready to run or some fatal error \r
+  @retval TRUE                  If driver is ready to run.\r
+  @retval FALSE                 If driver is not ready to run or some fatal error\r
                                 was found.\r
 \r
 **/\r
 BOOLEAN\r
 CoreIsSchedulable (\r
-  IN  EFI_CORE_DRIVER_ENTRY   *DriverEntry  \r
+  IN  EFI_CORE_DRIVER_ENTRY   *DriverEntry\r
   )\r
 {\r
   EFI_STATUS  Status;\r
@@ -220,6 +220,9 @@ CoreIsSchedulable (
   EFI_GUID    DriverGuid;\r
   VOID        *Interface;\r
 \r
+  Operator = FALSE;\r
+  Operator2 = FALSE;\r
+\r
   if (DriverEntry->After || DriverEntry->Before) {\r
     //\r
     // If Before or After Depex skip as CoreInsertOnScheduledQueueWhileProcessingBeforeAndAfter ()\r
@@ -228,14 +231,19 @@ CoreIsSchedulable (
     return FALSE;\r
   }\r
 \r
+  DEBUG ((DEBUG_DISPATCH, "Evaluate DXE DEPEX for FFS(%g)\n", &DriverEntry->FileName));\r
+\r
   if (DriverEntry->Depex == NULL) {\r
     //\r
     // A NULL Depex means treat the driver like an UEFI 2.0 thing.\r
     //\r
     Status = CoreAllEfiServicesAvailable ();\r
+    DEBUG ((DEBUG_DISPATCH, "  All UEFI Services Available                     = "));\r
     if (EFI_ERROR (Status)) {\r
+      DEBUG ((DEBUG_DISPATCH, "FALSE\n  RESULT = FALSE\n"));\r
       return FALSE;\r
     }\r
+    DEBUG ((DEBUG_DISPATCH, "TRUE\n  RESULT = TRUE\n"));\r
     return TRUE;\r
   }\r
 \r
@@ -247,13 +255,14 @@ CoreIsSchedulable (
 \r
 \r
   Iterator = DriverEntry->Depex;\r
-  \r
+\r
   while (TRUE) {\r
     //\r
     // Check to see if we are attempting to fetch dependency expression instructions\r
     // past the end of the dependency expression.\r
     //\r
     if (((UINTN)Iterator - (UINTN)DriverEntry->Depex) >= DriverEntry->DepexSize) {\r
+      DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Attempt to fetch past end of depex)\n"));\r
       return FALSE;\r
     }\r
 \r
@@ -269,21 +278,25 @@ CoreIsSchedulable (
       // If the code flow arrives at this point, there was a BEFORE or AFTER\r
       // that were not the first opcodes.\r
       //\r
+      DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected BEFORE or AFTER opcode)\n"));\r
       ASSERT (FALSE);\r
     case EFI_DEP_SOR:\r
       //\r
-      // These opcodes can only appear once as the first opcode.  If it is found \r
+      // These opcodes can only appear once as the first opcode.  If it is found\r
       // at any other location, then the dependency expression evaluates to FALSE\r
       //\r
       if (Iterator != DriverEntry->Depex) {\r
+        DEBUG ((DEBUG_DISPATCH, "  SOR\n"));\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected SOR opcode)\n"));\r
         return FALSE;\r
       }\r
+      DEBUG ((DEBUG_DISPATCH, "  SOR                                             = Requested\n"));\r
       //\r
       // Otherwise, it is the first opcode and should be treated as a NOP.\r
       //\r
       break;\r
 \r
-    case EFI_DEP_PUSH:  \r
+    case EFI_DEP_PUSH:\r
       //\r
       // Push operator is followed by a GUID. Test to see if the GUID protocol\r
       // is installed and push the boolean result on the stack.\r
@@ -293,98 +306,124 @@ CoreIsSchedulable (
       Status = CoreLocateProtocol (&DriverGuid, NULL, &Interface);\r
 \r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  PUSH GUID(%g) = FALSE\n", &DriverGuid));\r
         Status = PushBool (FALSE);\r
       } else {\r
+        DEBUG ((DEBUG_DISPATCH, "  PUSH GUID(%g) = TRUE\n", &DriverGuid));\r
         *Iterator = EFI_DEP_REPLACE_TRUE;\r
         Status = PushBool (TRUE);\r
       }\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
 \r
       Iterator += sizeof (EFI_GUID);\r
       break;\r
 \r
-    case EFI_DEP_AND:    \r
+    case EFI_DEP_AND:\r
+      DEBUG ((DEBUG_DISPATCH, "  AND\n"));\r
       Status = PopBool (&Operator);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
 \r
       Status = PopBool (&Operator2);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
 \r
       Status = PushBool ((BOOLEAN)(Operator && Operator2));\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
       break;\r
 \r
-    case EFI_DEP_OR:     \r
+    case EFI_DEP_OR:\r
+      DEBUG ((DEBUG_DISPATCH, "  OR\n"));\r
       Status = PopBool (&Operator);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
 \r
       Status = PopBool (&Operator2);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
 \r
       Status = PushBool ((BOOLEAN)(Operator || Operator2));\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
       break;\r
 \r
-    case EFI_DEP_NOT:    \r
+    case EFI_DEP_NOT:\r
+      DEBUG ((DEBUG_DISPATCH, "  NOT\n"));\r
       Status = PopBool (&Operator);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
 \r
       Status = PushBool ((BOOLEAN)(!Operator));\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
       break;\r
 \r
-    case EFI_DEP_TRUE:   \r
+    case EFI_DEP_TRUE:\r
+      DEBUG ((DEBUG_DISPATCH, "  TRUE\n"));\r
       Status = PushBool (TRUE);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
       break;\r
 \r
-    case EFI_DEP_FALSE: \r
+    case EFI_DEP_FALSE:\r
+      DEBUG ((DEBUG_DISPATCH, "  FALSE\n"));\r
       Status = PushBool (FALSE);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
       break;\r
 \r
-    case EFI_DEP_END:    \r
+    case EFI_DEP_END:\r
+      DEBUG ((DEBUG_DISPATCH, "  END\n"));\r
       Status = PopBool (&Operator);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
+      DEBUG ((DEBUG_DISPATCH, "  RESULT = %a\n", Operator ? "TRUE" : "FALSE"));\r
       return Operator;\r
 \r
     case EFI_DEP_REPLACE_TRUE:\r
+      CopyMem (&DriverGuid, Iterator + 1, sizeof (EFI_GUID));\r
+      DEBUG ((DEBUG_DISPATCH, "  PUSH GUID(%g) = TRUE\n", &DriverGuid));\r
+      \r
       Status = PushBool (TRUE);\r
       if (EFI_ERROR (Status)) {\r
+        DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unexpected error)\n"));\r
         return FALSE;\r
       }\r
 \r
       Iterator += sizeof (EFI_GUID);\r
       break;\r
 \r
-    default:      \r
+    default:\r
+      DEBUG ((DEBUG_DISPATCH, "  RESULT = FALSE (Unknown opcode)\n"));\r
       goto Done;\r
     }\r
-    \r
+\r
     //\r
     // Skip over the Dependency Op Code we just processed in the switch.\r
     // The math is done out of order, but it should not matter. That is\r