Skip to content

Commit

Permalink
kokkos(profiling): do not finalize in any backend
Browse files Browse the repository at this point in the history
`Kokkos::Profiling::finalize` should only be called in `pre_finalize_internal`, as described in #6633.
  • Loading branch information
romintomasetti committed Nov 29, 2023
1 parent 0d34280 commit c9d7bba
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 6 deletions.
2 changes: 0 additions & 2 deletions core/src/OpenMP/Kokkos_OpenMP_Instance.cpp
Expand Up @@ -304,8 +304,6 @@ void OpenMPInternal::finalize() {
}

m_initialized = false;

Kokkos::Profiling::finalize();
}

void OpenMPInternal::print_configuration(std::ostream &s) const {
Expand Down
2 changes: 0 additions & 2 deletions core/src/Serial/Kokkos_Serial.cpp
Expand Up @@ -58,8 +58,6 @@ void SerialInternal::finalize() {
m_thread_team_data.scratch_assign(nullptr, 0, 0, 0, 0, 0);
}

Kokkos::Profiling::finalize();

m_is_initialized = false;
}

Expand Down
2 changes: 0 additions & 2 deletions core/src/Threads/Kokkos_Threads_Instance.cpp
Expand Up @@ -672,8 +672,6 @@ void ThreadsInternal::finalize() {
s_threads_process.m_pool_size = 1;
s_threads_process.m_pool_fan_size = 0;
s_threads_process.m_pool_state = ThreadState::Inactive;

Kokkos::Profiling::finalize();
}

//----------------------------------------------------------------------------
Expand Down

0 comments on commit c9d7bba

Please sign in to comment.