summaryrefslogtreecommitdiff
path: root/integrationtests/readwrite_test.go
diff options
context:
space:
mode:
authorPaul Buetow <paul@buetow.org>2026-06-09 22:09:13 +0300
committerPaul Buetow <paul@buetow.org>2026-06-09 22:09:13 +0300
commit3cd5e655ee1768b4118815d1ea887acdd57eb498 (patch)
tree05285bc148faff79f931059aff97b8d3fbb9b126 /integrationtests/readwrite_test.go
parent57615945e6796950e7095a3ee8a97651ae3f1bd9 (diff)
test: add coverage for pidfd_send_signal and fadvise64
pidfd_send_signal (FamilyIPC, KindFd@arg0) and fadvise64 (KindFd, UNCLASSIFIED fd-based hint) previously had no end-to-end integration coverage despite correct classification/tracing. pidfd_send_signal: add a pidfd-send-signal ioworkload scenario that opens a pidfd for the current process and issues a sig-0 liveness probe (delivers nothing, safe to target self) via syscall.Syscall6 with the per-arch nr 424. TestPidfdSendSignal asserts enter_pidfd_send_signal is captured; pidfd_send_signal added to the pidfd -trace-syscalls list. fadvise64: add readwrite-fadvise64 and readwrite-fadvise64-ebadf scenarios using unix.Fadvise(fd, 0, 0, FADV_NORMAL), mirroring the readahead tests. TestReadwriteFadvise64 asserts enter_fadvise64 with Bytes==0 (UNCLASSIFIED: offset/len are hints, not bytes transferred) and positive duration; the ebadf variant asserts enter capture with Bytes==0 on the failing call. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Diffstat (limited to 'integrationtests/readwrite_test.go')
-rw-r--r--integrationtests/readwrite_test.go43
1 files changed, 43 insertions, 0 deletions
diff --git a/integrationtests/readwrite_test.go b/integrationtests/readwrite_test.go
index 8bfa539..636cc5c 100644
--- a/integrationtests/readwrite_test.go
+++ b/integrationtests/readwrite_test.go
@@ -302,6 +302,49 @@ func TestReadwriteReadaheadEbadf(t *testing.T) {
}, 0)
}
+func TestReadwriteFadvise64(t *testing.T) {
+ // fadvise64(2) is KindFd / UNCLASSIFIED: it declares a page-cache access hint
+ // and returns 0/-1 (no byte count, transfers no bytes to userspace), so the
+ // tracer must attribute zero bytes (not misread the 0/-1 return or the
+ // offset/len hint parameters as a byte count) while still capturing the fd
+ // (args[0]) on enter and timing the syscall.
+ result, _ := runScenarioResult(t, "readwrite-fadvise64", []ExpectedEvent{
+ {
+ PathContains: "fadvise64file.txt",
+ Tracepoint: "enter_fadvise64",
+ Comm: "ioworkload",
+ MinCount: 1,
+ },
+ })
+ exp := ExpectedEvent{
+ PathContains: "fadvise64file.txt",
+ Tracepoint: "enter_fadvise64",
+ Comm: "ioworkload",
+ }
+ // UNCLASSIFIED: no byte count is attributed for a successful fadvise64.
+ assertEventBytesEqual(t, result, exp, 0)
+ // Timing is captured end-to-end (enter/exit paired into a duration).
+ assertEventDurationPositive(t, result, exp)
+}
+
+func TestReadwriteFadvise64Ebadf(t *testing.T) {
+ // fadvise64 on an invalid fd fails with EBADF, but ior still captures the
+ // enter_fadvise64 tracepoint because arguments are read on syscall entry
+ // before the kernel returns the error. The UNCLASSIFIED -1 return must not
+ // be attributed as bytes.
+ result, _ := runScenarioResult(t, "readwrite-fadvise64-ebadf", []ExpectedEvent{
+ {
+ Tracepoint: "enter_fadvise64",
+ Comm: "ioworkload",
+ MinCount: 1,
+ },
+ })
+ assertEventBytesEqual(t, result, ExpectedEvent{
+ Tracepoint: "enter_fadvise64",
+ Comm: "ioworkload",
+ }, 0)
+}
+
func TestReadwriteCachestat(t *testing.T) {
// cachestat(2) is KindFd / UNCLASSIFIED: it queries page-cache residency for
// a file and returns 0/-1 (no byte count, no I/O bytes to userspace), so the