More improvements to the client.exp test harness.

Changes to account for unanticipated, but correct, output.

Initial version of pcp logger pmda.

Fix RHBZ#676474 by removing some incorrect asmlinkage() markers.

* tapset/nd_syscalls.stp: Remove asmlinkage() for do_fork, do_execve,
  compat_do_execve.

Test harness improvements for testing using a compile server.

- Always generate and authorize new server cert when a server starts.
- Only consider online servers when testing for trust in client.exp.

Don't report IPv6 entries returned by avahi, for now.

Fix a README typo of CONFIG_MODULE_UNLOAD

(reported by "wil_" on #systemtap)

Fix the indentation of systemtap_session::usage

Clean up exit paths

We need to strive not to leave files/processes around, so exit() calls
are reduced to only places where nothings been allocated yet.  Errors
are rarely so fatal to require immediate exit.

Use a ControlMaster for ssh remotes

We can start the ssh remote with a ControlMaster connection, using a
unique socket in the tmpdir, and then each subsequent remote command can
reuse that connection.

* remote.cxx (ssh_remote::open_control_master): Connect to the remote
  host with a ControlPath that later commands can reuse.  The relevant
  options are saved in member ssh_opts.
  (ssh_remote::close_control_master): Ask the control process to exit.
  (ssh_remote::get_uname, ::start, ::finish): Start using ssh_opts, so
  the ControlPath is always specified.

Precreate the main tmpdir

Create the primary temp dir early on, so we have a place to store
temporary data for remotes (like control socket paths).

* main.cxx (main): Create the tmpdir before remotes are started.
  (create_temp_dir, remove_temp_dir): Tolerate repeat calls.

Use a proper session copier when cloning

* session.cxx (systemtap_session): Define copy and assignment
  constructors without implementation, to avoid implicit behavior.
  Add a new copy constructor with a new kernel/arch.
  (systemtap_session::clone): Use the new copier.

Rework setup_server to ensure that servers get shut down on error.

Also reqorganize in order to save more output from server smoke tests.

accept and ignore %l (long) option in *printf family

* staptree.cxx (print_format::string_to_components): Parse 'l' type modifier.
* int1.stp: Test '#' flag and 'l' type modifier.
* int1.exp: Likewise.

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

Conflicts:
main.cxx

PR 11844 - --use-server-on-error option

- Now defaults to 'off'
- Errors no longer automatically result in retry using the server.
  This is done only for certain whitelisted errors.
  systemtap_session::set_try_server() is called for these errors.

Fix two more 'temporary' typos

Fix typo in message (temorary -> temporary) which broke compile server.

stap-server-connect uses this message to locate the temp directory.

Add NEWS item for process.mark $$parms

Support $$parms for sdt.

* tapsets.cxx (sdt_uprobe_var_expanding_visitor): Add
visit_target_symbol_arg visit_target_symbol_context
(visit_target_symbol_context): Handle $$name, $$provider, $$parms
(visit_target_symbol_arg): Handle symbol
* sdt_misc.exp:  Test $$parms.

PR 12428 - Document Compile Server/Client Setup and Admin

- New README.unprivileged
- Update info about unprivileged mode in stap.1 and in stap-server.8
- Don't bother autoconfing paths in stap-server.8. Refer to stappaths
  instead.

Add gettext-devel build requirement to spec file.

Avoid using $env(HOSTNAME)

When running crontab jobs the systemtap.server/server.exp tests
would not run because $env(HOSTNAME) is unset. HOSTNAME is
set by the bash shell and is not available in crontab run tests.
An equivalent [info hostname] is used in its place.

Revert accidental gettext clobbering

Wrapped lines, fixed some white space issues.

Some comments with the original strings have been left in for
ease of comparison. They will be removed for the final I18N commit.

gettext changes in main.cxx and session files

to invoke run $ xgettext -c -k_ main.cxx session.*
not every string has been set up for gettext translation, at this
point it is just to make sure it doesn't break stap.  The more
complicated strings will be I18N in time.

Autoconf groundwork for I18N of systemtap

TODO: localedir needs to be a variable and not hard-coded
further changes to files for .po file output are upcoming.

Cleanup to improve sdt on ppc.

* tapsets.cxx (sdt_uprobe_var_expanding_visitor::visit_target_symbol):
Use variable shift amount.  Don't cast unless probe type 3.
* sdt_misc.exp (type_flags): Don't use sup_flags
* sdt_types.c: Check __SIZEOF_SIZE_T__ for long long tests.

Assert that TMPDIR, if set, is a boring path

We don't want any excitement in the paths we must use.
e.g. Kbuild doesn't cope well with spaces in the M="/module path"

* session.cxx (systemtap_session::check_options): Restrict the
  characters allowed in TMPDIR to a universally-safe set.
* testsuite/parseko/cmdline22.stp: New, check that TMPDIR spaces are
  not allowed, even if that path really exists.

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

Ensure systemtap.stress/conversions.exp generates individual errors and warnings

PR12168 eliminated duplicated error messages and changed the count of
ERROR and WARNING messages. Adding the "-vv" option turns off the
duplication eliminate and allows an accurate count of the number
of times ERROR and WARNING messages occur.

PR 11844 - --use-server-on-error option

- Mechanism is in session.cxx, session.h.
- Code in main.cxx:passes_0_4 calls set_try_server upon compilation
  errors in phases 0-4.
- set_try_server (systemtap_session:dont_try_server) can be set for
  individual errors which do not warrant a retry (e.g. syntax errors)
  to override.

Make sure NSS and NSPR are properly initialized and shutdown.

- rpmReadConfigFiles initializes NSS, but does not shut it down.
  We now call rpmFreeCrypto if available or call NSS_Shutdown directly otherwise
  in missing_rpm_enlist
- In some versions of NSPR, PR_Init must be called once and only once before NSS_Init or
  NSS_InitReadOnly is called. In later versions, it is unnecessary and is ignored.
  systemtap_session::NSPR_Initialized is now static so that it is
  shared by all active session objects.

Remove unused variables and calls which initialized them.

Fixed BZ559599 by making target_set.stp use syscall probes for ia64.

* tapset/target_set.stp: Dwarfless probes don't work on ia64, so fall back
  to the 'syscall' probes (as we do for kernels < 2.6.18).

Fixed BZ661424 by checking for $prev_p first in __scheduler.ctxswitch.kp.

* tapset/scheduler.stp: Checks for '$prev_p' before checking for '$prev'
  so that i386 RHEL5 (which has both arguments) works correctly.

BZ673085: amplify documentation on initialization of probe alias variables

Avoid use of limits include in sdt.  Sign extend signed HI/SI values.

* sdt.h (limits): Remove.
(__sdt_type): Predefine type signness.
* tapsets.cxx (sdt_uprobe_var_expanding_visitor::visit_target_symbol):
Sign extend signed HI/SI values.
* sdt_asm.stp (rN): Add rNd tests

stap-report: also add a dummy "stap -vv -p4 ..." run

stap-report: dump more state

$XDG_DATA_* since that can interfere, and /proc*/*/kprobes*, for completeness

GCC 4.6 unused-variable warnings

We set a couple of variables that later code doesn't use.  GCC 4.6
warns about them, which our C*FLAGS=-Werror turned into a build failure.
Worked around these by adding a (void) var; to add a dummy use.

adding #ifdef CLONE_STOPPED to stop module compilation error

Audit some minus/dash/hyphen use in manpages

Groff treats "-" as a hyphen, U+2010.  When the intention is to produce
a literal ASCII minus sign, U+002D, it must be "\-".  This matters for
command-line options and examples to be copyable from the manpage.

I've updated cases that should obviously be escaped.  At a glance, I
think we have other cases that could be normal hyphens, not escaped, but
that's less of an issue.

Add the cache options from commit d105f66 to stap.1

Add support for real URI syntax in remote targets

We still only support SSH for now, but this opens the door for other
protocols.  All of the following should work:

  --remote host
  --remote user@host
  --remote ssh://host/
  --remote ssh://user@host/

URI paths, queries, and fragments are all parsed, but don't have any
meaning yet, so they're treated as an error.  I can imagine using this
for protocol-specific options though, e.g. ssh://host/?Compression=yes

* remote.cxx (class uri_decoder): New, breaks down a URI string.
  (ssh_remote::ssh_remote): Add a constructor using a uri_decoder.
  (remote::create): Redirect based on the scheme of URI-looking remote
  targets.  Assume all others are still plain [user@]host for ssh.

PR 11862: remove obsolete stap-client script and friends

Rework and improve warnings for missing NSS and avahi headers, libs and tools

PR 11862: remove obsolete stap-client script and friends

- Integrate stap-client-connect.c into csclient.cxx
- Remove stap-client-connect.c

PR 11862: remove obsolete stap-client script and friends

- Remove deprecated scripts
- Remove parts of stap-env used only by the removed scripts
- Remove man pages for removed scripts
- Remove parts of stap-client-connect which allowed it to be built
  as a standalone tool (used by stap-client).
- Update spec file. systemtap-client package removed.

Improve sdt asm test coverage.

* sdt_asm.S:  Improve coverage of tested registers and values.
* sdt_asm.stp: Likewise.

PR12413, Provide TID information in kprocess.create and kprocess.release

PR12412: Make the text agree with the example (should be reads)

Allow sdt to avoid application defined min/max macros.

* sdt.h:  Surround limits inclusion with push_macro and pop_macro.

PR12419: fix x86 uprobes decoding of "repnz ret"

BZ669082: support powerpc prior to 2.6.15, resolving conflict with PR4186

* buildrun.cxx (compile_pass): Omit passing of ARCH=foo for
  ancient powerpc.  This enables native ARCH=ppc64 builds,
  but sacrifices ppc-target cross-architecture cross-compilation.
  (make_tracequery_kmod, make_typequery_kmod): Ditto.

PR12411 cont'd: use enum type decl

* tapsets.cxx (sdt_uprobe_var_expanding_visitor): Make regwidths
  an enum type decl, not an anonymous-enum-typed variable.  Move
  DRI macro #define/#undef just around its use.

RHBZ669751: old ia64 kernels lack task_pt_regs()

... so define it in terms of ia64_task_regs()

Switch to using a register width table instead of heuristic code.

       * tapsets.cxx (sdt_uprobe_var_expanding_visitor): Add DRI and regwidths.
       Change dwarf_regs to be a regname/regwidth pair.  Change initializer.
       * (sdt_uprobe_var_expanding_visitor::visit_target_symbol):  Use it.

Fixed BZ669740 by making target_set.stp use syscall probes in kernels < 2.6.18.

* tapset/target_set.stp: Dwarfless probes don't work in kernels < 2.6.18.
  So, fall back to 'syscall' probes for those kernels (instead of
  'nd_syscall' probes.

PR 11445: Check that assert_is_my_proc check is within all uprobes/utrace callback entry points

Add comment explaining why a check for is_myproc() is not generated as a
safety net for process.end probes and their variants.

systemtap.apps infrastructure fixes.

mysql.exp (stap-mysql.sh): Use push/pop_macro to avoid mysql max/min macros.
xulrunner.exp (stap-xul.sh): Make JAVA_HOME platform independent.

Cleanup warnings caused by CFLAGS correction.

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

Conflicts:
configure
testsuite/configure

Configuration problems.

- CFLAGS was being cleared in configure.ac (oldCFLAGS typo)
- CXXFLAGS was not always being set (CXFXLAGS typo)
- pick up $(AM_CFLAGS) and @PIECFLAGS@ in all C compilations in Makefile.am

bump version numbers to 1.5, for future release cycle

datestamp release

README: build instructions suggest elfutils version 0.151

PR12399: make sdt.h semaphores visibility("hidden")

NEWS: sdt.h v3

PR 10657: Document changes in NEWS and in stap.1.

Fix myproc-unprivileged in embedded_expr

The macro assert_is_myproc() forms a statement, so we need to wrap it
in ({}) to use it in an expression list.

* translate.cxx (c_unparser::visit_embedded_expr): Wrap stmt as expr.

PR 10657 - semi-unprivileged tapset embedded-C functions.

Including the comment /* myproc-unprivileged */ within an embedded C
tapset function or embedded C expression makes it available for use
by unprivileged users and also automatically adds a call to
assert_is_myproc() within the function or expression (i.e. it is
equivalent to including the comment /* unprivileged */ and
manually adding the the call to assert_is_myproc).

All instances within the tapset source tree have been updated.

PR11756 - adding warnings for deprecated declarations
Added the --version-check  flag, this will check if systemtap
constructors being used are version dependent. If so, a warning
detailing what syntax element is displayed.

PR12115: defeat uninitialized-variable warnings

PR12115: defeat uninitialized-variable warning

NEWS: tweak --remote blurb wording

AUTHORS bump

Fix for undefined STAPCONF_PERF_STRUCTPID

Fix paragraph indentation of ssl options

Disable auto-unpriv when --remote is used

PR12115: Document --remote in NEWS, stap.1, and --help

Merge branch 'master' into remote

Integrate sdt-compat.h into testsuite sdt.h

Makefile.am: Use sdt_types.h instead of sdt-compat.h
Makefile.in: Likewise.
tapsets.cxx: Likewise.
testsuite/sys/sdt.h: Integrate sdt-compat.h.
sdt-compat.h: Remove.
sdt_types.h: New.

PR11203: Provide default init functions for initscript/systemtap.

Used on plattforms where /etc/rc.d/init.d/functions is not available.

PR11203: Provide default init functions for stap-server.

Used on plattforms where /etc/rc.d/init.d/functions is not available.

tapset compat: irq tracepoints to use @cast() for $action

year bump

PR12168: fix testcase to unsuppress warnings

compatibility: perf_event.h updates for kernel 2.6.37

* runtime/autoconf-perf-structpid.c: New file.
* buildrun.cxx: Use it to define STAPCONF_PERF_STRUCTPID.
* runtime/perf.c: Respect it for perf_event_create_kernel_counter param 3.

pfiles test: adapt to upstream kernel commit 53c3fa206

sk_peercred->pid -> sk_peer_pid->....

testsuite: accept post-PR11736 hit-count message

Inelegant workaround for systemtap.apps mysql.exp's use of min/max

* mysql.exp (do_abi_check): skip it.
(probes_mysql.h): #undef min/max prior to sdt.h inclusion, then re-#define

Parenthesize probe arguments properly.

* sdt-compat.h (STAP_CAST): Parenthesize.
(STAP_PROBE{1,10}): Parenthisize kprobe arguments too.

* sdt_misc.c (buz): Test probing a boolean expression.

* sdt_misc.exp (sup_flags): Compile tests with -Wall -Werror

* sdt_misc.stp (test_probe_1): New.

* sdt_misc_.d (test_probe_1): New.

* sdt_types.c (arr_struct, primary_colors_var): Only declare if needed.

PR12172: make implied --unprivileged --use-server emit messages only if verbose

Conditionalize sdt tests resulting in templates with anonymous types.

* sdt_types.c (arr_struct, primary_colors, incomplete_struct_type):
Make conditional.

* sdt_types.stp: Likewise.

PR12172: make unprivileged mode default under certain conditions.

--unprivileged --use-server is now the default for invocations requiring
phase 5 for members of the group stapusr who are not also members of
the group stapdev.

Fixed PR12338 by updating softirq.entry/softirq.exit probes for 2.6.37+.

* tapset/irq.stp: Added 'vec_nr' variable to softirq.entry/softirq.exit
  probes.  Used @defined() to make sure old variables are present.
* testsuite/buildok/irq-detailed.stp: Updated softirq.entry/softirq.exit
  test.

Editing the Beginner's guide section on target variables.

Make syscall.nfsservctl* tests optional for kernel 2.6.35.9-64.fc14.x86_64.

* testsuite/buildok/syscalls2-detailed.stp: Make syscall.nfsservctl and
  syscall.nfsservctl.return tests optional.

tweak wording/suppressibility of .function/.statement warning

Add warning for supposedly line-oriented .function() probes.

2010-12-20  David Smith  <dsmith@redhat.com>

Fix PR11388 by handling nd_syscall.mmap changes for 2.6.33+ kernels.

* tapset/nd_syscalls.stp: Use arch-generic sys_mmap_pgoff() for
  nd_syscall.mmap2 if kernel version is 2.6.33 or higher.
* tapset/i386/nd_syscalls.stp: Only use arch-specific nd_syscall.mmap*
  probes if kernel version is less than 2.6.33.
* tapset/ia64/nd_syscalls.stp: Ditto.
* tapset/s390/nd_syscalls.stp: Ditto.
* tapset/powerpc/nd_syscalls.stp: Ditto.  Also add missing
  nd_syscall.sigaltstack probe.
* tapset/x86_64/nd_syscalls.stp: Ditto.  Also move sys_mmap_pgoff() probe
  to nd_syscall.mmap2 alias (instead of nd_syscall.mmap alias).
* testsuite/buildok/nd_syscalls-detailed.stp: Add generic nd_syscall.mmap2
  test if kernel version is 2.6.33 or higher.
* testsuite/buildok/nd_syscalls-arch-detailed.stp: New test.
* testsuite/lib/stap_buildok.exp: Add new nd_syscalls-arch-detailed.stp
  test to known ia64 failures.