[gem5-dev] [M] Change in gem5/gem5[develop]: misc: Update .mailmap

2022-12-07 Thread Bobby Bruce (Gerrit) via gem5-dev 
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/66517?usp=email )


Change subject: misc: Update .mailmap
..

misc: Update .mailmap

This commit updates the mailmap since the initial commit in mid-July
2020: https://gem5-review.googlesource.com/c/public/gem5/+/29672.

`sort -u` has been run on this file so some previous entries have been
moved.

Change-Id: I46df1e9675f6f7057b680ca2abbcebdffd50462a
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/66517
Tested-by: kokoro 
Maintainer: Bobby Bruce 
Reviewed-by: Jason Lowe-Power 
---
M .mailmap
1 file changed, 179 insertions(+), 61 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved
  kokoro: Regressions pass
  Bobby Bruce: Looks good to me, approved




diff --git a/.mailmap b/.mailmap
index 5125666..49c438d 100644
--- a/.mailmap
+++ b/.mailmap
@@ -1,37 +1,43 @@
-ARM gem5 Developers 
 Abdul Mutaal Ahmad 
+adarshpatil 
+Adrià Armejach  Adrià Armejach  


 Adrian Herrera 
 Adrien Pesle 
-Adrià Armejach  Adrià Armejach  


 Akash Bagdia  Akash Bagdia 
 Alec Roelke  Alec Roelke 
+Alexander Klimov 
 Alexandru Dutu  Alexandru 
+Alex Richardson 
 Ali Jafri 
-Ali Saidi  Ali Saidi 
 Ali Saidi  Ali Saidi 
+Ali Saidi  Ali Saidi 
 Ali Saidi  Ali Saidi 
+Alistair Delva 
 Amin Farmahini 
 Anders Handler 
-Andrea Mondelli  Andrea Mondelli  

+Andrea Mondelli  Andrea Mondelli  

+Andrea Mondelli  Andrea Mondelli  


 Andrea Pellegrini 
-Andreas Hansson  Andreas Hansson  


 Andreas Hansson  Andreas Hansson 
 Andreas Hansson  Andreas Hansson  

+Andreas Hansson  Andreas Hansson  

 Andreas Hansson  Andreas Hansson  

-Andreas Sandberg  Andreas Sandberg  

 Andreas Sandberg  Andreas Sandberg  

+Andreas Sandberg  Andreas Sandberg  

 Andreas Sandberg  Andreas Sandberg  

 Andrew Bardsley  Andrew Bardsley  


 Andrew Lukefahr 
 Andrew Schultz 
 Andriani Mappoura 
-Ani Udipi 
+Angie Lee 
 Anis Peysieux 
+Ani Udipi 
 Anouk Van Laer 
-Arthur Perais 
+ARM gem5 Developers 
+Arthur Perais  Arthur Perais  


+Arun Rodrigues 
 Ashkan Tousi 
-Austin Harris 
-Richard D. Strong 
+Austin Harris  Austin Harris  


 Avishai Tvila 
 Ayaz Akram 
 Bagus Hanindhito 
@@ -41,80 +47,108 @@
 Bjoern A. Zeeb 
 Blake Hechtman  Blake Hechtman 
 Blake Hechtman  Blake Hechtman  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E) 

-Bobby R. Bruce 
+Bobby R. Bruce  Bobby Bruce  

 Boris Shingarov  Boris Shingarov  


 Brad Beckmann  Brad Beckmann 
 Brad Beckmann  Brad Beckmann  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E)  


 Brad Danofsky 
 Bradley Wang  Bradley 
+Brandon Potter  BKP 
 Brandon Potter  bpotter 
 Brandon Potter  Brandon Potter  


-Brandon Potter  BKP 
 Brian Grayson 
 Cagdas Dirik  cdirik 
+Carlos Falquez 
 Chander Sudanthi  Chander Sudanthi  

 Chander Sudanthi  Chander Sudanthi  


+Charles Jamieson 
+CHEN Meng 
 Chen Zou 
+Chia-You Chen 
+Chow, Marcus 
 Chris Adeniyi-Jones 
-Chris Emmons  Chris Emmons 
 Chris Emmons  Chris Emmons 
+Chris Emmons  Chris Emmons 
+Chris January 
 Christian Menard  Christian Menard  


-Christoph Pfister 
 Christopher Torng 
+Christoph Pfister 
 Chuan Zhu 
 Chun-Chen Hsu  Chun-Chen TK Hsu  


 Ciro Santilli 
 Clint Smullen 
