[gem5-dev] Change in gem5/gem5[develop]: python: Make DOT config generation optional

2020-05-20 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has submitted this change. (  
https://gem5-review.googlesource.com/c/public/gem5/+/29232 )


Change subject: python: Make DOT config generation optional
..

python: Make DOT config generation optional

By default, DOT configs are always generated when pydot is present.
This change allows a user to pass an empty --dot-config='' to disable
generating the DOT configuration. This can be useful to save space, or
to reduce Gem5 startup time when running many small regression tests.

This brings the behavior in-line with providing an empty
--dump_config='' and/or --json_config='' which similarly disables
generation of those output files.

Change-Id: I5bf39fda0409b948a8d14f3afa95db8fc78de6ee
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/29232
Reviewed-by: Jason Lowe-Power 
Maintainer: Jason Lowe-Power 
Tested-by: kokoro 
---
M src/python/m5/simulate.py
1 file changed, 3 insertions(+), 2 deletions(-)

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



diff --git a/src/python/m5/simulate.py b/src/python/m5/simulate.py
index 3317ae8..698dfbc 100644
--- a/src/python/m5/simulate.py
+++ b/src/python/m5/simulate.py
@@ -107,8 +107,9 @@
 except ImportError:
 pass

-do_dot(root, options.outdir, options.dot_config)
-do_ruby_dot(root, options.outdir, options.dot_config)
+if options.dot_config:
+do_dot(root, options.outdir, options.dot_config)
+do_ruby_dot(root, options.outdir, options.dot_config)

 # Initialize the global statistics
 stats.initSimStats()

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/29232
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: I5bf39fda0409b948a8d14f3afa95db8fc78de6ee
Gerrit-Change-Number: 29232
Gerrit-PatchSet: 3
Gerrit-Owner: Giacomo Travaglini 
Gerrit-Reviewer: Bobby R. Bruce 
Gerrit-Reviewer: Daniel Carvalho 
Gerrit-Reviewer: Giacomo Travaglini 
Gerrit-Reviewer: Jason Lowe-Power 
Gerrit-Reviewer: Nikos Nikoleris 
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: Make DOT config generation optional

2020-05-18 Thread Giacomo Travaglini (Gerrit) via gem5-dev
Giacomo Travaglini has uploaded this change for review. (  
https://gem5-review.googlesource.com/c/public/gem5/+/29232 )



Change subject: python: Make DOT config generation optional
..

python: Make DOT config generation optional

By default, DOT configs are always generated when pydot is present.
This change allows a user to pass --dot-config= to disable generating
the DOT configuration. This can be useful to save space, or to save
Gem5 startup time when running many small regression tests.

This brings the behavior in-line with --dump_config= and --json_config=

Change-Id: I5bf39fda0409b948a8d14f3afa95db8fc78de6ee
---
M src/python/m5/simulate.py
1 file changed, 3 insertions(+), 2 deletions(-)



diff --git a/src/python/m5/simulate.py b/src/python/m5/simulate.py
index 3317ae8..698dfbc 100644
--- a/src/python/m5/simulate.py
+++ b/src/python/m5/simulate.py
@@ -107,8 +107,9 @@
 except ImportError:
 pass

-do_dot(root, options.outdir, options.dot_config)
-do_ruby_dot(root, options.outdir, options.dot_config)
+if options.dot_config:
+do_dot(root, options.outdir, options.dot_config)
+do_ruby_dot(root, options.outdir, options.dot_config)

 # Initialize the global statistics
 stats.initSimStats()

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/29232
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: I5bf39fda0409b948a8d14f3afa95db8fc78de6ee
Gerrit-Change-Number: 29232
Gerrit-PatchSet: 1
Gerrit-Owner: Giacomo Travaglini 
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