On Sat, Jul 18, 2020 at 7:43 AM John Ogness <john.ogness@xxxxxxxxxxxxx> wrote: > > I expect this is handled correctly since the reader is not given any > parts until a full line is ready, but I will put more focus on testing > this to make sure. Yeah, the patches looked fine, but I only scanned them, and just wanted to make sure. Over the years, we've gotten printk wrong so many times that I get a bit paranoid. Things can look fine on the screen, but then have odd line breaks in the logs. Or vice versa. Or work fine on some machine, but consistently show some race on another. And some of the more complex features are hardly ever actually used - I'm not sure the optional message context (aka dictionary) is ever actually used. Yes, all the "dev_printk()" helpers fill it in with the device information (create_syslog_header()), and you _can_ use them if you know about them (ie journalctl -b _KERNEL_SUBSYSTEM=pci_bus but I sometimes wonder how many people use all this complexity. And how many people even know about it..) So there are hidden things in there that can easily break *subtly* and then take ages for people to notice, because while some are very obvious indeed ("why is my module list message broken up into a hundred lines?") others might be things people aren't even aware of. Maybe a lot of system tools use those kernel dictionaries. Maybe it would break immediately. I just sometimes wonder... Linus _______________________________________________ kexec mailing list kexec@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/kexec