Hide Forgot
Description of Problem: ----------------------- Red Hat Linux kernel source includes a customized logo that shows up during boot up on framebuffer-enabled kernels (i.e. VESA fb). The logo is defined in the include file "include/linux/linux_logo.h". In this header file, a comment instructs the user that, in order to be able to use this customized Red Hat logo instead of the default one (the Linux Penguin), the user has to edit the file "drivers/video/fbcon.c" file and change the definitions of LOGO_H and LOGO_W. The problem is that the comment reads: #define LOGO_H 800 #define LOGO_W 60 This is wrong. The logo height is 60 pixels, and the logo width is 800 pixels, so the comment should be fixed and swapped as: #define LOGO_H 60 #define LOGO_W 800 Please, see the diff unified patch at the end of this report. Apply it to the current kernel-source package (I generated the diff from 2.4.8-10) so it stays consistent. Version-Release number of selected component (if applicable): ------------------------------------------------------------- kernel-source-2.4.8-10 How Reproducible: ----------------- Always, since kernel-source-2.4.8-5 (AFAIK) Steps to Reproduce: ------------------- 1. Install kernel-source-2.4.18-10 (for example) 2. Look at /usr/src/linux-2.4.18-10/include/linux/linux_logo.h 3. Read the comments at the beginning of the file Actual Results: --------------- LOGO_H is 800, as read in the comments section of linux_logo.h LOGO_W is 60, as read in the comments section of linux_logo.h Expected Results: ----------------- LOGO_H should be 60, as read in the comments section of linux_logo.h LOGO_W should be 800, as read in the comments section of linux_logo.h Additional Information: ----------------------- A small unified diff patch to fix this: --- linux_logo.h 2002-09-16 15:34:27.000000000 +0200 +++ linux_logo.h.orig 2002-09-16 15:34:18.000000000 +0200 @@ -11,8 +11,8 @@ * * * Remember to modify drivers/video/fbcon.c: - * Change "#define LOGO_H 80" to "#define LOGO_H 60" - * Change "#define LOGO_W 80" to "#define LOGO_W 800" + * Change "#define LOGO_H 80" to "#define LOGO_H 800" + * Change "#define LOGO_W 80" to "#define LOGO_W 60" */ #ifndef __HAVE_ARCH_LINUX_LOGO
*blink* That header is autogenerated, but yes the comment is wrong. However I seem to not be able to find the cause of it being wrong on first sight; investigating further
ok found it; fixed for next build; thanks for the report
Hmmm... It seems the beautiful Red Hat boot logo has been removed from the kernel as of kernel-source-2.4.20-2.30. Instead, I see the standard penguin boot logo... What a pity! I liked it so much. I still have at hand the "linux_logo.h" file containing the original Red Hat boot logo so I can manually patch the kernel and preserve it in new kernels. Will the original 800x60 Red Hat boot logo (or an updated version) be included in new releases of the kernel-source RPM, or will the penguin boot logo stay forever?
This bug is not applicable anymore to current versions of the RedHat kernel.