lkml.org 
[lkml]   [2013]   [Aug]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH v2 04/20] mm, hugetlb: remove useless check about mapping type
Date
Joonsoo Kim <iamjoonsoo.kim@lge.com> writes:

> is_vma_resv_set(vma, HPAGE_RESV_OWNER) implys that this mapping is
> for private. So we don't need to check whether this mapping is for
> shared or not.
>
> This patch is just for clean-up.
>
> Signed-off-by: Joonsoo Kim <iamjoonsoo.kim@lge.com>

Reviewed-by: Aneesh Kumar K.V <aneesh.kumar@linux.vnet.ibm.com>

>
> diff --git a/mm/hugetlb.c b/mm/hugetlb.c
> index ea1ae0a..c017c52 100644
> --- a/mm/hugetlb.c
> +++ b/mm/hugetlb.c
> @@ -2544,8 +2544,7 @@ retry_avoidcopy:
> * at the time of fork() could consume its reserves on COW instead
> * of the full address range.
> */
> - if (!(vma->vm_flags & VM_MAYSHARE) &&
> - is_vma_resv_set(vma, HPAGE_RESV_OWNER) &&
> + if (is_vma_resv_set(vma, HPAGE_RESV_OWNER) &&
> old_page != pagecache_page)
> outside_reserve = 1;
>
> --
> 1.7.9.5



\
 
 \ /
  Last update: 2013-08-21 12:01    [W:0.332 / U:0.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site