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 public traces: update doc #7139

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

i#6662 public traces: update doc #7139

wants to merge 8 commits into from

Conversation

edeiana
Copy link
Contributor

@edeiana edeiana commented Dec 16, 2024

Updates the documentation on Public Google Workload Traces.
Gives a description of the new format: the #DR_ISA_REGDEPS
synthetic ISA that instructions follow, what TRACE_MARKER_TYPE_*
are present or have been modified, what is v2p.textproto.

Issue #6662

Updates the documentation on Public Google Workload Traces.
Gives a description of the new format: the #DR_ISA_REGDEPS
synthetic ISA that instructions follow, what TRACE_MARKER_TYPE_*
are present or have been modified, what is v2p.textproto.

Issue #6662
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved

\section sec_google_get Getting the Traces

The Google Workload Traces can be downloaded from:

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

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume we'd use the same bucket? Should we make version_1 and version_2 subdirs? Or just delete v1? Maybe just delete.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mhm, I'd keep version 1 in that bucket as is.
I added the deprecated section that says DynamoRIO 11.0 is the last version that will support these traces, so if somebody is still using them, they can still do so with DynamoRIO 11.0.
Can't we use a new bucket? "external_traces_v2"? Do we need a subdir?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's discuss offline.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The new one could be sthg like console.cloud.google.com/storage/browser/external-traces-v2.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's right, I created: https://console.cloud.google.com/storage/browser/external-traces-v2 (now empty).
I've added it as link to the new public traces.
I also added a section about deprecated public v1 traces with the old link to them.

I think it might be a good idea to also have DOI for citations? We could make one with https://zenodo.org/ . This way it's easy to track who/how many cite this work.

clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Outdated Show resolved Hide resolved
clients/drcachesim/docs/drcachesim.dox.in Show resolved Hide resolved
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.

2 participants