PR11684 cont'd

* doc/langref.tex: remove all references to tapset-defined functions.
  Leave in just "formatted output" a la printf.  Correct explanation
  of $1 token-pasting.  Still several bits left to fix later.

doc build: make clean should clean up dvi files too

Banish the SyScall

We don't need to deal with the SYSCALL_WRAPPERS-generated SyS_foo,
because since the dawn of that feature they have also provided a normal
sys_foo alias.  So as long as alias tracking is working, both in our
translator and in kallsyms for kprobe.function, we can be blissfully
ignorant.

http://sourceware.org/ml/systemtap/2010-q2/msg00551.html

Fixed PR11707 so that nd_syscall probes work on ppc64 systems.

* tapset/powerpc/registers.stp (_stp_get_register_by_offset): Fixed
  compile error.
  (_stp_arg): Fixed logic error.  The last case was missing an "if", which
  caused all _stp_arg() to only return the value of r10.

Added endian.stp/errno.stp build testcases.

* testsuite/buildok/endian-embedded.stp: New endian.stp testcase.
* testsuite/buildok/errno-embedded.stp: New errno.stp testcase.
* testsuite/buildok/ioscheduler-all-probes.stp: Made executable.

Added new dev.stp and ctime.stp build testcases and updated dentry testcase.

* testsuite/buildok/conversions-guru-embedded.stp: Renamed from
  set_kernel.stp.
* testsuite/buildok/ctime-embedded.stp: New testcase.
* testsuite/buildok/dentry-embedded.stp: Renamed from dentry.stp and added
  the rest of the functions from dentry.stp.
* testsuite/buildok/dev-embedded.stp: New testcase.

Improved aux_syscalls.stp build testcase.

* testsuite/buildok/aux_syscalls-embedded.stp: Added the rest of the
  functions from aux_syscalls.stp.

Small updates to conversion tapset and testcases.

* tapset/conversions.stp: Fixed documentation string.
* testsuite/buildok/conversions.stp: Added kernel_string_n() test.
* testsuite/buildok/conversions-embedded.stp: Ditto.

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

fixing the errors in langref.tex and reformatted the documentation in the tapset/*.stp

uprobes_ppc: Don't emulate privileged MSR opcodes

PR11698: Track aliases even with ppc function descriptors

We can include function descriptors in the symbol table, and just make
sure that we don't try to probe them with query_module_symtab.  This
lets update_symtab do its thing resolving aliased names.

* dwflpp.h (func_info): Add a descriptor flag.
* tapsets.cxx (symbol_table::add_symbol): Set the descriptor flag.
  (symbol_table::read_symbols): Not descriptors.
  (symbol_table::get_from_elf): Functions from powerpc rejected sections
  are function descriptors.
  (dwarf_query::query_module_symtab): Don't probe descriptors.

PR10327: Fix and test alias matching

The optimization in commit 4df79aa broke alias matching for functions
without wildcards, but we didn't have any testing. :(

* dwflpp.cxx (dwflpp::iterate_single_function): Call update_symtab after
  building the module function cache, to enable alias matching.
* testsuite/systemtap.base/func_alias.{exp,c}: New test.

Fixed BZ598615 to avoid a crash when setting registers on ia64.

* runtime/regs-ia64.c (ia64_store_register): Added a missing "return"
  statement that caused a crash when changing registers on ia64.

Make sure that the tapset documentation title descriptions are on a single line

The docproc program that extracts information information from the tapset
comments needs the title line to be a single line. Otherwise the second
line get counted as a description line and conflicts with the later
"Description:" section.

Addressing bug #11684 and eliminating duplication from /doc/langref.tex and /tapset/*.stp

PR11641: fix backtrace from uretprobe

This will partly revert commit 23258335. When not called
from a uretprobe hander, to pass -1 not 0 ensures all the
cases can be covered.

Added testcase for queue_stats.stp.

* testsuite/buildok/queue_stats-embedded.stp: New testcase.

Got tty.stp working everywhere.

* tapset/tty.stp: Large update to get working on RHEL4, RHEL5,
  RHEL6(beta), f12, f13, and rawhide.
* testsuite/buildok/tty-detailed.stp: Renamed from
  testsuite/buildok/tty.stp and added more functions.
* testsuite/buildok/tty-resize.stp: Separate probe testcase since it can
  fail because of PR 1155 (can't find inline function arguments).
* testsuite/buildok/tty-embedded.stp: New testcase.
* testsuite/systemtap.pass1-4/buildok.exp: Kfail the tty-resize.stp.

sdt.h: use /* c */ comments only

