On Tue, 05 Feb 2013 13:56:36 +0800 li guang <lig.fnst@xxxxxxxxxxxxxx> wrote: > ___ 2013-02-04______ 21:20 -0800___David Rientjes_________ > > On Tue, 5 Feb 2013, liguang wrote: > > > > > linux/mmzone.h included linux/memory_hotplug.h, > > > and linux/memory_hotplug.h also included > > > linux/mmzone.h, so there's a bad cirlular. > > > > > > > And both of these are protected by _LINUX_MMZONE_H and > > __LINUX_MEMORY_HOTPLUG_H, respectively, so what's the problem? > > obviously, It's a logical error, > and It has no more effect other than > combination of these 2 header files. > so, why don't we separate them? > Yup, flattening the hierarchy is nice. And having headers doing mutual inclusion like this is Just Weird. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>