Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG]: Memory leak in CUB's device_select_if test #1895

Closed
1 task done
Tracked by #1618
alliepiper opened this issue Jun 21, 2024 · 0 comments · Fixed by #1916
Closed
1 task done
Tracked by #1618

[BUG]: Memory leak in CUB's device_select_if test #1895

alliepiper opened this issue Jun 21, 2024 · 0 comments · Fixed by #1916
Assignees
@alliepiper
Labels
bug Something isn't working right.

Comments

@alliepiper
Copy link
Collaborator

Is this a duplicate?

Type of Bug

Something else

Component

CUB

Describe the bug

https://github.com/NVIDIA/cccl/actions/runs/9606424817/job/26497170531?pr=1879#step:5:2467

  	compute-sanitizer --tool memcheck --check-device-heap yes --leak-check full --padding 512 --track-stream-ordered-races all --check-warpgroup-mma yes --require-cuda-init no --check-exit-code yes --error-exitcode 1 --nvtx true /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  ========= COMPUTE-SANITIZER
  ===============================================================================
  All tests passed (4 assertions in 2 test cases)
  
  ========= Leaked 767 bytes at 0x7f4a2fc01000
  =========     Saved host backtrace up to driver entry point at allocation time
  =========     Host Frame: [0x28e3ce]
  =========                in /usr/lib/x86_64-linux-gnu/libcuda.so.1
  =========     Host Frame:libcudart_static_5382377d5c772c9d197c0cda9fd9742ee6ad893c [0xd2dfd]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:libcudart_static_f74e2f2bcf2cf49bd1a61332e1d15bd1e748f9cf [0x9da82]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:cudaMalloc [0xe2354]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:void C_A_T_C_H_T_E_M_P_L_A_T_E_T_E_S_T_F_U_N_C_0<metal::list<> >() [0x8a807]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:Catch::RunContext::invokeActiveTestCase() [0x2c072]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:Catch::RunContext::runCurrentTest(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >&) [0x444d7]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:Catch::RunContext::runTest(Catch::TestCase const&) [0x4d1ba]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:Catch::Session::runInternal() [0x5763f]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:Catch::Session::run() [0x57c2d]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame:main [0x21d45]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api
  =========     Host Frame: [0x23a8f]
  =========                in /usr/lib/x86_64-linux-gnu/libc.so.6
  =========     Host Frame:__libc_start_main [0x23b48]
  =========                in /usr/lib/x86_64-linux-gnu/libc.so.6
  =========     Host Frame:_start [0x22dc4]
  =========                in /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api

How to Reproduce

compute-sanitizer --tool memcheck --check-device-heap yes --leak-check full --padding 512 --track-stream-ordered-races all --check-warpgroup-mma yes --require-cuda-init no --check-exit-code yes --error-exitcode 1 --nvtx true /home/coder/cccl/build/cuda12.4-gcc13/cub-cpp17/bin/cub.cpp17.test.device_select_api

Expected behavior

No diagnostics emitted.

Reproduction link

No response

Operating System

No response

nvidia-smi output

No response

NVCC version

No response
@alliepiper alliepiper added the bug Something isn't working right. label Jun 21, 2024
This was referenced Jun 21, 2024
Draft
Open
@alliepiper alliepiper self-assigned this Jun 25, 2024
alliepiper added a commit to alliepiper/cccl that referenced this issue Jun 25, 2024
@alliepiper
Fix NVIDIA#1895; Memory leak in device_select_api test.
08ca1db
@alliepiper alliepiper mentioned this issue Jun 25, 2024
Merged
alliepiper added a commit to alliepiper/cccl that referenced this issue Jun 25, 2024
@alliepiper
Fix NVIDIA#1895; Memory leak in device_select_api test.
a21708a
alliepiper added a commit that referenced this issue Jun 25, 2024
@alliepiper
Sanitizer fixes (#1916)
82d74af 
* Add [skip-rapids] option.

* Remove dead skip-tests logic.

* Fix memory leak in CUB's radix sort custom test.

* Fix memory leak in CUB's test_util_device

* Fix memory leak in catch2_test_vsmem.

* Fix #1895; Memory leak in device_select_api test.

* Fix #1897; memory leak in test_block_radix_rank.cu

* Fix #1896; memory leak in CUB's NVRTC test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alliepiper alliepiper

Labels
bug Something isn't working right.
Projects
CCCL
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Sanitizer fixes alliepiper/cccl
1 participant
@alliepiper