option to display graph times relative to the start time

This avoids having humongous numbers displayed on the graphs' x
axis. Also, the dialog for adding a data set to a graph was cleaned
up.

* grapher/graph-dialog.glade: Add check box for display of relative
  time and a label for the data set list. Force the list of data sets
  to be larger.
* grapher/Graph.hxx (_timeBase): new member
* grapher/Graph.cxx (draw): Subtract _timeBase from displayed time
  value.
* grapher/GraphWidget.hxx (DataModelColumns): Add a column for a data
  set's title, which is optional at the moment.
* grapher/GraphWidget.hxx (_globalTimeBase, _timeBaseInitialized,
  _relativeTimesButton, _displayRelativeTimes): new members
* grapher/GraphWidget.hxx (GraphWidget): Hook up check button for
  displaying relative time.
  (on_expose_event): Determine base time if needed; set base time in
  graphs.
  (onDataDialogOpen): Insert graph data set's name (key) and title
  into the list of data sets.

Add a missing cairo context restore()

It's not clear if this was causing any ill effects, but it is
certainly bad form.

* grapher/GraphWidget.cxx (on_expose_event): add missing cr->restore().

Add new graph output to the last graph

restore event square color

* grapher/GraphStyle.cxx (GraphStyleEvent::draw): set event color in
  inner loop.

hover text on the additional graphs

* grapher/Graph.cxx (window2GraphCoords): Take the entire graph's
  position into account too.
* grapher/GraphWidget.cxx (GraphWidget): Hook up the data dialog's OK
  button.
  (onDataDialogOpen): If a data set doesn't have a title, use it's
  name (key) instead.

tweak multiline hover text to have proper interline spacing

* grapher/CairoWidget.cxx (CairoTextBox::draw): Use font information
  to caculate legible spacing. Also change the font to something more
  readable.
* testsuite/systemtap.examples/general/grapher.stp: Put carriage
  returns in the right spots.

support multiline data output from scripts run under the grapher

This is accompanied by support for multiline output in hover text.

* grapher/StapParser.cxx (ioCallback): Read data 'til the end of line
  character, not just '\n'. Be careful to use I/O functions that don't
  treat '\n' specially.
* grapher/StapParser.hxx: ditto
* grapher/CairoWidget.cxx (CairoTextBox::draw): Perform line breaks
  for hover text.
* testsuite/systemtap.examples/general/grapher.stp: Do multiline
  output of keyboard events. Also, fix longstanding breaking in the
  pty probe.

Merge commit 'origin/master'

Error check in hover text code for empty graphs

* grapher/GraphStyle.cxx(GraphStyleEvent::dataIndexAtPoint,
  GraphStyleBar::dataIndexAtPoint): check for empty graph data

PR11020: Check switch_file global flag inside inner loop for busy scripts

* runtime/staprun/relay.c (reader_thread): Check switch_file thread flags
  inside inner read-write loop. And clear the flags after switching file.

Hover text for event-style graphs

Don't draw the data value for an event; only display it in hover text.

* grapher/Graph.hxx (window2GraphCoords): new function
* grapher/Graph.cxx (window2GraphCoords): ditto
* grapher/GraphStyle.cxx (GraphStyleEvent::dataIndexAtPoint): new
  function implementation.

Fix PR 11034 by directly allocating per-cpu context data.
* translate.cxx (emit_common_header): Change type of 'contexts' to an
  array of struct context pointers.
  (emit_module_init): Allocate a context structure for each possible cpu.
  Free each if an error occurs.
  (emit_module_exit): Update contexts reference.  Free each possible cpu's
  context structure.
* tapsets.cxx (common_probe_entryfn_prologue): Use array instead of percpu
  data for context structure.

Make the hover text conform to data displayed.

Start of code to be more selective about the association between the
hover text and the underling graph. Also, show the data set name in hover text.

* grapher/GraphStyle.hxx (dataIndexAtPoint): New virtual function.
* grapher/GraphStyle.cxx (dataIndexAtPoint): Implementation for bar graphs
* grapher/GraphWidget.cxx (onHoverTimeout): Use dataIndexAtPoint.

Refactor drawing of different styles of graph into classes

* grapher/GraphStyle.cxx: New file
* grapher/GraphStyle.hxx: New file
* grapher/Graph(draw): Move much drawing code to GraphStyle::draw

Account for zoom factor when choosing hover text

* grapher/Graph.cxx (getTimeAtPoint): Divide by _zoomFactor too.

Make tapset and tests compatible with old kernel

* tapset/ioscheduler.stp : Make probe optional.
* tapset/irq.stp : Ditto.
* tapset/scheduler.stp : Ditto.
* tapset/tcpmib.stp : Ditto.
* tapset/linuxmib.stp : Switch variable by version.
* tapset/tty.stp : Ditto.
* testsuite/buildok/irq.stp : Make probe test optional.
* testsuite/buildok/tty.stp : Ditto.
* testsuite/systemtap.examples/network/netdev.stp : Ditto.
* testsuite/buildok/tcpmib-all-probes.stp : Add embedded functions.

minor correction

correcting license, forgot to change back yesterday while testing default Publican 1 license

Regenerate autotools files using autoreconf to pick up latest Makefile.am changes.

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

Make sure system types are defined.

uprobes ppc64: UPROBES_DEBUG build fix

* runtime/uprobes/uprobes_ppc.c (calc_offset): Fix printk types.

Use proper types for do_div

do_div accepts unsigned 64-bit integer type for dividend, signed types
would cause do_div's typecheck fail:

stat-common.c: In function 'needed_space':
stat-common.c:50: error: comparison of distinct pointer types lacks a cast
...same errors in time.c and tapset-timers.cxx's generated code...

A fix for time.c is special, on ppc32 cycles_t is 32-bit, so technically
we don't need do_div, but since the whole _stp_gettimeofday_ns() operates
on 64-bit types we'd better be safe and use uint64_t for the math.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

Change KERNEL_RELOC_SYMBOL to "_stext" on ppc32

Currently staprun fails at send_relocation_kernel(), this is because
ppc32 doesn't have ".__start" symbol. I think we should use _stext,
just as other arches.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

Implement _div64 and _mod64 for ppc32

_div64 and _mod64 arch-specific details were taken from gcc/longlong.h,
just as it was done for i386.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

Share ppc64 and ppc32 code where possible

* runtime/copy.c: Can use ppc64's code.
* runtime/regs.h: Ditto.
* runtime/string.h: Ditto.
* tapset/context.stp: Ditto.
* tapset/errno.stp: Ditto.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

Implement ppc32 variant of __is_user_regs

* tapset/nd_syscalls.stp: Implement ppc32 variant of __is_user_regs.
* tapset/syscalls.stp: Ditto.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

stack-ppc: Adjust for ppc32

Get rid of magic numbers, use appropriate defines from ptrace.h.

From now on stack-ppc.c should be suitable for ppc32.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

Rename stack-ppc64.c to stack-ppc.c

There should be no functional changes in this patch.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

Rename uprobes_ppc64.c to uprobes_ppc.c, use it for ppc32

The code *looks* generic enough, so I think it can be used for ppc32
without modifications.

Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

Convert string.stp documentation to standard markup.

* doc/SystemTap_Tapset_Reference/tapsets.tmpl: Add string.stp chapter.
* tapset/string.stp: Convert documentation to standard markup and add
  documentation that was missing.

some aesthetic tweaks to the grapher

* grapher/Graph.cxx (draw): Draw tick labels (times) below the graph.
* grapher/grapher.cxx (main): Open main window with a size big enough
  to display an entire graph.

Add hover text to the graph.

When the graph display is paused, leaving the mouse stationary over
the graph will display the data point under the pointer.

