diff options
| author |   Arnd Bergmann <arnd@arndb.de> | 2025-06-10 11:27:33 +0200 |
|---|---|---|
| committer |   Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> | 2025-06-12 21:29:52 +0300 |
| commit | 660942f2441df622d527f216009f332151843ce8 (patch) | |
| tree | 6d1040d8395d8322e6168c30d548eb16330f9520 | |
| parent | drm/prime: remove drm_prime_lookup_buf_by_handle (diff) | |
| download | wireguard-linux-660942f2441df622d527f216009f332151843ce8.tar.xz wireguard-linux-660942f2441df622d527f216009f332151843ce8.zip | |
drm: omapdrm: reduce clang stack usage
The thread sanitizer makes the stack usage explode from extra variable
spills in dispc_runtime_resume:
drivers/gpu/drm/omapdrm/dss/dispc.c:4735:27: error: stack frame size (1824) exceeds limit (1280) in 'dispc_runtime_resume' [-Werror,-Wframe-larger-than]
I could not figure out what exactly is going on here, but I see that
whenever dispc_restore_context() is not inlined, that function
and its caller shrink below 900 bytes combined of stack usage.
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20250610092737.2641862-1-arnd@kernel.org
Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
| -rw-r--r-- | drivers/gpu/drm/omapdrm/dss/dispc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/omapdrm/dss/dispc.c b/drivers/gpu/drm/omapdrm/dss/dispc.c index 533f70e8a4a6..cf055815077c 100644 --- a/drivers/gpu/drm/omapdrm/dss/dispc.c +++ b/drivers/gpu/drm/omapdrm/dss/dispc.c @@ -524,7 +524,7 @@ static void dispc_save_context(struct dispc_device *dispc) DSSDBG("context saved\n"); } -static void dispc_restore_context(struct dispc_device *dispc) +static noinline_for_stack void dispc_restore_context(struct dispc_device *dispc) { int i, j; |