From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751181AbdCYMpK (ORCPT ); Sat, 25 Mar 2017 08:45:10 -0400 Received: from mga14.intel.com ([192.55.52.115]:49581 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750718AbdCYMpI (ORCPT ); Sat, 25 Mar 2017 08:45:08 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,219,1486454400"; d="scan'208";a="1146849380" Subject: Re: [PATCH v5 2/5] perf report: Find the inline stack for a given address To: Ravi Bangoria , acme@kernel.org References: <1489700547-7260-1-git-send-email-yao.jin@linux.intel.com> <1489700547-7260-3-git-send-email-yao.jin@linux.intel.com> <58D619C1.50706@linux.vnet.ibm.com> Cc: jolsa@kernel.org, Linux-kernel@vger.kernel.org, ak@linux.intel.com, kan.liang@intel.com, milian.wolff@kdab.com, yao.jin@intel.com From: "Jin, Yao" Message-ID: Date: Sat, 25 Mar 2017 20:45:04 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <58D619C1.50706@linux.vnet.ibm.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Ravi, Arnaldo, The build error happens when BFD lib is not set in build environment. Anyway the patch series should be improved to get better compatibility for this case. For easy patch management, I send the v6 patch series for fixing this issue. Very sorry for the inconvenience. Thanks Jin Yao On 3/25/2017 3:18 PM, Ravi Bangoria wrote: > Hi Jin / Arnaldo, > > I see a build failure with this patch: > > On Friday 17 March 2017 03:12 AM, Jin Yao wrote: >> It would be useful for perf to support a mode to query the >> inline stack for a given callgraph address. This would simplify >> finding the right code in code that does a lot of inlining. >> >> The srcline.c has contained the code which supports to translate >> the address to filename:line_nr. This patch just extends the >> function to let it support getting the inline stacks. > ... >> + while (getline(&filename, &len, fp) != -1) { >> + if (filename_split(filename, &line_nr) != 1) { >> + free(filename); >> + goto out; >> + } >> + >> + if (inline_list__append(filename, NULL, line_nr, node) != 0) > util/srcline.c: In function ‘addr2inlines’: > util/srcline.c:403:7: error: too few arguments to function ‘inline_list__append’ > if (inline_list__append(filename, NULL, line_nr, node) != 0) > ^ > util/srcline.c:34:12: note: declared here > static int inline_list__append(char *filename, char *funcname, int line_nr, > ^ > util/srcline.c: At top level: > util/srcline.c:60:13: error: ‘inline_list__reverse’ defined but not used [-Werror=unused-function] > static void inline_list__reverse(struct inline_node *node) > ^ > cc1: all warnings being treated as errors > mv: cannot stat ‘util/.srcline.o.tmp’: No such file or directory > > Thanks, > Ravi >