* grapher/CairoWidget.hxx (CairoTextBox): new class
  (CairoWidget, CairoPlayButton): refector some play button-specific
  things from CairoWidget to CairoPlayButton.
* grapher/CairoWidget.cxx (CairoTextBox::draw): new function.
* grapher/GraphWidget.hxx (GraphWidget): new members for supporting
  hover text.
* grapher/GraphWidget.cxx (on_motion_notify_event): Set up hover text box.
  (establishHoverTimeout, onHoverTimeout, getGraphUnderPoint): new functions.

change time type from double to int64_t

* grapher/Graph.hxx (Graph): Change variables holding the time limits
  of the displayed graph from double to int64_t.
* grapher/Graph.cxx (Graph::draw): Do calculations of time differences
  using int64_t.
  (Graph::getExtents, Graph::setExtents): Change left and right
  arguments to int64_t.
* grapher/GraphData.hxx (GraphDataBase): Change time type to int64_t.
  (GraphDataBase::elementAsString): New function.
  (GraphData::elementAsString): Implementation of that function.
* grapher/StapParser.cxx (parseData): Parse time values from the stap
  script as 64 bit values.

Fix typo in scheduler tapset

* tapset/scheduler.stp (__scheduler.kthread_stop.kp): Change
  priority to prio.

minor modifications to allow book to build in latest Publican (1.x). to build in older versions of Publican, use old-syncandbuild.sh

minor modifications to allow book to build in latest Publican (1.x). to build in older versions of Publican, use old-syncandbuild.sh

minor modifications to allow book to build in latest Publican (1.x). to build in older versions of Publican, use old-syncandbuild.sh

minor modifications to allow book to build in latest Publican (1.x). to build in older versions of Publican, use old-syncandbuild.sh

minor modifications to allow book to build in latest Publican (1.x). to build in older versions of Publican, use old-syncandbuild.sh

PR10653: don't corrupt signal structs in itrace on rawhide utrace

* runtime/itrace.c (usr_itrace_report_signal): On modern utrace,
  must check/propagate utrace_signal_action().

Don't let _stp_alloc_percpu allocate too much memory.
* runtime/alloc.c (_stp_alloc_percpu): Refuses to allocate too much percpu
  memory.

PR10901: Add -D_FILE_OFFSET_BITS=64 to Makefile.am for stap and staprun

Fix build error when !HAVE_NSS

* runtime/staprun/staprun_funcs.c (assert_stap_module_permissions): Mark
  parameters unused.

docs: abbreviated stap-server news blurbs

Update NEWS and initscript/README.stap-server with details of changes
due to recent commits.

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

PR 10984 Additional Work. TOCTOU race checking access permissions before canonicalizing /lib/modules/KVER/systemtap.

Fix compilation warning of passing NULL to int parameter

* util.cxx (in_group_id): Replace NULL with 0.

Fix compilation error, use getgroups without fixed limits.

* util.cxx (in_group_id): Don't use NGROUPS_MAX.

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

Improve error messages related to module signing.

- Allow root, the owner of the uprobes build directory and the members of the
group owner of the uprobes buld directory to build uprobes.ko.

- When building uprobes.ko, make all generated files writable by the group
owner of the uprobes build directory.

- Don't change the group owner of the uprobes build directory during
'make install'

include modverify.h unconditionally.

Fix uninitialized fields in loc2c.

* loc2c.c (alloc_location, location_from_address, c_translate_constant):
Initialize ops and nops fields.

Add assert for detecting shared library reloading, PR11015.

* runtime/sym.c (_stp_tf_mmap_cb): Only look for first load of whole module
  that is executable. Add check for detecting reloading of module.

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

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

PR 10976, 10984. Loading of signed modules (script module and uprobes) limited to members of stapusr.

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

Conflicts:

configure

Ensure uprobes/Makefile never returns failure due to setting permissions.
Ensure uprobes install directory has group 'stap-server'.
Rename igid_in to in_group.

Updated.

