From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 4899 invoked by alias); 21 Jul 2010 15:18:05 -0000 Received: (qmail 4874 invoked by uid 426); 21 Jul 2010 15:18:05 -0000 Date: Wed, 21 Jul 2010 15:18:00 -0000 Message-ID: <20100721151805.4864.qmail@sourceware.org> From: fche@sourceware.org To: systemtap-cvs@sourceware.org Subject: [SCM] systemtap: system-wide probe/trace tool branch, rhel60, created. release-1.2-17-gc3ed551 X-Git-Refname: refs/heads/rhel60 X-Git-Reftype: branch X-Git-Oldrev: 0000000000000000000000000000000000000000 X-Git-Newrev: c3ed551be916af35ef72f6cb93f584423da3dede Mailing-List: contact systemtap-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: systemtap-cvs-owner@sourceware.org List-Archive: Reply-To: systemtap@sourceware.org X-SW-Source: 2010-q3/txt/msg00093.txt.bz2 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 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 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 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 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 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 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 Signed-off-by: Ananth N Mavinakayanahalli Reported-by: Josh Stone 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 Acked-by: Jim Keniston uprobes_ppc: Don't emulate privileged MSR opcodes commit 78df9821ec0a2de0fee9b57cc58b66d3a79b6af4 Author: Mark Wielaard 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 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 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 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 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 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 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 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 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 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 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