Skip to content

Commit

Permalink
ACPI: pfr_telemetry: use ACPI_FREE() to free acpi_object
Browse files Browse the repository at this point in the history
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: b0013e0 ("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>
  • Loading branch information
Wang ShaoBo authored and Rafael J. Wysocki committed Nov 23, 2022
1 parent eb70814 commit 0f2aa7f
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions drivers/acpi/pfr_telemetry.c
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down Expand Up @@ -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;
}
Expand Down Expand Up @@ -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;
}
Expand Down

0 comments on commit 0f2aa7f

Please sign in to comment.