]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Core/Dxe/Event/Timer.c
Apply code to avoid security warnings.
[mirror_edk2.git] / MdeModulePkg / Core / Dxe / Event / Timer.c
index c16326661720841a167d1e20d25ac1084d14336f..d0ca1fc9fca1c88aec758faee811dbbcfea7f1ff 100644 (file)
@@ -13,87 +13,60 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 **/\r
 \r
 \r
-#include <DxeMain.h>\r
+#include "DxeMain.h"\r
+#include "Event.h"\r
 \r
 //\r
-// Internal prototypes\r
+// Internal data\r
 //\r
-/**\r
-  Returns the current system time.\r
-\r
-  @return The current system time\r
-\r
-**/\r
-UINT64\r
-CoreCurrentSystemTime (\r
-  VOID\r
-  );\r
-\r
-/**\r
-  Checks the sorted timer list against the current system time.\r
-  Signals any expired event timer.\r
 \r
-  @param  CheckEvent             Not used \r
-  @param  Context                Not used\r
+LIST_ENTRY       mEfiTimerList = INITIALIZE_LIST_HEAD_VARIABLE (mEfiTimerList);\r
+EFI_LOCK         mEfiTimerLock = EFI_INITIALIZE_LOCK_VARIABLE (TPL_HIGH_LEVEL - 1);\r
+EFI_EVENT        mEfiCheckTimerEvent = NULL;\r
 \r
-**/\r
-VOID\r
-EFIAPI\r
-CoreCheckTimers (\r
-  IN EFI_EVENT            CheckEvent,\r
-  IN VOID                 *Context\r
-  );\r
+EFI_LOCK         mEfiSystemTimeLock = EFI_INITIALIZE_LOCK_VARIABLE (TPL_HIGH_LEVEL);\r
+UINT64           mEfiSystemTime = 0;\r
 \r
+//\r
+// Timer functions\r
+//\r
 /**\r
   Inserts the timer event.\r
 \r
-  @param  Event                  Points to the internal structure of timer event \r
+  @param  Event                  Points to the internal structure of timer event\r
                                  to be installed\r
 \r
 **/\r
 VOID\r
 CoreInsertEventTimer (\r
   IN IEVENT   *Event\r
-  );\r
-\r
-//\r
-// Internal data\r
-//\r
-\r
-STATIC LIST_ENTRY       mEfiTimerList = INITIALIZE_LIST_HEAD_VARIABLE (mEfiTimerList);\r
-STATIC EFI_LOCK         mEfiTimerLock = EFI_INITIALIZE_LOCK_VARIABLE (TPL_HIGH_LEVEL - 1);\r
-STATIC EFI_EVENT        mEfiCheckTimerEvent;\r
-\r
-STATIC EFI_LOCK         mEfiSystemTimeLock = EFI_INITIALIZE_LOCK_VARIABLE (TPL_HIGH_LEVEL);\r
-STATIC UINT64           mEfiSystemTime = 0;\r
+  )\r
+{\r
+  UINT64          TriggerTime;\r
+  LIST_ENTRY      *Link;\r
+  IEVENT          *Event2;\r
 \r
-//\r
-// Timer functions\r
-//\r
+  ASSERT_LOCKED (&mEfiTimerLock);\r
 \r
+  //\r
+  // Get the timer's trigger time\r
+  //\r
+  TriggerTime = Event->u.Timer.TriggerTime;\r
 \r
-/**\r
-  Initializes timer support.\r
+  //\r
+  // Insert the timer into the timer database in assending sorted order\r
+  //\r
+  for (Link = mEfiTimerList.ForwardLink; Link != &mEfiTimerList; Link = Link->ForwardLink) {\r
+    Event2 = CR (Link, IEVENT, u.Timer.Link, EVENT_SIGNATURE);\r
 \r
-**/\r
-VOID\r
-CoreInitializeTimer (\r
-  VOID\r
-  )\r
-{\r
-  EFI_STATUS  Status;\r
+    if (Event2->u.Timer.TriggerTime > TriggerTime) {\r
+      break;\r
+    }\r
+  }\r
 \r
-  Status = CoreCreateEvent (\r
-              EVT_NOTIFY_SIGNAL,\r
-              TPL_HIGH_LEVEL - 1,\r
-              CoreCheckTimers,\r
-              NULL,\r
-              &mEfiCheckTimerEvent\r
-              );\r
-  ASSERT_EFI_ERROR (Status);\r
+  InsertTailList (Link, &Event->u.Timer.Link);\r
 }\r
 \r
-\r
 /**\r
   Returns the current system time.\r
 \r
@@ -110,59 +83,15 @@ CoreCurrentSystemTime (
   CoreAcquireLock (&mEfiSystemTimeLock);\r
   SystemTime = mEfiSystemTime;\r
   CoreReleaseLock (&mEfiSystemTimeLock);\r
-  return SystemTime;\r
-}\r
-\r
-\r
-/**\r
-  Called by the platform code to process a tick.\r
-\r
-  @param  Duration               The number of 100ns elasped since the last call \r
-                                 to TimerTick\r
-\r
-**/\r
-VOID\r
-EFIAPI\r
-CoreTimerTick (\r
-  IN UINT64   Duration\r
-  )\r
-{\r
-  IEVENT          *Event;\r
-\r
-  //\r
-  // Check runtiem flag in case there are ticks while exiting boot services\r
-  //\r
 \r
-  CoreAcquireLock (&mEfiSystemTimeLock);\r
-\r
-  //\r
-  // Update the system time\r
-  //\r
-\r
-  mEfiSystemTime += Duration;\r
-\r
-  //\r
-  // If the head of the list is expired, fire the timer event\r
-  // to process it\r
-  //\r
-\r
-  if (!IsListEmpty (&mEfiTimerList)) {\r
-    Event = CR (mEfiTimerList.ForwardLink, IEVENT, u.Timer.Link, EVENT_SIGNATURE);\r
-\r
-    if (Event->u.Timer.TriggerTime <= mEfiSystemTime) {\r
-      CoreSignalEvent (mEfiCheckTimerEvent);\r
-    }\r
-  }\r
-\r
-  CoreReleaseLock (&mEfiSystemTimeLock);\r
+  return SystemTime;\r
 }\r
 \r
-\r
 /**\r
   Checks the sorted timer list against the current system time.\r
   Signals any expired event timer.\r
 \r
-  @param  CheckEvent             Not used \r
+  @param  CheckEvent             Not used\r
   @param  Context                Not used\r
 \r
 **/\r
