gem5
gem5 copied to clipboard
gem5
misc: Add Pannotia GPU tests
This adds the Pannotia GPU tests to be run weekly, though currently the binaries for these tests aren't compiling properly due to being unable to find m5ops, so they are still a work in progress.
Change-Id: I7feb28aa1921e67ecacd084e84edcf43a87241ba
Is there something different about this application vs. other applications using m5ops that is causing the compilation issues?
@mkjost0 : now that https://github.com/gem5/gem5-resources/pull/5 and https://github.com/gem5/gem5/pull/186 have been merged in, minimally lines 74-76 will need to change to use the new input args for FW. @jebraun3 please help @mkjost0 with what change is needed.
Maximally, we should work together to figure out how to update the YAML file to the new format James used in https://github.com/gem5/gem5/pull/186 in weekly.sh -- it looks like 74-76 don't use that format. We are happy to help in any way here to make it easy so James' subsequent patches can update the weekly YAML file.
@mkjost0 Sorry for responding to this so late. For FW to work, the options in line 76 would need to change to be --options="-f 1k_128k.gr -m default". To fully utilize the new changes, FW should be run on the underlying hardware (not in gem5) with the options --options="-f 1k_128k.gr -m generate", then in gem5 with the options --options="-m usemmap". Please let me know if you have any questions.
Closing as this appears to have abandoned. An issue has been created to track the adding of Pannotia to the GPU tests if desired: #1422
Closing as this appears to have abandoned. An issue has been created to track the adding of Pannotia to the GPU tests if desired: #1422
I was not aware the GPU tests had been removed from weekly ... do you know what they were failing on?
I
Closing as this appears to have abandoned. An issue has been created to track the adding of Pannotia to the GPU tests if desired: #1422
I was not aware the GPU tests had been removed from weekly ... do you know what they were failing on?
I don't have a good grasp of this, only to say that this was a change Melissa had yet to finish upon leaving. YOu can see I tried back in November to take over her work but this PR is dependent on a docker image she didn't provide sources for. I'd probably advise anyone taking over this task to just start from scratch otherwise it's confusing
I also was a little surprised we never got these in. I felt we had, but I checked and they were not translated from the jenkins to the GitHub Actions runner system. They were just here for someone else to pick up.