]> sourceware.org Git - systemtap.git/commitdiff
PR28449: runtime/transport: bump up _stp_subbuf_size to 64K
authorFrank Ch. Eigler <fche@redhat.com>
Mon, 15 Nov 2021 02:59:05 +0000 (21:59 -0500)
committerFrank Ch. Eigler <fche@redhat.com>
Mon, 15 Nov 2021 02:59:05 +0000 (21:59 -0500)
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.

runtime/transport/transport.c

index 3e128c92c964dbfe8d0f34369240292afd076d24..18ecccea290762098e539dfda7eab7a81cb2ba88 100644 (file)
@@ -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;
This page took 0.028085 seconds and 5 git commands to generate.