diff options
author | John Mainzer <mainzer@hdfgroup.org> | 2007-08-10 09:50:48 (GMT) |
---|---|---|
committer | John Mainzer <mainzer@hdfgroup.org> | 2007-08-10 09:50:48 (GMT) |
commit | c5cdb7d9577f4b2b8238cad5f148e7c2dba05141 (patch) | |
tree | 2b93afe9d18e254fa9c28c12578633dbc0f41b1b | |
parent | b742150e4c2bc115c531bc285a86dc38153594ca (diff) | |
download | hdf5-c5cdb7d9577f4b2b8238cad5f148e7c2dba05141.zip hdf5-c5cdb7d9577f4b2b8238cad5f148e7c2dba05141.tar.gz hdf5-c5cdb7d9577f4b2b8238cad5f148e7c2dba05141.tar.bz2 |
[svn-r14065] Second try at checking in the "cork the cache" modification.
First attempt contained slightly dated versions of the files
listed below.
Must also note that the mods were tested with h5committest,
and two other linux builds.
-rw-r--r-- | src/H5AC.c | 33 | ||||
-rw-r--r-- | src/H5ACprivate.h | 1 | ||||
-rw-r--r-- | src/H5ACpublic.h | 10 | ||||
-rw-r--r-- | src/H5Cprivate.h | 4 |
4 files changed, 14 insertions, 34 deletions
@@ -2040,7 +2040,7 @@ H5AC_resize_pinned_entry(H5F_t * f, ( H5C_get_trace_file_ptr(f->shared->cache, &trace_file_ptr) >= 0 ) && ( trace_file_ptr != NULL ) ) { - sprintf(trace, "H5AC_resize_pinned_entry 0x%lx %d %d", + sprintf(trace, "H5AC_resize_pinned_entry 0x%lx %d", (unsigned long)(((H5C_cache_entry_t *)thing)->addr), (int)new_size); } @@ -2611,7 +2611,6 @@ H5AC_get_cache_auto_resize_config(H5AC_t * cache_ptr, config_ptr->open_trace_file = FALSE; config_ptr->close_trace_file = FALSE; config_ptr->trace_file_name[0] = '\0'; - config_ptr->set_evictions_enabled = FALSE; config_ptr->evictions_enabled = evictions_enabled; config_ptr->set_initial_size = internal_config.set_initial_size; config_ptr->initial_size = internal_config.initial_size; @@ -2827,7 +2826,6 @@ H5AC_set_cache_auto_resize_config(H5AC_t * cache_ptr, /* const char * fcn_name = "H5AC_set_cache_auto_resize_config"; */ herr_t result; herr_t ret_value = SUCCEED; /* Return value */ - int name_len; H5C_auto_size_ctl_t internal_config; #if H5AC__TRACE_FILE_ENABLED H5AC_cache_config_t trace_config = H5AC__DEFAULT_CACHE_CONFIG; @@ -2941,16 +2939,14 @@ H5AC_set_cache_auto_resize_config(H5AC_t * cache_ptr, "H5C_set_cache_auto_resize_config() failed.") } - if ( config_ptr->set_evictions_enabled ) { - result = H5C_set_evictions_enabled((H5C_t *)cache_ptr, - config_ptr->evictions_enabled); + result = H5C_set_evictions_enabled((H5C_t *)cache_ptr, + config_ptr->evictions_enabled); - if ( result < 0 ) { + if ( result < 0 ) { - HGOTO_ERROR(H5E_CACHE, H5E_SYSTEM, FAIL, \ - "H5C_set_evictions_enabled() failed.") - } + HGOTO_ERROR(H5E_CACHE, H5E_SYSTEM, FAIL, \ + "H5C_set_evictions_enabled() failed.") } #ifdef H5_HAVE_PARALLEL @@ -2973,14 +2969,13 @@ done: ( trace_file_ptr != NULL ) ) { HDfprintf(trace_file_ptr, - "%s %d %d %d %d \"%s\" %d %d %d %d %f %d %d %ld %d %f %f %d %d %d %f %f %d %d %d %d %f %d %d\n", + "%s %d %d %d %d \"%s\" %d %d %d %f %d %d %ld %d %f %f %d %d %d %f %f %d %d %d %d %f %d %d\n", "H5AC_set_cache_auto_resize_config", trace_config.version, (int)(trace_config.rpt_fcn_enabled), (int)(trace_config.open_trace_file), (int)(trace_config.close_trace_file), trace_config.trace_file_name, - (int)(trace_config.set_evictions_enabled), (int)(trace_config.evictions_enabled), (int)(trace_config.set_initial_size), (int)(trace_config.initial_size), @@ -3041,8 +3036,8 @@ done: * * - Added code testing the evictions enabled field. At * present this consists of verifying that if - * set_evictions_enabled is TRUE and evictions_enabled - * is FALSE, then automatic cache resizing in disabled. + * evictions_enabled is FALSE, then automatic cache + * resizing in disabled. * * JRM - 7/28/07 * @@ -3112,13 +3107,6 @@ H5AC_validate_config(H5AC_cache_config_t * config_ptr) } } - if ( ( config_ptr->set_evictions_enabled != TRUE ) && - ( config_ptr->set_evictions_enabled != FALSE ) ) { - - HGOTO_ERROR(H5E_ARGS, H5E_BADVALUE, FAIL, \ - "config_ptr->set_evictions_enabled must be either TRUE or FALSE.") - } - if ( ( config_ptr->evictions_enabled != TRUE ) && ( config_ptr->evictions_enabled != FALSE ) ) { @@ -3126,8 +3114,7 @@ H5AC_validate_config(H5AC_cache_config_t * config_ptr) "config_ptr->evictions_enabled must be either TRUE or FALSE.") } - if ( ( config_ptr->set_evictions_enabled == TRUE ) && - ( config_ptr->evictions_enabled == FALSE ) && + if ( ( config_ptr->evictions_enabled == FALSE ) && ( ( config_ptr->incr_mode != H5C_incr__off ) || ( config_ptr->incr_mode != H5C_decr__off ) ) ) { diff --git a/src/H5ACprivate.h b/src/H5ACprivate.h index 0d10c88..628a269 100644 --- a/src/H5ACprivate.h +++ b/src/H5ACprivate.h @@ -201,7 +201,6 @@ extern hid_t H5AC_ind_dxpl_id; /* hbool_t open_trace_file = */ FALSE, \ /* hbool_t close_trace_file = */ FALSE, \ /* char trace_file_name[] = */ "", \ - /* hbool_t set_evictions_enabled = */ FALSE, \ /* hbool_t evictions_enabled = */ TRUE, \ /* hbool_t set_initial_size = */ TRUE, \ /* size_t initial_size = */ ( 1 * 1024 * 1024), \ diff --git a/src/H5ACpublic.h b/src/H5ACpublic.h index 76e5530..0ddef92 100644 --- a/src/H5ACpublic.h +++ b/src/H5ACpublic.h @@ -107,14 +107,9 @@ extern "C" { * The length of the path must not exceed H5AC__MAX_TRACE_FILE_NAME_LEN * characters. * - * set_evictions_enabled: Boolean flag indicating whether the cache - * evictions_enabled flag is to be set to the value specified - * in the evictions_enabled field below. - * * evictions_enabled: Boolean field used to either report the current - * evictions enabled status of the cache, or, when - * set_evictions_enabled is TRUE, to set the cache's evictions - * enabled status. + * evictions enabled status of the cache, or to set the cache's + * evictions enabled status. * * In general, the metadata cache should always be allowed to * evict entries. However, in some cases it is advantageous to @@ -346,7 +341,6 @@ typedef struct H5AC_cache_config_t hbool_t close_trace_file; char trace_file_name[H5AC__MAX_TRACE_FILE_NAME_LEN + 1]; - hbool_t set_evictions_enabled; hbool_t evictions_enabled; hbool_t set_initial_size; diff --git a/src/H5Cprivate.h b/src/H5Cprivate.h index e169bd6..d365755 100644 --- a/src/H5Cprivate.h +++ b/src/H5Cprivate.h @@ -37,7 +37,7 @@ #include "H5Fprivate.h" /* File access */ -#define H5C_DO_SANITY_CHECKS 1 +#define H5C_DO_SANITY_CHECKS 0 #define H5C_DO_EXTREME_SANITY_CHECKS 0 /* This sanity checking constant was picked out of the air. Increase @@ -51,7 +51,7 @@ /* H5C_COLLECT_CACHE_STATS controls overall collection of statistics * on cache activity. In general, this #define should be set to 0. */ -#define H5C_COLLECT_CACHE_STATS 1 +#define H5C_COLLECT_CACHE_STATS 0 /* H5C_COLLECT_CACHE_ENTRY_STATS controls collection of statistics * in individual cache entries. |