On Mon, Aug 11, 2003 at 03:45:16PM -0700, Richard Harke wrote: > On Monday 11 August 2003 11:51 am, Matthew Wilcox wrote: > > On Mon, Aug 11, 2003 at 10:10:31AM -0600, Bjorn Helgaas wrote: > > > Yup, I think you're on the right track. acpi_hp_csr_space is > > > not currently exported (see arch/ia64/kernel/ia64_ksyms.c). > > > Can you add it there and see whether it fixes your problem? > > > > > > If so, please send me a patch. And a pointer to the Nvidia > > > package would be nice, too. I see some Nvidia code in > > > drivers/char/agp/agpgart_be.c, but it doesn't reference > > > acpi_hp_csr_space(). > > > > Note that if a binary module (or the wrapper for one) is the only thing > > that needs a symbol exported, the normal policy is to not export it ... > > I find your comment a bit terse. If I understand you, if only 1 module > needs a kernel symbol, it wouldn't be exported and the module > affected would be incorporated into the kernel. But in that case > the config should be modified so that the module version cannot > be selected.
Yes, I'm a terse kind of person ;-) Modules that do not come with source cannot request that extra symbols be exported from the kernel. Now if it's agpgart that needs acpi_hp_csr_space, that's a different matter. -- "It's not Hollywood. War is real, war is primarily not about defeat or victory, it is about death. I've seen thousands and thousands of dead bodies. Do you think I want to have an academic debate on this subject?" -- Robert Fisk
