[gem5-dev] Change in gem5/gem5[develop]: base: Add a link to documentation in the remote GDB header file.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44608 ) Change subject: base: Add a link to documentation in the remote GDB header file. .. base: Add a link to documentation in

[gem5-dev] Change in gem5/gem5[develop]: base: Streamline the "send" method of the BaseRemoteGDB class.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44607 ) Change subject: base: Streamline the "send" method of the BaseRemoteGDB class. .. base: Streamline the "send" method of

[gem5-dev] Change in gem5/gem5[develop]: arch: Collapse unused size parameter from "as" VecPredReg method.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42000 ) Change subject: arch: Collapse unused size parameter from "as" VecPredReg method. .. arch: Collapse unused size parameter

[gem5-dev] Change in gem5/gem5[develop]: sim: Don't needlessly recreate ISA types in InstRecord.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42002 ) Change subject: sim: Don't needlessly recreate ISA types in InstRecord. .. sim: Don't needlessly recreate ISA types in

[gem5-dev] Change in gem5/gem5[develop]: arch,cpu: Separate printing and serialization of VecPredReg.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/41999 ) Change subject: arch,cpu: Separate printing and serialization of VecPredReg. .. arch,cpu: Separate printing and

[gem5-dev] Change in gem5/gem5[develop]: cpu: Use the built in << for VecReg and VecPredReg in ExeTrace.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42001 ) Change subject: cpu: Use the built in << for VecReg and VecPredReg in ExeTrace. .. cpu: Use the built in << for VecReg

[gem5-dev] Change in gem5/gem5[develop]: arch,mem: Use szext instead of sext as appropriate.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42386 ) Change subject: arch,mem: Use szext instead of sext as appropriate. .. arch,mem: Use szext instead of sext as appropriate.

[gem5-dev] Change in gem5/gem5[develop]: arch-sparc: Fix some bit manipulation bugs.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42603 ) Change subject: arch-sparc: Fix some bit manipulation bugs. .. arch-sparc: Fix some bit manipulation bugs. Several sext<>

[gem5-dev] Change in gem5/gem5[develop]: arch-sparc: Wrap overly long lines in the decoder definition.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/42383 ) Change subject: arch-sparc: Wrap overly long lines in the decoder definition. .. arch-sparc: Wrap overly long lines in

[gem5-dev] Change in gem5/gem5[develop]: base: Improve handling of thread IDs for remote GDB.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44606 ) Change subject: base: Improve handling of thread IDs for remote GDB. .. base: Improve handling of thread IDs for remote GDB.

[gem5-dev] Change in gem5/gem5[develop]: sim: Minor cleanup of the System class.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44605 ) Change subject: sim: Minor cleanup of the System class. .. sim: Minor cleanup of the System class. Fix style, move constant

[gem5-dev] Re: error from new test_convert.py?

2021-04-19 Thread Bobby Bruce via gem5-dev
My quick fix for this is here: https://gem5-review.googlesource.com/c/public/gem5/+/44625 -- Dr. Bobby R. Bruce Room 2235, Kemper Hall, UC Davis Davis, CA, 95616 web: https://www.bobbybruce.net On Mon, Apr 19, 2021 at 7:20 AM Bobby Bruce wrote: > I think I've got a solution to this (in my

[gem5-dev] Change in gem5/gem5[develop]: python,tests: Update pyunit tests to run in TestLib

2021-04-19 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/+/44625 ) Change subject: python,tests: Update pyunit tests to run in TestLib .. python,tests: Update pyunit tests to

[gem5-dev] Re: gem5 namespace

2021-04-19 Thread Jason Lowe-Power via gem5-dev
> > I still favor namespace gem5 - it'd be the "external external" API, i.e. > we probably wouldn't be using it within src/ that much, and it would be > used by other simulators... This is why I'm strongly in favor of lowercase gem5. When external projects link to gem5 (which is *going* to

[gem5-dev] Re: error from new test_convert.py?

2021-04-19 Thread Bobby Bruce via gem5-dev
I think I've got a solution to this (in my head at least), I'll try to push a patch today. I like having all the tests runnable in the `test` directory with `./main.py`, so it irks me this is a bit of a special case (not that we don't already have a few special cases, most notable the GTests). I

[gem5-dev] Re: error from new test_convert.py?

2021-04-19 Thread Giacomo Travaglini via gem5-dev
Yes, This is why I usually run using tests/gem5 as root directory ./tests/main.py [...] \ tests/gem5 <- root directory Testlib is basically trying to load every python file it finds on its walk. We should definitely add python unit tests to our presubmit runs IMHO. The simplest way is via

[gem5-dev] Change in gem5/gem5[develop]: scons: Allow declaring dependencies of tags

2021-04-19 Thread Daniel Carvalho (Gerrit) via gem5-dev
Daniel Carvalho has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/43587 ) Change subject: scons: Allow declaring dependencies of tags .. scons: Allow declaring dependencies of tags It was

[gem5-dev] Re: gem5 namespace

2021-04-19 Thread Bobby Bruce via gem5-dev
Nothing gets software devs as engaged as when discussing naming conventions :). I vote for "gem5" (lowercase) namespace, with all caps MACROS, but my sole reason for this is I've grown to flinch whenever I see "Gem5", which I admit isn't the best argument. I echo Daniel's comment that I care

[gem5-dev] Re: Upstreaming power-gem5

2021-04-19 Thread Sandipan Das via gem5-dev
Hi Boris, On 14/04/21 11:43 pm, Boris Shingarov wrote: > Hi Sandipan, > > I notice some of the commits (which were, if not blocking reviewing other > commits, but at least making comprehension of the whole body of commits > harder > for me) are ready for merge, for example >

[gem5-dev] Re: error from new test_convert.py?

2021-04-19 Thread Bobby Bruce via gem5-dev
The exception being thrown is "No module named 'm5'". m5 is not in the path, but could easily be added. >From looking at https://gem5-review.googlesource.com/c/public/gem5/+/39377, this was never intended to be run via `main.py`, though "test_convert.py" has the "test" prefix so our TestLib

[gem5-dev] Gem5 CPU Model & ISA Addition

2021-04-19 Thread Vignesh Manjunath via gem5-dev
Hello Team Good day!!! I am a researcher at Pro2Future, GmbH under Technical University of Graz, Austria. With an amazing platform like Gem5, I would like to use the framework to study the instruction behavior in the CPU based on a custom ISA and Pipeline model. As a part of the same, I

[gem5-dev] Change in gem5/gem5[develop]: sim,cpu: Move the remote GDB stub into the workload.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44617 ) Change subject: sim,cpu: Move the remote GDB stub into the workload. .. sim,cpu: Move the remote GDB stub into

[gem5-dev] Change in gem5/gem5[develop]: arch,base,sim: Construct the GDB stub with no threads.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44618 ) Change subject: arch,base,sim: Construct the GDB stub with no threads. .. arch,base,sim: Construct the GDB stub

[gem5-dev] Change in gem5/gem5[develop]: arch,base,sim: Move GDB stub creation into the arch Workloads.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44619 ) Change subject: arch,base,sim: Move GDB stub creation into the arch Workloads. .. arch,base,sim: Move GDB stub

[gem5-dev] Change in gem5/gem5[develop]: sim: Track ThreadContext-s in the Workload object.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44616 ) Change subject: sim: Track ThreadContext-s in the Workload object. .. sim: Track ThreadContext-s in the Workload

[gem5-dev] Change in gem5/gem5[develop]: arch: Eliminate the now unused remote_gdb.hh switching header.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44620 ) Change subject: arch: Eliminate the now unused remote_gdb.hh switching header. .. arch: Eliminate the now

[gem5-dev] Change in gem5/gem5[develop]: cpu,sim: Set ThreadContext's ContextID right away.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44615 ) Change subject: cpu,sim: Set ThreadContext's ContextID right away. .. cpu,sim: Set ThreadContext's ContextID

[gem5-dev] Change in gem5/gem5[develop]: base: When switching remote GDB threads, re-align on an inst boundary.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44613 ) Change subject: base: When switching remote GDB threads, re-align on an inst boundary. .. base: When switching

[gem5-dev] Change in gem5/gem5[develop]: base: Improve handling of thread IDs for remote GDB.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44606 ) Change subject: base: Improve handling of thread IDs for remote GDB. .. base: Improve handling of thread IDs for

[gem5-dev] Change in gem5/gem5[develop]: base: Streamline the "send" method of the BaseRemoteGDB class.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44607 ) Change subject: base: Streamline the "send" method of the BaseRemoteGDB class. .. base: Streamline the "send"

[gem5-dev] Change in gem5/gem5[develop]: arch-x86: If possible, use the workload to pick GDB arch.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44614 ) Change subject: arch-x86: If possible, use the workload to pick GDB arch. .. arch-x86: If possible, use the

[gem5-dev] Change in gem5/gem5[develop]: base: Don't wait for an inactive CPU for remote GDB.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44612 ) Change subject: base: Don't wait for an inactive CPU for remote GDB. .. base: Don't wait for an inactive CPU for

[gem5-dev] Change in gem5/gem5[develop]: base: Fill out the 'H' thread setting command in remote GDB.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44609 ) Change subject: base: Fill out the 'H' thread setting command in remote GDB. .. base: Fill out the 'H' thread

[gem5-dev] Change in gem5/gem5[develop]: base: Check the context ID when replacing a ThreadContext in GDB.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44610 ) Change subject: base: Check the context ID when replacing a ThreadContext in GDB. .. base: Check the context

[gem5-dev] Change in gem5/gem5[develop]: base: Add a link to documentation in the remote GDB header file.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44608 ) Change subject: base: Add a link to documentation in the remote GDB header file. .. base: Add a link to

[gem5-dev] Change in gem5/gem5[develop]: sim: Minor cleanup of the System class.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44605 ) Change subject: sim: Minor cleanup of the System class. .. sim: Minor cleanup of the System class. Fix style,

[gem5-dev] Change in gem5/gem5[develop]: base: Make the BaseRemoteGDB class able to handle multiple TCs.

2021-04-19 Thread Gabe Black (Gerrit) via gem5-dev
Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/44611 ) Change subject: base: Make the BaseRemoteGDB class able to handle multiple TCs. .. base: Make the

[gem5-dev] Change in gem5/gem5[develop]: util: Remove unused package import

2021-04-19 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44511 ) Change subject: util: Remove unused package import .. util: Remove unused package import Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: util: Fix cpt_upgrader format string

2021-04-19 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44510 ) Change subject: util: Fix cpt_upgrader format string .. util: Fix cpt_upgrader format string Change-Id:

[gem5-dev] Change in gem5/gem5[develop]: configs: Remove Ruby on ARM warning

2021-04-19 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44509 ) Change subject: configs: Remove Ruby on ARM warning .. configs: Remove Ruby on ARM warning The warning was