Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754701AbcKKKjg (ORCPT ); Fri, 11 Nov 2016 05:39:36 -0500 Received: from mail-by2nam03on0068.outbound.protection.outlook.com ([104.47.42.68]:47144 "EHLO NAM03-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751606AbcKKKje (ORCPT ); Fri, 11 Nov 2016 05:39:34 -0500 X-Greylist: delayed 10907 seconds by postgrey-1.27 at vger.kernel.org; Fri, 11 Nov 2016 05:39:34 EST Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jan.Glauber@cavium.com; Date: Fri, 11 Nov 2016 11:39:21 +0100 From: Jan Glauber To: Mark Rutland CC: Will Deacon , , Subject: Re: [PATCH v4 1/5] arm64: perf: Basic uncore counter support for Cavium ThunderX SOC Message-ID: <20161111103921.GE16907@hardcore> References: <73173d6ad2430eead5e9da40564a90a60961b6d9.1477741719.git.jglauber@cavium.com> <20161110165405.GH4418@leverpostej> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20161110165405.GH4418@leverpostej> User-Agent: Mutt/1.5.21 (2010-09-15) X-Originating-IP: [46.223.65.110] X-ClientProxiedBy: AM5PR0701CA0056.eurprd07.prod.outlook.com (10.169.145.146) To BN3PR07MB2580.namprd07.prod.outlook.com (10.167.5.8) X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;2:VG1NRRQ2cb/tkWNdxvp00r5r+el/odI2n2HGAwoy6Sv1R5eSKzJcYblvKRgA4C4beYeWPHT+D7gwkKenZnu/H+1pqiFOm5c1Wuioj1wDgqh8MJochmIQVbWBzCxhS0afAqAJmeAxqj2szQhB/n5YfEYzfrUzPKid7dDToY0frRg=;3:5FUB2ukEn47+LaLu92quw5JXtttsIgmH6M6vInecZQ59HExt3Wb0jj6xbzFlxgd1bl070JUcdkDoAwkJLGqm5BPzvq019z9Qe3oqzR3vAG45nWzwm0Xo4LxesNF9CZWNMUgUa9Zm3k7we8OnZIFaB46ogKYDasAf7fvkUBpMU8w= X-MS-Office365-Filtering-Correlation-Id: fe2c5a90-3d53-4e88-70e2-08d40a1f05a8 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001);SRVR:BN3PR07MB2580; X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;25:p+rNtfOrGW8Gw3BjcGStxKxvGK2zZTwa3TM9L2uALdBUvKYaxe5DD14Su3AFNSSNduACSUp4qqONzxGOGA+xJ2Q3U8JvgvkknOwNFQ3faCHLOcf6lnXqrQlbix80GxppgfumxxPVgkZvMEHyoW4+2LRwwZFuIAqxcuqDHYBSzFOim4rgSeLBpQXBuRXvTmvOSEafrW8h03HoNgMRaSFJROnwJBIBCad8c+0lOclHz5cY1Lb4vWhqTQSGYLqwSkQjzMa3KE9C8NldeAmqI3tKHqAYeC3+4CLIul9wdLnjqLcfP1HM0US7e1eyQArDPe7deCZCFDjHKWY9i3Eq9PR5KxgLAzlvlUVlWwNUTRyU9WRUdxQGiwy5wtC+u3Ad6+Cbiz240C0VGehF2Wrbja/8h7vVGljxmcLfAxbfovfZH3Azs0YT7mjY/EkVxCTNVKPiOpz2PIxtxdYxOeryULwrp3+GCZnjqCu0vMmQaM+2psxezLPciSc8k8OTWmEuOX6oQVHwkw4K/lYbjFc2EmV6olxEvMJwx0EDet77BBHNE+rPnBy1e/944fCN4HuyxQ7QqOZPfYabHIIaEHVnVyS+eSlnA/7TAj2adk1djQQO1NiQfN9a/oGL8FYggLpzXo57L1PZo1tIiHshhJJryuDtmIW0j/LQFfgOyoCRMYdGg5nIyl6rWd6IF0Jr29RakK4iNLTWG5VmmSEIr5REA3e/AwtcfQ20pYSpRnVL8SK5Y78nChWMcnsVC+oVnrhsX4Xn X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;31:I74u6FehSbsHVYs5bNPp/MzZSiSbosXxcG95wMHseZmUJ/t/ZB+2bxyJ0o3BoLv4C969YpbpEyqzbvwM8YfRnssYtfj4jbuJ9pfgqoMJM0lFY1rxZ21vQMgVAbDw2k1aAOwRXUyzNmUHwfMHLSZ1AUTnATJ6w3KLBwXKoXlkhHM/aRg8i952sYz+Hji2Z9WSyitPbMC7u2hWHdZksUAOIdLhXNzwnzZUQuJuxGJ/VHMsagG8ofuhXGGLHUNPsNin;20:U+6w+bfWNAiGQQeqkJR/FCrE3wRsSL35OdiVLyDrul5ZN5IPCaS6TXNYgUJ1zgsCy2eJygQmzJvuy33+tBLfeouVYlh11a9it3djxwFmSVqjarDNc0rZ5cOBCBM72V8zJ7QuoQ2Py39Lm1gU4jQyUkqQ39Qy4HSW7utshkGVqmrOSXdsqCBL7l79pjtsJaC1LpDjklnWC/lgthIP01jF01hdmR4vTOC8Sr5vN0ygVwYca0Mw1ca5y87zC4wysBy8H23y9b18xv0T9B23AUPERa+niFmukojYYhsIQC0xlMBVwTJpeTplwj2XHgl2Q5vSaLo0hNRzJpNVE1Nn1R1t75iSIWr8qtos5+nSWa5hNqs7+bohZckkU0/6pqX9fgVTv6UCPP8o8W8RoZeW4UeXun3/BYDUBqUObUM1mGEY2gCvVvdRTDjDlzpCvkVTsu1t0tc8/7JV3Es839gRmanWwzR9I2sI1O6XOZOyRm9RIoGRvwre2/5extbn/Czxk8xB3jfh23XqsXHejKi6RK+5kdu/BQ7UFr2OIObyhytJEHVwuurTmfnRvnF5gH6QCs2TZbzQME8o/pHBiPslmBhtOhk0Zz0LS2wxFcHFM+qLez0= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6040176)(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001);SRVR:BN3PR07MB2580;BCL:0;PCL:0;RULEID:;SRVR:BN3PR07MB2580; X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;4:HYVPV1JMY+Uvmb42qB0+4Sr1AVMpdllEdD5ryosAca/yLzqb9arjzSbrBJfslhXCzfaeKGCTzRNezoCiswqjelUzQYIUwjGGQMS/f1I9+3eB30xaQC1mcPgCwyI9FSS63TEKSONzmvmbV3/ZBF5L5kvW5yT7PbPYGbbWusX+yn8cmTIObKl82qa59Gtq7rm7C3f+CnYojOmqDUZ2jraZGVlOW8CXBfJQeskxUWCzpBqbohVq9e+fqoGO/C3fV7JkZ9q6UWccyoIijnlu5WPKPTNoBGqC28cMlHrTrrdZ/Jb19SbUEWsIQS5Vh96TwKJ9rOJ2s9W8BdmenJn0sCgmmIr01+Kq6ZT7VgHLj/KwwV/HvjPU9vE3kPsd9KVtagccWH7DIJl2R/UpRkZ7qDx2mQ== X-Forefront-PRVS: 012349AD1C X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(7916002)(189002)(199003)(24454002)(42186005)(76176999)(2906002)(50986999)(7736002)(305945005)(106356001)(54356999)(7846002)(97756001)(101416001)(9686002)(97736004)(1076002)(189998001)(33656002)(6116002)(105586002)(229853002)(68736007)(3846002)(8676002)(81156014)(81166006)(23726003)(4001350100001)(5660300001)(6916009)(33716001)(4326007)(46406003)(50466002)(47776003)(77096005)(66066001)(92566002)(110136003)(42882006)(83506001)(6666003)(2950100002)(586003)(18370500001);DIR:OUT;SFP:1101;SCL:1;SRVR:BN3PR07MB2580;H:hardcore;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BN3PR07MB2580;23:XSyEB5uCVvckCSoUwPKahw174MzcmbnskGTFlQsG9?= =?us-ascii?Q?zrz+3vk9aq4m/Wb3n6gIKmhpsnNJzWqITq6PbxyYlPCWYZEwVgEefdUg7EVx?= =?us-ascii?Q?zF3zH7O0oEFskDXf1bdiMRYoM3IV5CTInnDznIThC5GSZYacWUOUBjJwLhdM?= =?us-ascii?Q?HStyKD/7zc7dT37IICdytcDGXXxo2H1MCADEkhv/7n2PkkvAdPFP4AnoqFR/?= =?us-ascii?Q?zjsQBpe/aiL7XOB46dpct5erzJTNQZyKwS9iM+9YAXegZ2MaX8JL8AFr2eTx?= =?us-ascii?Q?ZwsnAb5YwSvv24zUzHUrpuwPgKtJ7Ml84BQLrJ1CTdHOAByDdkku/MrS8UrC?= =?us-ascii?Q?Oq3Jn8jjpk4PgMDaTlVa7g9vFmCFw8d/bwuqsk1LYKbX9cdCc/hWwtiQK6kq?= =?us-ascii?Q?C4c+ApVBXVoUKmlNTw3LmTl8smuEBbQhsLGeNsJN43JSiKm2Sx3oVkk5grFC?= =?us-ascii?Q?u81CU+aFTRwJv33teOQY2gse9dSM4A/+2YLDQJ9fMyZLUr3OIz59l06Mgvme?= =?us-ascii?Q?O76c308xMGwZZAzf/+0t4V9Wjanj0cSrO9S44JZyvH3qWXZ1d5tXxCB8VsyN?= =?us-ascii?Q?254cIuMoO0+aM/Q2Et8O9CfFxH0a0CpM+YwJnL9XlS1ABF/V74L7l1JoQne/?= =?us-ascii?Q?xXNchzfTEvKS7NYXws/CmsiCoPb2MFc37iAvYPKGCS7JI4Qwg/vqjF4YXoVq?= =?us-ascii?Q?RMhLPHkohK99ynoLzQa9BK8RwFURR2KTdx22f1p1jT5t/5fG7Zv3+/zIj3VJ?= =?us-ascii?Q?lv9Odq+WRmJATIZlaCXMhZbgElwR6ujeXqx17pLXdddAtni1W2ofQ1ABxr69?= =?us-ascii?Q?w5yHT9ge4lqPhLIboX686BCGyO7gLZdWEMA19W05UZpZIxoqKS1wBKDMOMAl?= =?us-ascii?Q?h/SH/rzohlSzIZhNzbP7GnuoqERZHwTWQitEldg0A+y3p7njSiWKqYTLFevg?= =?us-ascii?Q?v+fLrPVyCk5Xiktay6YOFpkaktFYI4a/DIQOz9LkqAxOZ3I/3Q83RljvE3hd?= =?us-ascii?Q?A9+YV1qCWXVHHwVtKZzsxGhvq/+5mll9x8/Xl9rl61FE8UOn9IRvyUoMt5ez?= =?us-ascii?Q?v2GZYpq5blzPzzS4DvOvxUcJUE0a8rMluQ5TX9bV31Iu73CdDs0ClQQZ1P5V?= =?us-ascii?Q?xqazDKsnf4VzSk2d7Rn4hDm0LNoS/A5QB9uT6PwpYNprUGIBZ5q9aJEW0JQV?= =?us-ascii?Q?Yy1qCE95LJvLbZ7vsd9wv70JBUp1RWv9be8?= X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2580;6:UY6xy6sGYSZEO9qPCew5H+IQIkPCi2Qyi3/SUHHvfS3r/t/1BPR9yGhTd0+k34lmCS6md8QF+ncd2RjuILZj/5UcAtxiSgmvdnUaUUDBGHzN5RFCU/cgZ23du1LhM/R86Pv+T+Dg+kqJE7SukBZY2JrKBsPOE+M6BQz3xIUA3N+A0sS1VYHa+T7ixfntQlqRzixlSX/PUu/TSqbdQnKerou9ilR1/kNgWVlZg5RnFkzWsBuvai/FfL0m6zE1MW1+OGgpmC6XWMnPMuJbyly33K7mxH3amw92KJHh4+8nOEhDOgPSM6bvNnOFCbVhLNlJ;5:GUqhYuBU2Su81BfDUB37K80kKa4B4PKbcwNmdlG2NBOQ5EqxMa6t3y27xDN5wGikvFMDk1URC/ObE/+8goxxv/JBgUtQKOfVhq9P9Et0qq0/SgZRN9F5z/Q4qrpzLFnUEiCTwHVhOyuXlTOmy5aCmEZXv+dW/CNtu3aSTttgvu8=;24:90dEljs0yBMgQaFIm9Jt6YcaG7DjuCjDknVwlMj1LGtQ/lhJJNaju6tu6fLwFdkj39Nz0+4Zt40hC6uxJMC6TQ6t++178cnTGaTl1P2LgdA=;7:Q7S79/EYuHTVhFK8EeDCih04bRzY3jmKcO8QLgwh037oPv7Cp0IGiKNykbhVDVaoBZfxcWxs3BV1Q47Rim7Rrzv8sE3ZG0PTpcabwKrchLdkBHMqLt6v3WMpQDj6tYhPTFzdetaSAldDN5ExzEFggkorfYebuhje5APXRcOqV2IDJgpVWLzlJbuzith8rSxeFJ82j2y8QcxOG2NSaZw16CVRc7/1X8pawcTH8LzzvsZdGDmBL3FzZtsfJqiesTc/D61kzynAc8d97Smfi/4o9NHv9HxAWkK2spS8G+y0D33UO2/BFkGh/z6Hbphm7eaLYYQ2e3Bjd7MTBl3puVQtklwbvjlKeHRULg/y5i8A4jU= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Nov 2016 10:39:31.3146 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR07MB2580 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1551 Lines: 52 Hi Mark, thanks for reviewing. One question below, for most of your other comments I think we need to come to a conclusion about the aggregation first. On Thu, Nov 10, 2016 at 04:54:06PM +0000, Mark Rutland wrote: > Hi Jan, > > Apologies for the delay in getting to this. > > On Sat, Oct 29, 2016 at 01:55:29PM +0200, Jan Glauber wrote: > > diff --git a/drivers/perf/uncore/uncore_cavium.c b/drivers/perf/uncore/uncore_cavium.c > > new file mode 100644 > > index 0000000..a7b4277 > > --- /dev/null > > +++ b/drivers/perf/uncore/uncore_cavium.c > > @@ -0,0 +1,351 @@ > > +/* > > + * Cavium Thunder uncore PMU support. > > + * > > + * Copyright (C) 2015,2016 Cavium Inc. > > + * Author: Jan Glauber > > + */ > > + > > +#include > > +#include > > +#include > > I believe the following includes are necessary for APIs and/or data > explicitly referenced by the driver code: > > #include > #include > #include > #include > #include > #include > #include > #include > #include > #include > #include > #include > #include > #include > > #include > > ... please add those here. Should I also add includes that are already in the included by uncore_cavium.h? I usually avoid includes that come through the "local" header file.