DynamoRIO / DynamoRIO/dynamorio

Add new cache management instruction support to drmemtrace

Open
#7,111 6 comments 0 reactions 1 assignee Claimed by @derekbruening View on GitHub
Component-DrMemtrace
Dominant language
C
Stars
3.2k
Forks
629
Avg merge
2d 15h
Merged PRs (30d)
31

Description

My pending comment in https://github.com/DynamoRIO/dynamorio/pull/7109 which adds CLFLUSHOPT opcode decoding support:

drmemtrace needs to be udpated as well: it looks like CLFLUSHOPT also invalidates all cache levels and so should be treated like CLFLUSH in memtraces (or would a simulator want to know about the subtle differences?). So just adding to 2 spots in instru.cpp https://github.com/DynamoRIO/dynamorio/blob/master/clients/drcachesim/tracer/instru.cpp#L227 and https://github.com/DynamoRIO/dynamorio/blob/master/clients/drcachesim/tracer/instru.cpp#L244 should be enough (optionally extend invariant_checker_test). Hmm. Maybe enough questions this should be separated out.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.