Page MenuHomeFreeBSD
Differential D16962

ddb: Enable 'thread <address>'
ClosedPublic
Actions

Authored by cem on Aug 30 2018, 10:56 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 17, 3:35 AM
Unknown Object (File)
Tue, Oct 14, 5:46 PM
Unknown Object (File)
Sun, Oct 12, 8:53 AM
Unknown Object (File)
Sun, Oct 12, 1:56 AM
Unknown Object (File)
Wed, Oct 8, 7:37 AM
Unknown Object (File)
Thu, Oct 2, 6:43 AM
Unknown Object (File)
Thu, Oct 2, 4:44 AM
Unknown Object (File)
Sat, Sep 20, 7:36 AM
View All Files
Subscribers
imp

Details

Reviewers
markj
kib
bwidawsk
bdrewery
jhb
Group Reviewers
manpages
Commits
rS339486: ddb: Enable 'thread <address>'
Summary

Prior to this change, the 'thread' command (to switch the debugger to another
thread) only accepts decimal-encoded tids. This is a minor nuisance and
sometimes stumbling block.

Use the same parsing logic as 'show thread <arg>' to enable 'thread' to accept
hex-encoded thread pointers in addition to decimal-encoded tids.

Test Plan

Untested other than compile. Will confirm before committing.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

cem created this revision.Aug 30 2018, 10:56 PM
Harbormaster completed remote builds in B19300: Diff 47502.Aug 30 2018, 10:56 PM
kib accepted this revision.Aug 31 2018, 7:59 AM
This revision is now accepted and ready to land.Aug 31 2018, 7:59 AM
markj added a comment.Aug 31 2018, 12:05 PM

Does ddb.4 need to be updated?

bwidawsk accepted this revision.Aug 31 2018, 5:45 PM

I think man page does need to be updated as markj said.

cem updated this revision to Diff 47553.Sep 1 2018, 4:19 AM

Document 'thread' (entirely absent) and 'show thread' tid behavior.

This revision now requires review to proceed.Sep 1 2018, 4:19 AM
Harbormaster completed remote builds in B19337: Diff 47553.Sep 1 2018, 4:19 AM
markj accepted this revision.Sep 1 2018, 5:12 AM
This revision is now accepted and ready to land.Sep 1 2018, 5:12 AM
Closed by commit rS339486: ddb: Enable 'thread <address>' (authored by cem). · Explain WhyOct 20 2018, 8:46 PM
This revision was automatically updated to reflect the committed changes.
Herald added a reviewer: manpages. · View Herald TranscriptOct 20 2018, 8:46 PM
Herald added a subscriber: imp. · View Herald Transcript

Revision Contents
Changeset List

PathSize
head/
share/
man/
man4/
21 lines
sys/
ddb/
2 lines
12 lines

Diff 49354

View Options

head/share/man/man4/ddb.4

Loading...
View Options

head/sys/ddb/db_command.c

Loading...
View Options

head/sys/ddb/db_thread.c

Loading...