Remove manual pointer casts from map-gen and pmap-gen

There were a lot of manual conversions from generic map_node* to the
KEYSYM variants (which had a duplicate of map_node at their heads).

Now there's a genuine map_node in each KEYSYM node, and we use the
trustworthy container_of and hlist_for_each_entry to go back and forth.

stapdyn: Workaround for an i686 Dyninst bug in argument passing

Dyninst appears to be filling only 32-bits for the index to
enter_dyninst_uprobe_regs, which makes the following arguments shift
badly too.  This is reported upstream, but we can detect and work around
it ourselves too.

stapdyn: Add tapset function print_regs

The underlying _stp_print_regs is more limited than the kernel version,
as some registers are not available, but this is still useful to see all
the information available to the probe.

Note: this makes it obvious that i686 actually doesn't have register
values right now.  Dyninst has documented that only POWER and AMD64
implement getRegisters, unfortunately.  This might be expanded by them
in a future release.

* runtime/regs.c: Split into linux/ and dyninst/ versions.
* tapset/context.stp (print_regs): Moved from linux/; now it's generic.
* stapdyn/mutatee.cxx (get_dwarf_registers): Add verbose logging, which
  also shows that i686 doesn't have registers. :(

Include --runtime=dyninst in error message for unprivileged user

* session.cxx : mention that --runtime=dyninst may be used in the case
 of an unprivileged user, when trying to probe userspace

stapdyn: Add support for return probes

Each address is converted to the BPatch_locExit of the function which
contains it.  Even $return is working fine from here!

* runtime/dyninst/uprobes.c (stp_dyninst_probe_flags): New, return the
  flags describing this probe, notably STAPDYN_PROBE_FLAG_RETURN.
* stapdyn/dynprobe.cxx (find_dynprobes): Read the flags from the module.
  (dynprobe_location::dynprobe_location): Parse return_p from the flags.
* stapdyn/mutatee.cxx (mutatee::instrument_dynprobe_target): For probes
  with return_p set, transform into BPatch_locExit points.
* tapsets.cxx (dwarf_builder::build): Let stapdyn return probes through.
  (uprobe_derived_probe_group::emit_module_dyninst_decls): Emit
  .flags=STAPDYN_PROBE_FLAG_RETURN for return probes.

stapdyn: Reduce LDADD to only those directly used

The only part of Dyninst that we actually use and need to link is
libdyninstAPI.  The rest of those libraries I had in dyninst_LIBS were
those needed indirectly by that one.  But trying to enumerate someone
else's dependencies is a recipe for failure -- e.g. it was already
broken by not including the new libdynElf too.

The reason I had this was to allow successful linking when using
--with-dyninst=/not/a/lib/searched/path without requiring also to set
LD_LIBRARY_PATH.  Since the enumeration is prone to break, and you'd
have to set LD_LIBRARY_PATH anyway to be able to run, I've decided it's
not worth trying to be fancy here.

configure: treat any positive --with-dyninst as a hard req

If the user gave --with-dyninst[=yes] or --with-dyninst=/path, treat a
failure in that feature test as a hard error.  If the user left dyninst
implicit, then don't make it an error.  And of course if the user gave
--without-dyninst, then don't test it at all.

Use the new kernel_functions to query inode-uretprobes

This gives a clean way to determine if this is a kernel with the new
uretprobe_register function yet.

Add System.map kernel functions to systemtap_session

The code to parse System.map was already in place for kprobe.function
validation; now it's generalized and the result made available in
systemtap_session::kernel_functions.

Move parse_kernel_config/exports into systemtap_session

These are operating on the session, with purely session-internal data,
so they might as well be member functions.

Prototype support for inode-uretprobes

Anton Arapov sent a test kernel build with new support for uretprobes,
and this patch adds stap support.  I've confirmed that it works on
x86_64, but i686 is crashing the target processes with SIGSEGV.

We need to find a dynamic (pass-2) way to detect the presence of
uretprobes in the kernel.  For now, I've just hacked it enabled, and it
will fail registration at runtime.

* buildrun.cxx (compile_pass): Autoconf uretprobes and its exports.
* runtime/linux/autoconf-inode-uretprobes.c: New, check for uretprobes.
* runtime/linux/runtime.h: Add function pointers for uretprobes.
* runtime/transport/transport.c (_stp_transport_init): Set the uretprobe
  function pointers if not exported.  (Anton's kernel does have exports,
  but we're prepared in case upstream doesn't accept those.)
* runtime/linux/uprobes-inode.c (stapiu_consumer): Let this double as
  either a uprobe_consumer or a uretprobe_consumer, chosen by return_p.
  (stapiu_probe_handler): Add a new generic function signature.
  (stapiu_uprobe_handler, stapiu_uretprobe_handler): New specific
  callbacks that mux into stapiu_probe_handler.
  (stapiu_register, stapiu_unregister): New functions to perform the
  appropriate kind of registration for a probe.
* tapsets.cxx (dwarf_builder::build): Hack the uretprobes check.
  (uprobe_derived_probe_group::emit_module_inode_decls): Use the new
  signature for the probe handler, and emit .return_p.

 spin-rawhide date command and correct ^Source regex

* scripts/spin-rawhide : adjust date to use +"%a..." syntax instead
 of cutting fields and remove backslash before ^Source sed regex

PR14707: Pass -x from stap to stapdyn

Commit 18e3d52a added -x to stapdyn, and I tested that it worked
directly, but I forgot to make stap pass its option along.  D'oh!

* buildrun.cxx (make_dyninst_run_command): Add -x s.target_pid.

Merge branch 'master' of ssh://sourceware.org/git/systemtap

Include stapgames in the testsuite

New systemtap.examples/stapgames directory added with the supporting
tapsets located in the 'tapset' subdirectory.  All tests run with -p4
specified.

* testsuite/systemtap.examples/stapgames/block.meta : block meta data
* testsuite/systemtap.examples/stapgames/block.stp : break a ceiling of
  blocks
* testsuite/systemtap.examples/stapgames/eater.meta : eater meta data
* testsuite/systemtap.examples/stapgames/eater.stp : eat the .'s before
  being eaten
* testsuite/systemtap.examples/stapgames/lifegame.meta : life meta data
* testsuite/systemtap.examples/stapgames/lifegame.stp : watch a
  'lifeform' morph and evolve
* testsuite/systemtap.examples/stapgames/pingpong.meta : ping pong meta
  data
* testsuite/systemtap.examples/stapgames/pingpong.stp : cursor bounces
  around terminal reflecting off edges
* testsuite/systemtap.examples/stapgames/tapset/audio.stp : audio
  functionality
* testsuite/systemtap.examples/stapgames/tapset/cursor.stp : cursor
  functionality and tracking
* testsuite/systemtap.examples/stapgames/tapset/game.stp : seed values
* testsuite/systemtap.examples/stapgames/tapset/gmtty.stp : tty functionality
* testsuite/systemtap.examples/stapgames/tapset/input.stp : controlling
  game input

PR14794: Only error if '--with-dyninst' can't find dyninst.

* configure.ac: Only error when dyninst can't be found and '--with-dyninst'
  was specified.
* configure (ac_compiler_gnu): Regenerated.

PR14794: enable --with-dyninst by default, if libraries/headers exist

* configure.ac (--with-dyninst): Search for headers/libraries, unless
  --without-dyninst.

releng: improve 'make gcov'

* Makefile.am (gcov): Rewrite to compile all the stap tools with the proper
  *FLAGS.  Iterate to run gcov in $SUBDIRS.
  (lcov): Zap.

