[Mono-bugs] [Bug 74639][Nor] New - instruction sync needed for atomic.h on ppc

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Sat, 16 Apr 2005 18:26:12 -0400 (EDT)

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by mass@akuma.org.


--- shadow/74639	2005-04-16 18:26:12.000000000 -0400
+++ shadow/74639.tmp.25679	2005-04-16 18:26:12.000000000 -0400
@@ -0,0 +1,28 @@
+Bug#: 74639
+Product: Mono: Runtime
+Version: 1.1
+OS Details: 
+Status: NEW   
+Priority: Normal
+Component: JIT
+AssignedTo: mono-bugs@ximian.com                            
+ReportedBy: mass@akuma.org               
+QAContact: mono-bugs@ximian.com
+TargetMilestone: ---
+Summary: instruction sync needed for atomic.h on ppc
+More fixes for atomic.h on ppc.
+The upcoming patch includes the changes suggested on the mailing list Wednesday (April 13th), as 
+well as addition of an isync call at the end of all the assembly routines.
+I believe this isync is neccessary - while the __volatile__ marker will prevent the assembly block 
+from being reordered with respect to other code by the compiler, the processor itself may still 
+reorder the execution of the instructions, for example executing branch prediction around the value 
+read from a refcount before that value has been successfully written to memory.