... else -ansi / -pedantic tests fail

authors bump

move #if 0 around STAPCONF_PROBE_KERNEL around loc2c

* buildrun.cxx: Always run autotest-kernel-probe.c.
* runtime/loc2c-runtime.h: Disable use of these probe functions here.

use probe_kernel_read for build-id verification if available

Post 1.1 I've been seeing the following on i686 (x86_64 is ok):

ERROR: Build-id mismatch: "kernel" vs. "vmlinux-2.6.34-8-default.debug" byte 0 (0x00 vs 0x00) rc -14 -14
Pass 5: run failed.  Try again with another '--vp 00001' option.

as per irc discussion, following test diff fixed the problem on i686, I've not
tested on any other archs.

Define the sdt probe type in one place.

tapsets.cxx (sdt_query): Move stap_sdt_probe_type from here...
sdt.h: ...to here.

Consolidate task_finder/vma tracker initialization.

After the task_finder is started, no new targets can reliably be added.
So make sure that all registration/enabling of the task_finder and/or
the vma tracker is done in one place (task_finder_derived_probe_group).

* task_finder.h: Defined enable_vma_tracker and vma_tracker_enabled.
* task_finder.cxx: Implement enable_vma_tracker and vma_tracker_enabled.
  (task_finder_derived_probe_group::emit_module_init): Also handle
  vma_tracker.
* elaborate.cxx (visit_embeddedcode): Use enable_vma_tracker and
  vma_tracker_enabled.
* session.h (systemtap_session): Remove need_vma_tracker.
* session.cxx (initialize): Remove need_vma_tracker initialization.
  (parse_cmdline): Enable vma tracking early when -d user mode module found.
* translate.cxx (c_unparser::emit_module_init): Don't try to start
  task finder and vma tracker here.
  (c_unparser::emit_module_exit): Don't try to stop task finder here.
  (emit_symbol_data_done): Don't emit static _stp_need_vma_tracker.
* runtime/sym.h: Remove static _stp_need_vma_tracker.
* runtime/sym.c (_stp_sym_init): Return error when registration failed.
* runtime/task_finder.c (stap_register_task_finder_target): Add sanity
  checks to detect double registration or registration after task finder
  was already started.
  (stap_start_task_finder): Sanity check that task finder wasn't already
  started.

Don't double print error message on bad stap command line option.

Bug introduced in commit db1354. getopt_long will already have printed
an appropriate error message in case of an invalid/unrecognized option
or an option missing an required argument. So don't print another error
message in that case.

* session.cxx (parse_cmdline): Silently handle '?' case.

PR11690: Remove a dtrace debug-print, and fix probe_perf

Stan's commit 278c975 fixes the core issue; I'm just polishing...

* dtrace.in: Comment out the debug-print before calling gcc.
* scripts/probe_perf/bench.sh: Pass -DSTAP_SDT_V2 to dtrace.

Fix tests for -DSTAP_SDT_V2.

dtrace -G -s foo.d builds foo.o so it must pass -DSTAP_SDT_V2 to gcc.  Add a -D
flag (sun dtrace also has a -D flag) and pass that to the gcc invocation.

* dtrace.in (semaphore_def_append): Make -DSTAP_SDT_V1 the default.
(generate): Likewise.
(main):  Add -D and pass to cpp and gcc.
* sdt_misc.exp: Run dtrace every iteration and pass it -DSTAP_SDT_V2.

Correct the hex-spelling of UPB2

Get rid of probe_perf tmps, and use a single awk

Fix 3-way merge result.

* sdt.h: Remove extra hunk from merge.

Merge branch 'master' of git://sources.redhat.com/git/systemtap

