Keith Owens <kaos@ocs.com.au> wrote:
>"Adam J. Richter" <adam@yggdrasil.com> wrote:
>> Note that this is not a "final" version. I plan to go
>>through all of the changes and bracket all of these new tables
>>with #ifdef MODULE...#endif so they do not result in complaints
>>about the table being defined static and never used in cases where
>>the driver is compiled directly into the kernel.
>This is cleaner. Append MODULE_ONLY after __initdata and remove the
>ifdef. It increases the size of initdata in the kernel, compared to
>ifdef, but since initdata is promptly reused as scratch space it should
>not be a problem.
[patch elided]
Thanks for the patch, but I think I'll stick with the ifdefs
for now, for the following reasons.
1. I think ifdef MODULE is more understandable to the casual observer.
2. There is often some other condition that I need to combine
with (CONFIG_PCI, CONFIG_ISAPNP, CONFIG_ISAPNP_MODULE).
3. There is often an existing ifdef in the right place that I
can just tuck the code into.
4. I would prefer that this change not have even a file size cost
to people who want to build minimal monolithic kernels
for applicance applications.
5. My feeling is that just the few kilobytes of file size cost
associated with #4 and knowing that absolutely nothing is
added for non-module usage will psychologically make
maintainers feel better about it and have even fewer misgivings
about integrating it.
6. We can expect the lines bracketing these table declarations
to be changed in the near future as the drivers are changed
to use the new PCI and isapnp interfaces or to use the ID
tables just to eliminate the old custom data structures that
hold the same information.
Thanks for the patch anyhow, though. It's a clever idea that
may be useful in other situations.
Adam J. Richter __ ______________ 4880 Stevens Creek Blvd, Suite 104
adam@yggdrasil.com \ / San Jose, California 95129-1034
+1 408 261-6630 | g g d r a s i l United States of America
fax +1 408 261-6631 "Free Software For The Rest Of Us."
-
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/
This archive was generated by hypermail 2b29 : Thu Nov 30 2000 - 21:00:13 EST