[gem5-dev] m5 utility patch series

Hi folks. I just uploaded a series (mostly small) patches which revamp the m5 utility as described in a design document I sent out a while ago. I've done some very preliminary sanity testing, but a lot more testing can/should be done to make sure I didn't screw anything up. One thing in

[gem5-dev] Change in gem5/gem5[develop]: misc: Remove the now unused M5OP_FOREACH_ANNOTATION macro.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27204 ) Change subject: misc: Remove the now unused M5OP_FOREACH_ANNOTATION macro. .. misc: Remove the now unused

[gem5-dev] Change in gem5/gem5[develop]: misc: Get rid of constants used for critical path annotation.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27222 ) Change subject: misc: Get rid of constants used for critical path annotation. .. misc: Get rid of constants

[gem5-dev] Change in gem5/gem5[develop]: util: Add SCons build files for the aarch64 verson of the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27213 ) Change subject: util: Add SCons build files for the aarch64 verson of the m5 utility. .. util: Add SCons build

[gem5-dev] Change in gem5/gem5[develop]: util: Refactor the string->register packing function in the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27229 ) Change subject: util: Refactor the string->register packing function in the m5 utility. .. util: Refactor the

[gem5-dev] Change in gem5/gem5[develop]: util: Allow overriding the magic address in the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27244 ) Change subject: util: Allow overriding the magic address in the m5 utility. .. util: Allow overriding the magic

[gem5-dev] Change in gem5/gem5[develop]: util: Use M5OP_FOREACH to build a table of function pointers.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27240 ) Change subject: util: Use M5OP_FOREACH to build a table of function pointers. .. util: Use M5OP_FOREACH to

[gem5-dev] Change in gem5/gem5[develop]: util: Move m5op_arm_A64.S into an aarch64 subdirectory.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27216 ) Change subject: util: Move m5op_arm_A64.S into an aarch64 subdirectory. .. util: Move m5op_arm_A64.S into an

[gem5-dev] Change in gem5/gem5[develop]: util: Use a dispatch table to call ops in the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27238 ) Change subject: util: Use a dispatch table to call ops in the m5 utility. .. util: Use a dispatch table to call

[gem5-dev] Change in gem5/gem5[develop]: util: Add support for multiple call types in the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27242 ) Change subject: util: Add support for multiple call types in the m5 utility. .. util: Add support for multiple

[gem5-dev] Change in gem5/gem5[develop]: misc,util: Remove explicit '; 's from M5OP_FOREACH.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27237 ) Change subject: misc,util: Remove explicit ';'s from M5OP_FOREACH. .. misc,util: Remove explicit ';'s from

[gem5-dev] Change in gem5/gem5[develop]: util: Update the m5 util Makefiles to not use javah.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27210 ) Change subject: util: Update the m5 util Makefiles to not use javah. .. util: Update the m5 util Makefiles to not

[gem5-dev] Change in gem5/gem5[develop]: util: Add an abstraction layer for call types in the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27241 ) Change subject: util: Add an abstraction layer for call types in the m5 utility. .. util: Add an abstraction

[gem5-dev] Change in gem5/gem5[develop]: util: Pull argument parsing functions out of m5.c.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27243 ) Change subject: util: Pull argument parsing functions out of m5.c. .. util: Pull argument parsing functions out

[gem5-dev] Change in gem5/gem5[develop]: util: Add missing functions to the java wrapper for m5 ops.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27225 ) Change subject: util: Add missing functions to the java wrapper for m5 ops. .. util: Add missing functions to the

[gem5-dev] Change in gem5/gem5[develop]: util: Eliminate the sw99param m5 utility command.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27232 ) Change subject: util: Eliminate the sw99param m5 utility command. .. util: Eliminate the sw99param m5 utility

[gem5-dev] Change in gem5/gem5[develop]: util: Improve the usage text of the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27231 ) Change subject: util: Improve the usage text of the m5 utility. .. util: Improve the usage text of the m5

[gem5-dev] Change in gem5/gem5[develop]: misc: Tidy up M5OP_FOREACH and the constants in m5ops.h.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27234 ) Change subject: misc: Tidy up M5OP_FOREACH and the constants in m5ops.h. .. misc: Tidy up M5OP_FOREACH and the

[gem5-dev] Change in gem5/gem5[develop]: util: Delete the execfile m5 utility command.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27233 ) Change subject: util: Delete the execfile m5 utility command. .. util: Delete the execfile m5 utility command.

[gem5-dev] Change in gem5/gem5[develop]: misc: Add the SE mode m5 ops to m5ops.h.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27236 ) Change subject: misc: Add the SE mode m5 ops to m5ops.h. .. misc: Add the SE mode m5 ops to m5ops.h. These

[gem5-dev] Change in gem5/gem5[develop]: util: Add a helpful macro for merging two tokens to m5ops.h.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27239 ) Change subject: util: Add a helpful macro for merging two tokens to m5ops.h. .. util: Add a helpful macro for

