sergiotarxz/libvips
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

tiny cache clean-up

Browse Source
This commit is contained in:
John Cupitt 2014-09-19 11:38:16 +01:00
parent ec3366257f
commit 862d7f03ea
2 changed files with 27 additions and 19 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ChangeLog
View File

@ -6,6 +6,7 @@
- vipsthumbnail retries with specified input profile if embedded profile is
broken
- add @profile option to pngsave, matching tiff and jpeg
- fix a race in the operation cache [Lovell]
8/9/14 started 7.40.8
- fix configure on rhel6 [Lovell]

45
libvips/iofuncs/cache.c
View File

@ -599,12 +599,9 @@ vips_cache_ref( VipsOperation *operation )
static void
vips_cache_insert( VipsOperation *operation )
{
/* It must not be in cache.
*/
if( g_hash_table_lookup( vips_cache_table, operation ) )
return;
VipsOperationCacheEntry *entry = g_new( VipsOperationCacheEntry, 1 );
entry = g_new( VipsOperationCacheEntry, 1 );
entry->operation = operation;
entry->time = 0;
entry->invalidate_id = 0;
@ -759,28 +756,38 @@ vips_cache_operation_buildp( VipsOperation **operation )
*operation = hit->operation;
}
/* We have to unlock between search and add so that more than one
* _build() can run at once.
*/
g_mutex_unlock( vips_cache_lock );
if( !hit ) {
if( vips_object_build( VIPS_OBJECT( *operation ) ) )
return( -1 );
/* Has to be after _build() so we can see output args.
*/
if( vips__cache_trace ) {
if( vips_operation_get_flags( *operation ) &
VIPS_OPERATION_NOCACHE )
printf( "vips cache : " );
else
printf( "vips cache+: " );
vips_object_print_summary( VIPS_OBJECT( *operation ) );
}
g_mutex_lock( vips_cache_lock );
if( !(vips_operation_get_flags( *operation ) &
VIPS_OPERATION_NOCACHE) )
vips_cache_insert( *operation );
/* If two threads call the same operation at the same time,
* we can get multiple adds. Let the first one win. See
* https://github.com/jcupitt/libvips/pull/181
*/
if( !g_hash_table_lookup( vips_cache_table, operation ) ) {
/* Has to be after _build() so we can see output args.
*/
if( vips__cache_trace ) {
if( vips_operation_get_flags( *operation ) &
VIPS_OPERATION_NOCACHE )
printf( "vips cache : " );
else
printf( "vips cache+: " );
vips_object_print_summary(
VIPS_OBJECT( *operation ) );
}
if( !(vips_operation_get_flags( *operation ) &
VIPS_OPERATION_NOCACHE) )
vips_cache_insert( *operation );
}
g_mutex_unlock( vips_cache_lock );
}