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 a9df942..f5c989a 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