This is the mail archive of the
systemtap-cvs@sourceware.org
mailing list for the systemtap project.
[SCM] systemtap: system-wide probe/trace tool branch, rhel60, created. release-1.2-17-gc3ed551
- From: fche at sourceware dot org
- To: systemtap-cvs at sourceware dot org
- Date: 21 Jul 2010 15:18:05 -0000
- Subject: [SCM] systemtap: system-wide probe/trace tool branch, rhel60, created. release-1.2-17-gc3ed551
- Reply-to: systemtap at sourceware dot org
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "systemtap: system-wide probe/trace tool".
The branch, rhel60 has been created
at c3ed551be916af35ef72f6cb93f584423da3dede (commit)
- Log -----------------------------------------------------------------
commit c3ed551be916af35ef72f6cb93f584423da3dede
Author: Mark Wielaard <mjw@redhat.com>
Date: Sun Jun 27 14:22:55 2010 +0200
Don't panic on utrace_barrier failures because process already ended.
* runtime/task_finder.c (stap_utrace_detach): When utrace_barrier returns
-ESRCH or -EALREADY treat it as success.
commit 8470d7c9f290f9c09e6971ef06f4c3bf6e5b2e32
Author: Frank Ch. Eigler <fche@elastic.org>
Date: Tue Jun 29 11:06:41 2010 -0400
Revert "rhbz596933: sdt.h "g" constraint on ppc is too broad"
This reverts commit 76095abacb12210cb5962ee7441da385ae448dd5.
commit a306dd9c5bd63cba37121c53191034d6399ee020
Author: Mark Wielaard <mjw@redhat.com>
Date: Fri Jun 4 11:34:23 2010 +0200
Add configure check for xmlto pdf with chapters titles starting with L...
There is a strange bug in older versions of xmlto when generating pdf.
https://bugzilla.redhat.com/show_bug.cgi?id=526273
So make sure to have a chapter title starting with L plus an refentry.
This will make sure the xmlto pdf support test fails on buggy versions.
configure.ac: Extend conftest.xml.
configure: Regenerated.
commit 76095abacb12210cb5962ee7441da385ae448dd5
Author: Frank Ch. Eigler <fche@elastic.org>
Date: Wed Jun 23 13:49:44 2010 -0400
rhbz596933: sdt.h "g" constraint on ppc is too broad
* includes/sys/sdt.h: Switch to inline-asm constraint set to "nro"
instead of "g", for all STAP_PROBE marker arguments.
commit 91099b03be25caeb326f4d1f156bf70a327c0db7
Author: David Smith <dsmith@redhat.com>
Date: Wed Apr 7 09:42:44 2010 -0500
PR 9871 (partial) fix. Removed all embedded-C in rpc.stp.
* tapset/rpc.stp: Replaced all embedded-C functions with script language.
... that is, nearly all ...; merged by hand from 7dfee5e and 9f61fce.
commit 05f04f9cb1684651a6a041b4ba026b1af83eebca
Author: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Date: Fri May 28 16:15:02 2010 +0530
Here is a patch implements a way for uprobes to skip single step
when a probe in on a nop instruction.
Currently its implemented for nop on x86/x86_64 and s390.
For powerpc, its implemented on all instructions that can be emulated.
(For this emulation code is borrowed from arch/powerpc/lib/sstep.c)
I havent tested it on s390. Updated with comments from Jim.
Signed-off-by: Srikar Dronamraju <srikar@linux.vnet.ibm.com>
Signed-off-by: Ananth N Mavinakayanahalli <ananth@in.ibm.com>
Reported-by: Josh Stone <jistone@redhat.com>
uprobes_ppc: define MSR_MASK and clean up error cases
- The emulate_step code was copied from the kernel, but needs to have
the #define MSR_MASK too.
- A case in emulate_step could fall through with an illegal instruction,
just break and return 0 instead.
- A few privileged instructions can't be emulated and so return -1, but
we don't want uprobe_emulate_insn to see that as non-zero -> TRUE.
At a minimum, this restores whatever SSOL-faulty behavior would have
existed before emulate_step was added. We should also add checks in
arch_validate_probed_insn to avoid such instructions in the first place.
Signed-off-by: Josh Stone <jistone@redhat.com>
Acked-by: Jim Keniston <jkenisto@us.ibm.com>
uprobes_ppc: Don't emulate privileged MSR opcodes
commit 78df9821ec0a2de0fee9b57cc58b66d3a79b6af4
Author: Mark Wielaard <mjw@redhat.com>
Date: Thu Jun 3 11:26:17 2010 +0200
Replace walk_stack field version guard with autoconf test.
The test for whether or not to assign print_context_stack to the
walk_stack stacktrace_ops field depended on the kernel version.
Replace with a proper runtime/autoconf test to make sure the field
always gets assigned when available.
* buildrun.cxx (compile_pass): Add output for STAP_CONF_WALK_STACK.
* runtime/autoconf-walk-stack.c: New test.
* runtime/stack.c (print_stack_ops): Assign walk_stack field
print_context_stack depending on STAP_CONF_WALK_STACK.
Guard uprobes using tests with uprobes_p.
PR11722 user_long(uaddr) should fetch a user space long (in case of 32-on-64).
user_long[_warn](uaddr) would fetch a kernel long, but that might be the
wrong size if the user space process was running in compat mode (32-on-64).
* tapset/conversions.stp (user_long): Fetch a compat_long_t if current
process is_compat_task().
(user_long_warn): Likewise.
commit 9e5c67f00d1e44699d12533a3db91b98b0ae7e2a
Author: Mark Wielaard <mjw@redhat.com>
Date: Wed Jun 9 11:50:48 2010 +0200
Loop on utrace_barrier if utrace_control UTRACE_DETACH returns -EINPROGRESS.
When utrace_control(tsk, eng, UTRACE_DETACH) returns -EINPROGRESS that
means there are still handlers running. So loop on utrace_barrier(tsk, eng)
in that case, till it no longer returns -ERESTARTSYS. That makes sure that
no engine handler will be called afterwards, so we can safely unload the
stap module. Not doing this might have caused PR11672
(utrace_report_syscall_exit crash), although we don't yet have a simple
reproducer for that issue.
* runtime/itrace.c (remove_usr_itrace_info): Loop on utrace_barrier if
utrace_control returned -EINPROGRESS.
* runtime/task_finder.c (stap_utrace_detach): Likewise.
(stap_utrace_detach_ops): Likewise. And warn if stap_utrace_detach
didn't return successfully.
(__stp_utrace_attach): Loop on -ERESTARTSYS after utrace_barrier.
(__stp_utrace_task_finder_target_quiesce): Likewise.
Add more error messages when something goes wrong during task_finder detach.
* runtime/task_finder.c (stap_utrace_detach): Report when utrace_barrier
fails.
(stap_utrace_detach_ops): Report how stap_utrace_detach failed.
commit 34c08f2a8f83c6ffa6939214448385e42982bace
Author: Josh Stone <jistone@redhat.com>
Date: Mon Jun 14 10:44:10 2010 -0700
Banish the SyScall
We don't need to deal with the SYSCALL_WRAPPERS-generated SyS_foo,
because since the dawn of that feature they have also provided a normal
sys_foo alias. So as long as alias tracking is working, both in our
translator and in kallsyms for kprobe.function, we can be blissfully
ignorant.
http://sourceware.org/ml/systemtap/2010-q2/msg00551.html
Fixed PR11710 so that nd_syscall probes work on s390x systems.
* tapset/s390/registers.stp (_stp_arg): Fixed logic error. The last case
was missing an "if", which caused all _stp_arg() calls to only return
the value of r6.
Fixed PR11707 so that nd_syscall probes work on ppc64 systems.
* tapset/powerpc/registers.stp (_stp_get_register_by_offset): Fixed
compile error.
(_stp_arg): Fixed logic error. The last case was missing an "if", which
caused all _stp_arg() to only return the value of r10.
commit 521c6939ff938f75e8b6db265835af3c4fc72011
Author: Josh Stone <jistone@redhat.com>
Date: Fri Jun 11 18:22:56 2010 -0700
PR11698: Track aliases even with ppc function descriptors
We can include function descriptors in the symbol table, and just make
sure that we don't try to probe them with query_module_symtab. This
lets update_symtab do its thing resolving aliased names.
* dwflpp.h (func_info): Add a descriptor flag.
* tapsets.cxx (symbol_table::add_symbol): Set the descriptor flag.
(symbol_table::read_symbols): Not descriptors.
(symbol_table::get_from_elf): Functions from powerpc rejected sections
are function descriptors.
(dwarf_query::query_module_symtab): Don't probe descriptors.
commit b67b3994ab0c3d1be8cb7a297af0f771c5259c3e
Author: Mark Wielaard <mjw@redhat.com>
Date: Wed May 26 12:24:21 2010 +0200
rhbz #596083 Make stap-prep resolve kernel-debuginfo-common package name.
The kernel-debuginfo-common package name is slightly different on different
distros. But the correct name is always required by the kernel-debuginfo
package. So don't add it to the CANDIDATES list, but let the yumdowloader
--resolve it. Tested on f13, rhel5 and rhel6 beta.
* stap-prep: Remove kernel-debuginfo-common from CANDIDATES, add --resolve
to yumdownloader.
rhbz #596083 Account for dot and dotless kernel variant in stap-prep.
uname -r can produce different kinds of output:
2.6.32-30.el6.x86_64 (no variant, but including ".arch")
2.6.18-194.3.1.el5debug ("variant", without dot, no arch)
2.6.33.4-95.fc13.i686.PAE (".arch.variant", two dots)
We didn't handle the dot before the variant used in newer distros.
* stap-prep: Remove either ".variant" or "variant" from the end of release
uname string.
commit 2c759d3f62ef9a996ab3b711693657cd8186fd9e
Author: Frank Ch. Eigler <fche@elastic.org>
Date: Fri May 21 13:25:03 2010 -0400
rhbz 594079: tapset/scsi.stp forward-porting for newer scsi sd/sr
* scsi.stp (scsi.ioentry): Rephrase in terms of scsi_prep_fn, sd_prep_fn
and sr_prep_fn.
commit 0a81d94ea5d1c5bb7ef5bb96e63fb77c3cac361e
Author: David Smith <dsmith@redhat.com>
Date: Tue May 18 13:19:23 2010 -0500
Fixed BZ 592830 by fixing signal.check_ignored.return for newer kernels.
* tapset/signal.stp (signal.check_ignored.return): Provide an alternative
for newer kernels where the sig_ignored() function has been inlined.
* tapset/errno.stp (return_str): New function.
* testsuite/buildok/signal-detailed.stp: Made the
signal.check_ignored.return test non-optional.
commit 987266b1e0f83e9676ccc7d7327f6088e8880c56
Author: David Smith <dsmith@redhat.com>
Date: Mon Apr 26 11:56:29 2010 -0500
PR 11338 (partial): Used @defined in nfsd.stp and general update.
* tapset/nfsd.stp: Large update to get working on RHEL4, RHEL5, f12, and
rawhide. Removed some embedded-C and kernel version checks.
* testsuite/buildok/nfsd-embedded.stp: Added the rest of the nfsd embedded
functions.
* testsuite/buildok/nfsd-detailed.stp: New detailed test of all nfsd
probes and convenience variables.
commit 9775a39d9bffff4978e765d9b5ec09c519b84d7c
Author: David Smith <dsmith@redhat.com>
Date: Fri Apr 16 09:12:06 2010 -0500
Fixed PR 11480 by updating rpc.stp and its tests.
* tapset/rpc.stp: Large update to get working on RHEL4, RHEL5, f12, and
rawhide.
* testsuite/buildok/rpc-detailed.stp: New detailed test of all rpc probes.
* testsuite/buildok/rpc-embedded.stp: Updated.
* testsuite/systemtap.pass1-4/buildok.exp: Removed kfail for
rpc-all-probes.stp.
Added atomic_read() embedded-C function and tests.
* tapset/atomic.stp: Added atomic_read().
* testsuite/buildok/atomic.stp: Added atomic_read() compile test.
* testsuite/systemtap.base/atomic.exp: Added atomic_read() tests.
* testsuite/lib/stap_run_error.exp (stap_run_error): Logs stap command and
tries to ensure inferior process is killed.
As a part of merging commit 6c62bc42:
Improved atomic.exp testcase by removing the need for an external module.
* testsuite/systemtap.base/atomic.exp: Removed the need for the external
module by putting the atomic_long_t inside the stap module.
* testsuite/systemtap.base/atomic_module.c: Removed.
* testsuite/systemtap.base/atomic_module.makefile: Ditto.
* testsuite/lib/stap_run_error.exp (stap_run_error): Better error handling.
commit 81b07590d1c5c43e26ecb416adfdf97bab9b7ae4
Author: Mark Wielaard <mjw@redhat.com>
Date: Wed May 12 14:16:26 2010 +0200
PR11590 Revert "PR6954: make ++/-- operation trigger automatic global printing"
This reverts commit 982b88bd950381434a8325e550eff9df59f59dcd.
Pretending that pre/postfix ++/-- operations only writes to its argument
messes up the optimization of a variable that is assigned and then
only used through post/prefix ++/--operators.
commit f5897cc71f3db79a21eab05ca1b718b294163229
Author: David Smith <dsmith@redhat.com>
Date: Tue Apr 20 13:10:46 2010 -0500
Fixed PR 3833 by updating nfs_proc.stp (and removing kernel version checks).
* tapset/nfs_proc.stp: Large update to get working on RHEL4, RHEL5, f12,
and rawhide.
* testsuite/buildok/nfs_proc-detailed.stp: New detailed test of all
nfs_proc probes and convenience variables.
-----------------------------------------------------------------------
hooks/post-receive
--
systemtap: system-wide probe/trace tool