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)
Tue, Nov 4, 2:59 AM
Unknown Object (File)
Mon, Nov 3, 1:39 PM
Unknown Object (File)
Mon, Nov 3, 1:37 PM
Unknown Object (File)
Mon, Nov 3, 1:34 PM
Unknown Object (File)
Mon, Nov 3, 1:33 PM
Unknown Object (File)
Sun, Nov 2, 5:59 PM
Unknown Object (File)
Mon, Oct 27, 7:50 PM
Unknown Object (File)
Sun, Oct 26, 11:41 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...