Re: [RFC][PATCH] ftrace/scripts: Work around for addition of metag magic but not relocations

From: Steven Rostedt
Date: Fri Jul 08 2016 - 17:32:48 EST


On Fri, 8 Jul 2016 12:18:50 -0700
Laura Abbott <labbott@xxxxxxxxxx> wrote:

> glibc recently did a sync up (94e73c95d9b5 "elf.h: Sync with the gabi
> webpage") that added a #define for EM_METAG but did not add relocations
>
> This triggers build errors:
>
> scripts/recordmcount.c: In function 'do_file':
> scripts/recordmcount.c:466:28: error: 'R_METAG_ADDR32' undeclared (first use in this function)
> case EM_METAG: reltype = R_METAG_ADDR32;
> ^~~~~~~~~~~~~~
> scripts/recordmcount.c:466:28: note: each undeclared identifier is reported only once for each function it appears in
> scripts/recordmcount.c:468:20: error: 'R_METAG_NONE' undeclared (first use in this function)
> rel_type_nop = R_METAG_NONE;
> ^~~~~~~~~~~~
>
> Work around this change with some more #ifdefery for the relocations.

Looks fine to me. I'll pull it in, and this probably should go stable
too.

-- Steve

>
> Signed-off-by: Laura Abbott <labbott@xxxxxxxxxx>
> ---
> I hit this while testing with a newer glibc. I'm not sure what the policy is
> about glibc, so this might just serve as an FYI for anyone else who
> is working on the bleeding edge.
>
> Fedora bug is at https://bugzilla.redhat.com/show_bug.cgi?id=1354034 for the
> curious.
> ---