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

PR: compare hammersim with gem5/develop v.23 #5

Open
wants to merge 16 commits into
base: develop-23
Choose a base branch
from

Conversation

kaustav-goswami
Copy link
Collaborator

@powerjg Please ignore the .github files for this review.

BobbyRBruce and others added 15 commits November 21, 2023 22:07
This was deprecated in C++14 and removed in C++17. This has been
replaced with std::random. This has been implemented to ensure
reproducible results despite (pseudo)random behavior.

Change-Id: Idd52bc997547c7f8c1be88f6130adff8a37b4116
Change-Id: I3bbcfd4dd9798149b37d4a2824fe63652e29786c
This is a simple copy of the current state of the .github on the develop
branch, as of 2023-07-27. The stable branch .github dir should never be
ahead of that on develop. Therefore this should be safe to do.

Change-Id: I1e39de2d1f923d1834d0a77f79a1ff3220964bba
Change-Id: I07e5e5f3bc95b78459b77c0f1170923f6c9daf18
Change-Id: Icd673083f23a465205bea12407bf265e2ba6fb4a
Change-Id: I5ae5081e0ac5524271e6c8300917d7d1e16d71ee
Change-Id: I7755f90c1b5d81ff1cf66920f229be921d47e844
This is done periodically as the Github Action's infrastructure reads
files from the repos main branch (`stable`) where as changes are made to
`develop`'s ".github" which should be made live ASAP.

This does not affect the gem5 build or any configuration scripts, only
how testing is performed via GitHub and other info used by Github (e.g.,
".github/ISSUE_TEMPLATE" which outlines the templates for creating
issues via our GitHub issues page.
This change updates DRAMInterface to to be compatible with gem5 v23

Signed-off-by: Kaustav Goswami <kggoswami@ucdavis.edu>
This change adds information on how to use HammerSim in the
README.md file.

Signed-off-by: Kaustav Goswami <kggoswami@ucdavis.edu>
This change adds a simple statistically generated variation map. It
uses VARIUS to generate the initial map. The map is formated into
a json file.

Change-Id: I2ad252b8fd8ffee95678293892c7232df890f18a
Signed-off-by: Kaustav Goswami <kggoswami@ucdavis.edu>
This change changes the default path for the device_file in the
util/hammersim directory.

Change-Id: I33aa2df7a474276ee9fe2e85df61add209babc47
Signed-off-by: Kaustav Goswami <kggoswami@ucdavis.edu>
This change adds sample configs to simulate rowhammer using
hammersim.

Change-Id: I8840c55498ad379e953550f765b4880901a120ca
Signed-off-by: Kaustav Goswami <kggoswami@ucdavis.edu>
This change cleans most of the non-important comments from the
hammersim codebase.

Change-Id: I316a067da79ca12c1e6dbb930887436237a687df
Signed-off-by: Kaustav Goswami <kggoswami@ucdavis.edu>
@kaustav-goswami kaustav-goswami self-assigned this Nov 22, 2023
@kaustav-goswami kaustav-goswami marked this pull request as draft November 22, 2023 06:20
This change resolves all the conflits encountered while rebasing.

Signed-off-by: Kaustav Goswami <kggoswami@ucdavis.edu>
@kaustav-goswami kaustav-goswami marked this pull request as ready for review November 22, 2023 06:31
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