diff options
author | Petr Oros <poros@redhat.com> | 2013-12-04 08:27:15 +0000 |
---|---|---|
committer | Jiri Kastner <jkastner@redhat.com> | 2014-01-30 14:59:55 +0100 |
commit | 443234bd24c794068a752d567bb7f4d903fbbaac (patch) | |
tree | fdd842667a2ae255b6e453210cf1bbac92f1e721 | |
parent | 7ec43cb842d412ff8586ba1d81e94b75bd3a9f90 (diff) | |
download | tuna-443234bd24c794068a752d567bb7f4d903fbbaac.tar.gz |
CLI: fix traceback where enter -p policy without prio
Tuna throwing value error when not specified priority.
Example: tuna -t PID_OF_SOME_PROCESS -p OTHER
Here "other" not priority, but scheduler policy.
For RR and FIFO default 1. For OTHER and BATCH default 0.
After patch apply, tuna use first part of string as policy and remaining part as priority.
All unacceptable values cause exception in schedutils library and tuna show error
Tested-by: Jiri Kastner <jkastner@redhat.com>
Signed-off-by: Petr Oros <poros@redhat.com>
Signed-off-by: Jiri Kastner <jkastner@redhat.com>
-rwxr-xr-x | tuna/tuna.py | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/tuna/tuna.py b/tuna/tuna.py index 6dfe740..0feb521 100755 --- a/tuna/tuna.py +++ b/tuna/tuna.py @@ -490,10 +490,14 @@ def thread_set_priority(tid, policy, rtprio): def threads_set_priority(tids, parm, affect_children = False): parms = parm.split(":") + rtprio = 0 policy = None - if len(parms) != 1: + if parms[0].upper() in ["OTHER", "BATCH", "IDLE", "FIFO", "RR"]: policy = schedutils.schedfromstr("SCHED_%s" % parms[0].upper()) - rtprio = int(parms[1]) + if len(parms) > 1: + rtprio = int(parms[1]) + elif parms[0].upper() in ["FIFO", "RR"]: + rtprio = 1 else: rtprio = int(parms[0]) |