Skip to content

Commit

Permalink
Make The_Device_Arena non-managed (AMReX-Codes#2998)
Browse files Browse the repository at this point in the history
The_Device_Arena used to be a separate Arena. We changed it to be an
alias of The_Arena to avoid memory fragmentation. However, the issue is
we don't have an Arena that can allocate non-managed memory unless
The_Arena is not managed. Because of performance concerns, we sometimes
want to allocate non-managed memory. Therefore, we make The_Device_Arena
an alias if and only if The_Arena is not managed.
  • Loading branch information
WeiqunZhang committed Oct 26, 2022
1 parent ab8c892 commit 7f3c908
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion Src/Base/AMReX_Arena.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,7 @@ Arena::Initialize ()
the_async_arena = new PArena(the_async_arena_release_threshold);

#ifdef AMREX_USE_GPU
if (the_arena->isDevice() || the_arena->isManaged()) {
if (the_arena->isDevice()) {
the_device_arena = the_arena;
} else {
the_device_arena = new CArena(0, ArenaInfo{}.SetDeviceMemory().SetReleaseThreshold
Expand Down
5 changes: 2 additions & 3 deletions Src/Base/AMReX_GpuContainers.H
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,8 @@ namespace Gpu {

/**
* \brief A PODVector that uses the standard memory Arena.
* Note that, on NVIDIA architectures, this Arena is actually
* managed.
*
* Note that the memory might or might not be managed depending
* on the amrex.the_arena_is_managed ParmParse parameter.
*/
template <class T>
using DeviceVector = PODVector<T, ArenaAllocator<T> >;
Expand Down

0 comments on commit 7f3c908

Please sign in to comment.