Re: [RFC] [PATCH] subarch cleanup

From: Russell King (rmk@arm.linux.org.uk)
Date: Thu Nov 21 2002 - 14:34:07 EST


On Thu, Nov 21, 2002 at 10:35:43AM -0800, Martin J. Bligh wrote:
> > Why do you need to move the .h files?
>
> Because they're in a silly place now. They should be whereever all
> the other include files are.
>
> > CFLAGS += -Iarch/i386/$(MACHINE_H) -Iarch/i386/mach-generic
> > That should achieve the same effect?
>
> Header files go under include ....

In this instance I'd disagree. Think about UML. UML has:

        include/asm-um/*.h
        include/asm-um/arch -> include/asm-i386

When building for UML, what happens if you need to get to a machine
specific file for something, and the i386 include files do:

        #include <asm/mach-generic/foo.h>

Yep, it fails.

Now guess why we in the ARM community haven't even bothered to look at
UML yet? There's over 1MB of include files that would need to be moved,
along with countless #include statements needing to be fixed up.

For something that would be nice to have, and probably run quite well on
the ARM architecture (due to some nice features ARM has, especially for
UML's jail mode) there isn't enough interest in it to warrant such a
painful reorganisation.

I'd therefore strongly recommend NOT going down the path of adding
subdirectories to include/asm-*.

-- 
Russell King (rmk@arm.linux.org.uk)                The developer of ARM Linux
             http://www.arm.linux.org.uk/personal/aboutme.html

- 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/



This archive was generated by hypermail 2b29 : Sat Nov 23 2002 - 22:00:37 EST