public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile
@ 2016-11-17 19:34 Michael Kinney
  2016-11-17 19:34 ` [Patch v2 1/3] MdePkg/Include: Add volatile to SynchronizationLib parameters Michael Kinney
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Michael Kinney @ 2016-11-17 19:34 UTC (permalink / raw)
  To: edk2-devel

Updates for V2
==============
* Split out patch that corrects function names in comment blocks
* Change InterlockedCompareExchangePointer() parameter from volatile VOID**
  to VOID * volatile *.
* Remove unnessary typecast in calls to InterlockedCompareExchangePointer()
* Fix missing copyright updates
* Minor formatting changes

Michael Kinney (3):
  MdePkg/Include: Add volatile to SynchronizationLib parameters
  MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs
  MdePkg/BaseSynchronizationLib: Fix function names in function headers

 MdePkg/Include/Library/SynchronizationLib.h              | 14 +++++++-------
 .../Ia32/InterlockedCompareExchange16.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange16.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange16.nasm               |  4 ++--
 .../Ia32/InterlockedCompareExchange32.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange32.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange32.nasm               |  4 ++--
 .../Ia32/InterlockedCompareExchange64.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange64.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange64.nasm               |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedDecrement.asm |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedDecrement.c   |  4 ++--
 .../Ia32/InterlockedDecrement.nasm                       |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedIncrement.asm |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedIncrement.c   |  4 ++--
 .../Ia32/InterlockedIncrement.nasm                       |  4 ++--
 MdePkg/Library/BaseSynchronizationLib/Synchronization.c  | 16 ++++++++--------
 .../Library/BaseSynchronizationLib/SynchronizationGcc.c  | 16 ++++++++--------
 .../Library/BaseSynchronizationLib/SynchronizationMsc.c  | 16 ++++++++--------
 .../X64/InterlockedCompareExchange16.asm                 |  6 +++---
 .../X64/InterlockedCompareExchange16.c                   |  4 ++--
 .../X64/InterlockedCompareExchange16.nasm                |  6 +++---
 .../X64/InterlockedCompareExchange32.asm                 |  6 +++---
 .../X64/InterlockedCompareExchange32.c                   |  4 ++--
 .../X64/InterlockedCompareExchange32.nasm                |  6 +++---
 .../X64/InterlockedCompareExchange64.asm                 |  6 +++---
 .../X64/InterlockedCompareExchange64.c                   |  4 ++--
 .../X64/InterlockedCompareExchange64.nasm                |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedDecrement.asm  |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedDecrement.c    |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedDecrement.nasm |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedIncrement.asm  |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedIncrement.c    |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedIncrement.nasm |  6 +++---
 34 files changed, 103 insertions(+), 103 deletions(-)

-- 
2.6.3.windows.1



^ permalink raw reply	[flat|nested] 6+ messages in thread

* [Patch v2 1/3] MdePkg/Include: Add volatile to SynchronizationLib parameters
  2016-11-17 19:34 [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Michael Kinney
@ 2016-11-17 19:34 ` Michael Kinney
  2016-11-17 19:34 ` [Patch v2 2/3] MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs Michael Kinney
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Michael Kinney @ 2016-11-17 19:34 UTC (permalink / raw)
  To: edk2-devel; +Cc: Liming Gao, Laszlo Ersek, Andrew Fish, Jeff Fan

The SpinLock functions in the SynchronicationLib use volatile
parameters to keep compiler from optimizing these functions
too much.  The volatile keyword is missing from the Interlocked*()
functions in this same library class.  Update the library class
to consistently use volatile on all functions in this class.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Andrew Fish <afish@apple.com>
Cc: Jeff Fan <jeff.fan@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
---
 MdePkg/Include/Library/SynchronizationLib.h | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/MdePkg/Include/Library/SynchronizationLib.h b/MdePkg/Include/Library/SynchronizationLib.h
index 7b97683..6cf3d71 100644
--- a/MdePkg/Include/Library/SynchronizationLib.h
+++ b/MdePkg/Include/Library/SynchronizationLib.h
@@ -1,7 +1,7 @@
 /** @file
   Provides synchronization functions.
 
-Copyright (c) 2006 - 2012, Intel Corporation. All rights reserved.<BR>
+Copyright (c) 2006 - 2016, 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
@@ -157,7 +157,7 @@ ReleaseSpinLock (
 UINT32
 EFIAPI
 InterlockedIncrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   );
 
 
@@ -179,7 +179,7 @@ InterlockedIncrement (
 UINT32
 EFIAPI
 InterlockedDecrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   );
 
 
@@ -204,7 +204,7 @@ InterlockedDecrement (
 UINT16
 EFIAPI
 InterlockedCompareExchange16 (
-  IN OUT  UINT16                    *Value,
+  IN OUT  volatile UINT16           *Value,
   IN      UINT16                    CompareValue,
   IN      UINT16                    ExchangeValue
   );
@@ -231,7 +231,7 @@ InterlockedCompareExchange16 (
 UINT32
 EFIAPI
 InterlockedCompareExchange32 (
-  IN OUT  UINT32                    *Value,
+  IN OUT  volatile UINT32           *Value,
   IN      UINT32                    CompareValue,
   IN      UINT32                    ExchangeValue
   );
@@ -258,7 +258,7 @@ InterlockedCompareExchange32 (
 UINT64
 EFIAPI
 InterlockedCompareExchange64 (
-  IN OUT  UINT64                    *Value,
+  IN OUT  volatile UINT64           *Value,
   IN      UINT64                    CompareValue,
   IN      UINT64                    ExchangeValue
   );
@@ -285,7 +285,7 @@ InterlockedCompareExchange64 (
 VOID *
 EFIAPI
 InterlockedCompareExchangePointer (
-  IN OUT  VOID                      **Value,
+  IN OUT  VOID                      * volatile *Value,
   IN      VOID                      *CompareValue,
   IN      VOID                      *ExchangeValue
   );
-- 
2.6.3.windows.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Patch v2 2/3] MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs
  2016-11-17 19:34 [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Michael Kinney
  2016-11-17 19:34 ` [Patch v2 1/3] MdePkg/Include: Add volatile to SynchronizationLib parameters Michael Kinney
