Re: [patch] [RFC] make WANT_PAGE_VIRTUAL a config option

From: Roman Zippel
Date: Fri Dec 17 2004 - 08:28:33 EST


On Thu, 16 Dec 2004, Dave Hansen wrote:

> Sorry I didn't provide this. My recent effort started to clean up some
> ugliness in some current patches that worked around this actually
> happening a few months ago. The original example didn't survive :)

There are two points, which did originally catch my attention.
1) Where/why do you want to remove the dependency on asm/page.h?
2) Does every structure really needs its own header?

If you want to do such a cleanup, it would be helpful to have some more
information about where you want to go, otherwise you may create a more
twisted maze of header files. The big question here is what further
cleanups are possible in this area?
What basically needs to be done is to separate the definitions from its
users, that doesn't mean every definition needs its own header file. Why
not create a single header file which collects a number of mm related
definitions? E.g. struct vm_area_struct is also used by a number of header
files, although its main users have already been separated into
asm/tlbflush.h and so created even more headers.
So to allow further header cleanup, we should look what other definitions
can be pulled out of mm.h and related headers. mm.h should probably stay a
high level header, but I'd also like to see a cleanup of asm/page.h. The
page table definitions in there should be available to every mm related
I had to look closer at this, but I did this for struct task_struct and
making it available for lowlevel header files. I rediffed the patches and
put them at

bye, Roman
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at