From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751398AbdGMJul (ORCPT ); Thu, 13 Jul 2017 05:50:41 -0400 Received: from ozlabs.org ([103.22.144.67]:37903 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751018AbdGMJuk (ORCPT ); Thu, 13 Jul 2017 05:50:40 -0400 From: Michael Ellerman To: Jiri Olsa Cc: Sukadev Bhattiprolu , Arnaldo Carvalho de Melo , Madhavan Srinivasan , Andi Kleen , ak@linux.intel.com, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, mikey@neuling.org, mpetlan@redhat.com Subject: Re: [GIT PULL] Please pull JSON files for Power9 PMU events In-Reply-To: <20170713080919.GA6531@krava> References: <20170712020827.GB31223@us.ibm.com> <87a849luvr.fsf@concordia.ellerman.id.au> <20170712144611.GN27350@kernel.org> <20170712164008.GC31223@us.ibm.com> <87y3rtj91u.fsf@concordia.ellerman.id.au> <20170713080919.GA6531@krava> User-Agent: Notmuch/0.21 (https://notmuchmail.org) Date: Thu, 13 Jul 2017 19:50:37 +1000 Message-ID: <87vamwk6mq.fsf@concordia.ellerman.id.au> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Jiri Olsa writes: > On Thu, Jul 13, 2017 at 01:43:41PM +1000, Michael Ellerman wrote: >> Sukadev Bhattiprolu writes: >> ... >> > >> > tools/perf/pmu-events/arch/powerpc/mapfile.csv | 3 + >> > .../perf/pmu-events/arch/powerpc/power9/cache.json | 176 +++++ >> > .../arch/powerpc/power9/floating-point.json | 44 ++ >> > .../pmu-events/arch/powerpc/power9/frontend.json | 446 +++++++++++ >> > .../pmu-events/arch/powerpc/power9/marked.json | 782 +++++++++++++++++++ >> > .../pmu-events/arch/powerpc/power9/memory.json | 158 ++++ >> >> In the map file we have "power9.json", but the files are power9/x.json. >> How does that work? > > hum, we take just the base name from that string and use it > as a map to the directory/struct with events.. so no harm done I figured it must be working somehow but it's a bit odd :) > I guess it's the leftover from when we switched from single file > to per topic event files.. x86 arch seems to have it right Yep. > I'll send the fix Thanks. cheers