From: Dave Jones <davej@suse.de>
To: Andrew Morton <akpm@zip.com.au>
Cc: ekonijn@xs4all.nl, Christoph Hellwig <hch@infradead.org>,
kernel-janitor-discuss@lists.sourceforge.net, linux-mm@kvack.org
Subject: Re: page-flags.h
Date: Sat, 4 May 2002 01:39:38 +0200 [thread overview]
Message-ID: <20020504013938.G30500@suse.de> (raw)
In-Reply-To: <3CD317DD.2C9FBD11@zip.com.au>; from akpm@zip.com.au on Fri, May 03, 2002 at 04:06:05PM -0700
On Fri, May 03, 2002 at 04:06:05PM -0700, Andrew Morton wrote:
> Part of my uncertainty here is that we just don't seem to
> have a "plan". Is the objective to completely flatten
> the include heirarchy, no nested includes, and make all
> .c files include all headers to which they (and their included
> headers) refer?
>
> That's pretty aggressive, but I think it's the only sane
> objective.
<linux/fs.h>, <linux/mm.h>, <linux/sched.h>, <linux/pagemap.h>
are usually the main culprits. Each of these suckers pulls in
dozens and dozens of includes.
So every time someone wants for eg, something trivial like
jiffies, they end up pulling in crap like <asm/mmx.h>
It's hurrendous how much stuff we have in some of the above
mentioned files which probably deserves their own files.
Updated graphs of these dependancies just went up to
ftp://ftp.kernel.org/pub/linux/kernel/people/davej/misc/graphs/
(old versions are in the parent dir before/after the last crapectomy)
Dave.
--
| Dave Jones. http://www.codemonkey.org.uk
| SuSE Labs
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/
next prev parent reply other threads:[~2002-05-03 23:39 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20020501192737.R29327@suse.de>
2002-05-01 17:34 ` page-flags.h Christoph Hellwig
2002-05-01 18:04 ` page-flags.h Dave Jones
2002-05-03 2:52 ` page-flags.h Andrew Morton
2002-05-03 8:24 ` page-flags.h Christoph Hellwig
2002-05-03 22:41 ` page-flags.h Erik van Konijnenburg
2002-05-03 23:06 ` page-flags.h Andrew Morton
2002-05-03 23:39 ` Dave Jones [this message]
2002-05-04 6:46 ` page-flags.h Erik van Konijnenburg
2002-05-14 20:25 ` page-flags.h Andrew Morton
2002-05-15 12:38 ` page-flags.h Christoph Hellwig
2002-08-29 19:04 ` page-flags.h Daniel Phillips
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20020504013938.G30500@suse.de \
--to=davej@suse.de \
--cc=akpm@zip.com.au \
--cc=ekonijn@xs4all.nl \
--cc=hch@infradead.org \
--cc=kernel-janitor-discuss@lists.sourceforge.net \
--cc=linux-mm@kvack.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox