diff options
author | Allen Byrne <byrn@hdfgroup.org> | 2020-10-01 14:04:36 (GMT) |
---|---|---|
committer | Allen Byrne <byrn@hdfgroup.org> | 2020-10-01 14:04:36 (GMT) |
commit | f1ba03cea5b82699a984c80bd2deac14fdc8df18 (patch) | |
tree | ebe777c3e0b83f4c4cec9212731da9ebe0a0cfd3 /src/H5FAdblkpage.c | |
parent | 10343c197906415388f2a4c8d292e21d25cf7381 (diff) | |
download | hdf5-f1ba03cea5b82699a984c80bd2deac14fdc8df18.zip hdf5-f1ba03cea5b82699a984c80bd2deac14fdc8df18.tar.gz hdf5-f1ba03cea5b82699a984c80bd2deac14fdc8df18.tar.bz2 |
Source formatted
Diffstat (limited to 'src/H5FAdblkpage.c')
-rw-r--r-- | src/H5FAdblkpage.c | 290 |
1 files changed, 136 insertions, 154 deletions
diff --git a/src/H5FAdblkpage.c b/src/H5FAdblkpage.c index effdeb2..24012ec 100644 --- a/src/H5FAdblkpage.c +++ b/src/H5FAdblkpage.c @@ -24,53 +24,44 @@ /* Module Declaration */ /**********************/ -#include "H5FAmodule.h" /* This source code file is part of the H5FA module */ - +#include "H5FAmodule.h" /* This source code file is part of the H5FA module */ /***********************/ /* Other Packages Used */ /***********************/ - /***********/ /* Headers */ /***********/ -#include "H5private.h" /* Generic Functions */ -#include "H5Eprivate.h" /* Error handling */ -#include "H5FApkg.h" /* Fixed Arrays */ -#include "H5FLprivate.h" /* Free Lists */ - +#include "H5private.h" /* Generic Functions */ +#include "H5Eprivate.h" /* Error handling */ +#include "H5FApkg.h" /* Fixed Arrays */ +#include "H5FLprivate.h" /* Free Lists */ /****************/ /* Local Macros */ /****************/ - /******************/ /* Local Typedefs */ /******************/ - /********************/ /* Package Typedefs */ /********************/ - /********************/ /* Local Prototypes */ /********************/ - /*********************/ /* Package Variables */ /*********************/ - /*****************************/ /* Library Private Variables */ /*****************************/ - /*******************/ /* Local Variables */ /*******************/ @@ -81,8 +72,6 @@ H5FL_DEFINE_STATIC(H5FA_dblk_page_t); /* Declare a free list to manage the page elements */ H5FL_BLK_DEFINE(page_elmts); - - /*------------------------------------------------------------------------- * Function: H5FA__dblk_page_alloc * @@ -95,44 +84,41 @@ H5FL_BLK_DEFINE(page_elmts); * *------------------------------------------------------------------------- */ -BEGIN_FUNC(PKG, ERR, -H5FA_dblk_page_t *, NULL, NULL, -H5FA__dblk_page_alloc(H5FA_hdr_t *hdr, size_t nelmts)) +BEGIN_FUNC(PKG, ERR, H5FA_dblk_page_t *, NULL, NULL, H5FA__dblk_page_alloc(H5FA_hdr_t *hdr, size_t nelmts)) - /* Local variables */ - H5FA_dblk_page_t *dblk_page = NULL; /* Fixed array data block page */ +/* Local variables */ +H5FA_dblk_page_t *dblk_page = NULL; /* Fixed array data block page */ - /* Check arguments */ - HDassert(hdr); +/* Check arguments */ +HDassert(hdr); - /* Allocate memory for the data block */ - if(NULL == (dblk_page = H5FL_CALLOC(H5FA_dblk_page_t))) - H5E_THROW(H5E_CANTALLOC, "memory allocation failed for fixed array data block page") +/* Allocate memory for the data block */ +if (NULL == (dblk_page = H5FL_CALLOC(H5FA_dblk_page_t))) + H5E_THROW(H5E_CANTALLOC, "memory allocation failed for fixed array data block page") - /* Share common array information */ - if(H5FA__hdr_incr(hdr) < 0) - H5E_THROW(H5E_CANTINC, "can't increment reference count on shared array header") - dblk_page->hdr = hdr; +/* Share common array information */ +if (H5FA__hdr_incr(hdr) < 0) + H5E_THROW(H5E_CANTINC, "can't increment reference count on shared array header") +dblk_page->hdr = hdr; - /* Set non-zero internal fields */ - dblk_page->nelmts = nelmts; +/* Set non-zero internal fields */ +dblk_page->nelmts = nelmts; - /* Allocate buffer for elements in data block page */ - if(NULL == (dblk_page->elmts = H5FL_BLK_MALLOC(page_elmts, nelmts * hdr->cparam.cls->nat_elmt_size))) - H5E_THROW(H5E_CANTALLOC, "memory allocation failed for data block page element buffer") +/* Allocate buffer for elements in data block page */ +if (NULL == (dblk_page->elmts = H5FL_BLK_MALLOC(page_elmts, nelmts * hdr->cparam.cls->nat_elmt_size))) + H5E_THROW(H5E_CANTALLOC, "memory allocation failed for data block page element buffer") - /* Set the return value */ - ret_value = dblk_page; +/* Set the return value */ +ret_value = dblk_page; CATCH - if(!ret_value) - if(dblk_page && H5FA__dblk_page_dest(dblk_page) < 0) - H5E_THROW(H5E_CANTFREE, "unable to destroy fixed array data block page") +if (!ret_value) + if (dblk_page && H5FA__dblk_page_dest(dblk_page) < 0) + H5E_THROW(H5E_CANTFREE, "unable to destroy fixed array data block page") -END_FUNC(PKG) /* end H5FA__dblk_page_alloc() */ +END_FUNC(PKG) /* end H5FA__dblk_page_alloc() */ - /*------------------------------------------------------------------------- * Function: H5FA__dblk_page_create * @@ -145,64 +131,62 @@ END_FUNC(PKG) /* end H5FA__dblk_page_alloc() */ * *------------------------------------------------------------------------- */ -BEGIN_FUNC(PKG, ERR, -herr_t, SUCCEED, FAIL, -H5FA__dblk_page_create(H5FA_hdr_t *hdr, haddr_t addr, size_t nelmts)) +BEGIN_FUNC(PKG, ERR, herr_t, SUCCEED, FAIL, + H5FA__dblk_page_create(H5FA_hdr_t *hdr, haddr_t addr, size_t nelmts)) - /* Local variables */ - H5FA_dblk_page_t *dblk_page = NULL; /* Fixed array data block page */ - hbool_t inserted = FALSE; /* Whether the header was inserted into cache */ +/* Local variables */ +H5FA_dblk_page_t *dblk_page = NULL; /* Fixed array data block page */ +hbool_t inserted = FALSE; /* Whether the header was inserted into cache */ #ifdef H5FA_DEBUG HDfprintf(stderr, "%s: Called, addr = %a\n", FUNC, addr); #endif /* H5FA_DEBUG */ - /* Sanity check */ - HDassert(hdr); +/* Sanity check */ +HDassert(hdr); - /* Allocate the data block page */ - if(NULL == (dblk_page = H5FA__dblk_page_alloc(hdr, nelmts))) - H5E_THROW(H5E_CANTALLOC, "memory allocation failed for fixed array data block page") +/* Allocate the data block page */ +if (NULL == (dblk_page = H5FA__dblk_page_alloc(hdr, nelmts))) + H5E_THROW(H5E_CANTALLOC, "memory allocation failed for fixed array data block page") - /* Set info about data block page on disk */ - dblk_page->addr = addr; - dblk_page->size = H5FA_DBLK_PAGE_SIZE(hdr, nelmts); +/* Set info about data block page on disk */ +dblk_page->addr = addr; +dblk_page->size = H5FA_DBLK_PAGE_SIZE(hdr, nelmts); #ifdef H5FA_DEBUG HDfprintf(stderr, "%s: dblk_page->size = %Zu\n", FUNC, dblk_page->size); #endif /* H5FA_DEBUG */ - /* Clear any elements in data block page to fill value */ - if((hdr->cparam.cls->fill)(dblk_page->elmts, nelmts) < 0) - H5E_THROW(H5E_CANTSET, "can't set fixed array data block page elements to class's fill value") +/* Clear any elements in data block page to fill value */ +if ((hdr->cparam.cls->fill)(dblk_page->elmts, nelmts) < 0) + H5E_THROW(H5E_CANTSET, "can't set fixed array data block page elements to class's fill value") - /* Cache the new fixed array data block page */ - if(H5AC_insert_entry(hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page->addr, dblk_page, H5AC__NO_FLAGS_SET) < 0) - H5E_THROW(H5E_CANTINSERT, "can't add fixed array data block page to cache") - inserted = TRUE; +/* Cache the new fixed array data block page */ +if (H5AC_insert_entry(hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page->addr, dblk_page, H5AC__NO_FLAGS_SET) < 0) + H5E_THROW(H5E_CANTINSERT, "can't add fixed array data block page to cache") +inserted = TRUE; - /* Add data block page as child of 'top' proxy */ - if(hdr->top_proxy) { - if(H5AC_proxy_entry_add_child(hdr->top_proxy, hdr->f, dblk_page) < 0) - H5E_THROW(H5E_CANTSET, "unable to add fixed array entry as child of array proxy") - dblk_page->top_proxy = hdr->top_proxy; - } /* end if */ +/* Add data block page as child of 'top' proxy */ +if (hdr->top_proxy) { + if (H5AC_proxy_entry_add_child(hdr->top_proxy, hdr->f, dblk_page) < 0) + H5E_THROW(H5E_CANTSET, "unable to add fixed array entry as child of array proxy") + dblk_page->top_proxy = hdr->top_proxy; +} /* end if */ CATCH - if(ret_value < 0) - if(dblk_page) { - /* Remove from cache, if inserted */ - if(inserted) - if(H5AC_remove_entry(dblk_page) < 0) - H5E_THROW(H5E_CANTREMOVE, "unable to remove fixed array data block page from cache") - - /* Destroy data block page */ - if(H5FA__dblk_page_dest(dblk_page) < 0) - H5E_THROW(H5E_CANTFREE, "unable to destroy fixed array data block page") - } /* end if */ +if (ret_value < 0) + if (dblk_page) { + /* Remove from cache, if inserted */ + if (inserted) + if (H5AC_remove_entry(dblk_page) < 0) + H5E_THROW(H5E_CANTREMOVE, "unable to remove fixed array data block page from cache") + + /* Destroy data block page */ + if (H5FA__dblk_page_dest(dblk_page) < 0) + H5E_THROW(H5E_CANTFREE, "unable to destroy fixed array data block page") + } /* end if */ -END_FUNC(PKG) /* end H5FA__dblk_page_create() */ +END_FUNC(PKG) /* end H5FA__dblk_page_create() */ - /*------------------------------------------------------------------------- * Function: H5FA__dblk_page_protect * @@ -216,58 +200,60 @@ END_FUNC(PKG) /* end H5FA__dblk_page_create() */ * *------------------------------------------------------------------------- */ -BEGIN_FUNC(PKG, ERR, -H5FA_dblk_page_t *, NULL, NULL, -H5FA__dblk_page_protect(H5FA_hdr_t *hdr, haddr_t dblk_page_addr, - size_t dblk_page_nelmts, unsigned flags)) +BEGIN_FUNC(PKG, ERR, H5FA_dblk_page_t *, NULL, NULL, + H5FA__dblk_page_protect(H5FA_hdr_t *hdr, haddr_t dblk_page_addr, size_t dblk_page_nelmts, + unsigned flags)) - /* Local variables */ - H5FA_dblk_page_t *dblk_page = NULL; /* Fixed array data block page */ - H5FA_dblk_page_cache_ud_t udata; /* Information needed for loading data block page */ +/* Local variables */ +H5FA_dblk_page_t * dblk_page = NULL; /* Fixed array data block page */ +H5FA_dblk_page_cache_ud_t udata; /* Information needed for loading data block page */ #ifdef H5FA_DEBUG HDfprintf(stderr, "%s: Called\n", FUNC); #endif /* H5FA_DEBUG */ - /* Sanity check */ - HDassert(hdr); - HDassert(H5F_addr_defined(dblk_page_addr)); +/* Sanity check */ +HDassert(hdr); +HDassert(H5F_addr_defined(dblk_page_addr)); - /* only the H5AC__READ_ONLY_FLAG is permitted */ - HDassert((flags & (unsigned)(~H5AC__READ_ONLY_FLAG)) == 0); +/* only the H5AC__READ_ONLY_FLAG is permitted */ +HDassert((flags & (unsigned)(~H5AC__READ_ONLY_FLAG)) == 0); - /* Set up user data */ - udata.hdr = hdr; - udata.nelmts = dblk_page_nelmts; - udata.dblk_page_addr = dblk_page_addr; +/* Set up user data */ +udata.hdr = hdr; +udata.nelmts = dblk_page_nelmts; +udata.dblk_page_addr = dblk_page_addr; - /* Protect the data block page */ - if(NULL == (dblk_page = (H5FA_dblk_page_t *)H5AC_protect(hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page_addr, &udata, flags))) - H5E_THROW(H5E_CANTPROTECT, "unable to protect fixed array data block page, address = %llu", (unsigned long long)dblk_page_addr) +/* Protect the data block page */ +if (NULL == (dblk_page = (H5FA_dblk_page_t *)H5AC_protect(hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page_addr, + &udata, flags))) + H5E_THROW(H5E_CANTPROTECT, "unable to protect fixed array data block page, address = %llu", + (unsigned long long)dblk_page_addr) - /* Create top proxy, if it doesn't exist */ - if(hdr->top_proxy && NULL == dblk_page->top_proxy) { - /* Add data block page as child of 'top' proxy */ - if(H5AC_proxy_entry_add_child(hdr->top_proxy, hdr->f, dblk_page) < 0) - H5E_THROW(H5E_CANTSET, "unable to add fixed array entry as child of array proxy") - dblk_page->top_proxy = hdr->top_proxy; - } /* end if */ +/* Create top proxy, if it doesn't exist */ +if (hdr->top_proxy && NULL == dblk_page->top_proxy) { + /* Add data block page as child of 'top' proxy */ + if (H5AC_proxy_entry_add_child(hdr->top_proxy, hdr->f, dblk_page) < 0) + H5E_THROW(H5E_CANTSET, "unable to add fixed array entry as child of array proxy") + dblk_page->top_proxy = hdr->top_proxy; +} /* end if */ - /* Set return value */ - ret_value = dblk_page; +/* Set return value */ +ret_value = dblk_page; CATCH - /* Clean up on error */ - if(!ret_value) { - /* Release the data block page, if it was protected */ - if(dblk_page && H5AC_unprotect(hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page->addr, dblk_page, H5AC__NO_FLAGS_SET) < 0) - H5E_THROW(H5E_CANTUNPROTECT, "unable to unprotect fixed array data block page, address = %llu", (unsigned long long)dblk_page->addr) - } /* end if */ +/* Clean up on error */ +if (!ret_value) { + /* Release the data block page, if it was protected */ + if (dblk_page && + H5AC_unprotect(hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page->addr, dblk_page, H5AC__NO_FLAGS_SET) < 0) + H5E_THROW(H5E_CANTUNPROTECT, "unable to unprotect fixed array data block page, address = %llu", + (unsigned long long)dblk_page->addr) +} /* end if */ -END_FUNC(PKG) /* end H5FA__dblk_page_protect() */ +END_FUNC(PKG) /* end H5FA__dblk_page_protect() */ - /*------------------------------------------------------------------------- * Function: H5FA__dblk_page_unprotect * @@ -281,28 +267,27 @@ END_FUNC(PKG) /* end H5FA__dblk_page_protect() */ * *------------------------------------------------------------------------- */ -BEGIN_FUNC(PKG, ERR, -herr_t, SUCCEED, FAIL, -H5FA__dblk_page_unprotect(H5FA_dblk_page_t *dblk_page, unsigned cache_flags)) +BEGIN_FUNC(PKG, ERR, herr_t, SUCCEED, FAIL, + H5FA__dblk_page_unprotect(H5FA_dblk_page_t *dblk_page, unsigned cache_flags)) - /* Local variables */ +/* Local variables */ #ifdef H5FA_DEBUG HDfprintf(stderr, "%s: Called\n", FUNC); #endif /* H5FA_DEBUG */ - /* Sanity check */ - HDassert(dblk_page); +/* Sanity check */ +HDassert(dblk_page); - /* Unprotect the data block page */ - if(H5AC_unprotect(dblk_page->hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page->addr, dblk_page, cache_flags) < 0) - H5E_THROW(H5E_CANTUNPROTECT, "unable to unprotect fixed array data block page, address = %llu", (unsigned long long)dblk_page->addr) +/* Unprotect the data block page */ +if (H5AC_unprotect(dblk_page->hdr->f, H5AC_FARRAY_DBLK_PAGE, dblk_page->addr, dblk_page, cache_flags) < 0) + H5E_THROW(H5E_CANTUNPROTECT, "unable to unprotect fixed array data block page, address = %llu", + (unsigned long long)dblk_page->addr) CATCH -END_FUNC(PKG) /* end H5FA__dblk_page_unprotect() */ +END_FUNC(PKG) /* end H5FA__dblk_page_unprotect() */ - /*------------------------------------------------------------------------- * Function: H5FA__dblk_page_dest * @@ -315,34 +300,31 @@ END_FUNC(PKG) /* end H5FA__dblk_page_unprotect() */ * *------------------------------------------------------------------------- */ -BEGIN_FUNC(PKG, ERR, -herr_t, SUCCEED, FAIL, -H5FA__dblk_page_dest(H5FA_dblk_page_t *dblk_page)) - - /* Sanity check */ - HDassert(dblk_page); - - /* Check if header field has been initialized */ - if(dblk_page->hdr) { - /* Check if buffer for data block page elements has been initialized */ - if(dblk_page->elmts) { - /* Free buffer for data block page elements */ - dblk_page->elmts = H5FL_BLK_FREE(page_elmts, dblk_page->elmts); - } /* end if */ - - /* Decrement reference count on shared info */ - if(H5FA__hdr_decr(dblk_page->hdr) < 0) - H5E_THROW(H5E_CANTDEC, "can't decrement reference count on shared array header") - dblk_page->hdr = NULL; +BEGIN_FUNC(PKG, ERR, herr_t, SUCCEED, FAIL, H5FA__dblk_page_dest(H5FA_dblk_page_t *dblk_page)) + +/* Sanity check */ +HDassert(dblk_page); + +/* Check if header field has been initialized */ +if (dblk_page->hdr) { + /* Check if buffer for data block page elements has been initialized */ + if (dblk_page->elmts) { + /* Free buffer for data block page elements */ + dblk_page->elmts = H5FL_BLK_FREE(page_elmts, dblk_page->elmts); } /* end if */ - /* Sanity check */ - HDassert(NULL == dblk_page->top_proxy); + /* Decrement reference count on shared info */ + if (H5FA__hdr_decr(dblk_page->hdr) < 0) + H5E_THROW(H5E_CANTDEC, "can't decrement reference count on shared array header") + dblk_page->hdr = NULL; +} /* end if */ - /* Free the data block page itself */ - dblk_page = H5FL_FREE(H5FA_dblk_page_t, dblk_page); +/* Sanity check */ +HDassert(NULL == dblk_page->top_proxy); -CATCH +/* Free the data block page itself */ +dblk_page = H5FL_FREE(H5FA_dblk_page_t, dblk_page); -END_FUNC(PKG) /* end H5FA__dblk_page_dest() */ +CATCH +END_FUNC(PKG) /* end H5FA__dblk_page_dest() */ |