linux-kernel - Re: [PATCH v4.2 0/2] console: Add persistent scrollback buffers for all VGA console

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Date:   Sat, 19 Nov 2016 18:26:01 +0000
From:   Andrey Utkin <andrey_utkin@...tmail.com>
To:     Manuel Schölling <manuel.schoelling@....de>
Cc:     plagnioj@...osoft.com, tomi.valkeinen@...com, jslaby@...e.cz,
        gregkh@...uxfoundation.org, linux-fbdev@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4.2 0/2] console: Add persistent scrollback buffers for all
 VGA console

Got into trouble sending my replies to LKML from Mutt, getting delivery
rejections related to encoding:

https://gist.github.com/andrey-utkin/b204666c34613858a34844283571ce17

Sorry for people who got excessive resends from me. For now I'm sending
this from web interface, hopefully it gets things right.

Hi Manuel,

I have tested your patches with v4.8.8 kernel on Gentoo system. Just
took me a little time to figure out that what my system uses as "raw
console" is not vgacon driver, but fbcon. In order to override defaults
and get VGA console, I had to add kernel bootargs "nofb nomodeset". A
pleasant byproduct of switching to VGA console was the fact that
the earliest printouts of kernel loading stage are preserved in
scrollback so I can look at them. However, vgacon is not compatible with
usage of modern graphical modes.

So your patch really works with native VGA console. This is very nice.
Thank you for filling this long-living feature gap.

Tested-by: Andrey Utkin <andrey_utkin@...tmail.com>

Now would you consider implementing same feature also for fbcon? As I
see it lacks same feature. I guess it may be harder because of support
of graphics (penguins on the top of screen, direct graphical output into
framebuffer-driven console - see "links -g", "fbgs", "mplayer -vo fbdev"
and so on).

P. S. I forgot to mention in my previous code review comments that both
patches have some checkpatch issues, and it should be quite easy to
perfect the patches to make checkpatch happy even in "--strict" mode.
You can use "--fix-inplace" by the way, it may solve most of that, just
review what exactly it changes so that it doesn't garble what is right.

Cheers!