aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/fs/iomap.c
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@linux-foundation.org>2016-10-25 11:27:31 -0700
committerLinus Torvalds <torvalds@linux-foundation.org>2016-12-14 21:08:18 -0800
commit5c2992ee7fd8a29d04125dc0aa3522784c5fa5eb (patch)
treecec604a649a4a8b46d0d1759587a9f0fa37a28a1 /fs/iomap.c
parentprintk: remove games with previous record flags (diff)
downloadwireguard-linux-5c2992ee7fd8a29d04125dc0aa3522784c5fa5eb.tar.xz
wireguard-linux-5c2992ee7fd8a29d04125dc0aa3522784c5fa5eb.zip
printk: remove console flushing special cases for partial buffered lines
It actively hurts proper merging, and makes for a lot of special cases. There was a good(ish) reason for doing it originally, but it's getting too painful to maintain. And most of the original reasons for it are long gone. So instead of having special code to flush partial lines to the console (as opposed to the record buffers), do _all_ the console writing from the record buffer, and be done with it. If an oops happens (or some other synchronous event), we will flush the partial lines due to the oops printing activity, so this does not affect that. It does mean that if you have a completely hung machine, a partial preceding line may not have been printed out. That was some of the original reason for this complexity, in fact, back when we used to test for the historical i386 "halt" instruction problem by doing pr_info("Checking 'hlt' instruction... "); if (!boot_cpu_data.hlt_works_ok) { pr_cont("disabled\n"); return; } halt(); halt(); halt(); halt(); pr_cont("OK\n"); and that model no longer works (it the 'hlt' instruction kills the machine, the partial line won't have been flushed, so you won't even see it). Of course, that was also back in the days when people actually had textual console output rather than a graphical splash-screen at bootup. How times change.. Cc: Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> Cc: Joe Perches <joe@perches.com> Cc: Steven Rostedt <rostedt@goodmis.org> Tested-by: Petr Mladek <pmladek@suse.com> Tested-by: Geert Uytterhoeven <geert@linux-m68k.org> Tested-by: Mark Rutland <mark.rutland@arm.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/iomap.c')
0 files changed, 0 insertions, 0 deletions