Add extra check for lock file before trying to call bmsubmit

*java/stapbm: add extra check for the lock file before submitting the check,
      change sdterr redirect (wasn't actually redirecting)

configure.ac: fix default --with-helper

configure.ac: fix spurious test -n

fix java_pid_str in absence of HAVE_JAVA_HELPER

systemtap.spec: install libHelperSDT as appropriate

configure.ac: rework configury to support --with-helper and --with-jdk options

The different options control where libHelperSDT.so and HelperSDT.jar end up.
Use --with-helper to install to a folder like libexec for subsequent symlinking
Use --with-jdk to install a jdk folder under ...ext/ and ...amd64/ directories

Allow java(foo).class(bar).method(foobazz:LINE) {} style syntax

*tapset-method.cxx: parse for the line number and edit the system function
    accordingly

Adjust stapbm to allow more than one rule per process

*java/stapbm: Relax script to allow more than one rule per process
      This required file locks to be used to ensure bminstall
      was only called once, as well as only allowing one
      rule to be submitted at a time.
*java/tapset-method.cxx: stapbm script always needs all parameters to be
 passed to ensure properly running behaviour

Remove stapbm.sh, renamed as stapbm

*java/stapbm.sh has been renamed to stapbm

Further remove unneeded functions

*tapset-method.cxx: Remove bminstall function and debug line

Add stapbm to autoconf and remove exact paths

*Makefile.in: remove rebase fragments
*configure: updated configure
*java/Makefile.am: add stapbm
*java/Makefile.in: resulting makefile
*tapset-method.cxx: updated script paths and helper location

Moved bm{install,submit} operations to pass 5

*java/stapbm: removed unneeded suffix from file name
*session.cxx: remove byteman operations/variables
*session.h: remove byteman operations/variables
*tapset-method.cxx: add new add new probe points and ensure that
    the main probe body isn't copied to all new
    probe points

merge changes (Makefiles: automatically install libHelperSDT)

incremental commit

Makefiles: automatically install libHelperSDT

Moving byteman operations to pass 5

Initial commit of stapbm.sh script

*java/stapbm.sh: script to build a byteman rule

Add .return style probes for java methods

Optimize the specific mark("X") probe actually specified

*java/HelperSDT.c: alter mark names to be more simple/predictable
*tapset-method.cxx: add function call to determine which mark name
    to return

Move stap_system for bminstall into its own function

*tapset-method.cxx: move the duplicate code into its own function

Only run bminstall.sh per specified java process once

*tapset-method.cxx: compare the script to see if it's already been installed
    if it has, don't install it again just let bmsubmit.sh
    redefine the probe

Only install rules once per java process

*session.cxx: re-enable the check before executing java_detach
*tapset-method.cxx: only bminstall.sh the process if its not already been recorded
    and check to see if we've installed any java procs already

Internationalize tapset-method strings

*tapset-method.cxx: add macros to internationalize strings

Allow more than one java probe point per script

*tapset-method.cxx: add filter to match probe against the class
    and method, if it doesn't match, then skip
    the probe point
*session.cxx: change how rules are uninstalled

Tweak how log file is specified for bmsubmit.sh commands

*session.cxx: alter bmcommand ordering and include explicit " -o"
*tapset-method.cxx: alter assignment of log file and ordering in
    bmsubmit.sh command

Allow java("fully qualified class") style syntax

*session.cxx: add class string member variable
*session.h: add member variable
*tapset-method.cxx: add class specification functionality

Cleanup and use stap_system() to launch commands

*tapset-method.cxx: use stap_system() to launch bmsubmit and bminstall
    commands instead of forking and managing it ourselves.
    Also, a small cleanup of commented out vars and varnames

Uninstall the specific byteman rule we generated

*session.cxx: add systemtap_session member variables and use
      stap_system to launch bmsubmit.sh uninstall command
*session.h: added member variable declarations
*tapset-method.cx: use member variables so we know which rule to uninstall
*tapsets.cxx: reorder which tapsets to register so normal process probes
      are registered first

Exit gracefully from a java process (uninstall rules)

*session.cxx: add java_detach() function, exec bmsubmit.sh -u
*session.h: declaration of java_detach
*tapset-method: change some local variables to systemtap_session
variables for java_detach

Add basic java/ Makefile.am to compile libHelperSDT.so

*Makefile.in's: autoreconf changes
*java/HelperSDT.c: fixed some warnings
*java/HelperSDT.h: removed several uneeded functions
*java/HelperSDT.java: removed several uneeded methods

Pipe output of bmsubmit.sh command to log file instead of System.out

*tapset-method.cxx: pipe output of bmsubmit.sh command to log file
*java/TODO: update the todo list

Change syntax from process(JAVAPID). to java(PID)

*tapset-method.cxx: change syntax keywords
*java/TODO: update todo file

Re-add HAVE_HELPER variable from merge, add conditional

*configure.ac: re-add HAVE_HELPER, undef HAVE_HELPER if bmsubmit.sh,
       bminstall.sh and java aren't found.

Add correct check for byteman (bmsubmit.sh & bminstall.sh)

*configure: autoconf generated
*configure.ac: added bminstall.sh and bmsubmit.sh check

Add java specific helper files

*java/HelperSDT.c: the actual value extraction and passing to stap
*java/HelperSDT.h: the jni header file
*java/HelperSDT.java: the java jni file
*java/README: how to manually compile and add the .jar and .so
*java/TODO: the TODO for automation
*tapset-method.cxx: fix the helper name

Added java/TODO file

*java/TODO: listed more todo items to keep track of

Make use of HAVE_HELPER config variable

*tapset-method.cxx: only create new probe point and exec byteman
                    if HAVE_HELPER / --with-helper was defined
                    and use the specified variable in the new
                    process("libHelperSDT.so").mark("*") probe

Properly define variable if --with-helper is defined

Add have_java configuration variable

Adding this variable will allow us to check if per-method probes
are even possible and at the same time define where the jni
library we need is.

Add more log messages and fix various formatting

*tapset-method.cxx: fix various formatting and add verbosity messages

Actually error out if more than 10 method parameters are specified

*tapset-method.cxx: if more than 10 method parameters are specified
    we need to error out due to the max parameters
    in the static marker

Fix method argument amount calculation

*tapset-method.cxx: previously the number of arguments in a method
    was determined by counting the number of ','s.
    Fix this by checking for distance between ()'s

Correct method name specified in generated byteman rule

*tapset-method.cxx: fix the method name passed to the stap probe

Initial java per method probing functionality

* Makefile.{am,in}: add tapset-method.cxx to file list
* session.cxx: add java_derived_probes to systemtap_session class
* session.h: likewise
* tapsets.cxx: add new token strings, call java probe point builder
* tapset-method.cxx: actual byteman exec and probe redefinition/builder

linux tapset: adapt to 3.9 movement of MAX_RT_PRIO to new linux/sched/rt.h

stapdyn: add a one-second timeout to the mutatee state-change ppoll

Intermittently, the filehandle-based notification hangs (with rawhide
dyninst 8.1.1 x86-64), but subsequent event polling works.  So let's
not let stapdyn block indefinitely on the former.

* stapdyn/mutator.cxx (mutator::run): Add a 10-second timeout.

PR15422: fix compiler warning when --runtime=dyninst loc2c's 32-on-64 binary

* runtime/dyninst/loc2c-runtime.h (*deref, *_user_asm): Explicitly
  widen incoming address to uintptr_t.

BZ915247: remove i18n stub support from python dtrace.in script

* Makefile.am (update-po): Drop dtrace.in from POTFILES.in.
* Makefile.in, po/*: Regenerate.
* dtrace.in: Remove gettext calls.

Handle an allocation failure in _stp_vsnprintf().

* runtime/vsprintf.c (_stp_vsnprintf): If _stp_reserve_bytes() fails,
  print an error and exit.

Fixed PR14491 by adding a real transport layer for dyninst.

* runtime/dyninst/transport.c: New file.
* runtime/dyninst/transport.h: New file.
* translate.cxx (c_unparser::emit_module_init): Emit a call to
  stp_session_init_finished() before probe registrations.
  (c_unparser::emit_module_exit): In dyninst mode, delay
  releasing the context.
* tapsets.cxx (common_probe_entryfn_epilogue): In dyninst mode, delay
  releasing the context.
* runtime/dyninst/stapdyn.h: Add stp_dyninst_session_init_finished()
  prototype.
* runtime/dyninst/shm.c (_stp_shm_destroy): Right before unmapping the
  shared memory, tear down the contexts.
* runtime/dyninst/runtime_context.h (__stp_runtime_contexts_free): New
  function.
* runtime/dyninst/runtime.h (stp_pthread_rwlock_init_shared): New function.
  (stp_dyninst_shm_connect): Initialize the transport by calling
  _stp_dyninst_transport_init().
  (stp_dyninst_session_init): Ditto.
  (stp_dyninst_session_init_finished): New function.
  (stp_dyninst_dtor): Shutdown the transport by calling
  _stp_dyninst_transport_shutdown().
* runtime/dyninst/print.c (_stp_print_flush): Now just calls
  _stp_dyninst_transport_write().
  (_stp_reserve_bytes): Now just calls
  _stp_dyninst_transport_reserve_bytes().
  (_stp_unreserve_bytes): Now just calls
  _stp_dyninst_transport_unreserve_bytes().
* runtime/dyninst/common_session_state.h: Add transport data to session
  data structure.
* runtime/common_probe_context.h: Add transport data to context structure.
* runtime/dyninst/io.c (_stp_vlog): New function.
  (_stp_warn): Now uses _stp_vlog().
  (_stp_error): Ditto.
  (_stp_softerror): Ditto.
  (_stp_dbug): Ditto.

Expand the SDT benchmark locations

- Add the loop index as an argument to stap:benchmark.
- Add stap:benchmark-thread-begin/end around each thread.
- Add stap:benchmark-begin/end around the whole thing.

Support "pid" interface of perf (for per-process case)

The interface of perf_event_create_kernel_counter() in kernel versions prior
2.6.37 takes the process id instead of a pointer to the task struct.

Signed-off-by: Torsten Polle <Torsten.Polle@gmx.de>

mutex-contention sample: drop compulsory NNNN argument used for array sizing

PR15363: tweak registration-warning message to print inode-offset

Sadly, virtual address to feed/decode objdump -d is not available.

PR15363 warn but don't error-abort, on inode-uprobes registration error

Fix weekdays in the rpm spec %changelog (per rpmlint)

uretprobes: Update for aarapov's v1 (posted April 3)

The uretprobes handler is now called ".ret_handler", with a new "func"
parameter for the breakpoint vaddr (which we don't use for now).

extend error::pass2 man page

Enable process.plt probes for ARM.

(Confirmed in binutils bfd/elf32-arm.c, elf32_arm_plt0_entry / elf32_arm_plt_entry.)

stapdyn runtime: don't use mktemp to create a file name

* runtime/dyninst/shm.c (_stp_shm_init): Just use a pid-based name; we only
  need mild uniqueness to avoid accidental collision.

BZ855981: Further tweak getcpu compatibility

- Keep using sched_getcpu() where possible - it's sometimes faster.
- Pass the correct parameters to syscall(getcpu).
- Direct the tapset cpu() through this compat code too.

BZ855981/948279, on second thought, use -march=i586 vice i686

BZ855981/948279, work around gcc default -march=i386 __sync_* incapability

* buildrun.cxx (compile_dyninst): Add CFLAGS += -march=i686 for arch=i386.

BZ855981: stapdyn runtime compatibility with RHEL5

Old kernels & old glibcs require some minor compatibility hacks.

* runtime/dyninst/linux_defs.h: Dummy-define CLOCK_MONOTONIC_RAW
  if required.
* runtime/dyninst/runtime.h: Dummy-define BITS_PER_LONG if required.
  _stp_runtime_entryfn_get_context: Use syscall(__NR_gettid) as
  fallback from __NR_getcpu if required.

PR15053: NEWS item and tweaked usage message for stapdyn

PR15315: Filter inode-uprobe processes in the handler

We already had process tracking, mainly for setting semaphores, but this
gives the list needed to check if an incoming uprobe hit is for a
process we care about.  So now stapiu_probe_prehandler will consult this
list before proceeding with the rest of the probe handler.

This also updates locking so that readers of this process list have a
more fine-grained lock, rather than the global stapiu_process_lock.
Code that needs to reserve/free a uprobe slot will both the global lock
and the specific target lock.

eatmydata: fix cut&paste'd description text

eatmydata: don't override $return for non-participating processes

samples: regenerate indexes, tweak keywords

eatmydata: new sample script

PR15306: Guard the stapdyn probe prologue for no shm

* runtime/dyninst/common_session_state.h (GET_SESSION_ATOMIC,
  _stap_hash_seed, probe_alibi, probe_timing, stp_session_context):
  Return NULL if the session (in shm) doesn't exist yet.
* tapsets.cxx (common_probe_entryfn_prologue): Skip the whole probe if
  the session_state isn't available at all (i.e. no shm yet).
  (common_probe_entryfn_epilogue): Close the session_state block.

mutator.cxx:481:42: error: expected primary-expression before '.' token

Signed-off-by: Adrian Negreanu <groleo@gmail.com>
Signed-off-by: Josh Stone <jistone@redhat.com>

Add stap-prep.1 to systemtap.spec

*systemtap.spec: add %{_mandir}/man1/stap-prep.1* to be packaged

man/Makefile.in: regenerate

install stap-prep.1 man page

stap-prep: debian: do not check for linux-kbuild

linux-headers* already depends on the right linux-kbuild* package and
our heuristic for guessing the package name is lacking. For example
with linux-image-3.2.0-0.bpo.4-amd64 version 3.2.35-2~bpo60+1 from
squeeze-backports it predicts "linux-kbuild-3.2.0" while the package
is actually named "linux-kbuild-3.2".

man stap.1: add reference to stap-prep (1)

man: add man page for stap-prep (1)

stap-prep: add support for deb based systems

Prepare stap-prep for support for non-rpm systems

configury: regenerate on fedora-18 with autoconf 2.69 etc.

Fixed PR15114 by adding uid/gid convenience variables to nfsd.proc.*.

* tapset/linux/nfsd.stp: Added uid/gid convenience variables to
  nfsd.proc.* aliases.
* testsuite/buildok/nfsd-detailed.stp: Added compile test for uid/gid
  convenience variables.

stap-report: include dpkg | grep linux on deb/ubuntu platforms

Add a few more ENABLE_NLS checks to be consistent

fixes for ENABLE_NLS=0

v2:
    [Josh Stone]: replace _F(_N(a,b,c),...) with _NF(a,b,c,...)

Signed-off-by: Adrian Marius Negreanu <adrian.m.negreanu@intel.com>
Signed-off-by: Josh Stone <jistone@redhat.com>

PR15053: stapdyn -G global variable setting support (initial patch)

* buildrun.cxx (make_dyninst_run_command): pass module arguments on to stapdyn.
* runtime/dyninst/runtime.h (set_int64_t): new function.
* runtime/dyninst/stapdyn.h (stp_global_setter): function prototype.
* stapdyn/mutator.cxx (mutator::mutator): add modoptions to mutator.
  (mutator::init_modoptions): new function.
  (mutator::run_module_init): call init_modoptions to initialize globals.
* stapdyn/mutator.h (class mutator): add modoptions to mutator.
* stapdyn/stapdyn.cxx (main): collect module arguments from command line.
* translate.cxx (struct c_unparser): new emit_global_init_* functions.
  (c_unparser::emit_global_param): stapdyn no longer calls this function.
  (c_unparser::emit_global_init_setters): new function.
  (c_unparser::emit_global_init): emit default initial values in stapdyn mode.
  (c_unparser::emit_global_init_type): new function.
  (c_unparser::emit_module_init): copy initial values of stapdyn globals
     to shared memory as appropriate.
  (translate_pass): emit static-initialization struct for stapdyn,
     and also emit the setter function to access this struct.
* translate.h (struct c_unparser): document new emit_global_init_* functions.

PR15279: Leave REG_IP alone in 3.9ish uprobes

We had experimental code to use uprobes that passed the IP as an extra
parameter, but that was dropped in favor of modifying pt_regs directly,
before even calling the handlers.  So this restores our handler to never
take an IP, and updates REG_IP only for older versions.

* buildrun.cxx: Don't run autoconf-inode-uprobes-noaddr.c.
* runtime/linux/autoconf-inode-uprobes-noaddr.c: Deleted.
* runtime/linux/uprobes-inode.c: Set STAPIU_NEEDS_REG_IP for older
  uprobes, and consolidate to a single handler function that uses that
  flag to gate IP-setting.
* testsuite/systemtap.context/uprobe_uaddr_mark.*: New test, makes sure
  that uaddr() matches the statement(ADDR) in pp().

PR15290: remove additional uretprobe cruft

PR15290: add support for aarapov's uretprobes

Signed-off-by: Adrian Marius Negreanu <adrian.m.negreanu@intel.com>
Signed-off-by: Josh Stone <jistone@redhat.com>

Remove stapdyn '-std=c++11' compile flag.

* stapdyn/Makefile.am: Remove "-std=c++11" compile flag.
* stapdyn/Makefile.in: Regenerated.
* stapdyn/dynprobe.cxx: Change '__typeof__' back to 'typeof'.
* stapdyn/mutator.cxx: Ditto.
* stapdyn/mutator.h: Ditto.

Include runtime/dyninst/stapdyn.h properly in stapdyn C++ code.

* stapdyn/dynprobe.cxx: Make sure when we include
  runtime/dyninst/stapdyn.h we use 'extern "C"'.
* stapdyn/mutatee.cxx: Ditto.
* stapdyn/mutator.cxx: Ditto.
* stapdyn/mutator.h: Ditto.

Added better process cleanup in 6 testcases.

* testsuite/systemtap.clone/probe_by_pid.exp: Better process cleanup.
* testsuite/systemtap.server/client.exp: Ditto.
* testsuite/systemtap.server/server.exp: Ditto.
* testsuite/systemtap.server/server_args.exp: Ditto.
* testsuite/systemtap.server/server_concurrency.exp: Ditto.
* testsuite/systemtap.server/server_locale.exp: Ditto.

Make sure dyninst stat/map locks are using PTHREAD_PROCESS_SHARED.

* runtime/dyninst/map_runtime.h (_stp_map_initialize_lock): Be sure to set
  'PTHREAD_PROCESS_SHARED' on mutexes by calling
  stp_pthread_mutex_init_shared().
* runtime/dyninst/stat_runtime.h: Ditto.
* runtime/dyninst/runtime.h: Add forward declaration of
  stp_pthread_mutex_init_shared().

Make sure macro chain tokens are available for error reporting.

Found by a combination of zzuf and valgrind. When reporting the chain
token we need a real copy, because the existing (junk) token might have
been discarded.

Another parse error found by zzuf. Don't crash when we find early EOF.

Example crasher: stap -e 'probe begin { for (i = 0; i++;'

Parse errors found by zzuf. Don't crash when we find early EOF.

Add cvs add && cvs commit operations for the tapset/ directory

*scripts/update-docs: add missing cvs operations for tapset files

run-staprun: Preserve the user PATH and HOME

Just for convenience, keeps PATH and HOME as the user's to make sure
-c CMD has a normal non-root environment.

Add options to help benchmark SDT

These are secret, undocumented, for developer use only.  Use stap
inception-style as the -c CMD of a stap which probes the SDT mark
process.provider("stap").mark("benchmark").

  --benchmark-sdt-loops=N
        Loop over the SDT probe point N times.

  --benchmark-sdt-threads=N
        Run the loop simultaneously in N threads.

stapdyn: fix the context->data_index

This regressed in commit c08bf9de, which lost the initialization of that
field.  Thus there was a data race in pmaps, stats, etc. in writing to
data_index 0, rather than the true index of the currently-owned context.