diff options
Diffstat (limited to 'generic/tclThreadAlloc.c')
| -rw-r--r--[-rwxr-xr-x] | generic/tclThreadAlloc.c | 144 | 
1 files changed, 92 insertions, 52 deletions
| diff --git a/generic/tclThreadAlloc.c b/generic/tclThreadAlloc.c index e32ad84..ddf888a 100755..100644 --- a/generic/tclThreadAlloc.c +++ b/generic/tclThreadAlloc.c @@ -10,8 +10,6 @@   *   * See the file "license.terms" for information on usage and redistribution of   * this file, and for a DISCLAIMER OF ALL WARRANTIES. - * - * RCS: @(#) $Id: tclThreadAlloc.c,v 1.27.2.1 2009/09/29 04:43:59 dgp Exp $   */  #include "tclInt.h" @@ -37,7 +35,9 @@   */  #define NOBJALLOC	800 -#define NOBJHIGH	1200 + +/* Actual definition moved to tclInt.h */ +#define NOBJHIGH	ALLOC_NOBJHIGH  /*   * The following union stores accounting information for each block including @@ -97,7 +97,9 @@ typedef struct Bucket {  /*   * The following structure defines a cache of buckets and objs, of which there - * will be (at most) one per thread. + * will be (at most) one per thread. Any changes need to be reflected in the + * struct AllocCache defined in tclInt.h, possibly also in the initialisation + * code in Tcl_CreateInterp().   */  typedef struct Cache { @@ -143,6 +145,26 @@ static Tcl_Mutex *objLockPtr;  static Cache sharedCache;  static Cache *sharedPtr = &sharedCache;  static Cache *firstCachePtr = &sharedCache; + +#if defined(HAVE_FAST_TSD) +static __thread Cache *tcachePtr; + +# define GETCACHE(cachePtr)			\ +    do {					\ +	if (!tcachePtr) {			\ +	    tcachePtr = GetCache();		\ +	}					\ +	(cachePtr) = tcachePtr;			\ +    } while (0) +#else +# define GETCACHE(cachePtr)			\ +    do {					\ +	(cachePtr) = TclpGetAllocCache();	\ +	if ((cachePtr) == NULL) {		\ +	    (cachePtr) = GetCache();		\ +	}					\ +    } while (0) +#endif  /*   *---------------------------------------------------------------------- @@ -293,6 +315,7 @@ TclpAlloc(      register int bucket;      size_t size; +#ifndef __LP64__      if (sizeof(int) >= sizeof(size_t)) {  	/* An unsigned int overflow can also be a size_t overflow */  	const size_t zero = 0; @@ -303,11 +326,9 @@ TclpAlloc(  	    return NULL;  	}      } +#endif -    cachePtr = TclpGetAllocCache(); -    if (cachePtr == NULL) { -	cachePtr = GetCache(); -    } +    GETCACHE(cachePtr);      /*       * Increment the requested size to include room for the Block structure. @@ -319,7 +340,7 @@ TclpAlloc(      blockPtr = NULL;      size = reqSize + sizeof(Block);  #if RCHECK -    ++size; +    size++;  #endif      if (size > MAXALLOC) {  	bucket = NBUCKETS; @@ -330,13 +351,13 @@ TclpAlloc(      } else {  	bucket = 0;  	while (bucketInfo[bucket].blockSize < size) { -	    ++bucket; +	    bucket++;  	}  	if (cachePtr->buckets[bucket].numFree || GetBlocks(cachePtr, bucket)) {  	    blockPtr = cachePtr->buckets[bucket].firstPtr;  	    cachePtr->buckets[bucket].firstPtr = blockPtr->nextBlock; -	    --cachePtr->buckets[bucket].numFree; -	    ++cachePtr->buckets[bucket].numRemoves; +	    cachePtr->buckets[bucket].numFree--; +	    cachePtr->buckets[bucket].numRemoves++;  	    cachePtr->buckets[bucket].totalAssigned += reqSize;  	}      } @@ -374,10 +395,7 @@ TclpFree(  	return;      } -    cachePtr = TclpGetAllocCache(); -    if (cachePtr == NULL) { -	cachePtr = GetCache(); -    } +    GETCACHE(cachePtr);      /*       * Get the block back from the user pointer and call system free directly @@ -396,8 +414,8 @@ TclpFree(      cachePtr->buckets[bucket].totalAssigned -= blockPtr->blockReqSize;      blockPtr->nextBlock = cachePtr->buckets[bucket].firstPtr;      cachePtr->buckets[bucket].firstPtr = blockPtr; -    ++cachePtr->buckets[bucket].numFree; -    ++cachePtr->buckets[bucket].numInserts; +    cachePtr->buckets[bucket].numFree++; +    cachePtr->buckets[bucket].numInserts++;      if (cachePtr != sharedPtr &&  	    cachePtr->buckets[bucket].numFree > bucketInfo[bucket].maxBlocks) { @@ -436,6 +454,7 @@ TclpRealloc(  	return TclpAlloc(reqSize);      } +#ifndef __LP64__      if (sizeof(int) >= sizeof(size_t)) {  	/* An unsigned int overflow can also be a size_t overflow */  	const size_t zero = 0; @@ -446,11 +465,9 @@ TclpRealloc(  	    return NULL;  	}      } +#endif -    cachePtr = TclpGetAllocCache(); -    if (cachePtr == NULL) { -	cachePtr = GetCache(); -    } +    GETCACHE(cachePtr);      /*       * If the block is not a system block and fits in place, simply return the @@ -461,7 +478,7 @@ TclpRealloc(      blockPtr = Ptr2Block(ptr);      size = reqSize + sizeof(Block);  #if RCHECK -    ++size; +    size++;  #endif      bucket = blockPtr->sourceBucket;      if (bucket != NBUCKETS) { @@ -514,18 +531,20 @@ TclpRealloc(   *	May move Tcl_Obj's from shared cached or allocate new Tcl_Obj's if   *	list is empty.   * + * Note: + *	If this code is updated, the changes need to be reflected in the macro + *	TclAllocObjStorageEx() defined in tclInt.h + *   *----------------------------------------------------------------------   */  Tcl_Obj *  TclThreadAllocObj(void)  { -    register Cache *cachePtr = TclpGetAllocCache(); +    register Cache *cachePtr;      register Tcl_Obj *objPtr; -    if (cachePtr == NULL) { -	cachePtr = GetCache(); -    } +    GETCACHE(cachePtr);      /*       * Get this thread's obj list structure and move or allocate new objs if @@ -554,7 +573,7 @@ TclThreadAllocObj(void)  	    }  	    while (--numMove >= 0) {  		objPtr = &newObjsPtr[numMove]; -		objPtr->internalRep.otherValuePtr = cachePtr->firstObjPtr; +		objPtr->internalRep.twoPtrValue.ptr1 = cachePtr->firstObjPtr;  		cachePtr->firstObjPtr = objPtr;  	    }  	} @@ -565,8 +584,8 @@ TclThreadAllocObj(void)       */      objPtr = cachePtr->firstObjPtr; -    cachePtr->firstObjPtr = objPtr->internalRep.otherValuePtr; -    --cachePtr->numObjects; +    cachePtr->firstObjPtr = objPtr->internalRep.twoPtrValue.ptr1; +    cachePtr->numObjects--;      return objPtr;  } @@ -583,6 +602,10 @@ TclThreadAllocObj(void)   * Side effects:   *	May move free Tcl_Obj's to shared list upon hitting high water mark.   * + * Note: + *	If this code is updated, the changes need to be reflected in the macro + *	TclAllocObjStorageEx() defined in tclInt.h + *   *----------------------------------------------------------------------   */ @@ -590,19 +613,17 @@ void  TclThreadFreeObj(      Tcl_Obj *objPtr)  { -    Cache *cachePtr = TclpGetAllocCache(); +    Cache *cachePtr; -    if (cachePtr == NULL) { -	cachePtr = GetCache(); -    } +    GETCACHE(cachePtr);      /*       * Get this thread's list and push on the free Tcl_Obj.       */ -    objPtr->internalRep.otherValuePtr = cachePtr->firstObjPtr; +    objPtr->internalRep.twoPtrValue.ptr1 = cachePtr->firstObjPtr;      cachePtr->firstObjPtr = objPtr; -    ++cachePtr->numObjects; +    cachePtr->numObjects++;      /*       * If the number of free objects has exceeded the high water mark, move @@ -701,16 +722,16 @@ MoveObjs(       */      while (--numMove) { -	objPtr = objPtr->internalRep.otherValuePtr; +	objPtr = objPtr->internalRep.twoPtrValue.ptr1;      } -    fromPtr->firstObjPtr = objPtr->internalRep.otherValuePtr; +    fromPtr->firstObjPtr = objPtr->internalRep.twoPtrValue.ptr1;      /*       * Move all objects as a block - they are already linked to each other, we       * just have to update the first and last.       */ -    objPtr->internalRep.otherValuePtr = toPtr->firstObjPtr; +    objPtr->internalRep.twoPtrValue.ptr1 = toPtr->firstObjPtr;      toPtr->firstObjPtr = fromFirstObjPtr;  } @@ -791,17 +812,9 @@ LockBucket(      Cache *cachePtr,      int bucket)  { -#if 0 -    if (Tcl_MutexTryLock(bucketInfo[bucket].lockPtr) != TCL_OK) { -	Tcl_MutexLock(bucketInfo[bucket].lockPtr); -	++cachePtr->buckets[bucket].numWaits; -	++sharedPtr->buckets[bucket].numWaits; -    } -#else      Tcl_MutexLock(bucketInfo[bucket].lockPtr); -#endif -    ++cachePtr->buckets[bucket].numLocks; -    ++sharedPtr->buckets[bucket].numLocks; +    cachePtr->buckets[bucket].numLocks++; +    sharedPtr->buckets[bucket].numLocks++;  }  static void @@ -940,7 +953,7 @@ GetBlocks(  		size = bucketInfo[n].blockSize;  		blockPtr = cachePtr->buckets[n].firstPtr;  		cachePtr->buckets[n].firstPtr = blockPtr->nextBlock; -		--cachePtr->buckets[n].numFree; +		cachePtr->buckets[n].numFree--;  		break;  	    }  	} @@ -997,8 +1010,8 @@ TclFinalizeThreadAlloc(void)      unsigned int i;      for (i = 0; i < NBUCKETS; ++i) { -        TclpFreeAllocMutex(bucketInfo[i].lockPtr); -        bucketInfo[i].lockPtr = NULL; +	TclpFreeAllocMutex(bucketInfo[i].lockPtr); +	bucketInfo[i].lockPtr = NULL;      }      TclpFreeAllocMutex(objLockPtr); @@ -1010,6 +1023,33 @@ TclFinalizeThreadAlloc(void)      TclpFreeAllocCache(NULL);  } +/* + *---------------------------------------------------------------------- + * + * TclFinalizeThreadAllocThread -- + * + *	This procedure is used to destroy single thread private resources used + *	in this file.  + * Called in TclpFinalizeThreadData when a thread exits (Tcl_FinalizeThread). + * + * Results: + *	None. + * + * Side effects: + *	None. + * + *---------------------------------------------------------------------- + */ + +void +TclFinalizeThreadAllocThread(void) +{ +    Cache *cachePtr = TclpGetAllocCache(); +    if (cachePtr != NULL) { +	TclpFreeAllocCache(cachePtr); +    } +} +  #else /* !(TCL_THREADS && USE_THREAD_ALLOC) */  /*   *---------------------------------------------------------------------- | 
