Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753683AbYKZDOx (ORCPT ); Tue, 25 Nov 2008 22:14:53 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751668AbYKZDOp (ORCPT ); Tue, 25 Nov 2008 22:14:45 -0500 Received: from ti-out-0910.google.com ([209.85.142.184]:4726 "EHLO ti-out-0910.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751403AbYKZDOo (ORCPT ); Tue, 25 Nov 2008 22:14:44 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=AkU5PUrYAtcfzYtEV2Hbg029KcaLcjJ/B93z/Aj7rDd1io9rhK2W1M/mVXJXCwt5Nh qejfrTqT5w3p75ggW9q7Q79U70DaWtZuC+ojfLeq2RXKeSyhi7d1UU1cXLFuKXMY5z5W igFYO4aoOquFM6ouCrxZyrzuaBwEX4Pg6Czds= Date: Wed, 26 Nov 2008 11:14:38 +0800 From: walimis To: Paul Mundt , Steven Rostedt , Liming Wang , Ingo Molnar , linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/1] ftrace: adding other non-leaving .text sections Message-ID: <20081126031438.GA16166@ubuntu.mshome.net> References: <1227666566-9044-1-git-send-email-liming.wang@windriver.com> <20081126030446.GA14685@linux-sh.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20081126030446.GA14685@linux-sh.org> User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1640 Lines: 36 On Wed, Nov 26, 2008 at 12:04:46PM +0900, Paul Mundt wrote: >On Tue, Nov 25, 2008 at 10:03:35PM -0500, Steven Rostedt wrote: >> >> On Wed, 26 Nov 2008, Liming Wang wrote: >> >> > Please review this patch, and I don't know whether new sections will conflict >> > with sched tracer or others. >> > This patch is against Steven tree's tip/devel latest commit. >> > >> > Impact: improve recordmcount.pl >> > >> > Besides .text section, there are three .text sections that won't >> > be freed after kernel booting. They are: .sched.text, .spinlock.text >> > and .kprobes.text, which contain functions we can trace. But the last >> > section ".kprobes.text" is particular, which has been marked as "notrace", >> > we ignore it. Thus we add other two sections. >> >> Yep, only those sections that we know will not disappear are OK. I was >> conservative with only using ".text" but I knew there were other sections >> that were OK, so I used the hash array to make it easy to add others. >> >Does your .text cover .text.head by the way? No, and not needed. Because .text.head is of booting code, to which function tracer can't trace. walimis >-- >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/ -- 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/