The patch titled memcg: fix mem_cgroup_update_mapped_file_stat oops has been removed from the -mm tree. Its filename was memcg-add-file-based-rss-accounting-fix-mem_cgroup_update_mapped_file_stat-oops.patch This patch was dropped because it was folded into memcg-add-file-based-rss-accounting.patch The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: memcg: fix mem_cgroup_update_mapped_file_stat oops From: Hugh Dickins <hugh@xxxxxxxxxxx> CONFIG_SPARSEMEM=y CONFIG_CGROUP_MEM_RES_CTLR=y cgroup_disable=memory bootup is oopsing in mem_cgroup_update_mapped_file_stat(). !SPARSEMEM is fine because its lookup_page_cgroup() contains an explicit check for NULL node_page_cgroup, but the SPARSEMEM version was missing a check for NULL section->page_cgroup. Should go in as a fix to memcg-add-file-based-rss-accounting.patch but it's curious that's the first thing to suffer from this divergence. Perhaps this is the wrong fix, and there should be an explicit mem_cgroup_disable() check somewhere else; but it would then seem dangerous that SPARSEMEM and !SPARSEMEM diverge in this way, and there are lots of lookup_page_cgroup NULL tests around. Signed-off-by: Hugh Dickins <hugh@xxxxxxxxxxx> Cc: Balbir Singh <balbir@xxxxxxxxxxxxxxxxxx> Cc: KAMEZAWA Hiroyuki <kamezawa.hiroyu@xxxxxxxxxxxxxx> Cc: Li Zefan <lizf@xxxxxxxxxxxxx> Cc: Paul Menage <menage@xxxxxxxxxx> Cc: Dhaval Giani <dhaval@xxxxxxxxxxxxxxxxxx> Cc: Daisuke Nishimura <nishimura@xxxxxxxxxxxxxxxxx> Cc: YAMAMOTO Takashi <yamamoto@xxxxxxxxxxxxx> Cc: KOSAKI Motohiro <kosaki.motohiro@xxxxxxxxxxxxxx> Cc: David Rientjes <rientjes@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- mm/page_cgroup.c | 2 ++ 1 file changed, 2 insertions(+) diff -puN mm/page_cgroup.c~memcg-add-file-based-rss-accounting-fix-mem_cgroup_update_mapped_file_stat-oops mm/page_cgroup.c --- a/mm/page_cgroup.c~memcg-add-file-based-rss-accounting-fix-mem_cgroup_update_mapped_file_stat-oops +++ a/mm/page_cgroup.c @@ -99,6 +99,8 @@ struct page_cgroup *lookup_page_cgroup(s unsigned long pfn = page_to_pfn(page); struct mem_section *section = __pfn_to_section(pfn); + if (!section->page_cgroup) + return NULL; return section->page_cgroup + pfn; } _ Patches currently in -mm which might be from hugh.dickins@xxxxxxxxxxxxx are origin.patch memcg-add-file-based-rss-accounting.patch memcg-add-file-based-rss-accounting-fix-mem_cgroup_update_mapped_file_stat-oops.patch memcg-fix-swap-accounting.patch memcg-fix-swap-accounting-update.patch -- To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html