summaryrefslogtreecommitdiffstats
path: root/drivers/powercap
diff options
context:
space:
mode:
authorxiongxin <xiongxin@kylinos.cn>2023-04-07 04:47:59 +0200
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2023-08-01 13:54:00 +0200
commit2fa00769b1e4bcf20cf0c967cb45f0cd1cea2b5c (patch)
tree24441b291087dcd60efc26eff5aff09278ea5dd2 /drivers/powercap
parentpowercap: arm_scmi: Remove recursion while parsing zones (diff)
downloadlinux-2fa00769b1e4bcf20cf0c967cb45f0cd1cea2b5c.tar.xz
linux-2fa00769b1e4bcf20cf0c967cb45f0cd1cea2b5c.zip
powercap: intel_rapl: Optimize rp->domains memory allocation
In the memory allocation of rp->domains in rapl_detect_domains(), there is an additional memory of struct rapl_domain allocated, optimize the code here to save sizeof(struct rapl_domain) bytes of memory. Test in Intel NUC (i5-1135G7). Signed-off-by: xiongxin <xiongxin@kylinos.cn> Tested-by: xiongxin <xiongxin@kylinos.cn> Reviewed-by: Srinivas Pandruvada<srinivas.pandruvada@linux.intel.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/powercap')
-rw-r--r--drivers/powercap/intel_rapl_common.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/powercap/intel_rapl_common.c b/drivers/powercap/intel_rapl_common.c
index 4e646e5e48f6..5e2a1c376a64 100644
--- a/drivers/powercap/intel_rapl_common.c
+++ b/drivers/powercap/intel_rapl_common.c
@@ -1485,7 +1485,7 @@ static int rapl_detect_domains(struct rapl_package *rp)
}
pr_debug("found %d domains on %s\n", rp->nr_domains, rp->name);
- rp->domains = kcalloc(rp->nr_domains + 1, sizeof(struct rapl_domain),
+ rp->domains = kcalloc(rp->nr_domains, sizeof(struct rapl_domain),
GFP_KERNEL);
if (!rp->domains)
return -ENOMEM;