From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965014AbVI0RER (ORCPT ); Tue, 27 Sep 2005 13:04:17 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965017AbVI0RER (ORCPT ); Tue, 27 Sep 2005 13:04:17 -0400 Received: from scrub.xs4all.nl ([194.109.195.176]:32734 "EHLO scrub.xs4all.nl") by vger.kernel.org with ESMTP id S965014AbVI0REQ (ORCPT ); Tue, 27 Sep 2005 13:04:16 -0400 Date: Tue, 27 Sep 2005 19:04:10 +0200 (CEST) From: Roman Zippel X-X-Sender: roman@scrub.home To: Herbert Poetzl cc: Andrew Morton , Linux Kernel ML Subject: Re: [Patch] eliminate CLONE_* duplications In-Reply-To: <20050927162242.GC21927@MAIL.13thfloor.at> Message-ID: References: <20050921092132.GA4710@MAIL.13thfloor.at> <20050921143954.GA10137@MAIL.13thfloor.at> <20050921151124.GB10137@MAIL.13thfloor.at> <20050921235810.GC18040@MAIL.13thfloor.at> <20050927162242.GC21927@MAIL.13thfloor.at> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Tue, 27 Sep 2005, Herbert Poetzl wrote: > > "logically organized" mainly means reducing dependencies by organizing > > them by their logical dependencies. > > did you consider that separating out the clone > stuff might be that basis for reducing dependencies? Not in this form, all users of this flag need other definitions from sched.h. > > The hardcoded defines actually do need fixing, frv is especially bad, > > as it even has hardcoded structure offsets. > > so instead of fixing the issue properly, we > 'mend' it by adding new code to */asm-offsets.c Using asm-offsets.c _is_ a proper solution. > > 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. > > which you obviously think is nothing I can do > 'properly' ... I don't know, but I know that it does require a large amount of experience in this area. bye, Roman