* 'master' of git://sources.redhat.com/git/systemtap:
  Tighten static user probe point operand contraints from "g" to "ro"

fix !CONFIG_FRAME_POINTER, !CONFIG_UTRACE case

In a kernel without FRAME_POINTERS (using the DWARF UNWINDER) and without
UTRACE I see the following in latest git master:
[...]
In file included from /tmp/stap/share/systemtap/runtime/stack.c:56:0,
                 from /tmp/stapPHqTen/stap_12068.c:52:
/tmp/stap/share/systemtap/runtime/stack-i386.c: In function "__stp_stack_print":
/tmp/stap/share/systemtap/runtime/stack-i386.c:69:25: error: implicit declaration of function "uprobe_get_pc".

AFAICT the autoconf test should result in STAPCONF_UPROBE_GET_PC=0 for the
case where CONFIG_UTRACE is unset since UPROBES requires UTRACE.

Signed-off-by: Tony Jones <tonyj@suse.de>

Tighten static user probe point operand contraints from "g" to "ro"

Give non-x86 a chance at uprobes SDT

* tapsets.cxx (sdt_uprobe_var_expanding_visitor): No need to assert that
  we know dwarf_regs unless we actually encountered STAP_SDT_V2 with
  arg_count > 0.  V1 should continue to work on other architectures.

Let probe_bench work on archs besides x86_64

The cycles code is too x86_64-centric, but for now at least it's been
neutered to let other archs report basic timing numbers.

* scripts/probe_bench/bench.stp: Use get_cycles() instead of rdtsc().
* scripts/probe_bench/bench.c (rdtsc): Return 0 unless on x86_64.  This
  needs more work and inline assembly to fit more architectures.

Ignore probe_bench's generated files

Short out errors in probe_bench

* scripts/probe_bench/bench.sh (setup_test, stap_test): Report when an
  error occurred and return.

Make sure the probe_bench can find sdt.h

* scripts/probe_bench/bench.sh (setup_test): Give -I$STAP/include to gcc.

PR11680: fix sdt.h STAP_SDT_V2 -> V1 auto-downgrading logic

Previously, the STAP_SDT_V2 -> V1 downgrade logic was half way down in the
header file, by which time some STAP_SDT_V2 definitions had already taken
effect.

* includes/sys/sdt.h: Make work on PPC et al.

Set vma_cache_p->dentry before use.

Bug (on old kernels) introduced by typo in commit ebf779.

* runtime/task_finder.c (__stp_call_mmap_callbacks_for_task): Make sure to
  set vma_cache_p->dentry before usage.

Add more error messages when something goes wrong during task_finder detach.

* runtime/task_finder.c (stap_utrace_detach): Report when utrace_barrier
  fails.
  (stap_utrace_detach_ops): Report how stap_utrace_detach failed.

Fix task_finder.c f_dentry -> dentry typo.

* runtime/task_finder.c (__stp_call_mmap_callbacks_for_task): Fix typo.

Loop on utrace_barrier if utrace_control UTRACE_DETACH returns -EINPROGRESS.

When utrace_control(tsk, eng, UTRACE_DETACH) returns -EINPROGRESS that
means there are still handlers running. So loop on utrace_barrier(tsk, eng)
in that case, till it no longer returns -ERESTARTSYS. That makes sure that
no engine handler will be called afterwards, so we can safely unload the
stap module. Not doing this might have caused PR11672
(utrace_report_syscall_exit crash), although we don't yet have a simple
reproducer for that issue.

* runtime/itrace.c (remove_usr_itrace_info): Loop on utrace_barrier if
  utrace_control returned -EINPROGRESS.
* runtime/task_finder.c (stap_utrace_detach): Likewise.
  (stap_utrace_detach_ops): Likewise. And warn if stap_utrace_detach
  didn't return successfully.
  (__stp_utrace_attach): Loop on -ERESTARTSYS after utrace_barrier.
  (__stp_utrace_task_finder_target_quiesce): Likewise.

Fix typo in stap.1.in

AUTHORS bump

uprobes_ppc: define MSR_MASK and clean up error cases

- The emulate_step code was copied from the kernel, but needs to have
  the #define MSR_MASK too.
- A case in emulate_step could fall through with an illegal instruction,
  just break and return 0 instead.
- A few privileged instructions can't be emulated and so return -1, but
  we don't want uprobe_emulate_insn to see that as non-zero -> TRUE.

At a minimum, this restores whatever SSOL-faulty behavior would have
existed before emulate_step was added.  We should also add checks in
arch_validate_probed_insn to avoid such instructions in the first place.

Signed-off-by: Josh Stone <jistone@redhat.com>
Acked-by: Jim Keniston <jkenisto@us.ibm.com>

added the XDG_DATA_DIRS to session.cxx and the corresponding man changes as specified in bug #11455

PR11678 Pick up ld.so (and other absolute shlibs) through stap --ldd.

* translate.cxx (add_unwindsym_ldd): Make scan slightly stricter, require
  shlib to start as absolute path and address to be hex. Also allow direct
  absolute shlib paths without soname.

Tweak schedtime.stp example for 2.6.35

$rq is removed from tracepoint in 2.6.35. Guard access using
@defined predicate.

Guard embeddedcode_info_pass from failures

We should not run this pass if there were previous errors, as
functioncall_traversing_visitor require referents on all calls.  Fixes
a segfault in semko/five.stp after an arg-count mismatch, as well as the
new semko/nofunc.stp with a completely missing function.

* elaborate.cxx (semantic_pass): Only do embeddedcode_info_pass if !rc.
* testsuite/semko/nofunc.stp: New, as I couldn't find such a test.

Improve identification of arity-mismatch errors

Instead of pointing at the declaration site, we now finger the use which
is trying to set a bad arity, and also mention who originally set the
arity if possible.  For example, semok/six.stp now says:

semantic error: inconsistent arity (1 vs. 2): identifier 'bar' at ../testsuite/semko/six.stp:6:3
        source:   bar[1, 2] = 3;  # inconsistent array dimensions
                  ^
semantic error: arity 1 first inferred here: identifier 'bar' at :5:3
        source:   bar[1] = 2;
                  ^

* staptree.cxx (vardecl::set_arity): Require a token from the caller,
  and report the original caller's token as well.
* elaborate.cxx (symresolution_info::visit_symbol): Pass the token.
  (symresolution_info::find_var): Ditto.
* parse.cxx (parser::parse_global): Ditto.
* tapset-mark.cxx (mark_derived_probe::mark_derived_probe): Ditto.
* tapset-utrace.cxx
  (utrace_var_expanding_visitor::visit_target_symbol_cached): Ditto.
* tapsets.cxx (tracepoint_derived_probe::tracepoint_derived_probe):
  Ditto.

Never permit homonymous vars with different arity

It was previously possible to generate a global array and local scalar
having the same name, because find_var was glossing over globals that
didn't have compatible_arity.

Fixes semko/nine, and improves semko/six to an arity-mismatch error
rather than a local-array error.

* elaborate.cxx (symresolution_info::find_var): Use set_arity without
  checking compatibile_arity, so that compatibility gets asserted.
  (symresolution_info::visit_foreach_loop): Add "missing global" hint.
  (symresolution_info::visit_arrayindex): Don't create local arrays.

Fork semko/two.stp into distinct errors

Now two.stp checks the failure of integer+string, and twotwo.stp checks
that stats can't be a function arg.  Both need -u to defeat optimization.

Use real probepoints in semko/two-ten

These tests were using non-existent probepoints (foo,p1,p2), leading to
an error that masks the real failure we're trying to test.

(A few were actually broken -- commits to follow...)

Don't abuse uprobes for tracking symbols, use -d ./usymbols --ldd.

* testsuite/systemtap.context/usymbols.exp: Remove fake process.function
  probe from script, add -d ./usymbols to stap invocation.

PR11665 Add umodname(addr) tapset function.

* runtime/sym.c (_stp_tf_mmap_cb): Always register module name in vma_map.
  (_stp_module_name): New function.
* tapset/ucontext.stp: New tapset.
* testsuite/buildok/ucontext.stp: New pass 4 test.
* testsuite/systemtap.context/usymbols.exp: Extend pass 5 test to include
  umodname(handler).

Increase TASK_FINDER_VMA_ENTRY_ITEMS default.

Somewhat arbitrary default, this is often way too much for tracking
single process, but often too little when tracking whole system.
FIXME Would be nice to make this dynamic. PR11671

* runtime/task_finder_vma.c: Clean up some (bad) comments.
  #define TASK_FINDER_VMA_ENTRY_ITEMS 1536.

Activate task finder vma tracker through tapset embedded-c /* pragma:vma */

The task finder vma tracker, _stp_sym_init(), was supposed to be activated
through defining STP_NEED_VMA_TRACKER in an included tapset. This didn't
actually work, since the define came too late. The _stp_sym_init() call
was however often triggered by the fact that any of these vma tracking
using tapset functions were used through task_finder_derived_probe_groups.
This patch makes it so that the vma tracker always gets activated when
they are used with any kind of probe.

* session.h (systemtap_session): New need_vma_tracker boolean.
* session.cxx (initialize): Initialize need_vma_tracker to false.
* elaborate.cxx (embeddedcode_info): New functioncall_traversing_visitor.
  (embeddedcode_info_pass): New semantic pass.
  (semantic_pass): Call embeddedcode_info_pass.
* translate.cxx (c_unparser::emit_module_init): Check whether
  session->need_vma_tracker is set to decide emitting _stp_sym_init.
  (c_unparser::emit_module_exit): Likewise for teardown.
* tapset/ucontext-symbols.stp: Remove STP_NEED_VMA_TRACKER define. Tag all
  embedded-c functions with /* pragma:vma */.
* tapset/ucontext-unwind.stp: Likewise.
* runtime/task_finder.c (__stp_task_finder_started): New static int.
  (stap_start_task_finder): Check and set __stp_task_finder_started.
  (stap_stop_task_finder): Likewise.

Remove unused vm_pgoff tracking from task_finder_vma.

* runtime/task_finder_vma.c (__stp_tf_vma_entry): Remove vm_pgoff field.
  (stap_add_vma_map_info): Remove vm_pgoff argument.
  (stap_remove_vma_map_info): Likewise.
  (stap_find_vma_map_info_user): Likewise.
* runtime/sym.c (_stp_tf_mmap_cb): Adjust vma_map callers.
  (_stp_module_relocate): Likewise.
  (_stp_mod_sec_lookup): Likewise.
* runtime/unwind.c (adjustStartLoc): Likewise.

Don't try to be clever with dentry lifetime.

Make caller of task_finder_vma responsible for dentry lifetime.
Must call stap_remove_vma_map_info when dentry goes away. Which
is what always happens because that is when the vma is unmapped.

* runtime/task_finder.c (__stp_call_mmap_callbacks_with_addr): Don't dget
  and don't dput.
  (__stp_call_mmap_callbacks_for_task): Only dget/dput for non-d_path case.
* runtime/task_finder_vma.c (stap_add_vma_map_info): Don't dget dentry.
  (stap_remove_vma_map_info): Don't dput dentry.
  (stap_drop_vma_maps): Likewise.

Remove unused arguments from stap_remove_vma_map_info.

* runtime/task_finder_vma.c (stap_remove_vma_map_info): Drop vm_end and
  vm_pgoff arguments.
* runtime/sym.c (_stp_tf_munmap_cb): Don't pass addr + length and 0.

Remove bogus (but wrongly used!) addr field from __stp_tf_vma_entry.

* runtime/task_finder_vma.c (__stp_tf_vma_entry): Remove addr field.
  (__stp_tf_get_vma_map_entry_internal): Compare vm_start, not addr.
  (stap_find_vma_map_info): Rename vm_addr argument to addr.

Track dentry for file backed vma in task_finder.

* runtime/task_finder.c (stap_task_finder_mmap_callback): Provide dentry.
  (__stp_call_mmap_callbacks): Take dentry.
  (__stp_call_mmap_callbacks_with_addr): Collect dentry.
  (__stp_call_mmap_callbacks_for_task): Likewise.
* runtime/task_finder_vma.c (__stp_tf_vma_entry): Add dentry.
  (stap_add_vma_map_info): Take and store dentry.
  (stap_remove_vma_map_info): Remove dentry.
  (stap_find_vma_map_info): Provide dentry.
  (stap_find_vma_map_info_user): Likewise.
  (stap_drop_vma_maps): Remove dentries.
* runtime/sym.c (_stp_tf_mmap_cb): Accept dentry and store it in vma_map.
  (_stp_module_relocate): Call stap_find_vma_map_info_user with NULL dentry.
  (_stp_mod_sec_lookup): Call stap_find_vma_map_info with NULL dentry.
* runtime/unwind.c (adjustStartLoc): Provide NULL dentry.
* runtime/kprobes-common.c (stap_kprobe_mmap_found): Accept dentry.
* runtime/kprobes-common.h (stap_kprobe_mmap_found): Likewise.
* runtime/uprobes-common.c (stap_uprobe_mmap_found): Likewise.
* runtime/uprobes-common.h (stap_uprobe_mmap_found): Likewise.

Rename __stp_tf_vma_initialize to stap_initialize_vma_map.

Makes it clear this isn't one of the internal task_finder_vma functions.

* runtime/sym.c (_stp_sym_init): Call stap_initialize_vma_map.
* runtime/task_finder_vma.c (stap_initialize_vma_map): Renamed from
  __stp_tf_vma_initialize.

Removed unused task_finder_vma code.

* runtime/task_finder_vma.c (__stp_tf_vma_table, __stp_tf_vma_hash,
  __stp_tf_get_vma_entry, __stp_tf_add_vma, __stp_tf_remove_vma_entry):
  Unused, so removed.

Tweak the ipmib update for 2.6.35

* tapset/ipmib.stp (ipmib.InNoRoutes, ipmib.InAddrErrors):
  Now using probe sufficiency instead of version checks.

PR11095: support process(). **provider()** .mark()

For STAP_SDT_V2 only for now.

* NEWS, stapprobes.3stap.in: Mention it.
* Makefile.am (stap_CPPFLAGS): Use -DSTAP_SDT_V2.
* dtrace.in: Preserve provider name, though squash it for STAP_SDT_V1.
* includes/sys/sdt.h: Likewise.
* tapsets.cxx (register_patterns): Accept .provider() probe point piece.
  (sdt_*_var_expanding_visotor): Store it.
  (sdt_query::*): Extend for provider matching, new magic codes.
* testsuite/systemtap.base/sdt_misc.exp: Use more formal interfaces
  to semaphores.

Add configure check for xmlto pdf with chapters titles starting with L...

There is a strange bug in older versions of xmlto when generating pdf.
https://bugzilla.redhat.com/show_bug.cgi?id=526273
So make sure to have a chapter title starting with L plus an refentry.
This will make sure the xmlto pdf support test fails on buggy versions.

configure.ac: Extend conftest.xml.
configure: Regenerated.

Make rpc.stap and tcpmib.stp compatible with 2.6.35

* tapset/rpc.stp: access structure member for om_{queue,rtt,execute}.
* tapset/tcpmib.stp: change _skb_dst to _skb_refdst.

Update ipmib.stp for 2.6.35

* tapset/ipmib.stp: Select ip_route_input_common for 2.6.35.

Replace walk_stack field version guard with autoconf test.

The test for whether or not to assign print_context_stack to the
walk_stack stacktrace_ops field depended on the kernel version.
Replace with a proper runtime/autoconf test to make sure the field
always gets assigned when available.

* buildrun.cxx (compile_pass): Add output for STAP_CONF_WALK_STACK.
* runtime/autoconf-walk-stack.c: New test.
* runtime/stack.c (print_stack_ops): Assign walk_stack field
  print_context_stack depending on STAP_CONF_WALK_STACK.

AUTHORS bump

Add environment variable fetching for current process

* tapset/context.stp: Add env_var.
* testsuite/buildok/context_test.stp: Add test for new function.

PR3672 cont'd: Change pretty-print "depth" to mean substructures

