diff options
author | Luis Claudio R. Goncalves <lgoncalv@redhat.com> | 2010-05-06 12:43:45 -0500 |
---|---|---|
committer | Clark Williams <williams@redhat.com> | 2010-05-06 12:43:45 -0500 |
commit | 8f4e1d0f96366d131825be0789e3b08e8b90f6dd (patch) | |
tree | b13eb4398e0c81ffda33e7765998715ad4b8605c /rteval.spec | |
parent | aafe28b72299ce0631174f116825ec93bd50a20d (diff) | |
download | rteval-8f4e1d0f96366d131825be0789e3b08e8b90f6dd.tar.gz rteval-8f4e1d0f96366d131825be0789e3b08e8b90f6dd.tar.xz rteval-8f4e1d0f96366d131825be0789e3b08e8b90f6dd.zip |
rteval: code cleanup and fixes
Changes:
* Closed /dev/null after use on each function opening it.
* Replaced os.kill(PID, SIGTERM) by subprocess.terminate() as the processes
have been started using subprocess.Popen(). Also added subprocess.wait()
after each terminate() call.
Then I noticed the subprocess python module in RHEL5 doesn't have the
methods terminate() and kill(), so I reverted that piece of the change
but kept the missing wait() calls.
* Imported the missing SIGKILL symbol in hackbench.py
* Removed the code snippet below from hackbench.py:runload():
count = 30
while count > 0 and p.poll() == None:
time.sleep(1.0)
count -= 1
if p.poll() == None:
os.kill(p.pid, SIGKILL)
Signed-off-by: Luis Claudio R. Gonçalves <lgoncalv@redhat.com>
Signed-off-by: Clark Williams <williams@redhat.com>
Diffstat (limited to 'rteval.spec')
0 files changed, 0 insertions, 0 deletions