]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdePkg/Library/BaseSynchronizationLib/SynchronizationMsc.c
MdePkg: Apply uncrustify changes
[mirror_edk2.git] / MdePkg / Library / BaseSynchronizationLib / SynchronizationMsc.c
index 90524a1017ae7c927f9893c73ecb1bf296f42611..f99895a39cf971060e0f163ab26c22ffcb9dc8f3 100644 (file)
@@ -1,14 +1,8 @@
 /** @file\r
   Implementation of synchronization functions.\r
 \r
-  Copyright (c) 2006 - 2008, 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
-\r
-  THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,\r
-  WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.\r
+  Copyright (c) 2006 - 2018, Intel Corporation. All rights reserved.<BR>\r
+  SPDX-License-Identifier: BSD-2-Clause-Patent\r
 \r
 **/\r
 \r
   Microsoft Visual Studio 7.1 Function Prototypes for read write barrier Intrinsics.\r
 **/\r
 \r
-void    _ReadWriteBarrier (void);\r
-#pragma intrinsic(_ReadWriteBarrier)\r
+void\r
+_ReadWriteBarrier (\r
+  void\r
+  );\r
 \r
+#pragma intrinsic(_ReadWriteBarrier)\r
 \r
-#define SPIN_LOCK_RELEASED          ((UINTN) 1)\r
-#define SPIN_LOCK_ACQUIRED          ((UINTN) 2)\r
+#define SPIN_LOCK_RELEASED  ((UINTN) 1)\r
+#define SPIN_LOCK_ACQUIRED  ((UINTN) 2)\r
 \r
 /**\r
   Retrieves the architecture specific spin lock alignment requirements for\r
   optimal spin lock performance.\r
 \r
   This function retrieves the spin lock alignment requirements for optimal\r
-  performance on a given CPU architecture. The spin lock alignment must be a\r
-  power of two and is returned by this function. If there are no alignment\r
+  performance on a given CPU architecture. The spin lock alignment is byte alignment.\r
+  It must be a power of two and is returned by this function. If there are no alignment\r
   requirements, then 1 must be returned. The spin lock synchronization\r
   functions must function correctly if the spin lock size and alignment values\r
   returned by this function are not used at all. These values are hints to the\r
@@ -47,7 +44,7 @@ GetSpinLockProperties (
   VOID\r
   )\r
 {\r
-  return 32;\r
+  return InternalGetSpinLockProperties ();\r
 }\r
 \r
 /**\r
@@ -69,14 +66,14 @@ GetSpinLockProperties (
 SPIN_LOCK *\r
 EFIAPI\r
 InitializeSpinLock (\r
-  OUT      SPIN_LOCK                 *SpinLock\r
+  OUT      SPIN_LOCK  *SpinLock\r
   )\r
 {\r
   ASSERT (SpinLock != NULL);\r
 \r
-  _ReadWriteBarrier();\r
+  _ReadWriteBarrier ();\r
   *SpinLock = SPIN_LOCK_RELEASED;\r
-  _ReadWriteBarrier();\r
+  _ReadWriteBarrier ();\r
 \r
   return SpinLock;\r
 }\r
@@ -104,7 +101,7 @@ InitializeSpinLock (
 SPIN_LOCK *\r
 EFIAPI\r
 AcquireSpinLock (\r
-  IN OUT  SPIN_LOCK                 *SpinLock\r
+  IN OUT  SPIN_LOCK  *SpinLock\r
   )\r
 {\r
   UINT64  Current;\r
@@ -116,11 +113,15 @@ AcquireSpinLock (
   INT64   Cycle;\r
   INT64   Delta;\r
 \r
-  if (PcdGet32 (PcdSpinLockTimeout) > 0) {\r
+  if (PcdGet32 (PcdSpinLockTimeout) == 0) {\r
+    while (!AcquireSpinLockOrFail (SpinLock)) {\r
+      CpuPause ();\r
+    }\r
+  } else if (!AcquireSpinLockOrFail (SpinLock)) {\r
     //\r
     // Get the current timer value\r
     //\r
-    Current = GetPerformanceCounter();\r
+    Current = GetPerformanceCounter ();\r
 \r
     //\r
     // Initialize local variables\r
@@ -144,27 +145,27 @@ AcquireSpinLock (
     if (Cycle < 0) {\r
       Cycle = -Cycle;\r
     }\r
+\r
     Cycle++;\r
 \r
     while (!AcquireSpinLockOrFail (SpinLock)) {\r
       CpuPause ();\r
       Previous = Current;\r
-      Current  = GetPerformanceCounter();\r
-      Delta = (INT64) (Current - Previous);\r
+      Current  = GetPerformanceCounter ();\r
+      Delta    = (INT64)(Current - Previous);\r
       if (Start > End) {\r
         Delta = -Delta;\r
       }\r
+\r
       if (Delta < 0) {\r
         Delta += Cycle;\r
       }\r
+\r
       Total += Delta;\r
       ASSERT (Total < Timeout);\r
     }\r
-  } else {\r
-    while (!AcquireSpinLockOrFail (SpinLock)) {\r
-      CpuPause ();\r
-    }\r
   }\r
+\r
   return SpinLock;\r
 }\r
 \r
@@ -188,12 +189,12 @@ AcquireSpinLock (
 BOOLEAN\r
 EFIAPI\r
 AcquireSpinLockOrFail (\r
-  IN OUT  SPIN_LOCK                 *SpinLock\r
+  IN OUT  SPIN_LOCK  *SpinLock\r
   )\r
 {\r
-  SPIN_LOCK   LockValue;\r
-  VOID        *Result;\r
-  \r
+  SPIN_LOCK  LockValue;\r
+  VOID       *Result;\r
+\r
   ASSERT (SpinLock != NULL);\r
 \r
   LockValue = *SpinLock;\r
@@ -201,13 +202,13 @@ AcquireSpinLockOrFail (
 \r
   _ReadWriteBarrier ();\r
   Result = InterlockedCompareExchangePointer (\r
-             (VOID**)SpinLock,\r
-             (VOID*)SPIN_LOCK_RELEASED,\r
-             (VOID*)SPIN_LOCK_ACQUIRED\r
-           );\r
+             (VOID **)SpinLock,\r
+             (VOID *)SPIN_LOCK_RELEASED,\r
+             (VOID *)SPIN_LOCK_ACQUIRED\r
+             );\r
 \r
   _ReadWriteBarrier ();\r
-  return (BOOLEAN) (Result == (VOID*) SPIN_LOCK_RELEASED);\r
+  return (BOOLEAN)(Result == (VOID *)SPIN_LOCK_RELEASED);\r
 }\r
 \r
 /**\r
@@ -227,10 +228,10 @@ AcquireSpinLockOrFail (
 SPIN_LOCK *\r
 EFIAPI\r
 ReleaseSpinLock (\r
-  IN OUT  SPIN_LOCK                 *SpinLock\r
+  IN OUT  SPIN_LOCK  *SpinLock\r
   )\r
 {\r
-  SPIN_LOCK    LockValue;\r
+  SPIN_LOCK  LockValue;\r
 \r
   ASSERT (SpinLock != NULL);\r
 \r
@@ -249,8 +250,7 @@ ReleaseSpinLock (
 \r
   Performs an atomic increment of the 32-bit unsigned integer specified by\r
   Value and returns the incremented value. The increment operation must be\r
-  performed using MP safe mechanisms. The state of the return value is not\r
-  guaranteed to be MP safe.\r
+  performed using MP safe mechanisms.\r
 \r
   If Value is NULL, then ASSERT().\r
 \r
@@ -262,7 +262,7 @@ ReleaseSpinLock (
 UINT32\r
 EFIAPI\r
 InterlockedIncrement (\r
-  IN      UINT32                    *Value\r
+  IN      volatile UINT32  *Value\r
   )\r
 {\r
   ASSERT (Value != NULL);\r
@@ -274,8 +274,7 @@ InterlockedIncrement (
 \r
   Performs an atomic decrement of the 32-bit unsigned integer specified by\r
   Value and returns the decremented value. The decrement operation must be\r
-  performed using MP safe mechanisms. The state of the return value is not\r
-  guaranteed to be MP safe.\r
+  performed using MP safe mechanisms.\r
 \r
   If Value is NULL, then ASSERT().\r
 \r
@@ -287,13 +286,44 @@ InterlockedIncrement (
 UINT32\r
 EFIAPI\r
 InterlockedDecrement (\r
-  IN      UINT32                    *Value\r
+  IN      volatile UINT32  *Value\r
   )\r
 {\r
   ASSERT (Value != NULL);\r
   return InternalSyncDecrement (Value);\r
 }\r
 \r
+/**\r
+  Performs an atomic compare exchange operation on a 16-bit unsigned integer.\r
+\r
+  Performs an atomic compare exchange operation on the 16-bit unsigned integer\r
+  specified by Value.  If Value is equal to CompareValue, then Value is set to\r
+  ExchangeValue and CompareValue is returned.  If Value is not equal to CompareValue,\r
+  then Value is returned.  The compare exchange operation must be performed using\r
+  MP safe mechanisms.\r
+\r
+  If Value is NULL, then ASSERT().\r
+\r
+  @param  Value         A pointer to the 16-bit value for the compare exchange\r
+                        operation.\r
+  @param  CompareValue  A 16-bit value used in a compare operation.\r
+  @param  ExchangeValue A 16-bit value used in an exchange operation.\r
+\r
+  @return The original *Value before exchange.\r
+\r
+**/\r
+UINT16\r
+EFIAPI\r
+InterlockedCompareExchange16 (\r
+  IN OUT  volatile UINT16  *Value,\r
+  IN      UINT16           CompareValue,\r
+  IN      UINT16           ExchangeValue\r
+  )\r
+{\r
+  ASSERT (Value != NULL);\r
+  return InternalSyncCompareExchange16 (Value, CompareValue, ExchangeValue);\r
+}\r
+\r
 /**\r
   Performs an atomic compare exchange operation on a 32-bit unsigned integer.\r
 \r
@@ -316,9 +346,9 @@ InterlockedDecrement (
 UINT32\r
 EFIAPI\r
 InterlockedCompareExchange32 (\r
-  IN OUT  UINT32                    *Value,\r
-  IN      UINT32                    CompareValue,\r
-  IN      UINT32                    ExchangeValue\r
+  IN OUT  volatile UINT32  *Value,\r
+  IN      UINT32           CompareValue,\r
+  IN      UINT32           ExchangeValue\r
   )\r
 {\r
   ASSERT (Value != NULL);\r
@@ -346,9 +376,9 @@ InterlockedCompareExchange32 (
 UINT64\r
 EFIAPI\r
 InterlockedCompareExchange64 (\r
-  IN OUT  UINT64                    *Value,\r
-  IN      UINT64                    CompareValue,\r
-  IN      UINT64                    ExchangeValue\r
+  IN OUT  volatile UINT64  *Value,\r
+  IN      UINT64           CompareValue,\r
+  IN      UINT64           ExchangeValue\r
   )\r
 {\r
   ASSERT (Value != NULL);\r
@@ -376,28 +406,28 @@ InterlockedCompareExchange64 (
 VOID *\r
 EFIAPI\r
 InterlockedCompareExchangePointer (\r
-  IN OUT  VOID                      **Value,\r
-  IN      VOID                      *CompareValue,\r
-  IN      VOID                      *ExchangeValue\r
+  IN OUT  VOID                      *volatile  *Value,\r
+  IN      VOID                                 *CompareValue,\r
+  IN      VOID                                 *ExchangeValue\r
   )\r
 {\r
   UINT8  SizeOfValue;\r
 \r
-  SizeOfValue = sizeof (*Value);\r
+  SizeOfValue = (UINT8)sizeof (*Value);\r
 \r
   switch (SizeOfValue) {\r
     case sizeof (UINT32):\r
-      return (VOID*)(UINTN)InterlockedCompareExchange32 (\r
-                             (UINT32*)Value,\r
-                             (UINT32)(UINTN)CompareValue,\r
-                             (UINT32)(UINTN)ExchangeValue\r
-                             );\r
+      return (VOID *)(UINTN)InterlockedCompareExchange32 (\r
+                              (volatile UINT32 *)Value,\r
+                              (UINT32)(UINTN)CompareValue,\r
+                              (UINT32)(UINTN)ExchangeValue\r
+                              );\r
     case sizeof (UINT64):\r
-      return (VOID*)(UINTN)InterlockedCompareExchange64 (\r
-                             (UINT64*)Value,\r
-                             (UINT64)(UINTN)CompareValue,\r
-                             (UINT64)(UINTN)ExchangeValue\r
-                             );\r
+      return (VOID *)(UINTN)InterlockedCompareExchange64 (\r
+                              (volatile UINT64 *)Value,\r
+                              (UINT64)(UINTN)CompareValue,\r
+                              (UINT64)(UINTN)ExchangeValue\r
+                              );\r
     default:\r
       ASSERT (FALSE);\r
       return NULL;\r