diff options
author | Raymond Lu <songyulu@hdfgroup.org> | 2009-02-11 19:22:59 (GMT) |
---|---|---|
committer | Raymond Lu <songyulu@hdfgroup.org> | 2009-02-11 19:22:59 (GMT) |
commit | 87cdac5fe981ce61404c3ac68eded4fad81608ea (patch) | |
tree | f2a516476c727df5db2c6983955844ad25f8a7ef /src/H5Dmpio.c | |
parent | 8fd5f783b4305485cb1293a90eaaf526e7882e78 (diff) | |
download | hdf5-87cdac5fe981ce61404c3ac68eded4fad81608ea.zip hdf5-87cdac5fe981ce61404c3ac68eded4fad81608ea.tar.gz hdf5-87cdac5fe981ce61404c3ac68eded4fad81608ea.tar.bz2 |
[svn-r16467] Performance Improvement(bug #1450). When a chunk is bigger than the cache size and isn't allocated
on disk, the library still loaded it in the cache, which is redundant. I changed it to bypass the
cache and added a test in dsets.c.
Tested on jam and smirom.
Diffstat (limited to 'src/H5Dmpio.c')
-rw-r--r-- | src/H5Dmpio.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/H5Dmpio.c b/src/H5Dmpio.c index 99ad001..19be413 100644 --- a/src/H5Dmpio.c +++ b/src/H5Dmpio.c @@ -1224,7 +1224,7 @@ if(H5DEBUG(D)) HGOTO_ERROR(H5E_STORAGE, H5E_CANTGET, FAIL, "couldn't get chunk info from skipped list") /* Load the chunk into cache and lock it. */ - if(H5D_chunk_cacheable(io_info, udata.addr)) { + if(H5D_chunk_cacheable(io_info)) { hbool_t entire_chunk = TRUE; /* Whether whole chunk is selected */ /* Compute # of bytes accessed in chunk */ @@ -1449,7 +1449,7 @@ if(H5DEBUG(D)) { HGOTO_ERROR(H5E_DATASET, H5E_CANTGET, FAIL, "can't switch to independent I/O") /* Load the chunk into cache and lock it. */ - if(H5D_chunk_cacheable(io_info, udata.addr)) { + if(H5D_chunk_cacheable(io_info)) { hbool_t entire_chunk = TRUE; /* Whether whole chunk is selected */ /* Compute # of bytes accessed in chunk */ |