+Cui Jin  Cui Jin 
 Curtis Dunham 
+Daecheol You 
 Dam Sunwoo 
 Dan Gibson 
 Daniel Carvalho  Daniel 
 Daniel Carvalho  Daniel R. Carvalho  


+Daniel Gerzhoy 
 Daniel Johnson 
 Daniel Sanchez 
+Davide Basilio Bartolini 
 David Guillen-Fandos  David Guillen  

 David Guillen-Fandos  David Guillen Fandos  


 David Hashe  David Hashe 
 David Oehmke 
+David Schall 
+Derek Christ 
 Derek Hower 
-Deyaun Guo  Deyuan Guo  

 Deyaun Guo  Deyuan Guo  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E)  

+Deyaun Guo  Deyuan Guo  

 Dibakar Gope  Dibakar Gope  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E) 

+Dimitrios Chasapis 
 Djordje Kovacevic  Djordje Kovacevic  


-Dongxue Zhang 
 Doğukan Korkmaztürk 
+Dongxue Zhang 
 Dylan Johnson 
 Earl Ou 
+eavivi 
+Éder F. Zulian 
 Edmund Grimley Evans 
+Eduardo José Gómez Hernández 
+Eliot Moss 
 Emilio Castillo  Emilio Castillo 
 Emilio Castillo  Emilio Castillo  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E) 

+Emily Brickey 
 Erfan Azarkhish 
+Erhu 
 Eric Van Hensbergen  Eric Van Hensbergen  


+Eric Ye 
 Erik Hallnor 
 Erik Tomusk 
 Faissal Sleiman  Faissal Sleiman  


 Fernando Endo 
+Franklin He 
 Gabe Black  Gabe Black 
 Gabe Black  Gabe Black 
+Gabe Loh  gloh 
 Gabor Dozsa 
+Gabriel Busnot 
+gauravjain14 
 Gedare Bloom  Gedare Bloom 
 Gene Wu  Gene WU 
 Gene WU  Gene Wu 
-Geoffrey Blake  Geoffrey Blake  


 Geoffrey Blake  Geoffrey Blake 
+Geoffrey Blake  Geoffrey Blake  


 Georg Kotheimer 
 Giacomo Gabrielli  Giacomo Gabrielli  


 Giacomo Travaglini 
 Glenn Bergmans 
+GWDx 
 Hamid Reza Khaleghzadeh  Hamid Reza Khaleghzadeh

[gem5-dev] [M] Change in gem5/gem5[release-staging-v22-1]: misc: Update .mailmap

2022-12-07 Thread Bobby Bruce (Gerrit) via gem5-dev 
Bobby Bruce has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/66531?usp=email )



Change subject: misc: Update .mailmap
..

misc: Update .mailmap

This commit updates the mailmap since the initial commit in mid-July
2020: https://gem5-review.googlesource.com/c/public/gem5/+/29672.

`sort -u` has been run on this file so some previous entries have been
moved.

Change-Id: I46df1e9675f6f7057b680ca2abbcebdffd50462a
---
M .mailmap
1 file changed, 175 insertions(+), 61 deletions(-)



diff --git a/.mailmap b/.mailmap
index 5125666..49c438d 100644
--- a/.mailmap
+++ b/.mailmap
@@ -1,37 +1,43 @@
-ARM gem5 Developers 
 Abdul Mutaal Ahmad 
+adarshpatil 
+Adrià Armejach  Adrià Armejach  


 Adrian Herrera 
 Adrien Pesle 
-Adrià Armejach  Adrià Armejach  


 Akash Bagdia  Akash Bagdia 
 Alec Roelke  Alec Roelke 
+Alexander Klimov 
 Alexandru Dutu  Alexandru 
+Alex Richardson 
 Ali Jafri 
-Ali Saidi  Ali Saidi 
 Ali Saidi  Ali Saidi 
+Ali Saidi  Ali Saidi 
 Ali Saidi  Ali Saidi 
+Alistair Delva 
 Amin Farmahini 
 Anders Handler 
-Andrea Mondelli  Andrea Mondelli  

+Andrea Mondelli  Andrea Mondelli  

+Andrea Mondelli  Andrea Mondelli  


 Andrea Pellegrini 
-Andreas Hansson  Andreas Hansson  


 Andreas Hansson  Andreas Hansson 
 Andreas Hansson  Andreas Hansson  

+Andreas Hansson  Andreas Hansson  

 Andreas Hansson  Andreas Hansson  

-Andreas Sandberg  Andreas Sandberg  

 Andreas Sandberg  Andreas Sandberg  

+Andreas Sandberg  Andreas Sandberg  

 Andreas Sandberg  Andreas Sandberg  

 Andrew Bardsley  Andrew Bardsley  


 Andrew Lukefahr 
 Andrew Schultz 
 Andriani Mappoura 
-Ani Udipi 
+Angie Lee 
 Anis Peysieux 
+Ani Udipi 
 Anouk Van Laer 
-Arthur Perais 
+ARM gem5 Developers 
+Arthur Perais  Arthur Perais  


+Arun Rodrigues 
 Ashkan Tousi 
-Austin Harris 
-Richard D. Strong 
+Austin Harris  Austin Harris  


 Avishai Tvila 
 Ayaz Akram 
 Bagus Hanindhito 
@@ -41,80 +47,108 @@
 Bjoern A. Zeeb 
 Blake Hechtman  Blake Hechtman 
 Blake Hechtman  Blake Hechtman  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E) 

-Bobby R. Bruce 
+Bobby R. Bruce  Bobby Bruce  

 Boris Shingarov  Boris Shingarov  


 Brad Beckmann  Brad Beckmann 
 Brad Beckmann  Brad Beckmann  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E)  


 Brad Danofsky 
 Bradley Wang  Bradley 
+Brandon Potter  BKP 
 Brandon Potter  bpotter 
 Brandon Potter  Brandon Potter  


-Brandon Potter  BKP 
 Brian Grayson 
 Cagdas Dirik  cdirik 
+Carlos Falquez 
 Chander Sudanthi  Chander Sudanthi  

 Chander Sudanthi  Chander Sudanthi  


+Charles Jamieson 
+CHEN Meng 
 Chen Zou 
+Chia-You Chen 
+Chow, Marcus 
 Chris Adeniyi-Jones 
-Chris Emmons  Chris Emmons 
 Chris Emmons  Chris Emmons 
+Chris Emmons  Chris Emmons 
+Chris January 
 Christian Menard  Christian Menard  


-Christoph Pfister 
 Christopher Torng 
+Christoph Pfister 
 Chuan Zhu 
 Chun-Chen Hsu  Chun-Chen TK Hsu  


 Ciro Santilli 
 Clint Smullen 
+Cui Jin  Cui Jin 
 Curtis Dunham 
+Daecheol You 
 Dam Sunwoo 
 Dan Gibson 
 Daniel Carvalho  Daniel 
 Daniel Carvalho  Daniel R. Carvalho  


+Daniel Gerzhoy 
 Daniel Johnson 
 Daniel Sanchez 
+Davide Basilio Bartolini 
 David Guillen-Fandos  David Guillen  

 David Guillen-Fandos  David Guillen Fandos  


 David Hashe  David Hashe 
 David Oehmke 
+David Schall 
+Derek Christ 
 Derek Hower 
-Deyaun Guo  Deyuan Guo  

 Deyaun Guo  Deyuan Guo  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E)  

+Deyaun Guo  Deyuan Guo  

 Dibakar Gope  Dibakar Gope  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E) 

+Dimitrios Chasapis 
 Djordje Kovacevic  Djordje Kovacevic  


-Dongxue Zhang 
 Doğukan Korkmaztürk 
+Dongxue Zhang 
 Dylan Johnson 
 Earl Ou 
+eavivi 
+Éder F. Zulian 
 Edmund Grimley Evans 
+Eduardo José Gómez Hernández 
+Eliot Moss 
 Emilio Castillo  Emilio Castillo 
 Emilio Castillo  Emilio Castillo  
ext:(%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E) 

+Emily Brickey 
 Erfan Azarkhish 
+Erhu 
 Eric Van Hensbergen  Eric Van Hensbergen  


+Eric Ye 
 Erik Hallnor 
 Erik Tomusk 
 Faissal Sleiman  Faissal Sleiman  


 Fernando Endo 
+Franklin He 
 Gabe Black  Gabe Black 
 Gabe Black  Gabe Black 
+Gabe Loh  gloh 
 Gabor Dozsa 
+Gabriel Busnot 
+gauravjain14 
 Gedare Bloom  Gedare Bloom 
 Gene Wu  Gene WU 
 Gene WU  Gene Wu 
-Geoffrey Blake  Geoffrey Blake  


 Geoffrey Blake  Geoffrey Blake 
+Geoffrey Blake  Geoffrey Blake  


 Georg Kotheimer 
 Giacomo Gabrielli  Giacomo Gabrielli  


 Giacomo Travaglini 
 Glenn Bergmans 
+GWDx 
 Hamid Reza Khaleghzadeh  Hamid Reza Khaleghzadeh  
ext:(%2C%20Lluc%20Alvarez%20%3Clluc.alvarez%40bsc.es%3E%2C%20Nilay%20Vaish%20%3Cnilay%40cs.wisc.edu%3E)  


+handsomeliu 
 Hanhwi Jang 
 Hoa Nguyen 
 Hongil Yoon 
 Hsuan Hsu 
+huangjs 
 Hussein Elnawawy 
 Ian Jiang 
 IanJiangICT 
 Ilias Vougioukas 
+Iru Cai 
 Isaac Richter 
 Isaac Sánchez Barrera 
 Ivan Pizarro 
@@ -123,104 +157,152 @@
 Jakub Jermar

[gem5-dev] [S] Change in gem5/gem5[develop]: configs: Alter x86-npb-benchmarks.py to exit after WORKEND

2022-12-07 Thread Bobby Bruce (Gerrit) via gem5-dev 
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/66513?usp=email )


Change subject: configs: Alter x86-npb-benchmarks.py to exit after WORKEND
..

configs: Alter x86-npb-benchmarks.py to exit after WORKEND

While the config script will still function without exiting the SimLoop
after the "WORKEND" exit event, there's no need for the simulation to
continue beyond this point.

Change-Id: I60691215e9516fa1eeb8b8502f2bc5a09de2969b
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/66513
Tested-by: kokoro 
Reviewed-by: Jason Lowe-Power 
Maintainer: Bobby Bruce 
---
M configs/example/gem5_library/x86-npb-benchmarks.py
1 file changed, 18 insertions(+), 2 deletions(-)

Approvals:
  Bobby Bruce: Looks good to me, approved
  kokoro: Regressions pass
  Jason Lowe-Power: Looks good to me, approved




diff --git a/configs/example/gem5_library/x86-npb-benchmarks.py  
b/configs/example/gem5_library/x86-npb-benchmarks.py

index 2cb3143..ff363e4 100644
--- a/configs/example/gem5_library/x86-npb-benchmarks.py
+++ b/configs/example/gem5_library/x86-npb-benchmarks.py
@@ -237,12 +237,11 @@
 # marked by `workend`.

 # We exepect that ROI ends with `workend` or `simulate() limit reached`.
-# Otherwise the simulation ended unexpectedly.
 def handle_workend():
 print("Dump stats at the end of the ROI!")

 m5.stats.dump()
-yield False
+yield True


 simulator = Simulator(

--
To view, visit  
https://gem5-review.googlesource.com/c/public/gem5/+/66513?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: I60691215e9516fa1eeb8b8502f2bc5a09de2969b
Gerrit-Change-Number: 66513
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Bobby Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
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

[gem5-dev] [S] Change in gem5/gem5[develop]: configs: Fix x86-gapbs-benchmarks.py example

2022-12-07 Thread Bobby Bruce (Gerrit) via gem5-dev 
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/66512?usp=email )


Change subject: configs: Fix x86-gapbs-benchmarks.py example
..

configs: Fix x86-gapbs-benchmarks.py example

With https://gem5-review.googlesource.com/c/public/gem5/+/64791 we
updated the configs/example/gem5_library to utilize the `m5.simulate`
module. The GAPBS benchmark example uses the "WORKBEGIN" and "WORKEND"
exit events to specify the ROI. The patch incorrectly assumed an "EXIT"
exit event were used.

As such, the
"test-gem5-library-example-x86-gapbs-benchmarks-ALL-x86_64-opt-MESI_Two_Level"
test was not properly running, causing the Nightly test to fail:
https://jenkins.gem5.org/job/nightly/444. This patch fixes this error.

Change-Id: I207fe3563c8d9c59bcb79428fe62d2d2bbccd013
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/66512
Maintainer: Bobby Bruce 
Tested-by: kokoro 
Reviewed-by: Jason Lowe-Power 
---
M configs/example/gem5_library/x86-gapbs-benchmarks.py
1 file changed, 30 insertions(+), 2 deletions(-)

Approvals:
  kokoro: Regressions pass
  Bobby Bruce: Looks good to me, approved
  Jason Lowe-Power: Looks good to me, approved




diff --git a/configs/example/gem5_library/x86-gapbs-benchmarks.py  
b/configs/example/gem5_library/x86-gapbs-benchmarks.py

index 638d34b..6ab3747 100644
--- a/configs/example/gem5_library/x86-gapbs-benchmarks.py
+++ b/configs/example/gem5_library/x86-gapbs-benchmarks.py
@@ -211,7 +211,7 @@
 )


-def handle_exit():
+def handle_workbegin():
 print("Done booting Linux")
 print("Resetting stats at the start of ROI!")
 m5.stats.reset()
@@ -219,6 +219,9 @@
 start_tick = m5.curTick()
 processor.switch()
 yield False  # E.g., continue the simulation.
+
+
+def handle_workend():
 print("Dump stats at the end of the ROI!")
 m5.stats.dump()
 yield True  # Stop the simulation. We're done.
@@ -227,7 +230,8 @@
 simulator = Simulator(
 board=board,
 on_exit_event={
-ExitEvent.EXIT: handle_exit(),
+ExitEvent.WORKBEGIN: handle_workbegin(),
+ExitEvent.WORKEND: handle_workend(),
 },
 )


--
To view, visit  
https://gem5-review.googlesource.com/c/public/gem5/+/66512?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: I207fe3563c8d9c59bcb79428fe62d2d2bbccd013
Gerrit-Change-Number: 66512
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Bobby Bruce 
Gerrit-Reviewer: Jason Lowe-Power 
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

[gem5-dev] [S] Change in gem5/gem5[release-staging-v22-1]: configs: Fix x86-gapbs-benchmarks.py example

2022-12-07 Thread Bobby Bruce (Gerrit) via gem5-dev 
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/66514?usp=email )


Change subject: configs: Fix x86-gapbs-benchmarks.py example
..

configs: Fix x86-gapbs-benchmarks.py example

With https://gem5-review.googlesource.com/c/public/gem5/+/64791 we
updated the configs/example/gem5_library to utilize the `m5.simulate`
module. The GAPBS benchmark example uses the "WORKBEGIN" and "WORKEND"
exit events to specify the ROI. The patch incorrectly assumed an "EXIT"
exit event were used.

As such, the
"test-gem5-library-example-x86-gapbs-benchmarks-ALL-x86_64-opt-MESI_Two_Level"
test was not properly running, causing the Nightly test to fail:
https://jenkins.gem5.org/job/nightly/444. This patch fixes this error.

Change-Id: I207fe3563c8d9c59bcb79428fe62d2d2bbccd013
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/66514
Tested-by: kokoro 
Reviewed-by: Bobby Bruce 
Maintainer: Bobby Bruce 
---
M configs/example/gem5_library/x86-gapbs-benchmarks.py
1 file changed, 30 insertions(+), 2 deletions(-)

Approvals:
  Bobby Bruce: Looks good to me, approved; Looks good to me, approved
  kokoro: Regressions pass




diff --git a/configs/example/gem5_library/x86-gapbs-benchmarks.py  
b/configs/example/gem5_library/x86-gapbs-benchmarks.py

index 638d34b..6ab3747 100644
--- a/configs/example/gem5_library/x86-gapbs-benchmarks.py
+++ b/configs/example/gem5_library/x86-gapbs-benchmarks.py
@@ -211,7 +211,7 @@
 )


-def handle_exit():
+def handle_workbegin():
 print("Done booting Linux")
 print("Resetting stats at the start of ROI!")
 m5.stats.reset()
@@ -219,6 +219,9 @@
 start_tick = m5.curTick()
 processor.switch()
 yield False  # E.g., continue the simulation.
+
+
+def handle_workend():
 print("Dump stats at the end of the ROI!")
 m5.stats.dump()
 yield True  # Stop the simulation. We're done.
@@ -227,7 +230,8 @@
 simulator = Simulator(
 board=board,
 on_exit_event={
-ExitEvent.EXIT: handle_exit(),
+ExitEvent.WORKBEGIN: handle_workbegin(),
+ExitEvent.WORKEND: handle_workend(),
 },
 )


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


Gerrit-Project: public/gem5
Gerrit-Branch: release-staging-v22-1
Gerrit-Change-Id: I207fe3563c8d9c59bcb79428fe62d2d2bbccd013
Gerrit-Change-Number: 66514
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Bobby Bruce 
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

[gem5-dev] [S] Change in gem5/gem5[release-staging-v22-1]: configs: Alter x86-npb-benchmarks.py to exit after WORKEND

2022-12-07 Thread Bobby Bruce (Gerrit) via gem5-dev 
Bobby Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/66515?usp=email )


Change subject: configs: Alter x86-npb-benchmarks.py to exit after WORKEND
..

configs: Alter x86-npb-benchmarks.py to exit after WORKEND

While the config script will still function without exiting the SimLoop
after the "WORKEND" exit event, there's no need for the simulation to
continue beyond this point.

Change-Id: I60691215e9516fa1eeb8b8502f2bc5a09de2969b
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/66515
Maintainer: Bobby Bruce 
Reviewed-by: Bobby Bruce 
Tested-by: kokoro 
---
M configs/example/gem5_library/x86-npb-benchmarks.py
1 file changed, 18 insertions(+), 2 deletions(-)

Approvals:
  kokoro: Regressions pass
  Bobby Bruce: Looks good to me, approved; Looks good to me, approved




diff --git a/configs/example/gem5_library/x86-npb-benchmarks.py  
b/configs/example/gem5_library/x86-npb-benchmarks.py

index 2cb3143..ff363e4 100644
--- a/configs/example/gem5_library/x86-npb-benchmarks.py
+++ b/configs/example/gem5_library/x86-npb-benchmarks.py
@@ -237,12 +237,11 @@
 # marked by `workend`.

 # We exepect that ROI ends with `workend` or `simulate() limit reached`.
-# Otherwise the simulation ended unexpectedly.
 def handle_workend():
 print("Dump stats at the end of the ROI!")

 m5.stats.dump()
