[gem5-dev] Re: kokoro failures due to bug in gem5-art?

2021-08-05 Thread Gabe Black via gem5-dev
One more: https://source.cloud.google.com/results/invocations/8f6a185e-0258-4f9c-b573-e79d1a86a5a1/targets/gem5%2Fgcp_ubuntu%2Fpresubmit/log On Thu, Aug 5, 2021 at 3:02 PM Gabe Black wrote: > Yeah, I wouldn't be surprised if it was specific to kokoro, or something > about how its networking is

[gem5-dev] Change in gem5/gem5[develop]: scons: Use the os.path prefix when using components of that module.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48126 ) Change subject: scons: Use the os.path prefix when using components of that module. .. scons: Use the os.path prefix when

[gem5-dev] Change in gem5/gem5[develop]: scons: Delete the comparison operators from SourceFile.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48125 ) Change subject: scons: Delete the comparison operators from SourceFile. .. scons: Delete the comparison operators from

[gem5-dev] Change in gem5/gem5[develop]: scons: Eliminate the SourceFile.basename property.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48124 ) Change subject: scons: Eliminate the SourceFile.basename property. .. scons: Eliminate the SourceFile.basename property.

[gem5-dev] Change in gem5/gem5[develop]: scons: Replace the SourceFile.filename property with attribute.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48123 ) Change subject: scons: Replace the SourceFile.filename property with attribute. .. scons: Replace the SourceFile.filename

[gem5-dev] Change in gem5/gem5[develop]: scons: Replace the extname property with os.path.splitext().

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48122 ) Change subject: scons: Replace the extname property with os.path.splitext(). .. scons: Replace the extname property with

[gem5-dev] Change in gem5/gem5[develop]: scons: Make all Executables strip-able, and de-special case .fast.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48120 ) Change subject: scons: Make all Executables strip-able, and de-special case .fast. .. scons: Make all Executables

[gem5-dev] Change in gem5/gem5[develop]: scons: Get rid of the unused "dirname" property of SourceFile.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48121 ) Change subject: scons: Get rid of the unused "dirname" property of SourceFile. .. scons: Get rid of the unused "dirname"

[gem5-dev] Running clang-format

2021-08-05 Thread Jason Lowe-Power via gem5-dev
Hi everyone, For fun(?), I have come up with a clang-format file that's pretty darn close to our current style. However, the problem is that most of our files don't actually meet our style guide! :'( So, running this clang-format generates an enormous changeset. Skimming through the (100,000+)

[gem5-dev] Re: python versions used by SCons and gem5

2021-08-05 Thread Jason Lowe-Power via gem5-dev
For MyPy that won't help.. the problem is that it looks for _m5/objects/.py *file* which doesn't exist on disk... we have to somehow convince it to look in the marshalled data. I was playing around with this, but I haven't figured it out yet. BTW, your suggested changes get a +1 from me. Cheers,

[gem5-dev] Re: python versions used by SCons and gem5

2021-08-05 Thread Gabe Black via gem5-dev
On Thu, Aug 5, 2021 at 3:08 PM Bobby Bruce wrote: > Hey Gabe, > > Just so we're on the same page, I'd like to take a moment to outline how I > understand this problem/solution. Please let me know if I've understood > correctly: > > We essentially have three components of the project which

[gem5-dev] Re: python versions used by SCons and gem5

2021-08-05 Thread Bobby Bruce via gem5-dev
Hey Gabe, Just so we're on the same page, I'd like to take a moment to outline how I understand this problem/solution. Please let me know if I've understood correctly: We essentially have three components of the project which interpret gem5 Python code: The gem5 binary, the Marshal binary, and

[gem5-dev] Re: kokoro failures due to bug in gem5-art?

2021-08-05 Thread Gabe Black via gem5-dev
Yeah, I wouldn't be surprised if it was specific to kokoro, or something about how its networking is set up. Those failures seem to have stopped happening now, or at least are happening much less. I don't remember seeing one for a while at least. Hopefully we can avoid making our timeout any

[gem5-dev] Re: kokoro failures due to bug in gem5-art?

2021-08-05 Thread Bobby Bruce via gem5-dev
That theory could be true, and I certainly don't have any better ideas, though I've never observed any hang on my local machine when recreating this issue. It could be something specific to Kokoro. I've fixed the gem5art error here: https://gem5-review.googlesource.com/c/public/gem5/+/49044. We

