Create debug_frame_hdr in target byte order

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

mips: add basic unwind support

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

mips: tapset: fix mips oddities in syscalls

Some mips syscalls have different prototypes to standard ones and
require special handling

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

tapset: fix build if arch does not define SA_RESTORER

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

mips: add a stub for stack unwind

This change adds a stub for processing stack unwind - this does not do
proper stack printing, but is enough to let the generated code compile
and run

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

mips: add tapset register definitions

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

mips: tapsets.cxx: add dwarf register definitions for SDT uprobes

Depending on the optimisations used, the compiler may generate either
numeric register definitions or typical aliases, so define both.
Since MIPS register names start with a dollar '$', add an escape
sequence to the register names when creating regname regular
expressions.

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

mips: add TIF_32BIT definition

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

testsuite: add support for mips64 build flags

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

tapset syscall-number database: add MIPS syscalls

This patch provides MIPS-specific syscall tables and extends
scripts/dump-syscall.sh to handle MIPS tables generation.

Signed-off-by: Marcin Nowakowski <marcin.nowakowski@imgtec.com>

Fixed 1382422 by making stap refer to 'dnf' or 'yum'.

* rpm_finder.cxx (missing_rpm_list_print): Ask the user to use the correct
  package manager for his system, either 'dnf' or 'yum'.

Introduce runtime optimizations for statistical computations per PR10234.

This update improves the performance of systemtap runtime statistical
computations by optimizing out unneeded parts of the __stp_stat_add()
function.  It is based on parametrizing and inlining it.  The stap
translator now generates _stp_stat_add(), or _stp_pmap_add_*() calls
that have additional "optimization" parameters respective to stats
in use for given global.  GCC uses this for optimizing the inlined
__stp_stat_add() calls.

The optimization effect significantly depends on compiler version,
platform architecture, and the stat operators being used for given
global.  At the moment, the available stat operators are @count,
@sum, @min, @max, @avg, and @variance. The most computionally
expensive is @variance.  The effect of optimizing @variance is
significant.  Other stat operators are computionally chap and so
the effect of their optimizations is relatively low.

Using gcc-6.2.1-1.fc26.x86_64, the @count, @sum, @min, and @max
optimizations brings approximately 8% run time shrinkage. The
@variance optimization shrinkage is up to 70% using this compiler.
For other architectures, namely for power, the optimization is less
effective.

runtime/map-gen.c: Pass the additional optimization parameters through
the map API generator macros.
runtime/map.c: Modify _new_map_set_stat() to accept optimization
        additional parameters.
runtime/map.h: Ditto.
runtime/pmap-gen.c: Pass the additional optimization parameters to
__stp_map_set*() and to _stp_pmap_add*().
runtime/stat-common.c: Add optimization params to __stp_stat_add().
runtime/stat.c: Add optimization params to _stp_stat_add().
tapsets.cxx: Generate parametrized calls to the runtime.
translate.cxx: Ditto.
testsuite/systemtap.base/optim_stats*: New testcase.

configury: regen configure

Don't use obsolete autoconf macro AC_PROG_CC_STDC

This has been obsolete since automake 1.8 as it's now part of AC_PROG_CC.

Add "sample" command to interactive mode

* Makefile.am: Updated using autoreconf in order to make DOCDIR available for interactive.cxx.
* Makefile.in: same
* doc/beginners/Makefile.in: same
* interactive.cxx: Added sample command, uses sqlite3.h.
* java/Makefile.in:: Updated using autoreconf in order to make DOCDIR available.
* testsuite/systemtap.examples/README: Added a small note about needing perl DBI to rerun perl script.
* testsuite/systemtap.examples/examples-index-gen.pl: As documentation files are generated, SQLite table is now populated with meta data.
* testsuite/systemtap.examples/index.html: Automatically updated documentation for new example after running perl script.
* testsuite/systemtap.examples/index.txt: same
* testsuite/systemtap.examples/keywords-index.html: same
* testsuite/systemtap.examples/keywords-index.txt: same
* testsuite/systemtap.examples/metadatabase.db: New database file that stores meta table.

man stap: note that need for --poison-cache option is likely a bug

Fix rhbz1378462 by updating the path to stap-gen-cert in stap-server.

Another PR20589 fix by updating user_buffer_quoted[_error]().

