gcc 4.6 -Wextra -Wall fixes

Some unused parameters removed, some extra
initialization here and there.

Add ia64 interpreter to whitelist

* dwflpp.cxx (iterate_over_libraries): Add /lib/ld-linux-ia64.so.2 to whitelist.

Fixed rpm_finder compilation when '!HAVE_LIBRPMIO && HAVE_NSS'.

* rpm_finder.cxx: Added needed include of nsscommon.h when we don't have
  librpmio but have nss.

Add ppc/s390 interpreter to whitelist.

* dwflpp.cxx (iterate_over_libraries): Add /lib64/ld64.so.1 to whitelist.

disable install targets from pcp/ subdir

These don't respect autoconf --prefix, and are not really
desired as part of the installed stap package anyway.

bump AUTHORS

Fix build problems with compile-server and client when avahi is not available.

NEWS entry for new compile server implementation.

i18n: po rebuild after CVE fixes

CVE-2011-1781, CVE-2011-1769: correct DW_OP_{mod,div} division-by-zero bug

    Probing a process with corrupted DWARF information, it has been
    possible to create a kernel-side divison-by-zero.  This fixes.

    Handle DW_OP_div/mod divide by zero. DW_OP_mod should work unsigned.

    * loc2c.c (translate): Use helper functions div_op and mod_op for
      DW_OP_div and DW_OP_mod operands. Set used_deref = true.
    * translate.cxx (translate_runtime): Emit STAP_MSG_LOC2C_03 define.
    * runtime/loc2c-runtime.h: Define dwarf_div_op and dwarf_mod_op macros.
    * runtime/unwind.c (compute_expr): Check for zero before executing
      DW_OP_mod or DW_OP_div.

Correct some rlimit settings for the translator when called by the server.

Some settings should have been specified in kilobytes.

nss-tools is no longer required by our RPMs.

Platform specific build issues:

Missing #includes.
Unused variables.

Set resource limits for the translator when called from the compile server.

Gettext a few lines, update /po and Makefiles

