From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757500Ab2EaAyt (ORCPT ); Wed, 30 May 2012 20:54:49 -0400 Received: from mail-qc0-f174.google.com ([209.85.216.174]:43704 "EHLO mail-qc0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754583Ab2EaAyr (ORCPT ); Wed, 30 May 2012 20:54:47 -0400 Date: Wed, 30 May 2012 20:54:43 -0400 From: Konrad Rzeszutek Wilk To: "Aneesh Kumar K.V" Cc: linux-mm@kvack.org, kamezawa.hiroyu@jp.fujitsu.com, dhillf@gmail.com, rientjes@google.com, mhocko@suse.cz, akpm@linux-foundation.org, hannes@cmpxchg.org, linux-kernel@vger.kernel.org, cgroups@vger.kernel.org, Andrea Arcangeli Subject: Re: [PATCH -V7 02/14] hugetlbfs: don't use ERR_PTR with VM_FAULT* values Message-ID: <20120531005442.GD401@localhost.localdomain> References: <1338388739-22919-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> <1338388739-22919-3-git-send-email-aneesh.kumar@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1338388739-22919-3-git-send-email-aneesh.kumar@linux.vnet.ibm.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, May 30, 2012 at 08:08:47PM +0530, Aneesh Kumar K.V wrote: > From: "Aneesh Kumar K.V" > > The current use of VM_FAULT_* codes with ERR_PTR requires us to ensure > VM_FAULT_* values will not exceed MAX_ERRNO value. Decouple the > VM_FAULT_* values from MAX_ERRNO. > > Signed-off-by: Aneesh Kumar K.V > Reviewed-by: KAMEZAWA Hiroyuki > Cc: Hillf Danton > Cc: Michal Hocko > Cc: Andrea Arcangeli > Cc: Johannes Weiner > --- > mm/hugetlb.c | 18 +++++++++++++----- > 1 file changed, 13 insertions(+), 5 deletions(-) > > diff --git a/mm/hugetlb.c b/mm/hugetlb.c > index e07d4cd..8ded02d 100644 > --- a/mm/hugetlb.c > +++ b/mm/hugetlb.c > @@ -1123,10 +1123,10 @@ static struct page *alloc_huge_page(struct vm_area_struct *vma, > */ > chg = vma_needs_reservation(h, vma, addr); > if (chg < 0) > - return ERR_PTR(-VM_FAULT_OOM); > + return ERR_PTR(-ENOMEM); > if (chg) > if (hugepage_subpool_get_pages(spool, chg)) > - return ERR_PTR(-VM_FAULT_SIGBUS); > + return ERR_PTR(-ENOSPC); Not enough space? Why not just pass what 'hugepage_subpool_get_pages' returns? > > spin_lock(&hugetlb_lock); > page = dequeue_huge_page_vma(h, vma, addr, avoid_reserve); > @@ -1136,7 +1136,7 @@ static struct page *alloc_huge_page(struct vm_area_struct *vma, > page = alloc_buddy_huge_page(h, NUMA_NO_NODE); > if (!page) { > hugepage_subpool_put_pages(spool, chg); > - return ERR_PTR(-VM_FAULT_SIGBUS); > + return ERR_PTR(-ENOSPC); -ENOMEM seems more appropiate? > } > } > > @@ -2496,6 +2496,7 @@ retry_avoidcopy: > new_page = alloc_huge_page(vma, address, outside_reserve); > > if (IS_ERR(new_page)) { > + int err = PTR_ERR(new_page); > page_cache_release(old_page); > > /* > @@ -2524,7 +2525,10 @@ retry_avoidcopy: > > /* Caller expects lock to be held */ > spin_lock(&mm->page_table_lock); > - return -PTR_ERR(new_page); > + if (err == -ENOMEM) > + return VM_FAULT_OOM; > + else > + return VM_FAULT_SIGBUS; Ah, you are doing it to translate it. Perhaps you should return -EFAULT for the really bad case where you need to do OOM and then for all the other cases return SIGBUS? Or maybe the other way around? ENOSPC doesn't seem like the right error. > } > > /* > @@ -2642,7 +2646,11 @@ retry: > goto out; > page = alloc_huge_page(vma, address, 0); > if (IS_ERR(page)) { > - ret = -PTR_ERR(page); > + ret = PTR_ERR(page); > + if (ret == -ENOMEM) > + ret = VM_FAULT_OOM; > + else > + ret = VM_FAULT_SIGBUS; > goto out; > } > clear_huge_page(page, address, pages_per_huge_page(h)); > -- > 1.7.10 > > -- > 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 >