[Guile-commits] GNU Guile branch, wip-nj-locks-nc, created. release_1-8-6-36-g9ec74a6

[Neil Jerram]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GNU Guile".

http://git.savannah.gnu.org/cgit/guile.git/commit/?id=9ec74a6a4e3d5b1d4b9eb23d1045aad2b0536e05

The branch, wip-nj-locks-nc has been created
        at  9ec74a6a4e3d5b1d4b9eb23d1045aad2b0536e05 (commit)

- Log -----------------------------------------------------------------
commit 9ec74a6a4e3d5b1d4b9eb23d1045aad2b0536e05
Author: Neil Jerram <address@hidden>
Date:   Wed Mar 25 22:50:46 2009 +0000

    Allow non-weak hash tables to be automatically thread-safe
    
    This patch allows a fat mutex to be associated with a hash table.
    When a hash table has an associated mutex, that mutex will be
    automatically locked and unlocked around all operations (include
    lookups) on the hash table's internal data.
    
    * libguile/hashtab.c (make_hash_table): Init mutex field to #f.
    
      (hashtable_mark): New function.
    
      (scm_hash_fn_get_handle, scm_hash_fn_create_handle_x,
      scm_hash_fn_remove_x, scm_hash_clear_x, scm_internal_hash_fold,
      scm_internal_hash_for_each_handle): If the hash table has a mutex,
      lock and unlock it around all accessing of the hash table's
      internals.
    
      (scm_hash_use_mutex_x): New function.
    
      (scm_hashtab_prehistory): Use hashtable_mark as hash table mark
      function.
    
    * libguile/hashtab.h (SCM_HASHTABLE_MUTEX, SCM_SET_HASHTABLE_MUTEX,
      scm_hash_use_mutex_x): New declarations.
    
      (scm_t_hashtable): New mutex field.

commit 10b11bab3317f72f74b59c1b5becf731215ae2f4
Author: Neil Jerram <address@hidden>
Date:   Wed Mar 25 22:34:23 2009 +0000

    Make the interned symbols hash thread-safe
    
    These changes introduce a mutex that protects all accesses to and
    modifications of the symbols hash.  They allow the test-define-race
    test to run without errors for a long time.  (Well, for 200s anyway.)
    
    * libguile/environments.c (obarray_enter, obarray_replace): Extra
      parameter on scm_i_rehash calls.
    
    * libguile/hashtab.c: Include symbols.h.
    
      (scm_i_rehash): New mutex parameter.  When non-NULL, unlock and
      relock this mutex around allocations, and recheck new size
      calculation (in case another thread has changed it).
    
      (rehash_after_gc): Special treatment for the symbol hash.
    
      (scm_hash_fn_create_handle_x, scm_hash_fn_remove_x): Extra parameter on 
scm_i_rehash calls.
    
    * libguile/hashtab.h (scm_i_rehash): New mutex parameter.
    
    * libguile/symbols.c (symbols): Rename scm_i_symbols and make non-static.
    
      (symbols_mutex): New variable.
    
      (scm_sys_symbols): symbols -> scm_i_symbols.
    
      (lookup_interned_symbol): Lock symbols_mutex while performing lookup.
    
      (intern_symbol): Add name, len and raw_hash parameters.  Lock
      symbols_mutex while checking and modifying the symbols hash.  Once
      the mutex is locked, recheck in case another thread has already
      interned the symbol.  Return interned symbol, in order to tell
      callers about this last case.
    
      (scm_i_c_mem2symbol, scm_i_mem2symbol, scm_take_locale_symboln):
      Update intern_symbol calls.
    
      (scm_i_rehash_symbols_after_gc): New function.
    
      (scm_symbols_prehistory): symbols -> scm_i_symbols.  Initialize
      symbols_mutex.
    
    * libguile/symbols.h (scm_i_symbols, scm_i_rehash_symbols_after_gc):
      New declarations.

commit 1f2707cc9473548863b483fd12d97afe7d0c94c2
Author: Neil Jerram <address@hidden>
Date:   Wed Mar 25 20:55:37 2009 +0000

    New test for thread-safe define
    
    Written by Linas Vepstas; modified by Neil Jerram.
    
    * test-suite/standalone/Makefile.am: Add test-define-race test.
    
    * test-suite/standalone/test-define-race.c: New test.

-----------------------------------------------------------------------


hooks/post-receive
-- 
GNU Guile