lkml.org 
[lkml]   [2012]   [Oct]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH] scripts/recordmcount.pl: Support build with -ffunction-sections.
From
Date
On Tue, 2012-10-30 at 14:51 +0000, Will Newton wrote:
> Scan any text section whose name begins with ".text." so
> we will find all the functions in a kernel built with
> -ffunction-sections.

A couple of things.

First, I'm very paranoid about a blanket "ok" on sections. We must
guarantee that all sections that starts with ".text" never is freed. And
if it is freed, that it must inform ftrace that it's about to free it
before it does. If we can not guarantee this, then we can't do it.

Second, most archs today do not use recordmcount.pl. They use the new
recordmcount.c file.

-- Steve

>
> Signed-off-by: Will Newton <will.newton@imgtec.com>
> ---
> scripts/recordmcount.pl | 8 ++++++--
> 1 files changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
> index b33446c..89461c4 100755
> --- a/scripts/recordmcount.pl
> +++ b/scripts/recordmcount.pl
> @@ -135,9 +135,13 @@ my %text_sections = (
> ".spinlock.text" => 1,
> ".irqentry.text" => 1,
> ".kprobes.text" => 1,
> - ".text.unlikely" => 1,
> );
>
> +sub is_valid_section
> +{
> + return defined($text_sections{$1}) || $1 =~ m/^\.text\./;
> +}
> +
> # Note: we are nice to C-programmers here, thus we skip the '||='-idiom.
> $objdump = 'objdump' if (!$objdump);
> $objcopy = 'objcopy' if (!$objcopy);
> @@ -502,7 +506,7 @@ while (<IN>) {
> $read_headers = 0;
>
> # Only record text sections that we know are safe
> - $read_function = defined($text_sections{$1});
> + $read_function = is_valid_section($1);
> # print out any recorded offsets
> update_funcs();
>




\
 
 \ /
  Last update: 2012-10-30 16:21    [W:0.044 / U:1.524 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site