mirror of
https://github.com/torvalds/linux.git
synced 2024-11-16 17:12:06 +00:00
7b768f07dc
arch_acpi_processor_cleanup_pdc() in x86 and ia64 results in memory allocated for _PDC objects that is never freed and will cause memory leak in case of physical CPU remove and add. Patch fixes the memory leak by freeing the objects soon after _PDC is evaluated. Reported-by: Bjorn Helgaas <bjorn.helgaas@hp.com> Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com> Signed-off-by: Len Brown <len.brown@intel.com>
86 lines
1.8 KiB
C
86 lines
1.8 KiB
C
/*
|
|
* arch/ia64/kernel/acpi-processor.c
|
|
*
|
|
* Copyright (C) 2005 Intel Corporation
|
|
* Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
|
|
* - Added _PDC for platforms with Intel CPUs
|
|
*/
|
|
|
|
#include <linux/kernel.h>
|
|
#include <linux/module.h>
|
|
#include <linux/init.h>
|
|
#include <linux/acpi.h>
|
|
|
|
#include <acpi/processor.h>
|
|
#include <asm/acpi.h>
|
|
|
|
static void init_intel_pdc(struct acpi_processor *pr)
|
|
{
|
|
struct acpi_object_list *obj_list;
|
|
union acpi_object *obj;
|
|
u32 *buf;
|
|
|
|
/* allocate and initialize pdc. It will be used later. */
|
|
obj_list = kmalloc(sizeof(struct acpi_object_list), GFP_KERNEL);
|
|
if (!obj_list) {
|
|
printk(KERN_ERR "Memory allocation error\n");
|
|
return;
|
|
}
|
|
|
|
obj = kmalloc(sizeof(union acpi_object), GFP_KERNEL);
|
|
if (!obj) {
|
|
printk(KERN_ERR "Memory allocation error\n");
|
|
kfree(obj_list);
|
|
return;
|
|
}
|
|
|
|
buf = kmalloc(12, GFP_KERNEL);
|
|
if (!buf) {
|
|
printk(KERN_ERR "Memory allocation error\n");
|
|
kfree(obj);
|
|
kfree(obj_list);
|
|
return;
|
|
}
|
|
|
|
buf[0] = ACPI_PDC_REVISION_ID;
|
|
buf[1] = 1;
|
|
buf[2] = ACPI_PDC_EST_CAPABILITY_SMP;
|
|
/*
|
|
* The default of PDC_SMP_T_SWCOORD bit is set for IA64 cpu so
|
|
* that OSPM is capable of native ACPI throttling software
|
|
* coordination using BIOS supplied _TSD info.
|
|
*/
|
|
buf[2] |= ACPI_PDC_SMP_T_SWCOORD;
|
|
|
|
obj->type = ACPI_TYPE_BUFFER;
|
|
obj->buffer.length = 12;
|
|
obj->buffer.pointer = (u8 *) buf;
|
|
obj_list->count = 1;
|
|
obj_list->pointer = obj;
|
|
pr->pdc = obj_list;
|
|
|
|
return;
|
|
}
|
|
|
|
/* Initialize _PDC data based on the CPU vendor */
|
|
void arch_acpi_processor_init_pdc(struct acpi_processor *pr)
|
|
{
|
|
pr->pdc = NULL;
|
|
init_intel_pdc(pr);
|
|
return;
|
|
}
|
|
|
|
EXPORT_SYMBOL(arch_acpi_processor_init_pdc);
|
|
|
|
void arch_acpi_processor_cleanup_pdc(struct acpi_processor *pr)
|
|
{
|
|
if (pr->pdc) {
|
|
kfree(pr->pdc->pointer->buffer.pointer);
|
|
kfree(pr->pdc->pointer);
|
|
kfree(pr->pdc);
|
|
pr->pdc = NULL;
|
|
}
|
|
}
|
|
|
|
EXPORT_SYMBOL(arch_acpi_processor_cleanup_pdc);
|