Re: Build failure triggered by recordmcount

[wu zhangjin <wuzhangjin@gmail.com>]

On Wed, Nov 24, 2010 at 1:09 AM, Steven Rostedt <rostedt@goodmis.org> 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@gmail.com>

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
>>
>>
>
>
>