Page MenuHomeFreeBSD

ddb: Add 'show active trace' command
ClosedPublic

Authored by cem on Aug 25 2016, 10:03 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 3 2024, 2:02 AM
Unknown Object (File)
Jan 9 2024, 11:20 AM
Unknown Object (File)
Jan 9 2024, 11:20 AM
Unknown Object (File)
Jan 9 2024, 11:20 AM
Unknown Object (File)
Dec 22 2023, 9:19 PM
Unknown Object (File)
Aug 25 2023, 12:04 PM
Unknown Object (File)
Aug 3 2023, 4:51 PM
Unknown Object (File)
Aug 3 2023, 4:51 PM
Subscribers

Details

Summary

'show active trace', or 'acttrace' for short, prints backtraces from running
threads only.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4936
Build 4999: arc lint + arc unit

Event Timeline

cem retitled this revision from to ddb: Add 'show active trace' command.
cem updated this object.
cem edited the test plan for this revision. (Show Details)
cem added reviewers: mjg, jhb.

Reduce duplication between all trace and active trace implementations.

sys/ddb/db_command.c
837

I think it would be beneficial (apart from introduction the acctrace command) to extend the default output with cpu number, if given thread is running.

Other than that looks good to me, thanks.

sys/ddb/db_command.c
837

The active_only (acttrace) output does include td_oncpu. It's the other clause of the if :-).

Or do you mean, I should add oncpu output to "all trace" as well?

sys/ddb/db_command.c
837

The latter.

sys/ddb/db_command.c
837

To be more specific, either only print CPU if the thread is running OR print CPU unconditionally along with the state, so that one can tell if it is on cpu (as opposed to having a note what last cpu it was on).

Print oncpu for running threads in both 'all trace' and 'active trace', per
mjg@.

cem marked an inline comment as done.Aug 26 2016, 1:46 AM

Does this look okay?

mjg edited edge metadata.

LGTM, thanks

This revision is now accepted and ready to land.Aug 26 2016, 2:33 AM
This revision was automatically updated to reflect the committed changes.