test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] framework/dts: Regist generate ASan report action to atexit
@ 2022-04-02  8:54 Jun Dong
  2022-04-06 12:01 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: Jun Dong @ 2022-04-02  8:54 UTC (permalink / raw)
  To: dts; +Cc: lijuan.tu, qingx.sun, junx.dong

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(-)

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:
-- 
2.33.1.windows.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [dts] [PATCH V1] framework/dts: Regist generate ASan report action to atexit
  2022-04-02  8:54 [dts] [PATCH V1] framework/dts: Regist generate ASan report action to atexit Jun Dong
@ 2022-04-06 12:01 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2022-04-06 12:01 UTC (permalink / raw)
  To: Dong, JunX, dts; +Cc: Sun, QingX

> -----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

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-04-06 12:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-02  8:54 [dts] [PATCH V1] framework/dts: Regist generate ASan report action to atexit Jun Dong
2022-04-06 12:01 ` Tu, Lijuan

test suite reviews and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://inbox.dpdk.org/dts/0 dts/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dts dts/ http://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git