git: f239db4800ee - main - ktrace: Remove CAPFAIL from default trace points

From: Jake Freeland <jfree_at_FreeBSD.org>
Date: Wed, 24 Apr 2024 00:29:04 UTC
The branch main has been updated by jfree:

URL: https://cgit.FreeBSD.org/src/commit/?id=f239db4800ee9e7ff8485f96b7a68e6c38178c3b

commit f239db4800ee9e7ff8485f96b7a68e6c38178c3b
Author:     Jake Freeland <jfree@FreeBSD.org>
AuthorDate: 2024-04-24 00:26:14 +0000
Commit:     Jake Freeland <jfree@FreeBSD.org>
CommitDate: 2024-04-24 00:26:14 +0000

    ktrace: Remove CAPFAIL from default trace points
    
    The CAPFAIL tracepoint was recently extended to report ECAPMODE
    capability  violations for processes that do not enter capability mode.
    This allows developers that are interested in Capsicumizing their
    programs to determine where violations are being raised.
    
    Previously, CAPFAIL only produced output for processes using Capsicum(4)
    capabilties. Thus, most ktrace users never received log output from the
    trace point. With the recent changes, this is no longer the case.
    
    Having this trace point enabled by default will produce output for all
    processes that use syscalls that are not permitted in capability mode.
    This may lead to confusion for users that are not familiar with the
    feature. Remove KTRFAC_CAPFAIL from ktrace's default points to avoid
    this.
    
    Approved by:    markj (mentor)
    Reviewed by:    markj
    MFC after:      1 month
    Differential Revision:  https://reviews.freebsd.org/D44887
---
 usr.bin/ktrace/ktrace.1 | 2 +-
 usr.bin/ktrace/ktrace.h | 7 +++----
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/usr.bin/ktrace/ktrace.1 b/usr.bin/ktrace/ktrace.1
index 5ccb206456b8..da39aac7800f 100644
--- a/usr.bin/ktrace/ktrace.1
+++ b/usr.bin/ktrace/ktrace.1
@@ -137,7 +137,7 @@ trace
 requests
 .It Cm +
 trace the default set of trace points -
-.Cm c , i , n , p , s , t , u , y
+.Cm c , i , n , s , t , u , y
 .El
 .It Ar command
 Execute
diff --git a/usr.bin/ktrace/ktrace.h b/usr.bin/ktrace/ktrace.h
index 1022f06532b1..bf3f70539310 100644
--- a/usr.bin/ktrace/ktrace.h
+++ b/usr.bin/ktrace/ktrace.h
@@ -31,13 +31,12 @@
 
 #define DEF_POINTS (KTRFAC_SYSCALL | KTRFAC_SYSRET | KTRFAC_NAMEI | \
 		    KTRFAC_GENIO | KTRFAC_PSIG | KTRFAC_USER | \
-		    KTRFAC_STRUCT | KTRFAC_SYSCTL | KTRFAC_CAPFAIL | \
-		    KTRFAC_STRUCT_ARRAY)
+		    KTRFAC_STRUCT | KTRFAC_SYSCTL | KTRFAC_STRUCT_ARRAY)
 
 #define PROC_ABI_POINTS (KTRFAC_PROCCTOR | KTRFAC_PROCDTOR)
 
-#define ALL_POINTS (DEF_POINTS | KTRFAC_CSW | PROC_ABI_POINTS | \
-		    KTRFAC_FAULT | KTRFAC_FAULTEND)
+#define ALL_POINTS (DEF_POINTS | KTRFAC_CAPFAIL | KTRFAC_CSW | \
+		    PROC_ABI_POINTS | KTRFAC_FAULT | KTRFAC_FAULTEND)
 
 #define DEF_TRACEFILE	"ktrace.out"