From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757295AbbIUQ3u (ORCPT ); Mon, 21 Sep 2015 12:29:50 -0400 Received: from smtprelay4.synopsys.com ([198.182.47.9]:58314 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753345AbbIUQ3s (ORCPT ); Mon, 21 Sep 2015 12:29:48 -0400 From: Alexey Brodkin To: "eranian@google.com" CC: "linux-kernel@vger.kernel.org" , "peterz@infradead.org" , "pgynther@google.com" , Vineet Gupta , "mingo@redhat.com" , "acme@kernel.org" Subject: Re: [PATCH] perf tools: Fix build errors with mipsel-linux-uclibc compiler Thread-Topic: [PATCH] perf tools: Fix build errors with mipsel-linux-uclibc compiler Thread-Index: AQHQ9Iq29HiPoeA3Y026Ucfuyp9OsA== Date: Mon, 21 Sep 2015 16:29:39 +0000 Message-ID: <1442852978.3859.15.camel@synopsys.com> Accept-Language: en-US, ru-RU Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.121.8.93] Content-Type: text/plain; charset="utf-8" Content-ID: <4D74A6A3BFE0CF48B1DA6EEE87C7E763@internal.synopsys.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id t8LGTt07032288 Hi Stephane, > Em Fri, Sep 04, 2015 at 04:01:54PM -0700, Petri Gynther escreveu: > > On Fri, Sep 4, 2015 at 3:48 PM, Arnaldo Carvalho de Melo > > wrote: > > > Em Fri, Sep 04, 2015 at 03:45:03PM -0700, Petri Gynther escreveu: > > >> This commit breaks the perf tool MIPS build because MIPS doesn't have > > >> CONFIG_PERF_REGS=y in .config-detected: > > >> > > >> commit bcc84ec65ad1bd9f777a1fade6f8e5e0c5808fa5 > > >> Author: Stephane Eranian > > >> Date: Mon Aug 31 18:41:12 2015 +0200 > > >> > > >> perf record: Add ability to name registers to record > > > > > > There was a fix for another arch that came after this one.. here it is, already > > > upstream, can you take a look? > > > > > > commit af4aeadd8c04303c0aa2d112145c3627e2ebd026 > > > Author: Stephane Eranian > > > Date: Tue Sep 1 11:30:14 2015 +0200 > > > > > > perf tools: Fix link time error with sample_reg_masks on non x86 > > > > > > This patch makes perf compile on non x86 platforms by defining a weak > > > symbol for sample_reg_masks[] in util/perf_regs.c. > > > > > > The patch also moves the REG() and REG_END() macros into the > > > util/per_regs.h header file. The macros are renamed to > > > SMPL_REG/SMPL_REG_END to avoid clashes with other header files. > > > > > > > > > > I have the above commit in my tree (as I have the latest Linus' tree > > as of right now). > > The MIPS build is still broken as I reported. > > Stephane, can you please take a look? The same problem happens when building perf on ARC. I'm on 4.3-rc2 and that's what I see: ------------------->8----------------- LINK .../linux-4.3-rc2/tools/perf/perf .../linux-4.3-rc2/tools/perf/libperf.a(libperf-in.o): In function `parse_regs': .../linux-4.3-rc2/tools/perf/util/parse-regs-options.c:28: undefined reference to `sample_reg_masks' .../linux-4.3-rc2/tools/perf/util/parse-regs-options.c:45: undefined reference to `sample_reg_masks' .../linux-4.3-rc2/tools/perf/util/parse-regs-options.c:38: undefined reference to `sample_reg_masks' collect2: error: ld returned 1 exit status ------------------->8----------------- Any chance for this to be discussed any further? -- Alexey{.n++%ݶw{.n+{G{ayʇڙ,jfhz_(階ݢj"mG?&~iOzv^m ?I