drm/panic: depends on !VT_CONSOLE
authorJocelyn Falempe <jfalempe@redhat.com>
Thu, 13 Jun 2024 15:40:16 +0000 (17:40 +0200)
committerJocelyn Falempe <jfalempe@redhat.com>
Tue, 18 Jun 2024 15:18:56 +0000 (17:18 +0200)
The race condition between fbcon and drm_panic can only occurs if
VT_CONSOLE is set. So update drm_panic dependency accordingly.
This will make it easier for Linux distributions to enable drm_panic
by disabling VT_CONSOLE, and keeping fbcon terminal.
The only drawback is that fbcon won't display the boot kmsg, so it
should rely on userspace to do that.
At least plymouth already handle this case with
https://gitlab.freedesktop.org/plymouth/plymouth/-/merge_requests/224

Suggested-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Jocelyn Falempe <jfalempe@redhat.com>
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240613154041.325964-1-jfalempe@redhat.com
drivers/gpu/drm/Kconfig

index a9df94291622ae24f54eb1719e90b755a3cca0bd..f5c989aed7e9013a1bddb2a3a41e943d1cd0bf8b 100644 (file)
@@ -107,7 +107,7 @@ config DRM_KMS_HELPER
 
 config DRM_PANIC
        bool "Display a user-friendly message when a kernel panic occurs"
-       depends on DRM && !FRAMEBUFFER_CONSOLE
+       depends on DRM && !(FRAMEBUFFER_CONSOLE && VT_CONSOLE)
        select DRM_KMS_HELPER
        select FONT_SUPPORT
        help