-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Fixed deadlock on pipeline.stop() with playback device in non realtime #8625
Conversation
2. added unit test test-non-realtime.py
@@ -0,0 +1,31 @@ | |||
# License: Apache 2.0. See LICENSE file in root directory. | |||
# Copyright(c) 2020 Intel Corporation. All Rights Reserved. |
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.
2021
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.
done
# and non realtime mode is not stuck due to deadlock of | ||
# pipeline stop thread and syncer blocking enqueue thread (DSO-15157) | ||
############################################################################################# | ||
test.start("Playback with non realtime doesn't stuck at stop") |
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.
doesn't get stuck
or
isn't stuck
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.
done
############################################################################################# | ||
test.start("Playback with non realtime doesn't stuck at stop") | ||
current_dir = os.path.dirname( os.path.abspath( __file__ )) | ||
filename = current_dir + os.sep + 'recording.bag' |
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.
Where is the file put??
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.
Shouldn't it be called recording_deadlock.bag
? I'm confused...
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.
fixed
@@ -0,0 +1,31 @@ | |||
# License: Apache 2.0. See LICENSE file in root directory. |
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.
Matan put his record & playback test under func/rec-play/
Talk to him and synchronize please
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.
done
[tracked on DSO-15157]