@ 2016-11-17 19:34 ` Michael Kinney
  2016-11-17 19:34 ` [Patch v2 3/3] MdePkg/BaseSynchronizationLib: Fix function names in function headers Michael Kinney
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Michael Kinney @ 2016-11-17 19:34 UTC (permalink / raw)
  To: edk2-devel; +Cc: Liming Gao, Laszlo Ersek, Andrew Fish, Jeff Fan

The SpinLock functions in the SynchronicationLib use volatile
parameters to keep compiler from optimizing these functions
too much.  The volatile keyword is missing from the Interlocked*()
functions in this same library instance.  Update the library instance
to consistently use volatile on all functions in the
SynchronizationLib class.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Andrew Fish <afish@apple.com>
Cc: Jeff Fan <jeff.fan@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
---
 .../Ia32/InterlockedCompareExchange16.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange16.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange16.nasm               |  4 ++--
 .../Ia32/InterlockedCompareExchange32.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange32.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange32.nasm               |  4 ++--
 .../Ia32/InterlockedCompareExchange64.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange64.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange64.nasm               |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedDecrement.asm |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedDecrement.c   |  4 ++--
 .../Ia32/InterlockedDecrement.nasm                       |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedIncrement.asm |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedIncrement.c   |  4 ++--
 .../Ia32/InterlockedIncrement.nasm                       |  4 ++--
 MdePkg/Library/BaseSynchronizationLib/Synchronization.c  | 16 ++++++++--------
 .../Library/BaseSynchronizationLib/SynchronizationGcc.c  | 16 ++++++++--------
 .../Library/BaseSynchronizationLib/SynchronizationMsc.c  | 16 ++++++++--------
 .../X64/InterlockedCompareExchange16.asm                 |  4 ++--
 .../X64/InterlockedCompareExchange16.c                   |  4 ++--
 .../X64/InterlockedCompareExchange16.nasm                |  4 ++--
 .../X64/InterlockedCompareExchange32.asm                 |  4 ++--
 .../X64/InterlockedCompareExchange32.c                   |  4 ++--
 .../X64/InterlockedCompareExchange32.nasm                |  4 ++--
 .../X64/InterlockedCompareExchange64.asm                 |  4 ++--
 .../X64/InterlockedCompareExchange64.c                   |  4 ++--
 .../X64/InterlockedCompareExchange64.nasm                |  4 ++--
 .../BaseSynchronizationLib/X64/InterlockedDecrement.asm  |  4 ++--
 .../BaseSynchronizationLib/X64/InterlockedDecrement.c    |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedDecrement.nasm |  4 ++--
 .../BaseSynchronizationLib/X64/InterlockedIncrement.asm  |  4 ++--
 .../BaseSynchronizationLib/X64/InterlockedIncrement.c    |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedIncrement.nasm |  4 ++--
 33 files changed, 86 insertions(+), 86 deletions(-)

diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.asm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.asm
index 92a0f49..7362fe9 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>
 ; Copyright (c) 2015, Linaro Ltd. All rights reserved.<BR>
 ; This program and the accompanying materials
 ; are licensed and made available under the terms and conditions of the BSD License
