Page MenuHomeFreeBSD
Differential D47648

ktrace.1: Add ARGS and ENVS as default trace points
ClosedPublic
Actions

Authored by jlduran on Sun, Nov 17, 5:55 PM.
Tags
Referenced Files
Unknown Object (File)
Tue, Nov 19, 10:21 AM
Unknown Object (File)
Mon, Nov 18, 9:48 PM
Unknown Object (File)
Mon, Nov 18, 7:48 PM
Unknown Object (File)
Mon, Nov 18, 4:55 PM
Unknown Object (File)
Mon, Nov 18, 4:26 AM
Unknown Object (File)
Sun, Nov 17, 8:08 PM
Subscribers
imp

Details

Reviewers
emaste
markj
Commits
rG2a657b4ee900: ktrace.1: Add ARGS and ENVS as default trace points
Summary

After commit 65a4daeaf324 ("ktrace: log execve(2) arguments and
environment"), "a" and "e" are part of the default set of trace points.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 60645
Build 57529: arc lint + arc unit

Event Timeline

jlduran created this revision.Sun, Nov 17, 5:55 PM
Herald added a subscriber: imp. · View Herald TranscriptSun, Nov 17, 5:55 PM
jlduran requested review of this revision.Sun, Nov 17, 5:55 PM
Harbormaster completed remote builds in B60645: Diff 146617.Sun, Nov 17, 5:55 PM
jlduran mentioned this in D47642: ktrace: Do not set args and envs as default.Sun, Nov 17, 5:55 PM
markj accepted this revision.Sun, Nov 17, 6:00 PM

Thank you.

Regarding these being the defaults, I don't have a strong opinion, but it seems natural for them to be enabled by default as they're effectively an extension of c. That is, they're pretty-printers for execve(2) arguments.

This revision is now accepted and ready to land.Sun, Nov 17, 6:00 PM
Closed by commit rG2a657b4ee900: ktrace.1: Add ARGS and ENVS as default trace points (authored by jlduran). · Explain WhySun, Nov 17, 6:14 PM
This revision was automatically updated to reflect the committed changes.
jlduran added a commit: rG2a657b4ee900: ktrace.1: Add ARGS and ENVS as default trace points.
jlduran added a comment.Sun, Nov 17, 6:17 PM
In D47648#1086370, @markj wrote:

Thank you.

You're welcome, and thank you for fixing the tests.

Regarding these being the defaults, I don't have a strong opinion, but it seems natural for them to be enabled by default as they're effectively an extension of c. That is, they're pretty-printers for execve(2) arguments.

Neither do I. Primarily motivated by fixing the tests without actually modifying them. Generally that will be my approach.

concussious.bugzilla_runbox.com added a project: manpages.Sun, Nov 17, 6:37 PM

Revision Contents
Changeset List

PathSize
usr.bin/
ktrace/
4 lines

Diff 146617

View Options

usr.bin/ktrace/ktrace.1

Loading...