Update opcode_mix initializers #6812
Comments
edeiana
added a commit
that referenced
this issue
May 16, 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
analysis_tool_tmpl_tdeprecated the use ofinitialize()andparallel_shard_init()in favor ofinitialize_stream()andparallel_shard_init_stream()respectively.These two methods allow initializations for the analysis tool as a whole (rather than doing them in the constructor) and for a trace shard (for parallel analysis tools).
The new methods add an additional
memtrace_stream_tparameter, which enables tools to query details of the underlying trace (e.g., the file type of the trace).Currently
opcode_mixstill uses the deprecated initializers. We want to update it to use the new initializers.The text was updated successfully, but these errors were encountered: