From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx130.postini.com [74.125.245.130]) by kanga.kvack.org (Postfix) with SMTP id 47E026B005C for ; Wed, 30 May 2012 20:54:48 -0400 (EDT) Received: by qabg27 with SMTP id g27so2731344qab.14 for ; Wed, 30 May 2012 17:54:47 -0700 (PDT) Date: Wed, 30 May 2012 20:54:43 -0400 From: Konrad Rzeszutek Wilk 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> Sender: owner-linux-mm@kvack.org List-ID: 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 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 > -- 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