On Mon, 12 Nov 2007 04:57:03 +0000 (GMT) Hugh Dickins <hugh@xxxxxxxxxxx> wrote: > > Could I confirm a change in the logic ? > > > > * Before this patch, wrong swapcache charge is added to one who > > called try_to_free_page(). > > try_to_free_pages? No, I don't think any wrong charge was made > there. Ah, sorry. I misundestood when it was added to swapcache... > It was when reading in swap pages. The usual way is by > swapin_readahead, which reads in a cluster of swap pages, which > are quite likely to belong to different memcgroups, but were all > charged to the one which is doing the fault on its target page. > Another way is in swapoff, where they all got charged to whoever > was doing the swapoff (and the charging in unuse_pte was a no-op). > I see. > > * After this patch, anonymous page's charge will drop to 0 when > > page_remove_rmap() is called. > > Yes, when its final (usually its only) page_remove_rmap is called. > Thank you for confirmation ! Regards, -Kame _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers