On Thu, 20 Dec 2007 21:54:15 +0530 Balbir Singh <[EMAIL PROTECTED]> wrote:
> >> struct page_cgroup *page_get_page_cgroup(struct page *page) > >> { > >> return (struct page_cgroup *) > >> (page->page_cgroup & ~PAGE_CGROUP_LOCK); > >> } > >> > >> I guess the issue is that often a "get" function has a complementary > >> "put" function, but this isn't one of them. Would page_page_cgroup > >> be a better name, perhaps? I don't know. > > > > Ah, yes, I mistakenly assumed it was a reference get. In that case I > > stand corrected and do not have any objections. > > > > I was going to say the same thing, page_get_page_cgroup() does not hold > any references. May be _get_ in the name is confusing. It is a bit unconventional. page_cgroup()? -- 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/