diff options
| author | Raymond Lu <songyulu@hdfgroup.org> | 2009-02-11 18:49:17 (GMT) |
|---|---|---|
| committer | Raymond Lu <songyulu@hdfgroup.org> | 2009-02-11 18:49:17 (GMT) |
| commit | 725b4d7ddb461c984c8afef2f7ea5d962f5def53 (patch) | |
| tree | efa7178b478577a6560859c56dd2f89f47c9d31c /src/H5Dmpio.c | |
| parent | 77e347b19fb09bd96aba18dd6017d6f678e19c4e (diff) | |
| download | hdf5-725b4d7ddb461c984c8afef2f7ea5d962f5def53.zip hdf5-725b4d7ddb461c984c8afef2f7ea5d962f5def53.tar.gz hdf5-725b4d7ddb461c984c8afef2f7ea5d962f5def53.tar.bz2 | |
[svn-r16465] 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 */ |