-
Notifications
You must be signed in to change notification settings - Fork 609
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.
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
Add synchronization to read/write operations in image decoder cache #1702
Conversation
Signed-off-by: Joaquin Anton <janton@nvidia.com>
} | ||
|
||
ImageCacheBlob::~ImageCacheBlob() { | ||
try { | ||
CUDA_CALL(cudaStreamSynchronize(cache_stream_)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A general remark: doing this try/terminate defeats the debugger.
/** | ||
* @brief Synchronizes internal cache CUDA stream with a provided stream before a cache reading | ||
* operation | ||
* @remarks To be called before launching a memory copy from a pointer provided by Get |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can also mention here that Add
doesn't need extra synchronization.
Signed-off-by: Joaquin Anton <janton@nvidia.com>
!build |
CI MESSAGE: [1099620]: BUILD STARTED |
CI MESSAGE: [1099620]: BUILD FAILED |
Signed-off-by: Joaquin Anton <janton@nvidia.com>
!build |
CI MESSAGE: [1099763]: BUILD STARTED |
CI MESSAGE: [1099763]: BUILD PASSED |
Signed-off-by: Joaquin Anton janton@nvidia.com
Why we need this PR?
Pick one, remove the rest
What happened in this PR?
Fill relevant points, put NA otherwise. Replace anything inside []
*Introduced an internal cache stream that it is used to synchronize read/write access
ImageDecoder with cached enabled
Usage of CUDA streams/events
Existing tests
NA
JIRA TASK: [Use DALI-XXXX or NA]