Revision: 17068
          http://sourceforge.net/p/edk2/code/17068
Author:   lersek
Date:     2015-03-20 12:17:02 +0000 (Fri, 20 Mar 2015)
Log Message:
-----------
MdePkg: BaseSynchronizationLib: fix Increment/Decrement retvals for ARM

The procedure call standard dictates that we move the result to r0 before
returning.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Tyler Smith <[email protected]>
Reviewed-by: Eugene Cohen <[email protected]>
Reviewed-by: Leif Lindholm <[email protected]>
Reviewed-by: Ronald Cron <[email protected]>
[[email protected]: cleaned up commit message]

Revision Links:
--------------
    http://sourceforge.net/p/edk2/code/0

Modified Paths:
--------------
    trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.S
    trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.asm

Modified: trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.S
===================================================================
--- trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.S      
2015-03-20 03:27:24 UTC (rev 17067)
+++ trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.S      
2015-03-20 12:17:02 UTC (rev 17068)
@@ -179,6 +179,7 @@
   cmp     r2, #0
   bne     TryInternalSyncIncrement
   dmb
+  mov     r0, r1
   bx      lr
 
 /**
@@ -208,4 +209,5 @@
   cmp     r2, #0
   bne     TryInternalSyncDecrement
   dmb
+  mov     r0, r1
   bx      lr

Modified: 
trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.asm
===================================================================
--- trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.asm    
2015-03-20 03:27:24 UTC (rev 17067)
+++ trunk/edk2/MdePkg/Library/BaseSynchronizationLib/Arm/Synchronization.asm    
2015-03-20 12:17:02 UTC (rev 17068)
@@ -178,6 +178,7 @@
   cmp     r2, #0
   bne     TryInternalSyncIncrement
   dmb
+  mov     r0, r1
   bx      lr
 
 /**
@@ -207,6 +208,7 @@
   cmp     r2, #0
   bne     TryInternalSyncDecrement
   dmb
+  mov     r0, r1
   bx      lr
 
   END


------------------------------------------------------------------------------
Dive into the World of Parallel Programming The Go Parallel Website, sponsored
by Intel and developed in partnership with Slashdot Media, is your hub for all
things parallel software development, from weekly thought leadership blogs to
news, videos, case studies, tutorials and more. Take a look and join the 
conversation now. http://goparallel.sourceforge.net/
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to