ACPI: pfr_telemetry: use ACPI_FREE() to free acpi_object
authorWang ShaoBo <bobo.shaobowang@huawei.com>
Fri, 18 Nov 2022 06:32:18 +0000 (14:32 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 31 Dec 2022 12:31:57 +0000 (13:31 +0100)
[ Upstream commit 0f2aa7fc2a9aee05bafb965d5b1638d3e74b4c61 ]

acpi_evaluate_dsm_typed()/acpi_evaluate_dsm() should be coupled
with ACPI_FREE() to free the ACPI memory, because we need to
track the allocation of acpi_object when ACPI_DBG_TRACK_ALLOCATIONS
enabled, so use ACPI_FREE() instead of kfree().

Fixes: b0013e037a8b ("ACPI: Introduce Platform Firmware Runtime Telemetry driver")
Signed-off-by: Wang ShaoBo <bobo.shaobowang@huawei.com>
Reviewed-by: Chen Yu <yu.c.chen@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/acpi/pfr_telemetry.c

index 9abf350..27fb6cd 100644 (file)
@@ -144,7 +144,7 @@ static int get_pfrt_log_data_info(struct pfrt_log_data_info *data_info,
        ret = 0;
 
 free_acpi_buffer:
-       kfree(out_obj);
+       ACPI_FREE(out_obj);
 
        return ret;
 }
@@ -180,7 +180,7 @@ static int set_pfrt_log_level(int level, struct pfrt_log_device *pfrt_log_dev)
                ret = -EBUSY;
        }
 
-       kfree(out_obj);
+       ACPI_FREE(out_obj);
 
        return ret;
 }
@@ -218,7 +218,7 @@ static int get_pfrt_log_level(struct pfrt_log_device *pfrt_log_dev)
        ret = obj->integer.value;
 
 free_acpi_buffer:
-       kfree(out_obj);
+       ACPI_FREE(out_obj);
 
        return ret;
 }