* tapset/uconversions.stp (user_buffer_quoted): Validate the
  'outlen' parameter, ensuring it is between 0 and MAXSTRINGLEN.
  (user_buffer_quoted_error): Ditto.
* tapset/linux/conversions.stp (kernel_buffer_quoted): Fix use of outlen.
* testsuite/buildok/conversions.stp: Move uconversion tests to
  uconversions.stp and update list of functions tested.
* testsuite/buildok/uconversions.stp: New file.

Let the @variance pretty printer show the bit_shift if specified.

Without this update, namely the cache doesn't behave correctly,
not able to distinguish between @variance() calls with different
bit-shifts.

man stap.1: update @variance text

Explain a little bit more about the bitshift and normalization.

Update @variance testcases after fixing PR20599.

This update fixes the reference values for @variance where PR20599
(histogram breaks @variance) manifested.

For linear_large.exp and linear_large_neg.exp, remove @variance
tests because of arithmetic overflow.

linear.exp: Update the @variance reference value.
linear_nearlyempty.exp: Ditto.
linear_over.exp: Ditto.
linear_overunder.exp: Ditto.
linear_under.exp: Ditto.
linear_large.exp: Remove the @variance subtest.
linear_large.stp: Ditto.
linear_large_neg.exp: Ditto.
linear_large_neg.stp: Ditto

Document @variance in the stap.1 man page.

Add a range check for the bit-shift of the @variance operator.

bump bundled unwind table size limit to 32M

It was reported that one version of oracle db has unwind tables in
excess of 24MB in size, and the prior stap limit of 16MB precludes
backtracing through it.

Fix PR20599 by only updating needed members of statistic_decl.

After introducing @variance in commit 63ead7fa, stat_ops and
bit_shift need to be tracked within the statistic_decl.  Before
this update, new histogram always rewrote these new members,
which was causing the problem described in PR20599.

elaborate.cxx: Only rewrite type, linear_low, linear_high, and
         linear_step, when binding new histogram with a global.

Fix PR20597 and move @avg() calculations out of runtime.

Revert those parts of commit 63ead7f, that introduce new "avg"
member of the struct stat_data and move computations of @avg()
from the code directly emitted by the translator to the runtime.

The goal of this update is both to fix PR20597, and to speed up
the runtime.  Note that @variance() calculations are based on the
avg too, but the variance vehicle carries its own avg_s, which is
an average scaled by certain bit_shift for numerical precision
improvement purpose.  Variance calculations are optimized out if
unneeded.

So after this change, @avg computations should be as fast as they
were before @variance has been introduced in commit 63ead7f.

elaborate.cxx: Calculate @avg based on sum and count.
translate.cxx: Ditto.
runtime/map.c: Ditto, plus remove the online @avg calculations.
runtime/stat.c: Remove online @avg calculations.
runtime/stat.h: Drop the 'avg' member from the struct stat_data.

Revert last change to flightrec*.exp testcases and add comments.

* testsuite/systemtap.base/flightrec1.exp: Revert last commit and add
  comments explaining what is going on. We expect the original 'stap'
  command to finish and leave stapio running in the background.
* testsuite/systemtap.base/flightrec2.exp: Ditto.
* testsuite/systemtap.base/flightrec3.exp: Ditto.
* testsuite/systemtap.base/flightrec4.exp: Ditto.
* testsuite/systemtap.base/flightrec5.exp: Ditto.

Fix PR200601 by updating __get_skb_iphdr() for newer 32-bit kernels.

* tapset/linux/ip.stp (__get_skb_iphdr): Updated for newer 32-bit
  kernels. Validates the skb structure, then calls the real
  skb_network_header() function.

Fix PR20589 by updating kernel_buffer_quoted().

* tapset/linux/conversions.stp (kernel_buffer_quoted): Validate the
  'outlen' parameter, ensuring it is between 0 and MAXSTRINGLEN.
  (kernel_buffer_quoted_error): Ditto.

copyright year bump

@variance runtime: fix map.c aggregate/calculations

gcc's indentation warnings pointed a finger of suspicion at the
new code that merges variance-related partial statistics across
stats structs.  Two blocks are now conditioned on STAT_OP_VARIANCE.

bump supported kernel version to 4.7

PR20423: tweak error message generation for case without e->components[]

... e.g. a case where a C function returns a struct, $return would
trigger the "... is being accessed instead of member" message, but the
operative tok* is the e node, not a member of the empty
e->components[].

