[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Remove m5ops_base declaration from ArmSystem

2020-07-03 Thread Hsuan Hsu (Gerrit) via gem5-dev
Hsuan Hsu has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/30914 )


Change subject: arch-arm: Remove m5ops_base declaration from ArmSystem
..

arch-arm: Remove m5ops_base declaration from ArmSystem

This declaration should have been removed but was accidentally re-added.
It keeps m5ops_base from being passed correctly from Python to C++ when
using ARM ISA, and hence triggers gem5 crash when the guest tries to
call m5ops. This change removes it again to fix the crash.

JIRA: https://gem5.atlassian.net/browse/GEM5-658

Change-Id: I8df4ff19ecc0d64255f24dc991f71b065d2a894e
Signed-off-by: Hsuan Hsu 
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/30914
Reviewed-by: Gabe Black 
Reviewed-by: Andreas Sandberg 
Maintainer: Andreas Sandberg 
Tested-by: kokoro 
---
M src/arch/arm/ArmSystem.py
1 file changed, 0 insertions(+), 4 deletions(-)

Approvals:
  Andreas Sandberg: Looks good to me, approved; Looks good to me, approved
  Gabe Black: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/arch/arm/ArmSystem.py b/src/arch/arm/ArmSystem.py
index 6555ea9..c4cc51f 100644
--- a/src/arch/arm/ArmSystem.py
+++ b/src/arch/arm/ArmSystem.py
@@ -79,10 +79,6 @@
 semihosting = Param.ArmSemihosting(NULL,
 "Enable support for the Arm semihosting by settings this  
parameter")


-m5ops_base = Param.Addr(0,
-"Base of the 64KiB PA range used for memory-mapped m5ops. Set to  
0 "

-"to disable.")
-
 # Set to true if simulation provides a PSCI implementation
 # This flag will be checked when auto-generating
 # a PSCI node. A client (e.g Linux) would then be able to

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/30914
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: I8df4ff19ecc0d64255f24dc991f71b065d2a894e
Gerrit-Change-Number: 30914
Gerrit-PatchSet: 2
Gerrit-Owner: Hsuan Hsu 
Gerrit-Reviewer: Andreas Sandberg 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-Reviewer: Hsuan Hsu 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: kokoro 
Gerrit-MessageType: merged
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: arch-arm: Remove m5ops_base declaration from ArmSystem

2020-07-02 Thread Hsuan Hsu (Gerrit) via gem5-dev
Hsuan Hsu has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/30914 )



Change subject: arch-arm: Remove m5ops_base declaration from ArmSystem
..

arch-arm: Remove m5ops_base declaration from ArmSystem

This declaration should have been removed but was accidentally re-added.
It keeps m5ops_base from being passed correctly from Python to C++ when
using ARM ISA, and hence triggers gem5 crash when the guest tries to
call m5ops. This change removes it again to fix the crash.

JIRA: https://gem5.atlassian.net/browse/GEM5-658

Change-Id: I8df4ff19ecc0d64255f24dc991f71b065d2a894e
Signed-off-by: Hsuan Hsu 
---
M src/arch/arm/ArmSystem.py
1 file changed, 0 insertions(+), 4 deletions(-)



diff --git a/src/arch/arm/ArmSystem.py b/src/arch/arm/ArmSystem.py
index 6555ea9..c4cc51f 100644
--- a/src/arch/arm/ArmSystem.py
+++ b/src/arch/arm/ArmSystem.py
@@ -79,10 +79,6 @@
 semihosting = Param.ArmSemihosting(NULL,
 "Enable support for the Arm semihosting by settings this  
parameter")


-m5ops_base = Param.Addr(0,
-"Base of the 64KiB PA range used for memory-mapped m5ops. Set to  
0 "

-"to disable.")
-
 # Set to true if simulation provides a PSCI implementation
 # This flag will be checked when auto-generating
 # a PSCI node. A client (e.g Linux) would then be able to

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/30914
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: I8df4ff19ecc0d64255f24dc991f71b065d2a894e
Gerrit-Change-Number: 30914
Gerrit-PatchSet: 1
Gerrit-Owner: Hsuan Hsu 
Gerrit-MessageType: newchange
___
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s