• Sam Lantinga's avatar
    Fixed bug 1128 · 58d7fc47
    Sam Lantinga authored
     Patrick Baggett      2011-02-16 22:58:33 PST
    
    This enhancement is for both x86/x64 Windows.
    
    The SDL implementation of mutexes uses the Win32 API interprocess
    synchronization primitive called a "Mutex". This implementation is subpar
    because it has a much higher overhead than an intraprocess mutex. The exact
    technical details are below, but my tests have shown that for reasonably high
    contention (10 threads on 4 physical cores), it has 13x higher overhead than
    the Win32 CriticalSection API.
    
    If this enhancement is accepted, I will write a patch to implement SDL mutexes
    using the critical section API, which should dramatically reduce overhead and
    improve scalability.
    
    
    -- Tech details --
    Normally, Win32 Mutexes are used across process boundaries to synchronize
    separate processes. In order to lock or unlock them, a user->kernel space
    transition is necessary, even in the uncontented case on a single CPU machine.
    Win32 CriticalSection objects can only be used within the same process virtual
    address space and thus to lock one, does not require a user->kernel space
    transition for the uncontended case, and additionally may spin a short while
    before going into kernel wait. This small spin allows a thread to obtain the
    lock if the mutex is released shortly after the thread starts spinning, in
    effect bypassing the overhead of user->kernel space transition which has higher
    overhead than the spinning itself.
    58d7fc47
Name
Last commit
Last update
VisualC Loading commit data...
VisualCE Loading commit data...
Xcode Loading commit data...
Xcode-iPhoneOS Loading commit data...
acinclude Loading commit data...
android-project Loading commit data...
build-scripts Loading commit data...
include Loading commit data...
src Loading commit data...
test Loading commit data...
.DISABLED-hgeol Loading commit data...
.hgignore Loading commit data...
.hgtags Loading commit data...
.indent.pro Loading commit data...
Android.mk Loading commit data...
BUGS Loading commit data...
Borland.html Loading commit data...
Borland.zip Loading commit data...
COPYING Loading commit data...
CREDITS Loading commit data...
INSTALL Loading commit data...
Makefile.ds Loading commit data...
Makefile.in Loading commit data...
Makefile.minimal Loading commit data...
Makefile.pandora Loading commit data...
Makefile.wiz Loading commit data...
NOTES Loading commit data...
README Loading commit data...
README-SDL.txt Loading commit data...
README.BeOS Loading commit data...
README.DirectFB Loading commit data...
README.HG Loading commit data...
README.MacOSX Loading commit data...
README.Platforms Loading commit data...
README.Porting Loading commit data...
README.Watcom Loading commit data...
README.WinCE Loading commit data...
README.android Loading commit data...
README.ds Loading commit data...
README.gesture Loading commit data...
README.iphoneos Loading commit data...
README.pandora Loading commit data...
README.touch Loading commit data...
SDL.spec.in Loading commit data...
TODO Loading commit data...
UNDER_CONSTRUCTION.txt Loading commit data...
VisualC.html Loading commit data...
Watcom-Win32.zip Loading commit data...
WhatsNew Loading commit data...
aclocal.m4 Loading commit data...
autogen.sh Loading commit data...
configure Loading commit data...
configure.in Loading commit data...
sdl-config.in Loading commit data...
sdl.m4 Loading commit data...
sdl.pc.in Loading commit data...