[gem5-dev] problem with docker image?

2021-05-24 Thread Gabe Black via gem5-dev
Hi folks. I've noticed kokoro runs occasionally failing with: bash: : command not found This is after the docker command runs it looks like, so I'm assuming this is a problem with our docker image? Any ideas? https://source.cloud.google.com/results/invocations/563a3724-49d8-4136-b30f-e19f85db833

[gem5-dev] Build failed in Jenkins: nightly #321

2021-05-24 Thread jenkins-no-reply--- via gem5-dev
See Changes: -- [...truncated 490.92 KB...] [ RUN ] TrieTestData.SingleEntry [ OK ] TrieTestData.SingleEntry (0 ms) [ RUN ] TrieTestData.TwoOverlappingEntries [ OK ] TrieTest

[gem5-dev] Change in gem5/gem5[develop]: mem: Add a ThreadContext helper constructor for PortProxy.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45863 ) Change subject: mem: Add a ThreadContext helper constructor for PortProxy. .. mem: Add a ThreadContext helper constructor for

[gem5-dev] Change in gem5/gem5[develop]: base: Construct debug flags on the heap

2021-05-24 Thread Jui-min Lee (Gerrit) via gem5-dev
Jui-min Lee has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45582 ) Change subject: base: Construct debug flags on the heap .. base: Construct debug flags on the heap Debug flags were directl

[gem5-dev] Change in gem5/gem5[develop]: scons: Increase the minimum version of clang to 6.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45621 ) Change subject: scons: Increase the minimum version of clang to 6. .. scons: Increase the minimum version of clang to 6. Cha

[gem5-dev] Change in gem5/gem5[develop]: cpu,fastmodel: Get rid of unused (read|set)FuncExeInst.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/45919 ) Change subject: cpu,fastmodel: Get rid of unused (read|set)FuncExeInst. .. cpu,fastmodel: Get rid of unused (read|

[gem5-dev] Change in gem5/gem5[develop]: fastmodel: Implement ThreadContext::sendFunctional.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45862 ) Change subject: fastmodel: Implement ThreadContext::sendFunctional. .. fastmodel: Implement ThreadContext::sendFunctional. T

[gem5-dev] Change in gem5/gem5[develop]: cpu: Add a sendFunctional function to the ThreadContext.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45861 ) Change subject: cpu: Add a sendFunctional function to the ThreadContext. .. cpu: Add a sendFunctional function to the ThreadC

[gem5-dev] Change in gem5/gem5[develop]: base: Use a lambda to simplify the stl_helpers template.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45900 ) Change subject: base: Use a lambda to simplify the stl_helpers template. .. base: Use a lambda to simplify the stl_helpers te

[gem5-dev] Re: test breakage for GCN3 on ToT?

2021-05-24 Thread Gabe Black via gem5-dev
Yes, some sort of fixed environment would solve the problem I think. Really for reproducibility in general dynamic binaries should not resolve libraries on the host, they should have their own set of libraries as if they were cross compiled. Updating the host (like I did) should not change the beha

[gem5-dev] Re: test breakage for GCN3 on ToT?

2021-05-24 Thread Jason Lowe-Power via gem5-dev
We shouldn't remove that test! It's a important feature that people rely on. The solution would be to run the test within a "known" environment. The docker containers we provide should work perfectly for that. Cheers, Jason On Mon, May 24, 2021 at 11:54 AM Bobby Bruce via gem5-dev wrote: > Hey

[gem5-dev] Re: test breakage for GCN3 on ToT?

2021-05-24 Thread Bobby Bruce via gem5-dev
Hey Gabe, When you say the 'hello' binary is dynamically linked, we're talking about the 'hello64-dynamic' binary? I agree these tests should probably be removed. (Does anyone have any objection to this?) This only appears to be half your problem though, as you've got "MatchStdoutNoPerf" tests fa

[gem5-dev] Change in gem5/gem5[develop]: cpu: Fix syntax in cpu/simple_thread.hh.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45860 ) Change subject: cpu: Fix syntax in cpu/simple_thread.hh. .. cpu: Fix syntax in cpu/simple_thread.hh. Change-Id: Id23de4651eb

[gem5-dev] Change in gem5/gem5[develop]: cpu: Fix style in cpu/simple/base.hh.

2021-05-24 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45859 ) Change subject: cpu: Fix style in cpu/simple/base.hh. .. cpu: Fix style in cpu/simple/base.hh. Change-Id: Ie5ef296ef82bf974c