Differential D51268

d.7: Document the DTrace scripting language
ClosedPublic
Actions

Authored by 0mp on Jul 11 2025, 8:45 PM.

Tags

Referenced Files

	Unknown Object (File)
	Wed, Sep 3, 7:31 AM

	Unknown Object (File)
	Wed, Aug 27, 4:06 PM

	Unknown Object (File)
	Wed, Aug 27, 3:08 PM

	Unknown Object (File)
	Wed, Aug 27, 5:00 AM

	Unknown Object (File)
	Tue, Aug 26, 10:20 PM

	Unknown Object (File)
	Tue, Aug 26, 12:19 PM

	Unknown Object (File)
	Sun, Aug 24, 2:45 PM

	Unknown Object (File)
	Sat, Aug 23, 11:07 PM

View All 54 Files

Subscribers

Details

Reviewers

bnovkov
christos
markj
gnn

Commits

rG193f2289fc9a: d.7: Document the DTrace scripting language

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

0mp created this revision.Jul 11 2025, 8:45 PM

Herald added subscribers: ziaee, imp. · View Herald TranscriptJul 11 2025, 8:45 PM

0mp requested review of this revision.Jul 11 2025, 8:45 PM

Harbormaster completed remote builds in B65365: Diff 158342.Jul 11 2025, 8:45 PM

0mp edited the summary of this revision. (Show Details)Jul 11 2025, 8:45 PM

0mp added a project: DTrace.

ziaee added inline comments.Jul 11 2025, 8:57 PM

share/man/man7/d.7
12	Maybe?

Improve Nd

Harbormaster completed remote builds in B65370: Diff 158351.Jul 12 2025, 7:28 AM

0mp added inline comments.Jul 12 2025, 7:28 AM

share/man/man7/d.7
12	Why not! Thank you :)

A few minor things, the rest looks fine.

share/man/man7/d.7
77	I don't think you need a comma here.
80	same here
155	s/handling a/handling of a/

Finish the first draft.

Harbormaster completed remote builds in B65371: Diff 158358.Jul 12 2025, 9:22 AM

0mp edited the summary of this revision. (Show Details)Jul 12 2025, 1:51 PM

0mp added reviewers: bnovkov, christos, markj.

0mp added a project: docs.

Address bcr's feedback
Reference ddb(4) as the kernel debugger, not kgdb

Harbormaster completed remote builds in B65378: Diff 158370.Jul 12 2025, 2:18 PM

Move cwd to BUGS and mention that it is not supported at the moment.

Harbormaster completed remote builds in B65381: Diff 158374.Jul 12 2025, 3:09 PM

Document execargs
Add a note what execname is.

Harbormaster completed remote builds in B65382: Diff 158379.Jul 12 2025, 3:46 PM

christos added inline comments.Jul 13 2025, 2:40 PM

share/man/man7/d.7
79	Isn't this sentence redundant, since the previous sentence already says that an enabled probed is defined by its probe ID?
96	Better to make this the second one in order to match the order of the probe description fields (provider, module, function, name).
148
197	I think here and below it's better to get rid of the "a/an".

Add bcr's comments

Harbormaster completed remote builds in B65403: Diff 158425.Jul 13 2025, 4:12 PM

Address feedback

Harbormaster completed remote builds in B65411: Diff 158459.Jul 14 2025, 12:11 PM

@ziaee, could you check the apropos(1) compliance? ;)

share/man/man7/d.7
79	So this comment is about: The enabled probe ID which uniquely identifies an enabled probe. An enabled probe is defined by its probe ID, its predicates, and its actions. I think it is necessary, because those 3 items are what makes the enabled probe unique. It is not redundant.

christos added inline comments.Jul 14 2025, 12:16 PM

share/man/man7/d.7
203	The first letter should be capitalized, no?

Capitalize descriptions of aggregation functions

Harbormaster completed remote builds in B65412: Diff 158462.Jul 14 2025, 12:21 PM

0mp marked an inline comment as done.Jul 14 2025, 12:21 PM

0mp added inline comments.

share/man/man7/d.7
203	Sure, why not.

0mp marked an inline comment as done.Jul 14 2025, 12:22 PM

christos accepted this revision.Jul 14 2025, 12:26 PM

This revision is now accepted and ready to land.Jul 14 2025, 12:26 PM

Thanks!

Closed by commit rG193f2289fc9a: d.7: Document the DTrace scripting language (authored by 0mp). · Explain WhyJul 14 2025, 1:02 PM

This revision was automatically updated to reflect the committed changes.

0mp added a commit: rG193f2289fc9a: d.7: Document the DTrace scripting language.

Herald added a reviewer: gnn. · View Herald TranscriptJul 14 2025, 1:02 PM

Revision Contents
Changeset List

Path

Size

cddl/

contrib/

opensolaris/

cmd/

dtrace/

3 lines

share/

man/

man7/

1 line

287 lines

5 lines

Diff 158464

cddl/contrib/opensolaris/cmd/dtrace/dtrace.1

Loading...

share/man/man7/Makefile

Loading...

share/man/man7/d.7

Loading...

share/man/man7/intro.7

Loading...