Re: [PATCH] v850 additions to include/linux/elf.h

[Miles Bader <miles@lsi.nec.co.jp>]

Alan Cox <alan@lxorguk.ukuu.org.uk> writes:
> > This patch adds more stuff to include/linux/elf.h for the v850 (used by
> > the new module loader).
> 
> To save cluttering a linux/*.h file couldnt the module loader for v850 
> include an asm/*.h file holding the extra info ?

I suppose so, but the v850 is not exceptional in this regard.
The various relocation type constants (R_...) in elf.h are completely
unused in the kernel on _every_ architecture, except by the new module
loader.

If tidying is needed, then it seems like the best thing would be to move
all the arch-specific stuff into the corresponding <asm/elf.h> file for
each architecture.  I presume the reason this hasn't been done is simply
convention -- userland elf.h files are also giant conglomerations of
defines for every supported architecture ...

-Miles
-- 
Next to fried food, the South has suffered most from oratory.
  			-- Walter Hines Page