@@ -30,7 +30,7 @@
 ; UINT16
 ; EFIAPI
 ; InternalSyncCompareExchange16 (
-;   IN      UINT16                    *Value,
+;   IN      volatile UINT16           *Value,
 ;   IN      UINT16                    CompareValue,
 ;   IN      UINT16                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.c b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.c
index 3d52137..0c3a4f6 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.c
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedCompareExchange16 function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>
   Copyright (c) 2015, Linaro Ltd. All rights reserved.<BR>
   This program and the accompanying materials
   are licensed and made available under the terms and conditions of the BSD License
@@ -36,7 +36,7 @@
 UINT16
 EFIAPI
 InternalSyncCompareExchange16 (
-  IN      UINT16                    *Value,
+  IN      volatile UINT16           *Value,
   IN      UINT16                    CompareValue,
   IN      UINT16                    ExchangeValue
   )
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.nasm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.nasm
index 8cabe66..7fbfd75 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange16.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>
 ; Copyright (c) 2015, Linaro Ltd. All rights reserved.<BR>
 ; This program and the accompanying materials
 ; are licensed and made available under the terms and conditions of the BSD License
@@ -28,7 +28,7 @@
 ; UINT16
 ; EFIAPI
 ; InternalSyncCompareExchange16 (
-;   IN      UINT16                    *Value,
+;   IN      volatile UINT16           *Value,
 ;   IN      UINT16                    CompareValue,
 ;   IN      UINT16                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.asm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.asm
index 78ea72c..1a4a611 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -29,7 +29,7 @@
 ; UINT32
 ; EFIAPI
 ; InternalSyncCompareExchange32 (
-;   IN      UINT32                    *Value,
+;   IN      volatile UINT32           *Value,
 ;   IN      UINT32                    CompareValue,
 ;   IN      UINT32                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.c b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.c
index a2c6838..cf80dde 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.c
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedCompareExchange32 function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -35,7 +35,7 @@
 UINT32
 EFIAPI
 InternalSyncCompareExchange32 (
-  IN      UINT32                    *Value,
+  IN      volatile UINT32           *Value,
   IN      UINT32                    CompareValue,
   IN      UINT32                    ExchangeValue
   )
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.nasm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.nasm
index 5393273..9064c65 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange32.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT32
 ; EFIAPI
 ; InternalSyncCompareExchange32 (
-;   IN      UINT32                    *Value,
+;   IN      volatile UINT32           *Value,
 ;   IN      UINT32                    CompareValue,
 ;   IN      UINT32                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.asm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.asm
index 0fcbd23..e7b02db 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -29,7 +29,7 @@
 ; UINT64
 ; EFIAPI
 ; InternalSyncCompareExchange64 (
-;   IN      UINT64                    *Value,
+;   IN      volatile UINT64           *Value,
 ;   IN      UINT64                    CompareValue,
 ;   IN      UINT64                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.c b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.c
index 73af6ef..0bdbd79 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.c
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedCompareExchange64 function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -34,7 +34,7 @@
 UINT64
 EFIAPI
 InternalSyncCompareExchange64 (
-  IN      UINT64                    *Value,
+  IN      volatile UINT64           *Value,
   IN      UINT64                    CompareValue,
   IN      UINT64                    ExchangeValue
   )
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.nasm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.nasm
index 206de40..c5c3e12 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedCompareExchange64.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT64
 ; EFIAPI
 ; InternalSyncCompareExchange64 (
-;   IN      UINT64                    *Value,
+;   IN      volatile UINT64           *Value,
 ;   IN      UINT64                    CompareValue,
 ;   IN      UINT64                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.asm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.asm
index 22cb0b2..b5fa447 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -29,7 +29,7 @@
 ; UINT32
 ; EFIAPI
 ; InternalSyncDecrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 InternalSyncDecrement   PROC
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.c b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.c
index 7f18e0b..354a0e7 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.c
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedDecrement function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -31,7 +31,7 @@
 UINT32
 EFIAPI
 InternalSyncDecrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   _asm {
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.nasm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.nasm
index a7414be..4c46041 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedDecrement.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT32
 ; EFIAPI
 ; InternalSyncDecrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 global ASM_PFX(InternalSyncDecrement)
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.asm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.asm
index 51675f6..61c2533 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -29,7 +29,7 @@
 ; UINT32
 ; EFIAPI
 ; InternalSyncIncrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 InternalSyncIncrement   PROC
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.c b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.c
index cf9f92a..c61a550 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.c
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.c
@@ -1,7 +1,7 @@
 /** @file
   InterLockedIncrement function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -31,7 +31,7 @@
 UINT32
 EFIAPI
 InternalSyncIncrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   _asm {
diff --git a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.nasm b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.nasm
index 24f24f5..3902c73 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/Ia32/InterlockedIncrement.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT32
 ; EFIAPI
 ; InternalSyncIncrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 global ASM_PFX(InternalSyncIncrement)
diff --git a/MdePkg/Library/BaseSynchronizationLib/Synchronization.c b/MdePkg/Library/BaseSynchronizationLib/Synchronization.c
index bb0c8e9..4f2e21b 100644
--- a/MdePkg/Library/BaseSynchronizationLib/Synchronization.c
+++ b/MdePkg/Library/BaseSynchronizationLib/Synchronization.c
@@ -244,7 +244,7 @@ ReleaseSpinLock (
 UINT32
 EFIAPI
 InterlockedIncrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   ASSERT (Value != NULL);
@@ -269,7 +269,7 @@ InterlockedIncrement (
 UINT32
 EFIAPI
 InterlockedDecrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   ASSERT (Value != NULL);
@@ -298,7 +298,7 @@ InterlockedDecrement (
 UINT16
 EFIAPI
 InterlockedCompareExchange16 (
-  IN OUT  UINT16                    *Value,
+  IN OUT  volatile UINT16           *Value,
   IN      UINT16                    CompareValue,
   IN      UINT16                    ExchangeValue
   )
@@ -329,7 +329,7 @@ InterlockedCompareExchange16 (
 UINT32
 EFIAPI
 InterlockedCompareExchange32 (
-  IN OUT  UINT32                    *Value,
+  IN OUT  volatile UINT32           *Value,
   IN      UINT32                    CompareValue,
   IN      UINT32                    ExchangeValue
   )
@@ -359,7 +359,7 @@ InterlockedCompareExchange32 (
 UINT64
 EFIAPI
 InterlockedCompareExchange64 (
-  IN OUT  UINT64                    *Value,
+  IN OUT  volatile UINT64           *Value,
   IN      UINT64                    CompareValue,
   IN      UINT64                    ExchangeValue
   )
@@ -389,7 +389,7 @@ InterlockedCompareExchange64 (
 VOID *
 EFIAPI
 InterlockedCompareExchangePointer (
-  IN OUT  VOID                      **Value,
+  IN OUT  VOID                      * volatile *Value,
   IN      VOID                      *CompareValue,
   IN      VOID                      *ExchangeValue
   )
@@ -401,13 +401,13 @@ InterlockedCompareExchangePointer (
   switch (SizeOfValue) {
     case sizeof (UINT32):
       return (VOID*)(UINTN)InterlockedCompareExchange32 (
-                             (UINT32*)Value,
+                             (volatile UINT32 *)Value,
                              (UINT32)(UINTN)CompareValue,
                              (UINT32)(UINTN)ExchangeValue
                              );
     case sizeof (UINT64):
       return (VOID*)(UINTN)InterlockedCompareExchange64 (
-                             (UINT64*)Value,
+                             (volatile UINT64 *)Value,
                              (UINT64)(UINTN)CompareValue,
                              (UINT64)(UINTN)ExchangeValue
                              );
diff --git a/MdePkg/Library/BaseSynchronizationLib/SynchronizationGcc.c b/MdePkg/Library/BaseSynchronizationLib/SynchronizationGcc.c
index 4b8c8e5..98c9648 100644
--- a/MdePkg/Library/BaseSynchronizationLib/SynchronizationGcc.c
+++ b/MdePkg/Library/BaseSynchronizationLib/SynchronizationGcc.c
@@ -260,7 +260,7 @@ ReleaseSpinLock (
 UINT32
 EFIAPI
 InterlockedIncrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   ASSERT (Value != NULL);
@@ -285,7 +285,7 @@ InterlockedIncrement (
 UINT32
 EFIAPI
 InterlockedDecrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   ASSERT (Value != NULL);
@@ -314,7 +314,7 @@ InterlockedDecrement (
 UINT16
 EFIAPI
 InterlockedCompareExchange16 (
-  IN OUT  UINT16                    *Value,
+  IN OUT  volatile UINT16           *Value,
   IN      UINT16                    CompareValue,
   IN      UINT16                    ExchangeValue
   )
@@ -345,7 +345,7 @@ InterlockedCompareExchange16 (
 UINT32
 EFIAPI
 InterlockedCompareExchange32 (
-  IN OUT  UINT32                    *Value,
+  IN OUT  volatile UINT32           *Value,
   IN      UINT32                    CompareValue,
   IN      UINT32                    ExchangeValue
   )
@@ -375,7 +375,7 @@ InterlockedCompareExchange32 (
 UINT64
 EFIAPI
 InterlockedCompareExchange64 (
-  IN OUT  UINT64                    *Value,
+  IN OUT  volatile UINT64           *Value,
   IN      UINT64                    CompareValue,
   IN      UINT64                    ExchangeValue
   )
@@ -405,7 +405,7 @@ InterlockedCompareExchange64 (
 VOID *
 EFIAPI
 InterlockedCompareExchangePointer (
-  IN OUT  VOID                      **Value,
+  IN OUT  VOID                      * volatile *Value,
   IN      VOID                      *CompareValue,
   IN      VOID                      *ExchangeValue
   )
@@ -417,13 +417,13 @@ InterlockedCompareExchangePointer (
   switch (SizeOfValue) {
     case sizeof (UINT32):
       return (VOID*)(UINTN)InterlockedCompareExchange32 (
-                             (UINT32*)Value,
+                             (volatile UINT32 *)Value,
                              (UINT32)(UINTN)CompareValue,
                              (UINT32)(UINTN)ExchangeValue
                              );
     case sizeof (UINT64):
       return (VOID*)(UINTN)InterlockedCompareExchange64 (
-                             (UINT64*)Value,
+                             (volatile UINT64 *)Value,
                              (UINT64)(UINTN)CompareValue,
                              (UINT64)(UINTN)ExchangeValue
                              );
diff --git a/MdePkg/Library/BaseSynchronizationLib/SynchronizationMsc.c b/MdePkg/Library/BaseSynchronizationLib/SynchronizationMsc.c
index db344b7..8bc6d26 100644
--- a/MdePkg/Library/BaseSynchronizationLib/SynchronizationMsc.c
+++ b/MdePkg/Library/BaseSynchronizationLib/SynchronizationMsc.c
@@ -262,7 +262,7 @@ ReleaseSpinLock (
 UINT32
 EFIAPI
 InterlockedIncrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   ASSERT (Value != NULL);
@@ -287,7 +287,7 @@ InterlockedIncrement (
 UINT32
 EFIAPI
 InterlockedDecrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
   ASSERT (Value != NULL);
@@ -316,7 +316,7 @@ InterlockedDecrement (
 UINT16
 EFIAPI
 InterlockedCompareExchange16 (
-  IN OUT  UINT16                    *Value,
+  IN OUT  volatile UINT16           *Value,
   IN      UINT16                    CompareValue,
   IN      UINT16                    ExchangeValue
   )
@@ -347,7 +347,7 @@ InterlockedCompareExchange16 (
 UINT32
 EFIAPI
 InterlockedCompareExchange32 (
-  IN OUT  UINT32                    *Value,
+  IN OUT  volatile UINT32           *Value,
   IN      UINT32                    CompareValue,
   IN      UINT32                    ExchangeValue
   )
@@ -377,7 +377,7 @@ InterlockedCompareExchange32 (
 UINT64
 EFIAPI
 InterlockedCompareExchange64 (
-  IN OUT  UINT64                    *Value,
+  IN OUT  volatile UINT64           *Value,
   IN      UINT64                    CompareValue,
   IN      UINT64                    ExchangeValue
   )
@@ -407,7 +407,7 @@ InterlockedCompareExchange64 (
 VOID *
 EFIAPI
 InterlockedCompareExchangePointer (
-  IN OUT  VOID                      **Value,
+  IN OUT  VOID                      * volatile *Value,
   IN      VOID                      *CompareValue,
   IN      VOID                      *ExchangeValue
   )
@@ -419,13 +419,13 @@ InterlockedCompareExchangePointer (
   switch (SizeOfValue) {
     case sizeof (UINT32):
       return (VOID*)(UINTN)InterlockedCompareExchange32 (
-                             (UINT32*)Value,
+                             (volatile UINT32*)Value,
                              (UINT32)(UINTN)CompareValue,
                              (UINT32)(UINTN)ExchangeValue
                              );
     case sizeof (UINT64):
       return (VOID*)(UINTN)InterlockedCompareExchange64 (
-                             (UINT64*)Value,
+                             (volatile UINT64*)Value,
                              (UINT64)(UINTN)CompareValue,
                              (UINT64)(UINTN)ExchangeValue
                              );
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm
index 7926662..e891922 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>
 ; Copyright (c) 2015, Linaro Ltd. All rights reserved.<BR>
 ; This program and the accompanying materials
 ; are licensed and made available under the terms and conditions of the BSD License
@@ -28,7 +28,7 @@
 ; UINT16
 ; EFIAPI
 ; InterlockedCompareExchange16 (
-;   IN      UINT16                    *Value,
+;   IN      volatile UINT16           *Value,
 ;   IN      UINT16                    CompareValue,
 ;   IN      UINT16                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.c b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.c
index 76aa6fb..8b51ecb 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.c
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedCompareExchange16 function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>
   Copyright (c) 2015, Linaro Ltd. All rights reserved.<BR>
   This program and the accompanying materials
   are licensed and made available under the terms and conditions of the BSD License
@@ -44,7 +44,7 @@ __int16 _InterlockedCompareExchange16(
 UINT16
 EFIAPI
 InternalSyncCompareExchange16 (
-  IN      UINT16                    *Value,
+  IN      volatile UINT16           *Value,
   IN      UINT16                    CompareValue,
   IN      UINT16                    ExchangeValue
   )
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm
index 1d72d07..efe3249 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, Intel Corporation. All rights reserved.<BR>
 ; Copyright (c) 2015, Linaro Ltd. All rights reserved.<BR>
 ; This program and the accompanying materials
 ; are licensed and made available under the terms and conditions of the BSD License
@@ -29,7 +29,7 @@
 ; UINT16
 ; EFIAPI
 ; InterlockedCompareExchange16 (
-;   IN      UINT16                    *Value,
+;   IN      volatile UINT16           *Value,
 ;   IN      UINT16                    CompareValue,
 ;   IN      UINT16                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm
index ee94ff7..269a3c7 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT32
 ; EFIAPI
 ; InterlockedCompareExchange32 (
-;   IN      UINT32                    *Value,
+;   IN      volatile UINT32           *Value,
 ;   IN      UINT32                    CompareValue,
 ;   IN      UINT32                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.c b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.c
index 9f8f3d3..ef407af 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.c
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedCompareExchange32 function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -44,7 +44,7 @@ long _InterlockedCompareExchange(
 UINT32
 EFIAPI
 InternalSyncCompareExchange32 (
-  IN      UINT32                    *Value,
+  IN      volatile UINT32           *Value,
   IN      UINT32                    CompareValue,
   IN      UINT32                    ExchangeValue
   )
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm
index 4c8c810..31142d0 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -28,7 +28,7 @@
 ; UINT32
 ; EFIAPI
 ; InterlockedCompareExchange32 (
-;   IN      UINT32                    *Value,
+;   IN      volatile UINT32           *Value,
 ;   IN      UINT32                    CompareValue,
 ;   IN      UINT32                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm
index b5bcd25..80d03fd 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT64
 ; EFIAPI
 ; InterlockedCompareExchange64 (
-;   IN      UINT64                    *Value,
+;   IN      volatile UINT64           *Value,
 ;   IN      UINT64                    CompareValue,
 ;   IN      UINT64                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.c b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.c
index 56805b3..a8f9fbb 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.c
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedCompareExchange64 function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -43,7 +43,7 @@ __int64 _InterlockedCompareExchange64(
 UINT64
 EFIAPI
 InternalSyncCompareExchange64 (
-  IN      UINT64                    *Value,
+  IN      volatile UINT64           *Value,
   IN      UINT64                    CompareValue,
   IN      UINT64                    ExchangeValue
   )
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm
index 72c350a..819daed 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -28,7 +28,7 @@
 ; UINT64
 ; EFIAPI
 ; InterlockedCompareExchange64 (
-;   IN      UINT64                    *Value,
+;   IN      volatile UINT64           *Value,
 ;   IN      UINT64                    CompareValue,
 ;   IN      UINT64                    ExchangeValue
 ;   );
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm
index fdea0d4..23f03c4 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT32
 ; EFIAPI
 ; InterlockedDecrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 InternalSyncDecrement   PROC
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.c b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.c
index 65faf01..da402cd 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.c
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.c
@@ -1,7 +1,7 @@
 /** @file
   InterlockedDecrement function
 
-  Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -38,9 +38,9 @@ long _InterlockedDecrement(
 UINT32
 EFIAPI
 InternalSyncDecrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
-  return _InterlockedDecrement (Value);
+  return _InterlockedDecrement ((long *)(UINTN)(Value));
 }
 
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm
index e140e44..780d902 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -28,7 +28,7 @@
 ; UINT32
 ; EFIAPI
 ; InterlockedDecrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 global ASM_PFX(InternalSyncDecrement)
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm
index 65abcf7..44f264f 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -27,7 +27,7 @@
 ; UINT32
 ; EFIAPI
 ; InterlockedIncrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 InternalSyncIncrement   PROC
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.c b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.c
index c0deb60..bbd1384 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.c
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.c
@@ -1,7 +1,7 @@
 /** @file
   InterLockedIncrement function
 
-  Copyright (c) 2006 - 2008, Intel Corporation. All rights reserved.<BR>
+  Copyright (c) 2006 - 2016, 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
@@ -38,9 +38,9 @@ long _InterlockedIncrement(
 UINT32
 EFIAPI
 InternalSyncIncrement (
-  IN      UINT32                    *Value
+  IN      volatile UINT32           *Value
   )
 {
-  return _InterlockedIncrement (Value);
+  return _InterlockedIncrement ((long *)(UINTN)(Value));
 }
 
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm
index 59d90be..e1c70a4 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm
@@ -1,6 +1,6 @@
 ;------------------------------------------------------------------------------
 ;
-; Copyright (c) 2006, Intel Corporation. All rights reserved.<BR>
+; Copyright (c) 2006 - 2016, 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
@@ -28,7 +28,7 @@
 ; UINT32
 ; EFIAPI
 ; InterlockedIncrement (
-;   IN      UINT32                    *Value
+;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
 global ASM_PFX(InternalSyncIncrement)
-- 
2.6.3.windows.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [Patch v2 3/3] MdePkg/BaseSynchronizationLib: Fix function names in function headers
  2016-11-17 19:34 [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Michael Kinney
  2016-11-17 19:34 ` [Patch v2 1/3] MdePkg/Include: Add volatile to SynchronizationLib parameters Michael Kinney
  2016-11-17 19:34 ` [Patch v2 2/3] MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs Michael Kinney
@ 2016-11-17 19:34 ` Michael Kinney
  2016-11-17 19:48 ` [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Laszlo Ersek
  2016-11-18  0:55 ` Gao, Liming
  4 siblings, 0 replies; 6+ messages in thread
From: Michael Kinney @ 2016-11-17 19:34 UTC (permalink / raw)
  To: edk2-devel; +Cc: Liming Gao, Laszlo Ersek, Andrew Fish, Jeff Fan

Some of the function names in function header comment blocks in
assembly files do not match the symbol name in the assembly sources.
Update function header comment blocks to match symbol name.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Cc: Andrew Fish <afish@apple.com>
Cc: Jeff Fan <jeff.fan@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Michael Kinney <michael.d.kinney@intel.com>
---
 .../Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm | 2 +-
 .../BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm        | 2 +-
 .../Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm | 2 +-
 .../BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm        | 2 +-
 .../Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm | 2 +-
 .../BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm        | 2 +-
 MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm      | 2 +-
 MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm     | 2 +-
 MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm      | 2 +-
 MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm     | 2 +-
 10 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm
index e891922..262adeb 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.asm
@@ -27,7 +27,7 @@
 ;------------------------------------------------------------------------------
 ; UINT16
 ; EFIAPI
-; InterlockedCompareExchange16 (
+; InternalSyncCompareExchange16 (
 ;   IN      volatile UINT16           *Value,
 ;   IN      UINT16                    CompareValue,
 ;   IN      UINT16                    ExchangeValue
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm
index efe3249..eebed5a 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange16.nasm
@@ -28,7 +28,7 @@
 ;------------------------------------------------------------------------------
 ; UINT16
 ; EFIAPI
-; InterlockedCompareExchange16 (
+; InternalSyncCompareExchange16 (
 ;   IN      volatile UINT16           *Value,
 ;   IN      UINT16                    CompareValue,
 ;   IN      UINT16                    ExchangeValue
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm
index 269a3c7..711b399 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.asm
@@ -26,7 +26,7 @@
 ;------------------------------------------------------------------------------
 ; UINT32
 ; EFIAPI
-; InterlockedCompareExchange32 (
+; InternalSyncCompareExchange32 (
 ;   IN      volatile UINT32           *Value,
 ;   IN      UINT32                    CompareValue,
 ;   IN      UINT32                    ExchangeValue
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm
index 31142d0..5ed4ba5 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange32.nasm
@@ -27,7 +27,7 @@
 ;------------------------------------------------------------------------------
 ; UINT32
 ; EFIAPI
-; InterlockedCompareExchange32 (
+; InternalSyncCompareExchange32 (
 ;   IN      volatile UINT32           *Value,
 ;   IN      UINT32                    CompareValue,
 ;   IN      UINT32                    ExchangeValue
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm
index 80d03fd..be429d8 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.asm
@@ -26,7 +26,7 @@
 ;------------------------------------------------------------------------------
 ; UINT64
 ; EFIAPI
-; InterlockedCompareExchange64 (
+; InternalSyncCompareExchange64 (
 ;   IN      volatile UINT64           *Value,
 ;   IN      UINT64                    CompareValue,
 ;   IN      UINT64                    ExchangeValue
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm
index 819daed..2b268b6 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedCompareExchange64.nasm
@@ -27,7 +27,7 @@
 ;------------------------------------------------------------------------------
 ; UINT64
 ; EFIAPI
-; InterlockedCompareExchange64 (
+; InternalSyncCompareExchange64 (
 ;   IN      volatile UINT64           *Value,
 ;   IN      UINT64                    CompareValue,
 ;   IN      UINT64                    ExchangeValue
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm
index 23f03c4..4dc3886 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.asm
@@ -26,7 +26,7 @@
 ;------------------------------------------------------------------------------
 ; UINT32
 ; EFIAPI
-; InterlockedDecrement (
+; InternalSyncDecrement (
 ;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm
index 780d902..60f4311 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedDecrement.nasm
@@ -27,7 +27,7 @@
 ;------------------------------------------------------------------------------
 ; UINT32
 ; EFIAPI
-; InterlockedDecrement (
+; InternalSyncDecrement (
 ;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm
index 44f264f..fc086b6 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.asm
@@ -26,7 +26,7 @@
 ;------------------------------------------------------------------------------
 ; UINT32
 ; EFIAPI
-; InterlockedIncrement (
+; InternalSyncIncrement (
 ;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
diff --git a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm
index e1c70a4..7f877b5 100644
--- a/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm
+++ b/MdePkg/Library/BaseSynchronizationLib/X64/InterlockedIncrement.nasm
@@ -27,7 +27,7 @@
 ;------------------------------------------------------------------------------
 ; UINT32
 ; EFIAPI
-; InterlockedIncrement (
+; InternalSyncIncrement (
 ;   IN      volatile UINT32           *Value
 ;   );
 ;------------------------------------------------------------------------------
-- 
2.6.3.windows.1



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile
  2016-11-17 19:34 [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Michael Kinney
                   ` (2 preceding siblings ...)
  2016-11-17 19:34 ` [Patch v2 3/3] MdePkg/BaseSynchronizationLib: Fix function names in function headers Michael Kinney
@ 2016-11-17 19:48 ` Laszlo Ersek
  2016-11-18  0:55 ` Gao, Liming
  4 siblings, 0 replies; 6+ messages in thread
From: Laszlo Ersek @ 2016-11-17 19:48 UTC (permalink / raw)
  To: Michael Kinney, edk2-devel

On 11/17/16 20:34, Michael Kinney wrote:
> Updates for V2
> ==============
> * Split out patch that corrects function names in comment blocks
> * Change InterlockedCompareExchangePointer() parameter from volatile VOID**
>   to VOID * volatile *.
> * Remove unnessary typecast in calls to InterlockedCompareExchangePointer()
> * Fix missing copyright updates
> * Minor formatting changes
> 
> Michael Kinney (3):
>   MdePkg/Include: Add volatile to SynchronizationLib parameters
>   MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs
>   MdePkg/BaseSynchronizationLib: Fix function names in function headers
> 
>  MdePkg/Include/Library/SynchronizationLib.h              | 14 +++++++-------
>  .../Ia32/InterlockedCompareExchange16.asm                |  4 ++--
>  .../Ia32/InterlockedCompareExchange16.c                  |  4 ++--
>  .../Ia32/InterlockedCompareExchange16.nasm               |  4 ++--
>  .../Ia32/InterlockedCompareExchange32.asm                |  4 ++--
>  .../Ia32/InterlockedCompareExchange32.c                  |  4 ++--
>  .../Ia32/InterlockedCompareExchange32.nasm               |  4 ++--
>  .../Ia32/InterlockedCompareExchange64.asm                |  4 ++--
>  .../Ia32/InterlockedCompareExchange64.c                  |  4 ++--
>  .../Ia32/InterlockedCompareExchange64.nasm               |  4 ++--
>  .../BaseSynchronizationLib/Ia32/InterlockedDecrement.asm |  4 ++--
>  .../BaseSynchronizationLib/Ia32/InterlockedDecrement.c   |  4 ++--
>  .../Ia32/InterlockedDecrement.nasm                       |  4 ++--
>  .../BaseSynchronizationLib/Ia32/InterlockedIncrement.asm |  4 ++--
>  .../BaseSynchronizationLib/Ia32/InterlockedIncrement.c   |  4 ++--
>  .../Ia32/InterlockedIncrement.nasm                       |  4 ++--
>  MdePkg/Library/BaseSynchronizationLib/Synchronization.c  | 16 ++++++++--------
>  .../Library/BaseSynchronizationLib/SynchronizationGcc.c  | 16 ++++++++--------
>  .../Library/BaseSynchronizationLib/SynchronizationMsc.c  | 16 ++++++++--------
>  .../X64/InterlockedCompareExchange16.asm                 |  6 +++---
>  .../X64/InterlockedCompareExchange16.c                   |  4 ++--
>  .../X64/InterlockedCompareExchange16.nasm                |  6 +++---
>  .../X64/InterlockedCompareExchange32.asm                 |  6 +++---
>  .../X64/InterlockedCompareExchange32.c                   |  4 ++--
>  .../X64/InterlockedCompareExchange32.nasm                |  6 +++---
>  .../X64/InterlockedCompareExchange64.asm                 |  6 +++---
>  .../X64/InterlockedCompareExchange64.c                   |  4 ++--
>  .../X64/InterlockedCompareExchange64.nasm                |  6 +++---
>  .../BaseSynchronizationLib/X64/InterlockedDecrement.asm  |  6 +++---
>  .../BaseSynchronizationLib/X64/InterlockedDecrement.c    |  6 +++---
>  .../BaseSynchronizationLib/X64/InterlockedDecrement.nasm |  6 +++---
>  .../BaseSynchronizationLib/X64/InterlockedIncrement.asm  |  6 +++---
>  .../BaseSynchronizationLib/X64/InterlockedIncrement.c    |  6 +++---
>  .../BaseSynchronizationLib/X64/InterlockedIncrement.nasm |  6 +++---
>  34 files changed, 103 insertions(+), 103 deletions(-)
> 

series
Reviewed-by: Laszlo Ersek <lersek@redhat.com>


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile
  2016-11-17 19:34 [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Michael Kinney
                   ` (3 preceding siblings ...)
  2016-11-17 19:48 ` [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Laszlo Ersek
@ 2016-11-18  0:55 ` Gao, Liming
  4 siblings, 0 replies; 6+ messages in thread
From: Gao, Liming @ 2016-11-18  0:55 UTC (permalink / raw)
  To: Kinney, Michael D, edk2-devel@lists.01.org

Reviewed-by: Liming Gao <liming.gao@intel.com>

-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Michael Kinney
Sent: Friday, November 18, 2016 3:35 AM
To: edk2-devel@lists.01.org
Subject: [edk2] [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile

Updates for V2
==============
* Split out patch that corrects function names in comment blocks
* Change InterlockedCompareExchangePointer() parameter from volatile VOID**
  to VOID * volatile *.
* Remove unnessary typecast in calls to InterlockedCompareExchangePointer()
* Fix missing copyright updates
* Minor formatting changes

Michael Kinney (3):
  MdePkg/Include: Add volatile to SynchronizationLib parameters
  MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs
  MdePkg/BaseSynchronizationLib: Fix function names in function headers

 MdePkg/Include/Library/SynchronizationLib.h              | 14 +++++++-------
 .../Ia32/InterlockedCompareExchange16.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange16.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange16.nasm               |  4 ++--
 .../Ia32/InterlockedCompareExchange32.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange32.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange32.nasm               |  4 ++--
 .../Ia32/InterlockedCompareExchange64.asm                |  4 ++--
 .../Ia32/InterlockedCompareExchange64.c                  |  4 ++--
 .../Ia32/InterlockedCompareExchange64.nasm               |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedDecrement.asm |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedDecrement.c   |  4 ++--
 .../Ia32/InterlockedDecrement.nasm                       |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedIncrement.asm |  4 ++--
 .../BaseSynchronizationLib/Ia32/InterlockedIncrement.c   |  4 ++--
 .../Ia32/InterlockedIncrement.nasm                       |  4 ++--
 MdePkg/Library/BaseSynchronizationLib/Synchronization.c  | 16 ++++++++--------
 .../Library/BaseSynchronizationLib/SynchronizationGcc.c  | 16 ++++++++--------
 .../Library/BaseSynchronizationLib/SynchronizationMsc.c  | 16 ++++++++--------
 .../X64/InterlockedCompareExchange16.asm                 |  6 +++---
 .../X64/InterlockedCompareExchange16.c                   |  4 ++--
 .../X64/InterlockedCompareExchange16.nasm                |  6 +++---
 .../X64/InterlockedCompareExchange32.asm                 |  6 +++---
 .../X64/InterlockedCompareExchange32.c                   |  4 ++--
 .../X64/InterlockedCompareExchange32.nasm                |  6 +++---
 .../X64/InterlockedCompareExchange64.asm                 |  6 +++---
 .../X64/InterlockedCompareExchange64.c                   |  4 ++--
 .../X64/InterlockedCompareExchange64.nasm                |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedDecrement.asm  |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedDecrement.c    |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedDecrement.nasm |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedIncrement.asm  |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedIncrement.c    |  6 +++---
 .../BaseSynchronizationLib/X64/InterlockedIncrement.nasm |  6 +++---
 34 files changed, 103 insertions(+), 103 deletions(-)

-- 
2.6.3.windows.1

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-11-18  0:55 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-17 19:34 [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Michael Kinney
2016-11-17 19:34 ` [Patch v2 1/3] MdePkg/Include: Add volatile to SynchronizationLib parameters Michael Kinney
2016-11-17 19:34 ` [Patch v2 2/3] MdePkg/BaseSynchronizationLib: Add volatile Interlocked*() APIs Michael Kinney
2016-11-17 19:34 ` [Patch v2 3/3] MdePkg/BaseSynchronizationLib: Fix function names in function headers Michael Kinney
2016-11-17 19:48 ` [Patch v2 0/3] MdePkg/BaseSynchronizationLib: Add volatile Laszlo Ersek
2016-11-18  0:55 ` Gao, Liming

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox