[gem5-dev] Jenkins build is back to normal : compiler-checks #198

2021-09-23 Thread jenkins-no-reply--- via gem5-dev 
See 

___
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]: misc: Update gem5's README file.

2021-09-23 Thread Gabe Black (Gerrit) via gem5-dev 
Gabe Black has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50755 )


Change subject: misc: Update gem5's README file.
..

misc: Update gem5's README file.

Remove references to alpha and SWIG. Change the list of configs to be a
list of configs and not ISAs even though they are frequently named after
the ISA, say that the options are in build_opts, add in missing top
level directories, update and generalize the description of full system,
re-wrap some text.

Change-Id: Id2fb9323dcb9d8bf8e86c78fb5e9aa6afed1c5b8
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/50755
Reviewed-by: Gabe Black 
Reviewed-by: Jason Lowe-Power 
Maintainer: Gabe Black 
Tested-by: kokoro 
---
M README
1 file changed, 19 insertions(+), 13 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved
  Gabe Black: Looks good to me, but someone else must approve; Looks good  
to me, approved

  kokoro: Regressions pass




diff --git a/README b/README
index e24e7a3..5803372 100644
--- a/README
+++ b/README
@@ -8,29 +8,35 @@
 http://www.gem5.org/documentation/learning_gem5/introduction.

 To build gem5, you will need the following software: g++ or clang,
-Python (gem5 links in the Python interpreter), SCons, SWIG, zlib, m4,
-and lastly protobuf if you want trace capture and playback
-support. Please see http://www.gem5.org/documentation/general_docs/building
-for more details concerning the minimum versions of the aforementioned  
tools.

+Python (gem5 links in the Python interpreter), SCons, zlib, m4, and lastly
+protobuf if you want trace capture and playback support. Please see
+http://www.gem5.org/documentation/general_docs/building for more details
+concerning the minimum versions of these tools.

 Once you have all dependencies resolved, type 'scons
-build//gem5.opt' where ARCH is one of ARM, NULL, MIPS, POWER, SPARC,
-or X86. This will build an optimized version of the gem5 binary (gem5.opt)
-for the the specified architecture. See
-http://www.gem5.org/documentation/general_docs/building for more details  
and

-options.
+build//gem5.opt' where CONFIG is one of the options in build_opts  
like
+ARM, NULL, MIPS, POWER, SPARC, X86, Garnet_standalone, etc. This will  
build an

+optimized version of the gem5 binary (gem5.opt) with the the specified
+configuration. See http://www.gem5.org/documentation/general_docs/building  
for

+more details and options.

-The basic source release includes these subdirectories:
+The main source tree includes these subdirectories:
+   - build_opts: pre-made default configurations for gem5
+   - build_tools: tools used internally by gem5's build process.
- configs: example simulation configuration scripts
- ext: less-common external packages needed to build gem5
+   - include: include files for use in other programs
+   - site_scons: modular components of the build system
- src: source code of the gem5 simulator
- system: source for some optional system software for simulated systems
- tests: regression tests
- util: useful utility programs and files

-To run full-system simulations, you will need compiled system firmware
-(console and PALcode for Alpha), kernel binaries and one or more disk
-images.
+To run full-system simulations, you may need compiled system firmware,  
kernel

+binaries and one or more disk images, depending on gem5's configuration and
+what type of workload you're trying to run. Many of those resources can be
+downloaded from http://resources.gem5.org, and/or from the git repository  
here:

+https://gem5.googlesource.com/public/gem5-resources/

 If you have questions, please send mail to gem5-us...@gem5.org


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50755
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: Id2fb9323dcb9d8bf8e86c78fb5e9aa6afed1c5b8
Gerrit-Change-Number: 50755
Gerrit-PatchSet: 4
Gerrit-Owner: Gabe Black 
Gerrit-Reviewer: Andreas Sandberg 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Daniel Carvalho 
Gerrit-Reviewer: Gabe Black 
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
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: python: Handle exceptions in gem5py.cc.

2021-09-23 Thread Gabe Black (Gerrit) via gem5-dev 
Gabe Black has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50870 )



Change subject: python: Handle exceptions in gem5py.cc.
..

python: Handle exceptions in gem5py.cc.

If an exception is thrown in python, it will emerge in c++ as a
py::error_already_set exception. If it was a SystemExit exception, we
should handle that gracefully and exit with the requested code. If it
was something else, we should print what() so the user has more info
about what went wrong.

Change-Id: I85a94b47865a49f9fbebfc22272249a5f0f59bf5
---
M src/python/gem5py.cc
1 file changed, 9 insertions(+), 1 deletion(-)



diff --git a/src/python/gem5py.cc b/src/python/gem5py.cc
index f2d8759..9b59a03 100644
--- a/src/python/gem5py.cc
+++ b/src/python/gem5py.cc
@@ -76,7 +76,15 @@
 py_argv.append(argv[i]);

 // Actually call the script.
-py::eval_file(argv[1]);
+try {
+py::eval_file(argv[1]);
+} catch (py::error_already_set &e) {
+if (e.matches(PyExc_SystemExit))
+return e.value().attr("code").cast();
+
+std::cerr << e.what();
+return 1;
+}

 return 0;
 }

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50870
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: I85a94b47865a49f9fbebfc22272249a5f0f59bf5
Gerrit-Change-Number: 50870
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black 
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

[gem5-dev] Change in gem5/gem5[develop]: misc: Merge branch v21.1.0.2 hotfix branch into develop

2021-09-23 Thread Bobby R. Bruce (Gerrit) via gem5-dev 
Bobby R. Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50828 )


Change subject: misc: Merge branch v21.1.0.2 hotfix branch into develop
..

misc: Merge branch v21.1.0.2 hotfix branch into develop

Change-Id: I2256ec1399528f4759a3c5c5e306d8c7a38c
---
M src/Doxyfile
M src/base/version.cc
3 files changed, 0 insertions(+), 8 deletions(-)

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




diff --git a/src/Doxyfile b/src/Doxyfile
index b9ea15a..dd52c25 100644
--- a/src/Doxyfile
+++ b/src/Doxyfile
@@ -31,11 +31,7 @@
 # This could be handy for archiving the generated documentation or
 # if some version control system is used.

-<<< HEAD   (c10982 tests: Update x86 boot tests to use x86-ubuntu-img)
 PROJECT_NUMBER = DEVELOP-FOR-V21-2
-===
-PROJECT_NUMBER = v21.1.0.2
->>> BRANCH (4666a4 misc: Update RELEASE-NOTES.md for v21.1.0.2)

 # The OUTPUT_DIRECTORY tag is used to specify the (relative or absolute)
 # base path where the generated documentation will be put.
diff --git a/src/base/version.cc b/src/base/version.cc
index 84c86ce..078a2f9 100644
--- a/src/base/version.cc
+++ b/src/base/version.cc
@@ -32,10 +32,6 @@
 /**
  * @ingroup api_base_utils
  */
-<<< HEAD   (c10982 tests: Update x86 boot tests to use x86-ubuntu-img)
 const char *gem5Version = "[DEVELOP-FOR-V21.2]";
-===
-const char *gem5Version = "21.1.0.2";
->>> BRANCH (4666a4 misc: Update RELEASE-NOTES.md for v21.1.0.2)

 } // namespace gem5

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50828
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: I2256ec1399528f4759a3c5c5e306d8c7a38c
Gerrit-Change-Number: 50828
Gerrit-PatchSet: 1
Gerrit-Owner: Bobby R. Bruce 
Gerrit-Reviewer: Bobby R. 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
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: tests: Fix broken compiler-tests.sh

2021-09-23 Thread Bobby R. Bruce (Gerrit) via gem5-dev 
Bobby R. Bruce has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50829 )


Change subject: tests: Fix broken compiler-tests.sh
..

tests: Fix broken compiler-tests.sh

Change-Id: I4ab0f91fe42249acc973be20201a1bd54e582d16
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/50829
Reviewed-by: Bobby R. Bruce 
Reviewed-by: Jason Lowe-Power 
Maintainer: Bobby R. Bruce 
Tested-by: kokoro 
---
M tests/compiler-tests.sh
1 file changed, 1 insertion(+), 1 deletion(-)

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




diff --git a/tests/compiler-tests.sh b/tests/compiler-tests.sh
index 699d134..12cf083 100755
--- a/tests/compiler-tests.sh
+++ b/tests/compiler-tests.sh
@@ -30,7 +30,7 @@

 # A subset of the above list: these images will build against every target,
 # ignoring builds_per_compiler.
-comprehensive=("gcc-version-11")
+comprehensive=("gcc-version-11"
"clang-version-11")

 # All build targets in build_opt/ which we want to build using each image.

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50829
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: I4ab0f91fe42249acc973be20201a1bd54e582d16
Gerrit-Change-Number: 50829
Gerrit-PatchSet: 2
Gerrit-Owner: Bobby R. Bruce 
Gerrit-Reviewer: Bobby R. 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
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

[gem5-dev] Change in gem5/gem5[develop]: scons: Add a comment about Python v.3.9 distutils.spawn fix.

2021-09-23 Thread Todd Bezenek (Gerrit) via gem5-dev 
Todd Bezenek has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/50869 )



Change subject: scons: Add a comment about Python v.3.9 distutils.spawn fix.
..

scons: Add a comment about Python v.3.9 distutils.spawn fix.

Added a comment about how to fix the error:

  ModuleNotFoundError: No module named 'distutils.spawn'.

in Ubuntu Linux to the file src/SConscript.

Change-Id: Iba2aaf6c8585d82093fdd01baa02388cfaa6c7ce
---
M src/SConscript
1 file changed, 7 insertions(+), 0 deletions(-)



diff --git a/src/SConscript b/src/SConscript
index fdd5830..ffeb503 100644
--- a/src/SConscript
+++ b/src/SConscript
@@ -503,6 +503,13 @@

 # Workaround for bug in SCons version > 0.97d20071212
 # Scons bug id: 2006 gem5 Bug id: 308
+#
+# Note: If you are seeing the error "ModuleNotFoundError: No module
+# named 'distutils.spawn'" thrown by the next line, try a command like
+# this one which installs the Python v.3.9 distutils module on Ubuntu
+# Linux:
+#
+#   $ sudo apt install python3.9-distutils
 for root, dirs, files in os.walk(base_dir, topdown=True):
 Dir(root[len(base_dir) + 1:])


--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/50869
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: Iba2aaf6c8585d82093fdd01baa02388cfaa6c7ce
Gerrit-Change-Number: 50869
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Bezenek 
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