Module Name: src Committed By: riastradh Date: Sun Feb 27 14:24:27 UTC 2022
Modified Files: src/sys/external/bsd/drm2/dist/drm/amd/amdgpu: amdgpu_acpi.c src/sys/external/bsd/drm2/dist/drm/nouveau: nouveau_acpi.c src/sys/external/bsd/drm2/dist/drm/nouveau/nvkm/subdev/mxm: nouveau_nvkm_subdev_mxm_base.c src/sys/external/bsd/drm2/dist/drm/radeon: radeon_acpi.c radeon_bios.c Log Message: drm: Use ACPI_FREE, not kfree, for ACPI_ALLOCATE_BUFFER result. Most of this code is not currently enabled, but it might be enabled soon. To generate a diff of this commit: cvs rdiff -u -r1.4 -r1.5 \ src/sys/external/bsd/drm2/dist/drm/amd/amdgpu/amdgpu_acpi.c cvs rdiff -u -r1.3 -r1.4 \ src/sys/external/bsd/drm2/dist/drm/nouveau/nouveau_acpi.c cvs rdiff -u -r1.3 -r1.4 \ src/sys/external/bsd/drm2/dist/drm/nouveau/nvkm/subdev/mxm/nouveau_nvkm_subdev_mxm_base.c cvs rdiff -u -r1.3 -r1.4 \ src/sys/external/bsd/drm2/dist/drm/radeon/radeon_acpi.c cvs rdiff -u -r1.9 -r1.10 \ src/sys/external/bsd/drm2/dist/drm/radeon/radeon_bios.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/external/bsd/drm2/dist/drm/amd/amdgpu/amdgpu_acpi.c diff -u src/sys/external/bsd/drm2/dist/drm/amd/amdgpu/amdgpu_acpi.c:1.4 src/sys/external/bsd/drm2/dist/drm/amd/amdgpu/amdgpu_acpi.c:1.5 --- src/sys/external/bsd/drm2/dist/drm/amd/amdgpu/amdgpu_acpi.c:1.4 Sat Dec 18 23:44:58 2021 +++ src/sys/external/bsd/drm2/dist/drm/amd/amdgpu/amdgpu_acpi.c Sun Feb 27 14:24:26 2022 @@ -1,4 +1,4 @@ -/* $NetBSD: amdgpu_acpi.c,v 1.4 2021/12/18 23:44:58 riastradh Exp $ */ +/* $NetBSD: amdgpu_acpi.c,v 1.5 2022/02/27 14:24:26 riastradh Exp $ */ /* * Copyright 2012 Advanced Micro Devices, Inc. @@ -24,7 +24,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: amdgpu_acpi.c,v 1.4 2021/12/18 23:44:58 riastradh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: amdgpu_acpi.c,v 1.5 2022/02/27 14:24:26 riastradh Exp $"); #include <linux/pci.h> #include <linux/acpi.h> @@ -117,7 +117,7 @@ static union acpi_object *amdgpu_atif_ca if (ACPI_FAILURE(status) && status != AE_NOT_FOUND) { DRM_DEBUG_DRIVER("failed to evaluate ATIF got %s\n", acpi_format_exception(status)); - kfree(buffer.pointer); + ACPI_FREE(buffer.pointer); return NULL; } @@ -206,7 +206,7 @@ static int amdgpu_atif_verify_interface( amdgpu_atif_parse_functions(&atif->functions, output.function_bits); out: - kfree(info); + ACPI_FREE(info); return err; } @@ -299,7 +299,7 @@ out: DRM_DEBUG_DRIVER("Notification %s, command code = %#x\n", (n->enabled ? "enabled" : "disabled"), n->command_code); - kfree(info); + ACPI_FREE(info); return err; } @@ -358,7 +358,7 @@ static int amdgpu_atif_query_backlight_c atif->backlight_caps.max_input_signal = characteristics.max_input_signal; out: - kfree(info); + ACPI_FREE(info); return err; } @@ -399,7 +399,7 @@ static int amdgpu_atif_get_sbios_request count = hweight32(req->pending); out: - kfree(info); + ACPI_FREE(info); return count; } @@ -530,7 +530,7 @@ static union acpi_object *amdgpu_atcs_ca if (ACPI_FAILURE(status) && status != AE_NOT_FOUND) { DRM_DEBUG_DRIVER("failed to evaluate ATCS got %s\n", acpi_format_exception(status)); - kfree(buffer.pointer); + ACPI_FREE(buffer.pointer); return NULL; } @@ -596,7 +596,7 @@ static int amdgpu_atcs_verify_interface( amdgpu_atcs_parse_functions(&atcs->functions, output.function_bits); out: - kfree(info); + ACPI_FREE(info); return err; } @@ -646,7 +646,7 @@ int amdgpu_acpi_pcie_notify_device_ready if (!info) return -EIO; - kfree(info); + ACPI_FREE(info); return 0; } @@ -708,14 +708,14 @@ int amdgpu_acpi_pcie_performance_request size = *(u16 *) info->buffer.pointer; if (size < 3) { DRM_INFO("ATCS buffer is too small: %zu\n", size); - kfree(info); + ACPI_FREE(info); return -EINVAL; } size = min(sizeof(atcs_output), size); memcpy(&atcs_output, info->buffer.pointer, size); - kfree(info); + ACPI_FREE(info); switch (atcs_output.ret_val) { case ATCS_REQUEST_REFUSED: Index: src/sys/external/bsd/drm2/dist/drm/nouveau/nouveau_acpi.c diff -u src/sys/external/bsd/drm2/dist/drm/nouveau/nouveau_acpi.c:1.3 src/sys/external/bsd/drm2/dist/drm/nouveau/nouveau_acpi.c:1.4 --- src/sys/external/bsd/drm2/dist/drm/nouveau/nouveau_acpi.c:1.3 Sat Dec 18 23:45:32 2021 +++ src/sys/external/bsd/drm2/dist/drm/nouveau/nouveau_acpi.c Sun Feb 27 14:24:27 2022 @@ -1,8 +1,8 @@ -/* $NetBSD: nouveau_acpi.c,v 1.3 2021/12/18 23:45:32 riastradh Exp $ */ +/* $NetBSD: nouveau_acpi.c,v 1.4 2022/02/27 14:24:27 riastradh Exp $ */ // SPDX-License-Identifier: MIT #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: nouveau_acpi.c,v 1.3 2021/12/18 23:45:32 riastradh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: nouveau_acpi.c,v 1.4 2022/02/27 14:24:27 riastradh Exp $"); #include <linux/pci.h> #include <linux/acpi.h> @@ -417,7 +417,7 @@ static int nouveau_rom_call(acpi_handle obj = (union acpi_object *)buffer.pointer; len = min(len, (int)obj->buffer.length); memcpy(bios+offset, obj->buffer.pointer, len); - kfree(buffer.pointer); + ACPI_FREE(buffer.pointer); return len; } Index: src/sys/external/bsd/drm2/dist/drm/nouveau/nvkm/subdev/mxm/nouveau_nvkm_subdev_mxm_base.c diff -u src/sys/external/bsd/drm2/dist/drm/nouveau/nvkm/subdev/mxm/nouveau_nvkm_subdev_mxm_base.c:1.3 src/sys/external/bsd/drm2/dist/drm/nouveau/nvkm/subdev/mxm/nouveau_nvkm_subdev_mxm_base.c:1.4 --- src/sys/external/bsd/drm2/dist/drm/nouveau/nvkm/subdev/mxm/nouveau_nvkm_subdev_mxm_base.c:1.3 Sat Dec 18 23:45:41 2021 +++ src/sys/external/bsd/drm2/dist/drm/nouveau/nvkm/subdev/mxm/nouveau_nvkm_subdev_mxm_base.c Sun Feb 27 14:24:27 2022 @@ -1,4 +1,4 @@ -/* $NetBSD: nouveau_nvkm_subdev_mxm_base.c,v 1.3 2021/12/18 23:45:41 riastradh Exp $ */ +/* $NetBSD: nouveau_nvkm_subdev_mxm_base.c,v 1.4 2022/02/27 14:24:27 riastradh Exp $ */ /* * Copyright 2011 Red Hat Inc. @@ -24,7 +24,7 @@ * Authors: Ben Skeggs */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: nouveau_nvkm_subdev_mxm_base.c,v 1.3 2021/12/18 23:45:41 riastradh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: nouveau_nvkm_subdev_mxm_base.c,v 1.4 2022/02/27 14:24:27 riastradh Exp $"); #include "mxms.h" @@ -158,7 +158,7 @@ wmi_wmmx_mxmi(struct nvkm_mxm *mxm, u8 v nvkm_debug(subdev, "WMMX MXMI returned non-integer\n"); } - kfree(obj); + ACPI_FREE(obj); return version; } @@ -195,7 +195,7 @@ mxm_shadow_wmi(struct nvkm_mxm *mxm, u8 obj->buffer.length, GFP_KERNEL); } - kfree(obj); + ACPI_FREE(obj); return mxm->mxms != NULL; } #endif Index: src/sys/external/bsd/drm2/dist/drm/radeon/radeon_acpi.c diff -u src/sys/external/bsd/drm2/dist/drm/radeon/radeon_acpi.c:1.3 src/sys/external/bsd/drm2/dist/drm/radeon/radeon_acpi.c:1.4 --- src/sys/external/bsd/drm2/dist/drm/radeon/radeon_acpi.c:1.3 Sat Dec 18 23:45:43 2021 +++ src/sys/external/bsd/drm2/dist/drm/radeon/radeon_acpi.c Sun Feb 27 14:24:27 2022 @@ -1,4 +1,4 @@ -/* $NetBSD: radeon_acpi.c,v 1.3 2021/12/18 23:45:43 riastradh Exp $ */ +/* $NetBSD: radeon_acpi.c,v 1.4 2022/02/27 14:24:27 riastradh Exp $ */ /* * Copyright 2012 Advanced Micro Devices, Inc. @@ -24,7 +24,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: radeon_acpi.c,v 1.3 2021/12/18 23:45:43 riastradh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: radeon_acpi.c,v 1.4 2022/02/27 14:24:27 riastradh Exp $"); #include <linux/acpi.h> #include <linux/pci.h> @@ -147,7 +147,7 @@ static union acpi_object *radeon_atif_ca if (ACPI_FAILURE(status) && status != AE_NOT_FOUND) { DRM_DEBUG_DRIVER("failed to evaluate ATIF got %s\n", acpi_format_exception(status)); - kfree(buffer.pointer); + ACPI_FREE(buffer.pointer); return NULL; } @@ -243,7 +243,7 @@ static int radeon_atif_verify_interface( radeon_atif_parse_functions(&atif->functions, output.function_bits); out: - kfree(info); + ACPI_FREE(info); return err; } @@ -306,7 +306,7 @@ out: DRM_DEBUG_DRIVER("Notification %s, command code = %#x\n", (n->enabled ? "enabled" : "disabled"), n->command_code); - kfree(info); + ACPI_FREE(info); return err; } @@ -346,7 +346,7 @@ static int radeon_atif_get_sbios_request count = hweight32(req->pending); out: - kfree(info); + ACPI_FREE(info); return count; } @@ -472,7 +472,7 @@ static union acpi_object *radeon_atcs_ca if (ACPI_FAILURE(status) && status != AE_NOT_FOUND) { DRM_DEBUG_DRIVER("failed to evaluate ATCS got %s\n", acpi_format_exception(status)); - kfree(buffer.pointer); + ACPI_FREE(buffer.pointer); return NULL; } @@ -538,7 +538,7 @@ static int radeon_atcs_verify_interface( radeon_atcs_parse_functions(&atcs->functions, output.function_bits); out: - kfree(info); + ACPI_FREE(info); return err; } @@ -588,7 +588,7 @@ int radeon_acpi_pcie_notify_device_ready if (!info) return -EIO; - kfree(info); + ACPI_FREE(info); return 0; } @@ -647,14 +647,14 @@ int radeon_acpi_pcie_performance_request size = *(u16 *) info->buffer.pointer; if (size < 3) { DRM_INFO("ATCS buffer is too small: %zu\n", size); - kfree(info); + ACPI_FREE(info); return -EINVAL; } size = min(sizeof(atcs_output), size); memcpy(&atcs_output, info->buffer.pointer, size); - kfree(info); + ACPI_FREE(info); switch (atcs_output.ret_val) { case ATCS_REQUEST_REFUSED: Index: src/sys/external/bsd/drm2/dist/drm/radeon/radeon_bios.c diff -u src/sys/external/bsd/drm2/dist/drm/radeon/radeon_bios.c:1.9 src/sys/external/bsd/drm2/dist/drm/radeon/radeon_bios.c:1.10 --- src/sys/external/bsd/drm2/dist/drm/radeon/radeon_bios.c:1.9 Sun Feb 27 14:23:16 2022 +++ src/sys/external/bsd/drm2/dist/drm/radeon/radeon_bios.c Sun Feb 27 14:24:27 2022 @@ -1,4 +1,4 @@ -/* $NetBSD: radeon_bios.c,v 1.9 2022/02/27 14:23:16 riastradh Exp $ */ +/* $NetBSD: radeon_bios.c,v 1.10 2022/02/27 14:24:27 riastradh Exp $ */ /* * Copyright 2008 Advanced Micro Devices, Inc. @@ -29,7 +29,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: radeon_bios.c,v 1.9 2022/02/27 14:23:16 riastradh Exp $"); +__KERNEL_RCSID(0, "$NetBSD: radeon_bios.c,v 1.10 2022/02/27 14:24:27 riastradh Exp $"); #include <linux/acpi.h> #include <linux/pci.h> @@ -229,7 +229,7 @@ static int radeon_atrm_call(acpi_handle obj = (union acpi_object *)buffer.pointer; memcpy(bios+offset, obj->buffer.pointer, obj->buffer.length); len = obj->buffer.length; - kfree(buffer.pointer); + ACPI_FREE(buffer.pointer); return len; }