gem5-dev@gem5.org

The gem5 Developer List

View all threads

[S] Change in gem5/gem5[develop]: arch-arm: Add an ArmAllRelease containing every defined extension

GT
Giacomo Travaglini (Gerrit)
Thu, May 25, 2023 7:30 AM

Giacomo Travaglini has submitted this change. (
https://gem5-review.googlesource.com/c/public/gem5/+/70958?usp=email )

Change subject: arch-arm: Add an ArmAllRelease containing every defined
extension
......................................................................

arch-arm: Add an ArmAllRelease containing every defined extension

This is probably the easiest way to instantiate a release containing
any implemented extension. It is alternatively possible to use the
latest release (e.g. Armv92 as of now).
This could be preferrable for consistency across simulations.
However if users want to always be up to date with development,
using ArmAllRelease will allow them to do so without the need
to change their configuration script

Change-Id: Ibca629e99da9b571f233de9d05a5a9186d02aa99
Signed-off-by: Giacomo Travaglini giacomo.travaglini@arm.com
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/70958
Tested-by: kokoro noreply+kokoro@google.com
Reviewed-by: Richard Cooper richard.cooper@arm.com
Maintainer: Jason Lowe-Power power.jg@gmail.com

M src/arch/arm/ArmSystem.py
1 file changed, 12 insertions(+), 0 deletions(-)

Approvals:
Jason Lowe-Power: Looks good to me, approved
kokoro: Regressions pass
Richard Cooper: Looks good to me, approved

diff --git a/src/arch/arm/ArmSystem.py b/src/arch/arm/ArmSystem.py
index b826f0d..c1f5e9f 100644
--- a/src/arch/arm/ArmSystem.py
+++ b/src/arch/arm/ArmSystem.py
@@ -224,6 +224,18 @@
extensions = Armv85.extensions + ["FEAT_SME"]

+class ArmAllRelease(ArmRelease):

  • """
  • A release containing any implemented extension.  It is alternatively
  • possible to use the latest release (e.g. Armv92 as of now).  This
    could be
  • preferrable for consistency across simulations.  However if users want
    to
  • always be up to date with development, using ArmAllRelease will allow
    them
  • to do so without the need to change their configuration script
  • """
  • extensions = ArmExtension.vals
  • class ArmSystem(System):
    type = "ArmSystem"
    cxx_header = "arch/arm/system.hh"

--
To view, visit
https://gem5-review.googlesource.com/c/public/gem5/+/70958?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Ibca629e99da9b571f233de9d05a5a9186d02aa99
Gerrit-Change-Number: 70958
Gerrit-PatchSet: 2
Gerrit-Owner: Giacomo Travaglini giacomo.travaglini@arm.com
Gerrit-Reviewer: Andreas Sandberg andreas.sandberg@arm.com
Gerrit-Reviewer: Giacomo Travaglini giacomo.travaglini@arm.com
Gerrit-Reviewer: Jason Lowe-Power power.jg@gmail.com
Gerrit-Reviewer: Richard Cooper richard.cooper@arm.com
Gerrit-Reviewer: kokoro noreply+kokoro@google.com

Giacomo Travaglini has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/70958?usp=email ) Change subject: arch-arm: Add an ArmAllRelease containing every defined extension ...................................................................... arch-arm: Add an ArmAllRelease containing every defined extension This is probably the easiest way to instantiate a release containing any implemented extension. It is alternatively possible to use the latest release (e.g. Armv92 as of now). This could be preferrable for consistency across simulations. However if users want to always be up to date with development, using ArmAllRelease will allow them to do so without the need to change their configuration script Change-Id: Ibca629e99da9b571f233de9d05a5a9186d02aa99 Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com> Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/70958 Tested-by: kokoro <noreply+kokoro@google.com> Reviewed-by: Richard Cooper <richard.cooper@arm.com> Maintainer: Jason Lowe-Power <power.jg@gmail.com> --- M src/arch/arm/ArmSystem.py 1 file changed, 12 insertions(+), 0 deletions(-) Approvals: Jason Lowe-Power: Looks good to me, approved kokoro: Regressions pass Richard Cooper: Looks good to me, approved diff --git a/src/arch/arm/ArmSystem.py b/src/arch/arm/ArmSystem.py index b826f0d..c1f5e9f 100644 --- a/src/arch/arm/ArmSystem.py +++ b/src/arch/arm/ArmSystem.py @@ -224,6 +224,18 @@ extensions = Armv85.extensions + ["FEAT_SME"] +class ArmAllRelease(ArmRelease): + """ + A release containing any implemented extension. It is alternatively + possible to use the latest release (e.g. Armv92 as of now). This could be + preferrable for consistency across simulations. However if users want to + always be up to date with development, using ArmAllRelease will allow them + to do so without the need to change their configuration script + """ + + extensions = ArmExtension.vals + + class ArmSystem(System): type = "ArmSystem" cxx_header = "arch/arm/system.hh" -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/70958?usp=email To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: public/gem5 Gerrit-Branch: develop Gerrit-Change-Id: Ibca629e99da9b571f233de9d05a5a9186d02aa99 Gerrit-Change-Number: 70958 Gerrit-PatchSet: 2 Gerrit-Owner: Giacomo Travaglini <giacomo.travaglini@arm.com> Gerrit-Reviewer: Andreas Sandberg <andreas.sandberg@arm.com> Gerrit-Reviewer: Giacomo Travaglini <giacomo.travaglini@arm.com> Gerrit-Reviewer: Jason Lowe-Power <power.jg@gmail.com> Gerrit-Reviewer: Richard Cooper <richard.cooper@arm.com> Gerrit-Reviewer: kokoro <noreply+kokoro@google.com>