From nobody Sun Feb 8 22:08:23 2026 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 66B422F56 for ; Fri, 31 Jan 2025 22:31:46 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738362706; cv=none; b=QtJbQxAVv4ou2t7pswnCxCJv8+xzOmdiCMUI/SYhlNBswHp3ov73Y6GFtkH0b1y4GoHa7nNR5AX94whdqzrcpKZ+GVH8gvN4SZwbD+8tp42zzr5fAmn3eyNunUi7boRf/F2AAOvu+jYztJZTmdVHT57kMsevdfllfhYi6g5Ugrc= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1738362706; c=relaxed/simple; bh=5WW6WvPEP+frdA3hs61x3sVE8CtyeNxuUnRpo9yZHgY=; h=From:Date:Subject:MIME-Version:Content-Type:Message-Id:To:Cc; b=mm1v7t6InGp1Rjlpfgqv6QpPLhaevGHvwJiHJI/7Pl56KZo+MWL4lIJw5lFzwOlqfCJDOx6zS2F4loxH/KXDvGQmLkX8mvGe2wfKtT4D9IhSUT1Z4rNI5IQNtKwgoQtMVndXJBnpNx6eNXDiGlJ+JtNzWSk3UHwq/8UJmZfdqkU= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=o7NKj2V9; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="o7NKj2V9" Received: by smtp.kernel.org (Postfix) with ESMTPSA id C011DC4CED1; Fri, 31 Jan 2025 22:31:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1738362705; bh=5WW6WvPEP+frdA3hs61x3sVE8CtyeNxuUnRpo9yZHgY=; h=From:Date:Subject:To:Cc:From; b=o7NKj2V9NreCfPwjo/ABeoPJeNsHYZ0OfjLHi1gpEhBUpyCk6dQlyGF7SHYw1NVDV ttFuFLpt2lcMNDRTcCYBQ/X/GDQdj9rJN+Wc3JDXNqDlo9M2FfRzJnW2j1YJeVpffI wcOFuTkYK8vA0D6LyWrIsdcabqHeZ6eqJKXiL8T/6bGEQd4i9+djXCIo8GgR8qcz/k WTsUiB0zXqpjta3TqqaDxy+VQFXOpxs7o/+jVcoCvBLZox56Y9VeQBpR4DEQpdVdSX G8mmKdyVZb9Lg0GUBFMxka5sH1P0wrMDur9ype0jgB+2Y2LyCh+cuw3jpGKtYnFK9e B6WdT/jpDmHbA== From: Nathan Chancellor Date: Fri, 31 Jan 2025 15:31:19 -0700 Subject: [PATCH] drm/amd/display: Respect user's CONFIG_FRAME_WARN more for dml files Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <20250131-amdgpu-respect-config_frame_warn-v1-1-68255384492b@kernel.org> X-B4-Tracking: v=1; b=H4sIADZPnWcC/x3NQQqDMBBA0avIrDtgorWkVylFpnESZ2EMk2oL4 t0bunyb/w8orMIF7s0ByrsUWVOFuTTgZ0qRUaZqsK29tqYzSMsU84bKJbN/o19TkDgGpYXHD2n CW+fIvWzfu4GhZrJykO9/8Xie5w8IikIQcgAAAA== X-Change-ID: 20250131-amdgpu-respect-config_frame_warn-739a9b24496e To: Jun Lei , Harry Wentland , Leo Li , Rodrigo Siqueira , Alex Deucher , =?utf-8?q?Christian_K=C3=B6nig?= Cc: amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman , Nathan Chancellor X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=openpgp-sha256; l=3488; i=nathan@kernel.org; h=from:subject:message-id; bh=5WW6WvPEP+frdA3hs61x3sVE8CtyeNxuUnRpo9yZHgY=; b=owGbwMvMwCUmm602sfCA1DTG02pJDOlz/f117m+ctIPHqld824FJlft5thWGpTBmVOfOmn98y jwRfrfujlIWBjEuBlkxRZbqx6rHDQ3nnGW8cWoSzBxWJpAhDFycAjARib+MDI314Xv0ryzi+fBQ Lu7x/R96lQ62Lixz956rN9BvEL0TFcDw38/dc7vGhGrT6K2+FlPDOaWftFofkl4zre/v2wcniov dmQA= X-Developer-Key: i=nathan@kernel.org; a=openpgp; fpr=2437CB76E544CB6AB3D9DFD399739260CB6CB716 Currently, there are several files in drm/amd/display that aim to have a higher -Wframe-larger-than value to avoid instances of that warning with a lower value from the user's configuration. However, with the way that it is currently implemented, it does not respect the user's request via CONFIG_FRAME_WARN for a higher stack frame limit, which can cause pain when new instances of the warning appear and break the build due to CONFIG_WERROR. Adjust the logic to switch from a hard coded -Wframe-larger-than value to only using the value as a minimum clamp and deferring to the requested value from CONFIG_FRAME_WARN if it is higher. Suggested-by: Harry Wentland Reported-by: Greg Kroah-Hartman Closes: https://lore.kernel.org/2025013003-audience-opposing-7f95@gregkh/ Signed-off-by: Nathan Chancellor --- drivers/gpu/drm/amd/display/dc/dml/Makefile | 14 +++++++++----- drivers/gpu/drm/amd/display/dc/dml2/Makefile | 22 +++++++++++++--------- 2 files changed, 22 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/amd/display/dc/dml/Makefile b/drivers/gpu/drm/= amd/display/dc/dml/Makefile index 46f9c05de16e8c9035f9e26c0b5c481c274d52ef..e1d500633dfad75e4f2265552be= 42a3e19dee6bf 100644 --- a/drivers/gpu/drm/amd/display/dc/dml/Makefile +++ b/drivers/gpu/drm/amd/display/dc/dml/Makefile @@ -29,11 +29,15 @@ dml_ccflags :=3D $(CC_FLAGS_FPU) dml_rcflags :=3D $(CC_FLAGS_NO_FPU) =20 ifneq ($(CONFIG_FRAME_WARN),0) -ifeq ($(filter y,$(CONFIG_KASAN)$(CONFIG_KCSAN)),y) -frame_warn_flag :=3D -Wframe-larger-than=3D3072 -else -frame_warn_flag :=3D -Wframe-larger-than=3D2048 -endif + ifeq ($(filter y,$(CONFIG_KASAN)$(CONFIG_KCSAN)),y) + frame_warn_limit :=3D 3072 + else + frame_warn_limit :=3D 2048 + endif + + ifeq ($(call test-lt, $(CONFIG_FRAME_WARN), $(frame_warn_limit)),y) + frame_warn_flag :=3D -Wframe-larger-than=3D$(frame_warn_limit) + endif endif =20 CFLAGS_$(AMDDALPATH)/dc/dml/display_mode_lib.o :=3D $(dml_ccflags) diff --git a/drivers/gpu/drm/amd/display/dc/dml2/Makefile b/drivers/gpu/drm= /amd/display/dc/dml2/Makefile index 91c4f3b4bd5f46ac5c1c74f665b06dbe61081917..21fd466dba26ef3359196d0b26b= c29125bb1507a 100644 --- a/drivers/gpu/drm/amd/display/dc/dml2/Makefile +++ b/drivers/gpu/drm/amd/display/dc/dml2/Makefile @@ -28,15 +28,19 @@ dml2_ccflags :=3D $(CC_FLAGS_FPU) dml2_rcflags :=3D $(CC_FLAGS_NO_FPU) =20 ifneq ($(CONFIG_FRAME_WARN),0) -ifeq ($(filter y,$(CONFIG_KASAN)$(CONFIG_KCSAN)),y) -ifeq ($(CONFIG_CC_IS_CLANG)$(CONFIG_COMPILE_TEST),yy) -frame_warn_flag :=3D -Wframe-larger-than=3D4096 -else -frame_warn_flag :=3D -Wframe-larger-than=3D3072 -endif -else -frame_warn_flag :=3D -Wframe-larger-than=3D2048 -endif + ifeq ($(filter y,$(CONFIG_KASAN)$(CONFIG_KCSAN)),y) + ifeq ($(CONFIG_CC_IS_CLANG)$(CONFIG_COMPILE_TEST),yy) + frame_warn_limit :=3D 4096 + else + frame_warn_limit :=3D 3072 + endif + else + frame_warn_limit :=3D 2048 + endif + + ifeq ($(call test-lt, $(CONFIG_FRAME_WARN), $(frame_warn_limit)),y) + frame_warn_flag :=3D -Wframe-larger-than=3D$(frame_warn_limit) + endif endif =20 subdir-ccflags-y +=3D -I$(FULL_AMD_DISPLAY_PATH)/dc/dml2 --- base-commit: 7f2b5237e313e39008a85b33ca94ab503a8fdff9 change-id: 20250131-amdgpu-respect-config_frame_warn-739a9b24496e Best regards, --=20 Nathan Chancellor