From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx165.postini.com [74.125.245.165]) by kanga.kvack.org (Postfix) with SMTP id 479496B004D for ; Tue, 20 Dec 2011 09:14:41 -0500 (EST) Date: Tue, 20 Dec 2011 15:14:37 +0100 From: Michal Hocko Subject: Re: [PATCH] mm: hugetlb: fix pgoff computation when unmapping page from vma Message-ID: <20111220141437.GJ10565@tiehlicka.suse.cz> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Hillf Danton Cc: KAMEZAWA Hiroyuki , Andrew Morton , linux-mm@kvack.org, LKML , Mel Gorman [CCing Mel] On Tue 20-12-11 21:45:51, Hillf Danton wrote: > The computation for pgoff is incorrect, at least with > > (vma->vm_pgoff >> PAGE_SHIFT) > > involved. It is fixed with the available method if HPAGE_SIZE is concerned in > page cache lookup. Have you seen this as a real issue? I guess nobody noticed as it is quite rare error case. Anyways, yes, looks good. > Cc: Michal Hocko > Cc: KAMEZAWA Hiroyuki > Cc: Andrew Morton > Signed-off-by: Hillf Danton Reviewed-by: Michal Hocko > --- > > --- a/mm/hugetlb.c Tue Dec 20 21:26:30 2011 > +++ b/mm/hugetlb.c Tue Dec 20 21:40:44 2011 > @@ -2315,8 +2315,7 @@ static int unmap_ref_private(struct mm_s > * from page cache lookup which is in HPAGE_SIZE units. > */ > address = address & huge_page_mask(h); > - pgoff = ((address - vma->vm_start) >> PAGE_SHIFT) > - + (vma->vm_pgoff >> PAGE_SHIFT); > + pgoff = linear_hugepage_index(vma, address); You have hstate so you can use vma_hugecache_offset directly > mapping = (struct address_space *)page_private(page); > > /* > Thanks -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org