Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757970AbYJVVkI (ORCPT ); Wed, 22 Oct 2008 17:40:08 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754871AbYJVVgG (ORCPT ); Wed, 22 Oct 2008 17:36:06 -0400 Received: from hrndva-omtalb.mail.rr.com ([71.74.56.125]:51449 "EHLO hrndva-omtalb.mail.rr.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757506AbYJVVfz (ORCPT ); Wed, 22 Oct 2008 17:35:55 -0400 Message-Id: <20081022212721.167005680@goodmis.org> User-Agent: quilt/0.46-1 Date: Wed, 22 Oct 2008 17:27:21 -0400 From: Steven Rostedt To: linux-kernel@vger.kernel.org Cc: Ingo Molnar , Frederic Weisbecker , Abhishek Sagar , "David S. Miller" , Thomas Gleixner , Peter Zijlstra , Andrew Morton , Linus Torvalds Subject: [PATCH 00/13 v2] ftrace: clean ups and fixes Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2203 Lines: 57 [ Changes since v1: Tested CONFIG_FTRACE & CONFIG_DYNAMIC_FTRACE CONFIG_FTRACE & !CONFIG_DYNAMIC_FTRACE !CONFIG_FTRACE on _both_ x86_64 and i386 kernels. Fixed typo in recordmcount.pl script s/xi386/i386/ Used fault return codes for ftrace_modify_code return. Added patch to use probe_kernel_read/write. Better changelog on ftrace_kill change. Use return code on WARN_ON*. Removed "notrace" from arch/*/kernel/ftrace.c and instead used the CFLAGS_REMOVE_ftrace.o = -pg in the Makefile. This change cleans up the look of the code, and prevents any further bugs from happening (forgetting a notrace). Added sparc64 in the removal of ftrace_mcount_set (forgot the quilt add in the first version) ] This is a series of patches to make ftrace more robust and clean ups. The first couple of patches fix the recordmount.pl script and changes it to only record the .text section functions. This means that the init sections will not be processed. I still have a patch to add notrace to the init sections, and not for safety reasons, but for perfomance. Since the init sections will not be processed, they will still call mcount. Note, mcount is just a ret, but why have the init code waste CPU cycles to call a stub function? A FTRACE_WARN_ON is added to change all WARN_ONS to not only print a warning, but also to disable ftrace as well. The later patches are a bit more drastic. Since the daemon is error prone, I stripped it out. In doing so, I have to disable dynamic ftrace from all archs that use it. The archs can get dynamic ftrace reenabled when they are ported to the recordmcount.pl method. (Arch maintainers, please contact me if you want help. I can do it with some information about your arch). Since the hash was created to work with the daemon, that too was stripped out, making the remaining code smaller and cleaner. The kprobe hooks in ftrace may need to be reworked. -- Steve -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/