From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759172Ab2AFXPV (ORCPT ); Fri, 6 Jan 2012 18:15:21 -0500 Received: from mail3.caviumnetworks.com ([12.108.191.235]:8840 "EHLO mail3.caviumnetworks.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755716Ab2AFXPU (ORCPT ); Fri, 6 Jan 2012 18:15:20 -0500 Message-ID: <4F078086.7060208@cavium.com> Date: Fri, 06 Jan 2012 15:15:18 -0800 From: David Daney User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101027 Fedora/3.0.10-1.fc12 Thunderbird/3.0.10 MIME-Version: 1.0 To: Steven Rostedt CC: David Daney , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH] recordmcount: Fix handling of elf64 big-endian objects. References: <1324345362-12230-1-git-send-email-ddaney.cavm@gmail.com> <1324350542.5916.48.camel@gandalf.stny.rr.com> <4F075CAE.2040709@gmail.com> <1325891377.7642.17.camel@gandalf.stny.rr.com> In-Reply-To: <1325891377.7642.17.camel@gandalf.stny.rr.com> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 06 Jan 2012 23:15:19.0528 (UTC) FILETIME=[0EC66E80:01CCCCC9] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 01/06/2012 03:09 PM, Steven Rostedt wrote: > On Fri, 2012-01-06 at 12:42 -0800, David Daney wrote: > >> Any more feedback on this patch? Should I look for it to appear in 3.3? > > OK, I tested it and then I noticed that mainline doesn't have > "HAVE_C_RECORDMCOUNT" set. Is that going in the PPC tree? I have no idea, I'm a MIPS guy at the moment. > I added that > to arch/powerpc/Kconfig, and did see the bug you saw. After adding your > patch, I did a diff of the available_filter_functions and had this: > >> .elf_core_extra_phdrs >> .elf_core_write_extra_phdrs >> .elf_core_write_extra_data >> .elf_core_extra_data_size > > These 4 functions were added after doing the change. Weird. > > Anyway, I'll run the change through my normal tests and then push it > out. > Thanks, David Daney