On Wed, Nov 24, 2010 at 1:09 AM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote: > On Mon, 2010-11-22 at 19:41 -0800, John Reiser wrote: >> It looks to me like the change which introduced "virtual functions" >> forgot about cross-platform endianness. Can anyone please test this patch? >> Thank you to Arnaud for supplying before+after data files do_mounts*.o. >> >> >> recordmcount: Honor endianness in fn_ELF_R_INFO > > Arnaud, can I get a "Tested-by" from you. > > Wu, can you give me your Acked-by: Acked-by: Wu Zhangjin <wuzhangjin@xxxxxxxxx> Thanks & Regards, Wu Zhangjin > > Thanks, > > -- Steve > >> >> --- >> scripts/recordmcount.h | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/scripts/recordmcount.h b/scripts/recordmcount.h >> index 58e933a..3966717 100644 >> --- a/scripts/recordmcount.h >> +++ b/scripts/recordmcount.h >> @@ -119,7 +119,7 @@ static uint_t (*Elf_r_sym)(Elf_Rel const *rp) = fn_ELF_R_SYM; >> static void fn_ELF_R_INFO(Elf_Rel *const rp, unsigned sym, unsigned type) >> { >> - rp->r_info = ELF_R_INFO(sym, type); >> + rp->r_info = _w(ELF_R_INFO(sym, type)); >> } >> static void (*Elf_r_info)(Elf_Rel *const rp, unsigned sym, unsigned type) = fn_ELF_R_INFO; >> -- 1.7.3.2 >> >> > > >