gem5-dev@gem5.org

The gem5 Developer List

View all threads

[XS] Change in gem5/gem5[develop]: configs: Make the configuration of the gicv4 parameter robust

RC
Richard Cooper (Gerrit)
Wed, Apr 12, 2023 7:41 AM

Richard Cooper has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/69685?usp=email )

Change subject: configs: Make the configuration of the gicv4 parameter
robust
......................................................................

configs: Make the configuration of the gicv4 parameter robust

Only the GICv3 model has a gicv4 parameter, causing the current
baremetal.py config to throw an exception when used with the
VExpress_GEM5_V1 platform containing a GICv2.

This patch checks for the existence of the gicv4 parameter, allowing
all VExpress platforms to be used.

Change-Id: I72667a9caee64fa497bda516217cd424050eb242

M configs/example/arm/baremetal.py
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/configs/example/arm/baremetal.py
b/configs/example/arm/baremetal.py
index 4af1ff1..345596d 100644
--- a/configs/example/arm/baremetal.py
+++ b/configs/example/arm/baremetal.py
@@ -142,7 +142,8 @@
system.auto_reset_addr = True

  # Using GICv3
  • system.realview.gic.gicv4 = False
  • if hasattr(system.realview.gic, "gicv4"):
  •    system.realview.gic.gicv4 = False
    
    system.highest_el_is_64 = True
    

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

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I72667a9caee64fa497bda516217cd424050eb242
Gerrit-Change-Number: 69685
Gerrit-PatchSet: 1
Gerrit-Owner: Richard Cooper richard.cooper@arm.com
Gerrit-MessageType: newchange

Richard Cooper has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/69685?usp=email ) Change subject: configs: Make the configuration of the gicv4 parameter robust ...................................................................... configs: Make the configuration of the gicv4 parameter robust Only the GICv3 model has a `gicv4` parameter, causing the current `baremetal.py` config to throw an exception when used with the VExpress_GEM5_V1 platform containing a GICv2. This patch checks for the existence of the `gicv4` parameter, allowing all VExpress platforms to be used. Change-Id: I72667a9caee64fa497bda516217cd424050eb242 --- M configs/example/arm/baremetal.py 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configs/example/arm/baremetal.py b/configs/example/arm/baremetal.py index 4af1ff1..345596d 100644 --- a/configs/example/arm/baremetal.py +++ b/configs/example/arm/baremetal.py @@ -142,7 +142,8 @@ system.auto_reset_addr = True # Using GICv3 - system.realview.gic.gicv4 = False + if hasattr(system.realview.gic, "gicv4"): + system.realview.gic.gicv4 = False system.highest_el_is_64 = True -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/69685?usp=email To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings Gerrit-Project: public/gem5 Gerrit-Branch: develop Gerrit-Change-Id: I72667a9caee64fa497bda516217cd424050eb242 Gerrit-Change-Number: 69685 Gerrit-PatchSet: 1 Gerrit-Owner: Richard Cooper <richard.cooper@arm.com> Gerrit-MessageType: newchange