Messages in this thread | | | Date | Wed, 12 Oct 2011 11:25:30 +0200 (CEST) | From | Thomas Gleixner <> | Subject | Re: [PATCH] irq: Add EXPORT_SYMBOL to function of irq generic-chip |
| |
On Wed, 12 Oct 2011, Nobuhiro Iwamatsu wrote:
> Although some functions of of irq generic-chip can be used, > since EXPORT_SYMBOL is not specified, it cannot use.
This does not make sense. In which context they can't be used?
> This is revised that EXPORT_SYMBOL can be added and referred > to in functions.
EXPORT_SYMBOL_GPL if at all please.
And I'd rather avoid the exports, except those
> +EXPORT_SYMBOL(irq_alloc_generic_chip); > +EXPORT_SYMBOL(irq_setup_generic_chip); > +EXPORT_SYMBOL(irq_setup_alt_chip); > +EXPORT_SYMBOL(irq_remove_generic_chip);
which are really necessary. To gain access to the various helper functions we're better off to create a data structure with function pointers.
struct irq_gc_functions { void (* irq_gc_noop)(...); ... };
Instantiate this structure in kernel/irq/generic-chip.c, statically initialize it with pointers to those helper functions and export that data structure. That's way more efficient than having the overhead of the symbol export for each of the functions.
Thanks,
tglx
| |