From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752436AbcF1Piz (ORCPT ); Tue, 28 Jun 2016 11:38:55 -0400 Received: from mail-bn1bon0098.outbound.protection.outlook.com ([157.56.111.98]:34656 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752128AbcF1Pix (ORCPT ); Tue, 28 Jun 2016 11:38:53 -0400 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jan.Glauber@cavium.com; Date: Tue, 28 Jun 2016 16:04:59 +0200 From: Jan Glauber To: Will Deacon CC: Mark Rutland , , Subject: Re: [PATCH v2 0/5] Cavium ThunderX uncore PMU support Message-ID: <20160628140459.GA27541@hardcore> References: <20160628102419.GA5425@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20160628102419.GA5425@arm.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-Originating-IP: [88.66.111.179] X-ClientProxiedBy: DB5PR06CA0014.eurprd06.prod.outlook.com (10.162.165.24) To BN3PR07MB2577.namprd07.prod.outlook.com (10.167.4.154) X-MS-Office365-Filtering-Correlation-Id: bbf548be-1443-4ac4-962b-08d39f5d3728 X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2577;2:IudxYaG+CIWhr49LWZ0MCIV3wUb69XB/ftjgxgrSYja4KKl1aAW5tA6NyfH+oIgW5HBMVuvUdf6D9om4VwMGVEIdj+aAvXuZd8PKWlaHaG9MISlry4OsyBRKgfPN4rCw3t1rp28FeMg+xV9aJVY1Snv66OBz0bQfh++lWWnT2ApfKvByTGYrzN6/f/x7Zx1F;3:Ca4slV17WGzabxIXk7EqcXRtsMxmWHpvbbteNMYg9tAnDNyKcTNlxNRal+w1b7mymNFq0hfw0Nw1Un9DxlambSZ6GBdpC+VLHTPUsnDS62lo7MvoWhwtoLMKZf3TYtyb X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BN3PR07MB2577; X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2577;25:EmZ1uDPmhNi/+micqGA7iAWMfE9JTdV9D7elOwLjW5N0vGTC2qRyT9trLu4ZcEB1wGNbtQpbs4yXmrUqpgene+TTwVEWXZvBa1yDhx+FGA390JjYrXJrnMjdF3MXMP5G3wSsZMIHwaI2a8u7SZHSER1H0j+SC92v8HJwT2M2iNgdxlyIziwS8fZHiKSaMNJyR8rD12NLaeB6jUPZa+r98OydE21PAtWen1SzqwVxzqKVffjE0F30Es132onKK63EQSP6S2CTsDfnVwoTj4xCEe1mfE6hOJOXuOaKOg9Qhn+kkEEB3SB3sX05Iv1KLaT65VLzPrcC+uW1cp2Ztu/SjyGpbRTR15NzR/lC29kJJiPXkn6wgQmcmk3z9ROlrHdAbA3/YDtlDJilEyzZG161JgOLKKgQ8kEiMEMUKLJPmUxngITJNiU1rOnQ5pBsfgGsPmqeTCWo2reh8Yr2wLqPQi5cSvrrOo5AYy0lpI0Y6lQfk5edwpnKcCw51hN3eJqRKT6XRnH6v7/pa20mJZOMygFxZ080EvRHKIph3MigCjUOpUuKXX6CoL72VZITrJ1CyMucsJ/cWDZa3lO/8dTPG2EJUV+HTFRlC2isbao+wHmrSFZ2GVTRA1GXrbbRvDt7Dx4kHm7cbrSav0aJNsBrGNDTFd9NZr4kj57fLfgQqYUeIsII5k75yShUVfq2UBIoIVly8i/F7eD9ceeXnku/FQ== X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2577;20:mG3ODg5PfwYbEvvqcVfVVHSuAyl7cNLmmw2CR4WsmUWzk8/6jRjYxfpB17zU89KpO7Gn8bCtwtBYbP1EQX49IC39K0WiQPunL3WYPj6iat8Z19g55rhvyFDSASsHB80EAKmqen4qMibT1J27W3pAWVJp37I2lVRwA1cAAFTUZOKBEGNiko8zYqijG7TkoKadx9fMYW4v3fzIqXNtym9EabSWUzFWAb9GXb/ADX8AsGjmoStPxfCES0vXGID9QElrnWrOKxl7XaEIovxD8AhHdPHmkiZhx/uGocTPspVZd1E3MPsLYrJeC1bFn/DP5Bg3v0g24iXFpIEicGBYJFGFlIXHwT/aCgnZKpFHfB8/RLomJ4YjUaktUbEdwk6A4tqXxeUi9FVszKp5H3FMp+ZpsyhblgpL3XKVeBufzZMBhJmktCRoCULmKP6vKQdwen+fGnXKwTTv1Uq3h3k1F23vR488nBs3Eba8IZ7QKkGgrwWBTCfU0zxSwSZbDjCB2IaFVR+zCBQv1BcmJvubRI6A118Rxez+0nG6sfBh1Jd6PBWOwo1sSwRaOcCJ/l8OspDzLTcmTodaV/VOL8wE6Sz6QLeYFCkX47wx5SNTX/rQKco= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(8121501046)(5005006)(10201501046)(3002001);SRVR:BN3PR07MB2577;BCL:0;PCL:0;RULEID:;SRVR:BN3PR07MB2577; X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2577;4:yFIpr1FzJFpRePvXedBeFfSTcv+F0hx9GHPQPZmbU/KPi0zB9gqxkA+cvXztwH0C07C/N9HnXyyRW7dYgoaz6tkCEC3DYc8ZYwqMFLgtzxmTz3N8hqrR5hPt3Zzx9HfLIVm16oKR2C6xc9C6tQNjQuClI+pmY4Trr8Cr7vq9W+dg3d7VezR2n8BlKjfpBnKF4AhGJ7RrS/Fm6KmeQLunnxG0/eh5Jh651axEvcxNeUm+ggEW31r25kgIaqMt2dtDo0BJFTD38sdG+aremqlHdgsojto7XE4y6axEtPPHiyi6Wgcsg5OPzQMjdXu4KHdEB8WHXdHQjFOXYdCxtrnRr0BGkJWaJxbWEbRaEjJorKFDhtMz0gbBq4Zxsc9ynUWW X-Forefront-PRVS: 0987ACA2E2 X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(7916002)(199003)(24454002)(189002)(2950100001)(77096005)(97736004)(305945005)(1076002)(9686002)(46406003)(33716001)(106356001)(50466002)(7736002)(8676002)(105586002)(189998001)(4001350100001)(81166006)(81156014)(110136002)(2906002)(92566002)(68736007)(3846002)(4326007)(47776003)(76176999)(23726003)(50986999)(66066001)(101416001)(7846002)(83506001)(42186005)(54356999)(33656002)(97756001)(586003)(6116002);DIR:OUT;SFP:1101;SCL:1;SRVR:BN3PR07MB2577;H:hardcore;FPR:;SPF:None;PTR:InfoNoRecords;A:1;MX:1;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;BN3PR07MB2577;23:9q2q/eZvNrSeEceIVnl9jP+XRzg0XaSQ5Y2MBv8w+?= =?us-ascii?Q?hdzvQJ1Sc3su2z81EROboCVKCTUN9sRBJsnAeQO81vjqaU424kyvxGA9aZri?= =?us-ascii?Q?3bmD6+DYCR91JPsesgg9tvCuXV3XfSyccqzMWm4PCEbQJeMgHQVhuALVUMz7?= =?us-ascii?Q?XOXWxbfojYZuG6QdpXO+aImvLibV4DGwiIrZ/8O2wpmTMAYKLue8RK0zirvb?= =?us-ascii?Q?6zjKmT+L0cyb3jngbvtMsLk7OWWbooF5uE4TRc0YQGID8pXAbmvUXvSFYfRX?= =?us-ascii?Q?uWDAMoEUFkVJeHAZ0oq03Qp2fVTjVecwJ2aFLketjubqrXZDgdcLXnqqgIAq?= =?us-ascii?Q?m8a/atnBlpRYV9yFmK450B4MmZF/q+RSF2S+0XCgy/inmAjRusBHc0jx060P?= =?us-ascii?Q?fY6ABCXRuSd/QUUT7zc0e5nCtETngZeFpanC7laVLcYIt/ImJkagw9aIzlf5?= =?us-ascii?Q?dkYWAxJtKipCrqKKNh7FrtQuIeYi2nokfLr5ahN4fvmOoYmhgP+Vc8Lqr/sL?= =?us-ascii?Q?yaESDQgLjmQMWdhHClOICCmfa43v5P8LtlTpN+yqhXRKbxdVmrHUhvM86R4C?= =?us-ascii?Q?pfc8yeHtjh36gw5lG8TQlX+Bi0ASU8I/ffCng+9aKAbbIZ989PvLWV9YAjtU?= =?us-ascii?Q?Ei7GxOzDUXyHyIclIwHx5lUorwoitN2deXAyDPqmg9BPxWqcF6S435jUQrp1?= =?us-ascii?Q?gnFnx6Sbk0VNwuKc+SllP15zkOjWv+0EQg/P5NE9L8pdTxNX8A/2+Edg/rCB?= =?us-ascii?Q?/27KXUOB4kMUAlsM651bvk/NagXV2Y5Cr17D75WYEr8qwPm/OEUy5esVbMji?= =?us-ascii?Q?6gZDZulGIvdiSA0Jj7vIv198ERxggK7zax7kY+EJmt8uW9fwIW++ASkQNTh/?= =?us-ascii?Q?TqcyRvAs61AXego7NfcDYNb5R/4xlYQ8NXyCXaKKqETnwvQ9pXG3KfmEBUWc?= =?us-ascii?Q?beHakHHfMTpjL4c6utf55l8bnZJgZZ2N0643Tp1qruiFnrY0cJEM4X7p7hmK?= =?us-ascii?Q?mbzBUTRhLtjUfwMWcJ7ru9pDd2QLY7olfHYYmv4ZRPKkBqexH5MfBKaO3vDf?= =?us-ascii?Q?cMYO/mU1nvJztIXGI55Hw0lJYwR?= X-Microsoft-Exchange-Diagnostics: 1;BN3PR07MB2577;6:cLzdNMGCuRD5wTE49NlxxbtR8aEZRsW01yrVhluKi8ehR28Jt33hfIoW8Tw2gTFear3cDO8JcvsER53C34tDK+mOy+HyoZhhzu2vDUSWXkWI9Gsr3Lx6a/vZD5AxmU8Ir/y9u9vA/P0nw8gyu724MvWzJMzqvijq18sT9KR4kf8Xa+diCTOX6G8Nt7IN476K9NjVP9SBm/ljZ48RMkmEBIC0L4UqRKAYpnR0np4qvFftJX6FZrmWiEf5ksLf5m98jV39p3VusjJcN9fWOO279eXriL0tYtEIKvhqiNyeMXx4N4cS/GQS3fsLrLvN1yMQ;5:+wJlEjSynHa8wBfKb+hqHeZ1U65qvZIb5jc39TV23Bu01eLt8+v4tOI+DOIHScqeN7i2RFGeQAPBhRhHH5Pzon+WxK1Qj8bkf6lxmWgdDjMGRaoKFYyfbq8C2q35xgO/SxFTNyb31T9QTXVaJ6Jkag==;24:c+L19RYcWoD8nzbqAnY8SIOFapLOVkC8Z+I6OiA22stlu9zW50pT/+T+gUSRVziwfJMl0Uvgjz86Q65kGWt/mkUrwxoX329BlDz0HoJOor4=;7:FCtTbmxT3P/twVMfW88BJgBC78myZRduymY68Pjpycy+ON1vYf2hhjglvM3h2P3VRKdyBfLWJbwQmynK8xXZ25oe8VsGhFe6NaYrZcSvy0aKWdoBsHymUe1SjQodinIEfdUkEjYnGVNUZ4NxOyADekV47R28aL1xwLRRHGthUIxVWq+FEGmpQ0QwTpNj0OeYbnbWr+8y3QZ03FojwMCRpYb5IZz4AUvwidLIpQ9clEkclgcgoznT8bR2ecC8nraYAo23i3C7Th8DDtibCRFUDw== SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Jun 2016 14:05:08.6285 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN3PR07MB2577 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 28, 2016 at 11:24:20AM +0100, Will Deacon wrote: > Hi Jan, > > On Wed, Mar 09, 2016 at 05:21:02PM +0100, Jan Glauber wrote: > > This patch series provides access to various counters on the ThunderX SOC. > > > > For details of the uncore implementation see patch #1. > > > > Patches #2-5 add the various ThunderX specific PMUs. > > > > As suggested I've put the files under drivers/perf/uncore. I would > > prefer this location over drivers/bus because not all of the uncore > > drivers are bus related. > > What's the status of these patches? Were you planning to send a new > version? > > Will Hi Will, I was half-way through with addressing Mark's review comments when got side-tracked. The principle question these patches raised remains open though in my opinion, how to determine the socket a device belongs to. There is no first-class interface to ask a device or the firmware which socket the device lives on. The options I see are: A) Using NUMA node information, depends on CONFIG_NUMA B) Decoding the socket bits of the PCI BAR address C) Using PCI topology information A is what I tried, but I agree that depending on CONFIG_NUMA is not a good solution. B would be easy but looks not very future-proof. So option C is what is left... thanks, Jan