Differential D46674

dtrace tests: Add a test case which validates FBT probe arguments
ClosedPublic
Actions

Authored by markj on Sep 14 2024, 6:56 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Sep 11, 8:00 AM

	Unknown Object (File)
	Fri, Sep 11, 7:57 AM

	Unknown Object (File)
	Fri, Sep 11, 7:57 AM

	Unknown Object (File)
	Thu, Sep 10, 8:33 AM

	Unknown Object (File)
	Wed, Sep 9, 11:50 PM

	Unknown Object (File)
	Wed, Sep 9, 9:45 AM

	Unknown Object (File)
	Wed, Sep 9, 2:08 AM

	Unknown Object (File)
	Sat, Sep 5, 10:15 AM

Subscribers

Details

Reviewers

gnn
avg

Group Reviewers

Commits

rG37a29946b91b: dtrace tests: Add a test case which validates FBT probe arguments
rGd439598dd0d3: dtrace tests: Add a test case which validates FBT probe arguments

Summary

MFC after: 2 weeks

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markj created this revision.Sep 14 2024, 6:56 PM

Herald added a reviewer: gnn. · View Herald TranscriptSep 14 2024, 6:56 PM

Herald added a subscriber: imp. · View Herald Transcript

markj requested review of this revision.Sep 14 2024, 6:56 PM

Harbormaster completed remote builds in B59486: Diff 143342.Sep 14 2024, 6:56 PM

markj added a parent revision: D46673: dtrace_test: Remove the dependency on dtraceall.Sep 14 2024, 6:56 PM

markj added a child revision: D46675: dtrace/amd64: Remove the dtrace_invop_callsite symbol.

LGTM

cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/fbtprovider/tst.argtest.ksh
32–33	It seems that DTrace supports args from 0 to 9 (10 of them in total). I do not see why args[9] would behave differently from args[8], but it might make sense to add it as well just for the sake of completeness.

This revision is now accepted and ready to land.Sep 15 2024, 9:09 AM

Include args[9] in the test.

This revision now requires review to proceed.Sep 15 2024, 1:32 PM

Harbormaster completed remote builds in B59496: Diff 143363.Sep 15 2024, 1:32 PM

markj marked an inline comment as done.Sep 15 2024, 1:32 PM

avg accepted this revision as: avg.Sep 15 2024, 8:10 PM

This revision is now accepted and ready to land.Sep 15 2024, 8:10 PM

Closed by commit rGd439598dd0d3: dtrace tests: Add a test case which validates FBT probe arguments (authored by markj). · Explain WhySep 19 2024, 9:54 AM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rGd439598dd0d3: dtrace tests: Add a test case which validates FBT probe arguments.

markj added a commit: rG37a29946b91b: dtrace tests: Add a test case which validates FBT probe arguments.Oct 4 2024, 3:57 PM

Revision Contents
Changeset List

Path

Size

cddl/

contrib/

opensolaris/

cmd/

dtrace/

test/

tst/

common/

fbtprovider/

tst.argtest.ksh

34 lines

tst.argtest.ksh.out

3 lines

usr.sbin/

dtrace/

tests/

common/

fbtprovider/

2 lines

sys/

cddl/

dev/

dtrace/

28 lines

Diff 143447

cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/fbtprovider/tst.argtest.ksh

Loading...

cddl/contrib/opensolaris/cmd/dtrace/test/tst/common/fbtprovider/tst.argtest.ksh.out

Loading...

cddl/usr.sbin/dtrace/tests/common/fbtprovider/Makefile

Loading...

sys/cddl/dev/dtrace/dtrace_test.c

Loading...