Fixed stale API calls to membrk API in gemmlike.

Details:
- Updated stale calls to the bli_membrk API within the 'gemmlike'
  sandbox. This API is now called bli_pba (packed block allocator).
  Ideally, this forgotten update would have been included as part of
  21911d6, which is when the branch where the membrk->pba changes was
  introduced was merged into 'master'.
- Comment updates.
This commit is contained in:
Field G. Van Zee
2021-08-12 14:06:53 -05:00
parent a32257eeab
commit e366665cd2
5 changed files with 17 additions and 17 deletions

View File

@@ -67,7 +67,7 @@ void PASTECH2(bls_,ch,opname) \
siz_t size_needed = sizeof( ctype ) * m_pack * k_pack; \
\
/* Check the mem_t entry provided by the caller. If it is unallocated,
then we need to acquire a block from the memory broker. */ \
then we need to acquire a block from the packed block allocator. */ \
if ( bli_mem_is_unalloc( mem ) ) \
{ \
if ( bli_thread_am_ochief( thread ) ) \
@@ -79,7 +79,7 @@ void PASTECH2(bls_,ch,opname) \
the current function before the other threads have a chance to
copy from it. (A barrier would fix that race condition, but then
again, I prefer to keep barriers to a minimum.) */ \
bli_membrk_acquire_m \
bli_pba_acquire_m \
( \
rntm, \
size_needed, \
@@ -104,8 +104,8 @@ void PASTECH2(bls_,ch,opname) \
else /* if ( bli_mem_is_alloc( mem ) ) */ \
{ \
/* If the mem_t entry provided by the caller does NOT contain a NULL
buffer, then a block has already been acquired from the memory
broker and cached by the caller. */ \
buffer, then a block has already been acquired from the packed
block allocator and cached by the caller. */ \
\
/* As a sanity check, we should make sure that the mem_t object isn't
associated with a block that is too small compared to the size of
@@ -123,12 +123,12 @@ void PASTECH2(bls_,ch,opname) \
above for why the acquisition needs to be directly to
the chief thread's passed-in mem_t and not a local
(temporary) mem_t. */ \
bli_membrk_release \
bli_pba_release \
( \
rntm, \
mem \
); \
bli_membrk_acquire_m \
bli_pba_acquire_m \
( \
rntm, \
size_needed, \
@@ -182,7 +182,7 @@ void PASTECH2(bls_,ch,opname) \
is allocated, which it should be. */ \
if ( bli_mem_is_alloc( mem ) ) \
{ \
bli_membrk_release \
bli_pba_release \
( \
rntm, \
mem \

View File

@@ -67,7 +67,7 @@ void PASTECH2(bls_,ch,opname) \
siz_t size_needed = sizeof( ctype ) * k_pack * n_pack; \
\
/* Check the mem_t entry provided by the caller. If it is unallocated,
then we need to acquire a block from the memory broker. */ \
then we need to acquire a block from the packed block allocator. */ \
if ( bli_mem_is_unalloc( mem ) ) \
{ \
if ( bli_thread_am_ochief( thread ) ) \
@@ -79,7 +79,7 @@ void PASTECH2(bls_,ch,opname) \
the current function before the other threads have a chance to
copy from it. (A barrier would fix that race condition, but then
again, I prefer to keep barriers to a minimum.) */ \
bli_membrk_acquire_m \
bli_pba_acquire_m \
( \
rntm, \
size_needed, \
@@ -104,8 +104,8 @@ void PASTECH2(bls_,ch,opname) \
else /* if ( bli_mem_is_alloc( mem ) ) */ \
{ \
/* If the mem_t entry provided by the caller does NOT contain a NULL
buffer, then a block has already been acquired from the memory
broker and cached by the caller. */ \
buffer, then a block has already been acquired from the packed
block allocator and cached by the caller. */ \
\
/* As a sanity check, we should make sure that the mem_t object isn't
associated with a block that is too small compared to the size of
@@ -123,12 +123,12 @@ void PASTECH2(bls_,ch,opname) \
above for why the acquisition needs to be directly to
the chief thread's passed-in mem_t and not a local
(temporary) mem_t. */ \
bli_membrk_release \
bli_pba_release \
( \
rntm, \
mem \
); \
bli_membrk_acquire_m \
bli_pba_acquire_m \
( \
rntm, \
size_needed, \
@@ -182,7 +182,7 @@ void PASTECH2(bls_,ch,opname) \
is allocated, which it should be. */ \
if ( bli_mem_is_alloc( mem ) ) \
{ \
bli_membrk_release \
bli_pba_release \
( \
rntm, \
mem \

View File

@@ -75,7 +75,7 @@ void bls_l3_thread_decorator
// Set the packing block allocator field of the rntm. This will be
// inherited by all of the child threads when they make local copies of
// the rntm below.
bli_membrk_rntm_set_membrk( rntm );
bli_pba_rntm_set_pba( rntm );
// Allcoate a global communicator for the root thrinfo_t structures.
thrcomm_t* restrict gl_comm = bli_thrcomm_create( rntm, n_threads );

View File

@@ -140,7 +140,7 @@ void bls_l3_thread_decorator
// Set the packing block allocator field of the rntm. This will be
// inherited by all of the child threads when they make local copies of
// the rntm below.
bli_membrk_rntm_set_membrk( rntm );
bli_pba_rntm_set_pba( rntm );
// Allocate a global communicator for the root thrinfo_t structures.
thrcomm_t* restrict gl_comm = bli_thrcomm_create( rntm, n_threads );

View File

@@ -68,7 +68,7 @@ void bls_l3_thread_decorator
bli_sba_rntm_set_pool( 0, array, rntm );
// Set the packing block allocator field of the rntm.
bli_membrk_rntm_set_membrk( rntm );
bli_pba_rntm_set_pba( rntm );
#ifndef SKIP_THRINFO_TREE
// Allcoate a global communicator for the root thrinfo_t structures.