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.

Add information about pretty printing the the SystemTap Beginners guide.

handle #!/usr/bin/env

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

tweak stapprobes blurb for .function -vs- .statement probes

Remove abusive use of monospace in langref.pdf

Expansion of Adrien Kunysz's patch in PR 12335. Use the \textt rather
than \tt to get the proper use of monospace fonts.

Clear semaphore when stap_uprobe slot is released.

uprobes-common.c (stap_uprobe_change_{plus,minus}): Clear semaphore.

PR 11922: Disable checking of DNS names on compile-server certificates.

The client now issues a warning and accepts the certificate.

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

* tapset/syscalls.stp: Use arch-generic sys_mmap_pgoff() for syscall.mmap2
  if kernel version is 2.6.33 or higher.
* tapset/i386/syscalls.stp: Only use arch-specific syscall.mmap* probes if
  kernel version is less than 2.6.33.
* tapset/ia64/syscalls.stp: Ditto.
* tapset/powerpc/syscalls.stp: Ditto.
* tapset/s390/syscalls.stp: Ditto.
* tapset/x86_64/syscalls.stp: Ditto.  Also move sys_mmap_pgoff() probe to
  syscall.mmap2 alias (instead of syscall.mmap alias).
* testsuite/buildok/syscalls-detailed.stp: Add generic syscall.mmap2 test
  if kernel version is 2.6.33 or higher.
* testsuite/buildok/syscalls-arch-detailed.stp: Only test arch-specific
  syscall.mmap* probes if kernel version is less than 2.6.33.

Fix s390 stack size problem.

* tapset/s390/nd_syscalls.stp (get_32mmap_args): Lower size of temporary
  variable to avoid stack size error.

Fix RHEL4 compile problem in runtime/staprun/staprun.c.

* runtime/staprun/staprun.c (remove_module): Only call prctl() if
  PR_SET_NAME is defined.

Fixed PR12330 by changing tapset/tty.stp to avoid ppc64 compile problem.

* tapset/tty.stp (_struct_winsize_u): Removed embedded-C function.
  (tty.resize): Removed calls to _struct_winsize_u() and just uses
  user_ushort() to get values in user memory.
* tapset/uconversions.stp (user_ushort): New function.
  (user_ushort_warn): Ditto.
* testsuite/buildok/conversions-embedded.stp: Added user_ushort() and
  user_ushort_warn() tests.
* testsuite/buildok/tty-embedded.stp: Removed (since all embedded-C
  functions have been removed from tty.stp).

A discussion about @defined to SystemTap Beginners Guide.

Move casting example in Beginners guide to extra/cast_example.stp

PR12309 part 2: ditch delayed_work for timers

* runtime/transport/transport.c: Remove all references to workqueues,
  delayed_work*; replace with same sort of timer use in relay_v2.c.

PR12309 part 1, use atomic_t for delayed_work state signaling

There was a possible race condition between multiple processes opening
the transport .ctl file concurrently.

* runtime/transport/transport.h (_stp_ctl_attached): Become atomic_t.
* runtime/transport/transport.c (_stp_attach, _stp_detach):
  Stop messing with it.
* runtime/transport/control.c (_stp_ctl_open|close_cmd): Use atomic_t
  and reorder logic to defeat race condition.

runtime: let staprun -d identify itself differently in task->comm

staprun.c (remove_module): Use prctl() to set task name to "staprun-d".

State which version systemtap @cast available in.

Free the URL of the ssl socket when finished with it.

PR 11922: Allow ip address as host name to match ip address in server certificate.

The default certificate function (SSL_AuthCertificate) fails to match DNS names
which resemble ip addresses. It then calls our badCertHandler callback. There
we get the chance to manually match the ip address.

Add description of @cast operator to beginners guide.

PR12115: New support for multiple remote targets

* session.cxx (systemtap_session::clone): Returns a systemtap_session
  for a requested arch/release.  If one such was already created, go
  ahead and use it.
* remote.h (run -> start, finish): Separate the launch from the wait.
* remote.cxx (direct, ssh_remote): Implement separated start and finish.
  (ssh_remote::ssh_remote): Use systemtap_session::clone to get a
  session matching the remote target.
* main.cxx (pass_5): Take a vector of targets to run in parallel.
  (cleanup): Perform cleanup for subsessions too.
  (main): Loop over all specified remote_uris.

PR12115: Prepare utils for multiple inferiors

We'll need to juggle multiple child process if we are to run multiple
remote targets in parallel.

* util.cxx (spawned_pid -> spawned_pids): No longer an only child,
  hopefully it won't act so spoiled anymore.
  (wait_for_spawn -> stap_waitpid): Now wait for a specific pid. Clear
  it from the set when dead (but not forgotten).
  (stap_spawn): Like stap_system, but return the pid and don't wait up.
  (stap_system): Call stap_spawn, then wait for it.
  (kill_stap_spawn): Must be fair, kill them all.

PR12115: Give remote-ssh a tty, and quiet status messages

Having a tty lets ssh pass signals through to the remote process.

* remote.cxx (ssh_remote): Use -t with ssh commands, and -q with ssh and
  scp commands.

PR12115: Early support for remote execution

This corresponds to the second draft posted on bugzilla.

* Makefile.am: Add remote.cxx to the build.
* Makefile.in: Regenerate.
* buildrun.cxx (run_pass -> make_run_command): Rename, and don't
  actually run the thing yet, so we can control *where* we run it.
* main.cxx (pass_5): Take a target where we want to run.
  (main): Check if we have a remote uri, and go with it.
* remote.cxx: New, define classes for local and ssh execution.
* remote.h: New, define the generic remote class.
* session.cxx (systemtap_session::systemtap_session): Move the logic to
  normalize machine architecture names to util.cxx.
  (systemtap_session::parse_cmdline): Add --remote.
* session.h (systemtap_session::remote_uris): New.
* util.cxx (normalize_machine): Moved from session.cxx.
  (wait_for_spawn): Refactor common code.
  (stap_system): Use wait_for_spawn.
  (stap_system_read): New, reads stdout from subprocess.
  Like popen, but with c++ streams and a signal-managed pid.

Split user_int64 in two on i386

* tapset/uconversions.stp (user_int64): Since __stp_get_user can't do
  64-bit reads on i386, we must read the value in two parts.
  (user_uint64): Make this just a wrapper for user_int64.

Fix user memory access on s390x.

* tapset/uconversions.stp (__STP_GET_USER): Switch back to using
  __stp_get_user() instead of kread() to read user memory.

Improve 386 precision op handling.

* tapsets.cxx (sdt_uprobe_var_expanding_visitor): Add probe_type.
(visit_target_symbol): Make default precision signed, unless asm probe.

* sdt_misc.exp (sdt_types):  Allow v1/v2 i386 unsigned test xfail

* sdt_types.c (unsigned_short_int_var, unsigned_int_var): Simplify

* sdt_types.stp: Likewise.

Add information about user-space stack backtrace to Beginners Guide.

Fix RHEL5 (older gcc) compile problem.

* tapsets.cxx (visit_target_symbol): Fix RHEL5 (older gcc) compile
  problem by changing '__SIZEOF_LONG__' to 'sizeof(long)'.

Fixed PR 12314 by correctly including linux/inotify.h.

* tapset/aux_syscalls.stp: Look for CONFIG_INOTIFY_USER (along with
  CONFIG_INOTIFY) when including linux/inotify.h.

Don't use /tmp as an arbitrary argument for -I or -R when testing.

It could contain .stp files which affect the test outcome.

Compare sdt probe args as is.

sdt_types.stp:  Don't mask negative probe args

Fix sdt probe arg qi/si fetching on 386

* tapsets.cxx (sdt_uprobe_var_expanding_visitor::visit_target_symbol):
Use long for default precision if "N@" is missing.

* sdt_misc.exp (pbtype_flags):  Don't use -g for STAP_SDT_V{2,3}
(V3 asm test): Fix arch detection

* sdt_types.stp: Mask negative arg values.

Update copyright information in Tapset Reference Manual.

Edit tapset documentation comments

Made the following changes to the documentation comments:
   - Remove the "General Syntax:" lines.
   - Remove trailing '.' for the summaries.
   - Make parameter information phrases rather than sentences

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

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

Bug 12099 - resolve script interpreters

testcase for 12099 - test script interpreter resolution

Edit the tapset documentation for conversions.stp

Made the following changes to the documentation comments:
  - Remove the "General Syntax:" lines.
  - Remove trailing '.' for the summaries.

Edit the tapset documentation for conversions-guru.stp

Made the following changes to the documentation comments:
  - Remove the "General Syntax:" lines.
  - Remove trailing '.' for the summaries.

Edit the tapset documentation for context-unwind.stp

Made the following changes to the documentation comments:
  - Remove the "General Syntax:" lines.
  - Remove trailing '.' for the summaries.

Edit the tapset documentation for context-symbols.stp

Made the following changes to the documentation comments:
  - Remove the "General Syntax:" lines.
  - Remove trailing '.' for the summaries and argumentation information.
  - Fix a few typos.

Edit the tapset documentation for context.stp

