On Thu, 14 Jun 2012 16:58:05 +0800 Li Zefan <lizefan@xxxxxxxxxx> wrote: > > +int hugetlb_cgroup_charge_cgroup(int idx, unsigned long nr_pages, > > > + struct hugetlb_cgroup **ptr) > > +{ > > + int ret = 0; > > + struct res_counter *fail_res; > > + struct hugetlb_cgroup *h_cg = NULL; > > + unsigned long csize = nr_pages * PAGE_SIZE; > > + > > + if (hugetlb_cgroup_disabled()) > > + goto done; > > + /* > > + * We don't charge any cgroup if the compound page have less > > + * than 3 pages. > > + */ > > + if (huge_page_order(&hstates[idx]) < HUGETLB_CGROUP_MIN_ORDER) > > + goto done; > > +again: > > + rcu_read_lock(); > > + h_cg = hugetlb_cgroup_from_task(current); > > + if (!h_cg) > > > In no circumstances should h_cg be NULL. > Aneesh? -- To unsubscribe from this list: send the line "unsubscribe cgroups" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html