We don't follow pointers anymore within a structure.  Now a single '$'
suffix means we print a shallow view of the structure, with {...} and
[...] in place of nested arrays/structs.  A double suffix '$$' means we
print everything contained in the struct.  Longer suffixes no longer
have meaning, and for now act the same as '$$'.

* tapsets.cxx (dwarf_pretty_print): Lose the "print_depth" field, and
  instead just check if this is a "full" print job.  Pass a bool
  indicating whether we're currently traversing the top-level, so we
  know which structs to expand.  Also wrap it all in try/catch so bad
  pointers don't kill us.

PR11664: Let return/next break out of try/catch

Try-catch works by faking a local "out" label and running the catch
block.  In the case of return and next, we have a "goto out" without any
error, so it should be allowed to progress to the real out label.

* translate.cxx (c_unparser::visit_try_block): Reorganize the nested
  labels to allow normal-fallthrough, normal-out, and error-out.
* testsuite/systemtap.base/overcatcher.stp: New test.

Correct uninitialization warning for gcc 4.1

* translate.cxx: provide default value for variable data.

PR11599: Support the new tracepoints API in 2.6.35

* buildrun.cxx (make_tracequery): Support DECLARE_TRACE_NOARGS.
* tapsets.cxx (tracepoint_derived_probe_group::emit_module_decls): Add
  an unused void* parameter for the tracepoint callbacks on 2.6.35+.

