Re: [PATCH] scripts/recordmcount.pl: Support build with-ffunction-sections.

[Steven Rostedt]

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@xxxxxxxxxx>
> ---
>  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();
> 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/