-yield False
+yield True


 simulator = Simulator(

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


Gerrit-Project: public/gem5
Gerrit-Branch: release-staging-v22-1
Gerrit-Change-Id: I60691215e9516fa1eeb8b8502f2bc5a09de2969b
Gerrit-Change-Number: 66515
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby Bruce 
Gerrit-Reviewer: Bobby Bruce 
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

[gem5-dev] Build failed in Jenkins: nightly #449

2022-12-07 Thread jenkins-no-reply--- via gem5-dev 
See 

Changes:

[yuhsingw] systemc: fix the payload and packet association in Gem5ToTlm bridge

[gabe.black] dev: Add an offset checking mechanism to RegisterBank.

[Bobby R. Bruce] python: Remove 'scheduleTickExit' in favor of 'exitSimLoop'

[yuhsingw] systemc: replace the deprecated std::iterator


--
[...truncated 1.96 MB...]
 [LINK]  -> RISCV/gem5.opt
scons: done building targets.
Downloading http://dist.gem5.org/dist/develop/arm/aarch-system-20220707.tar.bz2 
to /nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5out_ytf11zk -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview-simple-atomic.py
 /nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5outqshddxr8 -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview64-o3.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5outnamt8jod -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview-switcheroo-timing.py
 /nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5outzesbycyc -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview-simple-timing.py
 /nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5outk3t4yevv -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview64-minor-dual.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Starting Test Suite: realview-simple-atomic-ALL-x86_64-opt 
Starting Test Case: realview-simple-atomic-ALL-x86_64-opt
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5outeo8b8qv2 -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview-minor.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5out3_85hi67 -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview64-o3-dual.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5out7addmrou -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview-switcheroo-o3.py
 /nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Starting Test Suite: realview64-o3-ALL-x86_64-opt 
Starting Test Case: realview64-o3-ALL-x86_64-opt
Starting Test Suite: realview-switcheroo-timing-ALL-x86_64-opt 
Starting Test Case: realview-switcheroo-timing-ALL-x86_64-opt
Starting Test Suite: realview-simple-timing-ALL-x86_64-opt 
Starting Test Case: realview-simple-timing-ALL-x86_64-opt
Starting Test Suite: realview64-minor-dual-ALL-x86_64-opt 
Starting Test Case: realview64-minor-dual-ALL-x86_64-opt
Starting Test Suite: realview-minor-ALL-x86_64-opt 
Starting Test Case: realview-minor-ALL-x86_64-opt
Starting Test Suite: realview64-o3-dual-ALL-x86_64-opt 
Starting Test Case: realview64-o3-dual-ALL-x86_64-opt
Starting Test Suite: realview-switcheroo-o3-ALL-x86_64-opt 
Starting Test Case: realview-switcheroo-o3-ALL-x86_64-opt
Test: realview-switcheroo-timing-ALL-x86_64-opt Passed
Logging call to command: /nobackup/jenkins/workspace/nightly/build/ALL/gem5.opt 
-d /tmp/gem5outew1ykc7d -re --silent-redirect 
/nobackup/jenkins/workspace/nightly/tests/gem5/fs/linux/arm/run.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/configs/realview-o3.py 
/nobackup/jenkins/workspace/nightly/tests/gem5/resources/arm 
/nobackup/jenkins/workspace/nightly
Starting Test Suite:

[gem5-dev] Build failed in Jenkins: compiler-checks #446

2022-12-07 Thread jenkins-no-reply--- via gem5-dev 
See 


Changes:

[yuhsingw] systemc: fix the payload and packet association in Gem5ToTlm bridge

[gabe.black] dev: Add an offset checking mechanism to RegisterBank.

[Bobby R. Bruce] python: Remove 'scheduleTickExit' in favor of 'exitSimLoop'

[yuhsingw] systemc: replace the deprecated std::iterator


--
[...truncated 1.70 KB...]
  * Building target 'NULL.opt' with 'gcc-version-12'...
Done.
  * Building target 'NULL.fast' with 'gcc-version-12'...
Done.
  * Building target 'ALL.opt' with 'gcc-version-12'...
Done.
  * Building target 'ALL.fast' with 'gcc-version-12'...
Done.
  * Building target 'ARM_MESI_Three_Level.opt' with 'gcc-version-12'...
Done.
  * Building target 'ARM_MESI_Three_Level.fast' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MOESI_CMP_directory.opt' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MOESI_CMP_directory.fast' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MESI_Two_Level.opt' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MESI_Two_Level.fast' with 'gcc-version-12'...
Done.
  * Building target 'X86.opt' with 'gcc-version-12'...
Done.
  * Building target 'X86.fast' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MOESI_CMP_token.opt' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MOESI_CMP_token.fast' with 'gcc-version-12'...
Done.
  * Building target 'ARM_MESI_Three_Level_HTM.opt' with 'gcc-version-12'...
Done.
  * Building target 'ARM_MESI_Three_Level_HTM.fast' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MOESI_hammer.opt' with 'gcc-version-12'...
Done.
  * Building target 'NULL_MOESI_hammer.fast' with 'gcc-version-12'...
Done.
  * Building target 'RISCV.opt' with 'gcc-version-12'...
Done.
  * Building target 'RISCV.fast' with 'gcc-version-12'...
Done.
  * Building target 'X86_MOESI_AMD_Base.opt' with 'gcc-version-12'...
Done.
  * Building target 'X86_MOESI_AMD_Base.fast' with 'gcc-version-12'...
Done.
  * Building target 'ARM.opt' with 'gcc-version-12'...
Done.
  * Building target 'ARM.fast' with 'gcc-version-12'...
Done.
  * Building target 'MIPS.opt' with 'gcc-version-12'...
Done.
  * Building target 'MIPS.fast' with 'gcc-version-12'...
Done.
  * Building target 'GCN3_X86.opt' with 'gcc-version-12'...
Done.
  * Building target 'GCN3_X86.fast' with 'gcc-version-12'...
Done.
  * Building target 'X86_MI_example.opt' with 'gcc-version-12'...
Done.
  * Building target 'X86_MI_example.fast' with 'gcc-version-12'...
Done.
  * Building target 'SPARC.opt' with 'gcc-version-12'...
Done.
  * Building target 'SPARC.fast' with 'gcc-version-12'...
Done.
  * Building target 'POWER.opt' with 'gcc-version-12'...
Done.
  * Building target 'POWER.fast' with 'gcc-version-12'...
Done.
  * Building target 'Garnet_standalone.opt' with 'gcc-version-12'...
Done.
  * Building target 'Garnet_standalone.fast' with 'gcc-version-12'...
Done.
Starting build tests with 'gcc-version-11'...
  * Building target 'ARM_MOESI_hammer.opt' with 'gcc-version-11'...
Done.
  * Building target 'ARM_MOESI_hammer.fast' with 'gcc-version-11'...
Done.
Starting build tests with 'gcc-version-10'...
  * Building target 'NULL_MESI_Two_Level.opt' with 'gcc-version-10'...
Done.
  * Building target 'NULL_MESI_Two_Level.fast' with 'gcc-version-10'...
Done.
Starting build tests with 'gcc-version-9'...
  * Building target 'NULL.opt' with 'gcc-version-9'...
Done.
  * Building target 'NULL.fast' with 'gcc-version-9'...
Done.
Starting build tests with 'gcc-version-8'...
  * Building target 'NULL_MOESI_CMP_token.opt' with 'gcc-version-8'...
Done.
  * Building target 'NULL_MOESI_CMP_token.fast' with 'gcc-version-8'...
Done.
Starting build tests with 'gcc-version-7'...
  * Building target 'NULL_MOESI_hammer.opt' with 'gcc-version-7'...
Done.
  * Building target 'NULL_MOESI_hammer.fast' with 'gcc-version-7'...
Done.
Starting build tests with 'clang-version-14'...
'clang-version-14' was found in the comprehensive tests. All ISAs will be built.
  * Building target 'ARM_MOESI_hammer.opt' with 'clang-version-14'...
Done.
  * Building target 'ARM_MOESI_hammer.fast' with 'clang-version-14'...
Done.
  * Building target 'ARM_MESI_Three_Level.opt' with 'clang-version-14'...
Done.
  * Building target 'ARM_MESI_Three_Level.fast' with 'clang-version-14'...
Done.
  * Building target 'X86_MI_example.opt' with 'clang-version-14'...
Done.
  * Building target 'X86_MI_example.fast' with 'clang-version-14'...
Done.
  * Building target 'NULL_MOESI_CMP_token.opt' with 'clang-version-14'...
Done.
  * Building target 'NULL_MOESI_CMP_token.fast' with 'clang-version-14'...
Done.
  * Building target 'GCN3_X86.opt' with 'clang-version-14'...
Done.
  * Building

[gem5-dev] [S] Change in gem5/gem5[develop]: systemc: Enable DMI in the non-blocking/timing mode bridge.

2022-12-07 Thread Gabe Black (Gerrit) via gem5-dev 
Gabe Black has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/65754?usp=email )