[gem5-dev] Change in gem5/gem5[develop]: sim, misc: Rename M5OP_ANNOTATE to M5OP_RESERVED1.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27235 ) Change subject: sim,misc: Rename M5OP_ANNOTATE to M5OP_RESERVED1. .. sim,misc: Rename M5OP_ANNOTATE to

[gem5-dev] Change in gem5/gem5[develop]: util, scons: Generalize the aarch64 scons for the m5 util.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27217 ) Change subject: util,scons: Generalize the aarch64 scons for the m5 util. .. util,scons: Generalize the aarch64

[gem5-dev] Change in gem5/gem5[develop]: arm: Teach gem5 to recognize the gem5 semihosting immediate values.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27246 ) Change subject: arm: Teach gem5 to recognize the gem5 semihosting immediate values. .. arm: Teach gem5 to

[gem5-dev] Change in gem5/gem5[develop]: util: Remove the "pin" command from the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27226 ) Change subject: util: Remove the "pin" command from the m5 utility. .. util: Remove the "pin" command from the m5

[gem5-dev] Change in gem5/gem5[develop]: misc: Remove m5a_ (critical path annotation) prototypes from m5ops.h.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27220 ) Change subject: misc: Remove m5a_ (critical path annotation) prototypes from m5ops.h. .. misc: Remove m5a_

[gem5-dev] Change in gem5/gem5[develop]: util: Move source files into a src directory.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27209 ) Change subject: util: Move source files into a src directory. .. util: Move source files into a src directory.

[gem5-dev] Change in gem5/gem5[develop]: util: Further simplify the initParam implementation.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27230 ) Change subject: util: Further simplify the initParam implementation. .. util: Further simplify the initParam

[gem5-dev] Change in gem5/gem5[develop]: util: Undef M5OP at the end of assembly files that use M5OP_FOREACH.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27206 ) Change subject: util: Undef M5OP at the end of assembly files that use M5OP_FOREACH. .. util: Undef M5OP at

[gem5-dev] Change in gem5/gem5[develop]: util: Update jni_gem5Op.c so it will compile again.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27211 ) Change subject: util: Update jni_gem5Op.c so it will compile again. .. util: Update jni_gem5Op.c so it will

[gem5-dev] Change in gem5/gem5[develop]: sim: Simplify collecting the key value in initParam.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27228 ) Change subject: sim: Simplify collecting the key value in initParam. .. sim: Simplify collecting the key value in

[gem5-dev] Change in gem5/gem5[develop]: util, sparc: Use M5OP_FOREACH to define the m5op entry points for sparc.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27207 ) Change subject: util,sparc: Use M5OP_FOREACH to define the m5op entry points for sparc. .. util,sparc: Use

[gem5-dev] Change in gem5/gem5[develop]: misc: Use a const char * in the m5op m5_add_symbol.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27224 ) Change subject: misc: Use a const char * in the m5op m5_add_symbol. .. misc: Use a const char * in the m5op

[gem5-dev] Change in gem5/gem5[develop]: util: Add a semihosting implementation to the aarch64 m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27245 ) Change subject: util: Add a semihosting implementation to the aarch64 m5 utility. .. util: Add a semihosting

[gem5-dev] Change in gem5/gem5[develop]: util: Update the java wrapper names for m5 ops.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27223 ) Change subject: util: Update the java wrapper names for m5 ops. .. util: Update the java wrapper names for m5

[gem5-dev] Change in gem5/gem5[develop]: util: Remove the subfunc arg from M5OP_FOREACH.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27205 ) Change subject: util: Remove the subfunc arg from M5OP_FOREACH. .. util: Remove the subfunc arg from

[gem5-dev] Change in gem5/gem5[develop]: util, scons: Detect java and lua51 in the m5 utility scons files.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27214 ) Change subject: util,scons: Detect java and lua51 in the m5 utility scons files. .. util,scons: Detect java

[gem5-dev] Change in gem5/gem5[develop]: util, x86: Use M5OP_FOREACH to define the m5op entry points for x86.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27208 ) Change subject: util,x86: Use M5OP_FOREACH to define the m5op entry points for x86. .. util,x86: Use

[gem5-dev] Change in gem5/gem5[develop]: util: Delete the aarch64 Makefile for the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27215 ) Change subject: util: Delete the aarch64 Makefile for the m5 utility. .. util: Delete the aarch64 Makefile for

[gem5-dev] Change in gem5/gem5[develop]: util: Remove the annotation pseudo ops symbols from the m5 utility.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27203 ) Change subject: util: Remove the annotation pseudo ops symbols from the m5 utility. .. util: Remove the

[gem5-dev] Change in gem5/gem5[develop]: util: Convert the other architecture's m5 utility builds to scons.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27219 ) Change subject: util: Convert the other architecture's m5 utility builds to scons. .. util: Convert the other

