Ethan Furman | dbac8f4 | 2021-04-19 18:04:53 -0700 | [diff] [blame^] | 1 | import enum |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 2 | import errno |
Victor Stinner | d6debb2 | 2017-03-27 16:05:26 +0200 | [diff] [blame] | 3 | import os |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 4 | import random |
Guido van Rossum | 4f17e3e | 1995-03-16 15:07:38 +0000 | [diff] [blame] | 5 | import signal |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 6 | import socket |
Antoine Pitrou | f7d090c | 2017-06-29 16:40:14 +0200 | [diff] [blame] | 7 | import statistics |
Christian Heimes | 4fbc72b | 2008-03-22 00:47:35 +0000 | [diff] [blame] | 8 | import subprocess |
Victor Stinner | d6debb2 | 2017-03-27 16:05:26 +0200 | [diff] [blame] | 9 | import sys |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 10 | import threading |
Victor Stinner | d6debb2 | 2017-03-27 16:05:26 +0200 | [diff] [blame] | 11 | import time |
| 12 | import unittest |
| 13 | from test import support |
Hai Shi | a7f5d93 | 2020-08-04 00:41:24 +0800 | [diff] [blame] | 14 | from test.support import os_helper |
Berker Peksag | ce64391 | 2015-05-06 06:33:17 +0300 | [diff] [blame] | 15 | from test.support.script_helper import assert_python_ok, spawn_python |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 16 | try: |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 17 | import _testcapi |
| 18 | except ImportError: |
| 19 | _testcapi = None |
Christian Heimes | c06950e | 2008-02-28 21:17:00 +0000 | [diff] [blame] | 20 | |
Guido van Rossum | cc5a91d | 1997-04-16 00:29:15 +0000 | [diff] [blame] | 21 | |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 22 | class GenericTests(unittest.TestCase): |
| 23 | |
| 24 | def test_enums(self): |
| 25 | for name in dir(signal): |
| 26 | sig = getattr(signal, name) |
| 27 | if name in {'SIG_DFL', 'SIG_IGN'}: |
| 28 | self.assertIsInstance(sig, signal.Handlers) |
| 29 | elif name in {'SIG_BLOCK', 'SIG_UNBLOCK', 'SIG_SETMASK'}: |
| 30 | self.assertIsInstance(sig, signal.Sigmasks) |
| 31 | elif name.startswith('SIG') and not name.startswith('SIG_'): |
| 32 | self.assertIsInstance(sig, signal.Signals) |
| 33 | elif name.startswith('CTRL_'): |
| 34 | self.assertIsInstance(sig, signal.Signals) |
| 35 | self.assertEqual(sys.platform, "win32") |
| 36 | |
Ethan Furman | dbac8f4 | 2021-04-19 18:04:53 -0700 | [diff] [blame^] | 37 | CheckedSignals = enum._old_convert_( |
| 38 | enum.IntEnum, 'Signals', 'signal', |
| 39 | lambda name: |
| 40 | name.isupper() |
| 41 | and (name.startswith('SIG') and not name.startswith('SIG_')) |
| 42 | or name.startswith('CTRL_'), |
| 43 | source=signal, |
| 44 | ) |
| 45 | enum._test_simple_enum(CheckedSignals, signal.Signals) |
| 46 | |
| 47 | CheckedHandlers = enum._old_convert_( |
| 48 | enum.IntEnum, 'Handlers', 'signal', |
| 49 | lambda name: name in ('SIG_DFL', 'SIG_IGN'), |
| 50 | source=signal, |
| 51 | ) |
| 52 | enum._test_simple_enum(CheckedHandlers, signal.Handlers) |
| 53 | |
| 54 | Sigmasks = getattr(signal, 'Sigmasks', None) |
| 55 | if Sigmasks is not None: |
| 56 | CheckedSigmasks = enum._old_convert_( |
| 57 | enum.IntEnum, 'Sigmasks', 'signal', |
| 58 | lambda name: name in ('SIG_BLOCK', 'SIG_UNBLOCK', 'SIG_SETMASK'), |
| 59 | source=signal, |
| 60 | ) |
| 61 | enum._test_simple_enum(CheckedSigmasks, Sigmasks) |
| 62 | |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 63 | |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 64 | @unittest.skipIf(sys.platform == "win32", "Not valid on Windows") |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 65 | class PosixTests(unittest.TestCase): |
Christian Heimes | 4fbc72b | 2008-03-22 00:47:35 +0000 | [diff] [blame] | 66 | def trivial_signal_handler(self, *args): |
| 67 | pass |
| 68 | |
Thomas Wouters | ed03b41 | 2007-08-28 21:37:11 +0000 | [diff] [blame] | 69 | def test_out_of_range_signal_number_raises_error(self): |
| 70 | self.assertRaises(ValueError, signal.getsignal, 4242) |
| 71 | |
Thomas Wouters | ed03b41 | 2007-08-28 21:37:11 +0000 | [diff] [blame] | 72 | self.assertRaises(ValueError, signal.signal, 4242, |
Christian Heimes | 4fbc72b | 2008-03-22 00:47:35 +0000 | [diff] [blame] | 73 | self.trivial_signal_handler) |
Thomas Wouters | ed03b41 | 2007-08-28 21:37:11 +0000 | [diff] [blame] | 74 | |
Antoine Pietri | 5d2a27d | 2018-03-12 14:42:34 +0100 | [diff] [blame] | 75 | self.assertRaises(ValueError, signal.strsignal, 4242) |
| 76 | |
Thomas Wouters | ed03b41 | 2007-08-28 21:37:11 +0000 | [diff] [blame] | 77 | def test_setting_signal_handler_to_none_raises_error(self): |
| 78 | self.assertRaises(TypeError, signal.signal, |
| 79 | signal.SIGUSR1, None) |
| 80 | |
Christian Heimes | 4fbc72b | 2008-03-22 00:47:35 +0000 | [diff] [blame] | 81 | def test_getsignal(self): |
| 82 | hup = signal.signal(signal.SIGHUP, self.trivial_signal_handler) |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 83 | self.assertIsInstance(hup, signal.Handlers) |
Ezio Melotti | 19f2aeb | 2010-11-21 01:30:29 +0000 | [diff] [blame] | 84 | self.assertEqual(signal.getsignal(signal.SIGHUP), |
| 85 | self.trivial_signal_handler) |
Christian Heimes | 4fbc72b | 2008-03-22 00:47:35 +0000 | [diff] [blame] | 86 | signal.signal(signal.SIGHUP, hup) |
Ezio Melotti | 19f2aeb | 2010-11-21 01:30:29 +0000 | [diff] [blame] | 87 | self.assertEqual(signal.getsignal(signal.SIGHUP), hup) |
Christian Heimes | 4fbc72b | 2008-03-22 00:47:35 +0000 | [diff] [blame] | 88 | |
Antoine Pietri | 5d2a27d | 2018-03-12 14:42:34 +0100 | [diff] [blame] | 89 | def test_strsignal(self): |
Antoine Pietri | 019f5b3 | 2018-03-12 20:03:14 +0100 | [diff] [blame] | 90 | self.assertIn("Interrupt", signal.strsignal(signal.SIGINT)) |
| 91 | self.assertIn("Terminated", signal.strsignal(signal.SIGTERM)) |
Michael Osipov | 48ce489 | 2018-08-23 15:27:19 +0200 | [diff] [blame] | 92 | self.assertIn("Hangup", signal.strsignal(signal.SIGHUP)) |
Antoine Pietri | 5d2a27d | 2018-03-12 14:42:34 +0100 | [diff] [blame] | 93 | |
Victor Stinner | 32eb840 | 2016-03-15 11:12:35 +0100 | [diff] [blame] | 94 | # Issue 3864, unknown if this affects earlier versions of freebsd also |
Victor Stinner | 32eb840 | 2016-03-15 11:12:35 +0100 | [diff] [blame] | 95 | def test_interprocess_signal(self): |
| 96 | dirname = os.path.dirname(__file__) |
| 97 | script = os.path.join(dirname, 'signalinterproctester.py') |
| 98 | assert_python_ok(script) |
| 99 | |
Antoine Pitrou | 9d3627e | 2018-05-04 13:00:50 +0200 | [diff] [blame] | 100 | def test_valid_signals(self): |
| 101 | s = signal.valid_signals() |
| 102 | self.assertIsInstance(s, set) |
| 103 | self.assertIn(signal.Signals.SIGINT, s) |
| 104 | self.assertIn(signal.Signals.SIGALRM, s) |
| 105 | self.assertNotIn(0, s) |
| 106 | self.assertNotIn(signal.NSIG, s) |
| 107 | self.assertLess(len(s), signal.NSIG) |
| 108 | |
Gregory P. Smith | 38f11cc | 2019-02-16 12:57:40 -0800 | [diff] [blame] | 109 | @unittest.skipUnless(sys.executable, "sys.executable required.") |
| 110 | def test_keyboard_interrupt_exit_code(self): |
| 111 | """KeyboardInterrupt triggers exit via SIGINT.""" |
| 112 | process = subprocess.run( |
| 113 | [sys.executable, "-c", |
Gregory P. Smith | 414c625 | 2019-02-16 17:22:39 -0800 | [diff] [blame] | 114 | "import os, signal, time\n" |
| 115 | "os.kill(os.getpid(), signal.SIGINT)\n" |
| 116 | "for _ in range(999): time.sleep(0.01)"], |
Gregory P. Smith | 38f11cc | 2019-02-16 12:57:40 -0800 | [diff] [blame] | 117 | stderr=subprocess.PIPE) |
| 118 | self.assertIn(b"KeyboardInterrupt", process.stderr) |
| 119 | self.assertEqual(process.returncode, -signal.SIGINT) |
Gregory P. Smith | 414c625 | 2019-02-16 17:22:39 -0800 | [diff] [blame] | 120 | # Caveat: The exit code is insufficient to guarantee we actually died |
| 121 | # via a signal. POSIX shells do more than look at the 8 bit value. |
| 122 | # Writing an automation friendly test of an interactive shell |
| 123 | # to confirm that our process died via a SIGINT proved too complex. |
Gregory P. Smith | 38f11cc | 2019-02-16 12:57:40 -0800 | [diff] [blame] | 124 | |
Christian Heimes | 4fbc72b | 2008-03-22 00:47:35 +0000 | [diff] [blame] | 125 | |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 126 | @unittest.skipUnless(sys.platform == "win32", "Windows specific") |
| 127 | class WindowsSignalTests(unittest.TestCase): |
Antoine Pitrou | 9d3627e | 2018-05-04 13:00:50 +0200 | [diff] [blame] | 128 | |
| 129 | def test_valid_signals(self): |
| 130 | s = signal.valid_signals() |
| 131 | self.assertIsInstance(s, set) |
| 132 | self.assertGreaterEqual(len(s), 6) |
| 133 | self.assertIn(signal.Signals.SIGINT, s) |
| 134 | self.assertNotIn(0, s) |
| 135 | self.assertNotIn(signal.NSIG, s) |
| 136 | self.assertLess(len(s), signal.NSIG) |
| 137 | |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 138 | def test_issue9324(self): |
Brian Curtin | eccd4d9 | 2010-10-01 15:09:53 +0000 | [diff] [blame] | 139 | # Updated for issue #10003, adding SIGBREAK |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 140 | handler = lambda x, y: None |
Nick Coghlan | 60b3ac7 | 2013-08-03 22:56:30 +1000 | [diff] [blame] | 141 | checked = set() |
Brian Curtin | eccd4d9 | 2010-10-01 15:09:53 +0000 | [diff] [blame] | 142 | for sig in (signal.SIGABRT, signal.SIGBREAK, signal.SIGFPE, |
| 143 | signal.SIGILL, signal.SIGINT, signal.SIGSEGV, |
| 144 | signal.SIGTERM): |
Nick Coghlan | 60b3ac7 | 2013-08-03 22:56:30 +1000 | [diff] [blame] | 145 | # Set and then reset a handler for signals that work on windows. |
| 146 | # Issue #18396, only for signals without a C-level handler. |
| 147 | if signal.getsignal(sig) is not None: |
| 148 | signal.signal(sig, signal.signal(sig, handler)) |
| 149 | checked.add(sig) |
| 150 | # Issue #18396: Ensure the above loop at least tested *something* |
| 151 | self.assertTrue(checked) |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 152 | |
| 153 | with self.assertRaises(ValueError): |
| 154 | signal.signal(-1, handler) |
Brian Curtin | 80cd4bf | 2010-08-07 03:52:38 +0000 | [diff] [blame] | 155 | |
| 156 | with self.assertRaises(ValueError): |
| 157 | signal.signal(7, handler) |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 158 | |
Gregory P. Smith | 38f11cc | 2019-02-16 12:57:40 -0800 | [diff] [blame] | 159 | @unittest.skipUnless(sys.executable, "sys.executable required.") |
| 160 | def test_keyboard_interrupt_exit_code(self): |
| 161 | """KeyboardInterrupt triggers an exit using STATUS_CONTROL_C_EXIT.""" |
| 162 | # We don't test via os.kill(os.getpid(), signal.CTRL_C_EVENT) here |
| 163 | # as that requires setting up a console control handler in a child |
| 164 | # in its own process group. Doable, but quite complicated. (see |
| 165 | # @eryksun on https://github.com/python/cpython/pull/11862) |
| 166 | process = subprocess.run( |
| 167 | [sys.executable, "-c", "raise KeyboardInterrupt"], |
| 168 | stderr=subprocess.PIPE) |
| 169 | self.assertIn(b"KeyboardInterrupt", process.stderr) |
| 170 | STATUS_CONTROL_C_EXIT = 0xC000013A |
| 171 | self.assertEqual(process.returncode, STATUS_CONTROL_C_EXIT) |
| 172 | |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 173 | |
Benjamin Peterson | c68a4a0 | 2013-01-18 00:10:24 -0500 | [diff] [blame] | 174 | class WakeupFDTests(unittest.TestCase): |
| 175 | |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 176 | def test_invalid_call(self): |
| 177 | # First parameter is positional-only |
| 178 | with self.assertRaises(TypeError): |
| 179 | signal.set_wakeup_fd(signum=signal.SIGINT) |
| 180 | |
| 181 | # warn_on_full_buffer is a keyword-only parameter |
| 182 | with self.assertRaises(TypeError): |
| 183 | signal.set_wakeup_fd(signal.SIGINT, False) |
| 184 | |
Benjamin Peterson | c68a4a0 | 2013-01-18 00:10:24 -0500 | [diff] [blame] | 185 | def test_invalid_fd(self): |
Hai Shi | a7f5d93 | 2020-08-04 00:41:24 +0800 | [diff] [blame] | 186 | fd = os_helper.make_bad_fd() |
Victor Stinner | a7d03d9 | 2014-07-21 17:17:28 +0200 | [diff] [blame] | 187 | self.assertRaises((ValueError, OSError), |
| 188 | signal.set_wakeup_fd, fd) |
Benjamin Peterson | c68a4a0 | 2013-01-18 00:10:24 -0500 | [diff] [blame] | 189 | |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 190 | def test_invalid_socket(self): |
| 191 | sock = socket.socket() |
| 192 | fd = sock.fileno() |
| 193 | sock.close() |
| 194 | self.assertRaises((ValueError, OSError), |
| 195 | signal.set_wakeup_fd, fd) |
| 196 | |
Victor Stinner | d18ccd1 | 2014-07-24 21:58:53 +0200 | [diff] [blame] | 197 | def test_set_wakeup_fd_result(self): |
Victor Stinner | 1d8948e | 2014-07-24 22:51:05 +0200 | [diff] [blame] | 198 | r1, w1 = os.pipe() |
| 199 | self.addCleanup(os.close, r1) |
| 200 | self.addCleanup(os.close, w1) |
| 201 | r2, w2 = os.pipe() |
| 202 | self.addCleanup(os.close, r2) |
| 203 | self.addCleanup(os.close, w2) |
Victor Stinner | d18ccd1 | 2014-07-24 21:58:53 +0200 | [diff] [blame] | 204 | |
Victor Stinner | 7cea44d | 2014-08-27 14:02:36 +0200 | [diff] [blame] | 205 | if hasattr(os, 'set_blocking'): |
| 206 | os.set_blocking(w1, False) |
| 207 | os.set_blocking(w2, False) |
Victor Stinner | 3822760 | 2014-08-27 12:59:44 +0200 | [diff] [blame] | 208 | |
Victor Stinner | 1d8948e | 2014-07-24 22:51:05 +0200 | [diff] [blame] | 209 | signal.set_wakeup_fd(w1) |
Victor Stinner | c82a179 | 2014-07-24 22:55:12 +0200 | [diff] [blame] | 210 | self.assertEqual(signal.set_wakeup_fd(w2), w1) |
| 211 | self.assertEqual(signal.set_wakeup_fd(-1), w2) |
| 212 | self.assertEqual(signal.set_wakeup_fd(-1), -1) |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 213 | |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 214 | def test_set_wakeup_fd_socket_result(self): |
| 215 | sock1 = socket.socket() |
| 216 | self.addCleanup(sock1.close) |
Victor Stinner | 3822760 | 2014-08-27 12:59:44 +0200 | [diff] [blame] | 217 | sock1.setblocking(False) |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 218 | fd1 = sock1.fileno() |
| 219 | |
| 220 | sock2 = socket.socket() |
| 221 | self.addCleanup(sock2.close) |
Victor Stinner | 3822760 | 2014-08-27 12:59:44 +0200 | [diff] [blame] | 222 | sock2.setblocking(False) |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 223 | fd2 = sock2.fileno() |
| 224 | |
| 225 | signal.set_wakeup_fd(fd1) |
Victor Stinner | da565a7 | 2014-07-30 10:03:03 +0200 | [diff] [blame] | 226 | self.assertEqual(signal.set_wakeup_fd(fd2), fd1) |
| 227 | self.assertEqual(signal.set_wakeup_fd(-1), fd2) |
| 228 | self.assertEqual(signal.set_wakeup_fd(-1), -1) |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 229 | |
Victor Stinner | 3822760 | 2014-08-27 12:59:44 +0200 | [diff] [blame] | 230 | # On Windows, files are always blocking and Windows does not provide a |
| 231 | # function to test if a socket is in non-blocking mode. |
| 232 | @unittest.skipIf(sys.platform == "win32", "tests specific to POSIX") |
| 233 | def test_set_wakeup_fd_blocking(self): |
| 234 | rfd, wfd = os.pipe() |
| 235 | self.addCleanup(os.close, rfd) |
| 236 | self.addCleanup(os.close, wfd) |
| 237 | |
| 238 | # fd must be non-blocking |
| 239 | os.set_blocking(wfd, True) |
| 240 | with self.assertRaises(ValueError) as cm: |
| 241 | signal.set_wakeup_fd(wfd) |
| 242 | self.assertEqual(str(cm.exception), |
| 243 | "the fd %s must be in non-blocking mode" % wfd) |
| 244 | |
| 245 | # non-blocking is ok |
| 246 | os.set_blocking(wfd, False) |
| 247 | signal.set_wakeup_fd(wfd) |
| 248 | signal.set_wakeup_fd(-1) |
| 249 | |
Benjamin Peterson | c68a4a0 | 2013-01-18 00:10:24 -0500 | [diff] [blame] | 250 | |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 251 | @unittest.skipIf(sys.platform == "win32", "Not valid on Windows") |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 252 | class WakeupSignalTests(unittest.TestCase): |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 253 | @unittest.skipIf(_testcapi is None, 'need _testcapi') |
Charles-François Natali | 027f9a3 | 2011-10-02 18:36:05 +0200 | [diff] [blame] | 254 | def check_wakeup(self, test_body, *signals, ordered=True): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 255 | # use a subprocess to have only one thread |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 256 | code = """if 1: |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 257 | import _testcapi |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 258 | import os |
| 259 | import signal |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 260 | import struct |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 261 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 262 | signals = {!r} |
Victor Stinner | c13ef66 | 2011-05-25 02:35:58 +0200 | [diff] [blame] | 263 | |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 264 | def handler(signum, frame): |
| 265 | pass |
| 266 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 267 | def check_signum(signals): |
| 268 | data = os.read(read, len(signals)+1) |
| 269 | raised = struct.unpack('%uB' % len(data), data) |
Charles-François Natali | 027f9a3 | 2011-10-02 18:36:05 +0200 | [diff] [blame] | 270 | if not {!r}: |
| 271 | raised = set(raised) |
| 272 | signals = set(signals) |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 273 | if raised != signals: |
| 274 | raise Exception("%r != %r" % (raised, signals)) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 275 | |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 276 | {} |
| 277 | |
| 278 | signal.signal(signal.SIGALRM, handler) |
| 279 | read, write = os.pipe() |
Victor Stinner | 1db9e7b | 2014-07-29 22:32:47 +0200 | [diff] [blame] | 280 | os.set_blocking(write, False) |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 281 | signal.set_wakeup_fd(write) |
| 282 | |
| 283 | test() |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 284 | check_signum(signals) |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 285 | |
| 286 | os.close(read) |
| 287 | os.close(write) |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 288 | """.format(tuple(map(int, signals)), ordered, test_body) |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 289 | |
| 290 | assert_python_ok('-c', code) |
Victor Stinner | d49b1f1 | 2011-05-08 02:03:15 +0200 | [diff] [blame] | 291 | |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 292 | @unittest.skipIf(_testcapi is None, 'need _testcapi') |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 293 | def test_wakeup_write_error(self): |
| 294 | # Issue #16105: write() errors in the C signal handler should not |
| 295 | # pass silently. |
| 296 | # Use a subprocess to have only one thread. |
| 297 | code = """if 1: |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 298 | import _testcapi |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 299 | import errno |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 300 | import os |
| 301 | import signal |
| 302 | import sys |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 303 | from test.support import captured_stderr |
| 304 | |
| 305 | def handler(signum, frame): |
| 306 | 1/0 |
| 307 | |
| 308 | signal.signal(signal.SIGALRM, handler) |
| 309 | r, w = os.pipe() |
Victor Stinner | 1db9e7b | 2014-07-29 22:32:47 +0200 | [diff] [blame] | 310 | os.set_blocking(r, False) |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 311 | |
| 312 | # Set wakeup_fd a read-only file descriptor to trigger the error |
| 313 | signal.set_wakeup_fd(r) |
| 314 | try: |
| 315 | with captured_stderr() as err: |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 316 | signal.raise_signal(signal.SIGALRM) |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 317 | except ZeroDivisionError: |
| 318 | # An ignored exception should have been printed out on stderr |
| 319 | err = err.getvalue() |
| 320 | if ('Exception ignored when trying to write to the signal wakeup fd' |
| 321 | not in err): |
| 322 | raise AssertionError(err) |
| 323 | if ('OSError: [Errno %d]' % errno.EBADF) not in err: |
| 324 | raise AssertionError(err) |
| 325 | else: |
| 326 | raise AssertionError("ZeroDivisionError not raised") |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 327 | |
| 328 | os.close(r) |
| 329 | os.close(w) |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 330 | """ |
Antoine Pitrou | 8f0bdda | 2013-08-17 21:43:47 +0200 | [diff] [blame] | 331 | r, w = os.pipe() |
| 332 | try: |
| 333 | os.write(r, b'x') |
| 334 | except OSError: |
| 335 | pass |
| 336 | else: |
| 337 | self.skipTest("OS doesn't report write() error on the read end of a pipe") |
| 338 | finally: |
| 339 | os.close(r) |
| 340 | os.close(w) |
Antoine Pitrou | 6f6ec37 | 2013-08-17 20:27:56 +0200 | [diff] [blame] | 341 | |
| 342 | assert_python_ok('-c', code) |
| 343 | |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 344 | def test_wakeup_fd_early(self): |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 345 | self.check_wakeup("""def test(): |
| 346 | import select |
| 347 | import time |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 348 | |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 349 | TIMEOUT_FULL = 10 |
| 350 | TIMEOUT_HALF = 5 |
| 351 | |
Victor Stinner | 749a6a8 | 2015-03-30 22:09:14 +0200 | [diff] [blame] | 352 | class InterruptSelect(Exception): |
| 353 | pass |
| 354 | |
| 355 | def handler(signum, frame): |
| 356 | raise InterruptSelect |
| 357 | signal.signal(signal.SIGALRM, handler) |
| 358 | |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 359 | signal.alarm(1) |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 360 | |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 361 | # We attempt to get a signal during the sleep, |
| 362 | # before select is called |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 363 | try: |
| 364 | select.select([], [], [], TIMEOUT_FULL) |
Victor Stinner | 749a6a8 | 2015-03-30 22:09:14 +0200 | [diff] [blame] | 365 | except InterruptSelect: |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 366 | pass |
| 367 | else: |
| 368 | raise Exception("select() was not interrupted") |
| 369 | |
| 370 | before_time = time.monotonic() |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 371 | select.select([read], [], [], TIMEOUT_FULL) |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 372 | after_time = time.monotonic() |
| 373 | dt = after_time - before_time |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 374 | if dt >= TIMEOUT_HALF: |
| 375 | raise Exception("%s >= %s" % (dt, TIMEOUT_HALF)) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 376 | """, signal.SIGALRM) |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 377 | |
| 378 | def test_wakeup_fd_during(self): |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 379 | self.check_wakeup("""def test(): |
| 380 | import select |
| 381 | import time |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 382 | |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 383 | TIMEOUT_FULL = 10 |
| 384 | TIMEOUT_HALF = 5 |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 385 | |
Victor Stinner | 749a6a8 | 2015-03-30 22:09:14 +0200 | [diff] [blame] | 386 | class InterruptSelect(Exception): |
| 387 | pass |
| 388 | |
| 389 | def handler(signum, frame): |
| 390 | raise InterruptSelect |
| 391 | signal.signal(signal.SIGALRM, handler) |
| 392 | |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 393 | signal.alarm(1) |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 394 | before_time = time.monotonic() |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 395 | # We attempt to get a signal during the select call |
| 396 | try: |
| 397 | select.select([read], [], [], TIMEOUT_FULL) |
Victor Stinner | 749a6a8 | 2015-03-30 22:09:14 +0200 | [diff] [blame] | 398 | except InterruptSelect: |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 399 | pass |
| 400 | else: |
Victor Stinner | 749a6a8 | 2015-03-30 22:09:14 +0200 | [diff] [blame] | 401 | raise Exception("select() was not interrupted") |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 402 | after_time = time.monotonic() |
Victor Stinner | e40b3aa | 2011-07-04 17:35:10 +0200 | [diff] [blame] | 403 | dt = after_time - before_time |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 404 | if dt >= TIMEOUT_HALF: |
| 405 | raise Exception("%s >= %s" % (dt, TIMEOUT_HALF)) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 406 | """, signal.SIGALRM) |
Victor Stinner | d49b1f1 | 2011-05-08 02:03:15 +0200 | [diff] [blame] | 407 | |
| 408 | def test_signum(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 409 | self.check_wakeup("""def test(): |
| 410 | signal.signal(signal.SIGUSR1, handler) |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 411 | signal.raise_signal(signal.SIGUSR1) |
| 412 | signal.raise_signal(signal.SIGALRM) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 413 | """, signal.SIGUSR1, signal.SIGALRM) |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 414 | |
Victor Stinner | c13ef66 | 2011-05-25 02:35:58 +0200 | [diff] [blame] | 415 | @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'), |
| 416 | 'need signal.pthread_sigmask()') |
Victor Stinner | c13ef66 | 2011-05-25 02:35:58 +0200 | [diff] [blame] | 417 | def test_pending(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 418 | self.check_wakeup("""def test(): |
| 419 | signum1 = signal.SIGUSR1 |
| 420 | signum2 = signal.SIGUSR2 |
Victor Stinner | c13ef66 | 2011-05-25 02:35:58 +0200 | [diff] [blame] | 421 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 422 | signal.signal(signum1, handler) |
| 423 | signal.signal(signum2, handler) |
Victor Stinner | c13ef66 | 2011-05-25 02:35:58 +0200 | [diff] [blame] | 424 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 425 | signal.pthread_sigmask(signal.SIG_BLOCK, (signum1, signum2)) |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 426 | signal.raise_signal(signum1) |
| 427 | signal.raise_signal(signum2) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 428 | # Unblocking the 2 signals calls the C signal handler twice |
| 429 | signal.pthread_sigmask(signal.SIG_UNBLOCK, (signum1, signum2)) |
Charles-François Natali | 027f9a3 | 2011-10-02 18:36:05 +0200 | [diff] [blame] | 430 | """, signal.SIGUSR1, signal.SIGUSR2, ordered=False) |
Victor Stinner | c13ef66 | 2011-05-25 02:35:58 +0200 | [diff] [blame] | 431 | |
Christian Heimes | 5fb7c2a | 2007-12-24 08:52:31 +0000 | [diff] [blame] | 432 | |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 433 | @unittest.skipUnless(hasattr(socket, 'socketpair'), 'need socket.socketpair') |
| 434 | class WakeupSocketSignalTests(unittest.TestCase): |
| 435 | |
| 436 | @unittest.skipIf(_testcapi is None, 'need _testcapi') |
| 437 | def test_socket(self): |
| 438 | # use a subprocess to have only one thread |
| 439 | code = """if 1: |
| 440 | import signal |
| 441 | import socket |
| 442 | import struct |
| 443 | import _testcapi |
| 444 | |
| 445 | signum = signal.SIGINT |
| 446 | signals = (signum,) |
| 447 | |
| 448 | def handler(signum, frame): |
| 449 | pass |
| 450 | |
| 451 | signal.signal(signum, handler) |
| 452 | |
| 453 | read, write = socket.socketpair() |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 454 | write.setblocking(False) |
| 455 | signal.set_wakeup_fd(write.fileno()) |
| 456 | |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 457 | signal.raise_signal(signum) |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 458 | |
| 459 | data = read.recv(1) |
| 460 | if not data: |
| 461 | raise Exception("no signum written") |
| 462 | raised = struct.unpack('B', data) |
| 463 | if raised != signals: |
| 464 | raise Exception("%r != %r" % (raised, signals)) |
| 465 | |
| 466 | read.close() |
| 467 | write.close() |
| 468 | """ |
| 469 | |
| 470 | assert_python_ok('-c', code) |
| 471 | |
| 472 | @unittest.skipIf(_testcapi is None, 'need _testcapi') |
| 473 | def test_send_error(self): |
| 474 | # Use a subprocess to have only one thread. |
| 475 | if os.name == 'nt': |
| 476 | action = 'send' |
| 477 | else: |
| 478 | action = 'write' |
| 479 | code = """if 1: |
| 480 | import errno |
| 481 | import signal |
| 482 | import socket |
| 483 | import sys |
| 484 | import time |
| 485 | import _testcapi |
| 486 | from test.support import captured_stderr |
| 487 | |
| 488 | signum = signal.SIGINT |
| 489 | |
| 490 | def handler(signum, frame): |
| 491 | pass |
| 492 | |
| 493 | signal.signal(signum, handler) |
| 494 | |
| 495 | read, write = socket.socketpair() |
| 496 | read.setblocking(False) |
| 497 | write.setblocking(False) |
| 498 | |
| 499 | signal.set_wakeup_fd(write.fileno()) |
| 500 | |
| 501 | # Close sockets: send() will fail |
| 502 | read.close() |
| 503 | write.close() |
| 504 | |
| 505 | with captured_stderr() as err: |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 506 | signal.raise_signal(signum) |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 507 | |
| 508 | err = err.getvalue() |
| 509 | if ('Exception ignored when trying to {action} to the signal wakeup fd' |
| 510 | not in err): |
| 511 | raise AssertionError(err) |
| 512 | """.format(action=action) |
| 513 | assert_python_ok('-c', code) |
| 514 | |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 515 | @unittest.skipIf(_testcapi is None, 'need _testcapi') |
| 516 | def test_warn_on_full_buffer(self): |
| 517 | # Use a subprocess to have only one thread. |
| 518 | if os.name == 'nt': |
| 519 | action = 'send' |
| 520 | else: |
| 521 | action = 'write' |
| 522 | code = """if 1: |
| 523 | import errno |
| 524 | import signal |
| 525 | import socket |
| 526 | import sys |
| 527 | import time |
| 528 | import _testcapi |
| 529 | from test.support import captured_stderr |
| 530 | |
| 531 | signum = signal.SIGINT |
| 532 | |
| 533 | # This handler will be called, but we intentionally won't read from |
| 534 | # the wakeup fd. |
| 535 | def handler(signum, frame): |
| 536 | pass |
| 537 | |
| 538 | signal.signal(signum, handler) |
| 539 | |
| 540 | read, write = socket.socketpair() |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 541 | |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 542 | # Fill the socketpair buffer |
| 543 | if sys.platform == 'win32': |
| 544 | # bpo-34130: On Windows, sometimes non-blocking send fails to fill |
| 545 | # the full socketpair buffer, so use a timeout of 50 ms instead. |
| 546 | write.settimeout(0.050) |
| 547 | else: |
| 548 | write.setblocking(False) |
| 549 | |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 550 | written = 0 |
pxinwr | 1244c81 | 2020-12-01 05:48:33 +0800 | [diff] [blame] | 551 | if sys.platform == "vxworks": |
| 552 | CHUNK_SIZES = (1,) |
| 553 | else: |
| 554 | # Start with large chunk size to reduce the |
| 555 | # number of send needed to fill the buffer. |
| 556 | CHUNK_SIZES = (2 ** 16, 2 ** 8, 1) |
| 557 | for chunk_size in CHUNK_SIZES: |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 558 | chunk = b"x" * chunk_size |
| 559 | try: |
| 560 | while True: |
| 561 | write.send(chunk) |
| 562 | written += chunk_size |
Christian Heimes | 03c8ddd | 2020-11-20 09:26:07 +0100 | [diff] [blame] | 563 | except (BlockingIOError, TimeoutError): |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 564 | pass |
| 565 | |
| 566 | print(f"%s bytes written into the socketpair" % written, flush=True) |
| 567 | |
| 568 | write.setblocking(False) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 569 | try: |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 570 | write.send(b"x") |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 571 | except BlockingIOError: |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 572 | # The socketpair buffer seems full |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 573 | pass |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 574 | else: |
| 575 | raise AssertionError("%s bytes failed to fill the socketpair " |
| 576 | "buffer" % written) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 577 | |
| 578 | # By default, we get a warning when a signal arrives |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 579 | msg = ('Exception ignored when trying to {action} ' |
| 580 | 'to the signal wakeup fd') |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 581 | signal.set_wakeup_fd(write.fileno()) |
| 582 | |
| 583 | with captured_stderr() as err: |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 584 | signal.raise_signal(signum) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 585 | |
| 586 | err = err.getvalue() |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 587 | if msg not in err: |
| 588 | raise AssertionError("first set_wakeup_fd() test failed, " |
| 589 | "stderr: %r" % err) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 590 | |
| 591 | # And also if warn_on_full_buffer=True |
| 592 | signal.set_wakeup_fd(write.fileno(), warn_on_full_buffer=True) |
| 593 | |
| 594 | with captured_stderr() as err: |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 595 | signal.raise_signal(signum) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 596 | |
| 597 | err = err.getvalue() |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 598 | if msg not in err: |
| 599 | raise AssertionError("set_wakeup_fd(warn_on_full_buffer=True) " |
| 600 | "test failed, stderr: %r" % err) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 601 | |
| 602 | # But not if warn_on_full_buffer=False |
| 603 | signal.set_wakeup_fd(write.fileno(), warn_on_full_buffer=False) |
| 604 | |
| 605 | with captured_stderr() as err: |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 606 | signal.raise_signal(signum) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 607 | |
| 608 | err = err.getvalue() |
| 609 | if err != "": |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 610 | raise AssertionError("set_wakeup_fd(warn_on_full_buffer=False) " |
| 611 | "test failed, stderr: %r" % err) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 612 | |
| 613 | # And then check the default again, to make sure warn_on_full_buffer |
| 614 | # settings don't leak across calls. |
| 615 | signal.set_wakeup_fd(write.fileno()) |
| 616 | |
| 617 | with captured_stderr() as err: |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 618 | signal.raise_signal(signum) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 619 | |
| 620 | err = err.getvalue() |
Victor Stinner | 686b4b5 | 2018-07-18 18:29:54 +0200 | [diff] [blame] | 621 | if msg not in err: |
| 622 | raise AssertionError("second set_wakeup_fd() test failed, " |
| 623 | "stderr: %r" % err) |
Nathaniel J. Smith | 902ab80 | 2017-12-17 20:10:18 -0800 | [diff] [blame] | 624 | |
| 625 | """.format(action=action) |
| 626 | assert_python_ok('-c', code) |
| 627 | |
Victor Stinner | 1151710 | 2014-07-29 23:31:34 +0200 | [diff] [blame] | 628 | |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 629 | @unittest.skipIf(sys.platform == "win32", "Not valid on Windows") |
pxinwr | 1244c81 | 2020-12-01 05:48:33 +0800 | [diff] [blame] | 630 | @unittest.skipUnless(hasattr(signal, 'siginterrupt'), "needs signal.siginterrupt()") |
Christian Heimes | 8640e74 | 2008-02-23 16:23:06 +0000 | [diff] [blame] | 631 | class SiginterruptTest(unittest.TestCase): |
Jean-Paul Calderone | 9c39bc7 | 2010-05-09 03:25:16 +0000 | [diff] [blame] | 632 | |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 633 | def readpipe_interrupted(self, interrupt): |
Jean-Paul Calderone | 9c39bc7 | 2010-05-09 03:25:16 +0000 | [diff] [blame] | 634 | """Perform a read during which a signal will arrive. Return True if the |
| 635 | read is interrupted by the signal and raises an exception. Return False |
| 636 | if it returns normally. |
| 637 | """ |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 638 | # use a subprocess to have only one thread, to have a timeout on the |
| 639 | # blocking read and to not touch signal handling in this process |
| 640 | code = """if 1: |
| 641 | import errno |
| 642 | import os |
| 643 | import signal |
| 644 | import sys |
Jean-Paul Calderone | 9c39bc7 | 2010-05-09 03:25:16 +0000 | [diff] [blame] | 645 | |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 646 | interrupt = %r |
| 647 | r, w = os.pipe() |
Christian Heimes | 8640e74 | 2008-02-23 16:23:06 +0000 | [diff] [blame] | 648 | |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 649 | def handler(signum, frame): |
Charles-François Natali | 6e6c59b | 2015-02-07 13:27:50 +0000 | [diff] [blame] | 650 | 1 / 0 |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 651 | |
| 652 | signal.signal(signal.SIGALRM, handler) |
| 653 | if interrupt is not None: |
| 654 | signal.siginterrupt(signal.SIGALRM, interrupt) |
| 655 | |
Victor Stinner | dfde0d4 | 2011-07-01 15:58:39 +0200 | [diff] [blame] | 656 | print("ready") |
| 657 | sys.stdout.flush() |
| 658 | |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 659 | # run the test twice |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 660 | try: |
| 661 | for loop in range(2): |
| 662 | # send a SIGALRM in a second (during the read) |
| 663 | signal.alarm(1) |
| 664 | try: |
| 665 | # blocking call: read from a pipe without data |
| 666 | os.read(r, 1) |
Charles-François Natali | 6e6c59b | 2015-02-07 13:27:50 +0000 | [diff] [blame] | 667 | except ZeroDivisionError: |
| 668 | pass |
Victor Stinner | 56e8c29 | 2014-07-21 12:30:22 +0200 | [diff] [blame] | 669 | else: |
| 670 | sys.exit(2) |
| 671 | sys.exit(3) |
| 672 | finally: |
| 673 | os.close(r) |
| 674 | os.close(w) |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 675 | """ % (interrupt,) |
| 676 | with spawn_python('-c', code) as process: |
Christian Heimes | 8640e74 | 2008-02-23 16:23:06 +0000 | [diff] [blame] | 677 | try: |
Victor Stinner | 4527365 | 2011-06-22 22:15:51 +0200 | [diff] [blame] | 678 | # wait until the child process is loaded and has started |
| 679 | first_line = process.stdout.readline() |
| 680 | |
Victor Stinner | 0d63bac | 2019-12-11 11:30:03 +0100 | [diff] [blame] | 681 | stdout, stderr = process.communicate(timeout=support.SHORT_TIMEOUT) |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 682 | except subprocess.TimeoutExpired: |
| 683 | process.kill() |
Christian Heimes | 8640e74 | 2008-02-23 16:23:06 +0000 | [diff] [blame] | 684 | return False |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 685 | else: |
Victor Stinner | 4527365 | 2011-06-22 22:15:51 +0200 | [diff] [blame] | 686 | stdout = first_line + stdout |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 687 | exitcode = process.wait() |
| 688 | if exitcode not in (2, 3): |
Antoine Pitrou | dab4e8a | 2014-05-11 19:05:23 +0200 | [diff] [blame] | 689 | raise Exception("Child error (exit code %s): %r" |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 690 | % (exitcode, stdout)) |
| 691 | return (exitcode == 3) |
Christian Heimes | 8640e74 | 2008-02-23 16:23:06 +0000 | [diff] [blame] | 692 | |
| 693 | def test_without_siginterrupt(self): |
Victor Stinner | 3a7f0f0 | 2011-05-08 02:10:36 +0200 | [diff] [blame] | 694 | # If a signal handler is installed and siginterrupt is not called |
| 695 | # at all, when that signal arrives, it interrupts a syscall that's in |
| 696 | # progress. |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 697 | interrupted = self.readpipe_interrupted(None) |
| 698 | self.assertTrue(interrupted) |
Christian Heimes | 8640e74 | 2008-02-23 16:23:06 +0000 | [diff] [blame] | 699 | |
| 700 | def test_siginterrupt_on(self): |
Victor Stinner | 3a7f0f0 | 2011-05-08 02:10:36 +0200 | [diff] [blame] | 701 | # If a signal handler is installed and siginterrupt is called with |
| 702 | # a true value for the second argument, when that signal arrives, it |
| 703 | # interrupts a syscall that's in progress. |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 704 | interrupted = self.readpipe_interrupted(True) |
| 705 | self.assertTrue(interrupted) |
Christian Heimes | 8640e74 | 2008-02-23 16:23:06 +0000 | [diff] [blame] | 706 | |
| 707 | def test_siginterrupt_off(self): |
Victor Stinner | 3a7f0f0 | 2011-05-08 02:10:36 +0200 | [diff] [blame] | 708 | # If a signal handler is installed and siginterrupt is called with |
| 709 | # a false value for the second argument, when that signal arrives, it |
| 710 | # does not interrupt a syscall that's in progress. |
Victor Stinner | d628496 | 2011-06-20 23:28:09 +0200 | [diff] [blame] | 711 | interrupted = self.readpipe_interrupted(False) |
| 712 | self.assertFalse(interrupted) |
Jean-Paul Calderone | 9c39bc7 | 2010-05-09 03:25:16 +0000 | [diff] [blame] | 713 | |
| 714 | |
Brian Curtin | 3f004b1 | 2010-08-06 19:34:52 +0000 | [diff] [blame] | 715 | @unittest.skipIf(sys.platform == "win32", "Not valid on Windows") |
pxinwr | 1244c81 | 2020-12-01 05:48:33 +0800 | [diff] [blame] | 716 | @unittest.skipUnless(hasattr(signal, 'getitimer') and hasattr(signal, 'setitimer'), |
| 717 | "needs signal.getitimer() and signal.setitimer()") |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 718 | class ItimerTest(unittest.TestCase): |
| 719 | def setUp(self): |
| 720 | self.hndl_called = False |
| 721 | self.hndl_count = 0 |
| 722 | self.itimer = None |
Christian Heimes | cc47b05 | 2008-03-25 14:56:36 +0000 | [diff] [blame] | 723 | self.old_alarm = signal.signal(signal.SIGALRM, self.sig_alrm) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 724 | |
| 725 | def tearDown(self): |
Christian Heimes | cc47b05 | 2008-03-25 14:56:36 +0000 | [diff] [blame] | 726 | signal.signal(signal.SIGALRM, self.old_alarm) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 727 | if self.itimer is not None: # test_itimer_exc doesn't change this attr |
| 728 | # just ensure that itimer is stopped |
| 729 | signal.setitimer(self.itimer, 0) |
| 730 | |
| 731 | def sig_alrm(self, *args): |
| 732 | self.hndl_called = True |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 733 | |
| 734 | def sig_vtalrm(self, *args): |
| 735 | self.hndl_called = True |
| 736 | |
| 737 | if self.hndl_count > 3: |
| 738 | # it shouldn't be here, because it should have been disabled. |
| 739 | raise signal.ItimerError("setitimer didn't disable ITIMER_VIRTUAL " |
| 740 | "timer.") |
| 741 | elif self.hndl_count == 3: |
| 742 | # disable ITIMER_VIRTUAL, this function shouldn't be called anymore |
| 743 | signal.setitimer(signal.ITIMER_VIRTUAL, 0) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 744 | |
| 745 | self.hndl_count += 1 |
| 746 | |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 747 | def sig_prof(self, *args): |
| 748 | self.hndl_called = True |
| 749 | signal.setitimer(signal.ITIMER_PROF, 0) |
| 750 | |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 751 | def test_itimer_exc(self): |
| 752 | # XXX I'm assuming -1 is an invalid itimer, but maybe some platform |
| 753 | # defines it ? |
| 754 | self.assertRaises(signal.ItimerError, signal.setitimer, -1, 0) |
Christian Heimes | cc47b05 | 2008-03-25 14:56:36 +0000 | [diff] [blame] | 755 | # Negative times are treated as zero on some platforms. |
| 756 | if 0: |
| 757 | self.assertRaises(signal.ItimerError, |
| 758 | signal.setitimer, signal.ITIMER_REAL, -1) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 759 | |
| 760 | def test_itimer_real(self): |
| 761 | self.itimer = signal.ITIMER_REAL |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 762 | signal.setitimer(self.itimer, 1.0) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 763 | signal.pause() |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 764 | self.assertEqual(self.hndl_called, True) |
| 765 | |
R. David Murray | 44546f8 | 2010-04-21 01:59:28 +0000 | [diff] [blame] | 766 | # Issue 3864, unknown if this affects earlier versions of freebsd also |
Victor Stinner | 13ff245 | 2018-01-22 18:32:50 +0100 | [diff] [blame] | 767 | @unittest.skipIf(sys.platform in ('netbsd5',), |
Gregory P. Smith | 397cd8a | 2010-10-17 04:23:21 +0000 | [diff] [blame] | 768 | 'itimer not reliable (does not mix well with threading) on some BSDs.') |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 769 | def test_itimer_virtual(self): |
| 770 | self.itimer = signal.ITIMER_VIRTUAL |
| 771 | signal.signal(signal.SIGVTALRM, self.sig_vtalrm) |
| 772 | signal.setitimer(self.itimer, 0.3, 0.2) |
| 773 | |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 774 | start_time = time.monotonic() |
| 775 | while time.monotonic() - start_time < 60.0: |
Mark Dickinson | 9507887 | 2009-10-04 18:47:48 +0000 | [diff] [blame] | 776 | # use up some virtual time by doing real work |
| 777 | _ = pow(12345, 67890, 10000019) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 778 | if signal.getitimer(self.itimer) == (0.0, 0.0): |
| 779 | break # sig_vtalrm handler stopped this itimer |
Stefan Krah | f1da97304 | 2010-04-20 08:15:14 +0000 | [diff] [blame] | 780 | else: # Issue 8424 |
Benjamin Peterson | 9b140f6 | 2010-05-15 18:00:56 +0000 | [diff] [blame] | 781 | self.skipTest("timeout: likely cause: machine too slow or load too " |
| 782 | "high") |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 783 | |
| 784 | # virtual itimer should be (0.0, 0.0) now |
Ezio Melotti | 19f2aeb | 2010-11-21 01:30:29 +0000 | [diff] [blame] | 785 | self.assertEqual(signal.getitimer(self.itimer), (0.0, 0.0)) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 786 | # and the handler should have been called |
Ezio Melotti | 19f2aeb | 2010-11-21 01:30:29 +0000 | [diff] [blame] | 787 | self.assertEqual(self.hndl_called, True) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 788 | |
| 789 | def test_itimer_prof(self): |
| 790 | self.itimer = signal.ITIMER_PROF |
| 791 | signal.signal(signal.SIGPROF, self.sig_prof) |
Neal Norwitz | f5c7c2e | 2008-04-05 04:47:45 +0000 | [diff] [blame] | 792 | signal.setitimer(self.itimer, 0.2, 0.2) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 793 | |
Victor Stinner | 79d68f9 | 2015-03-19 21:54:09 +0100 | [diff] [blame] | 794 | start_time = time.monotonic() |
| 795 | while time.monotonic() - start_time < 60.0: |
Mark Dickinson | 7837347 | 2009-10-31 10:39:21 +0000 | [diff] [blame] | 796 | # do some work |
| 797 | _ = pow(12345, 67890, 10000019) |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 798 | if signal.getitimer(self.itimer) == (0.0, 0.0): |
| 799 | break # sig_prof handler stopped this itimer |
Stefan Krah | f1da97304 | 2010-04-20 08:15:14 +0000 | [diff] [blame] | 800 | else: # Issue 8424 |
Benjamin Peterson | 9b140f6 | 2010-05-15 18:00:56 +0000 | [diff] [blame] | 801 | self.skipTest("timeout: likely cause: machine too slow or load too " |
| 802 | "high") |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 803 | |
Neal Norwitz | f5c7c2e | 2008-04-05 04:47:45 +0000 | [diff] [blame] | 804 | # profiling itimer should be (0.0, 0.0) now |
Ezio Melotti | 19f2aeb | 2010-11-21 01:30:29 +0000 | [diff] [blame] | 805 | self.assertEqual(signal.getitimer(self.itimer), (0.0, 0.0)) |
Neal Norwitz | f5c7c2e | 2008-04-05 04:47:45 +0000 | [diff] [blame] | 806 | # and the handler should have been called |
Martin v. Löwis | 823725e | 2008-03-24 13:39:54 +0000 | [diff] [blame] | 807 | self.assertEqual(self.hndl_called, True) |
| 808 | |
Antoine Pitrou | 729780a | 2017-06-30 10:01:05 +0200 | [diff] [blame] | 809 | def test_setitimer_tiny(self): |
| 810 | # bpo-30807: C setitimer() takes a microsecond-resolution interval. |
| 811 | # Check that float -> timeval conversion doesn't round |
| 812 | # the interval down to zero, which would disable the timer. |
| 813 | self.itimer = signal.ITIMER_REAL |
| 814 | signal.setitimer(self.itimer, 1e-6) |
| 815 | time.sleep(1) |
| 816 | self.assertEqual(self.hndl_called, True) |
| 817 | |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 818 | |
Victor Stinner | 35b300c | 2011-05-04 13:20:35 +0200 | [diff] [blame] | 819 | class PendingSignalsTests(unittest.TestCase): |
| 820 | """ |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 821 | Test pthread_sigmask(), pthread_kill(), sigpending() and sigwait() |
| 822 | functions. |
Victor Stinner | 35b300c | 2011-05-04 13:20:35 +0200 | [diff] [blame] | 823 | """ |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 824 | @unittest.skipUnless(hasattr(signal, 'sigpending'), |
| 825 | 'need signal.sigpending()') |
| 826 | def test_sigpending_empty(self): |
| 827 | self.assertEqual(signal.sigpending(), set()) |
| 828 | |
| 829 | @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'), |
| 830 | 'need signal.pthread_sigmask()') |
| 831 | @unittest.skipUnless(hasattr(signal, 'sigpending'), |
| 832 | 'need signal.sigpending()') |
| 833 | def test_sigpending(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 834 | code = """if 1: |
| 835 | import os |
| 836 | import signal |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 837 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 838 | def handler(signum, frame): |
| 839 | 1/0 |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 840 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 841 | signum = signal.SIGUSR1 |
| 842 | signal.signal(signum, handler) |
| 843 | |
| 844 | signal.pthread_sigmask(signal.SIG_BLOCK, [signum]) |
| 845 | os.kill(os.getpid(), signum) |
| 846 | pending = signal.sigpending() |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 847 | for sig in pending: |
| 848 | assert isinstance(sig, signal.Signals), repr(pending) |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 849 | if pending != {signum}: |
| 850 | raise Exception('%s != {%s}' % (pending, signum)) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 851 | try: |
| 852 | signal.pthread_sigmask(signal.SIG_UNBLOCK, [signum]) |
| 853 | except ZeroDivisionError: |
| 854 | pass |
| 855 | else: |
| 856 | raise Exception("ZeroDivisionError not raised") |
| 857 | """ |
| 858 | assert_python_ok('-c', code) |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 859 | |
| 860 | @unittest.skipUnless(hasattr(signal, 'pthread_kill'), |
| 861 | 'need signal.pthread_kill()') |
| 862 | def test_pthread_kill(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 863 | code = """if 1: |
| 864 | import signal |
| 865 | import threading |
| 866 | import sys |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 867 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 868 | signum = signal.SIGUSR1 |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 869 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 870 | def handler(signum, frame): |
| 871 | 1/0 |
| 872 | |
| 873 | signal.signal(signum, handler) |
| 874 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 875 | tid = threading.get_ident() |
| 876 | try: |
| 877 | signal.pthread_kill(tid, signum) |
| 878 | except ZeroDivisionError: |
| 879 | pass |
| 880 | else: |
| 881 | raise Exception("ZeroDivisionError not raised") |
| 882 | """ |
| 883 | assert_python_ok('-c', code) |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 884 | |
Victor Stinner | 7f294d1 | 2011-06-10 14:02:10 +0200 | [diff] [blame] | 885 | @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'), |
| 886 | 'need signal.pthread_sigmask()') |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 887 | def wait_helper(self, blocked, test): |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 888 | """ |
| 889 | test: body of the "def test(signum):" function. |
| 890 | blocked: number of the blocked signal |
| 891 | """ |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 892 | code = '''if 1: |
| 893 | import signal |
| 894 | import sys |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 895 | from signal import Signals |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 896 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 897 | def handler(signum, frame): |
| 898 | 1/0 |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 899 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 900 | %s |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 901 | |
Ethan Furman | b775106 | 2021-03-30 21:17:26 -0700 | [diff] [blame] | 902 | blocked = %r |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 903 | signum = signal.SIGALRM |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 904 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 905 | # child: block and wait the signal |
| 906 | try: |
| 907 | signal.signal(signum, handler) |
| 908 | signal.pthread_sigmask(signal.SIG_BLOCK, [blocked]) |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 909 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 910 | # Do the tests |
| 911 | test(signum) |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 912 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 913 | # The handler must not be called on unblock |
| 914 | try: |
| 915 | signal.pthread_sigmask(signal.SIG_UNBLOCK, [blocked]) |
| 916 | except ZeroDivisionError: |
| 917 | print("the signal handler has been called", |
| 918 | file=sys.stderr) |
| 919 | sys.exit(1) |
| 920 | except BaseException as err: |
| 921 | print("error: {}".format(err), file=sys.stderr) |
| 922 | sys.stderr.flush() |
| 923 | sys.exit(1) |
| 924 | ''' % (test.strip(), blocked) |
Victor Stinner | 415007e | 2011-06-13 16:19:06 +0200 | [diff] [blame] | 925 | |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 926 | # sig*wait* must be called with the signal blocked: since the current |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 927 | # process might have several threads running, use a subprocess to have |
| 928 | # a single thread. |
| 929 | assert_python_ok('-c', code) |
Victor Stinner | af49460 | 2011-06-10 12:48:13 +0200 | [diff] [blame] | 930 | |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 931 | @unittest.skipUnless(hasattr(signal, 'sigwait'), |
| 932 | 'need signal.sigwait()') |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 933 | def test_sigwait(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 934 | self.wait_helper(signal.SIGALRM, ''' |
| 935 | def test(signum): |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 936 | signal.alarm(1) |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 937 | received = signal.sigwait([signum]) |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 938 | assert isinstance(received, signal.Signals), received |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 939 | if received != signum: |
| 940 | raise Exception('received %s, not %s' % (received, signum)) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 941 | ''') |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 942 | |
| 943 | @unittest.skipUnless(hasattr(signal, 'sigwaitinfo'), |
| 944 | 'need signal.sigwaitinfo()') |
| 945 | def test_sigwaitinfo(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 946 | self.wait_helper(signal.SIGALRM, ''' |
| 947 | def test(signum): |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 948 | signal.alarm(1) |
| 949 | info = signal.sigwaitinfo([signum]) |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 950 | if info.si_signo != signum: |
| 951 | raise Exception("info.si_signo != %s" % signum) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 952 | ''') |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 953 | |
| 954 | @unittest.skipUnless(hasattr(signal, 'sigtimedwait'), |
| 955 | 'need signal.sigtimedwait()') |
| 956 | def test_sigtimedwait(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 957 | self.wait_helper(signal.SIGALRM, ''' |
| 958 | def test(signum): |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 959 | signal.alarm(1) |
Victor Stinner | 643cd68 | 2012-03-02 22:54:03 +0100 | [diff] [blame] | 960 | info = signal.sigtimedwait([signum], 10.1000) |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 961 | if info.si_signo != signum: |
| 962 | raise Exception('info.si_signo != %s' % signum) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 963 | ''') |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 964 | |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 965 | @unittest.skipUnless(hasattr(signal, 'sigtimedwait'), |
| 966 | 'need signal.sigtimedwait()') |
| 967 | def test_sigtimedwait_poll(self): |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 968 | # check that polling with sigtimedwait works |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 969 | self.wait_helper(signal.SIGALRM, ''' |
| 970 | def test(signum): |
Victor Stinner | 9e8b82f | 2011-06-29 10:43:02 +0200 | [diff] [blame] | 971 | import os |
| 972 | os.kill(os.getpid(), signum) |
Victor Stinner | 643cd68 | 2012-03-02 22:54:03 +0100 | [diff] [blame] | 973 | info = signal.sigtimedwait([signum], 0) |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 974 | if info.si_signo != signum: |
| 975 | raise Exception('info.si_signo != %s' % signum) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 976 | ''') |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 977 | |
| 978 | @unittest.skipUnless(hasattr(signal, 'sigtimedwait'), |
| 979 | 'need signal.sigtimedwait()') |
| 980 | def test_sigtimedwait_timeout(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 981 | self.wait_helper(signal.SIGALRM, ''' |
| 982 | def test(signum): |
Victor Stinner | 643cd68 | 2012-03-02 22:54:03 +0100 | [diff] [blame] | 983 | received = signal.sigtimedwait([signum], 1.0) |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 984 | if received is not None: |
| 985 | raise Exception("received=%r" % (received,)) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 986 | ''') |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 987 | |
| 988 | @unittest.skipUnless(hasattr(signal, 'sigtimedwait'), |
| 989 | 'need signal.sigtimedwait()') |
| 990 | def test_sigtimedwait_negative_timeout(self): |
| 991 | signum = signal.SIGALRM |
Victor Stinner | 643cd68 | 2012-03-02 22:54:03 +0100 | [diff] [blame] | 992 | self.assertRaises(ValueError, signal.sigtimedwait, [signum], -1.0) |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 993 | |
Ross Lagerwall | bc80822 | 2011-06-25 12:13:40 +0200 | [diff] [blame] | 994 | @unittest.skipUnless(hasattr(signal, 'sigwait'), |
| 995 | 'need signal.sigwait()') |
Victor Stinner | 415007e | 2011-06-13 16:19:06 +0200 | [diff] [blame] | 996 | @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'), |
| 997 | 'need signal.pthread_sigmask()') |
Victor Stinner | 10c30d6 | 2011-06-10 01:39:53 +0200 | [diff] [blame] | 998 | def test_sigwait_thread(self): |
Victor Stinner | 415007e | 2011-06-13 16:19:06 +0200 | [diff] [blame] | 999 | # Check that calling sigwait() from a thread doesn't suspend the whole |
| 1000 | # process. A new interpreter is spawned to avoid problems when mixing |
| 1001 | # threads and fork(): only async-safe functions are allowed between |
| 1002 | # fork() and exec(). |
| 1003 | assert_python_ok("-c", """if True: |
| 1004 | import os, threading, sys, time, signal |
Victor Stinner | 10c30d6 | 2011-06-10 01:39:53 +0200 | [diff] [blame] | 1005 | |
Victor Stinner | 415007e | 2011-06-13 16:19:06 +0200 | [diff] [blame] | 1006 | # the default handler terminates the process |
| 1007 | signum = signal.SIGUSR1 |
| 1008 | |
| 1009 | def kill_later(): |
| 1010 | # wait until the main thread is waiting in sigwait() |
| 1011 | time.sleep(1) |
| 1012 | os.kill(os.getpid(), signum) |
| 1013 | |
| 1014 | # the signal must be blocked by all the threads |
| 1015 | signal.pthread_sigmask(signal.SIG_BLOCK, [signum]) |
| 1016 | killer = threading.Thread(target=kill_later) |
Victor Stinner | af49460 | 2011-06-10 12:48:13 +0200 | [diff] [blame] | 1017 | killer.start() |
| 1018 | received = signal.sigwait([signum]) |
| 1019 | if received != signum: |
| 1020 | print("sigwait() received %s, not %s" % (received, signum), |
| 1021 | file=sys.stderr) |
Victor Stinner | 415007e | 2011-06-13 16:19:06 +0200 | [diff] [blame] | 1022 | sys.exit(1) |
Victor Stinner | 10c30d6 | 2011-06-10 01:39:53 +0200 | [diff] [blame] | 1023 | killer.join() |
Victor Stinner | 415007e | 2011-06-13 16:19:06 +0200 | [diff] [blame] | 1024 | # unblock the signal, which should have been cleared by sigwait() |
| 1025 | signal.pthread_sigmask(signal.SIG_UNBLOCK, [signum]) |
| 1026 | """) |
Victor Stinner | af49460 | 2011-06-10 12:48:13 +0200 | [diff] [blame] | 1027 | |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 1028 | @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'), |
| 1029 | 'need signal.pthread_sigmask()') |
| 1030 | def test_pthread_sigmask_arguments(self): |
| 1031 | self.assertRaises(TypeError, signal.pthread_sigmask) |
| 1032 | self.assertRaises(TypeError, signal.pthread_sigmask, 1) |
| 1033 | self.assertRaises(TypeError, signal.pthread_sigmask, 1, 2, 3) |
| 1034 | self.assertRaises(OSError, signal.pthread_sigmask, 1700, []) |
Antoine Pitrou | 9d3627e | 2018-05-04 13:00:50 +0200 | [diff] [blame] | 1035 | with self.assertRaises(ValueError): |
| 1036 | signal.pthread_sigmask(signal.SIG_BLOCK, [signal.NSIG]) |
Serhiy Storchaka | d54cfb1 | 2018-05-08 07:48:50 +0300 | [diff] [blame] | 1037 | with self.assertRaises(ValueError): |
| 1038 | signal.pthread_sigmask(signal.SIG_BLOCK, [0]) |
| 1039 | with self.assertRaises(ValueError): |
| 1040 | signal.pthread_sigmask(signal.SIG_BLOCK, [1<<1000]) |
Antoine Pitrou | 9d3627e | 2018-05-04 13:00:50 +0200 | [diff] [blame] | 1041 | |
| 1042 | @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'), |
| 1043 | 'need signal.pthread_sigmask()') |
| 1044 | def test_pthread_sigmask_valid_signals(self): |
| 1045 | s = signal.pthread_sigmask(signal.SIG_BLOCK, signal.valid_signals()) |
| 1046 | self.addCleanup(signal.pthread_sigmask, signal.SIG_SETMASK, s) |
| 1047 | # Get current blocked set |
| 1048 | s = signal.pthread_sigmask(signal.SIG_UNBLOCK, signal.valid_signals()) |
| 1049 | self.assertLessEqual(s, signal.valid_signals()) |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 1050 | |
| 1051 | @unittest.skipUnless(hasattr(signal, 'pthread_sigmask'), |
| 1052 | 'need signal.pthread_sigmask()') |
| 1053 | def test_pthread_sigmask(self): |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1054 | code = """if 1: |
| 1055 | import signal |
| 1056 | import os; import threading |
| 1057 | |
| 1058 | def handler(signum, frame): |
| 1059 | 1/0 |
| 1060 | |
| 1061 | def kill(signum): |
| 1062 | os.kill(os.getpid(), signum) |
| 1063 | |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 1064 | def check_mask(mask): |
| 1065 | for sig in mask: |
| 1066 | assert isinstance(sig, signal.Signals), repr(sig) |
| 1067 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1068 | def read_sigmask(): |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 1069 | sigmask = signal.pthread_sigmask(signal.SIG_BLOCK, []) |
| 1070 | check_mask(sigmask) |
| 1071 | return sigmask |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1072 | |
Victor Stinner | b3e7219 | 2011-05-08 01:46:11 +0200 | [diff] [blame] | 1073 | signum = signal.SIGUSR1 |
Victor Stinner | 6fd49e1 | 2011-05-04 12:38:03 +0200 | [diff] [blame] | 1074 | |
Victor Stinner | d0e516d | 2011-05-03 14:57:12 +0200 | [diff] [blame] | 1075 | # Install our signal handler |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1076 | old_handler = signal.signal(signum, handler) |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 1077 | |
Victor Stinner | d0e516d | 2011-05-03 14:57:12 +0200 | [diff] [blame] | 1078 | # Unblock SIGUSR1 (and copy the old mask) to test our signal handler |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 1079 | old_mask = signal.pthread_sigmask(signal.SIG_UNBLOCK, [signum]) |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 1080 | check_mask(old_mask) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1081 | try: |
| 1082 | kill(signum) |
| 1083 | except ZeroDivisionError: |
| 1084 | pass |
| 1085 | else: |
| 1086 | raise Exception("ZeroDivisionError not raised") |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 1087 | |
Victor Stinner | d0e516d | 2011-05-03 14:57:12 +0200 | [diff] [blame] | 1088 | # Block and then raise SIGUSR1. The signal is blocked: the signal |
| 1089 | # handler is not called, and the signal is now pending |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 1090 | mask = signal.pthread_sigmask(signal.SIG_BLOCK, [signum]) |
| 1091 | check_mask(mask) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1092 | kill(signum) |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 1093 | |
Victor Stinner | d0e516d | 2011-05-03 14:57:12 +0200 | [diff] [blame] | 1094 | # Check the new mask |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1095 | blocked = read_sigmask() |
Giampaolo Rodola' | e09fb71 | 2014-04-04 15:34:17 +0200 | [diff] [blame] | 1096 | check_mask(blocked) |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 1097 | if signum not in blocked: |
| 1098 | raise Exception("%s not in %s" % (signum, blocked)) |
| 1099 | if old_mask ^ blocked != {signum}: |
| 1100 | raise Exception("%s ^ %s != {%s}" % (old_mask, blocked, signum)) |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 1101 | |
Victor Stinner | d0e516d | 2011-05-03 14:57:12 +0200 | [diff] [blame] | 1102 | # Unblock SIGUSR1 |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1103 | try: |
R David Murray | fc06999 | 2013-12-13 20:52:19 -0500 | [diff] [blame] | 1104 | # unblock the pending signal calls immediately the signal handler |
Victor Stinner | d0e516d | 2011-05-03 14:57:12 +0200 | [diff] [blame] | 1105 | signal.pthread_sigmask(signal.SIG_UNBLOCK, [signum]) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1106 | except ZeroDivisionError: |
| 1107 | pass |
| 1108 | else: |
| 1109 | raise Exception("ZeroDivisionError not raised") |
| 1110 | try: |
| 1111 | kill(signum) |
| 1112 | except ZeroDivisionError: |
| 1113 | pass |
| 1114 | else: |
| 1115 | raise Exception("ZeroDivisionError not raised") |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 1116 | |
Victor Stinner | d0e516d | 2011-05-03 14:57:12 +0200 | [diff] [blame] | 1117 | # Check the new mask |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1118 | unblocked = read_sigmask() |
Victor Stinner | 0a01f13 | 2011-07-04 18:06:35 +0200 | [diff] [blame] | 1119 | if signum in unblocked: |
| 1120 | raise Exception("%s in %s" % (signum, unblocked)) |
| 1121 | if blocked ^ unblocked != {signum}: |
| 1122 | raise Exception("%s ^ %s != {%s}" % (blocked, unblocked, signum)) |
| 1123 | if old_mask != unblocked: |
| 1124 | raise Exception("%s != %s" % (old_mask, unblocked)) |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1125 | """ |
| 1126 | assert_python_ok('-c', code) |
| 1127 | |
Victor Stinner | d554cdf | 2011-07-04 17:49:40 +0200 | [diff] [blame] | 1128 | @unittest.skipUnless(hasattr(signal, 'pthread_kill'), |
| 1129 | 'need signal.pthread_kill()') |
| 1130 | def test_pthread_kill_main_thread(self): |
| 1131 | # Test that a signal can be sent to the main thread with pthread_kill() |
| 1132 | # before any other thread has been created (see issue #12392). |
| 1133 | code = """if True: |
| 1134 | import threading |
| 1135 | import signal |
| 1136 | import sys |
| 1137 | |
| 1138 | def handler(signum, frame): |
| 1139 | sys.exit(3) |
| 1140 | |
| 1141 | signal.signal(signal.SIGUSR1, handler) |
| 1142 | signal.pthread_kill(threading.get_ident(), signal.SIGUSR1) |
| 1143 | sys.exit(2) |
| 1144 | """ |
| 1145 | |
| 1146 | with spawn_python('-c', code) as process: |
| 1147 | stdout, stderr = process.communicate() |
| 1148 | exitcode = process.wait() |
| 1149 | if exitcode != 3: |
| 1150 | raise Exception("Child error (exit code %s): %s" % |
| 1151 | (exitcode, stdout)) |
Victor Stinner | a929335 | 2011-04-30 15:21:58 +0200 | [diff] [blame] | 1152 | |
| 1153 | |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1154 | class StressTest(unittest.TestCase): |
| 1155 | """ |
| 1156 | Stress signal delivery, especially when a signal arrives in |
| 1157 | the middle of recomputing the signal state or executing |
| 1158 | previously tripped signal handlers. |
| 1159 | """ |
| 1160 | |
Antoine Pitrou | f7d090c | 2017-06-29 16:40:14 +0200 | [diff] [blame] | 1161 | def setsig(self, signum, handler): |
| 1162 | old_handler = signal.signal(signum, handler) |
| 1163 | self.addCleanup(signal.signal, signum, old_handler) |
| 1164 | |
| 1165 | def measure_itimer_resolution(self): |
| 1166 | N = 20 |
| 1167 | times = [] |
| 1168 | |
| 1169 | def handler(signum=None, frame=None): |
| 1170 | if len(times) < N: |
| 1171 | times.append(time.perf_counter()) |
| 1172 | # 1 µs is the smallest possible timer interval, |
| 1173 | # we want to measure what the concrete duration |
| 1174 | # will be on this platform |
| 1175 | signal.setitimer(signal.ITIMER_REAL, 1e-6) |
| 1176 | |
| 1177 | self.addCleanup(signal.setitimer, signal.ITIMER_REAL, 0) |
| 1178 | self.setsig(signal.SIGALRM, handler) |
| 1179 | handler() |
| 1180 | while len(times) < N: |
| 1181 | time.sleep(1e-3) |
| 1182 | |
| 1183 | durations = [times[i+1] - times[i] for i in range(len(times) - 1)] |
| 1184 | med = statistics.median(durations) |
| 1185 | if support.verbose: |
| 1186 | print("detected median itimer() resolution: %.6f s." % (med,)) |
| 1187 | return med |
| 1188 | |
| 1189 | def decide_itimer_count(self): |
| 1190 | # Some systems have poor setitimer() resolution (for example |
| 1191 | # measured around 20 ms. on FreeBSD 9), so decide on a reasonable |
| 1192 | # number of sequential timers based on that. |
| 1193 | reso = self.measure_itimer_resolution() |
| 1194 | if reso <= 1e-4: |
| 1195 | return 10000 |
| 1196 | elif reso <= 1e-2: |
| 1197 | return 100 |
| 1198 | else: |
| 1199 | self.skipTest("detected itimer resolution (%.3f s.) too high " |
| 1200 | "(> 10 ms.) on this platform (or system too busy)" |
| 1201 | % (reso,)) |
| 1202 | |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1203 | @unittest.skipUnless(hasattr(signal, "setitimer"), |
| 1204 | "test needs setitimer()") |
| 1205 | def test_stress_delivery_dependent(self): |
| 1206 | """ |
| 1207 | This test uses dependent signal handlers. |
| 1208 | """ |
Antoine Pitrou | f7d090c | 2017-06-29 16:40:14 +0200 | [diff] [blame] | 1209 | N = self.decide_itimer_count() |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1210 | sigs = [] |
| 1211 | |
| 1212 | def first_handler(signum, frame): |
| 1213 | # 1e-6 is the minimum non-zero value for `setitimer()`. |
| 1214 | # Choose a random delay so as to improve chances of |
| 1215 | # triggering a race condition. Ideally the signal is received |
| 1216 | # when inside critical signal-handling routines such as |
| 1217 | # Py_MakePendingCalls(). |
| 1218 | signal.setitimer(signal.ITIMER_REAL, 1e-6 + random.random() * 1e-5) |
| 1219 | |
| 1220 | def second_handler(signum=None, frame=None): |
| 1221 | sigs.append(signum) |
| 1222 | |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1223 | # Here on Linux, SIGPROF > SIGALRM > SIGUSR1. By using both |
| 1224 | # ascending and descending sequences (SIGUSR1 then SIGALRM, |
| 1225 | # SIGPROF then SIGALRM), we maximize chances of hitting a bug. |
Antoine Pitrou | f7d090c | 2017-06-29 16:40:14 +0200 | [diff] [blame] | 1226 | self.setsig(signal.SIGPROF, first_handler) |
| 1227 | self.setsig(signal.SIGUSR1, first_handler) |
| 1228 | self.setsig(signal.SIGALRM, second_handler) # for ITIMER_REAL |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1229 | |
| 1230 | expected_sigs = 0 |
Victor Stinner | 0d63bac | 2019-12-11 11:30:03 +0100 | [diff] [blame] | 1231 | deadline = time.monotonic() + support.SHORT_TIMEOUT |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1232 | |
| 1233 | while expected_sigs < N: |
| 1234 | os.kill(os.getpid(), signal.SIGPROF) |
| 1235 | expected_sigs += 1 |
| 1236 | # Wait for handlers to run to avoid signal coalescing |
Victor Stinner | 2cf4c20 | 2018-12-17 09:36:36 +0100 | [diff] [blame] | 1237 | while len(sigs) < expected_sigs and time.monotonic() < deadline: |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1238 | time.sleep(1e-5) |
| 1239 | |
| 1240 | os.kill(os.getpid(), signal.SIGUSR1) |
| 1241 | expected_sigs += 1 |
Victor Stinner | 2cf4c20 | 2018-12-17 09:36:36 +0100 | [diff] [blame] | 1242 | while len(sigs) < expected_sigs and time.monotonic() < deadline: |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1243 | time.sleep(1e-5) |
| 1244 | |
| 1245 | # All ITIMER_REAL signals should have been delivered to the |
| 1246 | # Python handler |
| 1247 | self.assertEqual(len(sigs), N, "Some signals were lost") |
| 1248 | |
| 1249 | @unittest.skipUnless(hasattr(signal, "setitimer"), |
| 1250 | "test needs setitimer()") |
| 1251 | def test_stress_delivery_simultaneous(self): |
| 1252 | """ |
| 1253 | This test uses simultaneous signal handlers. |
| 1254 | """ |
Antoine Pitrou | f7d090c | 2017-06-29 16:40:14 +0200 | [diff] [blame] | 1255 | N = self.decide_itimer_count() |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1256 | sigs = [] |
| 1257 | |
| 1258 | def handler(signum, frame): |
| 1259 | sigs.append(signum) |
| 1260 | |
Antoine Pitrou | f7d090c | 2017-06-29 16:40:14 +0200 | [diff] [blame] | 1261 | self.setsig(signal.SIGUSR1, handler) |
| 1262 | self.setsig(signal.SIGALRM, handler) # for ITIMER_REAL |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1263 | |
| 1264 | expected_sigs = 0 |
Victor Stinner | 0d63bac | 2019-12-11 11:30:03 +0100 | [diff] [blame] | 1265 | deadline = time.monotonic() + support.SHORT_TIMEOUT |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1266 | |
| 1267 | while expected_sigs < N: |
| 1268 | # Hopefully the SIGALRM will be received somewhere during |
| 1269 | # initial processing of SIGUSR1. |
| 1270 | signal.setitimer(signal.ITIMER_REAL, 1e-6 + random.random() * 1e-5) |
| 1271 | os.kill(os.getpid(), signal.SIGUSR1) |
| 1272 | |
| 1273 | expected_sigs += 2 |
| 1274 | # Wait for handlers to run to avoid signal coalescing |
Victor Stinner | 2cf4c20 | 2018-12-17 09:36:36 +0100 | [diff] [blame] | 1275 | while len(sigs) < expected_sigs and time.monotonic() < deadline: |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1276 | time.sleep(1e-5) |
| 1277 | |
| 1278 | # All ITIMER_REAL signals should have been delivered to the |
| 1279 | # Python handler |
| 1280 | self.assertEqual(len(sigs), N, "Some signals were lost") |
| 1281 | |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 1282 | @unittest.skipUnless(hasattr(signal, "SIGUSR1"), |
| 1283 | "test needs SIGUSR1") |
| 1284 | def test_stress_modifying_handlers(self): |
| 1285 | # bpo-43406: race condition between trip_signal() and signal.signal |
| 1286 | signum = signal.SIGUSR1 |
| 1287 | num_sent_signals = 0 |
| 1288 | num_received_signals = 0 |
| 1289 | do_stop = False |
| 1290 | |
| 1291 | def custom_handler(signum, frame): |
| 1292 | nonlocal num_received_signals |
| 1293 | num_received_signals += 1 |
| 1294 | |
| 1295 | def set_interrupts(): |
| 1296 | nonlocal num_sent_signals |
| 1297 | while not do_stop: |
| 1298 | signal.raise_signal(signum) |
| 1299 | num_sent_signals += 1 |
| 1300 | |
| 1301 | def cycle_handlers(): |
| 1302 | while num_sent_signals < 100: |
| 1303 | for i in range(20000): |
| 1304 | # Cycle between a Python-defined and a non-Python handler |
| 1305 | for handler in [custom_handler, signal.SIG_IGN]: |
| 1306 | signal.signal(signum, handler) |
| 1307 | |
| 1308 | old_handler = signal.signal(signum, custom_handler) |
| 1309 | self.addCleanup(signal.signal, signum, old_handler) |
Victor Stinner | 1fa17e8 | 2021-03-10 15:26:45 +0100 | [diff] [blame] | 1310 | |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 1311 | t = threading.Thread(target=set_interrupts) |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 1312 | try: |
Victor Stinner | 1fa17e8 | 2021-03-10 15:26:45 +0100 | [diff] [blame] | 1313 | ignored = False |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 1314 | with support.catch_unraisable_exception() as cm: |
Victor Stinner | 1fa17e8 | 2021-03-10 15:26:45 +0100 | [diff] [blame] | 1315 | t.start() |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 1316 | cycle_handlers() |
Victor Stinner | 1fa17e8 | 2021-03-10 15:26:45 +0100 | [diff] [blame] | 1317 | do_stop = True |
| 1318 | t.join() |
| 1319 | |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 1320 | if cm.unraisable is not None: |
| 1321 | # An unraisable exception may be printed out when |
| 1322 | # a signal is ignored due to the aforementioned |
| 1323 | # race condition, check it. |
| 1324 | self.assertIsInstance(cm.unraisable.exc_value, OSError) |
| 1325 | self.assertIn( |
| 1326 | f"Signal {signum} ignored due to race condition", |
| 1327 | str(cm.unraisable.exc_value)) |
Victor Stinner | 1fa17e8 | 2021-03-10 15:26:45 +0100 | [diff] [blame] | 1328 | ignored = True |
| 1329 | |
| 1330 | # bpo-43406: Even if it is unlikely, it's technically possible that |
| 1331 | # all signals were ignored because of race conditions. |
| 1332 | if not ignored: |
| 1333 | # Sanity check that some signals were received, but not all |
| 1334 | self.assertGreater(num_received_signals, 0) |
Antoine Pitrou | 68245b7 | 2021-03-05 10:32:50 +0100 | [diff] [blame] | 1335 | self.assertLess(num_received_signals, num_sent_signals) |
| 1336 | finally: |
| 1337 | do_stop = True |
| 1338 | t.join() |
| 1339 | |
| 1340 | |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 1341 | class RaiseSignalTest(unittest.TestCase): |
| 1342 | |
| 1343 | def test_sigint(self): |
Gregory P. Smith | 38f11cc | 2019-02-16 12:57:40 -0800 | [diff] [blame] | 1344 | with self.assertRaises(KeyboardInterrupt): |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 1345 | signal.raise_signal(signal.SIGINT) |
Vladimir Matveev | c24c6c2 | 2019-01-08 01:58:25 -0800 | [diff] [blame] | 1346 | |
| 1347 | @unittest.skipIf(sys.platform != "win32", "Windows specific test") |
| 1348 | def test_invalid_argument(self): |
| 1349 | try: |
| 1350 | SIGHUP = 1 # not supported on win32 |
| 1351 | signal.raise_signal(SIGHUP) |
| 1352 | self.fail("OSError (Invalid argument) expected") |
| 1353 | except OSError as e: |
| 1354 | if e.errno == errno.EINVAL: |
| 1355 | pass |
| 1356 | else: |
| 1357 | raise |
| 1358 | |
| 1359 | def test_handler(self): |
| 1360 | is_ok = False |
| 1361 | def handler(a, b): |
| 1362 | nonlocal is_ok |
| 1363 | is_ok = True |
| 1364 | old_signal = signal.signal(signal.SIGINT, handler) |
| 1365 | self.addCleanup(signal.signal, signal.SIGINT, old_signal) |
| 1366 | |
| 1367 | signal.raise_signal(signal.SIGINT) |
| 1368 | self.assertTrue(is_ok) |
| 1369 | |
Antoine Pitrou | c08177a | 2017-06-28 23:29:29 +0200 | [diff] [blame] | 1370 | |
Benjamin Peterson | 7483451 | 2019-11-19 20:39:14 -0800 | [diff] [blame] | 1371 | class PidfdSignalTest(unittest.TestCase): |
| 1372 | |
| 1373 | @unittest.skipUnless( |
| 1374 | hasattr(signal, "pidfd_send_signal"), |
| 1375 | "pidfd support not built in", |
| 1376 | ) |
| 1377 | def test_pidfd_send_signal(self): |
| 1378 | with self.assertRaises(OSError) as cm: |
| 1379 | signal.pidfd_send_signal(0, signal.SIGINT) |
| 1380 | if cm.exception.errno == errno.ENOSYS: |
| 1381 | self.skipTest("kernel does not support pidfds") |
Pablo Galindo | 8f0703f | 2019-12-29 21:35:54 +0000 | [diff] [blame] | 1382 | elif cm.exception.errno == errno.EPERM: |
| 1383 | self.skipTest("Not enough privileges to use pidfs") |
Benjamin Peterson | 7483451 | 2019-11-19 20:39:14 -0800 | [diff] [blame] | 1384 | self.assertEqual(cm.exception.errno, errno.EBADF) |
| 1385 | my_pidfd = os.open(f'/proc/{os.getpid()}', os.O_DIRECTORY) |
| 1386 | self.addCleanup(os.close, my_pidfd) |
damani42 | 38c878b | 2020-01-30 11:26:22 +0100 | [diff] [blame] | 1387 | with self.assertRaisesRegex(TypeError, "^siginfo must be None$"): |
Benjamin Peterson | 7483451 | 2019-11-19 20:39:14 -0800 | [diff] [blame] | 1388 | signal.pidfd_send_signal(my_pidfd, signal.SIGINT, object(), 0) |
| 1389 | with self.assertRaises(KeyboardInterrupt): |
| 1390 | signal.pidfd_send_signal(my_pidfd, signal.SIGINT) |
| 1391 | |
Zachary Ware | 38c707e | 2015-04-13 15:00:43 -0500 | [diff] [blame] | 1392 | def tearDownModule(): |
| 1393 | support.reap_children() |
Thomas Wouters | ed03b41 | 2007-08-28 21:37:11 +0000 | [diff] [blame] | 1394 | |
| 1395 | if __name__ == "__main__": |
Zachary Ware | 38c707e | 2015-04-13 15:00:43 -0500 | [diff] [blame] | 1396 | unittest.main() |