lkml.org 
[lkml]   [2000]   [Nov]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] CONFIG_EISA note in Documentation/Configure.help
Jeff Garzik wrote:
>
>Agreed, for the most part. If you know for sure you don't have an EISA
>machine, you can now disable CONFIG_EISA. IMHO ideally one should be
>able to eliminate code that is useless on all but a small subset of
>working machines.

Well, the CONFIG_EISA option is there. My little patch was just intended to
slightly enlighten those prone to "lets see what this option does". I
compiled test11-pre4 both with and without CONFIG_EISA and the difference is
very slight. Of course, if you had more items with EISA code, this difference
would be bigger.

848 -rw-r--r-- 1 root root 868179 Nov 14 13:32 bzImage
848 -rw-r--r-- 1 root root 867973 Nov 14 13:28 bzImage.no_eisa

The difference probably comes from my 3c59x driver.

I also uglied up the 3c59x.c code with #ifdef CONFIG_EISA around the
six sections relavant to EISA to see if that would save anything, and the
object file was only 318 bytes smaller, probably not worth the uglyness of
the six ifdefs. That modified code was not used in the above comparison.

I am running that modified code right now, BTW.

Jeff, I can send you the patch for the hacked up 3c59x.c if you're at all
interested.

Steven
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 12:45    [W:0.032 / U:1.076 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site