Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752072AbbH1Knw (ORCPT ); Fri, 28 Aug 2015 06:43:52 -0400 Received: from mail-by2on0071.outbound.protection.outlook.com ([207.46.100.71]:60292 "EHLO na01-by2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752002AbbH1Knu (ORCPT ); Fri, 28 Aug 2015 06:43:50 -0400 Authentication-Results: spf=none (sender IP is 165.204.84.221) smtp.mailfrom=amd.com; alien8.de; dkim=none (message not signed) header.d=none;alien8.de; dmarc=permerror action=none header.from=amd.com; X-WSS-ID: 0NTSGGO-07-2AI-02 X-M-MSG: Date: Fri, 28 Aug 2015 18:42:02 +0800 From: Huang Rui To: Ingo Molnar CC: Borislav Petkov , Jean Delvare , "Guenter Roeck" , Andy Lutomirski , "Andreas Herrmann" , Thomas Gleixner , Peter Zijlstra , "Rafael J. Wysocki" , Len Brown , John Stultz , =?iso-8859-1?Q?Fr=E9d=E9ric?= Weisbecker , , , , Andreas Herrmann , Aravind Gopalakrishnan , Borislav Petkov , "Fengguang Wu" , Aaron Lu , Tony Li Subject: Re: [PATCH 10/15] hwmon, fam15h_power: add compute unit accumulated power Message-ID: <20150828104202.GC4191@hr-slim.amd.com> References: <1440662866-28716-1-git-send-email-ray.huang@amd.com> <1440662866-28716-11-git-send-email-ray.huang@amd.com> <20150828080335.GA13309@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20150828080335.GA13309@gmail.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-EOPAttributedMessage: 0 X-Forefront-Antispam-Report: CIP:165.204.84.221;CTRY:US;IPV:NLI;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(2980300002)(428002)(3050300001)(24454002)(189002)(199003)(164054003)(23726002)(46406003)(50466002)(5003600100002)(47776003)(64706001)(97756001)(46102003)(53416004)(4001350100001)(110136002)(189998001)(101416001)(19580405001)(87936001)(19580395003)(77156002)(5001860100001)(97736004)(77096005)(4001540100001)(5001830100001)(92566002)(62966003)(33656002)(50986999)(83506001)(105586002)(68736005)(54356999)(76176999)(5007970100001)(86362001)(2950100001)(106466001);DIR:OUT;SFP:1101;SCL:1;SRVR:DM3PR12MB0858;H:atltwp01.amd.com;FPR:;SPF:None;PTR:InfoDomainNonexistent;MX:1;A:1;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;DM3PR12MB0858;2:AQXMgNE7lQvvNDu80NifcHzFzzwzHYUTkUQX72d8LVlitjaJedmN/8AxyqpKWhnggBFXe1jGWtFy0wxmT73cOC4iGkqrAaHNj5Glti8+8aSJ96iOnY561TvBXtgSN/8WMq4VSw7AECjvHwG4bw/yQrtNVGVOcuU66duFYTkkl4o=;3:tc3DSS8d7xxPHk9JKJhPBs2cV5bA0VmGVDROMg4brSX/Mfjf/35Ov5HZ8Xwwh5Qd4acIansUlshT9GXZ2FARnccPHLskYims1QQBeG5cygcWYU/Gj2d+c5Mt4W1wUJAu5rpsmAgC2T4fOSGpyIbXRZjAk9oqqk+tvu7xqVYN1+vNS5hYlkU68eDKhW1y4JaJMkLYsf57V48r14WfRmYztQ7YyRYx5ben9rN9XVRthn5qOWySQgXClBoeRG0KIvLi;25:p+gW/yU1kp8uO075tCbh9i2hhCwnzsmz4q+8dtPJOoJQy/SJHI0ArNuVlFUqAdK7gLXbGG6JktqbeqP0ab/KOPdsV8+3svtMsephP7LIlufmDlyqPa99RKGpwfYS5O08gInnVs+KymSgP5mD7jt6n4uvkieUSsz3Y95rEIlYLRoq7fzM4UyFPdLBf6emhU2uVfT2R+koXyqBXrcGB+PuYBPkPzwCFn8+ZHhcBN89PjQlRTuRbZl54Gp1olqWjljwbya3bmd3QlhyqGWu+2GepQ== X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:DM3PR12MB0858; X-Microsoft-Exchange-Diagnostics: 1;DM3PR12MB0858;20:/zvzrK0+HwpidL4ISVjcaxjhilpGM9NuHpRN1L0oq6Yj9oCb47Xg7k4xI1yirJ4SjbPSP73gn/VtbPPyKRg/aASkc2OOQJ6HxIs+ok/hppChAeNR0EXONUG3bv1d4dF+04v7JBt1uBA0MaGytOJ94TOYGujGDiFH+KjuJgGHPoetfcbw+8KfhYeRr/w4U9G6sPRnG9HJqapZUH4UitD8Xe8t0fhPtiJ+hFXMuRTr6z8Qb9GD0CC6YgkmGa9S0DEVvh9KtOti7GXl8HAMTeGJVsAW6pr3SgmLC70DzDfJDzguHIEPwc9QXMbTnZ3nNoAKY0ImQBRBAWqXvEBkyWZvNI7EdvU8aE9srdEyxueHDUdNEuCJpT9cqztZGEonK/rbA5D4aYNjMjWR73USJZSXqsx+V7MW0Hma4q1CGpomwD0lM1VvdRa1oD77Z+xxbjFMnsyo7sIh8lJX2+TJeKbbGNYustJIMAZAWncgN04lOXuSY3wvk1ZXklWFmUmF7yzV;4:CJZhTMctzYAMi9oNO6KRppXstNH/v1ZvWJmpFjLuprnXBRASYjHEwFJlWU7gUrd2gouith6jcB/AuLjKLAKKWhfBVYVzHhHWQLpyOKtSi2vNwbFsGH7c9lqAPG62Jo//re/Abfde9cXy5emDYt6NYRL5WZK3bWmyOPy691xJRlU0QOciTlbGDNpKaRS3oYWNVrZOLVIt+po0akSoPu/zp7kPZaa0bZwEHn9CAS2z5WzHKjKzA4Fj01Jt/W1Z97qxReOPtEmta9gNV3jI4t2HPxYCUtW3i0OlxXLdYsOHtD4ox+8S10RQWPjhyv8SH/q4 X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(5005006)(8121501046)(3002001);SRVR:DM3PR12MB0858;BCL:0;PCL:0;RULEID:;SRVR:DM3PR12MB0858; X-Forefront-PRVS: 0682FC00E8 X-Microsoft-Exchange-Diagnostics: =?us-ascii?Q?1;DM3PR12MB0858;23:buGdwj9yuGCM8X5TAyrtZFRZTV1E0IpVH+azuVlFe?= =?us-ascii?Q?EeI26CtDav0BHrzDv6ePE7vM3w8SL4Ll7YnGXmGCzoNWXijv48lOBAyLpzOa?= =?us-ascii?Q?1LFN9904/eOOmKRy96OB9ECKPVPpvAMSuDthWc7ivfvWHiaymd46GivI2CBB?= =?us-ascii?Q?QhYZ3/aROGzEqthJqqSn+fT61f5GtfimwnsUc+ViMKknZOcKZaF1sMuxNkK0?= =?us-ascii?Q?/rFkVZYo7cCr+iPs+PvkSvm0IwWHUA3WxLnk7NSANQuJlVD6lZlkGQvTwjil?= =?us-ascii?Q?iPKeBW7S/Xu+u4ITvW6NXO343/Pg2pEGSstf2+MiWDbcMW2/fbqtQPGK10es?= =?us-ascii?Q?MP5yzfRJ9cJi0ThsC5Yqso8PoeE5TZS0AzEpGIYwTgDd6d4bQWagcUAgJMPn?= =?us-ascii?Q?3GJl9Yehk5g9fnmU2XLTKVxA0UTdv5h+JQM487PZXO9fzkj8vLDQv0ZYzPCV?= =?us-ascii?Q?Hdo93H4bhjQ2z2rBYgrqaAM8ffAWy6DWY17lQNPiSIqehYHato2IHR1Avk8w?= =?us-ascii?Q?vsZWM7LCbEc6qN0fFy0eUp+e5glFl/JsEh/1t1YU1yLmqzUH8bUn/JMnrV4+?= =?us-ascii?Q?mxYm+5Jig14ic/UEz8GMUi5Sb61lhcKZpeRCsnrq/GUpxolihCey3wVjSe5g?= =?us-ascii?Q?1+mWKzR/ByK9b4jgKOTUuxhd/bJDqtcVoYQZYkSrnczJfLIW5qrlAAZKIfRl?= =?us-ascii?Q?AqACKmZ6Kj00uct5grKsEMbc44nB1kIlnmKkhj60sCqg71CAyWiHxDK/A8sE?= =?us-ascii?Q?HLwQGHQcH0phcKjzMvyC/OTBgEnjO+dz8XGwIR6LXrsxxXnHNSi9CXHF2X47?= =?us-ascii?Q?rKjwHb1+3NocdHJL7dtxe0slvtIEE9tbAavS4O3rHyuyQ6RuJV3gyeprz75g?= =?us-ascii?Q?hzIe1RKVKf0BTVET0Qw+f0eT+Knm4WipeMc66WnyMf3mejtazZH6M/YrruU5?= =?us-ascii?Q?4QJhYFtFbv2M4Csbdf9QJTosBbUJXhoVSn4sXJ1mJK4uv1lvD9G9Rdmf9UyE?= =?us-ascii?Q?Sm06KtpQtYUP9t6OvlDLnI4dqG0A4TjACKfh/gxdnrPaS7rrDZU/eKhr9QD2?= =?us-ascii?Q?CJuDiONB4I+qllOYKHUcFInMZ0T7PXLlwdXZyu17bx+v1iul08jbx5x2X9Xs?= =?us-ascii?Q?HBSOmnGyjnVWIfeh/K3T0dODGGEYTHoLKJ/Bd+1FdtfjgK4jGPwAQ=3D=3D?= X-Microsoft-Exchange-Diagnostics: 1;DM3PR12MB0858;5:Sf9ffV/lFnpo3P8adm1yoJ3LV4PIwUIv0q7lPkDMjYM2OH5e/JD5kLlYeWLtesJqoI2f9cHNHtnn71Q5JxZzQBTbQgTLex0nl+d90MIDhlqp4MaCnbpZAf2mRcjHyyqSaGsrgpEYURglmFYeanw1bw==;24:aQWdcyfYcQZhdYM6kzOJL6RlgWGeHt8e0AkfapS4bB1TEfpX3BHDjNKynnrhNjJcgh0iBKSQb8sNWynSt6yVB59nY8d73/oEFZWE14A9COQ=;20:aez+dX8D+tXB+HkyPnOD3Dqoy9Sg44EFMa9OmevVowjJdIWGhwVtlFqqMdVzMPW9RO7Vov68p6dGOHts7rjYfQ== SpamDiagnosticOutput: 1:23 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Aug 2015 10:43:38.6910 (UTC) X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d;Ip=[165.204.84.221];Helo=[atltwp01.amd.com] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM3PR12MB0858 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1477 Lines: 48 On Fri, Aug 28, 2015 at 10:03:35AM +0200, Ingo Molnar wrote: > > * Huang Rui wrote: > > > @@ -249,6 +254,21 @@ static int fam15h_power_init_data(struct pci_dev *f4, > > > > data->max_cu_acc_power = tmp; > > > > + cores_per_cu = amd_get_cores_per_cu(); > > + cu_num = boot_cpu_data.x86_max_cores / cores_per_cu; > > + > > + WARN_ON_ONCE(cu_num > MAX_CUS); > > + > > + for (cpu = 0; cpu < cu_num * cores_per_cu; cpu += cores_per_cu) { > > so 'cu_num * cores_per_cu' is really a roundabout way to say > boot_cpu_data.x86_max_cores? > Oh, yes. :) I will update it at v2. > > + cu = cpu / cores_per_cu; > > + if (rdmsrl_safe_on_cpu(cpu, MSR_F15H_CU_PWR_ACCUMULATOR, > > + &data->cu_acc_power[cu])) { > > + pr_err("Failed to read compute unit power accumulator MSR on core%d\n", > > + cpu); > > Please don't break printk lines mid-line - ignore checkpatch in this case. > OK, I got it. > Also, the message talks about 'core', while a CPU ID is printed. > Yes, but actually this value is for the compute unit which the core belongs to. E.X. the MSR_F15H_CU_PWR_ACCUMULATOR value is the same between core 0 and core 1. Because they belong to the same compute unit. Thanks, Rui -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/