From 3d05f13af9a8a63ab3fae36877bb8473b94b6d1a Mon Sep 17 00:00:00 2001 From: shadtorrie Date: Tue, 11 Aug 2020 10:08:33 -0700 Subject: [PATCH 1/3] adding suppression to flow when runnign vpr --- vtr_flow/scripts/python_libs/vtr/vpr/vpr.py | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py b/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py index 2a17f7bfda7..65060278df6 100644 --- a/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py +++ b/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py @@ -3,6 +3,7 @@ """ from collections import OrderedDict from pathlib import Path +from os import environ from vtr import ( find_vtr_file, CommandRunner, @@ -215,6 +216,14 @@ def run( else: cmd += ["--" + arg, str(value)] + #Extra options to fine-tune LeakSanitizer (LSAN) behaviour. + #Note that if VPR was compiled without LSAN these have no effect + # 'suppressions=...' Add the LeakSanitizer (LSAN) suppression file + # 'exitcode=12' Use a consistent exitcode (on some systems LSAN don't use the default exit code of 23) + # 'fast_unwind_on_malloc=0' Provide more accurate leak stack traces + + environ["LSAN_OPTIONS"] = "suppressions={} exitcode=23 fast_unwind_on_malloc=0".format(find_vtr_file("lsan.supp"))\ + command_runner.run_system_command( cmd, temp_dir=temp_dir, log_filename=log_filename, indent_depth=1 ) From 0fd6d4c7455688fbbf55cff30e29fa029a324acd Mon Sep 17 00:00:00 2001 From: shadtorrie Date: Tue, 11 Aug 2020 13:53:35 -0700 Subject: [PATCH 2/3] fixing pylint issue --- vtr_flow/scripts/python_libs/vtr/vpr/vpr.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py b/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py index 65060278df6..502e4a0f785 100644 --- a/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py +++ b/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py @@ -219,10 +219,13 @@ def run( #Extra options to fine-tune LeakSanitizer (LSAN) behaviour. #Note that if VPR was compiled without LSAN these have no effect # 'suppressions=...' Add the LeakSanitizer (LSAN) suppression file - # 'exitcode=12' Use a consistent exitcode (on some systems LSAN don't use the default exit code of 23) + # 'exitcode=12' Use a consistent exitcode + # (on some systems LSAN don't use the default exit code of 23) # 'fast_unwind_on_malloc=0' Provide more accurate leak stack traces - environ["LSAN_OPTIONS"] = "suppressions={} exitcode=23 fast_unwind_on_malloc=0".format(find_vtr_file("lsan.supp"))\ + environ["LSAN_OPTIONS"] = ( + "suppressions={} exitcode=23 fast_unwind_on_malloc=0".format(find_vtr_file("lsan.supp")) + ) command_runner.run_system_command( cmd, temp_dir=temp_dir, log_filename=log_filename, indent_depth=1 From f9b5977d7e9f3500e081da144e364b6e6b6dfffa Mon Sep 17 00:00:00 2001 From: shadtorrie Date: Wed, 12 Aug 2020 07:48:48 -0700 Subject: [PATCH 3/3] making minor change to start travis --- vtr_flow/scripts/python_libs/vtr/vpr/vpr.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py b/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py index 502e4a0f785..7838d99549c 100644 --- a/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py +++ b/vtr_flow/scripts/python_libs/vtr/vpr/vpr.py @@ -140,7 +140,7 @@ def run( temp_dir=Path("."), log_filename="vpr.out", vpr_exec=None, - vpr_args=None, + vpr_args=None ): """ Runs VPR with the specified configuration