Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751023AbdHaKxg (ORCPT ); Thu, 31 Aug 2017 06:53:36 -0400 Received: from mail-by2nam01on0077.outbound.protection.outlook.com ([104.47.34.77]:29600 "EHLO NAM01-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750790AbdHaKxd (ORCPT ); Thu, 31 Aug 2017 06:53:33 -0400 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Robert.Richter@cavium.com; Date: Thu, 31 Aug 2017 12:53:17 +0200 From: Robert Richter To: Zhangshaokun Cc: Ganapatrao Kulkarni , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Will.Deacon@arm.com, catalin.marinas@arm.com, mark.rutland@arm.com, acme@kernel.org, alexander.shishkin@linux.intel.com, peterz@infradead.org, mingo@redhat.com, jnair@caviumnetworks.com, Jonathan.Cameron@huawei.com, gklkml16@gmail.com, John Garry Subject: Re: [PATCH v6 4/4] perf vendor events arm64: Add ThunderX2 implementation defined pmu core events Message-ID: <20170831105317.GQ17961@rric.localdomain> References: <20170824120349.2323-1-ganapatrao.kulkarni@cavium.com> <20170824120349.2323-5-ganapatrao.kulkarni@cavium.com> <43dcc107-263a-5429-73d4-a5290c6610d6@hisilicon.com> <20170829124730.GH17961@rric.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [77.179.42.136] X-ClientProxiedBy: DB6PR1001CA0035.EURPRD10.PROD.OUTLOOK.COM (2603:10a6:4:55::21) To BL2PR07MB2338.namprd07.prod.outlook.com (2a01:111:e400:c753::16) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 9b6be1a3-92d1-4071-c533-08d4f05e844c X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);SRVR:BL2PR07MB2338; X-Microsoft-Exchange-Diagnostics: 1;BL2PR07MB2338;3:7CHo0lJUTio4GHmdszo7iLh3DPZnPMg4zVlR4kBLnKdgOras+dbDmsYy5IlwI7a6oKh7wMzlEegbAOAjmoq6Q+o6KSR/tEZl4dqVsVnekc9GSgfVP+oA7P/nzEP5PUbSDMVycsD5RfgJKW2RXJLZa0or6zGU1GgPgteGBKZ6cuqHlBvmElFEXyOmjjUP/AKm/7q5KoSHnmGBJ3m+dTo1sF9DQbDPYaqpnQucgCdeT0cdWNpK7M1etUZzQIOK4nSw;25:Y+RswDaf/ij/Wij2Wo3wA3rWITRnUptRXol86B748ur1Kp72Q2vKEV7pbgleVSr11G4o7Efod7snuKXptwk2aFLZuGyQHpPYAT3RifM0mTWZBs1Tkjg2KZXv7FkLJiEOcfWEWqGBnRh1bC0gB1mntbQQdg0I++CeH8nKVBm15HtsVwjo2cROV/hWq0F0Rr1SiSdM2vjQrOxcFXMVSVn8iX2wZUNCPjBJHCvmfh2dH5OrYZ0FdYXLvPvnXyCWVCxFMU6UOg+JtSWwvYm/ZLLMq5CyFlsuImAsqB6NqdnDQd6z0XVR80Tx0GokvFJYrTfVAVsyWd9hD6hf9+9NLDCddA==;31:y3HFCfzosWn69iUWf9mHkplLGbSMzDRLIjfJlYlp3kow2E2ctgf0lrlXKaYt3X9L0W15lDZiX69L2ir/ndX17zDpg8umKyrwjVSnzAvBBQlxESyv45u0TFZlBv5oOaXe41sTAZf9U5aEj9ZS67v+gnM8Q8LVZO7/OpfZHK+Gj2CK6KAcO0zW7KNdHjEXiPnjK35tdMsVmxvXaMdgQy4wrhGjBxW48OI7NV4pCFyVLtk= X-MS-TrafficTypeDiagnostic: BL2PR07MB2338: X-Microsoft-Exchange-Diagnostics: 1;BL2PR07MB2338;20:bkcwKKGedPwiPdhlI4FR1W1cldBHyG/WnSyHkivGMVjvzNxXElKLWRy3mUCtwq5iojlSdyq5CEXa2cFC2XlaJe/A640Xz4C6F6hI/J6mbkkv9ujgW3yTSQcA+Q8y9h/DdBqOL7Ymre9e22pOB7pBZ/5Ms8baWy01wLnzI9rLkdxBQvPfgjjFl0L5b9L1YPxvh12eytJSpyzZL4Jvl1ssW4rPn+dDQ2iboQLPF3VFvRcaDdRy8IqF53QOo0J9YXDZfA28mcviKAIsGVSUuybUVgkqu3xes6i9/jFbtGMTTmM7Nmj1mlvgp4c1uUH4meXMhULPj0KmdlNmefaOwuyQcFkzCqjBcAn8f1EsVqMWVfCPsfmCNl5AXB03X8JIwTooVVMgVbcfYxpk+0Ke/0AgfN1TvWjkX2HqGQTyWEAhACl7WB6T7voLgPoB4PL+zrh2yN3urGNrAVzgE91Bm/kBVde5IOzEB+LLVfJKozRnpR9zhHUuXo/S+CCQiX0pJ4xU;4:jfKIj7jU0cEcyrN7nk6TgeIM7zVs3GyaKvaSILXT8Hi1PgFSVfGDZPIbRAjXG1hJgZ1y4lDWkvmYBAo3ZvaZ78d2wm8Wcf7dauJu2eSCbQajpoqV8d5lbOQkANnCFw4lliTVvGl9OYwdtMppE4yZg83iWPdffh0cOH55OYCVzu8zKLvcraVKcsPIq8HlTcj8Gs8ACxEvRz4mv/kUCvWV3EWPJ/yMlDprNJL7KuOotjstL9mDLNlNkCRRxW8/usJh X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(601004)(2401047)(5005006)(8121501046)(93006095)(93001095)(100000703101)(100105400095)(10201501046)(3002001)(6041248)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123562025)(20161123560025)(20161123555025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:BL2PR07MB2338;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:BL2PR07MB2338; X-Forefront-PRVS: 04163EF38A X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(6009001)(199003)(24454002)(189002)(7736002)(54906002)(47776003)(39060400002)(8936002)(66066001)(9686003)(55016002)(101416001)(229853002)(23726003)(1076002)(3846002)(6116002)(53936002)(86362001)(478600001)(6506006)(305945005)(42186005)(4001350100001)(6916009)(2950100002)(33656002)(7416002)(5660300001)(25786009)(6246003)(83506001)(97736004)(105586002)(106356001)(54356999)(110136004)(53546010)(81166006)(4326008)(50986999)(93886005)(81156014)(2906002)(8676002)(68736007)(50466002)(72206003)(189998001)(6666003)(76176999)(18370500001);DIR:OUT;SFP:1101;SCL:1;SRVR:BL2PR07MB2338;H:rric.localdomain;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BL2PR07MB2338;23:nsTYLRUJEKBg6m4xkKGPlz+gys0EMCtigbfBy+xof?= =?us-ascii?Q?Q02T2Hx4wycuPCT6nwhbryy7EtgmvU8ef/DqpUg9NX3x5o4Hf4jAm0dYX0gW?= =?us-ascii?Q?Km5Rs26JZISBjxGwekPhRGV/lPhTeJDb0vfRbxOFB/FFpYwoKMEekXAnrOdV?= =?us-ascii?Q?1/WUEAXs9pCW/NisNnL1358SnWQAKBpTJYRYatnBk7+Iz6k4dAKrgVmhpDJk?= =?us-ascii?Q?vcSKHKgDWUfMD3t5yzimIlftIBxd7N0DHGiAvJz/kLdWfye/ppytF6y8wiYo?= =?us-ascii?Q?hCaAn1gM0Fu8Vcmaoc+EPJQ87NC9D2USDH+pxsYko5N1r/F1JFmkLoUVBjPb?= =?us-ascii?Q?08edjQLKJjQZjmBLkr3gfWLiFejB/JPtwml0GrFxhQ2QxHazBLnCP2735ygj?= =?us-ascii?Q?1KI4LiXwRsSAUCbSVDuQ3kGIoDJNoQHmJH8A+3y2kQVh6hYHLUoYD50tv00y?= =?us-ascii?Q?5REoJkmF3lMrNQWfmXQVYVKn8ZRyYg3gJeRtJb3pVFEzrYKZ0bfPruWjFPks?= =?us-ascii?Q?8CO8gDShfyfgsMKeBpMS0JKjWgwS/5wfNkJToV3QXIbfaWrnfc4l4E052W4t?= =?us-ascii?Q?X+0V1cjUtALQg4MBxpsscxdDexnq47pBkrLehbye9NoxWJMzVwaFycB47D2e?= =?us-ascii?Q?2zdKI6JS2qc0PB00MMzoqZ5RikVJMHAM7GvmwVHe4Zgr1aKK4L64KlCKZryi?= =?us-ascii?Q?BjkdzaUXUhh3cqjWDZLwactvFdcjyR9zQTzZKl3SWHvwKMU+ASylwAphg78r?= =?us-ascii?Q?WFIk6dHVqg1a2wPyTURNuquUhuLqpwntG+qVJBpl4r69uVPnTUCpleJJ/UBY?= =?us-ascii?Q?yCKDxTJiijxaPbtK+OMfVU9OoZfUcbejgO3feifr3fxJOVhCrVTKMzXdqiHp?= =?us-ascii?Q?wXuNij8Hj2cyC9cXyRkbEDv1zPpkLCohAAvYU6EaMU1yRJXgyu5+3Zk70WhB?= =?us-ascii?Q?M31yH3xbL6ibu+92lNv5kZm8YngU/jco/7FcjKxIFzBkZcGE1n1aYgTHsSn/?= =?us-ascii?Q?Usg4KsA9XFaAskwyb/XzCA3vQD90TuAlfCk/7gutTP30t/1PCI9Rju99MC6f?= =?us-ascii?Q?uqw4SzYRigS/hkX6W27pkeZop4Jh9AZ9+f/d4DNODjwhQ65qvhV+MPMPb5+x?= =?us-ascii?Q?aN6Jdtuu9bxfZ1Q9g1Kww4RKsV6aPjGQ2cmefpOtSaAkWTwkTg1AQMkf2AGt?= =?us-ascii?Q?giGoghHd3atr3caprGjOLaiVf7DsiF5n8gymb1qhQIv6ByEfH+ACrnG+zOCG?= =?us-ascii?Q?WhlvlknuYwaSbgOKqHklbf8WKFlySJWSlHu273D?= X-Microsoft-Exchange-Diagnostics: 1;BL2PR07MB2338;6:55NrD6EOtxkmtGQC9ugV5tVumVl8t8O8fapq9eG67maDP/Hv9gtGGe4m8iY7n9d2W0s/xO7ktQxGJ4dgRNYwecJaZji5emsP5hKgkECHTT86v0IrePvaHG9dP/Wfq59Jl3ijuD3U28q9TJrCitIje4yXkgqpToEZlibAA1uRMRfEx650AIPzb0FI2kc/0L9cxBFIyAADMBPFDMHDATUL++/02ObVwAluPXpreIFNHQayknDI3YgRZuSKvGaysUQ/nHK6wRb8Z+2ffd+Yx1/R00tPK8KCXITqd+po+hReb1kQLbI9ltxKqi1Eug8o20L3DegrPguCGmY24bhwrtSe2A==;5:EtM8ZmlRIMyTS1+RPf3jjjloCV8vT7lYuRbTPki+D8hGt42qLNWFgrd1k1NAmaXPeVMVBKz6ft9PeITkrQJLBC+XkvaIfUoN5QCqSYw+Tagu4g1+gc9ULr72nzfhnSpOaOcrZ5FK7hKzW+gGXu5Djg==;24:Uw4sjzJBcUdnVs1XB/2R+yDacYApglip9okb+Ne8oVQexPIJaaYXE6JLQL3lOX/Gi8tgBh/g7vsF8p+hlYJjMHb783KeBfHWUx+8xAksfIM=;7:9yjQX9k1LPYwbtuwm30I3RWAK1T5fIMpuBaPbrGYgM9R6KzLAm4rd3AN13PaKYrjTubCM1nCxX3HDoBLYRJtgfi6rAwLH3fwV8Uw9JKS+AYZCSgkKEtITFLAs2N8RoFEBKqTEUqcZP2TtFgx+wjZZpfStJHa0M6hdwI13QQKGdJ3JOXDHAj7CDFC2On20bq8Uh94mEUlOMda11dHaiTk4KQ2M5ARpeNHXv7UAeRCoTc= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: cavium.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Aug 2017 10:53:27.8766 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL2PR07MB2338 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1484 Lines: 35 Shaokun, On 31.08.17 18:29:01, Zhangshaokun wrote: > On 2017/8/29 20:47, Robert Richter wrote: > > On 29.08.17 17:26:00, Zhangshaokun wrote: > >> On 2017/8/24 20:03, Ganapatrao Kulkarni wrote: > >>> This is not a full event list, but a short list of useful events. > >>> > >>> Signed-off-by: Ganapatrao Kulkarni > >>> --- > >>> tools/perf/pmu-events/arch/arm64/mapfile.csv | 15 ++++++ > >>> .../arm64/thunderx2/implementation-defined.json | 62 ++++++++++++++++++++++ > >>> 2 files changed, 77 insertions(+) > >>> create mode 100644 tools/perf/pmu-events/arch/arm64/mapfile.csv > >>> create mode 100644 tools/perf/pmu-events/arch/arm64/thunderx2/implementation-defined.json > >>> > >> > >> I saw you also used thunderx2 in tools/perf/pmu-events/arch/arm64/, how about John's suggestion > >> that would use vendor sub-folder? > >> Of course, appreciate maintainer's comments. > > > > this would just add another level of subdirectories. I rather would > > prefer to have a per platform dir comparable to what is listed in > > > > arch/arm64/Kconfig.platforms > > > > Check it again that not all vendors have specific platform config per SoC family, > so this would not work for us (HiSilicon) and maybe some other vendors. Even if there is no specific platform defined in Kconfig, you still could create a platform dir in tools/perf/. There must not be a 1:1 mapping. DT compatible strings may give a hint for naming then. -Robert