tweak whitespace for gcc warning compliance

Handle testsuite 'spawn' calls correctly by calling 'close' and 'wait'.

* testsuite/systemtap.base/auto_path.exp: Add 'close' and 'wait' calls
  after a 'spawn'.
* testsuite/systemtap.base/const_op.exp: Ditto.
* testsuite/systemtap.base/kallsyms.exp: Ditto.
* testsuite/systemtap.base/timeout.exp: Ditto.
* testsuite/systemtap.context/symbols.exp: Kill spawned stap process and
  call 'close' before calling 'wait'.
* testsuite/systemtap.base/flightrec1.exp: Fix location of close/wait
  calls.
* testsuite/systemtap.base/flightrec2.exp: Ditto.
* testsuite/systemtap.base/flightrec3.exp: Ditto.
* testsuite/systemtap.base/flightrec4.exp: Ditto.
* testsuite/systemtap.base/flightrec5.exp: Ditto.
* testsuite/systemtap.base/crash.exp: Call 'catch' around individual
  'close' and 'wait' calls.
* testsuite/systemtap.maps/foreach_fail.exp: Ditto.
* testsuite/systemtap.base/backtrace.exp: Call 'catch' around 'wait'
  calls.
* testsuite/systemtap.base/cache_clean.exp: Ditto.
* testsuite/lib/systemtap.exp (setup_server): Add 'catch' around 'exec'.
  (stap_run_batch): Call 'close' before calling 'wait'.
* testsuite/lib/stap_compile.exp (stap_compile): Call 'close' before
  calling 'wait'.
* testsuite/systemtap.base/alternatives.exp: Ditto.
* testsuite/systemtap.base/at_var_unresolved.exp: Ditto.
* testsuite/systemtap.base/at_var_unresolved_lvalue.exp: Ditto.
* testsuite/systemtap.base/at_var_void_stmt.exp: Ditto.
* testsuite/systemtap.base/bz10078.exp: Ditto.
* testsuite/systemtap.base/bz1074541.exp: Ditto.
* testsuite/systemtap.base/bz1126645.exp: Ditto.
* testsuite/systemtap.base/bz13338.exp: Ditto.
* testsuite/systemtap.base/cmd_parse.exp: Ditto.
* testsuite/systemtap.base/debugpath.exp: Ditto.
* testsuite/systemtap.base/deref2.exp: Ditto.
* testsuite/systemtap.base/func_alias.exp: Ditto.
* testsuite/systemtap.base/multisym.exp: Ditto.
* testsuite/systemtap.base/pr13158.exp: Ditto.
* testsuite/systemtap.base/pr13306.exp: Ditto.
* testsuite/systemtap.base/pr14546.exp: Ditto.
* testsuite/systemtap.base/prologues.exp: Ditto.
* testsuite/systemtap.base/pthread_stacks.exp: Ditto.
* testsuite/systemtap.base/rep_ret.exp: Ditto.
* testsuite/systemtap.base/wakeup.exp: Ditto.
* testsuite/systemtap.base/warn_overflow.exp: Ditto.
* testsuite/systemtap.base/warnings.exp: Ditto.
* testsuite/systemtap.stress/whitelist.exp: Ditto.
* testsuite/systemtap.apps/tcl.exp: Update 'spawn' comment.

tweak whitespace and @variance conflicting-bitshift message

Reported-by: jhg_ on IRC.

Introduce new statistics operator @variance().

New statistics @variance() operator using the Welford's online algorithm
for per-cpu computation, and the Total Variance formula authored by
Niranjan Kamat and Arnab Nandi from the Ohio State University for the
cross-cpu aggregation.

AUTHORS refresh

systemtap: Blacklist powerpc:hash_fault tracepoint

Commit c4a048331603a3fe19c886498e957f5f2a577ed9 blacklists hcall_*
kernel tracepoints for powerpc to avoid recursive page faults and
kernel panic due to that. powerpc:hash_fault should be blacklisted
too, since, its placed in the same path of kernel code which is
invoked on a page fault.

Signed-off-by: Hemant Kumar <hemant@linux.vnet.ibm.com>

tweak error message for exhausted-overloaded-functions

Correct the error message generated for "all functions exhausted" to
focus on the token of the function-call site, rather than whatever
inner statement was last executed before a "next".

interactive mode: allow supplied script

