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.

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

i#6662 encodings2regdeps: data race bug fix #6809

Merged
merged 12 commits into from
May 16, 2024
Merged

i#6662 encodings2regdeps: data race bug fix #6809

merged 12 commits into from
May 16, 2024

Conversation

edeiana
Copy link
Contributor

@edeiana edeiana commented May 13, 2024

Fixes a data race due to multiple dr_standalone_init() done in parallel (per shard) by encodings2regdeps_filter_t.
dcontext is now initialized one time by record_filter_t and passed to its filters through the record_filter_info_t interface.

Avoids a data race in opcode_mix where all threads set the dcontext isa_mode to DR_ISA_REGDEPS for regdeps input traces.

Issue #6662 #6812

Fixes a data race due to multiple dr_standalone_init()
done in parallel (per shard) by encodings2regdeps_filter_t.

dcontext is now initialize one time by record_filter_t and
passed to its filters through the record_filter_info_t interface.

Issue #6662
is expected in dcontext_t initialization when
running record_filter_launcher.
DR_ISA_REGDEPS in opcode_mix dcontext.
@edeiana edeiana marked this pull request as ready for review May 14, 2024 09:33
@edeiana edeiana requested a review from abhinav92003 May 14, 2024 17:50
data race when we read the dcontext isa_mode in decode().
@edeiana edeiana requested a review from abhinav92003 May 15, 2024 07:49
clients/drcachesim/tools/filter/record_filter.h Outdated Show resolved Hide resolved
clients/drcachesim/tools/opcode_mix.cpp Outdated Show resolved Hide resolved
clients/drcachesim/tools/opcode_mix.cpp Show resolved Hide resolved
clients/drcachesim/tools/opcode_mix.h Outdated Show resolved Hide resolved
clients/drcachesim/tools/opcode_mix.cpp Show resolved Hide resolved
@edeiana edeiana merged commit 6705e16 into master May 16, 2024
17 checks passed
@edeiana edeiana deleted the i6662-data-race branch May 16, 2024 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants