Get PATH correct for fidning libexec tools for installcheck.

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

Look for stap-start-server and stap-stop-server in libexec.

Typos in install and rpm spec.

o add / suffix to stap_pkglibexecdir
o typo: %{name%} -> %{name}

PR10601: cleanup for i386, x86-64

* runtime/loc2c-runtime.h (usr_i386): Fix si/di ordering.  Add ip.
  (usr_x86_64): Add rip.
  (u_*_register): Use kernel standard ARRAY_SIZE() instead of S().

Merge remote branch 'origin/master'

Install %{libexecdir}/%{name}/stapio explicitely.

Don't install the entire %{libexecdir}/%{name} directory.

PR 10247, 10276, 10905: Move remaining client/server tools to libexec.

PR11015 Support shared library reloading (in different processes)

* runtime/task_finder_vma.c (stap_remove_vma_map_info): Return negative
  status on failure.
  (stap_find_vma_map_info): Likewise.
  (stap_find_vma_map_info_user): New function.
  (stap_drop_vma_maps): New function.
* runtime/sym.h (addr): Renamed to static_addr, to store addresses for
  sections which are always mapped at the same address.
  (_stp_module_relocate): Add extra struct task_struct *tsk argument.
* runtime/sym.c (_stp_tf_exec_cb): New callback, calls stap_drop_vma_maps.
  (_stp_tf_mmap_cb): Don't store address in module.section, but call
  stap_add_vma_map_info() per tsk->group_leader for matched module.
  Don't register empty/null modules.
  (_stp_module_relocate): Take extra struct task_struct *tsk argument,
  cache last tsk used. Only use section->static_addr for none dynamic
  modules. Use stap_find_vma_map_info_user() to locate dynamic modules.
  (_stp_mod_sec_lookup): Add extra argument unsigned long *rel_addr to
  optionally store relative address when module/section found.
  (_stp_kallsyms_lookup): Use _stp_mod_sec_lookup to find relative address.
  (_stp_sym_init): Register _stp_tf_exec_cb in stap_task_finder_target.
  Add error check to see if task finder could be initialized.
* dwflpp.cxx (emit_address): Pass NULL for kernel/modules and current for
  user tasks to _stp_module_relocate.
* runtime/transport/symbols.c (_stp_do_relocation): Set new static_addr
  _stp_section field.
* runtime/unwind.c (adjustStartLoc): Take new struct task_struct *tsk
  argument and pass to stap_find_vma_map_info_user and _stp_module_relocate
  to find adjusted addr.
  (_stp_search_unwind_hdr): Pass through struct task_struct *tsk.
  (unwind_frame): Likewise.
* tapset/context-symbols.stp (probemod): Add NULL to _stp_mod_sec_lookup
  call to indicate we aren't interested in relative address.
* tapsets.cxx (dwarf_derived_probe_group::emit_module_init): Pass NULL to
  _stp_module_relocate to indicate kernel/module address.

Tracepoint based tapset for memory subsystem.

conditionalize pty_write count variable on kernel version

* testsuite/systemtap.examples/general/grapher.stp: Conditionalize
  pty_write count variable on kernel version.

change namespace indentation to 0

A silly change, but this works better with other tools, like git
diff. I should have started the C++ files this way.

brown paper bag typo fixy

runtime: more build fix for non-utrace kernels

* runtime/stack-i386.c (_stp_stack_print): Make call to uprobe_get_pc()
  conditional on CONFIG_UTRACE.
* runtime/stack-x86_64.c: Ditto.

PR10601: unfork deref()

* runtime/loc2c-runtime.h: Remove k_ vs u_[store_]deref; share instead.
* tapsets.cxx: Remove k_ vs u_ redirection for *deref().

documentation: remove docs for nonexistentn perfctr probes

runtime: rhel4 build fix for uretprobe-wannabe stack tracebacks

* runtime/stack.c (_stp_stack_print): Comment out *retprobe logic if !CONFIG_UTRACE.

RHBZ543529: Make implicit -BCONFIG_MODULE_SIG=n

* buildrun.cxx (compile_pass): Disable module signing goo.

dwarf probes: skip inlined instances with null (buggy) entrypc

* dwflpp.cxx (resolve_prologue_endings): Skip entrypc=0.

runtime backtracing: port commit #2e7f8442 to uprobes1

    * runtime/uprobes/uprobes.c (uprobe_get_pc): new function

PR10601 cont'd, RHEL5 backward compatibility with more runtime/autoconf*

PR10601 part 1: i386 and x86-64 regset for dwarf fetch/store_register()s

* runtime/loc2c-runtime.h (fetch_register, store_register):
  forked into k_ (kernel) and u_ (user) varieties.  Implement
  i386 and x86_64 in terms of regset.h; fall back to k_* for
  other architectures.
* tapsets.cxx: (*::visit_target_symbol): Emit macros to map
  loc2c's fetch/store_register to loc2c-runtime's k_ or u_ as
  appopriate.

PR10601: comment on loc2c's use of [u]intptr_t for temp values

Recalculate sem address when tid changes.

* tapsets.cxx (uprobe_derived_probe_group::emit_module_decls): Emit
sdt_sem_tid.  Use it to recalculate sem address when task changes.

Remove externalvar.exp .exe and .so after test done.

Some tools (temporarily) still in $bindir.

s/pkglibexecdir/libexecdir/systemtap/

PR 10247 10276 10905: Infrastructure for moving underlying client/server tools to libexec.

Merge commit 'origin/master'

Remove uprobes.h declaration from runtime.h

Turns out that it breaks on kernels that don't have utrace.

* runtime/runtime.h : Don't include uprobes.h
* runtime/stack.c: Include uprobes.h
* runtime/stack-i386.c: Check if uprobes is included at all.
* runtime/stack-x86_64.c: ditto
* tapsets.cxx (uprobe_derived_probe_group::emit_module_decls): put
  uprobes.h include back in.

PR 10641: Do not allow -m when --unprivileged in the server.

Fix typos in NEWS

Remove blank line in ucontext-unwind.stp for SystemTap Tapset Reference Manual

The automatic documentation extraction doesn't allow blank lines between the
the comment and the actual code. Removed the problem blank line to allow
the information to be extracted for the SystemTap Tapset Reference Manual.

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

Add PROD2 to allow beginner's guide to be built again.

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

Add function to generate saner html file names for langref.

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

Correct path to trusted signer's database.

PR 10889: Reorganize client/server man pages. Document --unprivileged.

Merge commit 'origin/master'

support for a brief backtrace format

This only prints symbol+offset, or an address if the symbol isn't
known.

* runtime/runtime.h (SYM_VERBOSE_NO, SYM_VERBOSE_FULL,
  SYM_VERBOSE_BRIEF): new constants
* runtime/stack.c (_stp_stack_print): support brief format
* runtime/sym.c (_stp_func_print): ditto
* tapset/ucontext-unwind.stp (print_ubacktrace_brief): new function
* testsuite/systemtap.context/fib.c: new test program
* testsuite/systemtap.context/fib.stp: new test
* testsuite/systemtap.context/fib.exp: new test

backtrace through uprobes trampoline.

Only works in uretprobes for the moment.

* runtime/stack-x86_64.c (__stp_stack_print): Rewrite trampoline PC
  addresses if necessary.
* runtime/stack-i386.c (__stp_stack_print): ditto
* runtime/stack-arm.c (__stp_stack_print): Add extra argument
* runtime/stack-ppc.c (__stp_stack_print): ditto
* runtime/stack-s390.c (__stp_stack_print): ditto
* runtime/stack.c (_stap_stack_print): call __stp_stack_print with
  uretprobe_instance.
* testsuite/systemtap.context/uprobe_backtrace.stp: new test

PR 11097 fixed by correcting MAXNESTING test.

* translate.cxx (emit_function): Fixed MAXNESTING test to not allow too
  deep recursion.

function to translate from the uretprobe trampoline back to the original
return address

* runtime/uprobes2/uprobes.c (uprobe_get_pc): new function

set the IP in return probes to the returned-to instruction

It's easily available in kretprobes and uretprobes and is consistent
with the rest of the program state.

* translate.cxx (emit_common_header) : add uretprobe_instance to context.
* tapsets.cxx (common_probe_entryfn_prologue): Initialize ri in
  context to 0.
  (dwarf_derived_probe_group::emit_module_decls): Change IP to return
  address in kretprobes.
  (uprobe_derived_probe_group::emit_module_decls): enter_uretprobe_probe:
  set ri (uretprobe_instance) in context. Change IP to return
  address in uretprobes. Don't emit uprobe include and #define
* runtime/runtime.h : Add includes and #define for uprobes.
* runtime/stack.c (_stp_stack_print, _stp_stack_snprint): Add extra
  argument for uretprobe_instance.
* tapset/context-unwind.stp (print_backtrace, backtrace): Pass NULL
  for uretprobe_instance to _stp_stack_print.
* tapset/ucontext-unwind.stp (print_ubacktrace, ubacktrace): pass
  uretprobe_instance to _stp_stack_print
* testsuite/systemtap.context/uprobe_uaddr.exp : new test for uaddr in
  function probes
* testsuite/systemtap.context/uprobe_uaddr.stp : new file

Fix handling of multiple sdt uprobes with the same name.

tapsets.cxx (sdt_query::handle_query_module): Improve trace handling.
 (sdt_query::convert_location): Create new component instead of reusing old.

Remove unused variable.

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

Fixed DEBUG_MEM debug memory tracking.

* runtime/alloc.c (_stp_kmalloc): Fixed compilation when DEBUG_MEM is
  defined.  Improved allocated memory tracking.
  (_stp_kzalloc): Ditto.
  (_stp_vmalloc): Ditto.
  (_stp_alloc_percpu): Ditto.
  (_stp_kmalloc_node): Ditto.

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

Don't automatically authorize new root certificates as trusted signers.

Don't save build-id if it is located before _stext

This probably means that build-id will not be loaded at all and
happens for example with ARM kernel.

See also: http://sources.redhat.com/ml/systemtap/2009-q4/msg00574.html

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

Suppress output from stap-gen-cert when installing the systemtap-server package.

PR 10905: Initscript improvements: Automatically authorize the initscript servers as trusted servers and signers.

Clean up NSS after errors occur during verification.

PR 10905: stap-server initscript improvements

o Handle, -B, -I, -R options
o Allow specification of servers by pid
o Allow specification of servers by nickname.
o Advertise options used using avahi.

Do not use condition_codes() on arm

It does not exist in recent kernels.

Use regs->ARM_cpsr instead, this should work with all versions
of linux found in git repo (versions >=2.6.12-rc2). Difference
between condition_codes() and regs->ARM_cpsr should not matter
for systemtap.

Use env(SYSTEMTAP_TESTAPPS) for all systemtap.apps

mysql.exp: Use env(SYSTEMTAP_TESTAPPS).
 (mysqlrelease): Update and check for download failure.
postgres.exp: Use env(SYSTEMTAP_TESTAPPS).
stap-tcl.stp: Use .library("library").mark("mark").
tcl.exp: Likewise.

runtime build: fix alloc.c buildability on old gcc 3.4

* runtime/alloc.c (_stp_kmalloc): Move var decl back to top.

grapher: implement restarting a stap process

* grapher/StapParser.cxx (StapParser::disconnect): new function
* grapher/StapParser.hxx (StapProcess::StapProcess): initialize argv to 0
* grapher/grapher.cxx (StapLauncher::setArgs): Set argv to 0
  (StapLauncher launch, launchUsingParser): Refactor launch(),
  extracting function a that (re)launches a stap process using an
  existing parser.
  (StapLauncher::onChildDied): call disconnect() on dead parser.
  (GrapherWindow::_graphicalLauncher, setGraphicalLauncher): delete
  member, replacing with...
  (graphicalLauncher): new variable
  (ProcModelColumns): Store parser object in the list model instead of
  just a StapProcess object.
  (ProcWindow::onRestart): new function
  (ProcWindow::refresh): Preserve the list selection when the process
  list is refreshed.
  (ProcWindow::onSelectionChanged): Manage the restart button's state.

runtime: undefine DEBUG_MEM, retain alloc counting

DEBUG_MEM can & does evoke locking timeouts.

* runtime/alloc.c (*): Track __stp_allocated_memory regardless
  of DEBUG_MEM.

runtime: print stp_alloc statistics in startup printk

* runtime/alloc.c: Define DEBUG_MEM.
* runtime/print.c (_stp_print_kernel_info): Reformat/rescale mem values.

Merge commit 'origin/master'

grapher: integrate graph events from stdin with the stap process framework.

This was the original way to do graphing and had bitrotted some.

* grapher/StapParser.cxx (initIO): new catchHUP argument
  (ioCallback): Only disconnect signals, etc. on IN_HUP if catchHUP is
  true.
  (errIoCallback): Write error messages to stderr, not stdout.
* grapher/grapher.cxx (StapLauncher::launch): Don't catchHUP on our
  stap process children.
  (ProcWindow::refresh): Display something reasonable for the stap
  "process" that is feeding stdin.
  (main): Use StapParser::initIO to initialize parser reading from
  stdin.

grapher: more implementation stap process window

The process arguments are displayed, and the "kill" button is
insensitive when  a process is dead or nothing is selected.

* grapher/grapher.cxx (ProcWindow _killButton, _restartButton,
  _stapArgsLabel, _scriptArgsLabel): new members
  (ProcWindow::ProcWindow): hook 'em up
  (ProcWindow::onSelectionChanged): Enable / disable kill button and
  display process arguments.
* grapher/processwindow.glade: Replace bizzare handle window with a
  horizontal pane. Tweak various widget sizes.

grapher: change SIGCHLD handling and exit cleanup

The signal handler now calls waitpid() and writes the pid and status
to its pipe.

* grapher.cxx (ChildInfo): new class
  (handleChild): wait for zombies and write their status down the
  pipe.
  (ChildDeathReader::ioCallback): Read dead child's pid from signal
  handler pipe and emit signal.
  (ChildDeathReader::reap): delete
  (ChildDeathReader::Callback): delete
  (StapLauncher::StapLauncher): Connect to childDied signal.
  (StapLauncher setWinParams, reap, cleanUp): delete
  (onChildDied): new function that updates the parsers list when a
  child dies.
  (GrapherWindow): Remove ChildDeathReader::Callback base class
  (GrapherWindow::onParserListChanged): New function; exits if program
  is quitting and no parsers are left.
  (on_menu_file_quit): Kill all children; don't hide (and exit) unless
  there are no parsers.
  (main): Don't do any cleanup after Gtk loop exits.

Begin to parameterize loc2c for proper DWARF target address size.

* loc2c.c (stack_slot_type): New function.
(translate): Use it in place of STACK_TYPE and UTYPE macros.
(emit_loc_address, emit_loc_value, c_emit_location): Likewise.

Fix regression in statistic operations

In commit 98c783852039061db8c1611742660aaded0eab77 ("Use proper types
for do_div") I imprudently changed some variables to an unsigned type
while in some places the code actually relies on a sign.

So, let's be a bit smarter now and use temporary variables.

Reported-by: Wenji Huang <wenji.huang@oracle.com>
Signed-off-by: Anton Vorontsov <avorontsov@ru.mvista.com>

show the status of stap processes in the process window

Also, the "kill" button now works.

* grapher/StapParser.hxx (_ioConnection, _errIoConnection): new
  members for sigc connections.
  (initIo): New function
  (parsers, parserListChangedSignal): new variables
* grapher/StapParser.cxx (ioCallback): disconnect signals when child
  dies
  (initIo): new function
* grapher/grapher.cxx (StapLauncher): eliminate death callback in
  favor of childDied signal. Use global parsers list
  (ProcWindow::_listSelection): new member
  (ProcWindow::show, hide, onParserListChanged, onSelectionChanged,
  onKill): new functions
  (ProcWindow::ProcWindow): Set up cell renderer for status icon
* grapher/processwindow.glade: labels for display script and stap arguments

PR 10848 partial fix by using systemtap memory functions everywhere.

* runtime/addr-map.c (add_bad_addr_entry): Uses systemtap memory
  allocation/deallocation wrappers.
* runtime/itrace.c (create_itrace_info): Ditto.
  (remove_usr_itrace_info): Ditto.

Handle .probes section big endian 32 bit case.

sdt.h (STAP_PROBE_ADDR): Add 32 bit big endian case.
 (STAP_PROBE_DATA_): Use .balign
tapsets.cxx (sdt_query::get_next_probe): Stop if there is no probe name.

Merge commit 'origin/master'

grapher: start of a dialog for displaying active stap processes

The names of active scripts are displayed in a list; mock buttons
suggest being able to stop and restart them.

* grapher/processwindow.glade: new file
* grapher/Makefile.am: add processwindow.glade to installed files
* grapher/StapParser.hxx (StapProcess): new class
  (StapParser): factor out members that are now in StapProcess
  (ioCallback): Use the new childDied signal instead of aborting the
   whole grapher when a child dies.
* grapher/grapher.cxx (ProcModelColumns, ProcWindow): classes for
  displaying stap process window.
  (GrapherWindow::on_menu_proc_window): new function

refactor list of data sets out of GraphWidget into a global data structure.

Also, add a sigc signal for broadcasting data set changes.

* grapher/GraphData.hxx (GraphDataList) new typedef
  (getGraphData, graphDataSignal): new functions
* grapher/Graph.hxx (DatasetList): remove typedef in favor of
  systemtap::GraphDataList
* grapher/GraphWidget.cxx (GraphWidget, onGraphDataChanged): Use
  graphDataSignal to notice new data sets that need to be added. Use
  the global data set list instead of one embedded in GraphWidget.
* grapher/StapParser.hxx: delete the _widget member; use signals to
  broadcast that there are new data sets instead of calling into the widget.

Add .library("lib").mark("mark") and use it for .mark semaphores.

tapset-utrace.cxx (TOK_LIBRARY): New.
 (utrace_derived_probe::utrace_derived_probe): Add library and has_library.
 (utrace_builder::build): Handle library.
 (utrace_derived_probe_group::emit_probe_decl): Add sdt_sem_offset to
  emitted stap_utrace_probes.  Add stap_task_finder_target
  mmap_callback for handling shared library.  Handle sdt_sem_offset in
  emitted_stp_utrace_probe_cb.  Add stap_utrace_mmap_found.
 (register_tapset_utrace): Handle .library
tapset-utrace.cxx (TOK_LIBRARY): New.
 (base_query::base_query):  Add path and has_library.
 (dwarf_derived_probe::dwarf_derived_probe) Likewise.
 (dwarf_derived_probe::register_patterns):  Handle .library
 (sdt_query::convert_location): Likewise.
 (dwarf_builder::build):  Likewise.
 (uprobe_derived_probe_group::emit_module_decls): Emit
  sdt_sem_address.  Add sdt_sem_offset to emitted stap_uprobe_spec.
  Add offset and vm_flags to signature of stap_uprobe_change_plus, and
  handle sdt_sem_offset.  Allow writeable segments in emitted
  stap_uprobe_mmap_found.
sdt_misc.exp:  Test .library
util.cxx (find_executable):  Add env_path to sig and use it in getenv.
util.h (find_executable): Likewise.  Make "PATH" the default.
dtrace.in (provider): Turn on semaphores.
sdt.h: Likewise.

PR 10905 Ground Work: Save server status as PID files with the server config as contents.

Revert "PR 10641 fixed by checking module name in unprivileged mode."

This reverts commit b2ea60606801aa9bf243f22318ac4bd8a25094fe.
* main.cxx (checkOptions): Remove unprivileged mode module name check.
* testsuite/parseko/cmdline05.stp: Delete unneeded testcase.
* testsuite/parseok/cmdline03.stp: Ditto.

Add dtrace -I support.

dtrace -I is used to pass through include paths to cpp when run.
Thanks to David Malcolm <dmalcolm@redhat.com> for python coding and testing.

* dtrace.in: Add -I support.
* testsuite/systemtap.base/dtrace.exp: Add testcases for -C -I and -G -I.

tiny refactoring of signal and pipe code

PR 10641 fixed by checking module name in unprivileged mode.

* main.cxx (checkOptions): If in unprivileged mode, make sure module name
  starts with 'stap_'.
* testsuite/parseko/cmdline05.stp: New testcase.
* testsuite/parseok/cmdline03.stp: New testcase.

Small compile server shutdown fix.

* testsuite/lib/systemtap.exp (shutdown_server): Only remove the temporary
  stap script if it exists.

add copyright and license to grapher files

make the list items in the data set list unselectable

The checkbox selects the list, so it is just sloppy to allow the item
to be selected.

* grapher/GraphWidget.hxx (no_select_fun): New function; just returns
  false.
* grapher/GraphWidget.cxx (GraphWidget): Connect list store selection
  to function that prevents selection.

Change data set list display to use a check box for inclusion in a graph

Incidently, this supports removing a data set from a graph.

Also, remove the Add, Remove, and Cancel buttons from the data set dialog.

* grapher/GraphWidget.hxx (DataModelColumns::_dataEnabled): new member
  _listConnection: new member
* grapher/GraphWidget.cxx (GraphWidget): Delete connections for
  removed buttons. Connect to dialog show and hide signals instead of
  map. Add "Enabled" column to dialog list.
  (onDataAdd, onDataRemove): delete
  (onDataDialogOpen): Initialize list store with all data sets and
  hook up row_changed signal
  (onDataDialogClose, onRowChanged): new functions
* grapher/graph-dialog.glade: Remove obsolete buttons (Add, Remove, Cancel).

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

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().

Fix regression introduced by commit 379c585

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.