dwflpp: cache CU entry PCs for pr15123_retry_addr

Previously it was calling getscopes(pc) to see if we're at the start of
the outermost function, which may be a preamble for -mfentry.  But the
scopes are inside-out -- [0] is the innermost -- so inlines were
comparing against their own entry pc, then inappropriately applying the
mfentry workaround.  It also assumes we're already focussed on the CU,
which may not be the case for .callee probes that expand across CUs.
And besides all that, dwarf_getscopes is an expensive traversal.

Instead, we can just keep a cached set of known entry PCs, built from
our existing cache of concrete functions, and check if possible mfentry
PCs are in that set.

ppc64le: Fix LEP usage for probing

PPC64 ELF ABI v2 has a Global Entry Point and a Local Entry Point for
the functions. Debuginfo of ELF contains GEP which is same as entrypc
while symbol table contains GEP and offset, from which we can calculate
LEP. LEP is used to call function within single CU, when TOC pointer
update is not required. Placing a probe on LEP catches call from both
the GEP and the LEP but, by default, systemtap probes on GEP.

Commit b4c6a4b1cd00 ("Prioritize symbol table lookup for ppc64le") solve
this issue by storing LEP in symbol table and prioritizing symbol table
over debuginfo for ppc64le.

But there are few regression effect of this patch. Couple of examples
are given below.

1. If target program is compiled without optimization and user is
interested in function parameter, systemtap should probe after function
prologue. But above patch forces probe on LEP and which result in garbage
value of function parameter will get recorded.

  $ make verbose=1 installcheck RUNTESTFLAGS='at_var.exp -v --debug'
    ...
    # of expected passes        1
    # of unexpected failures    1

2. Probe on shared library function with parameter is failing at Pass 2.

  $ make verbose=1 installcheck RUNTESTFLAGS='exelib.exp -v --debug'
    ...
    # of expected passes        10
    # of unexpected failures    64

3. When symbol_name with offset is used to register kprobe, kernel itself
will find LEP and adds offset to it. Systemtap using LEP to find offset
is resulting in offset being added two times.
  GEP + lep_offset (by systemtap) + lep_offset (by kernel)

This can be solved by calculating LEP only at a time of adding a probe.
That will make effect of LEP local to that area and won't have any
regression effect.

After applying patch:

  $ make verbose=1 installcheck RUNTESTFLAGS='at_var.exp -v --debug'
    ...
    # of expected passes        2

  $ make verbose=1 installcheck RUNTESTFLAGS='exelib.exp -v --debug'
    ...
    # of expected passes        74

Fixes: Commit b4c6a4b1cd00 ("Prioritize symbol table lookup for ppc64le")
Reported-by: Athira Rajeev <atrajeev@linux.vnet.ibm.com>
[ Reported about issue with shared library ]
Signed-off-by: Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com>

ppc64le: Store correct function entry address in symbol_table

PPC64 ELF ABI v2 has a Global Entry Point and a Local Entry Point for
the functions. Debuginfo of ELF contains GEP which is same as entrypc
while symbol table contains GEP and offset, from which we can calculate
LEP. LEP is used to call function within single CU, when TOC pointer
update is not required. Placing a probe on LEP catches call from both
the GEP and the LEP but, by default, systemtap probes on GEP.

For ppc64le, Systemtap stores LEP in symbol table and prioritize symbol
table over debuginfo. But, storing LEP in symbol table has couple of
regression effect. As LEP is only required at a time of adding a probe,
don't store it in symbol table.

No need to prioritize symbol table as well because debuginfo and symbol
table both will contain Global Entry Point.

Revert commit b4c6a4b1cd00 ("Prioritize symbol table lookup for ppc64le")
partially.

Signed-off-by: Ravi Bangoria <ravi.bangoria@linux.vnet.ibm.com>

Add 'list-unfinished' target to the top-level Makefile.

* Makefile.am: Add 'list-unfinished' target (that just defers to
  testsuite/Makefile).
* Makefile.in: Regenerated.

don't colorize stap -L for non tty

interactive mode command aliases

Add more aliases for interactive mode commands.

Interactive mode example commands

After null input while in interactive mode, a list of example commands are printed.

Update help commnd for interactive mode

help SUBCOMMAND will now print help description for only that one command.

Passing a command that requires arguments with no arguments will trigger help COMMAND.

stap.1 man page interactive entry

