better state default value handling

iovisor · Aug 11, 2017 · 4870819 · 4870819
1 parent 843340d
commit 4870819
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/tools/offcputime.py b/tools/offcputime.py
@@ -88,8 +88,7 @@ def positive_nonzero_int(val):
  type=positive_nonzero_int,
  help="the amount of time in microseconds under which we " +
  "store traces (default U64_MAX)")
-parser.add_argument("--state", default=999,
- type=positive_int,
+parser.add_argument("--state", type=positive_int,
  help="filter on this thread state bitmask (eg, 2 == TASK_UNINTERRUPTIBLE" +
  ") see include/linux/sched.h")
 args = parser.parse_args()
@@ -184,7 +183,7 @@ def signal_ignore(signal, frame):
  thread_filter = '1'
 if args.state == 0:
  state_filter = 'prev->state == 0'
-elif args.state != 999:
+elif args.state:
  # these states are sometimes bitmask checked
  state_filter = 'prev->state & %d' % args.state
 else: