Messages in this thread | | | Date | Thu, 6 Jul 2006 11:39:22 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH 2/6] FRV: Fix FRV arch compile errors [try #3] |
| |
On Thu, 06 Jul 2006 19:25:03 +0100 David Howells <dhowells@redhat.com> wrote:
> Andrew Morton <akpm@osdl.org> wrote: > > > - The __init-style tags on declarations don't actually do anything and > > the compiler doesn't check for consistency with the definition - it's > > best to just omit it from the declaration. > > Well, you're wrong. They *do* do something. They stop the compiler using the > register-relative addressing reserved for small data. If this isn't in there, > then the linker will spit out a relocation error.
Sigh. So if we get this wrong (and we have, and we shall) then the error gets silently accepted by the toolchain until someone comes along and tries to link it on FVR.
Is there anything we can do about that? Mangling the names with some macro isn't attractive. Teach sparse about it? Dunno.
Right now we're showing a grand total of two identifiers tagged with __initdata in all of include/. Why isn't FRV blowing up all over the place? Is there something about nr_kernel_pages which made us get unlucky? - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |