Re: PATCH 2.4.17.2: make ext2 smaller

Jeff Garzik (jgarzik@mandrakesoft.com)
Sun, 02 Dec 2001 08:47:58 -0500


Ingo Oeser wrote:
>
> On Sun, Dec 02, 2001 at 08:03:27AM -0500, Jeff Garzik wrote:
> > Ingo Oeser wrote:
> > > Even this doesn't have to be done manually. Everything that is
> > > not covered by EXPORT_SYMBOL() in this case can be static, since
> >
> > And if !MODULE, then even EXPORT_SYMBOL symbols can become static, if
> > they are not used outside the compilation unit.
>
> If your compilation units are greater than the current
> granularity of modules: Yes.
>
> EXPORT_SYMBOL() symbols are Kernel-API, which is also exported to
> 3rd-party vendors with binary modules. So it makes little sense
> to me to make them static.

If !MODULES, modules are not supported by that kernel. It is completely
safe to make such functions static, if not used outside the compilation
unit. For all other cases, it is indeed wrong to make such them static.

Jeff

-- 
Jeff Garzik      | Only so many songs can be sung
Building 1024    | with two lips, two lungs, and one tongue.
MandrakeSoft     |         - nomeansno

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