Messages in this thread | | | From | Keith Owens <> | Subject | Re: [PATCH] Re: crc32 and lib.a (was Re: [PATCH] nbd in 2.5.3 does | Date | Sat, 02 Feb 2002 19:59:28 +1100 |
| |
On Sat, 02 Feb 2002 08:40:11 +0000, David Woodhouse <dwmw2@infradead.org> wrote: >kaos@ocs.com.au said: >> There is also a problem with exported symbols. To ld, EXPORT_SYMBOL >> looks like a reference to the symbol, > >Er, surely that's not a problem at all? This is desired behaviour? > >> but the export entry is irrelevant, what really matters is if any module >> refers to those symbols. > >Absolutely not. If we mark a symbol EXPORT_SYMBOL, we want it exported. No >questions asked.
Does anything still use this symbol or is it dead? Quite valid question.
>The export entry _is_ relevant; furthermore it's the _only_ thing that's >relevant, and there is no way of knowing if there's a module that isn't in >the tree, or maybe even a module that _is_ in the tree but not compiled >today, that needs the symbol in question.
I said that you get false positives because the code that uses the symbol might not be compiled in your kernel. The script flags symbols that are extern or exported and are not used in the current kernel. Somebody then has to go through the report and decide if those symbols are required under other config settings or by third party modules, assuming that they care about third party modules.
The script is NOT an automatic list of what can be cleaned up. I doubt that such a list can be generated in the face of the kernel config system and third party modules. It is only a starting point for symbols that need reviewing.
- 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/
| |