(PR14574 partial fix) Make several systemtap.printf/*.exp tests use dyninst.

* testsuite/lib/stap_run_binary.exp (stap_run_binary): Added optional
  argument support.
* testsuite/systemtap.printf/basic1.exp: Added dyninst support.
* testsuite/systemtap.printf/basic2.exp: Ditto.
* testsuite/systemtap.printf/basic3.exp: Ditto.
* testsuite/systemtap.printf/basic4.exp: Ditto.
* testsuite/systemtap.printf/basic5.exp: Ditto.
* testsuite/systemtap.printf/basic6.exp: Ditto.
* testsuite/systemtap.printf/bin1.exp: Ditto.
* testsuite/systemtap.printf/bin2.exp: Ditto.
* testsuite/systemtap.printf/bin3.exp: Ditto.
* testsuite/systemtap.printf/bin4.exp: Ditto.
* testsuite/systemtap.printf/bin5.exp: Ditto.
* testsuite/systemtap.printf/bin6.exp: Ditto.
* testsuite/systemtap.printf/char1.exp: Ditto.
* testsuite/systemtap.printf/char2.exp: Ditto.
* testsuite/systemtap.printf/int1.exp: Ditto.
* testsuite/systemtap.printf/memory1.exp: Ditto.
* testsuite/systemtap.printf/oct.exp: Ditto.
* testsuite/systemtap.printf/pretty-bits.exp: Ditto.
* testsuite/systemtap.printf/pretty-char.exp: Ditto.
* testsuite/systemtap.printf/print.exp: Ditto.
* testsuite/systemtap.printf/print_char.exp: Ditto.
* testsuite/systemtap.printf/printd.exp: Ditto.
* testsuite/systemtap.printf/println.exp: Ditto.
* testsuite/systemtap.printf/ptr-1.2.exp: Ditto.
* testsuite/systemtap.printf/ptr-1.3.exp: Ditto.
* testsuite/systemtap.printf/string1.exp: Ditto.
* testsuite/systemtap.printf/string2.exp: Ditto.

(PR14571 partial fix) Improve dyninst multithreading.

* translate.cxx (c_unparser::emit_common_header): Don't declare contexts,
  let the runtime handle it.
  (c_unparser::emit_module_init): Instead of allocating/freeing contexts
  directly, call the runtime functions
  _stp_runtime_contexts_alloc()/_stp_runtime_contexts/free().
  (c_unparser::emit_module_exit): Instead of waiting on contexts directly,
  call the runtime function _stp_runtime_context_wait().
* tapsets.cxx (common_probe_entryfn_prologue): Updated with new method of
  getting context in probe prologues. For dyninst, handle a NULL context.
  (common_probe_entryfn_epilogue): Call _stp_runtime_entryfn_put_context()
  in probe epilogue.
* runtime/dyninst/runtime_context.h: New file.
* runtime/linux/runtime_context.h: Ditto.
* runtime/common_probe_context.h: Added dyninst fields.
* runtime/dyninst/map_runtime.h: Make map locking optional.
* runtime/dyninst/print.c (_stp_print_init): Added multithreading support.
* runtime/dyninst/runtime.h: Remove the stapdyn_big_dumb_lock.
  (stp_dyninst_session_init): Call _stp_runtime_contexts_init() and
  _stp_print_init().
  (stp_dyninst_session_exit): Call _stp_print_cleanup().
* runtime/dyninst/stat_runtime.h: Make stat locking optional.
* runtime/stat.h: Ditto.
* runtime/linux/map_runtime.h (_stp_map_for_each_cpu): Remove unneeded macro.
* runtime/linux/runtime.h: Removed unused macros.
* runtime/linux/stat_runtime.h (_stp_stat_initialize_locks): Switched back
  to standard lkm macros.
* runtime/map-stat.c: Ditto.
* runtime/map.c: Ditto.
* runtime/pmap-gen.c: Ditto.
* runtime/stat.c: Ditto.
* runtime/map.h: Make locking optional.
* runtime/runtime.h: Added forward declarations for runtime_context.h.
* runtime/runtime_context.h: Include the correct runtime's
  runtime_context.h.
* runtime/vsprintf.c: Update method of getting context.

Allow spin-rawhide to be run from any directory

* scripts/spin-rawhide: replace HASH suffix with mktemp function
reposition constant/variable definitions to draw
  from newly cloned systemtap.git vs pwd
make STAP_MAJOR_VERSION grep more specific
make Source grep more specific

PR14707: Support stapdyn -x PID

* stapdyn/stapdyn.cxx (main): Parse -x and use attach_process for it.
* stapdyn/mutator.cxx (mutator::attach_process): Implement it.

PR14665: Start handling signals in stapdyn

This works somewhat to receives signals and kill the target process, but
sometimes it hangs.  I think it may work better when the notification FD
is resolved.

stapdyn: Detach the process in the mutatee dtor

This doesn't really matter for now, since we're only tracking a single
process that we created, but we'll need if for -x attaching.

stapdyn: Sketch out support for polling on Dyninst

By polling instead of waitForStatusChange, we can let signals break us
out of the run loop too.  However, the notification FD currently appears
broken in Dyninst, so this is "#if 0" for now.

* util.cxx (ppoll): Move the fallback ppoll() to a general location.
* remote.cxx (ppoll): Moved ppoll() from here.
* stapdyn/mutator.cxx: Add the #if 0 stub of notification FD support.

Added small fix for testsuite/systemtap.base/hash_add_buildid.exp.

* testsuite/systemtap.base/hash_add_buildid.exp: Wait for eu-readelf to
  finish to avoid a zombie process.

(PR14574 partial) Add dyninst to the systemtap.exelib/pthreadprobes.exp test.

* testsuite/systemtap.exelib/pthreadprobes.exp: Add dyninst testing.

stapdyn: Add some support for removing instrumentation

We'll need this for signal-handling cleanup, and for detaching from
-x PID target processes.

* stapdyn/mutatee.h (mutatee::snippets): New vector for all the handles
  to code that we've added with Dyninst insertSnippet.
* stapdyn/mutatee.cxx (mutatee::instrument_dynprobe_target): Save the
  insertSnippet handle in snippets.
  (mutatee::remove_instrumentation): New, remove all the snippets.
  (mutatee::unload_stap_dso): New placeholder for unloading our module
  from the target process.  Dyninst can't do this yet...
  (mutatee::~mutatee): Call remove_instrumentation and unload_stap_dso.

stapdyn: Manage mutatee lifetimes better

It will be more convenient to manage mutatee resources if there's one
clear owner of the process, so stop allowing implicit copies.

* stapdyn/mutatee.h (mutatee::mutatee): Don't implement copy ctors.
* stapdyn/mutator.h (mutator::mutatees): Use shared_ptr for this list.
* stapdyn/mutator.cxx (mutator::create_process): Save a shared_ptr.
  (mutator::run): Use pointer indirection for mutatees.
  (mutator::dynamic_library_callback): Ditto.

stapdyn: Manage mutator lifetime better

We need to have a global record of mutators in order for dyninst
callbacks to work, since they have no context attached.  But rather than
some singleton factory style, just keep a global list of all mutators
we've seen.

* stapdyn/mutator.cxx (g_mutators): New vector for all mutators.
  (g_dynamic_library_callback): Iterate callbacks for all g_mutators.
  (mutator::dynamic_library_callback): Do the individual work.
  (mutator::mutator): Add this to g_mutators.
  (mutator::~mutator): Remove this from g_mutators.
  (mutator::create, mutator::g_mutator): Removed.
* stapdyn/stapdyn.cxx (main): Use mutator new and auto_ptr delete.

stapdyn: Remove the 8-arg register workaround

This was only to work around an assertion failure in Dyninst, which has
been fixed by their commit f307d8e1c5d7bd6a20dc105c3e233b788f57d33b.

testsuite: for pretty-* test cases, use fixed /tmp/ file names

... so that systemtap.sum files from run to run contain the same string

testsuite: for stapsh-unix.stp test case, use fixed path name for tmp socket

... so that systemtap.sum files from run to run contain the same string

PR14774: fix multiply-wildcarded statement probes

* dwflpp.cxx (iterate_over_srcfile_lines): Fix handling of case where
  both function name and line number subfields are wildcards.

testsuite: make stap_run[23] more verbose when failing

If a stap_run[23] test case fails, in that the actual output varies
from the expected output, all remaining actual output lines were
neither compared nor reported in systemtap.log.

* testsuite/lib/stap_run2.exp (stap_run3): In case of a line mismatch,
  report any remaining actual lines.

PR14781: use kmem_cache_create with unique name

* runtime/stp_utrace.c (utrace_init): Instead of using the KMEM_CACHE macro,
  invoke kmem_cache_create directly, with cache names that are going to be
  unique across concurrent systemtap sessions.

PR14781 part 1: don't panic

* runtime/stp_utrace.stp (utrace_init): Don't SLAB_PANIC on duplicate
  kmem_cache names, as is sadly likely for concurrent inode-uprobes
  scripts running on CONFIG_DEBUG_VM kernels.
* runtime/linux/task_finder2.c (stap_start_task_finder): Deign to check
  the utrace_init return code, and fail with a touch of grey grace.

Updated locking comments in _stp_stat_get().

Add spin-rawhide script for automated rawhide releases

* scripts/spin-rawhide: freshly clones systemtap.git and fedpkg git
  repositories, update the .spec file accordingly and submit new build to
  bodhi

Log the command being executed in testsuite/lib/stap_run_exact.exp.

(PR14571 partial fix) Rework dyninst stat/map data storage.

* runtime/map.c: Switched from TLS to just an array of map data for
  dyninst mode. Pushed most of the kernel vs. dyninst code down to
  runtime/{linux,dyninst}/map_runtime.h.
* runtime/map-stat.c (_stp_map_new_hstat_log): Ditto.
* runtime/map.h: Ditto.
* runtime/pmap-gen.c: Ditto.
* runtime/stat.c (_stp_stat_add): Switched from TLS to just an array of
  stat data for dyninst mode. Pushed most of the kernel vs. dyninst code
  down to runtime/{linux,dyninst}/stat_runtime.h.
* runtime/stat-common.c: Ditto.
* runtime/stat.h: Ditto.
* runtime/dyninst/map_runtime.h: New file.
* runtime/dyninst/stat_runtime.h: New file.
* runtime/linux/map_runtime.h: New file.
* runtime/linux/stat_runtime.h: New file.
* runtime/dyninst/tls_data.c: Deleted file.

runtime: fix timers for old kernels

This is a regression from commit 2b5ecaf, which split the hrtimer code
into the runtime.  The kernel portion needs to vary the return type of
the callback function for old kernels, using int instead of the enum.
This patch defines hrtimer_return_t to abstract the difference.

tapset: Add the macro @choose_defined and use it

This returns the first parameter if it's @defined, otherwise it returns
the second parameter, replacing the common @defined(x)?x:y pattern.

I only updated the tapsets to use this, as I feel that's where we should
show off good code practice.  There are opportunities in the testsuite
that could use it too, but it's not necessary, and probably better to
leave existing tests undisturbed when possible.

NEWS: retroactively note previous probefunc() is available as --compatible=1.8

fix i18n whitespace nit

* main.cxx (passes_0_4): Drop extra space inside " foo " in listings
  of files parsed within tapset directories.

Adjust buildok/xtime.stp for kernel commit 1e75f.

time: Condense timekeeper.xtime into xtime_sec.

PR14742 - more complete dwz / DW_TAG_partial_unit support

Don't use DW_TAG_partial_units directly in dwflpp::iterate_over_cus
and dwarf_var_expanding_visitor::getcuscope. Descent into imported_units
in has_only_decl_members, dwflpp::iterate_over_globals,
dwflpp::iterate_over_types, dwflpp::iterate_over_labels
dwflpp::inner_die_containing_pc, dwflpp::print_locals,
dwflpp::print_members, dwflpp::find_struct_member and
dwarf_pretty_print::recurse_struct_members.

PR14745: note possibility of probe registration failure for timer.profile

Added a bug fix in _stp_stat_init()'s error handling code.

* runtime/stat.c (_stp_stat_init): Be sure to free memory allocated with
  _stp_alloc_percpu() with _stp_free_percpu() - not _stp_kfree().

samples: add baby-ltrace example for .plt probes

documentation: elaborate slightly on .plt probes

Fix .exported typo in stapprobes manpage.

Fixed PR14735 by adding timing ('-t') mode support for dyninst.

* tapsets.cxx (common_probe_entryfn_prologue): Added dyninst support for
  timing ("-t") mode. Disable overload processing for dyninst mode.
  (common_probe_entryfn_epilogue): Change funcion signature. Add dyninst
  support for timing ("-t") mode. Disable overload processing for dyninst
  mode.
  (dwarf_derived_probe_group::emit_module_decls): Update
  common_probe_entryfn_epilogue() call.
  (uprobe_derived_probe_group::emit_module_utrace_decls): Ditto.
  (kprobe_derived_probe_group::emit_module_decls): Ditto.
  (hwbkpt_derived_probe_group::emit_module_decls): Ditto.
  (tracepoint_derived_probe_group::emit_module_decls): Ditto.
* tapset-been.cxx (be_derived_probe_group::emit_module_decls): Ditto.
* tapset-itrace.cxx (itrace_derived_probe_group::emit_module_decls):
  Ditto.
* tapset-mark.cxx (mark_derived_probe_group::emit_module_decls): Ditto.
* tapset-netfilter.cxx (netfilter_derived_probe_group::emit_module_decls):
  Ditto.
* tapset-perfmon.cxx (perf_derived_probe_group::emit_module_decls): Ditto.
* tapset-procfs.cxx (procfs_derived_probe_group::emit_module_decls): Ditto.
* tapset-timers.cxx (timer_derived_probe_group::emit_module_decls): Ditto.
  (hrtimer_derived_probe_group::emit_module_decls): Ditto.
  (profile_derived_probe_group::emit_module_decls): Ditto.
* tapset-utrace.cxx (utrace_derived_probe_group::emit_module_decls): Ditto.
* runtime/dyninst/runtime.h: Added NSEC_PER_SEC and _stp_timespec_sub
  macros. Always include stat.c, as runtime/linux/runtime.h does.

stapdyn: Big refactoring

The main additions are the "mutator" class working with the overall
session, and the "mutatee" class working with the individual targets.
This is a bit of code churn, with no functional change, but it should
make it easier to keep track of our current state and add new features.

PR14731: fix PR12022 regression for synthetic foreach()

* elaborate.cxx (add_global_var_display): For foreach_loop created
  for otherwise-unused global, initialize sort_aggr => sc_none, the
  same way the parser would.

documentation: mention non-semaphore mode for dtrace(1) headers

Fixed PR14701 by adding dyninst timer probe support.

* tapset-timers.cxx (hrtimer_derived_probe_group::emit_interval): Removed
  function.
  (hrtimer_derived_probe_group::emit_module_decls): Pushed some code down
  to the runtime and added dyninst support.
  (hrtimer_derived_probe_group::emit_module_init): Ditto.
  (hrtimer_derived_probe_group::emit_module_exit): Ditto.
  (timer_builder::build): Throw semantic errors if 'timer.jiffies' or
  'timer.profile' probes are used in dyninst mode.
  (register_tapset_timers): Add fake privilege for 'timer.profile' probes
  when in dyninst mode (so we get a semantic error, not a privilege error).
* buildrun.cxx (compile_dyninst): Added '-lrt' for timer functions.
* testsuite/systemtap.pass1-4/buildok-dyninst.exp: Move some tests to the
  kfail list.
* runtime/timer.c: New file.
* runtime/dyninst/timer.c: Ditto.
* runtime/linux/timer.c: Ditto.

PR14712: Correct the bug number in a comment

I mixed this up in the log too; commits 529c7eae and 3df148cb
are really for PR14712, not PR14172.

testsuite: chmod a+x some semok testcases

PR12022 test cases for foreach (... @aggr ...) sorting

PR12022: lightly document new foreach (@aggr) sorting option

PR12022: support foreach sorting by user-selected aggregate operator

The syntax goes:

    foreach ([x,y] in array @avg -)  { }

inserting the desired sorting aggregator between the array name and
the +/- The runtime has been ready for this for a long time (see the
runtime/map.c SORT_* macros), finally time for the translator to let
users enjoy it.

* staptree.h (stat_component_type): Add sc_none possibility.
  (foreach_loop): Add sort_aggr field.
* parse.cxx (parse_foreach_loop): Parse optional statistics @ops.
* staptree.cxx (stat_op::print): Reject sc_none.
  (foreach_loop::print): Print statistics @ops.
* elaborate.cxx (typeresolution_info::visit_foreach_loop): Mark
  invalid @op sorting on non-statistics arrays.
* translate.cxx (c_unparser::visit_foreach_loop): Map sort_aggr
  to runtime SORT_* value, emitted as strings.
  (c_unparser::visit_stat_op): Reject sc_none.

man pages: elaborate more on probe syntax/semantics in stapprobes.3stap

stap.1 man page: take probe point syntax details back out

Seeing that stapprobes.3stap is fine, replace with a better cross-reference.

stap.1: mention a little about the probe point !/? operators

... even though they're listed in stapprobes.3stap also.

langref: add a sentence identifying the 'any' meaning of probe-point lists

PR14172: Fix the include order so vma tracking works

In commit 529c7eae, by moving the sym.c include, I broke the order
between #define HAVE_TASK_FINDER and the #ifdef in vma.c.

Reported-by: Yichun Zhang <agentzh@gmail.com>

PR14172: Fix for kernels without VM_EXECUTABLE

We were using VM_EXECUTABLE in two ways:

1) In task_finder for locating the process executable among all the
vmas.  Since around 2.6.26 there is also mm->exe_file, which will serve
this purpose just fine.

2) In uprobes to avoid relocation offset for semaphores in ET_EXEC
files.  This is actually incorrect, but harmless, because the callback
path for ET_EXEC targets already sets relocation=offset=0 anyway.  So we
can just remove the special case for VM_EXECUTABLE altogether.

* runtime/task_finder_vma.c (stap_find_exe_file): New, locate the
  process executable either by VM_EXECUTABLE or mm->exe_file.
* runtime/linux/task_finder.c (__stp_get_mm_path): Use stap_find_exe_file.
* runtime/linux/task_finder2.c (__stp_get_mm_path): Ditto.
* runtime/linux/uprobes-common.c (stap_uprobe_change_plus): Don't
  special case for VM_EXECUTABLE (and add a comment why).
* runtime/linux/uprobes-inode.c (stapiu_change_plus): Ditto.
  (stapiu_get_task_inode): Use stap_find_exe_file.

bump up SYSTEMTAP_CACHE_CLEAN_DEFAULT_INTERVAL_S to 300 (5 minutes)

testsuite: tweak net-sanity.exp test

It shouldn't use stap -vv, ashat creates too much noise in the .log file.
It shouldn't use 0xd34db33f as a known-bad address, because sometimes it's good.
Really good.  Mmmm, tasty, BBQ goodness good.  Honey, time to warm up the burners!

PR14245 stapio should not pass inherited relay_basedir_fd

jistone reported that new fd passing should be suppressed for stapio-spawned
child processes like target_cmd or function::system().

* staprun/stapio.c (main): Set CLOEXEC on relay_basedir_fd (if any).

PR14245 clean up error messages for staprun -d SOMETHING_AWFUL

jistone reported that the previously moved
"ERROR: no access to debugfs; try "chmod 0755 /sys/kernel/debug" as root"
message was appearing for erroneous staprun -d FOOBAR cases.

* ctl.c (init_ctl_channel): Print a more general error for any .ctl-file
  opening failure.

PR14245: fix staprun->stapio -F<fd> passing for -A (attach) mode

jistone reported that the recent -F<fd> code was not passing on
a suitable basedir fd to stapio's forked for re-attach (-A) mode.
Make it so.

* staprun.c (init_staprun): In attach_mode, do a dummy
  init_ctl_channel, just to gather a relay_basedir_fd .

stapdyn: remove the TODO in favor of bugzilla

staprun: let stap -vv (= staprun -v) print original/renamed module name before load

BZ863832 part 2, add stap-server home directory into %install too

stapdyn: Fixed the verbose logging of probes found

* stapdyn/stapdyn.cxx (find_uprobes): Use j to index the inner loop.

Set staprun's verbosity one less than stap

Prior to commit e520ea8, staprun would get one -v for s.verbosity>1 and
a second -v for s.verbosity>2.  That commit unbounded the number of -v
for staprun, but changed the off-by-one, and staprun and stapio are too
chatty for that.

This also now sets stapdyn verbosity the same way.

* buildrun.cxx (make_run_command): Give one less -v flag.
  (make_dyninst_run_command): Set stapdyn -v flags the same way.

Fixed PR14659 so that ptrace can be used on tasks probed with systemtap.

* runtime/stp_utrace.c (utrace_set_events): No longer set
  TIF_SYSCALL_TRACE on the target task.
  (utrace_reset): No longer clear TIF_SYSCALL_TRACE on the target task.
* testsuite/systemtap.base/ptrace.exp: New testcase.

BZ863832 add stap-server home directory to our rpm payload

dwalsh advises that doing so may make selinux more compliant with
our need for an unusually placed home directory for stap-server.

* systemtap.spec (-server %files): Add %dir /var/lib/stap-server.

PR14245: support /sys/kernel/debug mounted 0700

This is done by staprun passing a file descriptor for the
/sys/kernel/debug/systemtap/stap_MODULE directory from staprun
(running setuid) to stapio (running unprivileged, previously unable to
traverse to that path itself). This FD passing is done with a new
option -F<fd> for stapio (though by accident staprun also accepts (and
rejects) this option).

Since openat(2) is relatively recent, autoconf macros are used to back
down to graceful failure on older kernels, and to hide the new code.
New staprun always uses -F<fd> to stapio, even if permissions on
/sys/kernel/debug do not require it.

* staprun/common.c (relay_basedir_fd): New variable.
  (parse_args): Parse new -F: option.
  (usage): Document it.
* staprun/staprun.h: Corresponding changes.
* staprun/ctl.c (init_ctl_channel): Reorganize to try an incoming
  relay_basedir_fd first (with a faccessat cross-user check) first.
  Try to compute a relay_basedir_fd if not already set.
* staprun/mainloop.c (read_buffer_info): Note ignoring of this PR facility on
  RHEL4-era old_transport.
* staprun/relayfs.c (init_relayfs): Attempt to open relay_fd[] using
  relay_basedir_fd if specified.
* staprun/stapio.c: Top secret.
* staprun/staprun.c (main): Don't allow staprun itself to take -F, for it
  could be misused by a very bad person (tm).  However, arrange to pass
  it to stapio, if we have incidentally discovered a good relay_basedir_fd.
* staprun/staprun_funcs.c (mountfs): Drop access_debugfs() check at this
  point, as init_ctl_channel() will do the check later.

PR14245: have configury look for openat(2) syscall

... since it will be used for fd passing between staprun & stapio

staprun: arrange -v params commeasurate with stap -v

* buildrun.cxx (make_run_command): Instead of a meager zero-to-two
  -v's for staprun, add just as many as s.verbose currently indicates.

PR14555: tweak & simplify warning messages for _stext=0 message handling

PR14555: handle 0 _stext relocs from userspace by kallsyms_lookup_name fallback

* runtime/transport/symbols.c (_stp_do_relocation): For an incoming
  _stext=0 relocation (such as for /proc/sys/kernel/kptr_restrict = 2),
  fall back to kallsyms_lookup_name.

stap.1: note dyninst alternative to lkm in first paragraph

post-release version bump: 2.1 coming down the line

Add 2.0 release dates to NEWS and systemtap.spec

Make the tapcheck.sh look for all .stp files in the tapset directory

With the reorganization of the tapset directory there were some tapsets
multiple directories down.  tapcheck.sh was only checking the top level
of the tapset directory.  This meant that is was missing a many of the
tapsets that were in subdirectories.  This change makes the results more
accurate.

NEWS - prose tweak for %( systemtap_privilege ... entry

AUTHORS bump

Fix docs for %( systemtap_privilege tweak

change %( systemtap_privilege check to use stapdev/stapsys/stapusr names

PR14655: more politely fail if kallsyms_lookup_name unexported

* buildrun.cxx (compile_pass): Look for said export.
* runtime/sym.c: Provide a _stp_error()-laden backup.

PR14632: Some typos in documents

There were some articles duplicated in the SystemTap Beginners
Guide and typos the systemtap tutorial.

Rename session::is_usermode to runtime_usermode_p

Just appeasing the aesthetic sensibilities of a heckler...

PR14572: Set s.privilege = unprivileged for stapdyn

When running under Dyninst, we are effectively unprivileged by nature,
so setting s.privilege to reflect that helps restrict the available
probe types.

However, we still want to allow guru mode for setting target variables
and using embedded-C, so let systemtapr_:session::is_usermodea() pass.

* session.cxx (systemtap_session::parse_cmdline): For --runtime=dyninst,
  set the privilege level too.
  (systemtap_session::check_options): Allow -g for is_usermode().
* staptree.cxx (varuse_collecting_visitor::visit_embeddedcode): Allow
  embedded-C unrestricted for is_usermode().
  (varuse_collecting_visitor::visit_embedded_expr): Ditto.

Change NEWS to conform to added relnotes announcements.

PR14682: fix _stp_pmap_agg error exit path

commit 82523f19 changed the error-exit path of _stp_pmap_agg, but was
confused by the multiple (three!) levels of nested loops in effect at
the point of failure.  While the prior "return;" skipped an overall
(newly needed) aggregate-unlock; the current "break;" skipped too
little.  Switch to a proper simple goto to almost but not quite
return;.

testsuite: set -B CONFIG_DEBUG_INFO=y in testsuite rc to help debug crashes