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

Authored by 0mp on Jul 16 2025, 11:27 AM.

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

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

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

Path

Size

tools/

test/

stress2/

misc/

fullpath2.sh

2 lines

Diff 158759

View Options

tools/test/stress2/misc/fullpath2.sh

stress2/fullpath2: Clean up the vn_fullpath DTrace one-linerClosedPublicActions