cleanup --help text

modify interactive mode load behavior

The "load" command no longer appends to the existing script. The user
is prompted if they want to overwrite the existing file if the script is
not empty.

Add testcase for RHBZ1269062.

fix interactive mode empty script saving

When saving an empty script with the "save" or "edit" command,
a runtime error from join() causes interactive mode to exit.

Fix PR20504 by updating xfs/nfs4 tracepoint support for kernel 4.7+.

* tapsets.cxx (tracepoint_extra_decls): Add includes/definitions for xfs
  and nfs4 tracepoints on kernel 4.7+.

PR14787: colorize stap -L probe points

Fix a '--rlimit-*' option problem identified by BZ1368188.

* session.cxx (systemtap_session::parse_cmdline): Improve the '--rlimit-*'
  options to report an error and return if the option value isn't
  specified or doesn't convert properly. If the limit can't be set,
  return if the error isn't EPERM.
* man/stap.1.in: Remove confusing language in the '--rlimit-*' option
  descriptions.
* testsuite/parseko/rlimit-as01.stp: New test case.
* testsuite/parseko/rlimit-as02.stp: Ditto.
* testsuite/parseko/rlimit-cpu01.stp: Ditto.
* testsuite/parseko/rlimit-cpu02.stp: Ditto.
* testsuite/parseko/rlimit-fsize01.stp: Ditto.
* testsuite/parseko/rlimit-fsize02.stp: Ditto.
* testsuite/parseko/rlimit-nproc01.stp: Ditto.
* testsuite/parseko/rlimit-nproc02.stp: Ditto.
* testsuite/parseko/rlimit-stack01.stp: Ditto.
* testsuite/parseko/rlimit-stack02.stp: Ditto.

Update procmod_watcher.stp example for more modern kernels.

* testsuite/systemtap.examples/process/procmod_watcher.stp: Update example
  to report 'clone' syscalls as 'fork' syscalls (since on some
  architectures like aarch64 fork() is implemented using clone()). Also
  report 'exit_group' syscalls as 'exit' syscalls.

Tweak [gs]etgroups syscall test cases for aarch64.

* testsuite/systemtap.syscall/getgroups.c: Relax looking for an error
  return for aarch64.
* testsuite/systemtap.syscall/setgroups.c: Ditto.

Make [nd_]syscall.sched_[gs]etaffinity probes fully optional.

* tapset/linux/syscalls2.stp (syscall.sched_getaffinity): Make fully
  optional for kernels with no 'affinity' support.
  (syscall.sched_setaffinity): Ditto.
* tapset/linux/nd_syscalls2.stp (nd_syscall.sched_getaffinity): Ditto.
  (nd_syscall.sched_setaffinity): Ditto.

PR20149: place all .function probes at entry

Function probes with a line specifier different than the declaration
line are forced into entry probes.

* tapsets.cxx: query_cu: fall through line probing if .function()
  present

add section for -T in stap --help

PR15932: printf %m/%M user variant

* man/stap.1.in: update documentation regarding %m/%M
* runtime/vsprintf.c: m/M cases in _stp_vsprint_memory now check for '#'/STP_SPECIAL flag
* testsuite/systemtap.printf/print_user_buffer.*: test case for new variant

store auto_path string in probe point

stap startup message: tweak invitation to use '-i' interactive mode

PR20433: simplify strcpy NULL-tolerance with inlined ?: ""

Reworking commit b91ef70c, without big need for a new _stp_strcmp
library function.

some auto_path test case