Fixed tapset reference manual generation.
* doc/SystemTap_Tapset_Reference/Makefile.am: Check for file existence
  before doing compare.
* doc/SystemTap_Tapset_Reference/Makefile.in: Regenerated.
* tapset/ioblock.stp: Fixed comment so that docs can be built.

Rework setting of group and permissions on generated files and directories.

Correct tapset/ioscheduler.stp so documentation builds.

Make built files and directories writeable by the stap-server group individually.

Tweak cu-decl test

* testsuite/systemtap.base/cu-decl.exp: Check utrace.
* testsuite/lib/stap_compile.exp: Make catch after wait.

Correct block IO and IO scheduler tapset and test case

* tapset/ioblock.stp: Update comment and variables.
* testsuite/buildok/ioblock_test.stp: Add parameters.
* testsuite/buildok/ioscheduler.stp: Remove redundant parameters.

Added tracepoint-based probes to block IO and IO scheduler tapsets.
Added testcases for these probes.
Also, modified comments in ioblock.stp so that tapset docmentation can be automatically generated for this tapset.

Only attempt to change the group permissions of built items if we are root.

Allow members of the group stap-server to build the uprobes module.

Ensure that server status files are written by the initscript, not stap-start-server.

Report correct status when a server fails to start.

Fix typo on command to create /usr/share/systemtap/runtime/uprobes.

PR 10905 - Ensure that /usr/share/systemtap/runtime/uprobes is writeable by the
stap-server group.

cleanup: explain -5 magic value used in map-sorting code

build: disable make-silent mode in rpm builds

PR10983: Give preference to tracepoints in trace/events/

In 2.6.32-rc7, there are two power.h tracepoints headers, and only the
one in trace/events/ is valid.  In general, we can expect that
trace/events/ has newer headers, so we should search those first.

* tapsets.cxx (tracepoint_builder::init_dw): Search /events/ first.

PR 10981.  Fixed buildok/netdev.stp for RHEL5.
* tapset/networking.stp: Made 'netdev.change_rx_flag' and
  'netdev.get_stats' optional, since those functions don't exist on
  RHEL5.
* testsuite/buildok/netdev.stp: Ditto.

Don't kfail buildok/sched_test.stp and buildok/process_test.stp.
* testsuite/systemtap.pass1-4/buildok.exp: On rhel5 and rawhide, the
  buildok/sched_test.stp and buildok/process_test.stp tests pass
  sucessfully, so there is no reason to kfail them.

PR 5150.  Fixed nfs tapset by making probes optional.
* tapset/nfs_proc.stp: Made 'nfs.proc.read', 'nfs.proc.write', and
  'nfs.proc.commit' optional for newer kernels without those functions.
* testsuite/systemtap.pass1-4/buildok.exp: Expect nfs-all-probes.stp to
  pass.

Check in regenerated systemtap example indexes.

Clean up examples

Test cross-CU type discovery

Check that we can dereference a type declaration that is
defined in a separate CU from the function.

Make interrupts-by-dev.stp executable

PR4037: 32-bit staprun vs. 64-bit kernels just works (tm)

* configure.ac (PROCFLAGS): Don't define it.
* Makefile.am: Don't use it.

Document is_return(), module_name() and stp_pid() context tapset functions.

With tapset/aux_syscalls.stp fixes, test twentyseven.stp is not kfail.
* testsuite/systemtap.pass1-4/buildok.exp: Test twentyseven.stp should
  pass now.

Search other CUs of the module when resolving declarations.

* dwflpp.h (declaration_resolve_other_cus): New method.
  (iterate_over_globals): Mark as static and takes a CU to iterate over.
  (global_alias_caching_callback_cus): New method.
* dwflpp.cxx (global_alias_caching_callback_cus): New method.
  (declaration_resolve_other_cus): New method.
  (declaration_resolve): Call iterate_over_globals() with current cu.
  Call declaration_resolve_other_cus() when name not found.
  (iterate_over_globals): Takes cu_die to iterate over as argument.

Fixed aux_syscalls.stp for RHEL4.
* tapset/aux_syscalls.stp: Only define entries for IRQF_* flags if they
  exist.

PR 10974 fix.  Fixed aux_syscalls.stp for RHEL5.
* tapset/aux_syscalls.stp: Only define entries for
  IRQF_NOBALANCING/IRQF_IRQPOLL if they exist.

Use DW_AT_MIPS_linkage_name when available in vardie_from_symtable.

If there is a DW_AT_MIPS_linkage_name it encodes the actual name of
the variable as used in the symbol table.

* dwflpp.cpp (vardie_from_symtable): Check whether there is a
  DW_AT_MIPS_linkage_name attribute and use its value if so.

Remove caching of emit_address for kernel modules and shared libraries.

Caching of the address is only safe for kernel addresses that can never
change. For kernel module or dynamic shared library addresses it isn't
safe to cache the address since they can be unloaded, reloaded or mapped
differently in separate executables.

* dwflpp.cxx (emit_address): Remove static from addr definition for
  kernel and dynamic modules.

Explain "cheat" comment in sym.c (_stp_tf_mmap_cb).

We are abusing the "first" section address here to indicate where the
module (actually first segment) is loaded (which is why we are ignoring
the offset). It would be good to redesign the stp_module/stp_section
data structures to better align with the actual memory mappings we are
interested in (especially the "section" naming is slightly confusing
since what we really seem to mean are elf segments (which can contain
multiple elf sections).

* runtime/sym.c (_stp_tf_mmap_cb): Add cheat comment.

PR 5150 partial fixes.  Added support for nfs_file_fsync.
* tapset/nfs.stp(nfs.fop.fsync): Added support for nfs_file_fsync.
  (nfs.fop.fsync.return): Ditto.
  (nfs.aop.readpages.return): Fixed 'size' bug.
  (nfs.aop.write_begin): Renamed to make 'nfs.*.*' probes work correctly.
  (nfs.aop.write_end): Ditto.
  (__nfs.aop.write_begin): Ditto.
  (__nfs.aop.write_end): Ditto.
  (__nfs.aop.prepare_write): Ditto.
  (__nfs.aop.commit_write): Ditto.
  (nfs.fop.aio_read.return): Always sets 'units'.
  (nfs.fop.aio_write.return): Ditto.
  (nfs.fop.sendfile.return): Ditto.

Add testcase for retrieving $global vars from execs and shared libs.

Explicit testcase for PR10010 and PR10622.

* testsuite/systemtap.base/externalvar.c: New file.
* testsuite/systemtap.base/externalvar.exp: New file.
* testsuite/systemtap.base/externalvar.stp: New file.
* testsuite/systemtap.base/externalvar_lib.c: New file.

PR10010 Support $globals in shared libraries.

* dwflpp.cxx (dwflpp::emit_address): Enable task finder and emit a
  _stp_module_relocate for the ".dynamic" section when seeing a
  user-space dso address.
* runtime/sym.c (_stp_mod_sec_lookup): Remove .dynamic section addr cheat.
  (_stp_tf_mmap_cb): Add cheat here.

Allow modules to trigger task_finder vma_tracker.

* runtime/sym.h: Define _stp_need_vma_tracker.
* translate.cxx (emit_symbol_data_done): Output _stp_need_vma_tracker value.
  (c_unparser::emit_module_init): If STP_NEED_VMA_TRACKER isn't defined
  check _stp_need_vma_tracker to call _stp_sym_init().

PR 5150 partial fix.  Support nfs_write_begin()/nfs_write_end().
* tapset/nfs.stp: Added support for nfs_write_begin()/nfs_write_end(),
  which replaced nfs_prepare_write()/nfs_commit_write().

Add documentation for ansi.stp tapset.

* tapset/ansi.stp: Add documentation for every function.
* doc/SystemTap_Tapset_Reference/tapsets.tmpl: Add chapter on ansi.