From: Frank Ch. Eigler <fche@redhat.com>
Date: Mon, 15 Nov 2021 02:59:05 +0000 (-0500)
Subject: PR28449: runtime/transport: bump up _stp_subbuf_size to 64K
X-Git-Tag: release-4.6~4
X-Git-Url: https://sourceware.org/git/?a=commitdiff_plain;h=193da50838ce7e4e115d79dfcb120cddd50e98fa;p=systemtap.git

PR28449: runtime/transport: bump up _stp_subbuf_size to 64K

After commit cd48874296, it was reported that the kernel-side I/O
buffers could be exhausted more easily by the some tests in the
testsuite.  This resulted in "bufhdr corrupted ..." type messages
coming from the syscall.exp test case for example.

We bump up the $subject variable, so without "stap -s XYZ", the
default kernel->user transport will consist of 256 subbufs of 64K
each, per cpu.  (When a user does supply -s XYZ, the number gets
overridden, and the subbuf size may be quite a bit larger.)  With the
new default, the syscall.exp suite runs bufhdr-clean on an 8cpu box.
---

diff --git a/runtime/transport/transport.c b/runtime/transport/transport.c
index 3e128c92c..18ecccea2 100644
--- a/runtime/transport/transport.c
+++ b/runtime/transport/transport.c
@@ -73,7 +73,7 @@ static inline void _stp_unlock_inode(struct inode *inode);
 #include "control.c"
 
 static unsigned _stp_nsubbufs = 256;
-static unsigned _stp_subbuf_size = STP_BUFFER_SIZE;
+static unsigned _stp_subbuf_size = 8 * STP_BUFFER_SIZE; /* 64K */
 
 /* module parameters */
 static int _stp_bufsize;