Re: [PATCH] scripts/recordmcount.c: account for .softirqentry.text

From: Dmitry Vyukov
Date: Mon Sep 26 2016 - 11:12:33 EST


On Mon, Sep 26, 2016 at 5:06 PM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> On Mon, 26 Sep 2016 11:02:36 -0400
> Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
>
>> > ---
>> > scripts/recordmcount.c | 1 +
>> > 1 file changed, 1 insertion(+)
>> >
>> > diff --git a/scripts/recordmcount.c b/scripts/recordmcount.c
>> > index 42396a7..a68f031 100644
>> > --- a/scripts/recordmcount.c
>> > +++ b/scripts/recordmcount.c
>> > @@ -363,6 +363,7 @@ is_mcounted_section_name(char const *const txtname)
>> > strcmp(".sched.text", txtname) == 0 ||
>> > strcmp(".spinlock.text", txtname) == 0 ||
>> > strcmp(".irqentry.text", txtname) == 0 ||
>> > + strcmp(".softirqentry.text", txtname) == 0 ||
>> > strcmp(".kprobes.text", txtname) == 0 ||
>> > strcmp(".text.unlikely", txtname) == 0;
>> > }
>>
>
> Oh, some archs still use recordmcount.pl. I'm guessing that needs to be
> updated too:
>
> Signed-off-by: Steven Rostedt <rostedt@xxxxxxxxxxx>
> ---
> diff --git a/scripts/recordmcount.pl b/scripts/recordmcount.pl
> index 96e2486..2d48011 100755
> --- a/scripts/recordmcount.pl
> +++ b/scripts/recordmcount.pl
> @@ -134,6 +134,7 @@ my %text_sections = (
> ".sched.text" => 1,
> ".spinlock.text" => 1,
> ".irqentry.text" => 1,
> + ".softirqentry.text" => 1,
> ".kprobes.text" => 1,
> ".text.unlikely" => 1,
> );


Thanks, Steven. Mailed v2 with recordmcount.pl change.