Remove the STAP_LABEL machinery, which was replaced by gas `Nb' 'Nf' labels.

* sdt.h (STAP_CONCAT, STAP_LABEL_PREFIX,STAP_LABEL): Remove
(STAP_PROBEN_): Merge with STAP_PROBEN.

Test with and without semaphores using STAP_SDT_IMPLICIT_ENABLED.

* probe_perf/bench.sh: Use -DSTAP_SDT_IMPLICIT_ENABLED.

rhbz #596083 Account for dot and dotless kernel variant in stap-prep.

uname -r can produce different kinds of output:
2.6.32-30.el6.x86_64 (no variant, but including ".arch")
2.6.18-194.3.1.el5debug ("variant", without dot, no arch)
2.6.33.4-95.fc13.i686.PAE (".arch.variant", two dots)
We didn't handle the dot before the variant used in newer distros.

* stap-prep: Remove either ".variant" or "variant" from the end of release
  uname string.

PR11649: workaround for incomplete include/linux/event/kvm_*.h

* tapsets.cxx (tracepoint_extra_headers): Pass along systemtap_session.
  Add a kvm header if CONFIG_KVM.

Guard create_debug_frame_hdr body with _ELFUTILS_PREREQ(0,142).

loc2c: correct typo for i386 target register access on x86-64 kernel

PR3672 cont'd: Resolve declared structs

* tapsets.cxx (dwarf_pretty_print::recurse_struct): Use dwflpp to
  resolve DW_AT_declaration types.

Merge branch 'unwind_hdr'

PR10665 Make unwinder recognized synthesized debug_frame_hdr.

* runtime/unwind.c (_stp_search_unwind_hdr): Take is_ehframe argument.
  Recognize and use debug_hdr for binary searching the FDE.
  (unwind_frame): Always try binary hdr search.

Accept CIE versions 3 and 4.

* runtime/unwind.c (fde_pointer_type): Accept version 3 and 4.
  (unwind_frame): Likewise.

PR10665 Create a synthetic binary search table for a debug_frame section.

* runtime/sym.h (_stp_module): Add fields debug_hdr and debug_hdr_len.
* translate.cxx (create_debug_frame_hdr): New function.
  (get_unwind_data): Call new function when there is a debug_frame.
  (dump_unwindsyms): Output new debug_frame_hdr.

Add kprobes-common.{h,c}, which is analogous to uprobes-common.{h,c}

Add kprobes-common.{h,c}, which is analogous to uprobes-common.{h,c}

uprobes-common.h: New file.
uprobes-common.c: New file.

Implicit *_ENABLED() is no longer the default for sdt.h probes.
Cast V2 probe args.

sdt.h (STAP_SDT_IMPLICIT_ENABLED): New.
(STAP_TYPE): New.
(STAP_CAST): New.
(STAP_PROBE*_): Use STAP_TYPE and STAP_CASE.
sdt_misc.exp:  Use -DSTAP_SDT_IMPLICIT_ENABLED

Add () to execname in stap man page example.

Here is a patch implements a way for uprobes to skip single step
when a probe in on a nop instruction.

Currently its implemented for nop on x86/x86_64 and s390.
For powerpc, its implemented on all instructions that can be emulated.
(For this emulation code is borrowed from arch/powerpc/lib/sstep.c)

I havent tested it on s390. Updated with comments from Jim.

Signed-off-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Signed-off-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
Reported-by: Josh Stone <jistone@redhat.com>

Only build docproc when refdoc requested.

* doc/SystemTap_Tapset_Reference/Makefile.am: Move docproc under BUILDREFDOC
  conditional.
* doc/SystemTap_Tapset_Reference/Makefile.in: Regenerated.

PR3672: Add pretty-printing for compound types

This adds a new syntax for pretty-printing variables as strings:

  $var$         $var$$         $var->$            $var->$$
  $@cast(...)$  $@cast(...)$$  $@cast(...)->$     $@cast(...)->$$
  $var->foo->$  $var[1]->$     $@cast(...)->foo$  $@cast(...)[2]$

This is still a work in progress, but I deemed it now useful enough to
share.  See PR3672 for discussion of work remaining.

* parse.cxx (parser::parse_target_symbol_components): Parse the trailing
  $s as a new comp_pretty_print entry in the target_symbol.
* staptree.cxx (target_symbol::assert_no_components): Add a flag to
  assert whether pretty-printing is supported.
  (target_symbol::component::print): Support comp_pretty_print.
* dwflpp.cxx (dwflpp::translate_components): Skip the actual translation
  if the obstack "pool" is NULL.
  (dwflpp::type_die_for_local): Dig through a local and its components
  to get the type DIE that the pretty printing can start from.
  (dwflpp::type_die_for_return): Ditto for return values.
  (dwflpp::type_die_for_pointer): Ditto for pointers.
* tapsets.cxx (dwarf_pretty_print): New struct for recursing over a type
  DIE and producing a string representation.  (The real meat lies here!)
  (dwarf_var_expanding_visitor::visit_target_symbol): Determine when to
  use dwarf_pretty_print instead of the normal path.
  (dwarf_var_expanding_visitor::visit_target_symbol_context): Ditto.
  (tracepoint_var_expanding_visitor::visit_target_symbol_arg): Ditto.
  (tracepoint_var_expanding_visitor::visit_target_symbol_context): Ditto.
  (dwarf_cast_query, dwarf_cast_expanding_visitor): Move the function
  generation from the visitor to the query, and also support using
  dwarf_pretty_print as needed.

tracequery build fix

Similar to commit 392e08b7 -- we need to #undef TRACE_INCLUDE_FILE
before each header in the tracepoint query module, so they can be
queried all at once.

The side effect was that the all-inclusive tracequery build would fail,
so we'd fall back to building a separate tracequery for each header.
It's much faster when we can build them together.

* buildrun.cxx (make_tracequery): #undef TRACE_INCLUDE_FILE before each
  tracepoint header to prevent macro redefinition.

rhbz #596083 Make stap-prep resolve kernel-debuginfo-common package name.

The kernel-debuginfo-common package name is slightly different on different
distros. But the correct name is always required by the kernel-debuginfo
package. So don't add it to the CANDIDATES list, but let the yumdowloader
--resolve it. Tested on f13, rhel5 and rhel6 beta.

* stap-prep: Remove kernel-debuginfo-common from CANDIDATES, add --resolve
  to yumdownloader.

PR10245: allow "-" in sdt.h marker names

* tapsets.cxx (sdt_query ctor): Map - to __ in incoming marker names.
* NEWS: Note it.
* testsuite/systemtap.base/sdt_types.c, sdt_misc.exp: Test it.

make semok/thirtysix.stp test use stap -p2 instead of -p4