Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754912AbcKUPt0 (ORCPT ); Mon, 21 Nov 2016 10:49:26 -0500 Received: from terminus.zytor.com ([198.137.202.10]:40208 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754747AbcKUPtZ (ORCPT ); Mon, 21 Nov 2016 10:49:25 -0500 Date: Mon, 21 Nov 2016 07:49:14 -0800 From: tip-bot for Thomas Gleixner Message-ID: Cc: bigeasy@linutronix.de, tglx@linutronix.de, rjw@rjwysocki.net, mingo@kernel.org, hpa@zytor.com, linux-kernel@vger.kernel.org Reply-To: hpa@zytor.com, mingo@kernel.org, linux-kernel@vger.kernel.org, bigeasy@linutronix.de, rjw@rjwysocki.net, tglx@linutronix.de In-Reply-To: <20161117183541.8588-9-bigeasy@linutronix.de> References: <20161117183541.8588-9-bigeasy@linutronix.de> To: linux-tip-commits@vger.kernel.org Subject: [tip:smp/hotplug] powercap/intel_rapl: Add missing domain data update on hotplug Git-Commit-ID: 4a6162f9e1c140c7436818983065365168075b0c X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3144 Lines: 85 Commit-ID: 4a6162f9e1c140c7436818983065365168075b0c Gitweb: http://git.kernel.org/tip/4a6162f9e1c140c7436818983065365168075b0c Author: Thomas Gleixner AuthorDate: Thu, 17 Nov 2016 19:35:29 +0100 Committer: Thomas Gleixner CommitDate: Mon, 21 Nov 2016 16:37:06 +0100 powercap/intel_rapl: Add missing domain data update on hotplug The domain data of packages is only updated at init time, but new packages created by hotplug miss that treatment. Add it there and remove the global update at init time, because it's now obsolete. Signed-off-by: Thomas Gleixner Signed-off-by: Sebastian Andrzej Siewior Cc: "Rafael J. Wysocki" Cc: rt@linuxtronix.de Cc: linux-pm@vger.kernel.org Link: http://lkml.kernel.org/r/20161117183541.8588-9-bigeasy@linutronix.de Signed-off-by: Thomas Gleixner --- drivers/powercap/intel_rapl.c | 28 +++++++++++++--------------- 1 file changed, 13 insertions(+), 15 deletions(-) diff --git a/drivers/powercap/intel_rapl.c b/drivers/powercap/intel_rapl.c index 243b233..44b12e2 100644 --- a/drivers/powercap/intel_rapl.c +++ b/drivers/powercap/intel_rapl.c @@ -1164,24 +1164,20 @@ static const struct x86_cpu_id rapl_ids[] __initconst = { }; MODULE_DEVICE_TABLE(x86cpu, rapl_ids); -/* read once for all raw primitive data for all packages, domains */ -static void rapl_update_domain_data(void) +/* Read once for all raw primitive data for domains */ +static void rapl_update_domain_data(struct rapl_package *rp) { int dmn, prim; u64 val; - struct rapl_package *rp; - list_for_each_entry(rp, &rapl_packages, plist) { - for (dmn = 0; dmn < rp->nr_domains; dmn++) { - pr_debug("update package %d domain %s data\n", rp->id, - rp->domains[dmn].name); + for (dmn = 0; dmn < rp->nr_domains; dmn++) { + pr_debug("update package %d domain %s data\n", rp->id, + rp->domains[dmn].name); /* exclude non-raw primitives */ - for (prim = 0; prim < NR_RAW_PRIMITIVES; prim++) - if (!rapl_read_data_raw(&rp->domains[dmn], prim, - rpi[prim].unit, - &val)) - rp->domains[dmn].rdd.primitives[prim] = - val; + for (prim = 0; prim < NR_RAW_PRIMITIVES; prim++) { + if (!rapl_read_data_raw(&rp->domains[dmn], prim, + rpi[prim].unit, &val)) + rp->domains[dmn].rdd.primitives[prim] = val; } } @@ -1239,6 +1235,9 @@ static int rapl_package_register_powercap(struct rapl_package *rp) struct powercap_zone *power_zone = NULL; int nr_pl; + /* Update the domain data of the new package */ + rapl_update_domain_data(rp); + /* first we register package domain as the parent zone*/ for (rd = rp->domains; rd < rp->domains + rp->nr_domains; rd++) { if (rd->id == RAPL_DOMAIN_PACKAGE) { @@ -1357,8 +1356,7 @@ static int rapl_register_powercap(void) pr_debug("failed to register powercap control_type.\n"); return PTR_ERR(control_type); } - /* read the initial data */ - rapl_update_domain_data(); + list_for_each_entry(rp, &rapl_packages, plist) if (rapl_package_register_powercap(rp)) goto err_cleanup_package;