Change subject: systemc: Enable DMI in the non-blocking/timing mode bridge.
..

systemc: Enable DMI in the non-blocking/timing mode bridge.

Change-Id: Ia618081e2dbf8b49f62480ac5dc29f87100cd4f1
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/65754
Maintainer: Gabe Black 
Tested-by: kokoro 
Reviewed-by: Gabe Black 
---
M src/systemc/tlm_bridge/tlm_to_gem5.cc
1 file changed, 38 insertions(+), 2 deletions(-)

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




diff --git a/src/systemc/tlm_bridge/tlm_to_gem5.cc  
b/src/systemc/tlm_bridge/tlm_to_gem5.cc

index 47a2fba..c02efe7 100644
--- a/src/systemc/tlm_bridge/tlm_to_gem5.cc
+++ b/src/systemc/tlm_bridge/tlm_to_gem5.cc
@@ -207,6 +207,29 @@
 TlmToGem5Bridge::sendBeginResp(tlm::tlm_generic_payload ,
  sc_core::sc_time )
 {
+MemBackdoor::Flags flags;
+switch (trans.get_command()) {
+  case tlm::TLM_READ_COMMAND:
+flags = MemBackdoor::Readable;
+break;
+  case tlm::TLM_WRITE_COMMAND:
+flags = MemBackdoor::Writeable;
+break;
+  default:
+panic("TlmToGem5Bridge: "
+"received transaction with unsupported command");
+}
+Addr start_addr = trans.get_address();
+Addr length = trans.get_data_length();
+
+MemBackdoorReq req({start_addr, start_addr + length}, flags);
+MemBackdoorPtr backdoor = nullptr;
+
+bmp.sendMemBackdoorReq(req, backdoor);
+
+if (backdoor)
+trans.set_dmi_allowed(true);
+
 tlm::tlm_phase phase = tlm::BEGIN_RESP;

 auto status = socket->nb_transport_bw(trans, phase, delay);
@@ -574,12 +597,12 @@
 DPRINTF(TlmBridge, "register blocking interface");
 socket.register_b_transport(
 this, ::b_transport);
-socket.register_get_direct_mem_ptr(
-this, ::get_direct_mem_ptr);
 } else {
 panic("gem5 operates neither in Timing nor in Atomic mode");
 }

+socket.register_get_direct_mem_ptr(
+this, ::get_direct_mem_ptr);
 socket.register_transport_dbg(
 this, ::transport_dbg);


--
To view, visit  
https://gem5-review.googlesource.com/c/public/gem5/+/65754?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: Ia618081e2dbf8b49f62480ac5dc29f87100cd4f1
Gerrit-Change-Number: 65754
Gerrit-PatchSet: 4
Gerrit-Owner: Gabe Black 
Gerrit-Reviewer: Earl Ou 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Gabe Black 
Gerrit-Reviewer: Jui-min Lee 
Gerrit-Reviewer: Yu-hsin Wang 
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