On 20 Jul 2006, at 07:07, Adrian Bunk wrote: >> +struct start_info *xen_start_info; >> +EXPORT_SYMBOL(xen_start_info); > > EXPORT_SYMBOL_GPL? Possibly. >> +/* >> + * Point at the empty zero page to start with. We map the real >> shared_info >> + * page as soon as fixmap is up and running. >> + */ >> +struct shared_info *HYPERVISOR_shared_info = (struct shared_info >> *)empty_zero_page; >> +EXPORT_SYMBOL(HYPERVISOR_shared_info); >> ... > > EXPORT_SYMBOL_GPL? Interrupt-control macros (local_irq_enable/disable and friends) used this symbol, so GPLing it made non-GPL modules fail. We made a bunch of the macros proper functions so this may no longer be the case. -- Keir