diff mbox series

[V1] framework/dts: Regist generate ASan report action to atexit

Message ID 20220402085454.743-1-junx.dong@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] framework/dts: Regist generate ASan report action to atexit |

Checks

Context Check Description
ci/Intel-dts-suite-test success Testing OK

Commit Message

Jun Dong April 2, 2022, 8:54 a.m. UTC 
  Sometime, the framework does not generate ASan reports when exiting abnormally, 
because the operation of generating reports is at the end of the main flow of 
the framework, and there are no guarantee measures in case of abnormality. So
register the generate operation with the atexit module.

Signed-off-by: Jun Dong <junx.dong@intel.com>
---
 framework/dts.py | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

Comments

Tu, Lijuan April 6, 2022, 12:01 p.m. UTC | #1 
> -----Original Message-----
> From: Dong, JunX <junx.dong@intel.com>
> Sent: 2022年4月2日 16:55
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>; Sun, QingX <qingx.sun@intel.com>; Dong,
> JunX <junx.dong@intel.com>
> Subject: [dts] [PATCH V1] framework/dts: Regist generate ASan report action to
> atexit
> 
> Sometime, the framework does not generate ASan reports when exiting
> abnormally, because the operation of generating reports is at the end of the
> main flow of the framework, and there are no guarantee measures in case of
> abnormality. So register the generate operation with the atexit module.
> 
> Signed-off-by: Jun Dong <junx.dong@intel.com>

Acked-by: Lijuan Tu <lijuan.tu@intel.com>

Applied, thanks
diff mbox series

Patch

diff --git a/framework/dts.py b/framework/dts.py
index 1ffb1c23..dbb5a1ed 100644
--- a/framework/dts.py
+++ b/framework/dts.py
@@ -559,6 +559,8 @@  def run_all(
 
     # prepare ASan test
     ASanTestProcess.test_prepare(asan, output_dir)
+    # register generate ASan report action
+    atexit.register(ASanTestProcess.test_process)
 
     if not os.path.exists(output_dir):
         os.mkdir(output_dir)
@@ -676,9 +678,6 @@  def run_all(
 
     save_all_results()
 
-    # process ASan test report
-    ASanTestProcess.test_process()
-
 
 def show_speedup_options_messages(read_cache, skip_setup):
     if read_cache: