On Tuesday 15 January 2008 11:15:02 Harvey Harrison wrote: > On Tue, 2008-01-15 at 11:06 +0100, Andi Kleen wrote: > > On Tuesday 15 January 2008 10:29:51 Harvey Harrison wrote: > > > On Mon, 2008-01-14 at 23:16 +0100, Andi Kleen wrote: > > > > Instead of open coding the bit accesses uses standard style > > > > *PageDeferred* macros. > > > > > > > > > > Any reason these couldn't be static inlines in a shared header? > > > > The whole usage of that bit is private to the file > > True if pageattr_32|64.c get merged, not one I have looked at yet, > but wouldn't a static inline in the files then be better for the > typechecking?
If you pass in something other than a struct page * the compiler will already complain. Also all the other Page* accessor in page-flags.h are macros. -Andi -- 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/