Page MenuHomeFreeBSD
Differential D51341

stress2/fullpath2: Clean up the vn_fullpath DTrace one-liner
ClosedPublic
Actions

Authored by 0mp on Jul 16 2025, 11:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 10, 4:21 PM
Unknown Object (File)
Fri, Oct 10, 4:21 PM
Unknown Object (File)
Fri, Oct 10, 4:21 PM
Unknown Object (File)
Fri, Oct 10, 10:32 AM
Unknown Object (File)
Fri, Sep 26, 9:56 PM
Unknown Object (File)
Mon, Sep 15, 2:14 PM
Unknown Object (File)
Mon, Sep 15, 5:15 AM
Unknown Object (File)
Sep 13 2025, 6:47 AM
View All 27 Files
Subscribers
imp

Details

Reviewers
pho
christos
markj
bnovkov
Commits
rG31938394def0: stress2/fullpath2: Clean up the vn_fullpath DTrace one-liner
Summary
  • Remove -w; there is no need for destructive actions here.
  • Remove the glob from before "vn_fullpath"; the probe description matches vn_fullpath only anyway, so there is no need for the glob here.

Fixes: 113f2f0c76a7 stress2: Updated dtrace comment

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

0mp created this revision.Jul 16 2025, 11:27 AM
Herald added a subscriber: imp. · View Herald TranscriptJul 16 2025, 11:27 AM
0mp requested review of this revision.Jul 16 2025, 11:27 AM
Harbormaster completed remote builds in B65478: Diff 158593.Jul 16 2025, 11:27 AM
0mp added reviewers: pho, christos, markj, bnovkov.Jul 16 2025, 11:29 AM
christos accepted this revision.Jul 17 2025, 4:34 PM
This revision is now accepted and ready to land.Jul 17 2025, 4:34 PM
markj added a comment.Jul 17 2025, 4:58 PM

I think this is OBE? Commit 113f2f0c76a7ddc09cbadaa199e47cecec0f2814 has the same effect.

0mp added a comment.Jul 17 2025, 5:26 PM
In D51341#1173363, @markj wrote:

I think this is OBE? Commit 113f2f0c76a7ddc09cbadaa199e47cecec0f2814 has the same effect.

My patch improves the change committed by pho@. That's why I mention pho's patch in Fixes: .

Closed by commit rG31938394def0: stress2/fullpath2: Clean up the vn_fullpath DTrace one-liner (authored by 0mp). · Explain WhyJul 18 2025, 8:29 PM
This revision was automatically updated to reflect the committed changes.
0mp added a commit: rG31938394def0: stress2/fullpath2: Clean up the vn_fullpath DTrace one-liner.

Revision Contents
Changeset List

PathSize
tools/
test/
stress2/
misc/
2 lines

Diff 158759

View Options

tools/test/stress2/misc/fullpath2.sh

Loading...