On Wed, 21 Dec 2011 10:35:47 +0100 Michal Hocko <mhocko@xxxxxxx> wrote: > On Tue 20-12-11 15:30:36, Andrew Morton wrote: > > > > The patch titled > > Subject: memcg-clear-pc-mem_cgorup-if-necessary-fix-2-fix > > has been added to the -mm tree. Its filename is > > memcg-clear-pc-mem_cgorup-if-necessary-fix-2-fix.patch > > > > Before you just go and hit "reply", please: > > a) Consider who else should be cc'ed > > b) Prefer to cc a suitable mailing list as well > > c) Ideally: find the original patch on the mailing list and do a > > reply-to-all to that, adding suitable additional cc's > > > > *** Remember to use Documentation/SubmitChecklist when testing your code *** > > > > See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find > > out what to do about this > > > > The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ > > > > ------------------------------------------------------ > > From: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > > Subject: memcg-clear-pc-mem_cgorup-if-necessary-fix-2-fix > > > > ksm.c needs memcontrol.h, per Michal > > Just for record. It really doesn't need it at the moment because it gets > memcontrol.h via rmap.h resp. swap.h but I plan to remove memcontrol > include from those two. > I can do that in a separate patch if you prefer? Sure. It's generally bad to rely upon nested includes, especially one which the outer header file included for its own purposes (which might change). -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>