Makefile.am - dont include config.h runtime/staprun/config.h or git_version.h
Makefile.in - likewise
nsscommon.cxx - gettexted a few strings
po/* - regenerated files

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

Merge branch 'master' of git://oss.sgi.com/nathans/systemtap

* 'master' of git://oss.sgi.com/nathans/systemtap:
  Resolve a couple of issues from missing log file handling code.
  Minor cleanups after writing QA tests.
  Add code to deal with log file rotation.
  Unify separate tables for tracking log files, simpler code.
  Several additional metrics for log file PCP agent.
  Update the domain number comment now that one is reserved.
  Uncomment the seek-to-end-of-log-file code in pmdalogger.
  Remove further shared library remnants in pmdalogger build.

dtrace python i18n: make work with autoconf wackyness

autoconf likes to expand some @vars@ in terms of shell-script-like
constructs like @LOCALEDIR@ = "${datarootdir}/locale" and
@datarootdir@ = "${prefix}/share".  Since python doesn't interpolate
strings the same way as /bin/sh, this no workie. So we hard-code the
interpolation with a sequence of string.replace calls.

Confirmed working with LANG=fr_FR strace python ./dtrace |& grep /fr

* configure.ac: AC_SUBST a few more values.
* dtrace.in: Specially process ENABLE_NLS and similar values.

Add NULL check to second PR_Close(local_file_fd)

Add NULL check for PR_Close(local_file_fd)

Initialize local_file_fd to NULL

nsscommon.cxx - set to NULL as to ensure no uninitialized use

elf_getphdrnum is not in all versions of elfutils

* dwflpp.cxx (iterate_over_libraries): Use result of gelf_getphdr to
exit loop instead of result of elf_getphdrnum.

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

Read-only iteration should use const

* util.cxx (spawned_pids_t::killall): Use const_iterator.

remote: Add tests for manually-specified hosts

To run a basic test on hosts foo and bar, use:
  make installcheck RUNTESTFLAGS=remote.exp TESTREMOTES=foo,bar

* testsuite/systemtap.base/remote.exp: New test of --remote hosts.
* testsuite/systemtap.base/remote.stp: New.
* testsuite/Makefile.am: Add TESTREMOTES control of remote.exp.
* testsuite/Makefile.in: Regenerate.

Create a signal-safe type for tracking spawned pids

* util.cxx (spawned_pids_t): New type which wraps a set<pid_t>, masking
  signals on each access to ensure consistency in and out of the signal
  handler.  The spawned_pids global is the only instance.
  (stap_waitpid): Use !contains(pid) rather than count(pid)==0.
  (kill_stap_spawn): Use spawned_pids_t::killall().

Mask signals while removing the tmpdir

* main.cxx (remove_temp_dir): Rather than ignoring signals while we rm,
  just use stap_sigmasker so we can get missed signals when we're done.

Consolidate signal-masking into a utility class

* util.h (stap_sigmasker): New, masks our usual signals for the life of
  the stap_sigmasker object.
* remote.cxx (direct_stapsh::direct_stapsh): Use stap_sigmasker while
  spawning the stapsh child process.
  (ssh_remote::connect): Ditto for the ssh process.
  (remote::run): Use stap_sigmasker around the polling loop.

remote: Disambiguate the private target names

It's conceivable, however unlikely, that a user may have an actual host
named "direct" or "stapsh", which would conflict with our internal
methods if used as a --remote.  Such a user could say "ssh://direct" to
be explicit, but we can also hide ours a little better.  Those internal
names are now tested as proper URI schemes, e.g. "direct:...", so they
should never conflict with a user's legitimate target.

* remote.cxx (remote::create): Test for "direct" and "stapsh" only as
  the scheme of a decoded URI.
* main.cxx (main): Use "direct:" for non-remote use.
* testsuite/systemtap.base/stapsh.exp: Use "stapsh:" for testing.

Make sure remote children are waited for

* remote.cxx (ssh_remote::~ssh_remote): Call finish on destruction.
  (direct_stapsh::~direct_stapsh): Ditto.
  (direct::~direct): Ditto.

PR12749: Replace popen calls with stap_spawn_piped

The new form has the advantages that child processes are managed by
signals to stap, and that arguments are provided in a vector so they
don't need to be escaped.

* dwflpp.cxx (dwflpp::iterate_over_libraries): Convert popen call to
  stap_spawn_piped, followed by fdopen so the same FILE* operations are
  still supported.  Finish with fclose+stap_waitpid instead of pclose.
* tapsets.cxx (symbol_table::read_from_elf_file): Ditto.

Remove the unused git_revision()

The use of this function had been commented out for some time now, and
it contained an unescaped call to popen.  Rather than trying to fix dead
code, just remove it altogether.

Explicitly check the interpreter paths.

dwflpp.cxx (dwflpp::iterate_over_libraries):  Do explicit pathname comparison.

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

Conflicts:
nsscommon.c
runtime/staprun/Makefile.in

pmwebapi: prototype pcp web api daemon

* src/pmwebapi/main.c: New file.

pcp logger: disable install target, as this doesn't mesh with stap configury

Systemtap Compile Server Integration (rewrite):

- Rewrite stap-serverd in C++
- Rewrite related tools (stap-gen-cert, stap-authorize-cert, stap-sign-module)
  in C++. Integrate functionality into stap-serverd.
- Remove stap-server-connect (integrated into stap-serverd).
- Move all common NSS related code into nsscommon.cxx (renamed from nsscommon.c).
- Rename modsign.cxx to stap-sign-module.cxx.
- Update test suite with new expected messages.
- Update man pages.
- Remove obsolete tools (scripts).
- Remove test for certutil from configuation.

Bump postgres version used for systemtap.apps to 9.0.4

* stap-postgres.stp (postgresrelease):  Bump it to 9.0.4

Resolve a couple of issues from missing log file handling code.

If we do not stat the file on initial opening as well, the first
ino/dev comparison fails and we end up always reopening the file
at least once.

Whenever we have a successful stat call on logfile, keep latest
output in case we ever want to use last-modified times, etc.

Minor cleanups after writing QA tests.

Most interesting change is handling accidental use of a
directory name as a log file name, doesn't spam the log.

uprobes: impedance match insn tables with test_bit()

The kernel's test_bit expects its bitmap to be const volatile, but we
had ours as simply const.  On Fedora 15 with gcc 4.6, compiling uprobes
gave a few warnings like this:

    arch/x86/include/asm/bitops.h:319:2: warning: use of memory input
    without lvalue in asm operand 1 is deprecated [enabled by default]

That line is the asm statement in variable_test_bit().

The symptom noticed was that handle_riprel_insn was reading need_modrm:0
for opcode 0x89, when our table says it should be 1.  Who knows what
other havok ensued...

When our instruction tables are set const volatile to match test_bit(),
the warning goes away, and need_modrm is now computed correctly.

uprobes: remove #undef DEBUG_UPROBES_RIP, to make it controllable

Use iterate_over_libraries for --ldd instead of invoking ldd

* dwflpp.cxx (iterate_over_modules): Make data an opaque type.
(iterate_over_libraries): Likewise.
* dwflpp.h: Likewise.
* translate.cxx (add_unwindsym_iol_callback): New.
(query_module): New.
(add_unwindsym_ldd): Use them to iterate_over_libraries instead of ldd.
* library.exp:  Add --ldd test.

dtrace: Remove unused open_file function

dtrace: Push main logic into an actual main()

* dtrace.in (main): New, invoked using Python's __name__ idiom.
  (_provider.__typedef_append): Take add_typedefs as a parameter rather
  than pulling from the formerly global scope.
  (_provider.generate): Adjust __typedef_append calls.

dtrace: Fix tabbed indentation with 4 spaces

dtrace: Use dict.get() for conditional environment values

Add code to deal with log file rotation.

Depending on how the log file is rotated, it will often end up backed
by a new inode (new log file) as the previous one is renamed (usually
with yesterdays log timestamp).  When this happens, we'll stop getting
events ... unfortunate.

So, make use of the stat data queried at start of fetch now to ensure
we detect this scenario, close the old file and switch to the new one
seamlessly.  In the process, we need to make sure we handle the case
where the file doesn't exist.

Unify separate tables for tracking log files, simpler code.

Simplify the code a little, unifying the two separate tables
for tracking open log files.

Also added support for comments (#-prefixed) in the config file.

Dtrace script should honor $CPP, $CC, $CFLAGS

* dtrace (main):  Use $CPP, $CC, $CFLAGS if available
* dtrace.exp:  Add CFLAGS= CC= test

regen examples index

Better cleanup in insert_module().

* runtime/staprun/staprun_funcs.c (insert_module): Better cleanup on error
  exit.

eventcounter: more output tweaks

eventcount: improve output layout some more

eventcount script: improve startup message

perf: document default perf.*.sample(XX) value/behavior

eventcount: extend script with more info and prettier formatting

PR12508 Changed eventcount.stp to use target_set tapset

This change avoids cases where the target program runs several layers
away.

PR12508 eventcounting script

An eventcount.stp script has now been added which allows for event
counting in the format of 'stap eventcount.stp syscall.* process.end ...'
with a printout of tid's event and count.

NEWS - news blurb
testsuite/systemtap.examples/general/eventcount.meta - meta file
testsuite/systemtap.examples/general/eventcount.stp - script itself

Several additional metrics for log file PCP agent.

Add in counts of bytes and events seen per logfile, and the
current log file size for each.  This will aid debugging and
day-to-day monitoring of more complex, hierarchical (r)syslog
deployments, for example.

Added a fetch callback so initial code to check status of the
monitored log files could be added - will need to extend this
though, to deal with the common case of logfile rotation.

Minor cleanups - pmid_string field is not a string, rename;
do not explicitly initialise global variables to zero, as the
compiler will happily do that; metric semantics for a couple
of metrics not-quite-right (discrete vs instant).

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

Fix typo in tapset documentation

tapset/context.stp - target not 'targer'

Generate better file name for Tapset html

As a default xmlto uses reXXXX.html for file names for the generated html files.
This has two drawbacks: it is not meaningful to humans and is likely to change
between builds of the tapset reference manual. The added XMLTOHTMLPARAMS
option generate more nmemonic names for the files and generates files that
are less likely to change between builds.

Update the domain number comment now that one is reserved.

Uncomment the seek-to-end-of-log-file code in pmdalogger.

For real world uses, we need to do this so that even moderately
sized log files are not read from start to end whenever the PMDA
starts up - wastes CPU cycles and (re)generates events that may
have happened long ago.

Remove further shared library remnants in pmdalogger build.

remote: Normalize the shell used to invoke stapsh

We're using a little bit of shell magic on the remote side to tell
whether a system has stapsh available, but that magic depends on
specific shell functionality.  We now explicitly start it with
/bin/bash, so the user's choice of $SHELL doesn't matter.

* remote.cxx (ssh_remote::connect): Wrap the command in /bin/bash -c.

Loop server requests over every remotes' session

Now the list/trust-server options work on every unique session generated
by the requested remote targets.  This enables usage such as:
    $ stap --remote HOST --list-servers

* main.cxx (passes_0_4): Move the version/session banner to main.
  (main): Print the version just once, and the session banner for each
  unique session.  Perform server actions within the session loop.

Restore signals after removing the tmpdir

This is necessary because passes_0_4_again_with_server() will remove the
tmpdir before trying again on a server.  If it is then successful, the
proceeding run should be interruptible as normal.

* main.cxx (remove_temp_dir): Restore signals after removal.

usage message: identify parameters for --{list,trust}-servers=MAGIC

remote: Support ssh on custom ports

* remote.cxx (remote::create): Heuristically disambiguate between URI
  scheme:path and SSH host:port.
  (ssh_remote::create): Parse an optional :port in the host string.
  (ssh_remote::connect): Pass the port to ssh -p.
  (ssh_legacy_remote::open_control_master): Pass the port to both
  ssh -p and scp -P.

remote: Give a better message for ssh connection failures

* remote.cxx (ssh_remote::connect): Flag rc==255 as an ssh error.

remote: Allow prefixing lines with the host index

When --remote-prefix is given, each line from remote scripts will be
prefixed with "N: ", where N is the index of that host among all the
--remote options on the command line.

* remote.h (class remote): Add an optional prefix string.
* remote.cxx (remote::run): Set the prefix if desired.
  (stapsh::handle_poll): When using prefixes, read data line-wise.
* session.cxx (systemtap_session): Add bool use_remote_prefix.
  (systemtap_session::parse_cmdline): Set it with --remote-prefix.

tolerate kernel builds with missing .note.gnu.build-id sections

* translate.cxx (dump_unwindsyms): Add back the check "Don't save
build-id if it is located before _stext."
* runtime/sym.c (_stp_module_check): Don't do buildid check for
-DSTP_NO_BUILDID_CHECK.
(_stp_usermodule_check): Likewise.
* buildid.exp: Test -DSTP_NO_BUILDID_CHECK

Logger PMDA updates: remove DSO logic, cut back on debugging info, etc.

Changes from Ken McDonell <kenj@internode.on.net>

* pcp/src/pmdas/logger/logger.c (usage): Updated usage message.
  (logger_init): Removed DSO startup logic.
  (main): Ditto.
  (logger_end_contextCallBack): Only log debugging information when asked
  for.
  (logger_profile): Ditto.
  (logger_fetchCallBack): Ditto.
  (read_config): Ditto.
  (logger_pmid): Ditto.
  (logger_name): Ditto.
  (logger_children): Ditto.
* pcp/src/pmdas/logger/percontext.c (ctx_start): Ditto.
* pcp/src/pmdas/logger/event.c (event_create): Ditto.
  (event_fetch): Ditto.
* pcp/src/pmdas/logger/util.c (start_cmd): Ditto.
* pcp/src/pmdas/logger/root: Includes domain.h.

Added 'logger.perfile.{LOGFILE}.path' metric in the logger PMDA.

* pcp/src/pmdas/logger/logger.c (logger_fetchCallBack): Add support for
  new 'logger.perfile.{LOGFILE}.path' metric, which returns the logfile
  pathname.

PR12211 gettext the new err() and regenerate the po/ files

po/* - update with new strings
runtime/staprun/mainloop.c - gettext the err string

PR12211: Tweak the child-exit warning output

* runtime/staprun/mainloop.c (chld_proc): Use err() instead of dbug(0,)
  so we don't get the source:line dump, and remove the ':' as well.

Added help text for the dynamic metrics in the logger PMDA.

* pcp/src/pmdas/logger/logger.c (logger_text): New function that provides
  help for the dynamic metrics.
  (logger_init): Set up dynamic metrics help text.
* pcp/src/pmdas/logger/help: Removed commented out help for the dynamic
  metrics.

PR12211 stapio now reports abnormal child exit

NEWS - description of the new functionality
runtime/staprun/mainloop.c - changes to now handle and report the
exit status/signal.

Correct the local name in vfs.readv

Reported-By: Jake Maul <jakemaul@gmail.com>

Don't cache "N/A" in __find_bdevname()

Validate PMNS names in the logger PMDA install script.

* pcp/src/pmdas/logger/Install: Validate PMNS names.

Fix setjmp.exp and server tests for RHEL5 systems.

* testsuite/lib/systemtap.exp (start_server): To properly support older
  versions of tcl (as on RHEL5 systems), don't use the newer form of
  'catch'.  Instead, use the global 'errorCode' variable when getting the
  exit code of a child process.
* testsuite/systemtap.base/setjmp.exp: Ditto.

Add -DSTAP_SDT_ARG_CONSTRAINT

    * sdt.h (STAP_SDT_ARG_CONSTRAINT): Allow override of default nor constraint.
    * cxxclass.exp: Use it.

Added more testsuite error handling.

* testsuite/lib/systemtap.exp (start_server): Catch errors when starting a
  server.
* testsuite/systemtap.base/buildid.exp: Be sure to call wait after spawn.

Improve error handling in target_set.exp.

* testsuite/systemtap.base/target_set.exp: Handle EOF and add failure
  reasons.

Fixed 'test_check' compile line in network/tcp_init_cwnd.meta

* testsuite/systemtap.examples/network/tcp_init_cwnd.meta: Fix
  'test_check' entry to compile 'tcp_init_cwnd.stp', not
  'tcp_init_cwnd.meta'.

Enhance description for -c and format function::target better.

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

* 'master' of git://sources.redhat.com/git/systemtap:
  PR12645 (partial): Only insert uprobes in non-writable executable sections.

PR12645 (partial): Only insert uprobes in non-writable executable sections.

* runtime/uprobes-common.c (stap_uprobe_mmap_found): Only try to insert
  uprobes in non-writable executable sections.

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

* 'master' of git://sources.redhat.com/git/systemtap:
  PR12645 (partial): Improved uprobe/uretprobe error message.

i18n: regen po files

i18n: Fix accidental whitespace changes in stap -V banner

update-po: Use source-tree dtrace.in as input for po/POTFILES.in

PR12645 (partial): Improved uprobe/uretprobe error message.

* tapsets.cxx (uprobe_derived_probe_group::emit_module_decls): Improve new
  error messages by including the probe name.

Improved error handling in uprobe probes.

* tapsets.cxx (uprobe_derived_probe_group::emit_module_decls): Make sure
  we go through the probe epilogue even in an error in the emitted
  enter_uprobe_probe().

Improved error handling in uretprobe probes.

* tapsets.cxx (uprobe_derived_probe_group::emit_module_decls): Make sure
  we go through the probe epilogue even in an error.

module shutdown: be verbose upon a hang

In the generated module_exit() function, there exists an infinite loop
that waits for all probe handlers that might still be struggling along
to shut down, before authorizing removal of the probe module.  If
something is stuck though, it will stay stuck without any diagnostics
short of a hung stapio/staprun process that's sitting in the kernel.

This patch causes a message to be printk KERN_ERR'd, in case the
shutdown synchronization takes more than a second.

* translate.cxx (emit_module_exit): Print a message if holdon-spinning
  for more than a second.

Fixed minor error message bug in utrace_syscall_args.stp.

* testsuite/systemtap.base/utrace_syscall_args.stp: Fixed error message
  bug.