@@ -179,7 +108,6 @@ CoreCheckTimers (
   //\r
   // Check the timer database for expired timers\r
   //\r
-\r
   CoreAcquireLock (&mEfiTimerLock);\r
   SystemTime = CoreCurrentSystemTime ();\r
 \r
@@ -189,7 +117,6 @@ CoreCheckTimers (
     //\r
     // If this timer is not expired, then we're done\r
     //\r
-\r
     if (Event->u.Timer.TriggerTime > SystemTime) {\r
       break;\r
     }\r
@@ -210,11 +137,9 @@ CoreCheckTimers (
     // If this is a periodic timer, set it\r
     //\r
     if (Event->u.Timer.Period) {\r
-\r
       //\r
       // Compute the timers new trigger time\r
       //\r
-\r
       Event->u.Timer.TriggerTime = Event->u.Timer.TriggerTime + Event->u.Timer.Period;\r
 \r
       //\r
@@ -228,7 +153,6 @@ CoreCheckTimers (
       //\r
       // Add the timer\r
       //\r
-\r
       CoreInsertEventTimer (Event);\r
     }\r
   }\r
@@ -238,59 +162,81 @@ CoreCheckTimers (
 \r
 \r
 /**\r
-  Inserts the timer event.\r
-\r
-  @param  Event                  Points to the internal structure of timer event \r
-                                 to be installed\r
+  Initializes timer support.\r
 \r
 **/\r
 VOID\r
-CoreInsertEventTimer (\r
-  IN IEVENT   *Event\r
+CoreInitializeTimer (\r
+  VOID\r
   )\r
 {\r
-  UINT64          TriggerTime;\r
-  LIST_ENTRY      *Link;\r
-  IEVENT          *Event2;\r
+  EFI_STATUS  Status;\r
 \r
-  ASSERT_LOCKED (&mEfiTimerLock);\r
+  Status = CoreCreateEvent (\r
+             EVT_NOTIFY_SIGNAL,\r
+             TPL_HIGH_LEVEL - 1,\r
+             CoreCheckTimers,\r
+             NULL,\r
+             &mEfiCheckTimerEvent\r
+             );\r
+  ASSERT_EFI_ERROR (Status);\r
+}\r
+\r
+\r
+/**\r
+  Called by the platform code to process a tick.\r
+\r
+  @param  Duration               The number of 100ns elasped since the last call\r
+                                 to TimerTick\r
+\r
+**/\r
+VOID\r
+EFIAPI\r
+CoreTimerTick (\r
+  IN UINT64   Duration\r
+  )\r
+{\r
+  IEVENT          *Event;\r
 \r
   //\r
-  // Get the timer's trigger time\r
+  // Check runtiem flag in case there are ticks while exiting boot services\r
   //\r
-\r
-  TriggerTime = Event->u.Timer.TriggerTime;\r
+  CoreAcquireLock (&mEfiSystemTimeLock);\r
 \r
   //\r
-  // Insert the timer into the timer database in assending sorted order\r
+  // Update the system time\r
   //\r
+  mEfiSystemTime += Duration;\r
 \r
-  for (Link = mEfiTimerList.ForwardLink; Link  != &mEfiTimerList; Link = Link->ForwardLink) {\r
-    Event2 = CR (Link, IEVENT, u.Timer.Link, EVENT_SIGNATURE);\r
+  //\r
+  // If the head of the list is expired, fire the timer event\r
+  // to process it\r
+  //\r
+  if (!IsListEmpty (&mEfiTimerList)) {\r
+    Event = CR (mEfiTimerList.ForwardLink, IEVENT, u.Timer.Link, EVENT_SIGNATURE);\r
 \r
-    if (Event2->u.Timer.TriggerTime > TriggerTime) {\r
-      break;\r
+    if (Event->u.Timer.TriggerTime <= mEfiSystemTime) {\r
+      CoreSignalEvent (mEfiCheckTimerEvent);\r
     }\r
   }\r
 \r
-  InsertTailList (Link, &Event->u.Timer.Link);\r
+  CoreReleaseLock (&mEfiSystemTimeLock);\r
 }\r
 \r
 \r
 \r
-\r
 /**\r
   Sets the type of timer and the trigger time for a timer event.\r
 \r
-  @param  UserEvent              The timer event that is to be signaled at the \r
-                                 specified time \r
-  @param  Type                   The type of time that is specified in \r
-                                 TriggerTime \r
-  @param  TriggerTime            The number of 100ns units until the timer \r
-                                 expires \r
+  @param  UserEvent              The timer event that is to be signaled at the\r
+                                 specified time\r
+  @param  Type                   The type of time that is specified in\r
+                                 TriggerTime\r
+  @param  TriggerTime            The number of 100ns units until the timer\r
+                                 expires\r
 \r
-  @retval EFI_SUCCESS            The event has been set to be signaled at the \r
-                                 requested time \r
+  @retval EFI_SUCCESS            The event has been set to be signaled at the\r
+                                 requested time\r
   @retval EFI_INVALID_PARAMETER  Event or Type is not valid\r
 \r
 **/\r
@@ -314,7 +260,7 @@ CoreSetTimer (
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
-  if (Type < 0 || Type > TimerRelative  || !(Event->Type & EVT_TIMER)) {\r
+  if (Type < 0 || Type > TimerRelative  || (Event->Type & EVT_TIMER) == 0) {\r
     return EFI_INVALID_PARAMETER;\r
   }\r
 \r
@@ -323,7 +269,6 @@ CoreSetTimer (
   //\r
   // If the timer is queued to the timer database, remove it\r
   //\r
-\r
   if (Event->u.Timer.Link.ForwardLink != NULL) {\r
     RemoveEntryList (&Event->u.Timer.Link);\r
     Event->u.Timer.Link.ForwardLink = NULL;\r
@@ -347,5 +292,6 @@ CoreSetTimer (
   }\r
 \r
   CoreReleaseLock (&mEfiTimerLock);\r
+\r
   return EFI_SUCCESS;\r
 }\r