#1436 closed enhancement (fixed)
Add Signal Tracing to strace (easy)
Reported by: | bonefish | Owned by: | bonefish |
---|---|---|---|
Priority: | normal | Milestone: | R1 |
Component: | - General | Version: | R1/pre-alpha1 |
Keywords: | Cc: | ||
Blocked By: | Blocking: | ||
Platform: | All |
Description
ATM strace prints the traced syscalls only. It could (optionally) also trace signals.
Attachments (5)
Change History (12)
by , 17 years ago
by , 17 years ago
Attachment: | screen1.png added |
---|
follow-up: 2 comment:1 by , 17 years ago
by , 17 years ago
follow-up: 3 comment:2 by , 17 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
oh, some code for debug was not removed. Now the newpatch should fix the bug Replying to absabs:
when applied the patch, strace now can trace signals tested as following:
- in one terminal
strace sleep 666
- in another terminal
kill -s sig pid where sig is the signal such as URG\USR1 and so on.
see the screenshot
by , 17 years ago
Attachment: | newpatch.2 added |
---|
comment:3 by , 17 years ago
sorry for my mistake. Give up "patch\newpatch\newpatch.2", we need only the final.patch Replying to absabs:
oh, some code for debug was not removed. Now the newpatch should fix the bug Replying to absabs:
when applied the patch, strace now can trace signals tested as following:
- in one terminal
strace sleep 666
- in another terminal
kill -s sig pid where sig is the signal such as URG\USR1 and so on.
see the screenshot
follow-up: 5 comment:4 by , 17 years ago
Thanks for your patch! It looks very nice! Do you have the time to fix the coding style problems (constants have the 'k' prefix, so it would be kSignalName), and add an option to make signal tracing optional? If not, I can make the changes, too, just tell.
follow-up: 6 comment:5 by , 17 years ago
No problem. I will upload the patch later. :) Replying to axeld:
Thanks for your patch! It looks very nice! Do you have the time to fix the coding style problems (constants have the 'k' prefix, so it would be kSignalName), and add an option to make signal tracing optional? If not, I can make the changes, too, just tell.
by , 17 years ago
Attachment: | final.patch added |
---|
comment:6 by , 17 years ago
I have updated the final.patch. Thanks Replying to absabs:
No problem. I will upload the patch later. :) Replying to axeld:
Thanks for your patch! It looks very nice! Do you have the time to fix the coding style problems (constants have the 'k' prefix, so it would be kSignalName), and add an option to make signal tracing optional? If not, I can make the changes, too, just tell.
comment:7 by , 17 years ago
Thanks! Applied in hrev23436 with minor changes (-g now turns off signal tracing, it's enabled by default).
when applied the patch, strace now can trace signals tested as following:
strace sleep 666
kill -s sig pid where sig is the signal such as URG\USR1 and so on.
see the screenshot