[gem5-dev] Change in gem5/gem5[develop]: util: Add some additional flags for files in the m5 util.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27218 ) Change subject: util: Add some additional flags for files in the m5 util. .. util: Add some additional flags for

[gem5-dev] Change in gem5/gem5[develop]: util: Implement PIC assembly for the aarch64.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27212 ) Change subject: util: Implement PIC assembly for the aarch64. .. util: Implement PIC assembly for the aarch64.

[gem5-dev] Change in gem5/gem5[develop]: util: Correct some help text for the initparam m5 utility command.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27227 ) Change subject: util: Correct some help text for the initparam m5 utility command. .. util: Correct some help

[gem5-dev] Change in gem5/gem5[develop]: misc: Get rid of the m5op_flags.h header.

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27221 ) Change subject: misc: Get rid of the m5op_flags.h header. .. misc: Get rid of the m5op_flags.h header. This held

[gem5-dev] Change in gem5/gem5[develop]: scons: Enable LTO and partial linking with gcc >= 8.1.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27131 ) Change subject: scons: Enable LTO and partial linking with gcc >= 8.1. .. scons: Enable LTO and partial linking with gcc >=

[gem5-dev] Change in gem5/gem5[develop]: scons: Call summarize_warnings() when scons finishes building.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27130 ) Change subject: scons: Call summarize_warnings() when scons finishes building. .. scons: Call summarize_warnings() when

[gem5-dev] Change in gem5/gem5[develop]: arm: Add a callSemihosting method that figures out the width.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/25951 ) Change subject: arm: Add a callSemihosting method that figures out the width. .. arm: Add a callSemihosting method that

[gem5-dev] Change in gem5/gem5[develop]: arm: Add a gem5 specific pseudo op semihosting call.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/25950 ) Change subject: arm: Add a gem5 specific pseudo op semihosting call. .. arm: Add a gem5 specific pseudo op semihosting call.

[gem5-dev] Change in gem5/gem5[develop]: riscv: Fix RISCV builds by updating its use of pseudoInst().

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27163 ) Change subject: riscv: Fix RISCV builds by updating its use of pseudoInst(). .. riscv: Fix RISCV builds by updating its use

[gem5-dev] Change in gem5/gem5[develop]: dev-arm: Adjust idreg value in RealViewCtrl

Hello Ciro Santilli, Adrian Herrera, I'd like you to do a code review. Please visit https://gem5-review.googlesource.com/c/public/gem5/+/27184 to review the following change. Change subject: dev-arm: Adjust idreg value in RealViewCtrl

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: ARMv8.3 CompNum, SIMD complex number support

Jordi Vaquero has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27183 ) Change subject: arch-arm: ARMv8.3 CompNum, SIMD complex number support .. arch-arm: ARMv8.3 CompNum, SIMD

[gem5-dev] Change in gem5/gem5[develop]: riscv: Fix RISCV builds by updating its use of pseudoInst().

Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/27163 ) Change subject: riscv: Fix RISCV builds by updating its use of pseudoInst(). .. riscv: Fix RISCV builds by

[gem5-dev] Change in gem5/gem5[develop]: scons: Use the textwrap module to wrap warnings/errors neatly.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27128 ) Change subject: scons: Use the textwrap module to wrap warnings/errors neatly. .. scons: Use the textwrap module to wrap

[gem5-dev] Change in gem5/gem5[develop]: scons: Replace find_first_prog() with the built in Detect().

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27126 ) Change subject: scons: Replace find_first_prog() with the built in Detect(). .. scons: Replace find_first_prog() with the

[gem5-dev] Change in gem5/gem5[develop]: scons: Add a mechanism to accumulate warnings to reprint at the end.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27129 ) Change subject: scons: Add a mechanism to accumulate warnings to reprint at the end. .. scons: Add a mechanism to

[gem5-dev] Change in gem5/gem5[develop]: scons: Get rid of a redundant "Warning:" in the SConstruct.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27127 ) Change subject: scons: Get rid of a redundant "Warning:" in the SConstruct. .. scons: Get rid of a redundant "Warning:" in

[gem5-dev] Change in gem5/gem5[develop]: scons: Use the lsan-suppressions file when running internal commands.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27125 ) Change subject: scons: Use the lsan-suppressions file when running internal commands. .. scons: Use the lsan-suppressions

[gem5-dev] Change in gem5/gem5[develop]: misc: Add Gabe Black as the scons and util maintainer.

Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/27143 ) Change subject: misc: Add Gabe Black as the scons and util maintainer. .. misc: Add Gabe Black as the scons and util

[gem5-dev] Change in gem5/gem5[develop]: cpu-o3: Fix unset scoreboard in vector mode switching

Hsuan Hsu has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/26203 ) Change subject: cpu-o3: Fix unset scoreboard in vector mode switching .. cpu-o3: Fix unset scoreboard in vector mode