* main.cxx: Check for PATH directory in all include paths.
* testsuite/systemtap.base/auto_path.exp: New testcase.
* testsuite/systemtap.base/auto_path.c: Test program.
* testsuite/systemtap.base/tapset/PATH/*: Auto path tapsets.

move auto_path expansion to tapset resolution phase

* parse.cxx: parse_probe_points - Remove alias seen flag.
* staptree.h: struct probe_point - New auto_path flag.
* tapsets.cxx: dwarf_builder::build - Expand process component
  for /PATH/ directory.

Fix testsuite's list-unfinished makefile target

Do not only check systemtap.log in the main testsuite directory,
but also check other locations, namely the artifacts directory
used by the installcheck-parallel target.

Update interactive mode

* cmdline.cxx: introduce new '--interactive' switch
* cmdline.h: introduce new '--interactive' switch
* interactive.cxx: Print a few sample commands upon entering interactive mode.  Add new 'help' alias, '?'.
* session.cxx: Map new '--ineractive' switch to interactive mode.  When a script is not specified, new error message   suggests interactive mode.

RHBZ1312169: make stap-prep fall back to debuginfo-install

On some fedora / rhel boxes and their repo selections,
"debuginfo-install" can sometimes fetch the right files
even if "yum install" cannot.

Fix(ish) for PR20433 by avoiding a fedora kernel crash.

* runtime/vma.c (_stp_vma_mmap_cb): Use _stp_strcmp() to avoid a kernel
  crash.
* runtime/stp_string.c (_stp_strcmp): New function, a variant of strcmp()
  that treats NULL pointers as empty strings.
* runtime/stp_string.h: Add _stp_strcmp() declaration.

filename based auto path tapset support

Sometimes the process name may change with new versions and the
tapset must be modified. Process probe points in the PATH directory
may now have their arguments replaced by the full file path starting
from /PATH.

* parse.cxx: Pass alias_seen flag to parse_probe_points(). Add filepath
  argument to process component.

overloading testcase

* testsuite/semko/overload.stp: New testcase.

some function overloading optimizations

This removes some unnecessary code in the generated module.

* elaborate.cxx: Store next flag in functiondecl instead of in a set.
* staptree.h: struct functiondecl:: new has_next flag.
* translate.cxx: c_unparser::visit_functioncall - Don't generate
  redundant functioncall code for overloaded functions.

PR20217: overload warnings

Add a warning for overloaded functions that can't be reached
and also prevent them from being generated at all.

* elaborate.cxx: New struct function_next_check. New struct
  dead_overload_remover. New semantic_pass_overload function
  for optimization pass.

Fix a couple of conversions.stp tapset problems and improve its testing.

* tapset/linux/conversions.stp (kernel_buffer_quoted): Properly print the
  address as an unsigned long (instead of an unsigned int).
  (kernel_buffer_quoted_error): Add the faulting address in the error
  message.
* testsuite/buildok/conversions-embedded.stp: Add compile tests for
  several missing functions.

PR10485: auto-path tapset support

This prefixes the process probes placed in the tapset/PATH/ directory
with the location of the tapset.

* main.cxx: New collect_{stp,stpm} functions for passing to nftw(). Parse all
  .stp files placed in include_path[0]/PATH/. nftw() is now used instead of
  globbing for tapset searching to allow recursive traversal.
* parse.h: New pf_auto_path flag.
* parse.cxx: Prefix process argument with location of tapset.

PR19906: absolute pathnames for vfs.* probes

* tapset/linux/vfs.stp: Add a pathname variable to all probes.

PR20423: improve 'struct {...} being accessed instead of member' message

Esp. for anonymous structures, this message was not very helpful.
The new message is much more complete:

semantic error: 'struct {...}' (include/linux/types.h:175) is being accessed instead of a member such as '->counter': operator '->' at <input>:1:81
        source: probe kernel.function("dput") { println( @cast($dentry->d_inode, "struct inode")->i_count) }
                                                                                                ^

systemtap.spec: add systemd as BuildRequires & Requires if %{with_systemd}

It should have both "Requires: systemd" and "BuildRequires: systemd".
BuildRequires because it uses %{_unitdir} and %{_tmpfilesdir}, and
Requires because of systemctl in %post.

PR20416 cont'd: kprobe.function.return @entry too

Bonus round: jistone identified a twin in need of the same fix.

PR20416: handle @entry(@perf("..."))

The dwarf_var_expanding_visitor::visit_entry_op function inherited
some old logic from expanding $var's in .function().return probes.
This logic is not approproate for the more formal way by which current
code synthesizes a .call probe to collect @entry(EXPR) values.  Now we
pass the EXPR tree to the .call probe unmodified, for resolution
there, rather than in a fake non-.return context here.  This makes
@entry(@perf("...")) expressions work.

dejagnu testsuite driver: collect output during startup smoke test

Stop suppressing stderr; if something's wrong (like missing
kernel-debuginfo), let's show the user the whole message.

Fix a locking problem in the task_finder.

* runtime/linux/task_finder.c (stap_start_task_finder): Be sure to ulock
  the task if we exit early.
* runtime/linux/task_finder2.c (stap_start_task_finder): Ditto.

Add a non-optimized test to testsuite/systemtap.base/at_var.exp.

* testsuite/systemtap.base/at_var.exp: Add non-optimized test. Also, don't
  bother trying to run the executable if compilation failed.

Suppress empty builder error message

Use the standard testsuite 'kill' procedure in unprivileged_probes.exp.

Improve session state and transport state handling.

* translate.cxx (emit_module_init): At the start, only set the session
  state to STAP_SESSION_STARTING if we were in the
  STAP_SESSION_UNINITIALIZED state. Once the session is initialized, set
  the state to STAP_SESSION_RUNNING (if we're still in the
  STAP_SESSION_STARTING state).
* runtime/common_session_state.h: Add new 'STAP_SESSION_UNITIALIZED' state.
* runtime/dyninst/common_session_state.h (stp_session_init): Set initial
  session state to STAP_SESSION_UNITIALIZED.
* runtime/linux/common_session_state.h: Ditto.
* runtime/dyninst/linux_defs.h (atomic_cmpxchg): New function.
* runtime/transport/transport.c: Merge '_stp_start_called' and
  '_stp_exit_called' flags into a single flag called
  '_stp_transport_state'.

Merge branch 'csanting/PR19489'

pfaults.stp: correct the fault_entry_time index

Commit 31131f187a73 removed most manual tid() memoization, but missed
this particular use, and stap complained about the never-assigned 'id'.

Don't use as_root() to run stap within tapset_functions.exp

Most of the other as_root() in the testsuite is for stuff that
genuinely has to be root, like loading some test kernel module
that we'll probe.  For running the testsuite under a regular
user, stap{user,sys,dev} group membership mechanism should be
used instead.

stapprobes.3: Spell out that $context variables are probe-point specific.

Use our standard testsuite 'kill' procedure in several tests.

* testsuite/systemtap.base/flightrec1.exp: Use our "kill" procedure,
  instead of doing "exec kill ...".
* testsuite/systemtap.base/flightrec2.exp: Ditto.
* testsuite/systemtap.base/flightrec3.exp: Ditto.
* testsuite/systemtap.base/flightrec4.exp: Ditto.
* testsuite/systemtap.base/flightrec5.exp: Ditto.
* testsuite/systemtap.base/ipaddr.exp: Ditto.
* testsuite/systemtap.base/pr13158.exp: Ditto.
* testsuite/systemtap.base/remote.exp: Ditto.
* testsuite/systemtap.base/stapsh-unix.exp: Ditto.
* testsuite/systemtap.base/target_set.exp: Ditto.

Fix handling of internal transport flags.

* runtime/transport/transport.c: Remove unused '_stp_transport_mutex'
  mutex. Commit 440c8ee removed the mutex protection of the
  '_stp_start_called' and '_stp_exit_called' flags, so those flags have
  been converted to be atomics so that concurrent use of those flags see
  the correct values.

Make sure the unprivileged_myproc.exp testcase kills its processes.

* testsuite/systemtap.unprivileged/unprivileged_myproc.exp: Use library
  'kill' proc to make sure the test executables die.
* testsuite/lib/systemtap.exp (kill): Add 'AS_ROOT' argument, to kill the
  pid as root.

Update sendmmsg syscall test.

* testsuite/systemtap.syscall/sendmmsg.c: Update test to avoid a failur on
  32-bit RHEL6.

Tweak autocast-defined interactions further

- collapse basic @defined($foo) right away.
- last-ditch collapse other @defined(expr) to 1 or 0 depending on pe_unknown.
- run that last-ditch effort *before* turning on assert_resolvability.
- only run extra dead_control_remover for optimized runs
- in var_expanding_visitor, pass *any* unchanged expr through, so they
  may be decided later.  (e.g. for @choose_defined ternaries)

Increase the difficulty of semok/autocast14.stp

PR18079: support nested autocast / @defined

We now perform const-folding & dead-code-elision during the type
resolution loop, whenever an autocast expression gets evaluated.  This
way, @defined(foo()->mm) type expressions can work as nature intended.

This requires @defined() not to be short-circuit evaluated to 0 during
a random const_folding process, so a flag is introduced to control its
preservation or collapsing.  For the last (assert_resolvability) pass
in the type resolution loop, this flag is set to true, so that
genuinely unresolvable @defined($expressions) do get mapped to 0 in
time for a last elision.