search

emusolutions / LAGraph

This is a library plus a test harness for collecting algorithms that use the GraphBLAS
Other
0 stars 1 forks source link

`ctest_Init_errors` LC Crash #8

Closed jamesETsmith closed 1 year ago

jamesETsmith commented 1 year ago

General

This is a spinoff of #4 to tackle the problems with ctest_Init_errors.

Details

 ctest --test-dir build_lc -R Init_errors -V
Internal ctest changing into directory: /net/hyper120h-d/data/jsmith/apps/LAGraph/build_lc
UpdateCTestConfiguration  from :/net/hyper120h-d/data/jsmith/apps/LAGraph/build_lc/DartConfiguration.tcl
UpdateCTestConfiguration  from :/net/hyper120h-d/data/jsmith/apps/LAGraph/build_lc/DartConfiguration.tcl
Test project /net/hyper120h-d/data/jsmith/apps/LAGraph/build_lc
Constructing a list of tests
Done constructing a list of tests
Updating test list for fixtures
Added 0 tests to meet fixture requirements
Checking test dependency graph...
Checking test dependency graph end
test 12
    Start 12: ctest_Init_errors

12: Test command: /usr/bin/cmake "-E" "env" "/tools/lucata/bin/emusim.x" "--forward_return_value" "--" "test_Init_errors" "--no-exec"
12: Test timeout computed to be: 10000000
12:
12:         SystemC 2.3.3-Accellera --- Apr 21 2023 11:46:50
12:         Copyright (c) 1996-2018 by all Contributors,
12:         ALL RIGHTS RESERVED
12: Test Init_errors...                             RUNTIME FREE ERROR!
12: **Check Address (A) Register**
12: THROWING ILLEGAL EXCEPTION!
1/1 Test #12: ctest_Init_errors ................***Failed    0.32 sec

0% tests passed, 1 tests failed out of 1

Total Test time (real) =   0.32 sec

The following tests FAILED:
         12 - ctest_Init_errors (Failed)
Errors while running CTest
Output from these tests are in: /net/hyper120h-d/data/jsmith/apps/LAGraph/build_lc/Testing/Temporary/LastTest.log
Use "--rerun-failed --output-on-failure" to re-run the failed cases verbosely.
jamesETsmith commented 1 year ago

This problem is caused by calling GrB_Finalize() twice and we try to free our already-freed global objects. We just need to add checks if we've already run finalize. I'll push changes that fix this soon.