[ros-diffs] [tkreuzer] 59147: [RTL] - Fix a format specifier [NTOSKRNL] - Add a missing compiler memory barrier to KeIpiGenericCall - Remove a pointless cast

Author: tkreuzer
Date: Sun Jun  2 18:01:00 2013
New Revision: 59147

URL: http://svn.reactos.org/svn/reactos?rev=59147&view=rev
Log:
[RTL]
- Fix a format specifier

[NTOSKRNL]
- Add a missing compiler memory barrier to KeIpiGenericCall
- Remove a pointless cast

Modified:
    trunk/reactos/lib/rtl/image.c
    trunk/reactos/ntoskrnl/ke/ipi.c

Modified: trunk/reactos/lib/rtl/image.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/image.c?rev=59147&…
==============================================================================
--- trunk/reactos/lib/rtl/image.c	[iso-8859-1] (original)
+++ trunk/reactos/lib/rtl/image.c	[iso-8859-1] Sun Jun  2 18:01:00 2013
@@ -417,7 +417,8 @@
         case IMAGE_REL_BASED_MIPS_JMPADDR:
         default:
             DPRINT1("Unknown/unsupported fixup type %hu.\n", Type);
-            DPRINT1("Address %x, Current %u, Count %u, *TypeOffset %x\n",
Address, i, Count, SWAPW(*TypeOffset));
+            DPRINT1("Address %p, Current %u, Count %u, *TypeOffset %x\n",
+                    (PVOID)Address, i, Count, SWAPW(*TypeOffset));
             return (PIMAGE_BASE_RELOCATION)NULL;
         }
 

Modified: trunk/reactos/ntoskrnl/ke/ipi.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/ipi.c?rev=5914…
==============================================================================
--- trunk/reactos/ntoskrnl/ke/ipi.c	[iso-8859-1] (original)
+++ trunk/reactos/ntoskrnl/ke/ipi.c	[iso-8859-1] Sun Jun  2 18:01:00 2013
@@ -228,7 +228,11 @@
                         &Count);
 
         /* Spin until the other processors are ready */
-        while ((volatile ULONG)Count != 1) YieldProcessor();
+        while (Count != 1)
+        {
+            YieldProcessor();
+            KeMemoryBarrierWithoutFence();
+        }
     }
 #endif
 
@@ -248,7 +252,7 @@
     if (Affinity)
     {
         /* Sanity check */
-        ASSERT(Prcb == (volatile PKPRCB)KeGetCurrentPrcb());
+        ASSERT(Prcb == KeGetCurrentPrcb());
 
         /* FIXME: TODO */
         ASSERTMSG("Not yet implemented\n", FALSE);