Messages in this thread | | | Date | Mon, 2 Aug 2010 21:49:16 +0400 | From | Cyrill Gorcunov <> | Subject | Re: [PATCH -v3 2/2] x86: more early console output from compressed/misc.c |
| |
On Mon, Aug 02, 2010 at 02:18:39AM -0700, Yinghai Lu wrote: > > will get > > |Decompressing Linux... Parsing ELF... done. > |Booting the kernel. > > in serial console. > > reuse code from arch/x86/boot/ > and we can use printf if needed > > -v2: define BOOT_BOOT_H to avoid include boot.h > -v3: early_serial_base need to be static in misc.c ? > > Signed-off-by: Yinghai Lu <yinghai@kernel.org> > > --- > arch/x86/boot/compressed/misc.c | 65 +++++++++++++++++++++++++++++++++++++++- > arch/x86/boot/main.c | 6 +-- > 2 files changed, 66 insertions(+), 5 deletions(-) >
It seems eventually we just add ~80 lines of code in compare with what we had before, just to print "Decompressing..." by serial line. Not sure Yinghai if it worth it. Though having printf that early might be quite useful for those who debugging or developing new compressors/decompressors. If it would be small non-intrusive patch I would both hands for it but now I simply don't know (to be fair mine proposal based on your initial patch not better either ;)
-- Cyrill
| |