* [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-25 12:07 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
builds. This leads to some interesting failures, each resolved in this set.
With this set applied, I am able to obtain a successful allmodconfig Arm build.
v1 => v2:
- Remove superfluous change (these two override it)
- Mark only DRM_AMD_DC ("the new display engine) as Broken
- Change logic to only *include* working arches, not *preclude* them
Lee Jones (2):
drm/amdgpu: Temporarily disable broken Clang builds due to blown
stack-frame
Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is
enabled
drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
lib/Kconfig.debug | 1 +
2 files changed, 8 insertions(+)
--
2.38.1.584.g0f3c55d4c2-goog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-25 12:07 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: amd-gfx, dri-devel, llvm, linux-kernel, linux-arm-kernel
Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
builds. This leads to some interesting failures, each resolved in this set.
With this set applied, I am able to obtain a successful allmodconfig Arm build.
v1 => v2:
- Remove superfluous change (these two override it)
- Mark only DRM_AMD_DC ("the new display engine) as Broken
- Change logic to only *include* working arches, not *preclude* them
Lee Jones (2):
drm/amdgpu: Temporarily disable broken Clang builds due to blown
stack-frame
Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is
enabled
drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
lib/Kconfig.debug | 1 +
2 files changed, 8 insertions(+)
--
2.38.1.584.g0f3c55d4c2-goog
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
2022-11-25 12:07 ` Lee Jones
(?)
@ 2022-11-25 12:07 ` Lee Jones
-1 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
architectures built with Clang (all released versions), whereby the stack
frame gets blown up to well over 5k. This would cause an immediate kernel
panic on most architectures. We'll revert this when the following bug report
has been resolved: https://github.com/llvm/llvm-project/issues/41896.
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Lee Jones <lee@kernel.org>
---
drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
index 6925e0280dbe6..f4f3d2665a6b2 100644
--- a/drivers/gpu/drm/amd/display/Kconfig
+++ b/drivers/gpu/drm/amd/display/Kconfig
@@ -5,6 +5,7 @@ menu "Display Engine Configuration"
config DRM_AMD_DC
bool "AMD DC - Enable new display engine"
default y
+ depends on BROKEN || !CC_IS_CLANG || X86_64 || SPARC64 || ARM64
select SND_HDA_COMPONENT if SND_HDA_CORE
select DRM_AMD_DC_DCN if (X86 || PPC_LONG_DOUBLE_128)
help
@@ -12,6 +13,12 @@ config DRM_AMD_DC
support for AMDGPU. This adds required support for Vega and
Raven ASICs.
+ calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
+ architectures built with Clang (all released versions), whereby the stack
+ frame gets blown up to well over 5k. This would cause an immediate kernel
+ panic on most architectures. We'll revert this when the following bug report
+ has been resolved: https://github.com/llvm/llvm-project/issues/41896.
+
config DRM_AMD_DC_DCN
def_bool n
help
--
2.38.1.584.g0f3c55d4c2-goog
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
@ 2022-11-25 12:07 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
architectures built with Clang (all released versions), whereby the stack
frame gets blown up to well over 5k. This would cause an immediate kernel
panic on most architectures. We'll revert this when the following bug report
has been resolved: https://github.com/llvm/llvm-project/issues/41896.
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Lee Jones <lee@kernel.org>
---
drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
index 6925e0280dbe6..f4f3d2665a6b2 100644
--- a/drivers/gpu/drm/amd/display/Kconfig
+++ b/drivers/gpu/drm/amd/display/Kconfig
@@ -5,6 +5,7 @@ menu "Display Engine Configuration"
config DRM_AMD_DC
bool "AMD DC - Enable new display engine"
default y
+ depends on BROKEN || !CC_IS_CLANG || X86_64 || SPARC64 || ARM64
select SND_HDA_COMPONENT if SND_HDA_CORE
select DRM_AMD_DC_DCN if (X86 || PPC_LONG_DOUBLE_128)
help
@@ -12,6 +13,12 @@ config DRM_AMD_DC
support for AMDGPU. This adds required support for Vega and
Raven ASICs.
+ calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
+ architectures built with Clang (all released versions), whereby the stack
+ frame gets blown up to well over 5k. This would cause an immediate kernel
+ panic on most architectures. We'll revert this when the following bug report
+ has been resolved: https://github.com/llvm/llvm-project/issues/41896.
+
config DRM_AMD_DC_DCN
def_bool n
help
--
2.38.1.584.g0f3c55d4c2-goog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
@ 2022-11-25 12:07 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: amd-gfx, dri-devel, llvm, linux-kernel, linux-arm-kernel
calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
architectures built with Clang (all released versions), whereby the stack
frame gets blown up to well over 5k. This would cause an immediate kernel
panic on most architectures. We'll revert this when the following bug report
has been resolved: https://github.com/llvm/llvm-project/issues/41896.
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Lee Jones <lee@kernel.org>
---
drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
index 6925e0280dbe6..f4f3d2665a6b2 100644
--- a/drivers/gpu/drm/amd/display/Kconfig
+++ b/drivers/gpu/drm/amd/display/Kconfig
@@ -5,6 +5,7 @@ menu "Display Engine Configuration"
config DRM_AMD_DC
bool "AMD DC - Enable new display engine"
default y
+ depends on BROKEN || !CC_IS_CLANG || X86_64 || SPARC64 || ARM64
select SND_HDA_COMPONENT if SND_HDA_CORE
select DRM_AMD_DC_DCN if (X86 || PPC_LONG_DOUBLE_128)
help
@@ -12,6 +13,12 @@ config DRM_AMD_DC
support for AMDGPU. This adds required support for Vega and
Raven ASICs.
+ calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
+ architectures built with Clang (all released versions), whereby the stack
+ frame gets blown up to well over 5k. This would cause an immediate kernel
+ panic on most architectures. We'll revert this when the following bug report
+ has been resolved: https://github.com/llvm/llvm-project/issues/41896.
+
config DRM_AMD_DC_DCN
def_bool n
help
--
2.38.1.584.g0f3c55d4c2-goog
^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
2022-11-25 12:07 ` Lee Jones
(?)
@ 2022-11-25 12:11 ` Arnd Bergmann
-1 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2022-11-25 12:11 UTC (permalink / raw)
To: Lee Jones, Andrew Morton, Nathan Chancellor, Maarten Lankhorst,
mripard, Thomas Zimmermann, Dave Airlie, Daniel Vetter,
Nick Desaulniers, Tom Rix, Harry Wentland, Leo Li,
Rodrigo Siqueira, Alex Deucher, Christian König, Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
On Fri, Nov 25, 2022, at 13:07, Lee Jones wrote:
> calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> architectures built with Clang (all released versions), whereby the stack
> frame gets blown up to well over 5k. This would cause an immediate kernel
> panic on most architectures. We'll revert this when the following bug report
> has been resolved: https://github.com/llvm/llvm-project/issues/41896.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Lee Jones <lee@kernel.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
@ 2022-11-25 12:11 ` Arnd Bergmann
0 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2022-11-25 12:11 UTC (permalink / raw)
To: Lee Jones, Andrew Morton, Nathan Chancellor, Maarten Lankhorst,
mripard, Thomas Zimmermann, Dave Airlie, Daniel Vetter,
Nick Desaulniers, Tom Rix, Harry Wentland, Leo Li,
Rodrigo Siqueira, Alex Deucher, Christian König, Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
On Fri, Nov 25, 2022, at 13:07, Lee Jones wrote:
> calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> architectures built with Clang (all released versions), whereby the stack
> frame gets blown up to well over 5k. This would cause an immediate kernel
> panic on most architectures. We'll revert this when the following bug report
> has been resolved: https://github.com/llvm/llvm-project/issues/41896.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Lee Jones <lee@kernel.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
@ 2022-11-25 12:11 ` Arnd Bergmann
0 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2022-11-25 12:11 UTC (permalink / raw)
To: Lee Jones, Andrew Morton, Nathan Chancellor, Maarten Lankhorst,
mripard, Thomas Zimmermann, Dave Airlie, Daniel Vetter,
Nick Desaulniers, Tom Rix, Harry Wentland, Leo Li,
Rodrigo Siqueira, Alex Deucher, Christian König, Xinhui.Pan
Cc: amd-gfx, dri-devel, llvm, linux-kernel, linux-arm-kernel
On Fri, Nov 25, 2022, at 13:07, Lee Jones wrote:
> calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> architectures built with Clang (all released versions), whereby the stack
> frame gets blown up to well over 5k. This would cause an immediate kernel
> panic on most architectures. We'll revert this when the following bug report
> has been resolved: https://github.com/llvm/llvm-project/issues/41896.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Lee Jones <lee@kernel.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
2022-11-25 12:07 ` Lee Jones
(?)
(?)
@ 2022-11-28 5:52 ` Nathan Chancellor
-1 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:52 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, akpm, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, Nov 25, 2022 at 12:07:49PM +0000, Lee Jones wrote:
> calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> architectures built with Clang (all released versions), whereby the stack
> frame gets blown up to well over 5k. This would cause an immediate kernel
> panic on most architectures. We'll revert this when the following bug report
> has been resolved: https://github.com/llvm/llvm-project/issues/41896.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
> index 6925e0280dbe6..f4f3d2665a6b2 100644
> --- a/drivers/gpu/drm/amd/display/Kconfig
> +++ b/drivers/gpu/drm/amd/display/Kconfig
> @@ -5,6 +5,7 @@ menu "Display Engine Configuration"
> config DRM_AMD_DC
> bool "AMD DC - Enable new display engine"
> default y
> + depends on BROKEN || !CC_IS_CLANG || X86_64 || SPARC64 || ARM64
> select SND_HDA_COMPONENT if SND_HDA_CORE
> select DRM_AMD_DC_DCN if (X86 || PPC_LONG_DOUBLE_128)
> help
> @@ -12,6 +13,12 @@ config DRM_AMD_DC
> support for AMDGPU. This adds required support for Vega and
> Raven ASICs.
>
> + calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> + architectures built with Clang (all released versions), whereby the stack
> + frame gets blown up to well over 5k. This would cause an immediate kernel
> + panic on most architectures. We'll revert this when the following bug report
> + has been resolved: https://github.com/llvm/llvm-project/issues/41896.
> +
> config DRM_AMD_DC_DCN
> def_bool n
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
@ 2022-11-28 5:52 ` Nathan Chancellor
0 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:52 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, akpm, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, Nov 25, 2022 at 12:07:49PM +0000, Lee Jones wrote:
> calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> architectures built with Clang (all released versions), whereby the stack
> frame gets blown up to well over 5k. This would cause an immediate kernel
> panic on most architectures. We'll revert this when the following bug report
> has been resolved: https://github.com/llvm/llvm-project/issues/41896.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
> index 6925e0280dbe6..f4f3d2665a6b2 100644
> --- a/drivers/gpu/drm/amd/display/Kconfig
> +++ b/drivers/gpu/drm/amd/display/Kconfig
> @@ -5,6 +5,7 @@ menu "Display Engine Configuration"
> config DRM_AMD_DC
> bool "AMD DC - Enable new display engine"
> default y
> + depends on BROKEN || !CC_IS_CLANG || X86_64 || SPARC64 || ARM64
> select SND_HDA_COMPONENT if SND_HDA_CORE
> select DRM_AMD_DC_DCN if (X86 || PPC_LONG_DOUBLE_128)
> help
> @@ -12,6 +13,12 @@ config DRM_AMD_DC
> support for AMDGPU. This adds required support for Vega and
> Raven ASICs.
>
> + calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> + architectures built with Clang (all released versions), whereby the stack
> + frame gets blown up to well over 5k. This would cause an immediate kernel
> + panic on most architectures. We'll revert this when the following bug report
> + has been resolved: https://github.com/llvm/llvm-project/issues/41896.
> +
> config DRM_AMD_DC_DCN
> def_bool n
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
@ 2022-11-28 5:52 ` Nathan Chancellor
0 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:52 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, daniel, arnd, sunpeng.li, ndesaulniers, dri-devel,
Xinhui.Pan, maarten.lankhorst, linux-kernel, mripard,
harry.wentland, amd-gfx, tzimmermann, trix, alexander.deucher,
akpm, Rodrigo.Siqueira, airlied, christian.koenig,
linux-arm-kernel
On Fri, Nov 25, 2022 at 12:07:49PM +0000, Lee Jones wrote:
> calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> architectures built with Clang (all released versions), whereby the stack
> frame gets blown up to well over 5k. This would cause an immediate kernel
> panic on most architectures. We'll revert this when the following bug report
> has been resolved: https://github.com/llvm/llvm-project/issues/41896.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
> index 6925e0280dbe6..f4f3d2665a6b2 100644
> --- a/drivers/gpu/drm/amd/display/Kconfig
> +++ b/drivers/gpu/drm/amd/display/Kconfig
> @@ -5,6 +5,7 @@ menu "Display Engine Configuration"
> config DRM_AMD_DC
> bool "AMD DC - Enable new display engine"
> default y
> + depends on BROKEN || !CC_IS_CLANG || X86_64 || SPARC64 || ARM64
> select SND_HDA_COMPONENT if SND_HDA_CORE
> select DRM_AMD_DC_DCN if (X86 || PPC_LONG_DOUBLE_128)
> help
> @@ -12,6 +13,12 @@ config DRM_AMD_DC
> support for AMDGPU. This adds required support for Vega and
> Raven ASICs.
>
> + calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> + architectures built with Clang (all released versions), whereby the stack
> + frame gets blown up to well over 5k. This would cause an immediate kernel
> + panic on most architectures. We'll revert this when the following bug report
> + has been resolved: https://github.com/llvm/llvm-project/issues/41896.
> +
> config DRM_AMD_DC_DCN
> def_bool n
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 1/2] drm/amdgpu: Temporarily disable broken Clang builds due to blown stack-frame
@ 2022-11-28 5:52 ` Nathan Chancellor
0 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:52 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, arnd, sunpeng.li, ndesaulniers, dri-devel, Xinhui.Pan,
linux-kernel, amd-gfx, tzimmermann, trix, alexander.deucher,
akpm, Rodrigo.Siqueira, christian.koenig, linux-arm-kernel
On Fri, Nov 25, 2022 at 12:07:49PM +0000, Lee Jones wrote:
> calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> architectures built with Clang (all released versions), whereby the stack
> frame gets blown up to well over 5k. This would cause an immediate kernel
> panic on most architectures. We'll revert this when the following bug report
> has been resolved: https://github.com/llvm/llvm-project/issues/41896.
>
> Suggested-by: Arnd Bergmann <arnd@arndb.de>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> drivers/gpu/drm/amd/display/Kconfig | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig
> index 6925e0280dbe6..f4f3d2665a6b2 100644
> --- a/drivers/gpu/drm/amd/display/Kconfig
> +++ b/drivers/gpu/drm/amd/display/Kconfig
> @@ -5,6 +5,7 @@ menu "Display Engine Configuration"
> config DRM_AMD_DC
> bool "AMD DC - Enable new display engine"
> default y
> + depends on BROKEN || !CC_IS_CLANG || X86_64 || SPARC64 || ARM64
> select SND_HDA_COMPONENT if SND_HDA_CORE
> select DRM_AMD_DC_DCN if (X86 || PPC_LONG_DOUBLE_128)
> help
> @@ -12,6 +13,12 @@ config DRM_AMD_DC
> support for AMDGPU. This adds required support for Vega and
> Raven ASICs.
>
> + calculate_bandwidth() is presently broken on all !(X86_64 || SPARC64 || ARM64)
> + architectures built with Clang (all released versions), whereby the stack
> + frame gets blown up to well over 5k. This would cause an immediate kernel
> + panic on most architectures. We'll revert this when the following bug report
> + has been resolved: https://github.com/llvm/llvm-project/issues/41896.
> +
> config DRM_AMD_DC_DCN
> def_bool n
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
2022-11-25 12:07 ` Lee Jones
(?)
@ 2022-11-25 12:07 ` Lee Jones
-1 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
When enabled, KASAN enlarges function's stack-frames. Pushing quite a
few over the current threshold. This can mainly be seen on 32-bit
architectures where the present limit (when !GCC) is a lowly
1024-Bytes.
Signed-off-by: Lee Jones <lee@kernel.org>
---
lib/Kconfig.debug | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index c3c0b077ade33..82d475168db95 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -399,6 +399,7 @@ config FRAME_WARN
default 2048 if GCC_PLUGIN_LATENT_ENTROPY
default 2048 if PARISC
default 1536 if (!64BIT && XTENSA)
+ default 1280 if KASAN && !64BIT
default 1024 if !64BIT
default 2048 if 64BIT
help
--
2.38.1.584.g0f3c55d4c2-goog
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-25 12:07 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
When enabled, KASAN enlarges function's stack-frames. Pushing quite a
few over the current threshold. This can mainly be seen on 32-bit
architectures where the present limit (when !GCC) is a lowly
1024-Bytes.
Signed-off-by: Lee Jones <lee@kernel.org>
---
lib/Kconfig.debug | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index c3c0b077ade33..82d475168db95 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -399,6 +399,7 @@ config FRAME_WARN
default 2048 if GCC_PLUGIN_LATENT_ENTROPY
default 2048 if PARISC
default 1536 if (!64BIT && XTENSA)
+ default 1280 if KASAN && !64BIT
default 1024 if !64BIT
default 2048 if 64BIT
help
--
2.38.1.584.g0f3c55d4c2-goog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 38+ messages in thread
* [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-25 12:07 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 12:07 UTC (permalink / raw)
To: lee, arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: amd-gfx, dri-devel, llvm, linux-kernel, linux-arm-kernel
When enabled, KASAN enlarges function's stack-frames. Pushing quite a
few over the current threshold. This can mainly be seen on 32-bit
architectures where the present limit (when !GCC) is a lowly
1024-Bytes.
Signed-off-by: Lee Jones <lee@kernel.org>
---
lib/Kconfig.debug | 1 +
1 file changed, 1 insertion(+)
diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
index c3c0b077ade33..82d475168db95 100644
--- a/lib/Kconfig.debug
+++ b/lib/Kconfig.debug
@@ -399,6 +399,7 @@ config FRAME_WARN
default 2048 if GCC_PLUGIN_LATENT_ENTROPY
default 2048 if PARISC
default 1536 if (!64BIT && XTENSA)
+ default 1280 if KASAN && !64BIT
default 1024 if !64BIT
default 2048 if 64BIT
help
--
2.38.1.584.g0f3c55d4c2-goog
^ permalink raw reply related [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
2022-11-25 12:07 ` Lee Jones
(?)
@ 2022-11-25 13:40 ` Lee Jones
-1 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 13:40 UTC (permalink / raw)
To: arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
On Fri, 25 Nov 2022, Lee Jones wrote:
> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
> few over the current threshold. This can mainly be seen on 32-bit
> architectures where the present limit (when !GCC) is a lowly
> 1024-Bytes.
>
> Signed-off-by: Lee Jones <lee@kernel.org>
> ---
> lib/Kconfig.debug | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index c3c0b077ade33..82d475168db95 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -399,6 +399,7 @@ config FRAME_WARN
> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
> default 2048 if PARISC
> default 1536 if (!64BIT && XTENSA)
> + default 1280 if KASAN && !64BIT
> default 1024 if !64BIT
> default 2048 if 64BIT
> help
Note this also fixes 61 warnings when
(GCC && !GCC_PLUGIN_LATENT_ENTROPY)
... which as Arnd says should not be enabled by default. We'll
address that issue once this set has been applied.
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-25 13:40 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 13:40 UTC (permalink / raw)
To: arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
On Fri, 25 Nov 2022, Lee Jones wrote:
> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
> few over the current threshold. This can mainly be seen on 32-bit
> architectures where the present limit (when !GCC) is a lowly
> 1024-Bytes.
>
> Signed-off-by: Lee Jones <lee@kernel.org>
> ---
> lib/Kconfig.debug | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index c3c0b077ade33..82d475168db95 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -399,6 +399,7 @@ config FRAME_WARN
> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
> default 2048 if PARISC
> default 1536 if (!64BIT && XTENSA)
> + default 1280 if KASAN && !64BIT
> default 1024 if !64BIT
> default 2048 if 64BIT
> help
Note this also fixes 61 warnings when
(GCC && !GCC_PLUGIN_LATENT_ENTROPY)
... which as Arnd says should not be enabled by default. We'll
address that issue once this set has been applied.
--
Lee Jones [李琼斯]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-25 13:40 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-25 13:40 UTC (permalink / raw)
To: arnd, akpm, nathan, maarten.lankhorst, mripard, tzimmermann,
airlied, daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan
Cc: amd-gfx, dri-devel, llvm, linux-kernel, linux-arm-kernel
On Fri, 25 Nov 2022, Lee Jones wrote:
> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
> few over the current threshold. This can mainly be seen on 32-bit
> architectures where the present limit (when !GCC) is a lowly
> 1024-Bytes.
>
> Signed-off-by: Lee Jones <lee@kernel.org>
> ---
> lib/Kconfig.debug | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index c3c0b077ade33..82d475168db95 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -399,6 +399,7 @@ config FRAME_WARN
> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
> default 2048 if PARISC
> default 1536 if (!64BIT && XTENSA)
> + default 1280 if KASAN && !64BIT
> default 1024 if !64BIT
> default 2048 if 64BIT
> help
Note this also fixes 61 warnings when
(GCC && !GCC_PLUGIN_LATENT_ENTROPY)
... which as Arnd says should not be enabled by default. We'll
address that issue once this set has been applied.
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
2022-11-25 13:40 ` Lee Jones
(?)
@ 2022-11-25 13:57 ` Arnd Bergmann
-1 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2022-11-25 13:57 UTC (permalink / raw)
To: Lee Jones, Andrew Morton, Nathan Chancellor, Maarten Lankhorst,
mripard, Thomas Zimmermann, Dave Airlie, Daniel Vetter,
Nick Desaulniers, Tom Rix, Harry Wentland, Leo Li,
Rodrigo Siqueira, Alex Deucher, Christian König, Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
On Fri, Nov 25, 2022, at 14:40, Lee Jones wrote:
> On Fri, 25 Nov 2022, Lee Jones wrote:
>
>> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
>> few over the current threshold. This can mainly be seen on 32-bit
>> architectures where the present limit (when !GCC) is a lowly
>> 1024-Bytes.
>>
>> Signed-off-by: Lee Jones <lee@kernel.org>
>> ---
>> lib/Kconfig.debug | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
>> index c3c0b077ade33..82d475168db95 100644
>> --- a/lib/Kconfig.debug
>> +++ b/lib/Kconfig.debug
>> @@ -399,6 +399,7 @@ config FRAME_WARN
>> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
>> default 2048 if PARISC
>> default 1536 if (!64BIT && XTENSA)
>> + default 1280 if KASAN && !64BIT
>> default 1024 if !64BIT
>> default 2048 if 64BIT
>> help
>
> Note this also fixes 61 warnings when
>
> (GCC && !GCC_PLUGIN_LATENT_ENTROPY)
>
> ... which as Arnd says should not be enabled by default. We'll
> address that issue once this set has been applied.
Thanks a lot for checking this!
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-25 13:57 ` Arnd Bergmann
0 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2022-11-25 13:57 UTC (permalink / raw)
To: Lee Jones, Andrew Morton, Nathan Chancellor, Maarten Lankhorst,
mripard, Thomas Zimmermann, Dave Airlie, Daniel Vetter,
Nick Desaulniers, Tom Rix, Harry Wentland, Leo Li,
Rodrigo Siqueira, Alex Deucher, Christian König, Xinhui.Pan
Cc: linux-arm-kernel, linux-kernel, llvm, amd-gfx, dri-devel
On Fri, Nov 25, 2022, at 14:40, Lee Jones wrote:
> On Fri, 25 Nov 2022, Lee Jones wrote:
>
>> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
>> few over the current threshold. This can mainly be seen on 32-bit
>> architectures where the present limit (when !GCC) is a lowly
>> 1024-Bytes.
>>
>> Signed-off-by: Lee Jones <lee@kernel.org>
>> ---
>> lib/Kconfig.debug | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
>> index c3c0b077ade33..82d475168db95 100644
>> --- a/lib/Kconfig.debug
>> +++ b/lib/Kconfig.debug
>> @@ -399,6 +399,7 @@ config FRAME_WARN
>> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
>> default 2048 if PARISC
>> default 1536 if (!64BIT && XTENSA)
>> + default 1280 if KASAN && !64BIT
>> default 1024 if !64BIT
>> default 2048 if 64BIT
>> help
>
> Note this also fixes 61 warnings when
>
> (GCC && !GCC_PLUGIN_LATENT_ENTROPY)
>
> ... which as Arnd says should not be enabled by default. We'll
> address that issue once this set has been applied.
Thanks a lot for checking this!
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-25 13:57 ` Arnd Bergmann
0 siblings, 0 replies; 38+ messages in thread
From: Arnd Bergmann @ 2022-11-25 13:57 UTC (permalink / raw)
To: Lee Jones, Andrew Morton, Nathan Chancellor, Maarten Lankhorst,
mripard, Thomas Zimmermann, Dave Airlie, Daniel Vetter,
Nick Desaulniers, Tom Rix, Harry Wentland, Leo Li,
Rodrigo Siqueira, Alex Deucher, Christian König, Xinhui.Pan
Cc: amd-gfx, dri-devel, llvm, linux-kernel, linux-arm-kernel
On Fri, Nov 25, 2022, at 14:40, Lee Jones wrote:
> On Fri, 25 Nov 2022, Lee Jones wrote:
>
>> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
>> few over the current threshold. This can mainly be seen on 32-bit
>> architectures where the present limit (when !GCC) is a lowly
>> 1024-Bytes.
>>
>> Signed-off-by: Lee Jones <lee@kernel.org>
>> ---
>> lib/Kconfig.debug | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
>> index c3c0b077ade33..82d475168db95 100644
>> --- a/lib/Kconfig.debug
>> +++ b/lib/Kconfig.debug
>> @@ -399,6 +399,7 @@ config FRAME_WARN
>> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
>> default 2048 if PARISC
>> default 1536 if (!64BIT && XTENSA)
>> + default 1280 if KASAN && !64BIT
>> default 1024 if !64BIT
>> default 2048 if 64BIT
>> help
>
> Note this also fixes 61 warnings when
>
> (GCC && !GCC_PLUGIN_LATENT_ENTROPY)
>
> ... which as Arnd says should not be enabled by default. We'll
> address that issue once this set has been applied.
Thanks a lot for checking this!
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
2022-11-25 12:07 ` Lee Jones
(?)
(?)
@ 2022-11-28 5:53 ` Nathan Chancellor
-1 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:53 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, akpm, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, Nov 25, 2022 at 12:07:50PM +0000, Lee Jones wrote:
> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
> few over the current threshold. This can mainly be seen on 32-bit
> architectures where the present limit (when !GCC) is a lowly
> 1024-Bytes.
>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> lib/Kconfig.debug | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index c3c0b077ade33..82d475168db95 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -399,6 +399,7 @@ config FRAME_WARN
> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
> default 2048 if PARISC
> default 1536 if (!64BIT && XTENSA)
> + default 1280 if KASAN && !64BIT
> default 1024 if !64BIT
> default 2048 if 64BIT
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-28 5:53 ` Nathan Chancellor
0 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:53 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, akpm, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, Nov 25, 2022 at 12:07:50PM +0000, Lee Jones wrote:
> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
> few over the current threshold. This can mainly be seen on 32-bit
> architectures where the present limit (when !GCC) is a lowly
> 1024-Bytes.
>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> lib/Kconfig.debug | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index c3c0b077ade33..82d475168db95 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -399,6 +399,7 @@ config FRAME_WARN
> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
> default 2048 if PARISC
> default 1536 if (!64BIT && XTENSA)
> + default 1280 if KASAN && !64BIT
> default 1024 if !64BIT
> default 2048 if 64BIT
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-28 5:53 ` Nathan Chancellor
0 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:53 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, daniel, arnd, sunpeng.li, ndesaulniers, dri-devel,
Xinhui.Pan, maarten.lankhorst, linux-kernel, mripard,
harry.wentland, amd-gfx, tzimmermann, trix, alexander.deucher,
akpm, Rodrigo.Siqueira, airlied, christian.koenig,
linux-arm-kernel
On Fri, Nov 25, 2022 at 12:07:50PM +0000, Lee Jones wrote:
> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
> few over the current threshold. This can mainly be seen on 32-bit
> architectures where the present limit (when !GCC) is a lowly
> 1024-Bytes.
>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> lib/Kconfig.debug | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index c3c0b077ade33..82d475168db95 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -399,6 +399,7 @@ config FRAME_WARN
> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
> default 2048 if PARISC
> default 1536 if (!64BIT && XTENSA)
> + default 1280 if KASAN && !64BIT
> default 1024 if !64BIT
> default 2048 if 64BIT
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 2/2] Kconfig.debug: Provide a little extra FRAME_WARN leeway when KASAN is enabled
@ 2022-11-28 5:53 ` Nathan Chancellor
0 siblings, 0 replies; 38+ messages in thread
From: Nathan Chancellor @ 2022-11-28 5:53 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, arnd, sunpeng.li, ndesaulniers, dri-devel, Xinhui.Pan,
linux-kernel, amd-gfx, tzimmermann, trix, alexander.deucher,
akpm, Rodrigo.Siqueira, christian.koenig, linux-arm-kernel
On Fri, Nov 25, 2022 at 12:07:50PM +0000, Lee Jones wrote:
> When enabled, KASAN enlarges function's stack-frames. Pushing quite a
> few over the current threshold. This can mainly be seen on 32-bit
> architectures where the present limit (when !GCC) is a lowly
> 1024-Bytes.
>
> Signed-off-by: Lee Jones <lee@kernel.org>
Reviewed-by: Nathan Chancellor <nathan@kernel.org>
> ---
> lib/Kconfig.debug | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index c3c0b077ade33..82d475168db95 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -399,6 +399,7 @@ config FRAME_WARN
> default 2048 if GCC_PLUGIN_LATENT_ENTROPY
> default 2048 if PARISC
> default 1536 if (!64BIT && XTENSA)
> + default 1280 if KASAN && !64BIT
> default 1024 if !64BIT
> default 2048 if 64BIT
> help
> --
> 2.38.1.584.g0f3c55d4c2-goog
>
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
2022-11-25 12:07 ` Lee Jones
(?)
(?)
@ 2022-11-26 0:13 ` Andrew Morton
-1 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:13 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, nathan, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
I'm not sure who this patchset is aimed at, so I'll take my usual
approach of grabbing it and seeing who complains.
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
b339ec9c229aa is a year old and I've been doing arm allmodconfig for
ever. What am I missing here?
A broken arm allmodconfig is pretty irritating - I'm thinking that a
fix should be backported into -stable kernels. But I'm clearly missing
something here.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-26 0:13 ` Andrew Morton
0 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:13 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, daniel, arnd, sunpeng.li, ndesaulniers, dri-devel,
Xinhui.Pan, maarten.lankhorst, linux-kernel, mripard, nathan,
harry.wentland, amd-gfx, tzimmermann, trix, alexander.deucher,
Rodrigo.Siqueira, airlied, christian.koenig, linux-arm-kernel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
I'm not sure who this patchset is aimed at, so I'll take my usual
approach of grabbing it and seeing who complains.
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
b339ec9c229aa is a year old and I've been doing arm allmodconfig for
ever. What am I missing here?
A broken arm allmodconfig is pretty irritating - I'm thinking that a
fix should be backported into -stable kernels. But I'm clearly missing
something here.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-26 0:13 ` Andrew Morton
0 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:13 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, nathan, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
I'm not sure who this patchset is aimed at, so I'll take my usual
approach of grabbing it and seeing who complains.
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
b339ec9c229aa is a year old and I've been doing arm allmodconfig for
ever. What am I missing here?
A broken arm allmodconfig is pretty irritating - I'm thinking that a
fix should be backported into -stable kernels. But I'm clearly missing
something here.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-26 0:13 ` Andrew Morton
0 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:13 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, arnd, sunpeng.li, ndesaulniers, dri-devel, Xinhui.Pan,
linux-kernel, nathan, amd-gfx, tzimmermann, trix,
alexander.deucher, Rodrigo.Siqueira, christian.koenig,
linux-arm-kernel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
I'm not sure who this patchset is aimed at, so I'll take my usual
approach of grabbing it and seeing who complains.
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
b339ec9c229aa is a year old and I've been doing arm allmodconfig for
ever. What am I missing here?
A broken arm allmodconfig is pretty irritating - I'm thinking that a
fix should be backported into -stable kernels. But I'm clearly missing
something here.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
2022-11-26 0:13 ` Andrew Morton
(?)
(?)
@ 2022-11-28 7:47 ` Lee Jones
-1 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-28 7:47 UTC (permalink / raw)
To: Andrew Morton
Cc: arnd, nathan, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, 25 Nov 2022, Andrew Morton wrote:
> On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
>
> > Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> > now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> > builds. This leads to some interesting failures, each resolved in this set.
>
> I'm not sure who this patchset is aimed at, so I'll take my usual
> approach of grabbing it and seeing who complains.
>
> > With this set applied, I am able to obtain a successful allmodconfig Arm build.
>
> b339ec9c229aa is a year old and I've been doing arm allmodconfig for
> ever. What am I missing here?
>
> A broken arm allmodconfig is pretty irritating - I'm thinking that a
> fix should be backported into -stable kernels. But I'm clearly missing
> something here.
I will be taking these through all applicable Stable kernels.
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-28 7:47 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-28 7:47 UTC (permalink / raw)
To: Andrew Morton
Cc: arnd, nathan, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, 25 Nov 2022, Andrew Morton wrote:
> On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
>
> > Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> > now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> > builds. This leads to some interesting failures, each resolved in this set.
>
> I'm not sure who this patchset is aimed at, so I'll take my usual
> approach of grabbing it and seeing who complains.
>
> > With this set applied, I am able to obtain a successful allmodconfig Arm build.
>
> b339ec9c229aa is a year old and I've been doing arm allmodconfig for
> ever. What am I missing here?
>
> A broken arm allmodconfig is pretty irritating - I'm thinking that a
> fix should be backported into -stable kernels. But I'm clearly missing
> something here.
I will be taking these through all applicable Stable kernels.
--
Lee Jones [李琼斯]
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-28 7:47 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-28 7:47 UTC (permalink / raw)
To: Andrew Morton
Cc: llvm, daniel, arnd, sunpeng.li, ndesaulniers, dri-devel,
Xinhui.Pan, maarten.lankhorst, linux-kernel, mripard, nathan,
harry.wentland, amd-gfx, tzimmermann, trix, alexander.deucher,
Rodrigo.Siqueira, airlied, christian.koenig, linux-arm-kernel
On Fri, 25 Nov 2022, Andrew Morton wrote:
> On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
>
> > Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> > now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> > builds. This leads to some interesting failures, each resolved in this set.
>
> I'm not sure who this patchset is aimed at, so I'll take my usual
> approach of grabbing it and seeing who complains.
>
> > With this set applied, I am able to obtain a successful allmodconfig Arm build.
>
> b339ec9c229aa is a year old and I've been doing arm allmodconfig for
> ever. What am I missing here?
>
> A broken arm allmodconfig is pretty irritating - I'm thinking that a
> fix should be backported into -stable kernels. But I'm clearly missing
> something here.
I will be taking these through all applicable Stable kernels.
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-28 7:47 ` Lee Jones
0 siblings, 0 replies; 38+ messages in thread
From: Lee Jones @ 2022-11-28 7:47 UTC (permalink / raw)
To: Andrew Morton
Cc: llvm, arnd, sunpeng.li, ndesaulniers, dri-devel, Xinhui.Pan,
linux-kernel, nathan, amd-gfx, tzimmermann, trix,
alexander.deucher, Rodrigo.Siqueira, christian.koenig,
linux-arm-kernel
On Fri, 25 Nov 2022, Andrew Morton wrote:
> On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
>
> > Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> > now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> > builds. This leads to some interesting failures, each resolved in this set.
>
> I'm not sure who this patchset is aimed at, so I'll take my usual
> approach of grabbing it and seeing who complains.
>
> > With this set applied, I am able to obtain a successful allmodconfig Arm build.
>
> b339ec9c229aa is a year old and I've been doing arm allmodconfig for
> ever. What am I missing here?
>
> A broken arm allmodconfig is pretty irritating - I'm thinking that a
> fix should be backported into -stable kernels. But I'm clearly missing
> something here.
I will be taking these through all applicable Stable kernels.
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
2022-11-25 12:07 ` Lee Jones
(?)
(?)
@ 2022-11-26 0:17 ` Andrew Morton
-1 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:17 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, nathan, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
Oh, I get it. Clang. I'll tweak the above para to make that clearer.
cc:stable question still applies? How much trouble will these build
errors be causing people for the next N years?
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-26 0:17 ` Andrew Morton
0 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:17 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, daniel, arnd, sunpeng.li, ndesaulniers, dri-devel,
Xinhui.Pan, maarten.lankhorst, linux-kernel, mripard, nathan,
harry.wentland, amd-gfx, tzimmermann, trix, alexander.deucher,
Rodrigo.Siqueira, airlied, christian.koenig, linux-arm-kernel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
Oh, I get it. Clang. I'll tweak the above para to make that clearer.
cc:stable question still applies? How much trouble will these build
errors be causing people for the next N years?
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-26 0:17 ` Andrew Morton
0 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:17 UTC (permalink / raw)
To: Lee Jones
Cc: arnd, nathan, maarten.lankhorst, mripard, tzimmermann, airlied,
daniel, ndesaulniers, trix, harry.wentland, sunpeng.li,
Rodrigo.Siqueira, alexander.deucher, christian.koenig,
Xinhui.Pan, linux-arm-kernel, linux-kernel, llvm, amd-gfx,
dri-devel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
Oh, I get it. Clang. I'll tweak the above para to make that clearer.
cc:stable question still applies? How much trouble will these build
errors be causing people for the next N years?
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 38+ messages in thread
* Re: [PATCH v2 0/2] Fix a bunch of allmodconfig errors
@ 2022-11-26 0:17 ` Andrew Morton
0 siblings, 0 replies; 38+ messages in thread
From: Andrew Morton @ 2022-11-26 0:17 UTC (permalink / raw)
To: Lee Jones
Cc: llvm, arnd, sunpeng.li, ndesaulniers, dri-devel, Xinhui.Pan,
linux-kernel, nathan, amd-gfx, tzimmermann, trix,
alexander.deucher, Rodrigo.Siqueira, christian.koenig,
linux-arm-kernel
On Fri, 25 Nov 2022 12:07:48 +0000 Lee Jones <lee@kernel.org> wrote:
> Since b339ec9c229aa ("kbuild: Only default to -Werror if COMPILE_TEST") WERROR
> now defaults to COMPILE_TEST meaning that it's enabled for allmodconfig
> builds. This leads to some interesting failures, each resolved in this set.
Oh, I get it. Clang. I'll tweak the above para to make that clearer.
cc:stable question still applies? How much trouble will these build
errors be causing people for the next N years?
> With this set applied, I am able to obtain a successful allmodconfig Arm build.
^ permalink raw reply [flat|nested] 38+ messages in thread