[gem5-dev] Change in gem5/gem5[develop]: tests: Remove setuptools upgrade in gem5art-tests.sh

2021-08-05 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49043 ) Change subject: tests: Remove setuptools upgrade in gem5art-tests.sh .. tests: Remove setuptools upgrade in

[gem5-dev] Change in gem5/gem5[develop]: tests: Update pip prior to installing gem5art packages

2021-08-05 Thread Bobby R. Bruce (Gerrit) via gem5-dev
Bobby R. Bruce has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49044 ) Change subject: tests: Update pip prior to installing gem5art packages .. tests: Update pip prior to

[gem5-dev] Change in gem5/gem5[develop]: arch,cpu: Rename RegClass to RegClassType.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/45229 ) Change subject: arch,cpu: Rename RegClass to RegClassType. .. arch,cpu: Rename RegClass to RegClassType. This type is

[gem5-dev] Change in gem5/gem5[develop]: base: handle initial communication with GDB in `attach()`

2021-08-05 Thread Jan Vrany (Gerrit) via gem5-dev
Jan Vrany has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48182 ) Change subject: base: handle initial communication with GDB in `attach()` .. base: handle initial communication with GDB in

[gem5-dev] Change in gem5/gem5[develop]: base: Extract GDB command processing into separate function

2021-08-05 Thread Jan Vrany (Gerrit) via gem5-dev
Jan Vrany has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48181 ) Change subject: base: Extract GDB command processing into separate function .. base: Extract GDB command processing into

[gem5-dev] Re: Build failed in Jenkins: compiler-checks #145

2021-08-05 Thread Gabe Black via gem5-dev
https://gem5-review.googlesource.com/c/public/gem5/+/49023 On Mon, Aug 2, 2021 at 11:19 AM Jason Lowe-Power via gem5-dev < gem5-dev@gem5.org> wrote: > Looks like GCN is failing with clang-11. Could someone take a look? Bobby > is on vacation this week. > > Cheers, > Jason > > On Mon, Aug 2, 2021

[gem5-dev] Change in gem5/gem5[develop]: arch-gcn3: Fix initAtomicAccess.

2021-08-05 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/49023 ) Change subject: arch-gcn3: Fix initAtomicAccess. .. arch-gcn3: Fix initAtomicAccess. This function used

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

2021-08-05 Thread jenkins-no-reply--- via gem5-dev
See Changes: [Giacomo Travaglini] base: Add an exclude method to the AddrRange class [Giacomo Travaglini] python: Expose the AddrRange exclude to the python world [gabe.black] arm: Make the misc reg class return the name

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Explicitly implement I/DTLBI Ops in TLB

2021-08-05 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48148 ) Change subject: arch-arm: Explicitly implement I/DTLBI Ops in TLB .. arch-arm: Explicitly implement I/DTLBI Ops in

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Provide support for a multilevel-TLB in the ArmMMU

2021-08-05 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48149 ) Change subject: arch-arm: Provide support for a multilevel-TLB in the ArmMMU .. arch-arm: Provide support for a

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Distinguish Instruction from Data TLB entries

2021-08-05 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48147 ) Change subject: arch-arm: Distinguish Instruction from Data TLB entries .. arch-arm: Distinguish Instruction from

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Add a shared L2 TLB to the default ArmMMU

2021-08-05 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48150 ) Change subject: arch-arm: Add a shared L2 TLB to the default ArmMMU .. arch-arm: Add a shared L2 TLB to the default

[gem5-dev] Change in gem5/gem5[develop]: arch: Implement operator& for TypeTLB

2021-08-05 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/48463 ) Change subject: arch: Implement operator& for TypeTLB .. arch: Implement operator& for TypeTLB Change-Id:

[gem5-dev] Build failed in Jenkins: compiler-checks #148

2021-08-05 Thread jenkins-no-reply--- via gem5-dev
See Changes: [gabe.black] arch,cpu: Rename arch/generic/types.hh to pcstate.hh. [gabe.black] arch: Promote the micropc to the base PCState class. [gabe.black] scons: Disable the free-nonheap-object warning for