Skip to content

Commit

Permalink
Adds link to external-traces-v2, adds signal and uncompleted instr
Browse files Browse the repository at this point in the history
marker to the list of removed markers.
  • Loading branch information
edeiana committed Dec 18, 2024
1 parent 9b97a32 commit 5f52afa
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions clients/drcachesim/docs/drcachesim.dox.in
Original file line number Diff line number Diff line change
Expand Up @@ -1065,7 +1065,7 @@ A #DR_ISA_REGDEPS #instr_t contains the following information:
information is present only for decoded instructions (i.e., #instr_t generated by
decode() or decode_from_copy()). This information can be obtained using instr_length().
Be aware that in Google Workload Traces the instruction fetch size of a
#dynamorio::drmemtrace::memref_t# and the instr_length() of the corresponding fetched
#dynamorio::drmemtrace::memref_t and the instr_length() of the corresponding fetched
instruction do not match! For convenience reasons we kept the instruction fetch size to
be the same as the size of the original ISA instruction.

Expand All @@ -1083,6 +1083,8 @@ All markers of the original trace are present, except for:
- #dynamorio::drmemtrace::TRACE_MARKER_TYPE_SYSCALL_TRACE_START
- #dynamorio::drmemtrace::TRACE_MARKER_TYPE_SYSCALL_TRACE_END
- #dynamorio::drmemtrace::TRACE_MARKER_TYPE_SYSCALL_FAILED
- #dynamorio::drmemtrace::TRACE_MARKER_TYPE_SIGNAL_NUMBER
- #dynamorio::drmemtrace::TRACE_MARKER_TYPE_UNCOMPLETED_INSTRUCTION
Which have been removed.
Because tracing overhead results into inflated context switches, the
#dynamorio::drmemtrace::TRACE_MARKER_TYPE_CPU_ID values have been modified to
Expand All @@ -1106,7 +1108,7 @@ v2p.textproto format.

The Google Workload Traces can be downloaded from:

- [Google workload trace folder](TODO: add new link to Google Storage Bucket once known)
- [Google workload trace folder](https://console.cloud.google.com/storage/browser/external-traces-v2)

Directory structure:
- \verbatim
Expand Down

0 comments on commit 5f52afa

Please sign in to comment.