From 69c247de2ff3ca41be93cd90af539926418d023d Mon Sep 17 00:00:00 2001 From: Chris Down Date: Tue, 18 Feb 2020 13:54:37 -0500 Subject: 45463: test: kill: Document why we use SIGURG See discussion in workers/45460. --- Test/B11kill.ztst | 9 +++++++++ 1 file changed, 9 insertions(+) (limited to 'Test') diff --git a/Test/B11kill.ztst b/Test/B11kill.ztst index 40dd99cd3..dc6bf9b89 100644 --- a/Test/B11kill.ztst +++ b/Test/B11kill.ztst @@ -60,6 +60,15 @@ ?(eval):kill:2: not enough arguments # Regression tests: `kill ''` should not result in `kill 0`. +# +# We use SIGURG where an explicit sigspec can be provided as: +# +# 1. By default it's non-terminal, so even if we regress, we won't kill the +# test runner and other processes in the process group since we'll stop +# running this test before we get to the plain kill (and thus SIGTERM) +# cases; +# 2. It's also unlikely to be sent for any other reason during the process +# lifetime, so the test shouldn't be flaky. ( trap 'exit 11' URG -- cgit 1.4.1