Issue23948
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2015-04-14 15:24 by jpe, last changed 2022-04-11 14:58 by admin. This issue is now closed.
Messages (10) | |||
---|---|---|---|
msg240905 - (view) | Author: John Ehresman (jpe) * | Date: 2015-04-14 15:24 | |
os.kill() on Windows cannot act like it does on non-windows platforms because of differences in the underlying platforms. I think only kill() with a signal number of 9 (terminate process unconditionally) and a signal number of 0 (test to see if process exists) could be implemented. It isn't possibly to send the INT signal or a HUP signal to another process -- or at least it isn't possible without the use of questionable api's and code. Windows specific functions would be added for terminateprocess() and generateconsolectrlevent(). os.kill() on non-windows platforms would be left alone. |
|||
msg241067 - (view) | Author: Steve Dower (steve.dower) * | Date: 2015-04-15 01:08 | |
This feels like an unnecessary incompatibility between the platforms. I'd rather change the parameter values for CTRL+C events so we can distinguish when someone calls with that and then fix it internally on Windows. |
|||
msg241118 - (view) | Author: John Ehresman (jpe) * | Date: 2015-04-15 15:33 | |
Part of the issue here is that GenerateConsoleCtrlEvent doesn't work like kill w/ SIGINT does on unix -- it only works if the the target process id is 0 and then it generates the signal in all processes that share the console. An alternate proposal here is to expose the GenerateConsoleCtrlEvent functionality separately and only accept a signal number of 9 and maybe 0 in kill(). |
|||
msg241134 - (view) | Author: Eryk Sun (eryksun) * | Date: 2015-04-15 17:41 | |
> it only works if the the target process id is 0 and then it > generates the signal in all processes that share the console. You can target a process group, which is a subset of the console's attached processes. That's why the console host, conhost.exe, routes dispatching the event through the session CSRSS.EXE, which handles bookkeeping of Windows processes and threads, including groups. CSRSS creates a thread in each target, which starts at kernel32!CtrlRoutine. If a process is being debugged this routine raises a DBG_CONTROL_C or DBG_CONTROL_BREAK exception. Next it calls the registered handlers. The default handler, if called, exits the process with the exit code set to STATUS_CONTROL_C_EXIT (0xC000013A): 0:000> u kernel32!DefaultHandler l3 kernel32!DefaultHandler: 00000000`76f31290 4883ec28 sub rsp,28h 00000000`76f31294 b93a0100c0 mov ecx,0C000013Ah 00000000`76f31299 ff15e1bb0500 call qword ptr [kernel32! _imp_RtlExitUserProcess (00000000`76f8ce80)] Example: import os import sys import signal import subprocess import threading STATUS_CONTROL_C_EXIT = 0xC000013A p = subprocess.Popen([sys.executable, '-i'], stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.PIPE, creationflags=subprocess.CREATE_NEW_PROCESS_GROUP) p.stderr.read(1) # poor man's WaitForInputIdle os.kill(p.pid, signal.CTRL_BREAK_EVENT) watchdog = threading.Timer(5, p.terminate) watchdog.start() exitcode = p.wait() watchdog.cancel() if exitcode < 0: exitcode += 2 ** 32 assert exitcode == STATUS_CONTROL_C_EXIT |
|||
msg241135 - (view) | Author: John Ehresman (jpe) * | Date: 2015-04-15 17:53 | |
GenerateConsoleCtrlEvent has different limitations for CTRL_BREAK_EVENT and CTRL_C_EVENT according to MSDN; I was referring to the CTRL_C_EVENT limitations. Which python level signal handler will CTRL_BREAK_EVENT trigger? |
|||
msg241156 - (view) | Author: Eryk Sun (eryksun) * | Date: 2015-04-15 20:27 | |
> Which python level signal handler will CTRL_BREAK_EVENT > trigger? The CRT maps it to a non-standard signal, SIGBREAK (21). It's defined in the signal module. > GenerateConsoleCtrlEvent has different limitations for > CTRL_BREAK_EVENT and CTRL_C_EVENT according to MSDN; > I was referring to the CTRL_C_EVENT limitations. When creating a process group, the ConsoleFlags value in the process parameters is set to ignore CTRL_C_EVENT. You'll still get a DBG_CONTROL_C exception if a debugger is attached, but otherwise kernel32!CtrlRoutine ignores Ctrl+C. As you say, the docs state that "[CTRL_C_EVENT] cannot be generated for process groups". But it *does* work if you remove the ignore flag. This flag gets set/unset by assigning/removing a NULL handler. Here's a revised example to enable Ctrl+C in the child, disable Python's INT handler, and then generate CTRL_C_EVENT to kill the child and verify that the exit code is STATUS_CONTROL_C_EXIT. import os import sys import signal import subprocess import threading STATUS_CONTROL_C_EXIT = 0xC000013A p = subprocess.Popen([sys.executable, '-i', '-c', 'import ctypes, signal;' 'kernel32 = ctypes.windll.kernel32;' 'kernel32.SetConsoleCtrlHandler(None, 0);' 'signal.signal(signal.SIGINT, signal.SIG_DFL)'], stdin=subprocess.PIPE, stdout=subprocess.PIPE, stderr=subprocess.PIPE, creationflags=subprocess.CREATE_NEW_PROCESS_GROUP) p.stderr.read(1) # poor man's WaitForInputIdle os.kill(p.pid, signal.CTRL_C_EVENT) watchdog = threading.Timer(5, p.terminate) watchdog.start() exitcode = p.wait() watchdog.cancel() if exitcode < 0: exitcode += 2 ** 32 assert exitcode == STATUS_CONTROL_C_EXIT |
|||
msg241158 - (view) | Author: John Ehresman (jpe) * | Date: 2015-04-15 20:43 | |
Interesting -- I didn't know about removing the ignore flag in the child process. The question is whether this is close enough to the kill w/ SIGINT behavior on unix to use the same name. I think that there are enough differences to warrant a Windows specific function that calls GenerateConsoleCtrlEvent. |
|||
msg260183 - (view) | Author: Giampaolo Rodola' (giampaolo.rodola) * | Date: 2016-02-12 14:16 | |
See also #26350. |
|||
msg389143 - (view) | Author: Eryk Sun (eryksun) * | Date: 2021-03-20 04:05 | |
I'd prefer to change os.kill() to take the code path that generates a console control event only when the pid value is negative (i.e. a process group ID), with -1 reserved to send the event to all processes in the console session (i.e. console process group 0). Also, in this case, I'd map SIGINT and SIGBREAK to CTRL_C_EVENT and CTRL_BREAK_EVENT, instead of directly using the Windows API constants. Such a design conforms better with POSIX kill() [1], and it cleanly separates the GenerateConsoleCtrlEvent() and TerminateProcess() usage. The older usage with kill(non_negative_pid, console_control_event) could be retained by checking for the enum instances signal.CTRL_C_EVENT and signal.CTRL_BREAK_EVENT by object ID instead of by value. --- [1] https://pubs.opengroup.org/onlinepubs/9699919799/functions/kill.html |
|||
msg389166 - (view) | Author: John Ehresman (jpe) * | Date: 2021-03-20 15:49 | |
The original idea was to not use kill on win32 because developers often assume it will work like it does on unix-like OS's -- my claim is while kill could be improved on win32, it still won't support all the things kill can do elsewhere. I don't think this idea has much, if any support so I suggest closing this bug and opening other specific bugs to improve it on win32 for the ideas mentioned in the comments. |
History | |||
---|---|---|---|
Date | User | Action | Args |
2022-04-11 14:58:15 | admin | set | github: 68136 |
2021-12-11 21:00:22 | eryksun | set | status: open -> closed resolution: rejected stage: needs patch -> resolved |
2021-03-20 15:49:52 | jpe | set | messages: + msg389166 |
2021-03-20 04:05:29 | eryksun | set | versions:
+ Python 3.8, Python 3.9, Python 3.10, - Python 3.6 nosy: + paul.moore messages: + msg389143 components: + Extension Modules |
2021-03-20 03:51:30 | eryksun | set | messages: - msg256166 |
2016-02-12 14:16:44 | giampaolo.rodola | set | nosy:
+ giampaolo.rodola messages: + msg260183 |
2015-12-10 07:20:10 | eryksun | set | stage: needs patch messages: + msg256166 versions: + Python 3.6, - Python 3.5 |
2015-04-15 20:43:57 | jpe | set | messages: + msg241158 |
2015-04-15 20:27:26 | eryksun | set | messages: + msg241156 |
2015-04-15 17:53:56 | jpe | set | messages: + msg241135 |
2015-04-15 17:41:43 | eryksun | set | nosy:
+ eryksun messages: + msg241134 |
2015-04-15 15:33:37 | jpe | set | messages: + msg241118 |
2015-04-15 01:08:17 | steve.dower | set | messages: + msg241067 |
2015-04-14 15:24:21 | jpe | create |