On text-based systems the 'quiet' boot option will show printk levels
higher than CONSOLE_LOGLEVEL_QUIET.  The displaying of the Tux logo
during boot can cause some consoles to lose display data and as a result
confuse the end user.

Do not display the Tux logo on systems that are in 'quiet' boot.

v2: It helps to commit all my changes before sending them.  Remove extra
bracket.
v3: buildbot error fix: fbcon can be built as part of a module so export 
console_printk
v4: move console_printk change to separate patch, and drop logo cleanup

Signed-off-by: Prarit Bhargava <[email protected]>
Cc: Hans de Goede <[email protected]>
Cc: Marko Myllynen <[email protected]>
Cc: Steven Rostedt (VMware) <[email protected]>
Cc: Bartlomiej Zolnierkiewicz <[email protected]>
Cc: Kees Cook <[email protected]>
Cc: Daniel Vetter <[email protected]>
Cc: Thierry Reding <[email protected]>
Cc: Yisheng Xie <[email protected]>
Cc: Petr Mladek <[email protected]>
Cc: Sergey Senozhatsky <[email protected]>
Cc: [email protected]
---
 drivers/video/fbdev/core/fbcon.c | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/drivers/video/fbdev/core/fbcon.c b/drivers/video/fbdev/core/fbcon.c
index 8976190b6c1f..66e37bdff1d9 100644
--- a/drivers/video/fbdev/core/fbcon.c
+++ b/drivers/video/fbdev/core/fbcon.c
@@ -649,11 +649,14 @@ static void fbcon_prepare_logo(struct vc_data *vc, struct 
fb_info *info,
                kfree(save);
        }
 
+       if (logo_shown == FBCON_LOGO_DONTSHOW)
+               return;
+
        if (logo_lines > vc->vc_bottom) {
                logo_shown = FBCON_LOGO_CANSHOW;
                printk(KERN_INFO
                       "fbcon_init: disable boot-logo (boot-logo bigger than 
screen).\n");
-       } else if (logo_shown != FBCON_LOGO_DONTSHOW) {
+       } else {
                logo_shown = FBCON_LOGO_DRAW;
                vc->vc_top = logo_lines;
        }
@@ -1059,9 +1062,11 @@ static void fbcon_init(struct vc_data *vc, int init)
 
        cap = info->flags;
 
-       if (vc != svc || logo_shown == FBCON_LOGO_DONTSHOW ||
-           (info->fix.type == FB_TYPE_TEXT))
+       if (vc != svc || console_loglevel <= CONSOLE_LOGLEVEL_QUIET ||
+           (info->fix.type == FB_TYPE_TEXT)) {
                logo = 0;
+               logo_shown = FBCON_LOGO_DONTSHOW;
+       }
 
        if (var_to_display(p, &info->var, info))
                return;
-- 
2.17.2

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to