Skip to content
GitLab
Menu
Projects
Groups
Snippets
Loading...
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
Menu
Open sidebar
wangsen
rocm_bandwidth_test
Commits
b5f4387a
Unverified
Commit
b5f4387a
authored
Jul 01, 2019
by
Ramesh Errabolu
Committed by
GitHub
Jul 01, 2019
Browse files
Merge pull request #41 from RadeonOpenCompute/fixSignalLeak
Fix signal leak problem
parents
80050ad5
c4cfbb8f
Changes
1
Show whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
10 additions
and
9 deletions
+10
-9
rocm_bandwidth_test.cpp
rocm_bandwidth_test.cpp
+10
-9
No files found.
rocm_bandwidth_test.cpp
View file @
b5f4387a
...
@@ -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
{
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment