Remove unused exports and save 98Kb of kernel size

Remove unused exports and save 98Kb of kernel size

[Arjan van de Ven]

Hi,

I've made a patch to remove all EXPORT_SYMBOL's that aren't used in the
kernel; it's too big for the list so it can be found at

http://www.kernelmorons.org/unexport.patch

-rwxr-xr-x 1 root root 34476416 Apr  1 21:59 vmlinux.before
-rwxr-xr-x 1 root root 34378112 Apr  1 22:48 vmlinux.after

As you can see this saves 98Kb kernel size... that's not peanuts.

Signed-off-by: Arjan van de Ven <arjan@kernelmorons.org>

Re: Remove unused exports and save 98Kb of kernel size

[Marcel Holtmann]

Hi Arjan,

> I've made a patch to remove all EXPORT_SYMBOL's that aren't used in the
> kernel; it's too big for the list so it can be found at
> 
> http://www.kernelmorons.org/unexport.patch

no ack for net/bluetooth/ from me.

Regards

Marcel

Re: Remove unused exports and save 98Kb of kernel size

[Jörn Engel]

On Sat, 1 April 2006 23:18:58 +0200, Marcel Holtmann wrote:
> 
> > I've made a patch to remove all EXPORT_SYMBOL's that aren't used in the
> > kernel; it's too big for the list so it can be found at
> > 
> > http://www.kernelmorons.org/unexport.patch
> 
> no ack for net/bluetooth/ from me.

Why not?  Do you have patches pending for submission that will use
those exported symbols?

Jörn

-- 
The strong give up and move away, while the weak give up and stay.
-- unknown

Re: Remove unused exports and save 98Kb of kernel size

[Jörn Engel]

On Sat, 1 April 2006 23:05:45 +0200, Arjan van de Ven wrote:
> 
> I've made a patch to remove all EXPORT_SYMBOL's that aren't used in the
> kernel; it's too big for the list so it can be found at
> 
> http://www.kernelmorons.org/unexport.patch
> 
> -rwxr-xr-x 1 root root 34476416 Apr  1 21:59 vmlinux.before
> -rwxr-xr-x 1 root root 34378112 Apr  1 22:48 vmlinux.after
> 
> As you can see this saves 98Kb kernel size... that's not peanuts.
> 
> Signed-off-by: Arjan van de Ven <arjan@kernelmorons.org>

Is there a reason that you always leave the newline instead of
removing it as well?  Looks script-generated, so it should be a simple
change for the script to remove the newline as well.

Jörn

-- 
If System.PrivateProfileString("",
"HKEY_CURRENT_USER\Software\Microsoft\Office\9.0\Word\Security", "Level") <>
"" Then  CommandBars("Macro").Controls("Security...").Enabled = False
-- from the Melissa-source

Re: Remove unused exports and save 98Kb of kernel size

[Takashi Iwai]

At Sat, 01 Apr 2006 23:05:45 +0200,
Arjan van de Ven wrote:
> 
> Hi,
> 
> I've made a patch to remove all EXPORT_SYMBOL's that aren't used in the
> kernel; it's too big for the list so it can be found at
> 
> http://www.kernelmorons.org/unexport.patch
> 
> -rwxr-xr-x 1 root root 34476416 Apr  1 21:59 vmlinux.before
> -rwxr-xr-x 1 root root 34378112 Apr  1 22:48 vmlinux.after
> 
> As you can see this saves 98Kb kernel size... that's not peanuts.
> 
> Signed-off-by: Arjan van de Ven <arjan@kernelmorons.org>

Please be careful, many removals in sound will break under certain
kernel config (e.g. CONFIG_ISA).


Takashi

Re: Remove unused exports and save 98Kb of kernel size

[Marcel Holtmann]

Hi Joern,

> > > I've made a patch to remove all EXPORT_SYMBOL's that aren't used in the
> > > kernel; it's too big for the list so it can be found at
> > > 
> > > http://www.kernelmorons.org/unexport.patch
> > 
> > no ack for net/bluetooth/ from me.
> 
> Why not?  Do you have patches pending for submission that will use
> those exported symbols?

yes, I do.

Regards

Marcel

Re: Remove unused exports and save 98Kb of kernel size

[Valerie Henson]

On Mon, Apr 03, 2006 at 04:10:27PM +0200, J?rn Engel wrote:
> On Sat, 1 April 2006 23:05:45 +0200, Arjan van de Ven wrote:
> > 
> > I've made a patch to remove all EXPORT_SYMBOL's that aren't used in the
> > kernel; it's too big for the list so it can be found at
> > 
> > http://www.kernelmorons.org/unexport.patch
> > 
> > -rwxr-xr-x 1 root root 34476416 Apr  1 21:59 vmlinux.before
> > -rwxr-xr-x 1 root root 34378112 Apr  1 22:48 vmlinux.after
> > 
> > As you can see this saves 98Kb kernel size... that's not peanuts.
> > 
> > Signed-off-by: Arjan van de Ven <arjan@kernelmorons.org>
> 
> Is there a reason that you always leave the newline instead of
> removing it as well?  Looks script-generated, so it should be a simple
> change for the script to remove the newline as well.

Because it wasn't worth fixing for a silly April Fool's joke? :)

> On Sat, 1 April 2006 23:05:45 +0200, Arjan van de Ven wrote:
          ^^^^^^^
> > http://www.kernelmorons.org/unexport.patch
               ^^^^^^^^^^^^^^^^

Of course, there's a serious side to this patch.  Who knows how many
of these exported symbols are actually used?  I can imagine a config
option to turn off these symbols which is shipped as on by default for
a few weeks, in order to flush out people who are actually using these
symbols.

-VAL