Re: [Crash-utility] [PATCH] fix a mistake of output for task's state
----- Original Message -----
Hello Dave,
Please refer to the attachment. I think the string of state is not
correct.
--
--
Regards
Qiao Nuohan
--- a/task.c
+++ b/task.c
@@ -4633,7 +4633,7 @@ task_state_string(ulong task, char *buf, int verbose)
if (valid_task_state(_TRACING_STOPPED_) &&
(state & _TRACING_STOPPED_)) {
- sprintf(buf, "ST");
+ sprintf(buf, "TS");
valid++;
set++;
}
~
Actually it is shown as "ST" on purpose, because it goes back to
older kernels where there was no separate "traced" state, and both
"stopped" and "traced" states were essentially the same. The flag
bits keep changing over time, but the original list of states shown
by "ps" has always been:
crash> help ps
...
5. the task state (RU, IN, UN, ZO, ST, DE, SW).
...
However I agree with you that for clarity's sake the traced state should
be differentiated. But I think it would be better to change it to "TR"
instead of your "TS" suggestion -- note that the verbose translation
used by the "set" command shows "TASK_TRACED":
if (state & _TRACING_STOPPED_)
sprintf(&buf[strlen(buf)], "%sTASK_TRACED",
count++ ? "|" : "");
I'll update the "ps" help page with the new "TR" identifier.
Thanks,
Dave