Bug 74117 - [PATCH] for linux_logo.h from kernel-source
Summary: [PATCH] for linux_logo.h from kernel-source
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: kernel
Version: 7.3
Hardware: All
OS: Linux
medium
low
Target Milestone: ---
Assignee: Arjan van de Ven
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2002-09-16 13:44 UTC by Felipe Alfaro Solana
Modified: 2007-04-18 16:46 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2004-09-29 00:05:34 UTC
Embargoed:


Attachments (Terms of Use)

Description Felipe Alfaro Solana 2002-09-16 13:44:38 UTC
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

Comment 1 Arjan van de Ven 2002-09-16 13:52:51 UTC
*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

Comment 2 Arjan van de Ven 2002-09-16 13:58:48 UTC
ok found it; fixed for next build; thanks for the report

Comment 3 Felipe Alfaro Solana 2003-02-02 19:35:27 UTC
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?

Comment 4 Felipe Alfaro Solana 2004-09-29 00:05:34 UTC
This bug is not applicable anymore to current versions of the RedHat
kernel.


Note You need to log in before you can comment on or make changes to this bug.