Made the following changes to the documentation comments:
  - Remove the "General Syntax:" lines.
  - Remove trailing '.' for the summaries.
  - Add title and description for cpuid function

Remove the "General Syntax:" lines from ansi.stp

Add the queue_stat.stp tapset to the Tapset reference manual.

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

Add the *mib.stp tapsets to the SystemTap Tapset reference manual

Went through the comments in ipmib.stp, linuxmib.stp, and tcpmib.stp to
make them format properly. Then added a chapter in the SystemTap Tapset
reference manual for them.

task_backtrace should return addresses separated by spaces.

* tapset/context-unwind.stp (task_backtrace): Call _stp_stack_snprint_tsk
  with _STP_SYM_NONE.

Add stap_staticmarkers.stp to SystemTap tapset reference manual.

Increase environment limits to allow Tapset Reference manual to build.

Add the scheduler.stp to the Tapset Reference manual.

Add the indent.stp to the Tapset reference manual.

Minor edits to ucontext.stp documentation comment.

Add irq.stp to Tapset Reference manual

Add ucontext.stp to the tapset reference manual

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

Add conversions-guru.stp to tapset manual

Fixed 11794 by replacing returnstr() calls with return_str() when possible.

* tapset/errno.stp (returnstr): Updated documentation comment to note that
  this function should only be used in dwarfless probes.  Renamed
  'returnp' argument to 'format'.
  (return_str): Added documentation comment.
* tapset/aux_syscalls.stp: Replaced returnstr() function calls with
  return_str() function calls.
* tapset/i386/syscalls.stp: Ditto.
* tapset/ia64/syscalls.stp: Ditto.
* tapset/powerpc/syscalls.stp: Ditto.
* tapset/rpc.stp: Ditto.
* tapset/s390/syscalls.stp: Ditto.
* tapset/signal.stp: Ditto.
* tapset/syscalls.stp: Ditto.
* tapset/syscalls2.stp: Ditto.
* tapset/x86_64/syscalls.stp: Ditto.

Add errno.stp to the tapset documentation

Fix 'retstr' in sunrpc.clnt.shutdown_client.return.

* tapset/rpc.stp: In newer kernels, rpc_shutdown_client() has been changed
  to a void function.  Handle 'retstr' properly.

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

Support x86 1 byte and 2 byte registers.

* tapsets.cxx (sdt_uprobe_var_expanding_visitor): Add all register variants.
(get_register_width): New.
(visit_target_symbol): Use it for register case to extract register part.

* sdt_misc.exp: Add sdt asm test.
* sdt_asm.S: New.
* sdt_asm.stp: New.

Audit possible string buffer overruns.

* runtime/transport/procfs.c (_stp_register_ctl_channel_fs): Change
  sprintf() call to snprintf() call.
  (_stp_unregister_ctl_channel_fs): Ditto.
* runtime/transport/ring_buffer.c (_stp_transport_data_fs_init): Ditto.
* runtime/unwind.c (_stp_eh_enc_name): Ditto.
* runtime/io.c: Make sure STP_LOG_BUF_LEN isn't too short.
* runtime/staprun/staprun_funcs.c: Add comments about strcpy/strcat/sprintf
  appropriateness.
  (check_signature): Fix string length calculation.
* runtime/staprun/common.c: Add comments about strcpy/strcat/sprintf
  appropriateness.
* runtime/staprun/modverify.c: Ditto.
* runtime/staprun/relay.c: Ditto.
* runtime/staprun/relay_old.c: Ditto.
* runtime/staprun/staprun_funcs.c: Ditto.

Redistribute the Target Variable information

Redistribute the information about target variables in the manual. Also
added index entries for the information and edited the text.

Fixed possible string buffer overruns in several embedded-C functions.

* tapset/socket.stp (sock_flags_num2str): Use strlcat() instead of
  strcat() to guard against buffer overruns.  Instead of creating the
  string in a temporary variable and then copying it to THIS->__retvalue,
  just create the string in THIS->__retvalue.
  (msg_flags_num2str): Ditto.
* tapset/inet_sock.stp (daddr_to_string): Use snprintf() instead of
  sprintf() to guard against buffer overruns.
* tapset/signal.stp (sigset_mask_str): Ditto.
* tapset/s390/nd_syscalls.stp (get_32mmap_args): Use strlcat()/snprintf()
  instead of strcat()/sprintf() to guard against buffer overruns.
* tapset/s390/syscalls.stp: Removed unused function get_32mmap_args().
* tapset/aux_syscalls.stp: Minor string-related fixes.
* tapset/string.stp: Ditto.