From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F40CFC433FE for ; Thu, 28 Oct 2021 02:05:21 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id DC38861039 for ; Thu, 28 Oct 2021 02:05:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229822AbhJ1CHr (ORCPT ); Wed, 27 Oct 2021 22:07:47 -0400 Received: from szxga03-in.huawei.com ([45.249.212.189]:26198 "EHLO szxga03-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229534AbhJ1CHp (ORCPT ); Wed, 27 Oct 2021 22:07:45 -0400 Received: from dggeme762-chm.china.huawei.com (unknown [172.30.72.57]) by szxga03-in.huawei.com (SkyGuard) with ESMTP id 4Hfpkw5WD2z8tvq; Thu, 28 Oct 2021 10:03:52 +0800 (CST) Received: from huawei.com (10.67.189.2) by dggeme762-chm.china.huawei.com (10.3.19.108) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2308.15; Thu, 28 Oct 2021 10:05:14 +0800 From: Lexi Shao To: CC: , , , , , , , , , , , , Subject: Re: [PATCH] perf symbol: ignore $a/$d symbols for ARM modules Date: Thu, 28 Oct 2021 10:05:09 +0800 Message-ID: <20211028020509.39082-1-shaolexi@huawei.com> X-Mailer: git-send-email 2.12.3 In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [10.67.189.2] X-ClientProxiedBy: dggems706-chm.china.huawei.com (10.3.19.183) To dggeme762-chm.china.huawei.com (10.3.19.108) X-CFilter-Loop: Reflected Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 27/10/2021 23:10, James Clark wrote: >On 27/10/2021 13:31, Lexi Shao wrote: >> On 27/10/2021 18:24, James Clark wrote: >>> On 27/10/2021 10:52, Lexi Shao wrote: >>>> On ARM machine, kernel symbols from modules can be resolved to $a >>>> instead of printing the actual symbol name. Ignore symbols starting with >>>> "$" when building kallsyms rbtree. >>>> >>>> A sample stacktrace is shown as follows: >>>> >>>> c0f2e39c schedule_hrtimeout+0x14 ([kernel.kallsyms]) >>>> bf4a66d8 $a+0x78 ([test_module]) >>>> c0a4f5f4 kthread+0x15c ([kernel.kallsyms]) >>>> c0a001f8 ret_from_fork+0x14 ([kernel.kallsyms]) >>>> >>>> On ARM machine, $a/$d symbols are used by the compiler to mark the >>>> beginning of code/data part in code section. These symbols are filtered >>>> out when linking vmlinux(see scripts/kallsyms.c ignored_prefixes), but >>>> are left on modules. So there are $a symbols in /proc/kallsyms which >>>> share the same addresses with the actual module symbols and confuses perf >>>> when resolving symbols. >>> >>> Hi Lexi, >>> >>> Is it worth using or re-implementing the entire is_ignored_symbol() function >>>from scripts/kallsyms.c? It seems like this change only fixes one occurrence, >>> but is_ignored_symbol() has a big list of other cases. >>> >>> Unless those cases are different? >>> >>> Thanks >>> James >> >> Hi James, >> >> I don't think it's necessary to cover all the cases listed in >> is_ignored_symbol(). As long as the symbols are unique and not overlapping >> with each other, they should't cause problems in resolving symbols. So most >> of the cases in is_ignored_symbol() should be irrelevant. > >Is it possible to do the filtering of the module symbols somewhere else like >in kernel/kallsyms.c? I'm not that familiar with it but it seems like >at one point when populating kallsyms '$...' functions are filtered out, but >when modules are loaded the symbols are not filtered because another code path is >used? > >If kallsyms has some duplicate addresses in there then isn't the bug in kallsyms >rather than perf? And another tool could get confused too. > >James Yes we can filter these symbols out when adding module symbols to the kallsyms list. I will send another patch that modifies module.c to ignore '$' symbols when loading module. I think it's worth applying this patch in the sense that perf userspace tool can be used on all 5.x kernels, and people don't have to update the kernel to fix it. Lexi > >> >> Lexi >> >>> >>>> >>>> After this patch, the module symbol name is printed: >>>> >>>> c0f2e39c schedule_hrtimeout+0x14 ([kernel.kallsyms]) >>>> bf4a66d8 test_func+0x78 ([test_module]) >>>> c0a4f5f4 kthread+0x15c ([kernel.kallsyms]) >>>> c0a001f8 ret_from_fork+0x14 ([kernel.kallsyms]) >>>> >>>> Signed-off-by: Lexi Shao >>>> --- >>>> tools/perf/util/symbol.c | 4 ++++ >>>> 1 file changed, 4 insertions(+) >>>> >>>> diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c >>>> index 0fc9a5410739..35116aed74eb 100644 >>>> --- a/tools/perf/util/symbol.c >>>> +++ b/tools/perf/util/symbol.c >>>> @@ -702,6 +702,10 @@ static int map__process_kallsym_symbol(void *arg, const char *name, >>>> if (!symbol_type__filter(type)) >>>> return 0; >>>> >>>> + /* Ignore local symbols for ARM modules */ >>>> + if (name[0] == '$') >>>> + return 0; >>>> + >>>> /* >>>> * module symbols are not sorted so we add all >>>> * symbols, setting length to 0, and rely on >>>> >>>