search

ledatelescope / bifrost

A stream processing framework for high-throughput applications.
BSD 3-Clause "New" or "Revised" License
64 stars 29 forks source link

Allow `cudaMemoryTypeUnregistered` #220

Closed jaycedowell closed 7 months ago

jaycedowell commented 10 months ago

This PR addresses a change in behavior of cudaPointerGetAttributes starting in CUDA 11 . Before CUDA 11 "normal" system memory (that allocated outside of CUDA) would generate an error return code. Starting with CUDA 11 the call returns success with a memoryType of cudaMemoryTypeUnregistered. This updates memory.cpp to allow this type for CUDA 10+ and map it to BF_SPACE_SYSTEM. cudaMemoryTypeHost is also now mapped to BF_SPACE_CUDA_HOST.

codecov[bot] commented 10 months ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (842e8f4) 65.39% compared to head (af5a401) 65.39%. Report is 4 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #220 +/- ## ======================================= Coverage 65.39% 65.39% ======================================= Files 67 67 Lines 5840 5840 ======================================= Hits 3819 3819 Misses 2021 2021 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.