From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx150.postini.com [74.125.245.150]) by kanga.kvack.org (Postfix) with SMTP id 3EBF16B0005 for ; Wed, 6 Mar 2013 17:22:48 -0500 (EST) Date: Wed, 6 Mar 2013 14:22:46 -0800 From: Andrew Morton Subject: Re: [PATCH resend] rmap: recompute pgoff for unmapping huge page Message-Id: <20130306142246.b333f350f713dbbf3e931d93@linux-foundation.org> In-Reply-To: References: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Hillf Danton Cc: Linux-MM , LKML , Michal Hocko , Michel Lespinasse On Mon, 4 Mar 2013 20:47:31 +0800 Hillf Danton wrote: > [Resend due to error in delivering to linux-kernel@vger.kernel.org, > caused probably by the rich format provided by the mail agent by default.] > > We have to recompute pgoff if the given page is huge, since result based on > HPAGE_SIZE is not approapriate for scanning the vma interval tree, as shown > by commit 36e4f20af833(hugetlb: do not use vma_hugecache_offset() for > vma_prio_tree_foreach) > > ... > > @@ -1513,6 +1513,9 @@ static int try_to_unmap_file(struct page > unsigned long max_nl_size = 0; > unsigned int mapcount; > > + if (PageHuge(page)) > + pgoff = page->index << compound_order(page); > + > mutex_lock(&mapping->i_mmap_mutex); > vma_interval_tree_foreach(vma, &mapping->i_mmap, pgoff, pgoff) { > unsigned long address = vma_address(page, vma); Also, what does this patch actually do? I have a canned response nowadays: When writing a changelog, please describe the end-user-visible effects of that bug, so that others can more easily decide which kernel version(s) should be fixed, and so that downstream kernel maintainers can more easily work out whether this patch will fix a problem which they or their customers are observing. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org