Add A Sample To Exapmle
Frank Ch. Eigler
fche@redhat.com
Mon Nov 7 13:46:00 GMT 2016
Hi -
> I add a sample systemtap script to source tree. Wish it can be useful:)
> Can anybody help me to review this patch?
> [...]
> diff --git a/testsuite/systemtap.examples/index.html b/testsuite/systemtap.examples/index.html
> index d9a7d47..5952356 100644
> --- a/testsuite/systemtap.examples/index.html
> +++ b/testsuite/systemtap.examples/index.html
> [...]
(This is OK, but you don't have to include diffs of machine-generated
files for review.)
> [...]
> +global record
This is fine, except that there is no limit in your script as to how
many records may be kept alive. This record array will grow and grow
and eventually overflow with a MAXMAPENTRIES-related error. Consider
global record%
to activate wrapping (LRU) mode, so that older entries will be
reused when the array fills up.
> +probe begin {
> + warn("Printing tcp retransmission")
> +}
Please consider log() or such instead of warn().
Otherwise looks good. What do you think about the suggestions?
- FChE
More information about the Systemtap
mailing list