Skip to content

GitLab

Commit c4cfbb8f authored by Ramesh Errabolu's avatar Ramesh Errabolu
Browse files

Fix signal leak problem

parent 80050ad5
Hide whitespace changes
Inline Side-by-side
Showing with 10 additions and 9 deletions
rocm_bandwidth_test.cpp
View file @ c4cfbb8f
...@@ -705,6 +705,16 @@ void RocmBandwidthTest::Run() { ...@@ -705,6 +705,16 @@ void RocmBandwidthTest::Run() {
} }
void RocmBandwidthTest::Close() { void RocmBandwidthTest::Close() {
if (init_src_ != NULL) {
hsa_signal_destroy(init_signal_);
hsa_amd_memory_pool_free(init_src_);
}
if (validate_) {
hsa_amd_memory_pool_free(validate_dst_);
}
hsa_status_t status = hsa_shut_down(); hsa_status_t status = hsa_shut_down();
ErrorCheck(status); ErrorCheck(status);
return; return;
...@@ -802,15 +812,6 @@ RocmBandwidthTest::~RocmBandwidthTest() { ...@@ -802,15 +812,6 @@ RocmBandwidthTest::~RocmBandwidthTest() {
delete link_type_matrix_; delete link_type_matrix_;
delete link_weight_matrix_; delete link_weight_matrix_;
delete active_agents_list_; delete active_agents_list_;
if (init_src_ != NULL) {
hsa_signal_destroy(init_signal_);
hsa_amd_memory_pool_free(init_src_);
}
if (validate_) {
hsa_amd_memory_pool_free(validate_dst_);
}
} }
std::string RocmBandwidthTest::GetVersion() const { std::string RocmBandwidthTest::GetVersion() const {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment