Hi,
(Sorry for the delay.)
On Thu, 22 Sep 2005, Herbert Poetzl wrote:
> _what_ do you consider 'logically organized' because
> putting all the CLONE_* stuff into a separate file is
> pretty logical for me ... but obviously not for you.
"logically organized" mainly means reducing dependencies by organizing
them by their logical dependencies. If a large header file is included by
a lot of other files, some parts maybe separated to reduce header
dependencies. The same can be done for config dependencies, so that a
config change doesn't necessarily recompiles the whole kernel.
Your change doesn't reduce any dependecies and it's not such a big
cleanup:
arch/alpha/kernel/asm-offsets.c | 2 --
arch/alpha/kernel/entry.S | 1 +
arch/cris/arch-v10/kernel/asm-offsets.c | 3 ---
arch/cris/arch-v10/kernel/entry.S | 1 +
arch/cris/arch-v32/kernel/asm-offsets.c | 3 ---
arch/frv/kernel/kernel_thread.S | 2 +-
arch/ia64/ia32/ia32_entry.S | 3 ++-
arch/ia64/kernel/asm-offsets.c | 4 ----
arch/parisc/kernel/entry.S | 4 +---
arch/ppc/kernel/asm-offsets.c | 2 --
arch/ppc/kernel/misc.S | 1 +
arch/ppc64/kernel/asm-offsets.c | 3 ---
arch/ppc64/kernel/misc.S | 1 +
arch/v850/kernel/asm-offsets.c | 4 ----
arch/v850/kernel/entry.S | 1 +
include/asm-cris/arch-v10/offset.h | 3 ---
include/asm-cris/arch-v32/offset.h | 3 ---
include/linux/clone.h | 32 ++++++++++++++++++++++++++++++++
include/linux/sched.h | 30 +-----------------------------
19 files changed, 42 insertions(+), 61 deletions(-)
The noise generated by the separation is larger than the avoided
duplication.
The hardcoded defines actually do need fixing, frv is especially bad, as
it even has hardcoded structure offsets.
> I have absolutely no problem with different, more
> logical splitups, and I'm willing to break down the
> entire sched.h if that will help the cause ... so
> please enlighten me here ...
sched.h is especially challenging due to dependencies between headers
under asm and linux. It's not just splitting sched.h, it also requires
analyzing its dependencies.
bye, Roman
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
|
|