Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751646AbdGZPNe (ORCPT ); Wed, 26 Jul 2017 11:13:34 -0400 Received: from mail-by2nam03on0070.outbound.protection.outlook.com ([104.47.42.70]:22432 "EHLO NAM03-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751510AbdGZPNc (ORCPT ); Wed, 26 Jul 2017 11:13:32 -0400 Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Jan.Glauber@cavium.com; Date: Wed, 26 Jul 2017 17:13:14 +0200 From: Jan Glauber To: Borislav Petkov Cc: Suzuki K Poulose , Mark Rutland , Will Deacon , linux-arm-kernel@lists.infradead.org, "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v8 1/3] perf: cavium: Support memory controller PMU counters Message-ID: <20170726151314.GA10696@hc> References: <20170725150422.4775-1-jglauber@cavium.com> <20170725150422.4775-2-jglauber@cavium.com> <72145781-e9ec-036f-f752-b4756fef08ee@arm.com> <20170726111946.GA6273@hc> <20170726131058.GA8665@hc> <131179fe-42e7-f286-5bd4-801f4c93d5f9@arm.com> <20170726145522.GC28875@nazgul.tnic> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170726145522.GC28875@nazgul.tnic> User-Agent: Mutt/1.5.21 (2010-09-15) X-Originating-IP: [88.67.130.225] X-ClientProxiedBy: AM5PR0601CA0033.eurprd06.prod.outlook.com (10.173.86.147) To CO2PR07MB2583.namprd07.prod.outlook.com (10.166.201.22) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: ac00921a-a3d5-4163-ee00-08d4d438ddb1 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(300000503095)(300135400095)(201703131423075)(201703031133081)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);SRVR:CO2PR07MB2583; X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2583;3:aGRNmHr1Ck5IbxAP5Ip7CG61FvDxYS9zTnp4IpFJvqCY7WQZCXhVDBw4OYvnENhyTwMLTh2TL1vqLNws2unq5umeyNQ4q78w47JdSO+hWtGp6Jvhiac/RgEN7DaxmciijPHOJAvCEfuI6DVRhZc7t+o2jEC42RL7KRAyX6zdylC2XpvY8m0BGSHJdP8prO9Si/tCyGJazFjEhh34kplj0/sBPnasJHYm3grRBGxQO3i6cvD5Fh2DN1UD/QyBJOHC6KIPejaPtMQy9IILr0XfbzFHXf8yy8IIQVEWeDTlgVC+zpoGjxiSUsR71NCDE4HaijY0Bild3m4nkc75NY4gA12j0Z39zIyCirbp6a1aGpqdt/3w3+JEpZCNhhO2c+ljAf5TC2NuuTO3QEqA6vTV0EUDQDK3FxjBGiZVLJOV13ZeBS/hinUKNPpHqO9QdJhHousSmbf8AVNklM6x3Izo1O8bNdIrbcauNo+3HEGXE0Al2ggoaL8AokVrWWO9lM2oEs0w1yuzg/2Yvzy0gyjxDF2ZpJmYzIxECxTzDx/AfAF2w6CG3I3eL6qioEWZhsQ0pnSL1zDkg5F8UzrMe99x9Fvqr6/nU9cXVt6L+LAbX4SlOBgrL1H6L3wLCjzl63HIxy0/htKh5R1odufOqLQgTofBL0ctAp7B5bl4USM8JNfK4xqRwLVFt+nxfEboPsysi+M5iQ+4ggKYrZA+lMSgkIt+U9VIDafWLhFtVSn2qtg= X-MS-TrafficTypeDiagnostic: CO2PR07MB2583: X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2583;25:/kK21Epg/EVC82/5z08/w26R1ZKOGU/fNxIz+UcEOBq3NWJG6bDAWpJ40/CdiVSrfMRXIM10hdiAWoGVN/acV6ecntJqQ/PHSbiDvZKJWdTF/4/bFleWWYf0Cd2h1QILti5LAU3kAev4jkkfnB/GKZFIoV0/VB9ApgcGx+yLvliK3ZMG05HwUh5lDOwgTOnpR2ITZSqV/GVsy3dSzgm4AHwuZkYZoJtndMjnvpcf21ixJp7XYZs1VlS219f1wkF6UZUggzjamn+KsLViFJ4/UaFncFmefWlqnvVdXgTSSZipydnFlOyfTxQMROKuDQzzOC1wXo1503R52i36Ij3VS9bHx4lr6aOFI+H+USbXtdmr7dFvnOPHQu/Rxj+sHdAUPJyDjqt81KKwJoMmogVEgvs3go2hCfh+KPbwXJWgeQpe1bwvyWr/GMlbvUUv8z+lsSWhmtVvsdvpx6lrhOMzSZdECG2SnDrPS3k+MWvIvRFiPd4bipWKU4N1UEIs5+lotJT/j2diihKR1nrqcutyYJeNJlh2ahx5C0EPtH62a7Ad1mbpltBrOSZ6YAp4/vGTL8iasTO8Sss+Cm1RTwJX6T0WSjna2YWK7xlll6MIys59Hq/VPdgXxdx3N/XbquzATIcSOQeCbalLEcBGUNMm9q+ZoJLcRqtvDXu9/frbZdYO620e54YnPQudo9pntFgLwwtCQxo5BaRoIkACvgrLwHv6+vUVp4aQvrd8a3/0vRJuFej82BoUAn7k5i7CVC+Qjq8v09s8lg7Uc30nNt0QzKK80ncYkq3JY+Zqtc5S28/hlYkwWBl51Aq4LBFGw/aRDFVTesPkUwGO1QhI99LFPcs7fx3AvjwlNALgUVMXWAu+cUE/6/qaQwGRdkn9cy8XWJwe+pFVCo9jI0TfA2xgfW0oRB3eJuqbvZi9hUP0rK8= X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2583;31:aiCUr2X5L6QFJNnnAafo892SXystJYTYkqQG+jU47bZMlEnMNP1izuxihob1YiVhWOd8fdPlLddr9QxaHPQZlVYvzeUeqoeEjZbSo4n3/yh53uIE2MEaPbPLHMhNON8Jr2PlXzJvnGpVBm3ovL3djjefs7iwyH7W+JVIGpKvd/ZH0U4Ow805WMcp0fRVqVONz8h/k2kltdS7/e5VXcnvz1Lxo6U4GM+Gk1Xz5pqTCAnBEDgeFefKjrL4FFMbbEfEUt1aLledlCg31kcB7RSNv105nQ/3uEk+Xn8eZgLCJb6Jf2h+HSK3hDVkRHEUqauxxCStZku+4S2mS0RLKYT5AfLlu6a6jzpTc7bR63aCgyA1NG9fy/8KZZ6KHsjGUVA7wBImLXQsgO2a3rIORfB+v61MxKvwB5r3pIXBYX8lGKizfFZHSd7xnL/eGrzYLLVEy1ogHfW/jPrin15mjkOP2KItVD0d5c/yTmlKsKb+h/PAa/NylC7zPdjYzk73X2ntNOO4JNbt0tdXVS0l9NBFSAJ9yGEBBad4q5Zb/wndELm7fEOxCJG+lk8XznJC7pjuBrXKx+oUjCnzh60gyAEqnhlxXrZhtxUg4Un9HRS6CAfa3uIWilWShbidg5zmjvTkavf2dhEj0czwSIP76q1sdH1d5zapx4Z8t1U0Fge8cF92n5uduiWyA/kRsgKZUzJnwQCqw7z9TvbJVC7n7AABBQ== X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2583;20:0X6l5PBLU+5wAb56VS8zGRS9/5gV8/odbTPHziBo0FxgbLz8Bbxb5AWHAR6Kfq99gMMJn0Dcx91VrxGOhViAuJ8OCY4VhY2XhHQWfTUT4UiLaYvQuWI6fcbSzZL1+2ITuCUno0Fh/snkzmLi+/6WqdGnUbu0Si4xObHrF7FEtujy/qIO1Bot/7NFcJh+KVg8MFzPlVe3l4N+LGpwKjzZu2MOpwo3YxpZjteL1c6QVAjHqY7SLgddOKlK1lYRCYfQrMgAP17VcmSQmg5CbgxSNWEmRgIeF+EkNHmk+Tgj1QnpSoJ4VyAWpLCoZtwQAFbZY4Apc6KrfH+0aBFUD5PVFUZgH+wGxXcILDaodeS/8lHmoiucELAzoMQgMUcNhj/0FIK0izx3Rn3HK/6uu65DO2++oZAZB2hEBZdrmGkCH0dJzksW7gO0/OmTPwyDNax3GyfntPrfH5VxqzLkE/T/OvhLEQes/Hf96UuMCcp28bTM3sD2wDlw3zmpsOUBQlx7hz1uVazV7YcsEdS4xZdtFTW0+2qIz+OUA3PCf3n8M26EKanAzt36HK1ri3U3b1TBd78MVoYh9I5n7dXHB18v47z7Rp2DCGGj9gJyhnRCPH0= X-Exchange-Antispam-Report-Test: UriScan:(20558992708506); 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)(3002001)(10201501046)(93006095)(100000703101)(100105400095)(6041248)(20161123564025)(20161123558100)(20161123560025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123555025)(20161123562025)(6072148)(100000704101)(100105200095)(100000705101)(100105500095);SRVR:CO2PR07MB2583;BCL:0;PCL:0;RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);SRVR:CO2PR07MB2583; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;CO2PR07MB2583;4:q/1N+pN2uVMvOb8pIljdFtYOh3/VJmk/E8pJdZ8eUE?= =?us-ascii?Q?LlbVDk9chEtQ6TjE+cKTZvVd844OjDnHRdK1ac4U4veeeLc+ggJd+AS4Fipk?= =?us-ascii?Q?9Wyu9gINLraXO0L9ZsuIMdYHK5Fta4na/QkZAjMpOQbtSNnyfbghGsmFI0wb?= =?us-ascii?Q?lipSM18Miejubl+eig2VYkjA2XNQ0EBaxkcnn6FnoxxrHvrDq6J3itf5scrX?= =?us-ascii?Q?X1tcCs76wpliKqhIBD+q8a2JWUogZRs1XqrTHlsmKDHTagGbwleuW5dtJxA5?= =?us-ascii?Q?ca/XhufxZdPEm+XnPOIJzTCd8tu3obt8Q3hXOIC0GeGQnPlYuqvMzx+7rJKi?= =?us-ascii?Q?JrF+dxfMc1Om8ueLrA4PkaAmCdrZ96k3kLv0Vu1lIBAHrAUjjhs+4sEwP2VH?= =?us-ascii?Q?wvWJCIMNmSUK2552l2dyvWsxGoTEdBpqmjo56iB7ME/T23kdj7U9TIi7XMwD?= =?us-ascii?Q?CNA7istGWydmkLia0ksnyKj5GK5YtrmOG89FHk6zfoqzYGSG2I3/RjByTPe3?= =?us-ascii?Q?aBv4llTjzLU5gRzEi/oBuNAKv8RXcXHJJGc3iaPmqke8Y0v5nfDVnUJizEe8?= =?us-ascii?Q?PHmtl5x2rNGa1UAiJO2NFb2qN/2Pw74Tgx/IixUjcN2CFXPh2dnypX+vUFzk?= =?us-ascii?Q?+P5c2My/pXP9XkRmrfcq+f7ZftZKfp/KPkPO/A5XtYmAGd7Evhqqa9ja1j7Y?= =?us-ascii?Q?8LmiJrCqVNJLfnJJzhBbGjNNv7spiXsu1M61IJUXrHz9bbhIaGOtd8XkQKwY?= =?us-ascii?Q?QNpaD8380EayHp4nPFy2JUx7N63C/enYDLehBMnArQEd1UnARcMLdp4hegNy?= =?us-ascii?Q?QByvW5M44bGTQ/mIZr55CazHa2WpKQYLkxBljNVKbqlCNcGHvDpvXHRem+ic?= =?us-ascii?Q?hjIyj3UfB5s88i38kZ+67sA37F8FIZ6GVLaqQhGSArAN10tNW/nK5vknKQbZ?= =?us-ascii?Q?BJ0qnujvdjUW9jhEhcOHk3YVBepwD7GHV+LKQvFfn4R2xJERv3f76FPtn+a/?= =?us-ascii?Q?LeDJixv4WJutxTYv+14Mx1U5qonBXqRY+r5H6k8QLoefYCQXMEoQWA0GMAMC?= =?us-ascii?Q?Mm+roxCUmGsFRyKQdIBL0k1h8+UxtvWx/d7NLFKeC07qlFiVO5aZZ93IZngX?= =?us-ascii?Q?PyN5Do7NCcbva2wHhZPQPXTDaOdSsBSGre3xp5AF+QHbyJyEHNtg=3D=3D?= X-Forefront-PRVS: 038002787A X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(7370300001)(4630300001)(6009001)(39850400002)(39400400002)(39410400002)(39840400002)(39450400003)(189002)(199003)(76104003)(24454002)(42186005)(101416001)(4001350100001)(33656002)(50986999)(7350300001)(76176999)(105586002)(50466002)(66066001)(5660300001)(54356999)(47776003)(106356001)(5890100001)(83506001)(72206003)(478600001)(33716001)(189998001)(68736007)(7736002)(23726003)(6116002)(3846002)(2906002)(81166006)(81156014)(8676002)(38730400002)(229853002)(110136004)(4326008)(1076002)(6666003)(6916009)(42882006)(2950100002)(54906002)(55016002)(6246003)(25786009)(53936002)(97736004)(9686003)(93886004)(305945005)(6496005)(18370500001);DIR:OUT;SFP:1101;SCL:1;SRVR:CO2PR07MB2583;H:hc;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:3;LANG:en; X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;CO2PR07MB2583;23:66AFue79Ta4oUbuTFHTyWpwP0FbGsbpYNF+RnX35E?= =?us-ascii?Q?OVWr2AUy0tyOebPDWHbTpxNydCrNSYxSB5rSOaIB5onlHSCmqex5WvfXEpnb?= =?us-ascii?Q?qveyeokbV+kyZwV/NWWTXAQaBgwMCnJMxz/kI8BslYvmVuKMUfaadLactVEB?= =?us-ascii?Q?BgQ59Y4TwWy66DtdXfdSCmzYvXgIOYkJeZFwWgI2JlmUaK3sHFpSXdupT5CU?= =?us-ascii?Q?S+k4ubdWNSuOYuQ8jaTiooq/+x2FKxHr/wQybWHC2JrwG9FyGnMrDZlVc8c1?= =?us-ascii?Q?cy3ppor13WVeJfOLrFtrWbA67pfy42ToYSBvpj5u22ky8HbaYeFXDHZrLlDi?= =?us-ascii?Q?wt9ftTSqfySKSQSJOr9vyVPyDA8pTQtZdYs9YTGidYLj5zBrbVrsN8Q8zjpl?= =?us-ascii?Q?tv1zLpaE1otS/zew8E+rd0RqW1VGttVmUKOMNDpU/5jLRIvyxAIXM7Kt970t?= =?us-ascii?Q?6yp90r9K4l9w8Q58eHyj1UEgtQfbewveFv6yqSd3KTYU1TmSqoJ1JjMiSy/c?= =?us-ascii?Q?XHZyGj35bUBdpQfw0Qjq22Ed0mHSrHghC79vy5tjgDZLULRZUl5L5ENLOqOZ?= =?us-ascii?Q?uZnb35XJrATgB9L37UvWZz3oeX02q8jmQgDdi/iok+rm+ihScNsPZo2BDsu3?= =?us-ascii?Q?SwjNaqA8C4Bpei1ORl9P1XFLOa2y5+faPpOnhXvpu8mY16BphFHiIw4D1xvu?= =?us-ascii?Q?oRArmgfQVwVbhQ4nS9dfSu0QSyuYeTc2WJMrrE3KaTIpR5LlBkbRgPZu0OkY?= =?us-ascii?Q?omvGH4v/GqA3LQI/CJwDtGZ0MS2FQs/JinP+r+TrLOOoVxHzvYAYZeLBYFLU?= =?us-ascii?Q?Cm9ocL6RajScdUYa2cLi1dtF6bd/Fco2ZD5Y8sbB/ac+4eUyHvS4qxcKEdzF?= =?us-ascii?Q?+XlqmiY9VTHomdIIMOZzhdOfVMWqrtPXhQ7O0eQ06WfQ8OzuqQ6TiknsfGe0?= =?us-ascii?Q?5t/gyBhT2iHE1L3ZC6sC2IyVhuGaScfH2E/jyowHL3LSarQBZhURSb+WaqGp?= =?us-ascii?Q?EhcDF8IG7L9QdxBNZ/2qoKnv9ZYs3Lfm6QvJXUghn3AZpgHdp2amyRL6Ri64?= =?us-ascii?Q?N+9HFpoMrWzf1VnohLTNEZVQgS7u6xVKCKwFYVwOeeC5TwWdnku6i5MG2bxd?= =?us-ascii?Q?ZwXDwF7RjemIeRtJJkVcpFErAcEp7uIStSBLW27CoUdcMBed/VD26MMNgvqP?= =?us-ascii?Q?dvbvJNhKbKqCNN7ZJaZMxM8SwmKhYEe2zcFVTyJ/vlEbaUyI+DwIrQP5O6Rv?= =?us-ascii?Q?oHUl96MMqHdm/6KLZ6HwPXB046liAXjK0Lny1QOQbJLHB1XfWxFY3FBpr9zt?= =?us-ascii?Q?uCFGF9GEyX79m5RWs02duS0AgwjzoPG2B+JcGT0nSbkCS4rMSJqs3jlTC8tQ?= =?us-ascii?Q?iWrUt089re+r7bc4gwPaPyj0LITM2DX50ouav1rZD91Z59rsytKivkqigfLe?= =?us-ascii?Q?mDSWxhdio6cSIRd1QBdxJcYcoMJKvRkwai/9riwIW2GQ5/R5TIcPwHtNfK0h?= =?us-ascii?Q?8qzTzSFqXp7CQ=3D=3D?= X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;CO2PR07MB2583;6:ht0q7Sd9Kv6pSI/+l5jNzLFwjBqBGc0RyKou60K4wo?= =?us-ascii?Q?LMLMXD24ZS6skdX9SLcXipuLrslHnhTyX4YH+DzRAjbXZeGLK3JOJIEh1cR6?= =?us-ascii?Q?usDJivMWYvFYPh+WRP8/Z8fsfBmYSiuLzavCLJyG6iSZiQZG14nrur05b0ze?= =?us-ascii?Q?njuagG8QT4zMq9mxQolflPSDBedxvNYW7N0w4TW457JV11IN5JnmIWH2xyQx?= =?us-ascii?Q?BZDy+M9sgAjYiT6sjog7nRSROaCekaVbY735kFp5ogS1Nd8oeVwhJ2Tpq4aB?= =?us-ascii?Q?kAEjngSbBDlE5w9A4eKxM0NOtbZMAVg9VXGQ7gewSZYpzeJaDpEw5MxWNRXw?= =?us-ascii?Q?TJxCbJAp4Bfsq06suyFIWZ1WY1YGzK2hBG9IO8ynvyxo0cABp9Am0H2mW4Bx?= =?us-ascii?Q?4geOk6G9HlOAUxsKiU4TuB6IXB4wL7OqlNloPqs2fmrk6ibYXx7VhMbh/7sS?= =?us-ascii?Q?DoqKeIQNXas7I3yFcNcqU70IwYHl5eir0rOPpzEXk8eh5/+5xrg9nV/BsEAb?= =?us-ascii?Q?cVhu0kApSZw87u3J3lbV6EzfquLpr3SbJs7JoiqmszJAkCyjzwek2IY8umZu?= =?us-ascii?Q?wGxMKzaBIHsMkMh+8p3jQfl0mI9p5flSB4i5ODSzl22TNi8As2wDS2BejCGl?= =?us-ascii?Q?8O9t/QF82usg2ckhIrk+5vfw76C/VQB3HksYIsKF1XtwR5yTcd9lTm1QUYnj?= =?us-ascii?Q?xfE5QZrnkanrvm+AYWvj+Gr4V96AE0EE7mn542PwvTbTD6IQIhZEI2fQ8HDK?= =?us-ascii?Q?qgcZJPyzZexa7Bk+7Xmgnc6RrVzkKJ23Zg6voCDuMHMVdK8cLCQd0VjBnAVe?= =?us-ascii?Q?xQwGp8p/nWkDJqO9wIrvGjOztx38zfwNdrpy/+ZlCuDdpZdTSFKTUl3h0l2j?= =?us-ascii?Q?HUAqZdOseUrj3b+KSmM/Rg+WaKJ3C6tH9eIjqIj0/Tt+BI9//LYljYnFWRWo?= =?us-ascii?Q?agzODXhPjeOvXIvCAsMMjf/RdLauORVPQJsD4NvXcBME1IjBIULGUVjhKm7b?= =?us-ascii?Q?o=3D?= X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2583;5:3N4ih2ws46+msOutBgVVDkHFLC9Z9Eq53kFbDFkrAxcgz3HnVjRe0RgU5DvAKg8U107Udstb7zfpvlkqOCDMfzQ3+lh1FTUTlrLrul8GHe8RYSi76BnBlZ10bM7oDkMMDf/U403tOcq/FuUsZ4fNdZKmigoWDjrmDDBwtlQ3LP31zUV0QyMMbrJSZqVp9eWEOyJsQQKQuOd4GPYOE8C38d8XlRk9w1RiGxpKwOj+ab0YsT3mDMG8lS9ydFVlLsWFq4JKbD2kk5U+Jme9yAIK3l+Z06q96blxUwIVpE/QBbYaEgJFZSjdI6ALgxAXQx4lXq868lTZwBkArY2IpZKCTaTWZMCNPcBhD6nagGhZUDBzVfxPu4SOhVpUoykgrZEaGf9NIEBI09ULjHH0fyqUJ3o/nYagArN2TlIBUWaQMarwwvLoQV/ARtyyQf3pKkZMrkSQabtnCDw9+yszid96GcOduOu6Fnhe1oLAVSOW86nbNoi+qpteTLqmjlTmFiVR;24:DyBm3R0N6Vfirbfxz4J0lDR6mxTbzQcLSklI9m331vVio6z1MCnFJpYl7gby92nfY4x+/4Y+C3Cktme+ZpTEYe4PvsFBPlWru8bPUI3XfeQ= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;CO2PR07MB2583;7:XPR3TzbMMpyliQ0qOpFHQkkMk/C2LLZx3p49ADqoyNlwdPpuX66yFY67fMpg8tpj9DzN/KQySTjgM2OwTGE/7/evvjYFtFfSO/YyPPT2GTmwDWP6mDtkOv+Q1d6dIGMFZO37r7o79dp7+cizfvK5BbvlysKTXJtTxGsDSTg3tLwuDrIJdfftwkGreqULgEEiHCTEso/jqdDovw8wjuFuRdR0WcV6BVKpCkmx6nFVH0WyGYUSdG+t2ejwtOuoijcNqUaj/+k+AEFP7YDsZbQLIuM5B+RaxS1zu7li74WE5+nFEI2iewzn2ShTv9nJs+xNkCnwl3GEYaqqQp7rmkqOdVNi6Nq5uVePmP96Y/AcmMtU2XIKmO5thhuGPXxEbnjLhkhZYmBm/w4bbpu1XlTGhtkaMuX5pdAcW1KYdnUdH11ppvjlv2G+rWgBYm9B3QV7rbMsWUgMlblxgtTALTVdg0hpNBCXapVCyKpyZgNKRqtmQY0dyaM6bFocZek9QH/t/4UgwW+HOFYUvy8m8tlxQg92CUb4mlpP59uOAiLBkR7slktMpOw/KCIi4Q7DXFLr5A2s3djieD7TE9t6dWU+EyQ1xv43PobVuu6do6Ldfbyiw6ZVoGqaR2bioPbGQDTTEwUPVq3ElnBznYOCJGYWm/sx59PteuhB2tNP4BtTpiv94iOz5n2Vjs88ljIuMwcBs2IxiJbta38c21OhYOnNSH/L/diI5yymYlpJSjuO9o/WfLLOX5rKW1EanyJfc2RDKZMmwfJh1Ql4EHVI7/Og67M4cTj3R+5qkrQhB6dbq2I= X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 26 Jul 2017 15:13:25.2041 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO2PR07MB2583 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2210 Lines: 55 On Wed, Jul 26, 2017 at 04:55:22PM +0200, Borislav Petkov wrote: > On Wed, Jul 26, 2017 at 03:35:25PM +0100, Suzuki K Poulose wrote: > > So the Cavium EDACs, which appear as PCI devices have a PMU attached to it. > > Cavium EDACs? > > So let me set something straight first: An EDAC driver simply talks to > some RAS IP block and reports errors. It shouldn't have anything to do > with a PMU. > > > In order to build this PMU driver as a module, we need a way to load the module > > automatically based on the PCI id. However, since the EDAC driver already > > registers with that PCI id, we cannot use the same for the PMU. Ideally, > > So this is strange. There's a single PCI ID but multiple functionalities > behind it? Yes, but I would still not call a memory controller a RAS IP block. There are a number of registers on the memory controller (or on the OCX TLK interconnect), and while some of them are RAS related there are also other registers in the same device like the counters we want to access via PMU code. > > the PMU driver should be loaded when the EDAC driver is loaded. But looking > > at the links above, it looks like you don't like the idea of triggering a > > probe of the PMU component from the EDAC driver. We may be able to get rid > > of the PMU specific information from the EDAC driver by maintaining the PCI > > id of the device in the PMU driver. But we may still need to make sure that > > the PMU driver gets a chance to probe the PMU when the device is available. > > > > What do you think is the best option here ? > > Can either of the two - EDAC or PMU driver - use an alternate detection > method? I'm currently using pci_get_device(vendor-ID, device-ID, ...) which works fine. > For example, we moved the main x86 EDAC drivers we moved to x86 CPU > family, model, stepping detection from PCI IDs because the PCI IDs were > clumsy to use. I'm also looking for CPU implementor (MIDR), I could check for the model too but I still need to detect devices based on PCI IDs as the model check is not sufficient here (only multi-socket ThunderX has OCX TLK devices). --Jan > -- > Regards/Gruss, > Boris. > > ECO tip #101: Trim your mails when you reply. > --