Patrick Williams | 03514f1 | 2024-04-05 07:04:11 -0500 | [diff] [blame] | 1 | From c645c83628b2290855cbd225e13c038ab75a7f74 Mon Sep 17 00:00:00 2001 |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 2 | From: Hongxu Jia <hongxu.jia@windriver.com> |
| 3 | Date: Fri, 26 Aug 2016 02:02:49 -0400 |
| 4 | Subject: [PATCH 05/11] fix incorrect timeout while system time changed |
| 5 | |
| 6 | While system time changed by NTP, invoking timeout_command |
| 7 | breaks with incorrect timeout. |
| 8 | -------- |
| 9 | |05:40:55,872 INFO program: Running... mount -t ext2 -o |
| 10 | defaults,ro /dev/sda2 /mnt/sysimage |
| 11 | |01:40:55,086 DEBUG program: 10 seconds timeout |
| 12 | -------- |
| 13 | |
| 14 | Use numbert count to replace current time count could workaround |
| 15 | the issue. |
| 16 | |
| 17 | Upstream-Status: Pending |
| 18 | |
| 19 | Signed-off-by: Hongxu Jia <hongxu.jia@windriver.com> |
| 20 | --- |
| 21 | blivet/util.py | 6 +++--- |
| 22 | 1 file changed, 3 insertions(+), 3 deletions(-) |
| 23 | |
| 24 | diff --git a/blivet/util.py b/blivet/util.py |
Patrick Williams | 03514f1 | 2024-04-05 07:04:11 -0500 | [diff] [blame] | 25 | index 58117ae..5bc5804 100644 |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 26 | --- a/blivet/util.py |
| 27 | +++ b/blivet/util.py |
Patrick Williams | 03514f1 | 2024-04-05 07:04:11 -0500 | [diff] [blame] | 28 | @@ -176,14 +176,14 @@ def timeout_command(argv, timeout, *args, **kwargs): |
Brad Bishop | 1a4b7ee | 2018-12-16 17:11:34 -0800 | [diff] [blame] | 29 | """call shell-command and either return its output or kill it |
| 30 | if it doesn't normally exit within timeout seconds and return None""" |
| 31 | import subprocess, datetime, os, time, signal |
| 32 | - start = datetime.datetime.now() |
| 33 | + count = 0 |
| 34 | |
| 35 | try: |
| 36 | proc = subprocess.Popen(argv, *args, **kwargs) |
| 37 | while proc.poll() is None: |
| 38 | time.sleep(0.1) |
| 39 | - now = datetime.datetime.now() |
| 40 | - if timeout != -1 and (now - start).seconds> timeout: |
| 41 | + count += 1 |
| 42 | + if timeout != -1 and count > timeout*10: |
| 43 | os.kill(proc.pid, signal.SIGKILL) |
| 44 | os.waitpid(-1, os.WNOHANG) |
| 45 | program_log.debug("%d seconds timeout" % timeout) |