On Thu, 2007-12-20 at 21:54 +0530, Balbir Singh wrote: > > 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. OK, you three had the entire conversation outing me before I even fot to respond! :) Yeah, I thought it was a refcount acquisition. Balbir, it's fine the way it stands (except for the slightly confusing name). -- Dave -- 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/
- References:
- [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- From: Balbir Singh <[email protected]>
- Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- From: Dave Hansen <[email protected]>
- Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- From: Hugh Dickins <[email protected]>
- Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- From: Peter Zijlstra <[email protected]>
- Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- From: Hugh Dickins <[email protected]>
- Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- From: Peter Zijlstra <[email protected]>
- Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- From: Balbir Singh <[email protected]>
- [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- Prev by Date: Re: not needed patch
- Next by Date: Re: [rfc][patch] mm: madvise(WILLNEED) for anonymous memory
- Previous by thread: Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- Next by thread: Re: [PATCH] Move page_assign_page_cgroup to VM_BUG_ON in free_hot_cold_page
- Index(es):
 |